Renamed tab variable to activeTab for greater clarity.

module/VuFind/src/VuFind/Controller/AbstractRecord.php

@@ -495,7 +495,7 @@ class AbstractRecord extends AbstractBase
 
         $driver = $this->loadRecord();
         $view = $this->createViewModel();
-        $view->tab = strtolower($tab);
+        $view->activeTab = strtolower($tab);
         $view->defaultTab = strtolower($this->defaultTab);
 
         // Set up next/previous record links (if appropriate)

themes/blueprint/templates/record/view.phtml

@@ -86,7 +86,7 @@
     <ul>
       <? foreach ($tabs as $tab => $desc): ?>
       <? // add current tab to breadcrumbs if applicable:
-         $isCurrent = (strtolower($this->tab) == strtolower($tab));
+         $isCurrent = (strtolower($this->activeTab) == strtolower($tab));
          if ($isCurrent) {
              $this->layout()->breadcrumbs .= '<span>&gt;</span><em>' . $this->transEsc($desc) . '</em>';
          }
@@ -102,7 +102,7 @@
 
 
   <div class="recordsubcontent">
-    <?=$this->record($this->driver)->getTab($this->tab)?>
+    <?=$this->record($this->driver)->getTab($this->activeTab)?>
   </div>
 
   <span class="Z3988" title="<?=$this->escapeHtml($this->driver->getOpenURL())?>"></span>

themes/jquerymobile/templates/record/header-navbar.phtml

@@ -8,7 +8,7 @@
     <ul>
       <? foreach ($tabs as $tab => $desc): ?>
         <li>
-          <a rel="external"<?=(strtolower(isset($this->tab) ? $this->tab : '') == strtolower($tab)) ? ' class="ui-btn-active"' : ''?> href="<?=$this->recordLink()->getTabUrl($this->driver, $tab)?>"><?=$this->transEsc($desc)?></a>
+          <a rel="external"<?=(strtolower(isset($this->activeTab) ? $this->activeTab : '') == strtolower($tab)) ? ' class="ui-btn-active"' : ''?> href="<?=$this->recordLink()->getTabUrl($this->driver, $tab)?>"><?=$this->transEsc($desc)?></a>
         </li>
       <? endforeach; ?>
     </ul>

themes/jquerymobile/templates/record/view.phtml

 <?
     // Grab tab contents up front -- this will set the page title, which we need to
     // do before we display the page header below.
-    $tab = $this->record($this->driver)->getTab($this->tab);
+    $tab = $this->record($this->driver)->getTab($this->activeTab);
 ?>
 <div data-role="page" id="Record-view">
   <?=$this->mobileMenu()->header(array('searchLink' => $this->searchOptions($this->searchClassId)->getSearchHomeAction()))?>
   <div class="record" data-role="content" data-record-id="<?=$this->escapeHtml($this->driver->getUniqueId())?>">
     <?=$this->flashmessages()?>
-    <? if ($this->tab == $this->defaultTab): ?>
+    <? if ($this->activeTab == $this->defaultTab): ?>
       <?=$this->record($this->driver)->getCoreMetadata()?>
     <? else: ?>
       <h3><?=$this->escapeHtml($this->driver->getBreadcrumb())?></h3>