From 49da7cd462488e58572730da2ccd8dbf8aed839c Mon Sep 17 00:00:00 2001 From: =?utf8?q?Joonas=20Kylm=C3=A4l=C3=A4?= Date: Tue, 13 Apr 2021 11:22:16 +0300 Subject: [PATCH] Bug 28139: Handle Processing status with HoldsAutoFill pref turned on To test: 1: Turn on HoldsAutoFill setting 2: Place a item-level hold on item X at Branch A 3: Make the hold to Processing state: $ koha-mysql kohadev > select * from reserves; # look up the reserve_id > UPDATE reserves SET found = 'P', priority = 0 WHERE reserve_id = XXX; 4: Check in the item X at Branch A - A pop-up asking confirmation comes 5: Apply patch & restart plack 6: Check in the item X again at Branch A - now the hold is confirmed automatically Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens Signed-off-by: Jonathan Druart --- circ/returns.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/circ/returns.pl b/circ/returns.pl index 28c0b832f3..07e28f4c47 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -417,7 +417,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" || $reserve->{'ResFound'} eq "Transferred" ) && C4::Context->preference('HoldsAutoFill') ) { + if ( ( $reserve->{'ResFound'} eq "Reserved" || $reserve->{'ResFound'} eq "Processing" || $reserve->{'ResFound'} eq "Transferred" ) && C4::Context->preference('HoldsAutoFill') ) { my $item = Koha::Items->find( $itemnumber ); my $biblio = $item->biblio; -- 2.39.5