MAN-172: No fixed address and error fixes

ministryofjustice · Feb 5, 2025 · eef139f · eef139f
1 parent 97d44aa
commit eef139f
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 21 deletions.
diff --git a/server/views/pages/edit-contact-details/edit-contact-details.njk b/server/views/pages/edit-contact-details/edit-contact-details.njk
@@ -55,7 +55,10 @@
     }) if personalDetails.mainAddress.to }}
 
     {% if errorMessages %}
-        {{ govukErrorSummary({ titleText: "There is a problem", errorList: errorMessages | toErrorList }) }}
+    {% set errorList = errorMessages | toErrorList %}
+        {% if errorList > 0 %}
+            {{ govukErrorSummary({ titleText: "There is a problem", errorList: errorMessages | toErrorList }) }}
+        {% endif %}
     {% endif %}
 
 {% endblock %}

diff --git a/server/views/pages/personal-details.njk b/server/views/pages/personal-details.njk
@@ -38,13 +38,16 @@
 {% block pageContent %}
     {% set contactDetails %}
         {% set mainAddress = personalDetails.mainAddress %}
-
             {% set address %}
                 {% if personalDetails.mainAddress %}
-                    {% set addressAsList = addressToList(mainAddress, noFixedAddress) %}
-                    {% if addressAsList.length > 0 %}
-                        {{ addressAsList.join('<br>') | safe }}
-                        <br/>
+                    {% if mainAddress.noFixedAddress === true %}
+                        No fixed address
+                    {% else %}
+                        {% set addressAsList = addressToList(mainAddress) %}
+                            {% if addressAsList.length > 0 %}
+                                {{ addressAsList.join('<br>') | safe }}
+                                <br/>
+                            {% endif %}
                     {% endif %}
                     <p class="govuk-!-margin-top-2 govuk-!-margin-bottom-0">
                         <span class="govuk-summary-list__key govuk-!-padding-bottom-0" data-qa="addressTypeLabel">Type of address</span>

diff --git a/server/views/pages/personal-details/_address.njk b/server/views/pages/personal-details/_address.njk
@@ -8,12 +8,11 @@
 
 {% set addressHtml %}
   {% set fullAddress %}
-    {% if address.noFixedAbode %}
-      No fixed abode<br />
-      {{ address.type }}
+    {% if address.noFixedAddress %}
+      No fixed address
+    {% else %}
+      {{ addressToList(address).join('<br>') | safe }}
     {% endif %}
-
-    {{ addressToList(address).join('<br>') | safe }}
   {% endset %}
 
   {{ govukSummaryList({

diff --git a/server/views/pages/personal-details/addresses.njk b/server/views/pages/personal-details/addresses.njk
@@ -34,18 +34,19 @@
   iconFallbackText: 'Warning'
 }) }} -->
 
-<p>
-    <a href="{{ deliusDeepLinkUrl('AddressandAccommodation', crn) }}" target="_blank">Update addresses on NDelius (opens in new tab)</a>
-</p>
-
 {% set address = addressOverview.mainAddress %}
-<span data-qa="mainAddressSection">
-    {% set addressType = 'address' %}
-    {% set loopIndex = '' %}
-    {% include './_address.njk' %}
-</span>
+{% if addressOverview.mainAddress %}
+    <p>
+        <a href="{{ deliusDeepLinkUrl('AddressandAccommodation', crn) }}" target="_blank">Update addresses on NDelius (opens in new tab)</a>
+    </p>
+    <span data-qa="mainAddressSection">
+        {% set addressType = 'address' %}
+        {% set loopIndex = '' %}
+        {% include './_address.njk' %}
+    </span>
+{% endif %}
 
-{% if addressOverview.otherAddresses.length > 0 %}
+    {% if addressOverview.otherAddresses.length > 0 %}
     <h2 class="govuk-heading-l govuk-!-margin-top-8">Other addresses</h2>
     <p>
         <a href="{{ deliusDeepLinkUrl('AddressandAccommodation', crn) }}" target="_blank">Edit all other addresses on NDelius (opens in new tab)</a>