From a2d831df8adfdca498f707ea50a28bf78aeb9c74 Mon Sep 17 00:00:00 2001
From: Thomas Berezansky <tsbere@mvlc.org>
Date: Wed, 21 Dec 2011 16:19:47 -0500
Subject: [PATCH] TPac: Don't show part selector on hold result page

Instead, duplicate the hold list's "part name in parens" display, when a
part was selected.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
---
 .../src/templates/opac/parts/place_hold_result.tt2 | 30 ++++++++++------------
 1 file changed, 13 insertions(+), 17 deletions(-)

diff --git a/Open-ILS/src/templates/opac/parts/place_hold_result.tt2 b/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
index 7b6585923a..87e2d40340 100644
--- a/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
+++ b/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
@@ -34,24 +34,20 @@
                         [% IF override %] checked='checked' [% ELSE %] disabled='disabled' [% END %]/>
                 </td>
                 <td>
-                    <div class='hold-items-list-title'>[% attrs.title_extended | html %]</div>
+                    [%
+                        title = attrs.title_extended;
+                        IF hdata.selected_part AND hdata.parts AND hdata.parts.size > 0;
+                            FOREACH part IN hdata.parts;
+                                IF hdata.selected_part == part.id;
+                                    title = l('[_1] ([_2])', title, part.label);
+                                END;
+                            END;
+                        END;
+                    %]
+                        
+                    <div class='hold-items-list-title'>[% title | html %]</div>
                     [% IF hdata.parts %]
-                        [% IF hdata.parts.size > 0 %]
-                        <div style='padding-left: 10px'>
-                            <span>[% hdata.part_required ? l('Select a Part:') : l('Select a Part (optional):') %]</span>
-                            <select name='part'>
-                                [% IF !hdata.part_required %]
-                                <option [% UNLESS hdata.selected_part %]selected='selected'[% END %] value=''>[% l('- All Parts -') %]</option>
-                                [% END %]
-                                [% FOR part IN hdata.parts %]
-                                <option value='[% part.id %]' 
-                                    [% IF (hdata.selected_part || '') == part.id %]selected='selected'[% END %]>[% part.label | html %]</option>
-                                [% END %]
-                            </select>
-                        </div>
-                        [% ELSE %]
-                        <input type='hidden' name='part' value=''/>
-                        [% END %]
+                        <input type='hidden' name='part' value='[% hdata.selected_part || '' %]'/>
                     [% END %]
                     <div>
                         [% IF hdata.hold_success %]
-- 
2.11.0