Attachment 169128 Details for Bug 37400 – Bug 37400 - In returns.pl don't search for a patron unless needed

[patch] Bug 37400 - In returns.pl don't search for a patron unless needed

Bug-37400---In-returnspl-dont-search-for-a-patron-.patch (text/plain), 1.15 KB, created by Kyle M Hall (khall) on 2024-07-18 11:29:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2024-07-18 11:29:19 UTC

Size: 1.15 KB

patch

obsolete

>From 58d28d99efdc071049f99be5c551ab5e0adbd347 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Thu, 18 Jul 2024 07:27:47 -0400
>Subject: [PATCH] Bug 37400 - In returns.pl don't search for a patron unless
> needed
>
>In returns.pl we find a patron, then use it in an if statement.
>
>Test Plan:
>Cancel a hold with other holds remaining on the record.
>No change should be noted.
>---
> circ/returns.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/circ/returns.pl b/circ/returns.pl
>index 3fd20c65913..1e0dfc0e603 100755
>--- a/circ/returns.pl
>+++ b/circ/returns.pl
>@@ -173,8 +173,8 @@ if ( $query->param('reserve_id') ) {
>     # FIXME do we need to do this if we didn't take the cancel_reserve branch above?
>     my ( undef, $nextreservinfo, undef ) = CheckReserves($item);
> 
>-    my $patron = Koha::Patrons->find( $nextreservinfo->{'borrowernumber'} );
>     if ( $userenv_branch ne $nextreservinfo->{'branchcode'} ) {
>+        my $patron = Koha::Patrons->find( $nextreservinfo->{'borrowernumber'} );
>         $template->param(
>             itemtitle      => $biblio->title,
>             itembiblionumber => $biblio->biblionumber,
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 37400: 169128 | 169208