LP#1661688: Add a link and other tweaks to alternate hold pickup feature
authorKathy Lussier <klussier@masslnc.org>
Wed, 21 Feb 2018 20:19:34 +0000 (15:19 -0500)
committerGalen Charlton <gmc@equinoxinitiative.org>
Wed, 28 Feb 2018 21:48:59 +0000 (16:48 -0500)
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>
Open-ILS/src/extras/ils_events.xml
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm
Open-ILS/src/sql/Pg/950.data.seed-values.sql
Open-ILS/src/sql/Pg/upgrade/XXXX.schema.hold_checkout.sql
Open-ILS/src/templates/staff/circ/share/t_event_override_dialog.tt2
Open-ILS/web/js/ui/default/staff/circ/services/circ.js

index a4573b4..473b2d6 100644 (file)
                <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 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"> 
        </event>
        <event code='7017' textcode='ROUTE_TO_COPY_LOCATION'>
                <desc xml:lang="en-US"> 
index caed8bb..deb9541 100644 (file)
@@ -1135,6 +1135,7 @@ sub check_captured_holds {
 
         if ($holdau) {
             $payload->{patron_name} = $holdau->first_given_name . ' ' . $holdau->family_name;
 
         if ($holdau) {
             $payload->{patron_name} = $holdau->first_given_name . ' ' . $holdau->family_name;
+            $payload->{patron_id} = $holdau->id;
         } else {
             $payload->{patron_name} = "???";
         }
         } else {
             $payload->{patron_name} = "???";
         }
index 65f3a70..f6c7d19 100644 (file)
@@ -16739,14 +16739,6 @@ VALUES
                'coust', 'description'),
        'bool');
 
                '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
 INSERT INTO config.org_unit_setting_type
 ( name, grp, label, description, datatype )
 VALUES
index db79223..fa7b41d 100644 (file)
@@ -15,13 +15,5 @@ VALUES
                'coust', 'description'),
        'bool');
 
                '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;
 
 COMMIT;
 
index 0adeb11..398796f 100644 (file)
@@ -13,8 +13,8 @@
         <div class="panel-body">
           <div ng-if="copy_barcode" class="strong-text-2">{{copy_barcode}}</div>
           {{evt.desc}}
         <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>
             <div>
                <label><input type="checkbox" ng-model="formdata.clearHold"/> 
                [% l('Cancel this hold upon checkout?') %]</label>
index 3bf866c..d2a72f0 100644 (file)
@@ -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.patronName = $scope.holdEvent.payload.patron_name;
                    $scope.holdID = $scope.holdEvent.payload.hold_id;
+                   $scope.patronID = $scope.holdEvent.payload.patron_id;
                 }
 
                 $scope.auto_override =
                 }
 
                 $scope.auto_override =