diff options
author | Dave Lawrence <dlawrence@mozilla.com> | 2013-01-09 21:08:58 +0100 |
---|---|---|
committer | Dave Lawrence <dlawrence@mozilla.com> | 2013-01-09 21:08:58 +0100 |
commit | 864425625db466dbbbc346e41d2acddbce101ecf (patch) | |
tree | eeba895bc138717b0028a9d81893c8d662df16c0 /extensions/BMO/Extension.pm | |
parent | f200da35c6283c71b7c29b2fc88d655b1e26db06 (diff) | |
download | bugzilla-864425625db466dbbbc346e41d2acddbce101ecf.tar.gz bugzilla-864425625db466dbbbc346e41d2acddbce101ecf.tar.xz |
- In bug_end_of_create, first check for the existence of a group before checking to
see if the bug is in the group. $bug->in_group expects and object to be passed.
(caused issues with automated testing)
Diffstat (limited to 'extensions/BMO/Extension.pm')
-rw-r--r-- | extensions/BMO/Extension.pm | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm index ca6133e82..c35e5688f 100644 --- a/extensions/BMO/Extension.pm +++ b/extensions/BMO/Extension.pm @@ -666,10 +666,11 @@ sub bug_end_of_create { my $bug = $args->{'bug'}; foreach my $group_name (keys %group_auto_cc) { - if ($bug->in_group(Bugzilla::Group->new({ name => $group_name }))) { + my $group_obj = Bugzilla::Group->new({ name => $group_name }); + if ($group_obj && $bug->in_group($group_obj)) { my $ra_logins = exists $group_auto_cc{$group_name}->{$bug->product} - ? $group_auto_cc{$group_name}->{$bug->product} - : $group_auto_cc{$group_name}->{'_default'}; + ? $group_auto_cc{$group_name}->{$bug->product} + : $group_auto_cc{$group_name}->{'_default'}; foreach my $login (@$ra_logins) { $bug->add_cc($login); } |