From bab4bb9adb4db0b20ef94771ef78ecf654497788 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 18 May 2020 17:50:11 +0200 Subject: [PATCH] Bug 25531: Debar patron if needed when checkin is backdated If think this case does not apply to real-life, but the logic needs to be fixed. If an item is due now, and AddReturn is called now with a return date in the future, the issue is overdue and the patron must be debarred. However it is not as we compare with now and not the return date Signed-off-by: Victor Grousset/tuxayo Signed-off-by: Martin Renvoize Signed-off-by: Joy Nelson (cherry picked from commit 0c238db2f5763db48fe3d451b599ffd6b0e42869) Signed-off-by: Victor Grousset/tuxayo --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 727fbf890b..c91d05643a 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2036,7 +2036,7 @@ sub AddReturn { my $fix = _FixOverduesOnReturn( $borrowernumber, $item->itemnumber, $exemptfine ); defined($fix) or warn "_FixOverduesOnReturn($borrowernumber, $item->itemnumber...) failed!"; # zero is OK, check defined - if ( $issue and $issue->is_overdue ) { + if ( $issue and $issue->is_overdue($return_date) ) { # fix fine days my ($debardate,$reminder) = _debar_user_on_return( $patron_unblessed, $item_unblessed, dt_from_string($issue->date_due), $return_date ); if ($reminder){ -- 2.20.1