From 145e4f7a1e104416da3fe5866ae537e68e4e9480 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 9 Mar 2016 11:28:04 +0000 Subject: [PATCH] Bug 16013: Make classification sources deletable Because of a wrong variable scope, the classification sources are not deletable. Test plan: Create a classification sources (admin/classsources.pl) Try to delete it Without this patch, the classification source won't be deleted With this patch, it will work! Signed-off-by: Srdjan Signed-off-by: Katrin Fischer Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com (cherry picked from commit f6463317db08b9a4218ad94a2d191a5f5216c8c5) Signed-off-by: Julian Maurice --- admin/classsources.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/admin/classsources.pl b/admin/classsources.pl index 0b7ee31776..cc2fe87f7b 100755 --- a/admin/classsources.pl +++ b/admin/classsources.pl @@ -58,7 +58,7 @@ if ($op eq "add_source") { $rule_code); $display_lists = 1; } elsif ($op eq "delete_source") { - delete_class_source_form($template); + delete_class_source_form($template, $source_code); } elsif ($op eq "delete_source_confirmed") { delete_class_source($template, $source_code); $display_lists = 1; @@ -146,7 +146,7 @@ sub edit_class_source { sub delete_class_source_form { - my ($template) = @_; + my ($template, $source_code) = @_; $template->param( delete_class_source_form => 1, confirm_op => "delete_source_confirmed", -- 2.39.5