Bug 13779: (QA follow-up) Correct checkauth call in offline_circ/download.pl
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 7 Sep 2018 07:50:40 +0000 (09:50 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 14 Sep 2018 17:27:29 +0000 (17:27 +0000)
Script offline_circ/download.pl does not use the return values of
checkauth. So should not assign them to bad chosen variables that do
not correspond with checkauth but with get_template_and_user.

Test plan:
Look for $template, $loggedinuser, $cookie, $flags in the code.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

offline_circ/download.pl

index 713057c..2b6c67e 100755 (executable)
@@ -27,8 +27,7 @@ use C4::Context;
 use C4::Koha;
 
 my $query = new CGI;
-my ( $template, $loggedinuser, $cookie, $flags ) =
-  checkauth( $query, undef, { circulate => "circulate_remaining_permissions" },
+checkauth( $query, undef, { circulate => "circulate_remaining_permissions" },
     "intranet" );
 
 my $page     = $query->param('page') || 0;