Attachment 77427 Details for Bug 20660 – Bug 20660: Return date override affects debarments in AddReturn

[patch] Bug 20660: Return date override affects debarments in AddReturn

Bug-20660-Return-date-override-affects-debarments-.patch (text/plain), 1.08 KB, created by Josef Moravec on 2018-08-02 12:20:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2018-08-02 12:20:12 UTC

Size: 1.08 KB

patch

obsolete

>From f8293596a98ff517eef010f30f8bbdafddc7f015 Mon Sep 17 00:00:00 2001
>From: David Bourgault <dav.bour@gmail.com>
>Date: Tue, 1 May 2018 14:03:52 -0400
>Subject: [PATCH] Bug 20660: Return date override affects debarments in
> AddReturn
>
>Re-did the patch to follow best practices.
>
>TEST PLAN:
>0. Apply first patch only
>1. prove t/db_dependent/Circulation.t
>2. Apply second patch
>3. prove t/db_dependent/Circulation.t
>
>prove should fail the first time and pass the second time.
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>---
> C4/Circulation.pm | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 87812a3..a1f86b9 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1988,6 +1988,7 @@ sub AddReturn {
> 
>         if ( $issue and $issue->is_overdue ) {
>         # fix fine days
>+            $today = $return_date if $return_date;
>             $today = $dropboxdate if $dropbox;
>             my ($debardate,$reminder) = _debar_user_on_return( $patron_unblessed, $item, dt_from_string($issue->date_due), $today );
>             if ($reminder){
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 20660: 74857 | 74955 | 74956 | 77426 | 77427 | 77448 | 77449