From: Terran McCanna <tmccanna@georgialibraries.org>
Date: Tue, 14 Jan 2014 01:49:10 +0000 (-0500)
Subject: temporary workaround for holds error when user isn't logged in and doesn't select... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=eae42fd2b8bbfcab3ac6a8cd45c52af3f8afe86d;p=evergreen%2Fpines.git

temporary workaround for holds error when user isn't logged in and doesn't select a pickup location
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm
index 5c0a1830e7..7354c6d7dd 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm
@@ -94,7 +94,7 @@ sub load_getit {
         ]);
     }
 
-    # If user is logged in and has a default pickup location, grab it
+    # If user is logged in and has a default pickup location for holds, grab it
     if ($ctx->{user}) {
         my $set_map = $self->ctx->{user_setting_map};
         if ($$set_map{'opac.default_pickup_location'}) {
@@ -123,8 +123,8 @@ sub login_and_place_hold {
     return Apache2::Const::HTTP_BAD_REQUEST 
         unless $pickup_lib =~ /^\d+$/;
 
-    #If a pickup library hasn't been selected, reload page
-    if ($pickup_lib == '0') {
+    #Temporary workaround for selection problem - if PINES (value 1) is selected, reload page
+    if ($pickup_lib == '1') {
         return $self->load_login;
     }