Author: sevein
Date: Thu Oct 15 11:52:29 2009
New Revision: 3755

Log:
Omit layer when count($holdings) = 0. Cosmetic changes.

Modified:
   trunk/apps/qubit/modules/repository/templates/_contextMenu.php

Modified: trunk/apps/qubit/modules/repository/templates/_contextMenu.php
==============================================================================
--- trunk/apps/qubit/modules/repository/templates/_contextMenu.php      Thu Oct 
15 11:51:55 2009        (r3754)
+++ trunk/apps/qubit/modules/repository/templates/_contextMenu.php      Thu Oct 
15 11:52:29 2009        (r3755)
@@ -1,34 +1,33 @@
+<?php if (substr($currentAction, 0, 4) == 'show' && count($holdings)):
+  // Only display holdings on "show" templates  to avoid form submit
+  // issues with using pager ?>
 <div class="context-column-box">
   <div class="contextMenu">
-    
-    <!-- Only display holdings on "show" templates to avoid form submit issues
-         with using pager -->
-    <?php if (substr($currentAction, 0, 4) == 'show' && count($holdings)): ?>
-    <div class="label">
-      <?php echo sfConfig::get('app_ui_label_holdings') ?>
-    </div>
+
+    <div class="label"><?php echo sfConfig::get('app_ui_label_holdings') 
?></div>
     <div class="holdings">
-    <ul>
+      <ul>
       <?php foreach ($holdings as $holding): ?>
         <li><?php echo link_to(render_title($holding), 
'informationobject/show?id='.$holding->getId()) ?></li>
       <?php endforeach; ?>
-    </ul>
+      </ul>
     </div>
-    
+
     <?php if ($pager->haveToPaginate()): ?>
     <div class="pager">
       <?php $links = $pager->getLinks(); ?>
       <?php if ($pager->getPage() != $pager->getFirstPage()): ?>
-     <?php echo link_to('< '.__('previous'), 
'repository/show?id='.$repository->getId().'&holdingsPage='.($pager->getPage()-1))
 ?>
+        <?php echo link_to('< '.__('previous'), 
'repository/show?id='.$repository->getId().'&holdingsPage='.($pager->getPage()-1))
 ?>
       <?php endif; ?>
       <?php foreach ($links as $page): ?>
         <?php echo ($page == $pager->getPage()) ? '<strong>'.$page.'</strong>' 
: link_to($page, 
'repository/show?id='.$repository->getId().'&holdingsPage='.$page) ?>
       <?php endforeach ?>
       <?php if ($pager->getPage() != $pager->getLastPage()): ?>
-     <?php echo link_to(__('next').' >', 
'repository/show?id='.$repository->getId().'&holdingsPage='.($pager->getPage()+1))
 ?>
+        <?php echo link_to(__('next').' >', 
'repository/show?id='.$repository->getId().'&holdingsPage='.($pager->getPage()+1))
 ?>
       <?php endif; ?>
     </div>
     <?php endif; ?>
-    <?php endif; // Only display holdings for show template ?>
+
   </div>
 </div>
+<?php endif; // Only display holdings for show template ?>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Qubit Toolkit Commits" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.ca/group/qubit-commits?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to