From 864425625db466dbbbc346e41d2acddbce101ecf Mon Sep 17 00:00:00 2001 From: Dave Lawrence Date: Wed, 9 Jan 2013 15:08:58 -0500 Subject: - 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) --- extensions/BMO/Extension.pm | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'extensions/BMO') 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); } -- cgit v1.2.3-24-g4f1b