From 7205fd2c6b5882cb03ae461b037f73fbbc47bfa4 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 9 May 2019 15:53:56 +0100 Subject: [PATCH] Bug 22877: Prevent double overdue processing for lost and returned Content-Type: text/plain; charset=utf-8 Signed-off-by: Martin Renvoize Signed-off-by: Liz Rea Signed-off-by: Marcel de Rooy --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index e1b0ed6348..9dd1a23774 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -1960,7 +1960,7 @@ sub AddReturn { MarkIssueReturned( $borrowernumber, $item->itemnumber, $return_date, $patron->privacy ); }; unless ( $@ ) { - if ( C4::Context->preference('CalculateFinesOnReturn') && $is_overdue ) { + if ( C4::Context->preference('CalculateFinesOnReturn') && $is_overdue && !$item->itemlost ) { _CalculateAndUpdateFine( { issue => $issue, item => $item_unblessed, borrower => $patron_unblessed, return_date => $return_date } ); } } else { -- 2.11.0