Attachment 155044 Details for Bug 34668 – Bug 34668: Add popup warn librarians of waiting holds when checking out

[patch] Bug 34668: Add popup warn librarians of waiting holds when checking out

Bug-34668-Add-popup-warn-librarians-of-waiting-hol.patch (text/plain), 7.73 KB, created by Shi Yao Wang on 2023-08-31 19:46:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Shi Yao Wang

Created: 2023-08-31 19:46:23 UTC

Size: 7.73 KB

patch

obsolete

>From b7c33e1ddf1fbb4bd922beae4cf39a0bad048044 Mon Sep 17 00:00:00 2001
>From: Shi Yao Wang <shi-yao.wang@inlibro.com>
>Date: Thu, 31 Aug 2023 13:31:16 -0400
>Subject: [PATCH] Bug 34668: Add popup warn librarians of waiting holds when
> checking out
>
>When there are holds waiting for patrons, sometimes the librarian misses
>the "Holds waiting here" display. This patch adds a modal popup warning
>when checking out an item for a patron with waiting holds.
>
>Test plan:
>1) find a user (user1)
>2) find a biblio (biblio1)
>3) add a hold for biblio1 to user1
>   (search biblio1 > Holds > find user1 > Place hold)
>4) checkout biblio1 to another user if not already checked out
>   and checkin through circulation page (not through the user page)
>   > confirm hold
>5) there should be a "Holds waiting here (1)" section added on user1 page
>
>6) checkout any items that isn't the one on hold for user1
>   -> notice it just checks out as normal
>7) apply patch and update database
>8) go to administration > system preferences > switch "WaitingNotifyAtCheckout" to "Notify"
>9) refresh page and redo step 6
>   -> notice there is now a modal to warn the librarian of a waiting hold
>   click Ok to proceed with the checkout, click outside the modal to not do anything
>---
> circ/circulation.pl                           |  6 ++++
> ...668-add_WaitingNotifyAtCheckout_syspref.pl | 16 +++++++++
> installer/data/mysql/mandatory/sysprefs.sql   |  1 +
> .../admin/preferences/circulation.pref        |  6 ++++
> .../prog/en/modules/circ/circulation.tt       | 34 ++++++++++++++++++-
> 5 files changed, 62 insertions(+), 1 deletion(-)
> create mode 100644 installer/data/mysql/atomicupdate/bug_34668-add_WaitingNotifyAtCheckout_syspref.pl
>
>diff --git a/circ/circulation.pl b/circ/circulation.pl
>index 9b925e29ad..1ed869a80e 100755
>--- a/circ/circulation.pl
>+++ b/circ/circulation.pl
>@@ -547,6 +547,12 @@ if ( $patron ) {
>     );
>     $template->param( patron_messages => $patron_messages );
> 
>+    if (C4::Context->preference("WaitingNotifyAtCheckout") ) {
>+        #Check for waiting holds
>+        my $waiting_holds = $patron->holds->search({ found => 'W', branchcode => $branch })->count;
>+        
>+        $template->param( waiting_holds => $waiting_holds );
>+    }
> }
> 
> my $fast_cataloging = 0;
>diff --git a/installer/data/mysql/atomicupdate/bug_34668-add_WaitingNotifyAtCheckout_syspref.pl b/installer/data/mysql/atomicupdate/bug_34668-add_WaitingNotifyAtCheckout_syspref.pl
>new file mode 100644
>index 0000000000..d7a8848ed3
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_34668-add_WaitingNotifyAtCheckout_syspref.pl
>@@ -0,0 +1,16 @@
>+use Modern::Perl;
>+
>+return {
>+    bug_number  => "34668",
>+    description => "Add syspref WaitingNotifyAtCheckout",
>+    up          => sub {
>+        my ($args) = @_;
>+        my ( $dbh, $out ) = @$args{qw(dbh out)};
>+
>+        # Do you stuffs here
>+        $dbh->do(q{INSERT IGNORE INTO `systempreferences` (variable,value,options,explanation,type) VALUES('WaitingNotifyAtCheckout','0',NULL,'If ON, notify librarians of waiting holds for the patron whose items they are checking out.','YesNo') });
>+
>+        say $out "Added new system preference 'WaitingNotifyAtCheckout'";
>+
>+    },
>+};
>diff --git a/installer/data/mysql/mandatory/sysprefs.sql b/installer/data/mysql/mandatory/sysprefs.sql
>index 30197591a8..a3eeae93c2 100644
>--- a/installer/data/mysql/mandatory/sysprefs.sql
>+++ b/installer/data/mysql/mandatory/sysprefs.sql
>@@ -793,6 +793,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
> ('viewMARC','1','','Allow display of MARC view of bibiographic records','YesNo'),
> ('virtualshelves','1','','If ON, enables Lists management','YesNo'),
> ('WaitingNotifyAtCheckin','0',NULL,'If ON, notify librarians of waiting holds for the patron whose items they are checking in.','YesNo'),
>+('WaitingNotifyAtCheckout','0',NULL,'If ON, notify librarians of waiting holds for the patron whose items they are checking out.','YesNo'),
> ('WebBasedSelfCheck','0',NULL,'If ON, enables the web-based self-check system','YesNo'),
> ('WhenLostChargeReplacementFee','1',NULL,'If ON, Charge the replacement price when a patron loses an item.','YesNo'),
> ('WhenLostForgiveFine','0',NULL,'If ON, Forgives the fines on an item when it is lost.','YesNo'),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>index 55feb4d444..89f7ce8798 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>@@ -97,6 +97,12 @@ Circulation:
>                   1: Notify
>                   0: "Don't notify"
>             - librarians of waiting holds for the patron whose items they are checking in.
>+        -
>+            - pref: WaitingNotifyAtCheckout
>+              choices:
>+                  1: Notify
>+                  0: "Don't notify"
>+            - librarians of waiting holds for the patron whose items they are checking out.
>         -
>             - pref: FilterBeforeOverdueReport
>               choices:
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>index 3388e387a5..a6cfd4c55b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -666,7 +666,11 @@
>                                             <a href="#" title="Checkout settings"><i class="fa-solid fa-sliders"></i></a>
>                                         </div>
> 
>-                                        <button type="submit" class="btn btn-primary">Check out</button>
>+                                        [% IF waiting_holds %]
>+                                            <button type="button" class="btn btn-primary" data-toggle="modal" data-target="#circ-warnwaitingholds-modal">Check out</button>
>+                                        [% ELSE %]
>+                                            <button type="submit" class="btn btn-primary">Check out</button>
>+                                        [% END %]
> 
>                                         <div class="circ-settings">
> 
>@@ -865,6 +869,28 @@
>     </div> 
> </div> 
> 
>+[% IF waiting_holds %]
>+    <div id="circ-warnwaitingholds-modal" class="modal fade audio-alert-action block">
>+        <div class="modal-dialog">
>+            <div class="modal-content">
>+                <div class="modal-header">
>+                    <h3>This patron has waiting holds</h3>
>+                </div>
>+                <div class="modal-body">
>+                    <ul>
>+                        <li>
>+                            This patron has waiting holds that are available for checkout
>+                        </li>
>+                    </ul>
>+                </div>
>+                <div class="modal-footer">
>+                    <button type="button" class="btn btn-primary" data-dismiss="modal">Ok</button>
>+                </div>
>+            </div>
>+        </div>
>+    </div>
>+[% END %]
>+
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'datatables.inc' %]
>     [% INCLUDE 'columns_settings.inc' %]
>@@ -931,6 +957,12 @@
>             var newin = window.open(link, 'popup', 'width=600,height=400,resizable=1,toolbar=0,scrollbars=1,top');
>         }
>         $(document).ready(function() {
>+            [% IF waiting_holds %]
>+                $('#circ-warnwaitingholds-modal .btn-primary').on('click',function() {
>+                    $('#mainform').submit();
>+                });
>+            [% END %]
>+
>             $('#mainform').on('submit',function() {
>                 if ($("#barcode") && $("#barcode").val()) {
>                     $('#barcode').on('keypress',function(event) {
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 34668: 155044 | 155045 | 155143 | 155188 | 156992 | 157243 | 157312 | 157315 | 157316 | 158359 | 158528 | 158529 | 158530 | 160673 | 160674 | 160675 | 160676