LP#1838995: Change end-user terminology user/miker/lp-1838995-hold-subscriptions-rebase
authorMike Rylander <mrylander@gmail.com>
Wed, 29 Jul 2020 18:37:40 +0000 (14:37 -0400)
committerMike Rylander <mrylander@gmail.com>
Wed, 19 Aug 2020 14:04:18 +0000 (10:04 -0400)
After a community suggestion and weighing some options, this commit
changes the end-user visible terminology of mixed "batch holds" (the
prevailing staff-side term) and "hold subscriptions" (the pevailing
patron-side term) to the single term "hold groups".

Signed-off-by: Mike Rylander <mrylander@gmail.com>
26 files changed:
Open-ILS/examples/fm_IDL.xml
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Account.pm
Open-ILS/src/sql/Pg/950.data.seed-values.sql
Open-ILS/src/sql/Pg/upgrade/XXXX.data.hold_buckets.sql
Open-ILS/src/templates/opac/myopac/ebook_holds.tt2
Open-ILS/src/templates/opac/myopac/ebook_holds_ready.tt2
Open-ILS/src/templates/opac/myopac/hold_history.tt2
Open-ILS/src/templates/opac/myopac/hold_subscriptions.tt2
Open-ILS/src/templates/opac/myopac/holds.tt2
Open-ILS/src/templates/opac/parts/place_hold.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/index.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_bucket_info.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_event.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_event_create.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_grid_menu.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_list.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_pending.tt2
Open-ILS/src/templates/staff/cat/bucket/batch_hold/t_view.tt2
Open-ILS/src/templates/staff/circ/patron/index.tt2
Open-ILS/src/templates/staff/circ/patron/t_hold_subscriptions.tt2
Open-ILS/src/templates/staff/circ/patron/t_search_results.tt2
Open-ILS/src/templates/staff/navbar.tt2
Open-ILS/web/js/ui/default/staff/cat/bucket/batch_hold/app.js
Open-ILS/web/js/ui/default/staff/services/user-bucket.js
docs/RELEASE_NOTES_NEXT/Circulation/hold-subscriptions.adoc

index a72dbd6..d8228da 100644 (file)
@@ -7288,7 +7288,7 @@ SELECT  usr,
             </actions>
         </permacrud>
        </class>
-       <class id="abhe" controller="open-ils.cstore open-ils.pcrud" oils_obj:fieldmapper="action::batch_hold_event" oils_persist:tablename="action.batch_hold_event" reporter:label="Batch Hold Event">
+       <class id="abhe" controller="open-ils.cstore open-ils.pcrud" oils_obj:fieldmapper="action::batch_hold_event" oils_persist:tablename="action.batch_hold_event" reporter:label="Hold Group Event">
                <fields oils_persist:primary="id" oils_persist:sequence="action.batch_hold_event_id_seq">
                        <field name="id" reporter:datatype="id" />
                        <field name="staff" reporter:datatype="link"/>
@@ -7306,14 +7306,14 @@ SELECT  usr,
                </links>
                <permacrud xmlns="http://open-ils.org/spec/opensrf/IDL/permacrud/v1">
                        <actions>
-                <create permission="MANAGE_BATCH_HOLDS" global_required="true"/>
+                <create permission="MANAGE_HOLD_GROUPS" global_required="true"/>
                 <retrieve/>
-                <update permission="MANAGE_BATCH_HOLDS" global_required="true"/>
-                <delete permission="MANAGE_BATCH_HOLDS" global_required="true"/>
+                <update permission="MANAGE_HOLD_GROUPS" global_required="true"/>
+                <delete permission="MANAGE_HOLD_GROUPS" global_required="true"/>
             </actions>
         </permacrud>
        </class>
-       <class id="abhem" controller="open-ils.cstore open-ils.pcrud" oils_obj:fieldmapper="action::batch_hold_event_map" oils_persist:tablename="action.batch_hold_event_map" reporter:label="Batch Hold Event Map">
+       <class id="abhem" controller="open-ils.cstore open-ils.pcrud" oils_obj:fieldmapper="action::batch_hold_event_map" oils_persist:tablename="action.batch_hold_event_map" reporter:label="Hold Group Event Map">
                <fields oils_persist:primary="id" oils_persist:sequence="action.batch_hold_event_map_id_seq">
                        <field name="id" reporter:datatype="id" />
                        <field name="batch_hold_event" reporter:datatype="link"/>
