Attachment 168336 Details for Bug 37126 – Bug 37126: Add link to patron account when checking out to statistical patron ends checkout

[patch] Bug 37126: Add link to patron account when checking out to statistical patron ends checkout

Bug-37126-Add-link-to-patron-account-when-checking.patch (text/plain), 2.83 KB, created by David Nind on 2024-07-02 03:56:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2024-07-02 03:56:15 UTC

Size: 2.83 KB

patch

obsolete

>From 8c6f69c3ee91b30aefdaa0245d74aea4efbccace Mon Sep 17 00:00:00 2001
>From: Eric Garcia <cubingguy714@gmail.com>
>Date: Mon, 1 Jul 2024 17:53:30 +0000
>Subject: [PATCH] Bug 37126: Add link to patron account when checking out to
> statistical patron ends checkout
>
>When checking out an item to a statistical patron, where that item
>is already checked out to another patron, the patron details in the
>message do not link to that patron.
>
>This patch fixes this so that the patron details are now linked.
>
>This could be useful so that staff can more easily access the patron's
>details, and can, for example, check for any incorrect charges.
>
>Test plan:
>1. Set up a statistical patron for local use:[1]
>   1.1 Add a statistical patron category (Administration > Patrons and
>       circulation > Patron categories), for example:
>       - Category code: STAT
>       - Description: Statistical
>       - Enrollment period: 99
>       - Category type: Statistical
>   1.2 Add a new statistical paton (Patrons > +New patron > Statistical),
>       for example:
>       - Surname: Reference Statistical Patron
>       - Card number: refstat
>2. Check out an item to a standard patron.
>3. Check out the same item to the statistical patron created in step 1.2.
>4. Note that when the item is checked out to the statistical patron,
>   the patron name and number in the message are not linked:
>   "Item checked in from: PATRON_NAME (BORROWERNUMBER)".
>5. Apply the patch.
>6. Repeat steps 2 and 3.
>7. Note that the patron details in the message are now linked.
>
>[1] Monday Minutes: Statistical Patron and Local Use:
>    https://bywatersolutions.com/education/monday-minutes-statistical-patron-and-local-use
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>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 ff812fe5c1..fdba5cb170 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -473,7 +473,7 @@
>                                 [% IF ( STATS ) %]
>                                     <li>Local use recorded</li>
>                                     [% IF ( CHECKEDIN ) %]
>-                                        <li>Item checked in from: [% INCLUDE 'patron-title.inc' patron=BORROWER %]</li>
>+                                        <li>Item checked in from: <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% BORROWER.borrowernumber | uri %]">[% INCLUDE 'patron-title.inc' patron=BORROWER %]</a></li>
>                                     [% END %]
>                                     [% IF ( MESSAGES.ResFound ) %]
>                                         <li>Item on hold, please checkin.</li>
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 37126: 167878 | 168331 | 168336 | 168371