From 62f617bcacaf58c04257cfd6ffd9f06caeb9d81d Mon Sep 17 00:00:00 2001 From: Chris Sharp Date: Mon, 29 Aug 2022 12:45:16 -0400 Subject: [PATCH] declare the variable once, then update the values Signed-off-by: Chris Sharp --- Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Ecard.pm | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Ecard.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Ecard.pm index 69db0767a3..a230855f02 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Ecard.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Ecard.pm @@ -20,14 +20,15 @@ my $U = 'OpenILS::Application::AppUtils'; my $self = shift; my $path = $self->apache->path_info; my $update_type = 'register'; +my @api_fields; if ($path =~ m|opac/erenew/submit|) { - my $update_type = 'renew'; + $update_type = 'renew'; } # Create different api_fields array if registration or renewal if ($update_type == 'renew') { - my @api_fields = ( + @api_fields = ( {name => 'vendor_username', required => 1}, {name => 'vendor_password', required => 1}, {name => 'patron_id', required => 1}, #needed for matching @@ -57,7 +58,7 @@ if ($update_type == 'renew') { {name => 'temp_renewal'} #new boolean value ); } else { - my @api_fields = ( + @api_fields = ( {name => 'vendor_username', required => 1}, {name => 'vendor_password', required => 1}, {name => 'first_given_name', class => 'au', required => 1}, -- 2.11.0