Attachment 119994 Details for Bug 27924 – Bug 27924: Display number of holds awaiting pickup on check out screens

[patch] Bug 27924: Display number of holds awaiting pickup on check out screens

Bug-27924-Display-number-of-holds-awaiting-pickup-.patch (text/plain), 8.23 KB, created by Owen Leonard on 2021-04-22 10:58:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2021-04-22 10:58:03 UTC

Size: 8.23 KB

patch

obsolete

>From 90173adfb8296b3f4335d66ffe04d1e891b85e78 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 21 Apr 2021 16:25:48 +0000
>Subject: [PATCH] Bug 27924: Display number of holds awaiting pickup on check
> out screens
>
>This patch modifies the checkout page so that waiting holds are
>displayed separately depending on whether they are waiting at the
>current branch or not. A count of each number of waiting holds is
>displayed too.
>
>Unrelated change: A missing </li> has been added for markup validity.
>
>To test, apply the patch and view the checkout screen in these
>situations:
>
> - A patron with no waiting holds.
> - A patron with one or more holds waiting at the current library.
> - A patron with one or more holds waiting at another library.
> - A patron with holds waiting at both the current library and at other
>   libraries.
>
>In each case, the display of waiting hold information should be correct,
>including the count of holds of each kind.
>---
> .../prog/en/modules/circ/circulation.tt            | 66 +++++++++++++++++-----
> 1 file changed, 51 insertions(+), 15 deletions(-)
>
>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 2078d9c77b..fb3abec220 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -830,7 +830,7 @@
>                                     [% SET ClaimReturnedWarningThreshold = Koha.Preference('ClaimReturnedWarningThreshold') %]
>                                     [% SET return_claims = patron.return_claims %]
>                                     [% IF return_claims.count > ClaimReturnedWarningThreshold %]
>-                                        <li><span class="circ-hlt return-claims">Return claims: Patron has [% return_claims.count | html %] RETURN CLAIMS.</span>
>+                                        <li><span class="circ-hlt return-claims">Return claims: Patron has [% return_claims.count | html %] RETURN CLAIMS.</span></li>
>                                     [% END %]
> 
> 
>@@ -869,24 +869,60 @@
> 
>                                 [% IF WaitingHolds.count %]
>                                     <div id="holdswaiting" class="circmessage">
>-                                        <h4>Holds waiting:</h4>
>+                                        [% SET waiting_here = 0 %]
>+                                        [% SET waiting_elsewhere = 0 %]
>                                         [% FOREACH w IN WaitingHolds %]
>+                                            [% IF ( w.branch.branchcode == Branches.GetLoggedInBranchcode()  ) %]
>+                                                [% waiting_here = waiting_here + 1 %]
>+                                            [% ELSE %]
>+                                                [% waiting_elsewhere = waiting_elsewhere + 1 %]
>+                                            [% END %]
>+                                        [% END %]
>+
>+                                        [% IF ( waiting_here > 0 ) %]
>+                                            <h4>Holds waiting here ([% waiting_here | html %])</h4>
>                                             <ul>
>-                                                <li>
>-                                                    <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% w.biblio.biblionumber | uri %]">[% w.biblio.title | html %]</a>
>-                                                    ([% ItemTypes.GetDescription( w.item.effective_itemtype ) | html %]),
>-                                                    [% IF ( w.biblio.author ) %] by [% w.biblio.author | html %] [% END %]
>-                                                    [% IF ( w.item.itemcallnumber ) %] [[% w.item.itemcallnumber | html %]] [% END %]
>-                                                    Hold placed on [% w.reservedate | $KohaDates %].
>-
>-                                                    <br/>
>-                                                    [% IF ( w.branch.branchcode == Branches.GetLoggedInBranchcode()  ) %]<strong class="waitinghere">[% ELSE %]<strong>[% END %]
>-                                                        [% SET expires_on = w.expirationdate %]
>-                                                        Waiting at [% w.branch.branchname | html %] [% IF expires_on %] until [% expires_on | $KohaDates %] [% END %]
>-                                                    </strong>
>-                                                </li>
>+                                                [% FOREACH w IN WaitingHolds %]
>+                                                    [% IF ( w.branch.branchcode == Branches.GetLoggedInBranchcode()  ) %]
>+                                                        <li>
>+                                                            <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% w.biblio.biblionumber | uri %]">[% w.biblio.title | html %]</a>
>+                                                            ([% ItemTypes.GetDescription( w.item.effective_itemtype ) | html %]),
>+                                                            [% IF ( w.biblio.author ) %] by [% w.biblio.author | html %] [% END %]
>+                                                            [% IF ( w.item.itemcallnumber ) %] [[% w.item.itemcallnumber | html %]] [% END %]
>+                                                            Hold placed on [% w.reservedate | $KohaDates %].
>+                                                            <br />
>+                                                            <strong class="waitinghere">
>+                                                                [% SET expires_on = w.expirationdate %]
>+                                                                Waiting here [% IF expires_on %] until [% expires_on | $KohaDates %] [% END %]
>+                                                            </strong>
>+                                                        </li>
>+                                                    [% END %]
>+                                                [% END %]
>                                             </ul>
>                                         [% END %]
>+
>+                                        [% IF ( waiting_elsewhere > 0 ) %]
>+                                            <h4>Holds waiting at other libraries ([% waiting_elsewhere | html %])</h4>
>+                                            <ul>
>+                                                [% FOREACH w IN WaitingHolds %]
>+                                                    [% IF ( w.branch.branchcode != Branches.GetLoggedInBranchcode()  ) %]
>+                                                        <li>
>+                                                            <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% w.biblio.biblionumber | uri %]">[% w.biblio.title | html %]</a>
>+                                                            ([% ItemTypes.GetDescription( w.item.effective_itemtype ) | html %]),
>+                                                            [% IF ( w.biblio.author ) %] by [% w.biblio.author | html %] [% END %]
>+                                                            [% IF ( w.item.itemcallnumber ) %] [[% w.item.itemcallnumber | html %]] [% END %]
>+                                                            Hold placed on [% w.reservedate | $KohaDates %].
>+                                                            <br />
>+                                                            <strong>
>+                                                                [% SET expires_on = w.expirationdate %]
>+                                                                Waiting at [% w.branch.branchname | html %] [% IF expires_on %] until [% expires_on | $KohaDates %] [% END %]
>+                                                            </strong>
>+                                                        </li>
>+                                                    [% END %]
>+                                                [% END %]
>+                                            </ul>
>+                                        [% END %]
>+
>                                     </div>
>                                 [% END # /IF WaitingHolds.count %]
> 
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27924: 118083 | 119962 | 119994 | 120006 | 120030