Attachment 118595 Details for Bug 27032 – Bug 27032: (follow-up) Pass rather than fetch variables

[patch] Bug 27032: (follow-up) Pass rather than fetch variables

Bug-27032-follow-up-Pass-rather-than-fetch-variabl.patch (text/plain), 1.69 KB, created by Nick Clemens (kidclamp) on 2021-03-22 13:30:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2021-03-22 13:30:38 UTC

Size: 1.69 KB

patch

obsolete

>From 83c0962b7f3f11c700954ef7b6304be912cedf2f Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Mon, 22 Mar 2021 13:29:13 +0000
>Subject: [PATCH] Bug 27032: (follow-up) Pass rather than fetch variables
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> C4/Circulation.pm | 18 +++++++++++-------
> 1 file changed, 11 insertions(+), 7 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 921c6bccec..2b7e8e3029 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -2791,7 +2791,12 @@ sub CanBookBeRenewed {
>             return ( 0, 'overdue');
>         }
> 
>-        $auto_renew = _CanBookBeAutoRenewed($borrowernumber, $itemnumber);
>+        $auto_renew = _CanBookBeAutoRenewed({
>+            patron     => $patron,
>+            item       => $item,
>+            branchcode => $branchcode,
>+            issue      => $issue
>+        });
>         return ( 0, $auto_renew  ) if $auto_renew =~ 'auto_account_expired';
>         return ( 0, $auto_renew  ) if $auto_renew =~ 'auto_too_late';
>         return ( 0, $auto_renew  ) if $auto_renew =~ 'auto_too_much_oweing';
>@@ -4271,12 +4276,11 @@ sub _CalculateAndUpdateFine {
> }
> 
> sub _CanBookBeAutoRenewed {
>-    my ( $borrowernumber, $itemnumber ) = @_;
>-
>-    my $item = Koha::Items->find($itemnumber);
>-    my $issue = $item->checkout;
>-    my $patron = $issue->patron;
>-    my $branchcode = _GetCircControlBranch( $item->unblessed, $patron->unblessed );
>+    my ( $params ) = @_;
>+    my $patron = $params->{patron};
>+    my $item = $params->{item};
>+    my $branchcode = $params->{branchcode};
>+    my $issue = $params->{issue};
> 
>     my $issuing_rule = Koha::CirculationRules->get_effective_rules(
>         {
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27032: 113666 | 113667 | 113668 | 118593 | 118594 | 118595 | 121492 | 121493 | 121494 | 124780 | 124781 | 124782