diff options
author | David Lawrence <dkl@mozilla.com> | 2015-05-05 05:07:39 +0200 |
---|---|---|
committer | Byron Jones <glob@mozilla.com> | 2015-05-05 05:07:39 +0200 |
commit | 99fc3b3c3db2e1ffa00a94d5e88cefc29f98c49b (patch) | |
tree | b55c77ae4e6681f2fa362cd5bac06f909df57522 | |
parent | 5ff4a99bb9f9547f92a53b87489237464d7b4962 (diff) | |
download | bugzilla-99fc3b3c3db2e1ffa00a94d5e88cefc29f98c49b.tar.gz bugzilla-99fc3b3c3db2e1ffa00a94d5e88cefc29f98c49b.tar.xz |
Bug 1159282: form.dev-engagement-event: stop creating a second "discussion" bug
-rw-r--r-- | extensions/BMO/Extension.pm | 74 |
1 files changed, 0 insertions, 74 deletions
diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm index c362d37f7..76c73afb3 100644 --- a/extensions/BMO/Extension.pm +++ b/extensions/BMO/Extension.pm @@ -1658,8 +1658,6 @@ sub _post_dev_engagement { my $error_mode_cache = Bugzilla->error_mode; Bugzilla->error_mode(ERROR_MODE_DIE); - my $discussion_bug; - my @warnings; eval { # Add attachment containing tab delimited field values for # spreadsheet import. @@ -1721,81 +1719,9 @@ EOF filename => 'dev_engagement_submission.html', mimetype => 'text/html', }); - - # File discussion bug - Bugzilla->set_user(Bugzilla::User->new({ name => 'nobody@mozilla.org' })); - my $new_user = Bugzilla->user; - - # HACK: User needs to be in the editbugs and primary bug's group to allow - # setting of dependencies. - $new_user->{'groups'} = [ - Bugzilla::Group->new({ name => 'editbugs' }), - Bugzilla::Group->new({ name => 'mozilla-employee-confidential' }) - ]; - - my $recipients = { changer => $new_user }; - $vars->{original_reporter} = $old_user; - - $discussion_bug = Bugzilla::Bug->create({ - short_desc => '[discussion] ' . $parent_bug->short_desc, - product => 'Developer Engagement', - component => 'Events Request Discussion', - keywords => ['event-discussion-needs-review'], - bug_severity => 'normal', - groups => ['mozilla-employee-confidential'], - comment => 'This is the discussion for the request ' . - 'described in bug ' . $parent_bug->id . '.', - op_sys => 'All', - rep_platform => 'All', - version => 'unspecified' - }); - - # Set the needinfo flag on the discussion bug - my @new_flags; - foreach my $type (@{ $discussion_bug->flag_types }) { - next if $type->name ne 'needinfo'; - foreach my $requestee (DEV_ENGAGE_DISCUSS_NEEDINFO()) { - # needinfo'ing a disable account throws an error - warn instead - my $requestee_object = Bugzilla::User->new({ name => $requestee, cache => 1 }); - if (!$requestee_object || !$requestee_object->is_enabled) { - push @warnings, "Failed to needinfo $requestee on dev-engagement bug (does not exist or disabled)"; - next; - } - my $needinfo_flag = { - type_id => $type->id, - status => '?', - requestee => $requestee, - }; - push(@new_flags, $needinfo_flag); - } - } - $discussion_bug->set_flags(\@new_flags, []) if @new_flags; - $discussion_bug->update($discussion_bug->creation_ts); - Bugzilla::BugMail::Send($discussion_bug->id, $recipients); - - # Add discussion comment to the parent bug pointing to new bug - # and dependency link - $parent_bug->set_all({ dependson => { add => [ $discussion_bug->id ] } }); - $parent_bug->add_comment('This request is being discussed in bug ' . - $discussion_bug->id); }; - my $error = $@; - - Bugzilla->set_user($old_user); - Bugzilla->error_mode($error_mode_cache); - - foreach my $warning (@warnings) { - warn $warning . "\n"; - } - # No matter what happened, ensure the parent bug gets marked as updated - # There's no need to send mail for parent bug $parent_bug->update($parent_bug->creation_ts); - - if ($error || !$discussion_bug) { - warn "Failed to create additional dev-engagement bug: $error\n" if $error; - $vars->{'message'} = 'dev_engagement_creation_failed'; - } } sub _add_attachment { |