From da9456263fa06a2baecc6b6549ef1f4c8afdd9f5 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: wainuiwitikapark --- tags/review.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tags/review.pl b/tags/review.pl index 428fac9571..0f640f2e2c 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,7 +58,7 @@ 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 ($tag, $js_reply); + my $tag; my $status; if ($tag = $input->param('test')) { my $check = is_approved($tag);