process_bug.cgi 14.6 KB
Newer Older
1
#!/usr/bin/perl -wT
2 3 4
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
terry%netscape.com's avatar
terry%netscape.com committed
5
#
6 7
# This Source Code Form is "Incompatible With Secondary Licenses", as
# defined by the Mozilla Public License, v. 2.0.
terry%netscape.com's avatar
terry%netscape.com committed
8

9 10 11 12 13 14 15 16 17 18 19 20
# Implementation notes for this file:
#
# 1) the 'id' form parameter is validated early on, and if it is not a valid
# bugid an error will be reported, so it is OK for later code to simply check
# for a defined form 'id' value, and it can assume a valid bugid.
#
# 2) If the 'id' form parameter is not defined (after the initial validation),
# then we are processing multiple bugs, and @idlist will contain the ids.
#
# 3) If we are processing just the one id, then it is stored in @idlist for
# later processing.

21
use 5.10.1;
22
use strict;
23
use lib qw(. lib);
24

25
use Bugzilla;
26
use Bugzilla::Constants;
27
use Bugzilla::Bug;
28
use Bugzilla::User;
29
use Bugzilla::Util;
30
use Bugzilla::Error;
31
use Bugzilla::Flag;
32
use Bugzilla::Status;
33
use Bugzilla::Token;
34

35
use List::MoreUtils qw(firstidx);
36 37
use Storable qw(dclone);

38
my $user = Bugzilla->login(LOGIN_REQUIRED);
39

40
my $cgi = Bugzilla->cgi;
41
my $dbh = Bugzilla->dbh;
42
my $template = Bugzilla->template;
43
my $vars = {};
44

45 46 47 48
######################################################################
# Subroutines
######################################################################

49
# Tells us whether or not a field should be changed by process_bug.
50
sub should_set {
51
    # check_defined is used for fields where there's another field
52 53 54 55
    # whose name starts with "defined_" and then the field name--it's used
    # to know when we did things like empty a multi-select or deselect
    # a checkbox.
    my ($field, $check_defined) = @_;
56
    my $cgi = Bugzilla->cgi;
57 58
    if ( defined $cgi->param($field) 
         || ($check_defined && defined $cgi->param("defined_$field")) )
59 60 61 62 63 64
    {
        return 1;
    }
    return 0;
}

65 66 67 68
######################################################################
# Begin Data/Security Validation
######################################################################

69 70
# Create a list of objects for all bugs being modified in this request.
my @bug_objects;
71
if (defined $cgi->param('id')) {
72
  my $bug = Bugzilla::Bug->check_for_edit(scalar $cgi->param('id'));
73 74
  $cgi->param('id', $bug->id);
  push(@bug_objects, $bug);
75
} else {
76
    foreach my $i ($cgi->param()) {
77
        if ($i =~ /^id_([1-9][0-9]*)/) {
78
            my $id = $1;
79
            push(@bug_objects, Bugzilla::Bug->check_for_edit($id));
80
        }
81 82 83
    }
}

84
# Make sure there are bugs to process.
85
scalar(@bug_objects) || ThrowUserError("no_bugs_chosen", {action => 'modify'});
86

87
my $first_bug = $bug_objects[0]; # Used when we're only updating a single bug.
88

89 90 91 92
# Delete any parameter set to 'dontchange'.
if (defined $cgi->param('dontchange')) {
    foreach my $name ($cgi->param) {
        next if $name eq 'dontchange'; # But don't delete dontchange itself!
93 94
        # Skip ones we've already deleted (such as "defined_$name").
        next if !defined $cgi->param($name);
95 96
        if ($cgi->param($name) eq $cgi->param('dontchange')) {
            $cgi->delete($name);
97
            $cgi->delete("defined_$name");
98 99
        }
    }
100 101
}

102
# do a match on the fields if applicable
103
Bugzilla::User::match_field({
104 105 106 107 108
    'qa_contact'                => { 'type' => 'single' },
    'newcc'                     => { 'type' => 'multi'  },
    'masscc'                    => { 'type' => 'multi'  },
    'assigned_to'               => { 'type' => 'single' },
});
109

110
print $cgi->header() unless Bugzilla->usage_mode == USAGE_MODE_EMAIL;
111

