From a59f2d06a1eb6056373023c1605ac1ecc02f92ed Mon Sep 17 00:00:00 2001 From: Stefan Schantl Date: Wed, 14 Apr 2021 20:19:27 +0200 Subject: [PATCH] ids.cgi: Use newly intruduced functions when removing a provider. Signed-off-by: Stefan Schantl --- html/cgi-bin/ids.cgi | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/html/cgi-bin/ids.cgi b/html/cgi-bin/ids.cgi index b3a7d9f522..e728202040 100644 --- a/html/cgi-bin/ids.cgi +++ b/html/cgi-bin/ids.cgi @@ -937,23 +937,18 @@ if ($cgiparams{'RULESET'} eq $Lang::tr{'ids apply'}) { ## Remove provider from the list of used providers. # } elsif ($cgiparams{'PROVIDERS'} eq $Lang::tr{'remove'}) { - my %used_providers = (); + # Assign a nice human-readable variable. + my $id = $cgiparams{'ID'}; - # Read-in provider settings file. - &General::readhasharray($IDS::providers_settings_file, \%used_providers); + # Grab the provider name bevore deleting. + my $provider = &get_provider_handle($id); - # Grab the provider name bevore deleting it from hash. - my $provider = $used_providers{$cgiparams{'ID'}}[0]; - - # Drop entry from the hash. - delete($used_providers{$cgiparams{'ID'}}); + # Remove the provider. + &remove_provider($id); # Undef the given ID. undef($cgiparams{'ID'}); - # Write the changed hash to the provide settings file. - &General::writehasharray($IDS::providers_settings_file, \%used_providers); - # Lock the webpage and print message. &working_notice("$Lang::tr{'ids apply ruleset changes'}"); -- 2.39.5