From 0632bb7f7072cc8bd16b295314432456f546cc93 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 13 Aug 2020 18:42:56 +0200 Subject: [PATCH] Bug 18501: but.. what was the meaning of holdingbranch here? Signed-off-by: Martin Renvoize --- C4/Circulation.pm | 7 ------- 1 file changed, 7 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 450e4668fb..a07b6ecfe6 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2035,7 +2035,6 @@ sub AddReturn { # the holdingbranch is updated if the document is returned to another location. # this is always done regardless of whether the item was on loan or not - my $item_holding_branch = $item->holdingbranch; if ($item->holdingbranch ne $branch) { $item->holdingbranch($branch)->store; } @@ -2065,12 +2064,6 @@ sub AddReturn { if ( $item->itemlost ) { $messages->{'WasLost'} = 1; unless ( C4::Context->preference("BlockReturnOfLostItems") ) { - my $refunded = $item->set_found( - { - holdingbranch => $item_holding_branch, - } - ); - $messages->{'LostItemFeeRefunded'} = $refunded; } } -- 2.20.1