From c40de80dffb8175ebe9eb6f39030bb62d7bbe178 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berick@esilibrary.com>
Date: Fri, 7 Oct 2011 16:11:33 -0400
Subject: [PATCH] TPac; display mono part in account holds list/edit

Part label is appended to the title, wrapped in (parens).

Signed-off-by: Bill Erickson <berick@esilibrary.com>
---
 Open-ILS/src/templates/opac/myopac/holds.tt2      | 24 +++++++++++------------
 Open-ILS/src/templates/opac/myopac/holds/edit.tt2 | 11 ++++++++---
 2 files changed, 19 insertions(+), 16 deletions(-)

diff --git a/Open-ILS/src/templates/opac/myopac/holds.tt2 b/Open-ILS/src/templates/opac/myopac/holds.tt2
index f008fb722a..16c1e1d465 100644
--- a/Open-ILS/src/templates/opac/myopac/holds.tt2
+++ b/Open-ILS/src/templates/opac/myopac/holds.tt2
@@ -37,18 +37,10 @@
                         <option id='myopac_holds_actions_none' value=''>
                         -- [% l("Actions for selected holds") %] --
                         </option>
-                        <option value='suspend'>
-                            [% l("Suspend") %]
-                        </option>
-                        <option value='activate'>
-                            [% l("Activate") %]
-                        </option>
-                        <!-- XXX maybe later <option value='thaw_date'>
-                            [% l("Set Active Date") %]
-                        </option> -->
-                        <option value='cancel'>
-                            [% l("Cancel") %]
-                        </option>
+                        <option value='suspend'>[% l("Suspend") %]</option>
+                        <option value='activate'>[% l("Activate") %]</option>
+                        <!-- XXX maybe later <option value='thaw_date'>[% l("Set Active Date") %]</option> -->
+                        <option value='cancel'>[% l("Cancel") %]</option>
                     </select>
                 </td>
                 <td width="1" style="padding-left:9px;">
@@ -135,7 +127,13 @@
                     </td>
                     <td width="138">
                         <div style="margin-top:10px;margin-bottom:10px;">
-                            <a href="[% ctx.opac_root %]/record/[% hold.hold.bre.id %]">[% attrs.title | html %]</a>
+                            [% 
+                                title = attrs.title;
+                                IF ahr.hold_type == 'P';
+                                    title = l('[_1] ([_2])', title, hold.hold.part.label);
+                                END;
+                            %]
+                            <a href="[% ctx.opac_root %]/record/[% hold.hold.bre.id %]">[% title | html %]</a>
                         </div>
                     </td>
                     <td width="123">
diff --git a/Open-ILS/src/templates/opac/myopac/holds/edit.tt2 b/Open-ILS/src/templates/opac/myopac/holds/edit.tt2
index 256b385ffb..b6d6059d4d 100644
--- a/Open-ILS/src/templates/opac/myopac/holds/edit.tt2
+++ b/Open-ILS/src/templates/opac/myopac/holds/edit.tt2
@@ -22,12 +22,17 @@
         <a href="[% ctx.opac_root %]/myopac/holds">[% l('List all holds') %]</a>
     </div>
     <div id="hold_editor">
-        [% IF hold %]
-            <h1>[% attrs.title | html %]</h1>
+        [%  IF hold;
+                title = attrs.title;
+                IF ahr.hold_type == 'P';
+                    title = l('[_1] ([_2])', title, hold.hold.part.label);
+                END 
+        %]
+            <h1>[% title | html %]</h1>
             <h2>[% attrs.author | html %]</h2>
             [% IF attrs.format_icon %]<p>
                 <strong>[% l('Format:') %]</strong>
-                <img src="[% ctx.media_prefix %]/images/[% attrs.format_icon %]" alt="[% attrs.format %]" title="[% attrs.title | html %]" />
+                <img src="[% ctx.media_prefix %]/images/[% attrs.format_icon %]" alt="[% attrs.format %]" title="[% title | html %]" />
             </p>[% END %]
             <p>
                 <strong>[% l('Status') %]</strong>: [% hold.human_status %]
-- 
2.11.0