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>
(cherry picked from commit d7e754ea3c)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Kyle M Hall 2024-07-18 07:27:47 -04:00 committed by Fridolin Somers
parent 7525e88ca7
commit c9bef445a2
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

@ -177,8 +177,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,