From: Thomas Berezansky <tsbere@mvlc.org>
Date: Wed, 21 Dec 2011 21:19:47 +0000 (-0500)
Subject: TPac: Don't show part selector on hold result page
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=e981f761f16dcad55276b4df65303ae7ed40df3e;p=evergreen%2Fmasslnc.git

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>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
---

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 %]