Render error summary on cya (in case a developer overrides the statem…

…ent of truth)
hmcts · Jan 12, 2018 · cb0980c · cb0980c
1 parent bd5c8c3
commit cb0980c
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 12 deletions.
diff --git a/templates/look-and-feel/components/errors.njk b/templates/look-and-feel/components/errors.njk
@@ -0,0 +1,14 @@
+{% macro errorSummary(fields, title='There was a problem') -%}
+  {%- if fields.validated and not fields.valid -%}
+    <div class="error-summary" role="group" aria-labelledby="error-summary-heading" tabindex="-1">
+      <h1 class="heading-medium error-summary-heading" id="error-summary-heading">
+        {{ title }}
+      </h1>
+      <ul class="error-summary-list">
+        {%- for error in fields.errors -%}
+          <li><a href="#{{error.id}}">{{error.message}}</a></li>
+        {%- endfor -%}
+      </ul>
+    </div>
+  {%- endif -%}
+{%- endmacro %}
diff --git a/templates/look-and-feel/layouts/check_your_answers.html b/templates/look-and-feel/layouts/check_your_answers.html
@@ -3,6 +3,7 @@
 {% from "look-and-feel/components/check-your-answers.njk" import answer %}
 {% from "look-and-feel/components/phase_banner.njk" import phaseBanner %}
 {% from "look-and-feel/components/fields.njk" import hiddenInput %}
+{% from "look-and-feel/components/errors.njk" import errorSummary %}
 
 {% set defaultContent = {
   title: 'Check your answers',
@@ -24,6 +25,8 @@
 
 {{ header(pageContent.title | default(defaultContent.title)) }}
 
+  {{ errorSummary(fields) }}
+
   {% for section in _sections %}
 
     {% if section.atLeast1Completed %}

diff --git a/templates/look-and-feel/layouts/question.html b/templates/look-and-feel/layouts/question.html
@@ -1,6 +1,7 @@
 {% extends "look-and-feel/layouts/two_thirds.html" %}
 {% from "look-and-feel/components/header.njk" import header %}
 {% from "look-and-feel/components/phase_banner.njk" import phaseBanner %}
+{% from "look-and-feel/components/errors.njk" import errorSummary %}
 
 {% block page_title %}{{ title }}{% endblock %}
 
@@ -15,18 +16,7 @@
 
 <form action="{{ postUrl | default(path if path else url) }}" method="post" class="form">
 
-  {%- if fields.validated and not fields.valid -%}
-    <div class="error-summary" role="group" aria-labelledby="error-summary-heading" tabindex="-1">
-      <h1 class="heading-medium error-summary-heading" id="error-summary-heading">
-        There was a problem
-      </h1>
-      <ul class="error-summary-list">
-        {%- for error in fields.errors -%}
-          <li><a href="#{{error.id}}">{{error.message}}</a></li>
-        {%- endfor -%}
-      </ul>
-    </div>
-  {%- endif -%}
+  {{ errorSummary(fields) }}
 
   {% block fields -%}
   {%- endblock %}