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=0e83fea1990dcbadb3c49315efa497d4bf8f5fc9;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 c50e1dd24b..fa42b12541 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm
@@ -93,7 +93,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'}) {
@@ -101,7 +101,6 @@ sub load_getit {
         }
     }
 
-
     $self->ctx->{page} = 'getit'; # repair the page
     return Apache2::Const::OK;
 }
@@ -117,6 +116,11 @@ sub login_and_place_hold {
     return Apache2::Const::HTTP_BAD_REQUEST 
         unless $pickup_lib =~ /^\d+$/;
 
+    #Temporary workaround for selection problem - if PINES (value 1) is selected, reload page
+    if ($pickup_lib == '1') {
+        return $self->load_login;
+    }
+
     my $new_uri = $self->apache->unparsed_uri;
     my $sep = ($new_uri =~ /\?/) ? '&' : '?';