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>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Eric Garcia 2024-07-01 17:53:30 +00:00 committed by Katrin Fischer
parent 49ecc5c797
commit 464f9296d4
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834

View file

@ -473,7 +473,7 @@
[% IF ( STATS ) %] [% IF ( STATS ) %]
<li>Local use recorded</li> <li>Local use recorded</li>
[% IF ( CHECKEDIN ) %] [% 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 %] [% END %]
[% IF ( MESSAGES.ResFound ) %] [% IF ( MESSAGES.ResFound ) %]
<li>Item on hold, please checkin.</li> <li>Item on hold, please checkin.</li>