Attachment 96408 Details for Bug 24259 – Bug 24259: Handle non existing default circ rule in Charges::Fees

[patch] Bug 24259: Handle non existing default circ rule in Charges::Fees

Bug-24259-Handle-non-existing-default-circ-rule-in.patch (text/plain), 1.07 KB, created by Jonathan Druart on 2019-12-18 09:53:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2019-12-18 09:53:03 UTC

Size: 1.07 KB

patch

obsolete

>From decdc13e0d0f8b90e4fee45511070ee38e817957 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 18 Dec 2019 10:50:43 +0100
>Subject: [PATCH] Bug 24259: Handle non existing default circ rule in
> Charges::Fees
>
>Can't call method "lengthunit" on an undefined value at
>/usr/share/koha/lib/Koha/Charges/Fees.pm line 101.
>
>Test plan:
>1 - Remove all/all rule form circulation rules
>2 - Define a rental fee per day for an itemtype
>3 - Checkout an item of this type ot patron
>4 - Should get note of no circ rule defined
>5 - Confirm the checkout
>=> item is checked out
>---
> Koha/Charges/Fees.pm | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/Koha/Charges/Fees.pm b/Koha/Charges/Fees.pm
>index 42847983f4..9098a29cd0 100644
>--- a/Koha/Charges/Fees.pm
>+++ b/Koha/Charges/Fees.pm
>@@ -98,6 +98,8 @@ sub accumulate_rentalcharge {
>             branchcode   => $self->library->id
>         }
>     );
>+    return 0 unless $issuing_rule;
>+
>     my $units = $issuing_rule->lengthunit;
>     my $rentalcharge_increment =
>       ( $units eq 'days' )
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24259: 96407 | 96408 | 96488 | 96525