From f4e21f4199dfa99d202de893897b3971bbe7aea6 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 17 Apr 2026 09:18:54 +0200 Subject: [PATCH] Bug 42254: (QA follow-up) Remove unused var, fixing a FIXME Test plan: Read the patch. Look for the var in the script. Signed-off-by: Marcel de Rooy Signed-off-by: Lucas Gass --- tags/review.pl | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/tags/review.pl b/tags/review.pl index 2e754ca1f7..2f67d6baf8 100755 --- a/tags/review.pl +++ b/tags/review.pl @@ -46,7 +46,7 @@ sub ajax_auth_cgi { # returns CGI object my ($auth_status) = check_cookie_auth( $sessid, $needed_flags ); if ( $auth_status ne "ok" ) { - #FIMXE: This should return a HTTP error and not a script + #FIXME: This should return a HTTP error and not a script output_with_http_headers $input, undef, "window.alert('Your CGI session cookie ($sessid) is not current. " . "Please refresh the page and try again.');\n", 'js'; @@ -58,9 +58,8 @@ sub ajax_auth_cgi { # returns CGI object if ( is_ajax() ) { my $input = &ajax_auth_cgi($needed_flags); my $operator = C4::Context->userenv->{'number'}; # must occur AFTER auth - my $js_reply; - my $op = $input->param('op') || q{}; - my $tag = $input->param('tag'); + my $op = $input->param('op') || q{}; + my $tag = $input->param('tag'); my $status; if ( $op eq 'test' ) { my $check = is_approved($tag);