refs #20658 [finc] BARF specify current page menu entry

* improve sourcecode

themes/finc/templates/myresearch/menu.phtml

@@ -34,9 +34,7 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
     <?php endif; ?>
     <?php if ($this->ils()->checkFunction('getMyTransactionHistory', $capabilityParams)): ?>
       <li class="facet">
-        <a href="<?=$this->url('myresearch-historicloans')?>"<?=$this->active == 'historicloans' ? ' class="active"' : ''?>
-          <?=$this->active == 'historicloans' ? ' aria-current="page"' : ''?>
-        >
+        <a href="<?=$this->url('myresearch-historicloans')?>"<?=$this->active == 'historicloans' ? ' class="active" aria-current="page"' : ''?>>
         <i class="fa fa-fw fa-history" aria-hidden="true"></i> <?=$this->transEsc('Loan History')?>
       </a>
       </li>
@@ -45,8 +43,8 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
 
       <li class="facet">
         <a href="<?=$this->url('myresearch-holds')?>" class="flex<?=$this->active == 'holds' ? ' active' : ''?>"
-          <?=$this->active == 'holds' ? ' aria-current="page"' : ''?>
-        >
+        <?=$this->active == 'holds' ? ' aria-current="page"' : ''?>
+      >
         <span class="flex-col"><i class="fa fa-fw fa-flag" aria-hidden="true"></i>&nbsp;<?=$this->transEsc('Holds and Recalls')?></span>
         <span class="holds-status status hidden"><i class="fa fa-spin fa-spinner" aria-hidden="true"></i></span>
         <?php /* nxt line finc specific - CK */ ?>
@@ -58,8 +56,8 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
     <?php if ($this->ils()->checkFunction('StorageRetrievalRequests', $capabilityParams)): ?>
       <li class="facet">
         <a href="<?=$this->url('myresearch-storageretrievalrequests')?>" class="flex<?=$this->active == 'storageRetrievalRequests' ? ' active' : ''?>"
-          <?=$this->active == 'storageRetrievalRequests' ? ' aria-current="page"' : ''?>
-        >
+        <?=$this->active == 'storageRetrievalRequests' ? ' aria-current="page"' : ''?>
+      >
         <span class="flex-col"><i class="fa fa-fw fa-archive" aria-hidden="true"></i> <?=$this->transEsc('Storage Retrieval Requests')?></span>
         <span class="storageretrievalrequests-status status hidden"><i class="fa fa-spin fa-spinner" aria-hidden="true"></i></span>
         <?php /* nxt line finc specific - CK */ ?>
@@ -71,8 +69,8 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
     <?php if ($this->ils()->checkFunction('ILLRequests', $capabilityParams)): ?>
       <li class="facet">
         <a href="<?=$this->url('myresearch-illrequests')?>" class="flex<?=$this->active == 'ILLRequests' ? ' active' : ''?>"
-          <?=$this->active == 'ILLRequests' ? ' aria-current="page"' : ''?>
-        >
+        <?=$this->active == 'ILLRequests' ? ' aria-current="page"' : ''?>
+      >
         <span class="flex-col"><i class="fa fa-fw fa-exchange" aria-hidden="true"></i>&nbsp;<?=$this->transEsc('Interlibrary Loan Requests')?></span>
         <span class="illrequests-status status hidden"><i class="fa fa-spin fa-spinner" aria-hidden="true"></i></span>
         <?php /* nxt line finc specific - CK */ ?>
@@ -84,25 +82,21 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
     <?php if ($this->ils()->checkCapability('getMyFines', $capabilityParams)): ?>
       <li class="facet">
         <a href="<?=$this->url('myresearch-fines')?>" class="flex<?=$this->active == 'fines' ? ' active' : ''?>"
-          <?=$this->active == 'fines' ? ' aria-current="page"' : ''?>
-        >
+        <?=$this->active == 'fines' ? ' aria-current="page"' : ''?>
+      >
         <span class="flex-col"><i class="fa fa-fw fa-usd" aria-hidden="true"></i>&nbsp;<?=$this->transEsc('Fines')?></span>
         <span class="fines-status status hidden"><i class="fa fa-spin fa-spinner" aria-hidden="true"></i></span>
       </a>
       </li>
     <?php endif; ?>
     <li class="facet">
-      <a href="<?=$this->url('myresearch-profile')?>"<?=$this->active == 'profile' ? ' class="active"' : ''?>
-        <?=$this->active == 'profile' ? ' aria-current="page"' : ''?>
-      >
+      <a href="<?=$this->url('myresearch-profile')?>"<?=$this->active == 'profile' ? ' class="active" aria-current="page"' : ''?>>
       <i class="fa fa-fw fa-user" aria-hidden="true"></i> <?=$this->transEsc('Profile')?>
     </a>
     </li>
     <?php if ($user && $user->libraryCardsEnabled()): ?>
       <li class="facet">
-        <a href="<?=$this->url('librarycards-home')?>"<?=$this->active == 'librarycards' ? ' class="active"' : ''?>
-          <?=$this->active == 'librarycards' ? ' aria-current="page"' : ''?>
-        >
+        <a href="<?=$this->url('librarycards-home')?>"<?=$this->active == 'librarycards' ? ' class="active" aria-current="page"' : ''?>>
         <i class="fa fa-fw fa-barcode" aria-hidden="true"></i> <?=$this->transEsc('Library Cards')?>
       </a>
       </li>
@@ -110,9 +104,7 @@ $capabilityParams = $patron ? ['patron' => $patron] : [];
   <?php endif; ?>
   <?php if ($this->accountCapabilities()->getSavedSearchSetting() === 'enabled'): ?>
     <li class="facet">
-      <a href="<?=$this->url('search-history')?>?require_login"<?=$this->active == 'history' ? ' class="active"' : ''?>
-        <?=$this->active == 'history' ? ' aria-current="page"' : ''?>
-      >
+      <a href="<?=$this->url('search-history')?>?require_login"<?=$this->active == 'history' ? ' class="active" aria-current="page"' : ''?>>
       <i class="fa fa-fw fa-search" aria-hidden="true"></i> <?=$this->transEsc('history_saved_searches')?>
     </a>
     </li>