From: Stefan Schantl <stefan.schantl@ipfire.org>
Date: Tue, 23 Mar 2021 16:38:26 +0000 (+0100)
Subject: ids.cgi: Correctly use "enabled" for checked checkboxes.
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=39fcff9011110fdaa3ae9407a376ab8f44305113;p=people%2Fstevee%2Fipfire-2.x.git

ids.cgi: Correctly use "enabled" for checked checkboxes.

Signed-off-by: Stefan Schantl <stefan.schantl@ipfire.org>
---

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'";
 		}