LP#1661688: Add a link and other tweaks to alternate hold pickup feature

Adding a couple of tweaks to the alternate hold pickup feature:
  * Created a link for the hold patron so that staff can easily retrieve the record and potentially verify that the user allows the alternate patron to pick
up their holds.
  * Tweaked the language in the event dialog.
  * Removed the new OU setting from actor.org_unit_setting so that a default value is not set.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

diff --git a/Open-ILS/src/extras/ils_events.xml b/Open-ILS/src/extras/ils_events.xml
index a4573b4..473b2d6 100644 (file)
--- a/Open-ILS/src/extras/ils_events.xml
+++ b/Open-ILS/src/extras/ils_events.xml
@@ -892,7 +892,7 @@
                <desc xml:lang="en-US">This requested action would result in a negative patron balance</desc>
        </event>
        <event code='7016' textcode='ITEM_ON_HOLDS_SHELF'>
-               <desc xml:lang="en-US">This item is currently on the holds shelf for another patron</desc>
+               <desc xml:lang="en-US">This item is currently on the holds shelf for another patron:</desc>
        </event>
        <event code='7017' textcode='ROUTE_TO_COPY_LOCATION'>
                <desc xml:lang="en-US"> 

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm
index caed8bb..deb9541 100644 (file)
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm
@@ -1135,6 +1135,7 @@ sub check_captured_holds {
 
         if ($holdau) {
             $payload->{patron_name} = $holdau->first_given_name . ' ' . $holdau->family_name;
+            $payload->{patron_id} = $holdau->id;
         } else {
             $payload->{patron_name} = "???";
         }

diff --git a/Open-ILS/src/sql/Pg/950.data.seed-values.sql b/Open-ILS/src/sql/Pg/950.data.seed-values.sql
index 65f3a70..f6c7d19 100644 (file)
--- a/Open-ILS/src/sql/Pg/950.data.seed-values.sql
+++ b/Open-ILS/src/sql/Pg/950.data.seed-values.sql
@@ -16739,14 +16739,6 @@ VALUES
                'coust', 'description'),
        'bool');
 
-INSERT INTO actor.org_unit_setting (
-    org_unit, name, value
-) VALUES (
-    (SELECT id FROM actor.org_unit WHERE parent_ou IS NULL),
-    'circ.clear_hold_on_checkout',
-    'false'
-);
-
 INSERT INTO config.org_unit_setting_type
 ( name, grp, label, description, datatype )
 VALUES

diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.hold_checkout.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.hold_checkout.sql
index db79223..fa7b41d 100644 (file)
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.hold_checkout.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.hold_checkout.sql
@@ -15,13 +15,5 @@ VALUES
                'coust', 'description'),
        'bool');
 
-INSERT INTO actor.org_unit_setting (
-    org_unit, name, value
-) VALUES (
-    (SELECT id FROM actor.org_unit WHERE parent_ou IS NULL),
-    'circ.clear_hold_on_checkout',
-    'false'
-);
-
 COMMIT;
 

diff --git a/Open-ILS/src/templates/staff/circ/share/t_event_override_dialog.tt2 b/Open-ILS/src/templates/staff/circ/share/t_event_override_dialog.tt2
index 0adeb11..398796f 100644 (file)
--- a/Open-ILS/src/templates/staff/circ/share/t_event_override_dialog.tt2
+++ b/Open-ILS/src/templates/staff/circ/share/t_event_override_dialog.tt2
@@ -13,8 +13,8 @@
         <div class="panel-body">
           <div ng-if="copy_barcode" class="strong-text-2">{{copy_barcode}}</div>
           {{evt.desc}}
-          <div ng-if="evt.textcode == 'ITEM_ON_HOLDS_SHELF'">
-             [% l('for ') %] {{patronName}}.
+          <div ng-if="evt.textcode == 'ITEM_ON_HOLDS_SHELF'"> 
+              <a target="_blank" href="[% ctx.base_path %]/staff/circ/patron/{{patronID}}/checkout">{{patronName}}</a>.
             <div>
                <label><input type="checkbox" ng-model="formdata.clearHold"/> 
                [% l('Cancel this hold upon checkout?') %]</label>

diff --git a/Open-ILS/web/js/ui/default/staff/circ/services/circ.js b/Open-ILS/web/js/ui/default/staff/circ/services/circ.js
index 3bf866c..d2a72f0 100644 (file)
--- a/Open-ILS/web/js/ui/default/staff/circ/services/circ.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/services/circ.js
@@ -752,6 +752,7 @@ function($uibModal , $q , egCore , egAlertDialog , egConfirmDialog,  egAddCopyAl
 
                    $scope.patronName = $scope.holdEvent.payload.patron_name;
                    $scope.holdID = $scope.holdEvent.payload.hold_id;
+                   $scope.patronID = $scope.holdEvent.payload.patron_id;
                 }
 
                 $scope.auto_override =