From 10af741387a75d29c6b7782c6ce3d2c07cad9517 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 21 Feb 2020 12:37:44 +0100 Subject: [PATCH] Bug 24413: Do not remove the restrictions from AddReturn As we are now removing them from MarkIssueReturned they should not be removed from AddReturn as well. Also I think this will fix a regression, if $doreturn is not set (in case the item is withdrawn and BlockReturnOfWithdrawnItems or the item is lost and BlockReturnOfLostItems, and other specific cases). Signed-off-by: Victor Grousset/tuxayo Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize --- C4/Circulation.pm | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 7b6d3d64e5..7a3b21b903 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2126,16 +2126,6 @@ sub AddReturn { if C4::Context->preference("ReturnLog"); } - # Remove any OVERDUES related debarment if the borrower has no overdues - if ( $borrowernumber - && $patron->debarred - && C4::Context->preference('AutoRemoveOverduesRestrictions') - && !Koha::Patrons->find( $borrowernumber )->has_overdues - && @{ GetDebarments({ borrowernumber => $borrowernumber, type => 'OVERDUES' }) } - ) { - DelUniqueDebarment({ borrowernumber => $borrowernumber, type => 'OVERDUES' }); - } - # Check if this item belongs to a biblio record that is attached to an # ILL request, if it is we need to update the ILL request's status if (C4::Context->preference('CirculateILL')) { -- 2.39.5