lp1481844 Change Available Copies Display
authorKathy Lussier <klussier@masslnc.org>
Wed, 5 Aug 2015 16:50:53 +0000 (12:50 -0400)
committerBen Shum <bshum@biblio.org>
Wed, 19 Aug 2015 01:47:08 +0000 (21:47 -0400)
The Show link in the available copies area of the record summary will now
display for any org unit that owns a copy of the title, even if those copies
are unavailable. This commit also changes the language in the display to read
"x of y copies available at z library."

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/templates/opac/parts/record/copy_counts.tt2
docs/RELEASE_NOTES_NEXT/OPAC/change-available-copies-display.adoc [new file with mode: 0644]

index 6e95ae7..eee85e8 100644 (file)
@@ -7,6 +7,7 @@
         ou_hiding_disabled = ctx.org_hiding_disabled();
         WHILE depth < depths;
             ou_avail = ctx.copy_summary.$depth.available;
         ou_hiding_disabled = ctx.org_hiding_disabled();
         WHILE depth < depths;
             ou_avail = ctx.copy_summary.$depth.available;
+            ou_count = ctx.copy_summary.$depth.count;
             ou_id = ctx.copy_summary.$depth.org_unit;
             cp_org_unit = ctx.get_aou(ou_id);
             skip_me = !ou_hiding_disabled AND !ctx.org_within_hiding_scope(ou_id);
             ou_id = ctx.copy_summary.$depth.org_unit;
             cp_org_unit = ctx.get_aou(ou_id);
             skip_me = !ou_hiding_disabled AND !ctx.org_within_hiding_scope(ou_id);
@@ -18,9 +19,9 @@
             displayed_ous.$ou_name = 1;
     %]
     <li>
             displayed_ous.$ou_name = 1;
     %]
     <li>
-    [% l('[quant,_1,copy,copies] at [_2].', ou_avail, ou_name) | html %]
+    [% l('[_1] of [quant,_2,copy,copies] available at [_3].', ou_avail, ou_count, ou_name) | html %]
     [%- this_depth = ctx.get_aou(ou_id).ou_type.depth;
     [%- this_depth = ctx.get_aou(ou_id).ou_type.depth;
-        IF ou_avail > 0 && this_depth != ctx.copy_depth %]
+        IF ou_count > 0 && this_depth != ctx.copy_depth %]
     <a href="[% mkurl('', {copy_depth => this_depth}, ['copy_offset']); %]"
        title="[% l('Show copies at [_1]', ou_name); %]">
        [%- l('(Show)'); %]</a>
     <a href="[% mkurl('', {copy_depth => this_depth}, ['copy_offset']); %]"
        title="[% l('Show copies at [_1]', ou_name); %]">
        [%- l('(Show)'); %]</a>
diff --git a/docs/RELEASE_NOTES_NEXT/OPAC/change-available-copies-display.adoc b/docs/RELEASE_NOTES_NEXT/OPAC/change-available-copies-display.adoc
new file mode 100644 (file)
index 0000000..b470a38
--- /dev/null
@@ -0,0 +1,11 @@
+Change to Available Copies Display
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+The _Show_ link in the available copies area of the record summary will now
+display for any org unit that owns a copy of a particular title, even if all
+those copies are unavailable. The _Show_ link will not display if a) the copy
+display is already scoped to that org unit or b) the org unit does not own
+copies of the title. 
+
+The language has also been changed to read "x of y copies available at z
+library." 
+