112 113
# Check for a mid-air collision. Currently this only works when updating
# an individual bug.
114
my $delta_ts = $cgi->param('delta_ts') || '';
115 116 117 118 119

if ($delta_ts) {
    my $delta_ts_z = datetime_from($delta_ts)
      or ThrowCodeError('invalid_timestamp', { timestamp => $delta_ts });

120 121
    my $first_delta_tz_z =  datetime_from($first_bug->delta_ts);

122 123
    if ($first_delta_tz_z ne $delta_ts_z) {
        ($vars->{'operations'}) = $first_bug->get_activity(undef, $delta_ts);
124 125 126

        # Always sort midair collision comments oldest to newest,
        # regardless of the user's personal preference.
127 128
        my $comments = $first_bug->comments({ order => 'oldest_to_newest',
                                              after => $delta_ts });
129

130 131
        # Show midair if previous changes made other than CC
        # and/or one or more comments were made
132
        my $do_midair = scalar @$comments ? 1 : 0;
133 134 135 136

        if (!$do_midair) {
            foreach my $operation (@{ $vars->{'operations'} }) {
                foreach my $change (@{ $operation->{'changes'} }) {
137 138 139 140
                    if ($change->{'fieldname'} ne 'cc') {
                        $do_midair = 1;
                        last;
                    }
141 142 143 144 145 146 147 148 149
                }
                last if $do_midair;
            }
        }

        if ($do_midair) {
            $vars->{'title_tag'} = "mid_air";
            $vars->{'comments'} = $comments;
            $vars->{'bug'} = $first_bug;
150 151
            # The token contains the old delta_ts. We need a new one.
            $cgi->param('token', issue_hash_token([$first_bug->id, $first_bug->delta_ts]));
152 153 154 155 156 157

            # Warn the user about the mid-air collision and ask them what to do.
            $template->process("bug/process/midair.html.tmpl", $vars)
                || ThrowTemplateError($template->error());
            exit;
        }
158
    }
159
}
160

161 162 163 164 165 166
# We couldn't do this check earlier as we first had to validate bug IDs
# and display the mid-air collision page if delta_ts changed.
# If we do a mass-change, we use session tokens.
my $token = $cgi->param('token');

if ($cgi->param('id')) {
167
    check_hash_token($token, [$first_bug->id, $delta_ts || $first_bug->delta_ts]);
168 169 170 171 172 173 174 175 176
}
else {
    check_token_data($token, 'buglist_mass_change', 'query.cgi');
}

######################################################################
# End Data/Security Validation
######################################################################

177 178
$vars->{'title_tag'} = "bug_processed";

179
my $action;
180
if (defined $cgi->param('id')) {
181
    $action = $user->setting('post_bug_submit_action');
182 183

    if ($action eq 'next_bug') {
184 185
        my $bug_list_obj = $user->recent_search_for($first_bug);
        my @bug_list = $bug_list_obj ? @{$bug_list_obj->bug_list} : ();
186
        my $cur = firstidx { $_ eq $cgi->param('id') } @bug_list;
187
        if ($cur >= 0 && $cur < $#bug_list) {
188 189 190
            my $next_bug_id = $bug_list[$cur + 1];
            detaint_natural($next_bug_id);
            if ($next_bug_id and $user->can_see_bug($next_bug_id)) {
191
                # We create an object here so that $bug->send_changes can use it
192 193 194
                # when displaying the header.
                $vars->{'bug'} = new Bugzilla::Bug($next_bug_id);
            }
195 196 197 198
        }
    }
    # Include both action = 'same_bug' and 'nothing'.
    else {
199
        $vars->{'bug'} = $first_bug;
200 201 202 203 204 205 206
    }
}
else {
    # param('id') is not defined when changing multiple bugs at once.
    $action = 'nothing';
}

207 208 209
# Component, target_milestone, and version are in here just in case
# the 'product' field wasn't defined in the CGI. It doesn't hurt to set
# them twice.
210 211 212
my @set_fields = qw(op_sys rep_platform priority bug_severity
                    component target_milestone version
                    bug_file_loc status_whiteboard short_desc
213
                    deadline remaining_time estimated_time
214
                    work_time set_default_assignee set_default_qa_contact
215
                    cclist_accessible reporter_accessible
216
                    product confirm_product_change
217
                    bug_status resolution dup_id bug_ignored);
