From eef139f7154ceeb0d535eaa393cf8f2f1caab210 Mon Sep 17 00:00:00 2001
From: Paul McPhee <paul.mcphee@digital.justice.gov.uk>
Date: Wed, 5 Feb 2025 11:01:58 +0000
Subject: [PATCH] MAN-172: No fixed address and error fixes

---
 .../edit-contact-details.njk                  |  5 ++++-
 server/views/pages/personal-details.njk       | 13 +++++++-----
 .../views/pages/personal-details/_address.njk |  9 ++++----
 .../pages/personal-details/addresses.njk      | 21 ++++++++++---------
 4 files changed, 27 insertions(+), 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
index bdbee5c3..3c910048 100644
--- 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
index ac24f4e7..db61222f 100644
--- 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
index 8459f0b1..e8bf2ddc 100644
--- 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
index 37c12bed..6de102b5 100644
--- 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>