Bug 16667: Unused variable and function call in circulation.pl
authorTomas Cohen Arazi <tomascohen@theke.io>
Sun, 5 Jun 2016 11:46:06 +0000 (08:46 -0300)
committerJulian Maurice <julian.maurice@biblibre.com>
Thu, 16 Jun 2016 09:46:21 +0000 (11:46 +0200)
This patch removes an unused occurence of the $branches variable. Probably a
leftover from a recent rewrite.

To test:
- Run:
  $ git grep '$branches' circ/circulation.pl
=> FAIL: Only on occurence of the variable
- Apply the patch
- Run:
  $ git grep '$branches' circ/circulation.pl
=> SUCCESS: The variable has been removed
- Sign off :-D

Regards

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 2bff91eb4e733a5c658e17fb93f13497768ba2e6)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
(cherry picked from commit 9d8b407d7e0fc220268c4df3a0bebe4f508eae25)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

circ/circulation.pl

index c421085..63aa1fd 100755 (executable)
@@ -117,8 +117,6 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user (
     }
 );
 
-my $branches = GetBranches();
-
 my $force_allow_issue = $query->param('forceallow') || 0;
 if (!C4::Auth::haspermission( C4::Context->userenv->{id} , { circulate => 'force_checkout' } )) {
     $force_allow_issue = 0;