gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 82535a4887)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Kyle M Hall 2024-07-18 07:27:47 -04:00 committed by Lucas Gass
parent c20fede430
commit d7e754ea3c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
circ/returns.pl
View File

@ -179,8 +179,8 @@ if ( $query->param('reserve_id') && $op eq 'cud-affect_reserve') {
# 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,