From f703087eec36da5a957ae5bcd28f3a0e23fde770 Mon Sep 17 00:00:00 2001 From: Byron Jones Date: Mon, 30 Apr 2012 22:17:20 +0800 Subject: Bug 750144: default sec-groups for products should always be filable --- extensions/BMO/Extension.pm | 2 +- extensions/BMO/lib/Data.pm | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm index e7f6554e4..ab0be0819 100644 --- a/extensions/BMO/Extension.pm +++ b/extensions/BMO/Extension.pm @@ -581,7 +581,7 @@ sub bug_check_groups { : [ map { trim($_) } split(',', $group_names) ]; foreach my $name (@$group_names) { - if ($always_fileable_group{$name}) { + if (exists $always_fileable_group{$name}) { my $group = new Bugzilla::Group({ name => $name }) or next; $add_groups->{$group->id} = $group; } diff --git a/extensions/BMO/lib/Data.pm b/extensions/BMO/lib/Data.pm index 929498e74..798162fbd 100644 --- a/extensions/BMO/lib/Data.pm +++ b/extensions/BMO/lib/Data.pm @@ -335,4 +335,7 @@ our %product_sec_groups = ( "_default" => 'core-security' ); +# Default security groups for products should always been fileable +map { $always_fileable_group{$_} = 1 } values %product_sec_groups; + 1; -- cgit v1.2.3-24-g4f1b