218 219
push(@set_fields, 'assigned_to') if !$cgi->param('set_default_assignee');
push(@set_fields, 'qa_contact')  if !$cgi->param('set_default_qa_contact');
220 221 222 223 224
my %field_translation = (
    bug_severity => 'severity',
    rep_platform => 'platform',
    short_desc   => 'summary',
    bug_file_loc => 'url',
225 226
    set_default_assignee   => 'reset_assigned_to',
    set_default_qa_contact => 'reset_qa_contact',
227
    confirm_product_change => 'product_change_confirmed',
228 229
);

230
my %set_all_fields = ( other_bugs => \@bug_objects );
231
foreach my $field_name (@set_fields) {
232
    if (should_set($field_name, 1)) {
233 234 235 236 237
        my $param_name = $field_translation{$field_name} || $field_name;
        $set_all_fields{$param_name} = $cgi->param($field_name);
    }
}

238
if (should_set('keywords')) {
239 240
    my $action = $cgi->param('keywordaction') || '';
    # Backward-compatibility for Bugzilla 3.x and older.
241 242 243 244
    $action = 'remove' if $action eq 'delete';
    $action = 'set'    if $action eq 'makeexact';
    $set_all_fields{keywords}->{$action} = $cgi->param('keywords');
}
245 246 247
if (should_set('comment')) {
    $set_all_fields{comment} = {
        body       => scalar $cgi->param('comment'),
248
        is_private => scalar $cgi->param('comment_is_private'),
249 250
    };
}
251 252 253 254 255 256 257
if (should_set('see_also')) {
    $set_all_fields{'see_also'}->{add} = 
        [split(/[\s,]+/, $cgi->param('see_also'))];
}
if (should_set('remove_see_also')) {
    $set_all_fields{'see_also'}->{remove} = [$cgi->param('remove_see_also')];
}
258 259 260 261
foreach my $dep_field (qw(dependson blocked)) {
    if (should_set($dep_field)) {
        if (my $dep_action = $cgi->param("${dep_field}_action")) {
            $set_all_fields{$dep_field}->{$dep_action} =
262
                [split(/[\s,]+/, $cgi->param($dep_field))];
263 264 265 266 267 268
        }
        else {
            $set_all_fields{$dep_field}->{set} = $cgi->param($dep_field);
        }
    }
}
269 270 271 272 273 274
# Formulate the CC data into two arrays of users involved in this CC change.
if (defined $cgi->param('newcc')
    or defined $cgi->param('addselfcc')
    or defined $cgi->param('removecc')
    or defined $cgi->param('masscc')) 
{
275
    my (@cc_add, @cc_remove);
276 277 278 279
    # If masscc is defined, then we came from buglist and need to either add or
    # remove cc's... otherwise, we came from show_bug and may need to do both.
    if (defined $cgi->param('masscc')) {
        if ($cgi->param('ccaction') eq 'add') {
280
            @cc_add = $cgi->param('masscc');
281
        } elsif ($cgi->param('ccaction') eq 'remove') {
282
            @cc_remove = $cgi->param('masscc');
283 284
        }
    } else {
285
        @cc_add = $cgi->param('newcc');
286
        push(@cc_add, $user) if $cgi->param('addselfcc');
287 288

        # We came from show_bug which uses a select box to determine what cc's
289 290
        # need to be removed...
        if ($cgi->param('removecc') && $cgi->param('cc')) {
291
            @cc_remove = $cgi->param('cc');
292 293 294
        }
    }

295
    $set_all_fields{cc} = { add => \@cc_add, remove => \@cc_remove };
296
}
297 298 299 300 301

# Fields that can only be set on one bug at a time.
if (defined $cgi->param('id')) {
    # Since aliases are unique (like bug numbers), they can only be changed
    # for one bug at a time.
302
    if (defined $cgi->param('alias')) {
303 304 305 306 307 308 309 310 311 312 313 314
        $set_all_fields{alias} = $cgi->param('alias');
    }
}

