From 2c8e612f1e58411e20b966ae7714b1bf619571b8 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Mon, 12 Apr 2021 14:18:20 +0000 Subject: [PATCH] Bug 28136: Handle Transferred status for ResFound MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit bug 25690 added a new 'Transferred' status to 'ResFound', this status needs to be handled in circ/returns.pl To test: 1 - Place a hold on an item at Branch B for pickup at Branch A 2 - Check in the item at Branch B - confirm hold and transfer 3 - Check in the item at Branch A - nothing happens? 4 - Apply patch 5 - Checkin in the item at Branch A - hold popup appears 6 - Clear the hold and place it again 7 - Set system preference 'HoldsAutoFill' to do 8 - Check in the item at Branch B - hold is found and confirmed 9 - Check in the item at Branch A - hold is found and confirmed Signed-off-by: Joonas Kylmälä Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart (cherry picked from commit 9d7f55a8748c5e148d805195e314f189c886a492) Signed-off-by: Fridolin Somers --- circ/returns.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/circ/returns.pl b/circ/returns.pl index d9359a0b66..1a83264817 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -415,7 +415,7 @@ if ( $messages->{'ResFound'}) { my $patron = Koha::Patrons->find( $reserve->{borrowernumber} ); my $holdmsgpreferences = C4::Members::Messaging::GetMessagingPreferences( { borrowernumber => $reserve->{'borrowernumber'}, message_name => 'Hold_Filled' } ); my $branchCheck = ( $userenv_branch eq $reserve->{branchcode} ); - if ( $reserve->{'ResFound'} eq "Reserved" && C4::Context->preference('HoldsAutoFill') ) { + if ( ( $reserve->{'ResFound'} eq "Reserved" || $reserve->{'ResFound'} eq "Transferred" ) && C4::Context->preference('HoldsAutoFill') ) { my $item = Koha::Items->find( $itemnumber ); my $biblio = $item->biblio; @@ -441,7 +441,7 @@ if ( $messages->{'ResFound'}) { $template->param( waiting => $branchCheck ? 1 : undef, ); - } elsif ( $reserve->{'ResFound'} eq "Reserved" || $reserve->{'ResFound'} eq "Processing" ) { + } elsif ( $reserve->{'ResFound'} eq "Reserved" || $reserve->{'ResFound'} eq "Processing" || $reserve->{'ResFound'} eq "Transferred" ) { $template->param( intransit => $branchCheck ? undef : 1, transfertodo => $branchCheck ? undef : 1, -- 2.39.5