From 56005d6684b42d6b15ec982c48e9d77a96d1858d Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Wed, 13 Jul 2011 12:24:38 -0400
Subject: [PATCH] Preset correct default pickup lib for staff placing holds for
 patrons

Also, for hold placement, only enable org unit selection where
can_have_vols is true.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---
 Open-ILS/web/js/ui/default/opac/staff.js                   |  7 +++++++
 Open-ILS/web/templates/default/opac/parts/org_selector.tt2 | 11 +++++++++--
 Open-ILS/web/templates/default/opac/parts/place_hold.tt2   |  9 ++++-----
 Open-ILS/xul/staff_client/server/patron/holds.js           |  3 ++-
 4 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/Open-ILS/web/js/ui/default/opac/staff.js b/Open-ILS/web/js/ui/default/opac/staff.js
index b5ef7f5285..16234b569f 100644
--- a/Open-ILS/web/js/ui/default/opac/staff.js
+++ b/Open-ILS/web/js/ui/default/opac/staff.js
@@ -20,6 +20,13 @@ function staff_hold_usr_input_disabler(input) {
 }
 function staff_hold_usr_prepop() {
     if (xulG && xulG.patron_barcode) {
+        var sel = document.getElementById("pickup_lib");
+        for (var i = 0; i < sel.options.length; i++) {
+            if (sel.options[i].value == xulG.patron_home_ou) {
+                sel.selectedIndex = i;
+                break;
+            }
+        }
         document.getElementById("hold_usr_input").value = xulG.patron_barcode;
         document.getElementById("hold_usr_input").disabled = false;
         document.getElementById("hold_usr_is_requestor_not").checked = true;
diff --git a/Open-ILS/web/templates/default/opac/parts/org_selector.tt2 b/Open-ILS/web/templates/default/opac/parts/org_selector.tt2
index 2daf63ad75..d8d3fc788e 100644
--- a/Open-ILS/web/templates/default/opac/parts/org_selector.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/org_selector.tt2
@@ -1,6 +1,13 @@
 [%
-    BLOCK build_org_selector_options; %]
-        <option value='[% walker.id %]' [% IF walker.id == value %] selected='selected' [% END %]>
+    BLOCK build_org_selector_options;
+        disabled = '';
+        IF can_have_vols_only;
+            UNLESS walker.ou_type.can_have_vols == 't';
+                disabled = 'disabled="disabled"';
+            END;
+        END;
+%]
+        <option value='[% walker.id %]' [% IF walker.id == value %] selected='selected' [% END %][% disabled %]>
             [%
                 pad = walker.ou_type.depth * 2;
                 FOR idx IN [0..pad]; '&nbsp;'; END;
diff --git a/Open-ILS/web/templates/default/opac/parts/place_hold.tt2 b/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
index 9e90c7ac1e..69c469ad35 100644
--- a/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
@@ -58,15 +58,14 @@
             [% END %]
             <p>
                 [% | l(attrs.title, ctx.get_aou(ctx.default_pickup_lib).name) %]
-                You would like to place a hold on <strong><q>[_1]</q></strong> to be picked up at [_2].
-                If this is correct, press <strong>SUBMIT</strong>.
+                You would like to place a hold on <strong><q>[_1]</q></strong>.<br />
+                If this is correct, confirm your pickup location and click <strong>SUBMIT</strong>.
                 [% END %]
             </p>
             <p>
-                [% l('If you would like to change the library pick up location, select from the location dropdown menu.') %]
-                <br class="clear-both" />
+                [% l('Pickup location:') %]
                 [% PROCESS "default/opac/parts/org_selector.tt2";
-                    PROCESS build_org_selector name='pickup_lib' value=ctx.default_pickup_lib %]
+                    PROCESS build_org_selector name='pickup_lib' value=ctx.default_pickup_lib id='pickup_lib' can_have_vols_only=1 %]
             </p>
             <p>
                 [% |l %]If you use the Traveling Library Center (TLC) and ABC Express
diff --git a/Open-ILS/xul/staff_client/server/patron/holds.js b/Open-ILS/xul/staff_client/server/patron/holds.js
index 234b2b615e..5dd912a1ff 100644
--- a/Open-ILS/xul/staff_client/server/patron/holds.js
+++ b/Open-ILS/xul/staff_client/server/patron/holds.js
@@ -1341,7 +1341,8 @@ patron.holds.prototype = {
                                                 obj.error.standard_unexpected_error_alert('holds.js, opac_hold_placed(): ',E);
                                             }
                                         },
-                                        'patron_barcode' : obj.patron_barcode
+                                        'patron_barcode' : obj.patron_barcode,
+                                        'patron_home_ou': patron.util.retrieve_au_via_id(ses(),obj.patron_id).home_ou()
                                     },
                                     'url_prefix' : xulG.url_prefix,
                                     'url' : xulG.url_prefix( urls.browser )
-- 
2.11.0