@@ -, +, @@ - Make sure the variable is not actually used: $ git grep '$circControlBranch' - Sign off :-D --- C4/Circulation.pm | 6 ------ 1 file changed, 6 deletions(-) --- a/C4/Circulation.pm +++ a/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; --