From 17df41d8db093577d9b380b44de1e62ca85164f9 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 Signed-off-by: Jonathan Druart --- C4/Circulation.pm | 7 ------- 1 file changed, 7 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index bb235b9df6..dfdbb1a5fa 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2008,7 +2008,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; } @@ -2040,12 +2039,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