Bug 344448: editparams.cgi doesn't let me clear a group - Patch by Frédéric Buclin...

diff --git a/Bugzilla/Config/Common.pm b/Bugzilla/Config/Common.pm
index 9fe648d01387102528fd84a0f9fb3b371f69d1cf..30b484468a1971291593f70e521f7ee03a105e5a 100644 (file)
--- a/Bugzilla/Config/Common.pm
+++ b/Bugzilla/Config/Common.pm
@@ -154,6 +154,7 @@ sub check_opsys {
 
 sub check_group {
     my $group_name = shift;
+    return "" unless $group_name;
     my $group = new Bugzilla::Group({'name' => $group_name});
     unless (defined $group) {
         return "Must be an existing group name";

diff --git a/Bugzilla/Config/GroupSecurity.pm b/Bugzilla/Config/GroupSecurity.pm
index 08b202cda6540ec01aa28f3224633f0d38b6c251..0235a8cb16613a22baa62de32f480096dddcd90c 100644 (file)
--- a/Bugzilla/Config/GroupSecurity.pm
+++ b/Bugzilla/Config/GroupSecurity.pm
@@ -94,6 +94,7 @@ sub get_param_list {
 
 sub _get_all_group_names {
     my @group_names = map {$_->name} Bugzilla::Group::get_all_groups();
+    unshift(@group_names, '');
     return \@group_names;
 }
 1;