Attachment 83549 Details for Bug 22049 – Bug 22049: (follow-up) Remove leftover variable

[patch] Bug 22049: (follow-up) Remove leftover variable

Bug-22049-follow-up-Remove-leftover-variable.patch (text/plain), 1.67 KB, created by Tomás Cohen Arazi (tcohen) on 2018-12-31 11:33:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2018-12-31 11:33:54 UTC

Size: 1.67 KB

patch

obsolete

>From 2d975447c8e8fb85a5109ef61515e403ed7ac2c8 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Mon, 31 Dec 2018 08:28:03 -0300
>Subject: [PATCH] Bug 22049: (follow-up) Remove leftover variable
>
>The $circControlBranch variable was originally set to be used to pick
>the right dropbox branch. It was only used in MarkIssueReturned, to get
>the right Koha::Calendar object. As this responsability was moved top to
>the AddReturn caller, and the fact that _GetCircControlBranch is
>actually used for fines rules, there's no use for it in this context.
>And it was left on the previous patch as a mistake.
>
>To test:
>- Make sure the variable is not actually used:
>  $ git grep '$circControlBranch'
>=> SUCCESS: removed variable is not actually used.
>- Sign off :-D
>---
> C4/Circulation.pm | 6 ------
> 1 file changed, 6 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 3726652d54..ed0271e1a0 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1895,13 +1895,7 @@ sub AddReturn {
>         my $is_overdue;
>         die "The item is not issed and cannot be returned" unless $issue; # Just in case...
>         $patron or warn "AddReturn without current borrower";
>-		my $circControlBranch;
>         if ($dropbox) {
>-            # define circControlBranch only if dropbox mode is set
>-            # don't allow dropbox mode to create an invalid entry in issues (issuedate > today)
>-            # FIXME: check issuedate > returndate, factoring in holidays
>-
>-            $circControlBranch = _GetCircControlBranch($item,$patron_unblessed);
>             $is_overdue = $issue->is_overdue( $dropboxdate );
>         } else {
>             $is_overdue = $issue->is_overdue;
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 22049: 83542 | 83543 | 83544 | 83548 | 83549 | 83625 | 83626 | 83627 | 83654 | 83655 | 83656 | 83727 | 83728 | 83729 | 83730 | 85498