From 5199bb74dcdef65b78a8e740a93ddd3c3ea9766a Mon Sep 17 00:00:00 2001 From: Josef Moravec Date: Tue, 22 Aug 2017 08:58:11 +0000 Subject: [PATCH] Bug 19116: Hold not set to waiting after transfer Test plan: 0) Do not apply the patch 1) Place hold on item from another branch 2) Switch to that branch 3) Check them in at the other branch to set them into transport status (T) 4) Switch back to your homebranch 5) Check items in again, use the different confirm buttons and compare: Only "confirm and print" will be set to waiting, "confirm" remains in transport. 6) Apply the patch 7) Repeat 1-5 - now should work as expected - the hold is marked waiting on "confirm" button too 8) Check the hold from the same branch, to make sure this doesn't add regression Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi Signed-off-by: Jonathan Druart (cherry picked from commit 80fd5cd0dbfe8c5612325ca65251d36a97fcfc1c) Signed-off-by: Fridolin Somers (cherry picked from commit 2a0dc0f1d28ad03dd48326caff120dfe3b3daacd) Signed-off-by: Katrin Fischer --- C4/Reserves.pm | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 3e450b4927..70f980a2e7 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -919,7 +919,11 @@ sub CheckReserves { my $priority = 10000000; foreach my $res (@reserves) { if ( $res->{'itemnumber'} == $itemnumber && $res->{'priority'} == 0) { - return ( "Waiting", $res, \@reserves ); # Found it + if ($res->{'found'} eq 'W') { + return ( "Waiting", $res, \@reserves ); # Found it, it is waiting + } else { + return ( "Reserved", $res, \@reserves ); # Found determinated hold, e. g. the tranferred one + } } else { my $borrowerinfo; my $iteminfo; -- 2.39.5