summaryrefslogtreecommitdiffstats
path: root/process_bug.cgi
diff options
context:
space:
mode:
authorDylan William Hardison <dylan@hardison.net>2016-11-07 21:25:06 +0100
committerDylan William Hardison <dylan@hardison.net>2016-11-07 21:25:19 +0100
commit35671f4ab3397f4c0d5ea96e0269d94c3d6bd2ac (patch)
tree019b79c2e4195081c7248ee7c3024aa2eadd465b /process_bug.cgi
parentb15c80df9de4b5d2b72639bbcf84aa4b3c208fa6 (diff)
downloadbugzilla-35671f4ab3397f4c0d5ea96e0269d94c3d6bd2ac.tar.gz
bugzilla-35671f4ab3397f4c0d5ea96e0269d94c3d6bd2ac.tar.xz
Bug 1313766 - Bugzilla::Bug->send_changes() should not output HTML directly
Diffstat (limited to 'process_bug.cgi')
-rwxr-xr-xprocess_bug.cgi87
1 files changed, 49 insertions, 38 deletions
diff --git a/process_bug.cgi b/process_bug.cgi
index 80018cfb3..0b1b30dbf 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -111,8 +111,6 @@ my $user_match_fields = {
Bugzilla::Hook::process('bug_user_match_fields', { fields => $user_match_fields });
Bugzilla::User::match_field($user_match_fields);
-print $cgi->header() unless Bugzilla->usage_mode == USAGE_MODE_EMAIL;
-
# Check for a mid-air collision. Currently this only works when updating
# an individual bug.
my $delta_ts = $cgi->param('delta_ts') || '';
@@ -158,6 +156,7 @@ if ($delta_ts) {
$cgi->param('token', issue_hash_token([$first_bug->id, $first_bug->delta_ts]));
# Warn the user about the mid-air collision and ask them what to do.
+ print $cgi->header() unless Bugzilla->usage_mode == USAGE_MODE_EMAIL;
$template->process("bug/process/midair.html.tmpl", $vars)
|| ThrowTemplateError($template->error());
exit;
@@ -186,6 +185,9 @@ $vars->{'title_tag'} = "bug_processed";
my $action;
if (defined $cgi->param('id')) {
$action = $user->setting('post_bug_submit_action');
+ if ($action ne 'nothing' && $action ne 'same_bug' && $action ne 'next_bug') {
+ ThrowCodeError("invalid_post_bug_submit_action");
+ }
if ($action eq 'next_bug') {
my $bug_list_obj = $user->recent_search_for($first_bug);
@@ -376,6 +378,7 @@ if (defined $cgi->param('id')) {
##############################
# Do Actual Database Updates #
##############################
+my $sent_changes;
foreach my $bug (@bug_objects) {
my $changes = $bug->update();
@@ -389,50 +392,58 @@ foreach my $bug (@bug_objects) {
}
}
- my $recipient_count = $bug->send_changes($changes, $vars);
+ $sent_changes = $bug->send_changes($changes);
}
# Delete the session token used for the mass-change.
delete_token($token) unless $cgi->param('id');
-if (Bugzilla->usage_mode == USAGE_MODE_EMAIL) {
- # Do nothing.
-}
-elsif ($action eq 'next_bug' or $action eq 'same_bug') {
- my $bug = $vars->{'bug'};
- if ($bug and $user->can_see_bug($bug)) {
- if ($action eq 'same_bug') {
- # $bug->update() does not update the internal structure of
- # the bug sufficiently to display the bug with the new values.
- # (That is, if we just passed in the old Bug object, we'd get
- # a lot of old values displayed.)
- $bug = new Bugzilla::Bug($bug->id);
- $vars->{'bug'} = $bug;
- }
- $vars->{'bugs'} = [$bug];
- if ($action eq 'next_bug') {
- $vars->{'nextbug'} = $bug->id;
- }
+# BMO: add show_bug_format hook for experimental UI work
+my $format_params = {
+ format => scalar $cgi->param('format'),
+ ctype => scalar $cgi->param('ctype'),
+};
+Bugzilla::Hook::process('show_bug_format', $format_params);
+my $format = $template->get_format("bug/show",
+ $format_params->{format},
+ $format_params->{ctype});
+
+if (Bugzilla->usage_mode != USAGE_MODE_EMAIL) {
+ print $cgi->header();
+
+ foreach my $sent_change (@$sent_changes) {
+ my $params = $sent_change->{params};
+ my $sent_bugmail = $sent_change->{sent_bugmail};
+ $vars->{$_} = $params->{$_} foreach keys %$params;
+ $vars->{'sent_bugmail'} = $sent_bugmail;
+ $template->process("bug/process/results.html.tmpl", $vars)
+ || ThrowTemplateError($template->error());
+ $vars->{'header_done'} = 1;
+ }
- # BMO: add show_bug_format hook for experimental UI work
- my $format_params = {
- format => scalar $cgi->param('format'),
- ctype => scalar $cgi->param('ctype'),
- };
- Bugzilla::Hook::process('show_bug_format', $format_params);
- my $format = $template->get_format("bug/show",
- $format_params->{format},
- $format_params->{ctype});
- $template->process($format->{template}, $vars)
- || ThrowTemplateError($template->error());
- exit;
+ if ($action eq 'next_bug' or $action eq 'same_bug') {
+ my $bug = $vars->{'bug'};
+ if ($bug and $user->can_see_bug($bug)) {
+ if ($action eq 'same_bug') {
+ # $bug->update() does not update the internal structure of
+ # the bug sufficiently to display the bug with the new values.
+ # (That is, if we just passed in the old Bug object, we'd get
+ # a lot of old values displayed.)
+ $bug = Bugzilla::Bug->new($bug->id);
+ $vars->{'bug'} = $bug;
+ }
+ $vars->{'bugs'} = [$bug];
+ if ($action eq 'next_bug') {
+ $vars->{'nextbug'} = $bug->id;
+ }
+
+ $template->process($format->{template}, $vars)
+ || ThrowTemplateError($template->error());
+ exit;
+ }
}
-} elsif ($action ne 'nothing') {
- ThrowCodeError("invalid_post_bug_submit_action");
-}
-# End the response page.
-unless (Bugzilla->usage_mode == USAGE_MODE_EMAIL) {
+ # End the response page.
$template->process("bug/navigate.html.tmpl", $vars)
|| ThrowTemplateError($template->error());
$template->process("global/footer.html.tmpl", $vars)