From ee23a22820a8de0d65f1824f26b05785f6b80edd Mon Sep 17 00:00:00 2001
From: Andrew Newton <andy.newton@dvsa.gov.uk>
Date: Mon, 5 Aug 2024 12:13:55 +0100
Subject: [PATCH] fix: Fix application status tag cases. (#174) (#175)

* fix: Fix application status tag cases.

* fix: Fix application status tag cases.

fix: Fix application status tag cases.

fix: Stop rendering empty nav menu on sign-in page.
---
 module/Olcs/view/layouts/base.phtml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/module/Olcs/view/layouts/base.phtml b/module/Olcs/view/layouts/base.phtml
index 87f3acd43..9ebf73878 100644
--- a/module/Olcs/view/layouts/base.phtml
+++ b/module/Olcs/view/layouts/base.phtml
@@ -193,12 +193,12 @@ $isLoggedIn = $currentUser->isLoggedIn();
                   : $this->translate('header-vehicle-operator-licensing');
           ?>
         </a>
+        <?php  if ($isLoggedIn): ?>
         <nav aria-label="Menu" class="govuk-header__navigation">
         <button type="button" class="govuk-header__menu-button govuk-js-header-toggle" aria-controls="navigation" aria-label="<?php echo $this->escapeHtml($this->translate('show.or.hide.top.level.navigation')); ?>">
             <?php echo $this->escapeHtml($this->translate('menu')); ?>
         </button>
           <?php
-          if ($isLoggedIn) {
               // get Home navigation
               $navigation = $this->navigation('navigation')->getContainer()->findBy('id', 'home');
 
@@ -219,9 +219,9 @@ $isLoggedIn = $currentUser->isLoggedIn();
 
               // This seems irrelevant, but it's very important as the partial is set elsewhere
               $menu->setPartial(null);
-          }
           ?>
          </nav>
+        <?php endif; ?>
       </div>
     </div>
   </header>