From 910be2636cd3a7da3f916c4ac7f20c1c17bd6cc5 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Thu, 6 Jan 2022 12:38:05 +0000 Subject: [PATCH] Bug 29808: Do not receive transfer for checked out items This patch adds a check to StockRotationItem->advance to check if an item is checked out from the destination branch. If so, we do not receive the transfer To test: 1 - Enable StockRotation system preference 2 - Browse to Tools->Stock rotation->New Rota and create a new rota, set as cyclical 3 - Manage->stages and add two stages, each with 0 day duration 4 - Manage->items - add an item 5 - Checkout the item from the first branch of rotation 6 - Tools->Stock rotation->Manage->items->move to next stage 7 - Error: [Transfer item is currently checked out] at /usr/share/perl5/Exception/Class/Base.pm line 88 8 - Check in the item at first branch 9 - Checkout the item from the second stage branch 10 - Apply patch 11 - Tools->Stock rotation->Manage->items->move to next stage 12 - No error! 13 - Check in item at second branch 14 - Checkout from first branch 15 - Tools->Stock rotation->Manage->items->move to next stage 16 - Still no error! Signed-off-by: Martin Renvoize Signed-off-by: Fridolin Somers Signed-off-by: Kyle M Hall (cherry picked from commit 92a15ba56d21bf01cc0b65620465f86fe522e3d5) Signed-off-by: Andrew Fuerste-Henry --- Koha/StockRotationItem.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Koha/StockRotationItem.pm b/Koha/StockRotationItem.pm index 9ee92c10e6..8ed624240f 100644 --- a/Koha/StockRotationItem.pm +++ b/Koha/StockRotationItem.pm @@ -272,7 +272,9 @@ sub advance { } }; $transfer->receive - if $item->holdingbranch eq $new_stage->branchcode_id; # Already at branch + if $item->holdingbranch eq $new_stage->branchcode_id && !$item->checkout; + # If item is already at branch, and not checked out + # If item is checked out, the return will either receive or initiate the transfer return $transfer; } -- 2.39.5