Attachment 111034 Details for Bug 26232 – Bug 26232: Empty smart rule fine grace period

[patch] Bug 26232: Empty smart rule fine grace period

Bug-26232-Empty-smart-rule-fine-grace-period.patch (text/plain), 1.03 KB, created by Didier Gautheron on 2020-10-01 11:33:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Didier Gautheron

Created: 2020-10-01 11:33:02 UTC

Size: 1.03 KB

patch

obsolete

>From 71e53c4424e2b695644fe513c6658eecd7025f3a Mon Sep 17 00:00:00 2001
>From: Didier Gautheron <didier.gautheron@biblibre.com>
>Date: Tue, 18 Aug 2020 07:40:50 +0200
>Subject: [PATCH] Bug 26232: Empty smart rule fine grace period
>
>Staff can create smart rules with empty (undefined) fine grace period.
>In Overdues.pm undefined firstremind means 0, do the same in Circulation.pm.
>
>To test:
>1. Apply these patches
>2. Run:
>   kshell
>   prove t/db_dependent/Circulation.t
>---
> C4/Circulation.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 5c05326123..702399151c 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -2321,7 +2321,7 @@ sub _calculate_new_debar_dt {
> 
>     # grace period is measured in the same units as the loan
>     my $grace =
>-      DateTime::Duration->new( $unit => $issuing_rule->{firstremind} );
>+      DateTime::Duration->new( $unit => $issuing_rule->{firstremind} // 0);
> 
>     my $deltadays = DateTime::Duration->new(
>         days => $chargeable_units
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26232: 108432 | 108433 | 108434 | 111034 | 112403 | 112404 | 112427 | 112428