From f703087eec36da5a957ae5bcd28f3a0e23fde770 Mon Sep 17 00:00:00 2001
From: Byron Jones <bjones@mozilla.com>
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(-)

(limited to 'extensions/BMO')

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