index fdab54d..45175e1 100644 (file)
@@ -216,8 +216,8 @@ sub test_and_create_batch_hold_event {
 
     my $org = $e->requestor->ws_ou || $e->requestor->home_ou;
     # the perm locaiton shouldn't really matter here since holds
-    # will exist all over and MANAGE_BATCH_HOLDS should be universal
-    my $evt = $U->check_perms($e->requestor->id, $org, 'MANAGE_BATCH_HOLDS');
+    # will exist all over and MANAGE_HOLD_GROUPS should be universal
+    my $evt = $U->check_perms($e->requestor->id, $org, 'MANAGE_HOLD_GROUPS');
     return $evt if $evt;
 
     my $rand_setting = $U->ou_ancestor_setting_value($org, 'holds.subscription.randomize');
@@ -372,7 +372,7 @@ __PACKAGE__->register_method(
         desc => q/This is for batch creating a set of holds where every field is identical except for the target users./,
         params => [
             { desc => 'Authentication token', type => 'string' },
-            { desc => 'Batch Hold Event ID to roll back', type => 'number' },
+            { desc => 'Hold Group Event ID to roll back', type => 'number' },
         ],
         return => {
             desc => 'Stream of objects structured as {total=>X, count=>Y} on success, event on error',
@@ -387,7 +387,7 @@ sub rollback_batch_hold_event {
     return $e->die_event unless $e->checkauth;
 
     my $org = $e->requestor->ws_ou || $e->requestor->home_ou;
-    my $evt = $U->check_perms($e->requestor->id, $org, 'MANAGE_BATCH_HOLDS');
+    my $evt = $U->check_perms($e->requestor->id, $org, 'MANAGE_HOLD_GROUPS');
     return $evt if $evt;
 
     my $batch_event = $e->retrieve_action_batch_hold_event($event_id);
index d3ead5c..11b79e0 100644 (file)
@@ -1588,7 +1588,7 @@ sub load_place_hold {
 
         if ($ctx->{hold_subscription}) {
             # this is a batch event, hold "user" is a bucket id
-            $logger->info("Hold Subscription Event requested for user bucket: " . $ctx->{hold_subscription});
+            $logger->info("Hold Group Event requested for user bucket: " . $ctx->{hold_subscription});
             $usr = $e->retrieve_container_user_bucket($ctx->{hold_subscription});
         }
     }
index b116b07..2fbd101 100644 (file)
@@ -1940,7 +1940,7 @@ INSERT INTO permission.perm_list ( id, code, description ) VALUES
     'View booking resource types', 'ppl', 'description')),
  ( 622, 'VIEW_BOOKING_RESOURCE', oils_i18n_gettext(622,
     'View booking resources', 'ppl', 'description')),
- ( 623, 'MANAGE_BATCH_HOLDS', oils_i18n_gettext(623,
+ ( 623, 'MANAGE_HOLD_GROUPS', oils_i18n_gettext(623,
     'Manage batch (subscription) hold events', 'ppl', 'description'))
 ;
 
@@ -2855,7 +2855,7 @@ INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (4, oils_i18n_get
 INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (5, oils_i18n_gettext(5, 'Staff forced', 'ahrcc', 'label'));
 INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (6, oils_i18n_gettext(6, 'Patron via OPAC', 'ahrcc', 'label'));
 INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (7, oils_i18n_gettext(7, 'Patron via SIP', 'ahrcc', 'label'));
-INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (8, oils_i18n_gettext(8, 'Subscription Event rollback', 'ahrcc', 'label'));
+INSERT INTO action.hold_request_cancel_cause (id,label) VALUES (8, oils_i18n_gettext(8, 'Hold Group Event rollback', 'ahrcc', 'label'));
 SELECT SETVAL('action.hold_request_cancel_cause_id_seq', 100);
 
 
@@ -5734,7 +5734,7 @@ INSERT INTO container.user_bucket_type (code,label) VALUES ('folks:hold.view', o
 INSERT INTO container.user_bucket_type (code,label) VALUES ('folks:hold.cancel', oils_i18n_gettext('folks:hold.cancel', 'Cancel Holds', 'cubt', 'label'));
 
 INSERT INTO container.user_bucket_type (code,label) SELECT code,label FROM container.copy_bucket_type where code = 'staff_client';
-INSERT INTO container.user_bucket_type (code,label) VALUES ('hold_subscription', oils_i18n_gettext('hold_subscription', 'Hold Subscription Container', 'cubt', 'label'));
+INSERT INTO container.user_bucket_type (code,label) VALUES ('hold_subscription', oils_i18n_gettext('hold_subscription', 'Hold Group Container', 'cubt', 'label'));
 
 ----------------------------------
 -- MARC21 record structure data --
@@ -17155,7 +17155,7 @@ INSERT INTO action_trigger.environment (
 );
 
 INSERT INTO action_trigger.event_definition (active, owner, name, hook, validator, reactor, delay, delay_field, group_field, cleanup_success, template)
-    VALUES ('f', 1, 'Subscription Hold Placed for Patron Email Notification', 'hold_request.success', 'NOOP_True', 'SendEmail', '30 minutes', 'request_time', 'usr', 'CreateHoldNotification',
+    VALUES ('f', 1, 'Hold Group Hold Placed for Patron Email Notification', 'hold_request.success', 'NOOP_True', 'SendEmail', '30 minutes', 'request_time', 'usr', 'CreateHoldNotification',
 $$
 [%- USE date -%]
 [%- user = target.0.usr -%]
@@ -17188,7 +17188,7 @@ INSERT INTO action_trigger.event_definition (
     active, owner, name, hook, validator, reactor, cleanup_success,
     delay, delay_field, group_field, template
 ) VALUES (
-    false, 1, 'Subscription Hold Placed for Patron SMS Notification', 'hold_request.success', 'NOOP_True',
+    false, 1, 'Hold Group Hold Placed for Patron SMS Notification', 'hold_request.success', 'NOOP_True',
     'SendSMS', 'CreateHoldNotification', '00:30:00', 'shelf_time', 'sms_notify',
     '[%- USE date -%]
 [%- user = target.0.usr -%]
index f48fe33..9337cf7 100644 (file)
@@ -17,7 +17,7 @@ CREATE TABLE action.batch_hold_event_map (
     hold                INT     NOT NULL REFERENCES action.hold_request (id) ON UPDATE CASCADE ON DELETE CASCADE
 );
 
-INSERT INTO container.user_bucket_type (code,label) VALUES ('hold_subscription','Hold Subscription Container');
+INSERT INTO container.user_bucket_type (code,label) VALUES ('hold_subscription','Hold Group Container');
 
 INSERT INTO config.org_unit_setting_type
     (name, label, description, grp, datatype)
@@ -41,13 +41,13 @@ VALUES (
 
 -- Committer! Verify permission id before pushing!
 INSERT INTO permission.perm_list (id,code,description)
-  VALUES ( 623, 'MANAGE_BATCH_HOLDS', oils_i18n_gettext(623, 'Manage batch (subscription) hold events', 'ppl', 'description'));
+  VALUES ( 623, 'MANAGE_HOLD_GROUPS', oils_i18n_gettext(623, 'Manage hold groups and hold group events', 'ppl', 'description'));
 
 INSERT INTO action.hold_request_cancel_cause (id,label)
-  VALUES ( 8, oils_i18n_gettext(8, 'Subscription Event rollback', 'ahrcc', 'label'));
+  VALUES ( 8, oils_i18n_gettext(8, 'Hold Group Event rollback', 'ahrcc', 'label'));
 
 INSERT INTO action_trigger.event_definition (active, owner, name, hook, validator, reactor, delay, delay_field, group_field, cleanup_success, template)
-    VALUES ('f', 1, 'Subscription Hold Placed for Patron Email Notification', 'hold_request.success', 'NOOP_True', 'SendEmail', '30 minutes', 'request_time', 'usr', 'CreateHoldNotification',
+    VALUES ('f', 1, 'Hold Group Hold Placed for Patron Email Notification', 'hold_request.success', 'NOOP_True', 'SendEmail', '30 minutes', 'request_time', 'usr', 'CreateHoldNotification',
 $$
 [%- USE date -%]
 [%- user = target.0.usr -%]
@@ -81,7 +81,7 @@ INSERT INTO action_trigger.event_definition (
     active, owner, name, hook, validator, reactor, cleanup_success,
     delay, delay_field, group_field, template
 ) VALUES (
-    false, 1, 'Subscription Hold Placed for Patron SMS Notification', 'hold_request.success', 'NOOP_True',
+    false, 1, 'Hold Group Hold Placed for Patron SMS Notification', 'hold_request.success', 'NOOP_True',
     'SendSMS', 'CreateHoldNotification', '00:30:00', 'shelf_time', 'sms_notify',
     '[%- USE date -%]
 [%- user = target.0.usr -%]
index 3cc6a83..95dd8d5 100644 (file)
@@ -23,7 +23,7 @@
         </div>
         [% IF ctx.my_hold_subscriptions.size > 0 %]
         <div class="align">
-            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Subscriptions") %]</a>
+            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Groups") %]</a>
         </div>
         [% END %]
         <div class="align">
index c5b1f11..d1ce496 100644 (file)
@@ -23,7 +23,7 @@
         </div>
         [% IF ctx.my_hold_subscriptions.size > 0 %]
         <div class="align">
-            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Subscriptions") %]</a>
+            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Groups") %]</a>
         </div>
         [% END %]
         <div class="align">
index bbd0b20..c5da38e 100644 (file)
@@ -25,7 +25,7 @@
         [% END %]
         [% IF ctx.my_hold_subscriptions.size > 0 %]
         <div class="align">
-            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Subscriptions") %]</a>
+            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Groups") %]</a>
         </div>
         [% END %]
         <div class="align selected">
index 79fa632..51fa081 100644 (file)
@@ -25,7 +25,7 @@
         [% END %]
         [% IF ctx.my_hold_subscriptions.size > 0 %]
         <div class="align selected">
-            <a href='#'>[% l("Hold Subscriptions") %]</a>
+            <a href='#'>[% l("Hold Groups") %]</a>
         </div>
         [% END %]
         <div class="align">
@@ -34,7 +34,7 @@
     </div>
 
     <div class="header_middle">
-        <span style="float:left;">[% l("Current Hold Subscriptions") %]</span>
+        <span style="float:left;">[% l("Current Hold Groups") %]</span>
         <span style="float:right;">
             <a class="hide_me" href="#">[% l('Export List') %]</a>
         </span>
@@ -47,7 +47,7 @@
             <big><strong>[% l('No subscriptions found.') %]</strong></big>
         </div>
         [% ELSE %]
-        <table id='acct_holds_hist_header' class='table_no_border_space table_no_cell_pad' title="[% l('Hold Subscriptions') %]">
+        <table id='acct_holds_hist_header' class='table_no_border_space table_no_cell_pad' title="[% l('Hold Groups') %]">
             <thead>
                 <tr>
                     <td><span>[% l('Name') %]</span></td>
index 5c2630f..60acf7d 100644 (file)
@@ -26,7 +26,7 @@
         [% END %]
         [% IF ctx.my_hold_subscriptions.size > 0 %]
         <div class="align">
-            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Subscriptions") %]</a>
+            <a href='[% mkurl('hold_subscriptions', {}, ['limit','offset','available','sort','sort_type']) %]'>[% l("Hold Groups") %]</a>
         </div>
         [% END %]
         <div class="align">
index 5d811ed..809def3 100644 (file)
@@ -137,10 +137,10 @@ function maybeToggleNumCopies(obj) {
                   name="hold_usr_is_requestor" value="2"
                   />
               <label for="hold_usr_is_subscription">
-                  [% l("Place hold for patron Hold Subscription:") %]
+                  [% l("Place hold for patron Hold Group:") %]
               </label>
               <select id='select_hold_subscription' name='hold_subscription'>
-                  <option selected='selected' value=''>[% l('- Subscriptions -') %]</option>
+                  <option selected='selected' value=''>[% l('- Hold Groups -') %]</option>
                   [% FOR sub IN ctx.hold_subscriptions %]
                   <option value='[% sub.id %]'>[% sub.name | html %]</option>
                   [% END %]
index 841f59c..3cee21d 100644 (file)
@@ -1,6 +1,6 @@
 [%
   WRAPPER "staff/base.tt2";
-  ctx.page_title = l("Batch Hold Buckets"); 
+  ctx.page_title = l("Hold Group Buckets"); 
   ctx.page_app = "egCatBatchHoldBuckets";
   ctx.page_ctrl = "BatchHoldBucketCtrl";
 %]
     s.EVENT_ROLLBACK_TITLE =
       "[% l('Cancel all holds created by selected events') %]";
     s.EVENT_NO_TARGET =
-      "[% l('No target provided for hold subscription') %]";
+      "[% l('No target provided for hold group') %]";
     s.EVENT_INVALID_TARGET =
-      "[% l('Invalid target provided for hold subscription') %]";
+      "[% l('Invalid target provided for hold group') %]";
     s.EVENT_CREATE_SUMMARY =
-      "[% l('Created subscription hold for [_1] of [_2] patrons', '{{success}}', '{{total}}') %]";
+      "[% l('Created holds for [_1] of [_2] patrons', '{{success}}', '{{total}}') %]";
   }])
 </script>
 [% END %]
@@ -37,7 +37,7 @@ changing routes with each tab selection anyway.
 <ul class="nav nav-tabs">
   <li ng-class="{active : tab == 'list'}">
     <a href="./cat/bucket/batch_hold/list">
-        [% l('Subscriptions') %]
+        [% l('Hold Groups') %]
     </a>
   </li>
   <li ng-class="{active : tab == 'view'}">
index c320397..bfa0f61 100644 (file)
@@ -1,6 +1,6 @@
 
 <div ng-show="bucket()">
-  <strong>[% l('Subscription #{{bucket().id()}}: {{bucket().name()}}') %]</strong>
+  <strong>[% l('Hold Group #{{bucket().id()}}: {{bucket().name()}}') %]</strong>
   <span ng-show="bucket().description()"> / {{bucket().description()}}</span>
   <br/>
   <span>
@@ -16,6 +16,6 @@
 </div>
 
 <div ng-show="!bucket()">
-  <strong>[% l('No Subscription Selected') %]</strong>
+  <strong>[% l('No Hold Group Selected') %]</strong>
 </div>
 
index ebecad6..58806bd 100644 (file)
@@ -8,10 +8,10 @@
   persist-key="cat.bucket.batch_hold.events">
 
 
-  <eg-grid-menu-item label="[% l('New Batch Hold Event') %]" standalone="true"
+  <eg-grid-menu-item label="[% l('New Hold Group Event') %]" standalone="true"
     handler="openCreateEventDialog"></eg-grid-menu-item>
 
-  <eg-grid-action label="[% l('Cancel batch holds') %]"
+  <eg-grid-action label="[% l('Cancel hold group event') %]"
     handler="rollbackEvent"></eg-grid-action>
 
   <eg-grid-field path="id" required hidden></eg-grid-field>
index bf44fd0..3b965a0 100644 (file)
@@ -6,7 +6,7 @@
     <div class="modal-header">
       <button type="button" class="close" 
         ng-click="cancel()" aria-hidden="true">&times;</button>
-      <h4 class="modal-title">[% l('New Batch Hold Event') %]</h4>
+      <h4 class="modal-title">[% l('New Hold Group Event') %]</h4>
     </div>
     <div class="modal-body">
       <div class="form-group">
index cd3d693..f23e59f 100644 (file)
@@ -1,15 +1,15 @@
 
 <!-- global grid menu displayed on every Bucket page -->
-<eg-grid-menu-item label="[% l('New Subscription') %]" standalone="true" 
+<eg-grid-menu-item label="[% l('New Hold Group') %]" standalone="true" 
   handler="openCreateBucketDialog"></eg-grid-menu-item>
 
-<eg-grid-menu-item label="[% l('Edit Subscription') %]" 
+<eg-grid-menu-item label="[% l('Edit Hold Group') %]" 
   handler="openEditBucketDialog"></eg-grid-menu-item>
 
-<eg-grid-menu-item label="[% l('Delete Subscription') %]" 
+<eg-grid-menu-item label="[% l('Delete Hold Group') %]" 
   handler="openDeleteBucketDialog"></eg-grid-menu-item>
 
-<eg-grid-menu-item label="[% l('Shared Subscription') %]" 
+<eg-grid-menu-item label="[% l('Shared Hold Group') %]" 
   handler="openSharedBucketDialog"></eg-grid-menu-item>
 
 <eg-grid-menu-item divider="true"></eg-grid-menu-item>
index b2de6a0..ea83402 100644 (file)
@@ -7,7 +7,7 @@
   grid-controls="gridControls"
   persist-key="cat.bucket.batch_hold.list">
 
-  <eg-grid-menu-item label="[% l('New Subscription') %]" standalone="true"
+  <eg-grid-menu-item label="[% l('New Hold Group') %]" standalone="true"
     handler="openCreateBucketDialog"></eg-grid-menu-item>
 
   <eg-grid-field path="id" required hidden></eg-grid-field>
index 81b83af..5e441d6 100644 (file)
   items-provider="gridDataProvider"
   persist-key="cat.bucket.batch_hold.pending">
 
-  <eg-grid-menu-item label="[% l('Add All To Subscription') %]" standalone="true"
+  <eg-grid-menu-item label="[% l('Add All To Hold Group') %]" standalone="true"
     handler="addAllPending"></eg-grid-menu-item>
 
   <!-- actions drop-down -->
-  <eg-grid-action label="[% l('Add To Subscription') %]" 
+  <eg-grid-action label="[% l('Add To Hold Group') %]" 
     handler="addToBucket"></eg-grid-action>
 
   <eg-grid-action label="[% l('Clear List') %]" 
index 240b172..45c0c4b 100644 (file)
@@ -5,7 +5,7 @@
   idl-class="au"
   auto-fields="true"
   grid-controls="gridControls"
-  menu-label="[% l('Subscriptions') %]"
+  menu-label="[% l('Hold Groups') %]"
   persist-key="cat.bucket.batch_hold.view">
 
   [% INCLUDE 'staff/cat/bucket/batch_hold/t_grid_menu.tt2' %]
index cd2b94b..1afb3eb 100644 (file)
@@ -78,7 +78,7 @@ angular.module('egCoreMod').run(['egStrings', function(s) {
   s.PAGE_TITLE_PATRON_ITEMS_OUT = "[% l('Items Out') %]";
   s.PAGE_TITLE_PATRON_EDIT = "[% l('Edit') %]";
   s.MERGE_SELF_NOT_ALLOWED = "[% l('Logged in account cannot be merged') %]"
-  s.REMOVE_HOLD_SUBSCRIPTIONS = "[% l('Remove selected Hold Subscriptions for user?') %]"
+  s.REMOVE_HOLD_SUBSCRIPTIONS = "[% l('Remove selected Hold Groups for user?') %]"
 }]);
 </script>
 
@@ -194,7 +194,7 @@ angular.module('egCoreMod').run(['egStrings', function(s) {
           </li>
           <li>
             <a href="./circ/patron/{{patron().id()}}/hold_subscriptions">
-              [% l('Hold Subscriptions') %]
+              [% l('Hold Groups') %]
             </a>
           </li>
           <li>
index 8ec23a1..a33c935 100644 (file)
@@ -1,4 +1,4 @@
-<div class="strong-text-2">[% l('Hold Subscriptions') %]</div>
+<div class="strong-text-2">[% l('Hold Groups') %]</div>
 
 <eg-grid
   features="allowAll,-display"
@@ -9,7 +9,7 @@
   items-provider="gridDataProvider"
   persist-key="circ.patron.batch_hold.list">
 
-<eg-grid-action label="[% l('Remove Subscription') %]"
+<eg-grid-action label="[% l('Remove Hold Groups') %]"
   handler="removeSubscriptions"></eg-grid-action>
 
   <eg-grid-field path="id" required hidden></eg-grid-field>
index 909215b..6d4e028 100644 (file)
@@ -24,7 +24,7 @@
     handler="addToBucket" disabled="need_one_selected"></eg-grid-menu-item>
 
   <eg-grid-menu-item ng-repeat="bkt in bucketSvc.allSubscriptions"
-    label="[% 'Hold Subscription: ' %] {{bkt.name()}}" handler-data="bkt"
+    label="[% 'Hold Groups: ' %] {{bkt.name()}}" handler-data="bkt"
     handler="addToBucket" disabled="need_one_selected"></eg-grid-menu-item>
 
   <eg-grid-field label="[% l('ID') %]" path='id' visible></eg-grid-field>
index ed52833..e400c75 100644 (file)
           <li>
             <a href="./cat/bucket/batch_hold/list" target="_self">
               <span class="glyphicon glyphicon-list-alt"></span>
-              [% l('Hold Subscriptions') %]
+              [% l('Hold Groups') %]
             </a>
           </li>
           <li class="divider"></li>
index 790dd79..f7ff64e 100644 (file)
@@ -12,7 +12,7 @@ angular.module('egUserBucketMod', ['egCoreMod'])
 
     var service = {
         allBuckets : [], // un-fleshed user buckets
-        allSubscriptions : [], // un-fleshed user buckets for hold subscriptions
+        allSubscriptions : [], // un-fleshed user buckets for hold groups
         barcodeString : '', // last scanned barcode
         barcodeRecords : [], // last scanned barcode results
         currentBucket : null, // currently viewed bucket
index fe125de..13277e6 100644 (file)
@@ -1,26 +1,26 @@
-Hold Subscriptions
-^^^^^^^^^^^^^^^^^^
+Hold Groups
+^^^^^^^^^^^
 
-This feature allows staff to add multiple users to a named hold subscription
+This feature allows staff to add multiple users to a named hold group
 bucket and place title-level holds for a record for that entire set of users.
-Users can be added to such a hold subscription bucket from either the patron
+Users can be added to such a hold group bucket from either the patron
 search result interface, via the Add to Bucket dropdown, or through a dedicated
-Hold Subscription interface available from the Circulation menu.  Adding new
-patrons to a subscription bucket will require staff have the PLACE_HOLD
+Hold Group interface available from the Circulation menu.  Adding new
+patrons to a hold group bucket will require staff have the PLACE_HOLD
 permission.
 
-Holds can be placed for the users in a subscription bucket either directly from
+Holds can be placed for the users in a hold group bucket either directly from
 the normal staff-place hold interface in the embedded OPAC, or by supplying the
-record ID within the subscription bucket interface.  In the latter case, the
+record ID within the hold group bucket interface.  In the latter case, the
 list of users for which a hold was attempted but failed to be placed can be
 downloaded by staff in order to address any placement issues.  Placing a
-subscription bucket hold will requires staff have the MANAGE_BATCH_HOLDS
+hold group bucket hold will requires staff have the MANAGE_HOLD_GROUPS
 permission, which is new with this development.
 
-In the event of a mistaken subscription hold, staff with the MANAGE_BATCH_HOLDS
+In the event of a mistaken hold group hold, staff with the MANAGE_HOLD_GROUPS
 permission will have the ability to cancel all unfulfilled holds created as
-part of a subscription hold event.
+part of a hold group event.
 
-A link to the title's hold interface is available from the list of subscription
-hold events in the dedicated subscription hold interface.
+A link to the title's hold interface is available from the list of hold group
+events in the dedicated hold group interface.