TPAC: Suppress copy counts for libraries that own zero copies collab/dyrcona/tpac-suppress-preferred-library-count-with-0
authorDan Scott <dan@coffeecode.net>
Wed, 28 Mar 2012 02:33:31 +0000 (22:33 -0400)
committerJason Stephenson <jstephenson@mvlc.org>
Wed, 28 Mar 2012 19:11:39 +0000 (15:11 -0400)
Rather than slavishly showing the copy counts for libraries even when
the library has no copies to show, avoid wasting the precious display
space and suppress the output.

This was a problem for both search scope libraries and preferred library
in the search results, and for just preferred library in record details.
This commit makes the behaviour consistent, in favour of suppressing the
display of copy counts when there's nothing to display.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Open-ILS/src/templates/opac/parts/record/copy_counts.tt2
Open-ILS/src/templates/opac/parts/result/copy_counts.tt2

index 47f8f44..a573161 100644 (file)
@@ -27,6 +27,7 @@
         ou_id = attrs.plib_copy_counts.$depth.org_unit;
         UNLESS depth < 0 || displayed_ous.exists(ou_name);
     %]
+    [%- IF attrs.plib_copy_counts.$depth.count > 0; %]
     <li class="preferred">[%
         l('[_1] of [quant,_2,copy,copies] available at [_3].',
             attrs.plib_copy_counts.$depth.available,
@@ -37,5 +38,6 @@
        l('(Show preferred library)');
     %]</a></li>
     [%- END %]
+    [%- END %]
     </ul>
 </span>
index 943066e..d95d3a7 100644 (file)
@@ -4,6 +4,7 @@
     WHILE depth < depths;
         ou_name = ctx.get_aou(attrs.copy_counts.$depth.org_unit).name;
         displayed_ous.$ou_name = 1;
+        IF attrs.copy_counts.$depth.count > 0;
 %]
 <div class="result_count">
     [% l('[_1] of [quant,_2,copy,copies] available at [_3].',
@@ -12,7 +13,8 @@
         ou_name) | html
     %]
 </div>
-[%- depth = depth + 1;
+[%-     END;
+    depth = depth + 1;
     END;
 
     depth = attrs.plib_copy_counts.size - 1;
@@ -20,6 +22,7 @@
     UNLESS displayed_ous.exists(ou_name);
     
 %]
+[%- IF attrs.plib_copy_counts.$depth.count > 0; %]
 <div class="result_count preferred">[%
      l('[_1] of [quant,_2,copy,copies] available at [_3].',
         attrs.plib_copy_counts.$depth.available,
@@ -28,3 +31,4 @@
     %]<span> [% l('(Preferred)') %]</span>
 </div>
 [%- END %]
+[%- END %]