my %is_private;
foreach my $field (grep(/^defined_isprivate/, $cgi->param())) {
    $field =~ /(\d+)$/;
    my $comment_id = $1;
    $is_private{$comment_id} = $cgi->param("isprivate_$comment_id");
}
$set_all_fields{comment_is_private} = \%is_private;

315 316 317 318
my @check_groups = $cgi->param('defined_groups');
my @set_groups = $cgi->param('groups');
my ($removed_groups) = diff_arrays(\@check_groups, \@set_groups);
$set_all_fields{groups} = { add => \@set_groups, remove => $removed_groups };
319

320
my @custom_fields = Bugzilla->active_custom_fields;
321 322 323 324 325 326
foreach my $field (@custom_fields) {
    my $fname = $field->name;
    if (should_set($fname, 1)) {
        $set_all_fields{$fname} = [$cgi->param($fname)];
    }
}
327

328 329
# We are going to alter the list of removed groups, so we keep a copy here.
my @unchecked_groups = @$removed_groups;
330
foreach my $b (@bug_objects) {
331 332 333 334 335 336 337 338
    # Don't blindly ask to remove unchecked groups available in the UI.
    # A group can be already unchecked, and the user didn't try to remove it.
    # In this case, we don't want remove_group() to complain.
    my @remove_groups;
    foreach my $g (@{$b->groups_in}) {
        push(@remove_groups, $g->name) if grep { $_ eq $g->name } @unchecked_groups;
    }
    local $set_all_fields{groups}->{remove} = \@remove_groups;
339
    $b->set_all(\%set_all_fields);
340 341
}

342
if (defined $cgi->param('id')) {
343
    # Flags should be set AFTER the bug has been moved into another
344 345
    # product/component. The structure of flags code doesn't currently
    # allow them to be set using set_all.
346 347 348
    my ($flags, $new_flags) = Bugzilla::Flag->extract_flags_from_cgi(
        $first_bug, undef, $vars);
    $first_bug->set_flags($flags, $new_flags);
349 350 351 352 353 354 355 356

    # Tags can only be set to one bug at once.
    if (should_set('tag')) {
        my @new_tags = split(/[\s,]+/, $cgi->param('tag'));
        my ($tags_removed, $tags_added) = diff_arrays($first_bug->tags, \@new_tags);
        $first_bug->remove_tag($_) foreach @$tags_removed;
        $first_bug->add_tag($_) foreach @$tags_added;
    }
357
}
358

359 360 361 362
##############################
# Do Actual Database Updates #
##############################
foreach my $bug (@bug_objects) {
363
    my $changes = $bug->update();
364 365

    if ($changes->{'bug_status'}) {
366
        my $new_status = $changes->{'bug_status'}->[1];
367 368 369 370
        # We may have zeroed the remaining time, if we moved into a closed
        # status, so we should inform the user about that.
        if (!is_open_state($new_status) && $changes->{'remaining_time'}) {
            $vars->{'message'} = "remaining_time_zeroed"
371
              if $user->is_timetracker;
372 373
        }
    }
374

375
    $bug->send_changes($changes, $vars);
terry%netscape.com's avatar
terry%netscape.com committed
376 377
}

378 379 380
# Delete the session token used for the mass-change.
delete_token($token) unless $cgi->param('id');

381 382 383
if (Bugzilla->usage_mode == USAGE_MODE_EMAIL) {
    # Do nothing.
}
384 385 386 387 388 389 390 391 392 393
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;
394
        }
395
        $vars->{'bugs'} = [$bug];
396 397 398
        if ($action eq 'next_bug') {
            $vars->{'nextbug'} = $bug->id;
        }
399 400 401 402
        $template->process("bug/show.html.tmpl", $vars)
          || ThrowTemplateError($template->error());
        exit;
    }
403 404
} elsif ($action ne 'nothing') {
    ThrowCodeError("invalid_post_bug_submit_action");
terry%netscape.com's avatar
terry%netscape.com committed
405
}
406

407
# End the response page.
408 409 410 411 412 413 414 415
unless (Bugzilla->usage_mode == USAGE_MODE_EMAIL) {
    $template->process("bug/navigate.html.tmpl", $vars)
        || ThrowTemplateError($template->error());
    $template->process("global/footer.html.tmpl", $vars)
        || ThrowTemplateError($template->error());
}

1;