TPAC: Improve copy display in record summary

In the copy counts, offer a link to display the copies at parent
libraries that have available copies. For example, if you are browsing
at BR1 and there are no available copies, but you see that SYS1 has
available copies, clicking "(Show)" beside the SYS1 available copy count
will change the search scope to SYS1 and display the copies for BR1 and
BR2.

Also, avoid displaying the copy table header if there are no physical
copies to display (for example, if there are located URIs that make the
copy show up in search results).

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

diff --git a/Open-ILS/src/templates/opac/parts/record/copy_counts.tt2 b/Open-ILS/src/templates/opac/parts/record/copy_counts.tt2
index 905789d..9d85ae7 100644 (file)
--- a/Open-ILS/src/templates/opac/parts/record/copy_counts.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/copy_counts.tt2
@@ -4,12 +4,17 @@
     [%- depths = ctx.copy_summary.size;
         depth = 0;
         WHILE depth < depths;
+            ou_avail = ctx.copy_summary.$depth.available;
+            ou_id = ctx.copy_summary.$depth.org_unit;
     %]
     <li>
-    [% l('[quant,_1,copy,copies] at [_2].', 
-        ctx.copy_summary.$depth.available,
-        ctx.get_aou(ctx.copy_summary.$depth.org_unit).name)
+    [% l('[quant,_1,copy,copies] at [_2].', ou_avail, ctx.get_aou(ou_id).name)
         | html %]
+    [%- IF ou_avail > 0 && ou_id != CGI.param('loc'); %]
+    <a href="[% mkurl('', {loc => ou_id}); %]"
+       title="[% l('Show copies at [_1]', ctx.get_aou(ou_id).name); %]">
+       [%- l('(Show)'); %]</a>
+    [%- END; %]
     </li>
     [%- depth = depth + 1; END %]
     </ul>

diff --git a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
index 03d5f16..8148dd9 100644 (file)
--- a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
@@ -1,11 +1,17 @@
 [%-
 FOREACH copy_info IN ctx.copies;
+    IF copy_info.call_number_label != '##URI##';
+        has_copies = 'true';
+    END;
     IF copy_info.part_label != '';
         has_parts = 'true';
+    END;
+    IF has_parts && has_copies;
         LAST;
     END;
 END;
 %]
+[%- IF has_copies; %]
 <table cellpadding="0" cellspacing="0" border="0" width="100%" id="rdetails_status">
     <thead>
         <tr>
@@ -132,4 +138,4 @@ END;
         </tr>
     </tbody>
 </table>
-
+[% END; %]