From 8dd748d5bcf3baa773141609982febc18bfca853 Mon Sep 17 00:00:00 2001
From: Terran McCanna <tmccanna@georgialibraries.org>
Date: Mon, 13 Jan 2014 20:49:10 -0500
Subject: [PATCH] temporary workaround for holds error when user isn't logged
 in and doesn't select a pickup location

---
 Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

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 =~ /\?/) ? '&' : '?';
 
-- 
2.11.0