Bugzilla – Attachment 162968 Details for
Bug 33174
Have better indication when one is cancelling multiple holds on a record
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 33174: Add warning about how many holds are being cancelled
Bug-33174-Add-warning-about-how-many-holds-are-bei.patch (text/plain), 2.99 KB, created by
Kyle M Hall (khall)
on 2024-03-08 14:58:01 UTC
(
hide
)
Description:
Bug 33174: Add warning about how many holds are being cancelled
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2024-03-08 14:58:01 UTC
Size:
2.99 KB
patch
obsolete
>From f9f11fc24a53e0e54bade861e15f3f067492854c Mon Sep 17 00:00:00 2001 >From: Lucas Gass <lucas@bywatersolutions.com> >Date: Wed, 8 Mar 2023 21:12:39 +0000 >Subject: [PATCH] Bug 33174: Add warning about how many holds are being > cancelled > >To test: >1. Apply patch >2. Find or make a record with many holds on it. >3. Go to the Holds tab ( request.pl ). >4. Select some hodls to cancel and press "Cancel selected" >5. A modal popup will appear. Notice the warning telling you exactly how many holds you are about to cancel. Confirm the count is accurate. >6. Try using the select all button and make sure the warning is accurate. >7. Make sure you can cancel the holds. > >Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt >index 1d668cde78f..864ac6f0b9f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt >@@ -1263,7 +1263,7 @@ > > <div class="modal-body"> > <p>Are you sure you want to cancel this hold?</p> >- >+ <div id="cancel_hold_alert" class="alert alert-danger"></div> > <fieldset class="action"> > [% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %] > [% IF hold_cancellation.count %] >@@ -1341,6 +1341,7 @@ > } > > var MSG_CANCEL_SELECTED = _("Cancel selected (%s)"); >+ var MSG_CANCEL_ALERT = _("This action will cancel <span class='badge'>%s</span> hold(s)."); > $.fn.select2.defaults.set("width", "100%" ); > $.fn.select2.defaults.set("dropdownAutoWidth", true ); > >@@ -1709,6 +1710,7 @@ > $('.select_hold', table).prop('checked', !count); > $(this).prop('checked', !count); > $('.cancel_selected_holds').html(MSG_CANCEL_SELECTED.format($('.holds_table .select_hold:checked').length)); >+ $('#cancel_hold_alert').html( MSG_CANCEL_ALERT.format($('.holds_table .select_hold:checked').length)); > localStorage.selectedHolds = $('.holds_table .select_hold:checked').toArray().map(el => $(el).data('id')); > }); > >@@ -1717,6 +1719,7 @@ > var count = $('.select_hold:not(:checked)', table).length; > $('.select_hold_all', table).prop('checked', !count); > $('.cancel_selected_holds').html(MSG_CANCEL_SELECTED.format($('.holds_table .select_hold:checked').length)); >+ $('#cancel_hold_alert').html( MSG_CANCEL_ALERT.format($('.holds_table .select_hold:checked').length)); > localStorage.selectedHolds = $('.holds_table .select_hold:checked').toArray().map(el => $(el).data('id')); > }); > >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 33174
:
147985
|
147989
| 162968