From 1b225de9d30ab15686145ff2db7b4e1f0109660f Mon Sep 17 00:00:00 2001 From: Stefan Schantl Date: Tue, 23 Mar 2021 17:38:26 +0100 Subject: [PATCH] ids.cgi: Correctly use "enabled" for checked checkboxes. Signed-off-by: Stefan Schantl --- html/cgi-bin/ids.cgi | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/html/cgi-bin/ids.cgi b/html/cgi-bin/ids.cgi index c57e0fd521..4047b91e94 100644 --- a/html/cgi-bin/ids.cgi +++ b/html/cgi-bin/ids.cgi @@ -675,7 +675,14 @@ if ($cgiparams{'RULESET'} eq $Lang::tr{'save'}) { # Assign some nice human-readable values. my $provider = $cgiparams{'PROVIDER'}; my $subscription_code = $cgiparams{'SUBSCRIPTION_CODE'}; - my $status_autoupdate = $cgiparams{'ENABLE_AUTOUPDATE'}; + my $status_autoupdate; + + # Handle autoupdate checkbox. + if ($cgiparams{'ENABLE_AUTOUPDATE'} eq "on") { + $status_autoupdate = "enabled"; + } else { + $status_autoupdate = "disabled"; + } # Check if we are going to add a new provider. if ($cgiparams{'PROVIDERS'} eq "$Lang::tr{'add'}") { @@ -1464,7 +1471,7 @@ END # Check if an existing provider should be edited. if($cgiparams{'PROVIDERS'} eq "$Lang::tr{'edit'}") { # Check if autoupdate is enabled for this provider. - if ($used_providers{$cgiparams{'ID'}}[2] eq "on") { + if ($used_providers{$cgiparams{'ID'}}[2] eq "enabled") { # Set the checkbox to be checked. $checked{'ENABLE_AUTOUPDATE'} = "checked='checked'"; } -- 2.39.5