process_bug.cgi 15.2 KB
Newer Older
1
#!/usr/bin/perl -wT
2
# -*- Mode: perl; indent-tabs-mode: nil -*-
terry%netscape.com's avatar
terry%netscape.com committed
3
#
4 5 6 7 8 9 10 11 12 13
# The contents of this file are subject to the Mozilla Public
# License Version 1.1 (the "License"); you may not use this file
# except in compliance with the License. You may obtain a copy of
# the License at http://www.mozilla.org/MPL/
#
# Software distributed under the License is distributed on an "AS
# IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
# implied. See the License for the specific language governing
# rights and limitations under the License.
#
terry%netscape.com's avatar
terry%netscape.com committed
14
# The Original Code is the Bugzilla Bug Tracking System.
15
#
terry%netscape.com's avatar
terry%netscape.com committed
16
# The Initial Developer of the Original Code is Netscape Communications
17 18 19 20
# Corporation. Portions created by Netscape are
# Copyright (C) 1998 Netscape Communications Corporation. All
# Rights Reserved.
#
terry%netscape.com's avatar
terry%netscape.com committed
21
# Contributor(s): Terry Weissman <terry@mozilla.org>
22
#                 Dan Mosedale <dmose@mozilla.org>
23
#                 Dave Miller <justdave@syndicomm.com>
24
#                 Christopher Aillon <christopher@aillon.com>
25
#                 Myk Melez <myk@mozilla.org>
26
#                 Jeff Hedlund <jeff.hedlund@matrixsi.com>
27
#                 Frédéric Buclin <LpSolit@gmail.com>
28
#                 Lance Larsh <lance.larsh@oracle.com>
29
#                 Akamai Technologies <bugzilla-dev@akamai.com>
30
#                 Max Kanat-Alexander <mkanat@bugzilla.org>
terry%netscape.com's avatar
terry%netscape.com committed
31

32 33 34 35 36 37 38 39 40 41 42 43
# 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.

44 45
use strict;

46
use lib qw(. lib);
47

48
use Bugzilla;
49
use Bugzilla::Constants;
50
use Bugzilla::Bug;
51
use Bugzilla::BugMail;
52
use Bugzilla::Mailer;
53
use Bugzilla::User;
54
use Bugzilla::Util;
55
use Bugzilla::Error;
56
use Bugzilla::Field;
57
use Bugzilla::Product;
58
use Bugzilla::Component;
59
use Bugzilla::Keyword;
60
use Bugzilla::Flag;
61
use Bugzilla::Status;
62
use Bugzilla::Token;
63

64
use List::MoreUtils qw(firstidx);
65 66
use Storable qw(dclone);

67
my $user = Bugzilla->login(LOGIN_REQUIRED);
68

69
my $cgi = Bugzilla->cgi;
70
my $dbh = Bugzilla->dbh;
71
my $template = Bugzilla->template;
72
my $vars = {};
73

74 75 76 77
######################################################################
# Subroutines
######################################################################

78
# Tells us whether or not a field should be changed by process_bug.
79
sub should_set {
80
    # check_defined is used for fields where there's another field
81 82 83 84
    # 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) = @_;
85
    my $cgi = Bugzilla->cgi;
86 87
    if ( defined $cgi->param($field) 
         || ($check_defined && defined $cgi->param("defined_$field")) )
88 89 90 91 92 93
    {
        return 1;
    }
    return 0;
}

94 95 96 97
######################################################################
# Begin Data/Security Validation
######################################################################

98 99
# Create a list of objects for all bugs being modified in this request.
my @bug_objects;
100
if (defined $cgi->param('id')) {
101 102 103
  my $bug = Bugzilla::Bug->check(scalar $cgi->param('id'));
  $cgi->param('id', $bug->id);
  push(@bug_objects, $bug);
104
} else {
105
    foreach my $i ($cgi->param()) {
106
        if ($i =~ /^id_([1-9][0-9]*)/) {
107
            my $id = $1;
108
            push(@bug_objects, Bugzilla::Bug->check($id));
109
        }
110 111 112
    }
}

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

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

118 119 120 121
# 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!
122 123
        # Skip ones we've already deleted (such as "defined_$name").
        next if !defined $cgi->param($name);
124 125
        if ($cgi->param($name) eq $cgi->param('dontchange')) {
            $cgi->delete($name);
126
            $cgi->delete("defined_$name");
127 128
        }
    }
129 130
}

131
# do a match on the fields if applicable
132
Bugzilla::User::match_field({
133 134 135 136 137
    'qa_contact'                => { 'type' => 'single' },
    'newcc'                     => { 'type' => 'multi'  },
    'masscc'                    => { 'type' => 'multi'  },
    'assigned_to'               => { 'type' => 'single' },
});
138

139
print $cgi->header() unless Bugzilla->usage_mode == USAGE_MODE_EMAIL;
140

141 142
# Check for a mid-air collision. Currently this only works when updating
# an individual bug.
143
if (defined $cgi->param('delta_ts'))
144
{
145 146 147 148 149 150 151 152
    my $delta_ts_z = datetime_from($cgi->param('delta_ts'));
    my $first_delta_tz_z =  datetime_from($first_bug->delta_ts);
    if ($first_delta_tz_z ne $delta_ts_z) {
        ($vars->{'operations'}) =
            Bugzilla::Bug::GetBugActivity($first_bug->id, undef,
                                          scalar $cgi->param('delta_ts'));

        $vars->{'title_tag'} = "mid_air";
153
    
154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169
        ThrowCodeError('undefined_field', { field => 'longdesclength' })
            if !defined $cgi->param('longdesclength');

        $vars->{'start_at'} = $cgi->param('longdesclength');
        # Always sort midair collision comments oldest to newest,
        # regardless of the user's personal preference.
        $vars->{'comments'} = $first_bug->comments({ order => "oldest_to_newest" });
        $vars->{'bug'} = $first_bug;

        # 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]));
        # 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;
    }
170
}
171

172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187
# 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')) {
    check_hash_token($token, [$first_bug->id, $first_bug->delta_ts]);
}
else {
    check_token_data($token, 'buglist_mass_change', 'query.cgi');
}

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

188 189
$vars->{'title_tag'} = "bug_processed";

190
my $action;
191
if (defined $cgi->param('id')) {
192
    $action = $user->setting('post_bug_submit_action');
193 194

    if ($action eq 'next_bug') {
195 196
        my $bug_list_obj = $user->recent_search_for($first_bug);
        my @bug_list = $bug_list_obj ? @{$bug_list_obj->bug_list} : ();
197
        my $cur = firstidx { $_ eq $cgi->param('id') } @bug_list;
198
        if ($cur >= 0 && $cur < $#bug_list) {
199 200 201
            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)) {
202
                # We create an object here so that $bug->send_changes can use it
203 204 205
                # when displaying the header.
                $vars->{'bug'} = new Bugzilla::Bug($next_bug_id);
            }
206 207 208 209
        }
    }
    # Include both action = 'same_bug' and 'nothing'.
    else {
210
        $vars->{'bug'} = $first_bug;
211 212 213 214 215 216 217
    }
}
else {
    # param('id') is not defined when changing multiple bugs at once.
    $action = 'nothing';
}

218 219 220 221 222 223
# For each bug, we have to check if the user can edit the bug the product
# is currently in, before we allow them to change anything.
foreach my $bug (@bug_objects) {
    if (!Bugzilla->user->can_edit_product($bug->product_obj->id) ) {
        ThrowUserError("product_edit_denied",
                      { product => $bug->product });
224
    }
225
}
226

227 228 229
# 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.
230 231 232
my @set_fields = qw(op_sys rep_platform priority bug_severity
                    component target_milestone version
                    bug_file_loc status_whiteboard short_desc
233
                    deadline remaining_time estimated_time
234
                    work_time set_default_assignee set_default_qa_contact
235
                    cclist_accessible reporter_accessible 
236 237
                    product confirm_product_change
                    bug_status resolution dup_id);
238 239
push(@set_fields, 'assigned_to') if !$cgi->param('set_default_assignee');
push(@set_fields, 'qa_contact')  if !$cgi->param('set_default_qa_contact');
240 241 242 243 244
my %field_translation = (
    bug_severity => 'severity',
    rep_platform => 'platform',
    short_desc   => 'summary',
    bug_file_loc => 'url',
245 246
    set_default_assignee   => 'reset_assigned_to',
    set_default_qa_contact => 'reset_qa_contact',
247
    confirm_product_change => 'product_change_confirmed',
248 249
);

250
my %set_all_fields = ( other_bugs => \@bug_objects );
251
foreach my $field_name (@set_fields) {
252
    if (should_set($field_name, 1)) {
253 254 255 256 257
        my $param_name = $field_translation{$field_name} || $field_name;
        $set_all_fields{$param_name} = $cgi->param($field_name);
    }
}

258
if (should_set('keywords')) {
259 260
    my $action = $cgi->param('keywordaction') || '';
    # Backward-compatibility for Bugzilla 3.x and older.
261 262 263 264
    $action = 'remove' if $action eq 'delete';
    $action = 'set'    if $action eq 'makeexact';
    $set_all_fields{keywords}->{$action} = $cgi->param('keywords');
}
265 266 267
if (should_set('comment')) {
    $set_all_fields{comment} = {
        body       => scalar $cgi->param('comment'),
268
        is_private => scalar $cgi->param('comment_is_private'),
269 270
    };
}
271 272 273 274 275 276 277
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')];
}
278 279 280 281 282 283 284 285 286 287 288
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} =
                [split(/\s,/, $cgi->param($dep_field))];
        }
        else {
            $set_all_fields{$dep_field}->{set} = $cgi->param($dep_field);
        }
    }
}
289 290 291 292 293 294
# 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')) 
{
295
    my (@cc_add, @cc_remove);
296 297 298 299
    # 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') {
300
            @cc_add = $cgi->param('masscc');
301
        } elsif ($cgi->param('ccaction') eq 'remove') {
302
            @cc_remove = $cgi->param('masscc');
303 304
        }
    } else {
305 306 307 308
        @cc_add = $cgi->param('newcc');
        push(@cc_add, Bugzilla->user) if $cgi->param('addselfcc');

        # We came from show_bug which uses a select box to determine what cc's
309 310
        # need to be removed...
        if ($cgi->param('removecc') && $cgi->param('cc')) {
311
            @cc_remove = $cgi->param('cc');
312 313 314
        }
    }

315
    $set_all_fields{cc} = { add => \@cc_add, remove => \@cc_remove };
316
}
317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334

# 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.
    if (Bugzilla->params->{"usebugaliases"} && defined $cgi->param('alias')) {
        $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;

335 336 337 338
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 };
339

340
my @custom_fields = Bugzilla->active_custom_fields;
341 342 343 344 345 346
foreach my $field (@custom_fields) {
    my $fname = $field->name;
    if (should_set($fname, 1)) {
        $set_all_fields{$fname} = [$cgi->param($fname)];
    }
}
347

348 349
# We are going to alter the list of removed groups, so we keep a copy here.
my @unchecked_groups = @$removed_groups;
350
foreach my $b (@bug_objects) {
351 352 353 354 355 356 357 358
    # 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;
359
    $b->set_all(\%set_all_fields);
360 361
}

362
if (defined $cgi->param('id')) {
363
    # Flags should be set AFTER the bug has been moved into another
364 365
    # product/component. The structure of flags code doesn't currently
    # allow them to be set using set_all.
366 367 368
    my ($flags, $new_flags) = Bugzilla::Flag->extract_flags_from_cgi(
        $first_bug, undef, $vars);
    $first_bug->set_flags($flags, $new_flags);
369
}
370

371 372 373 374
##############################
# Do Actual Database Updates #
##############################
foreach my $bug (@bug_objects) {
375
    my $changes = $bug->update();
376 377

    if ($changes->{'bug_status'}) {
378
        my $new_status = $changes->{'bug_status'}->[1];
379 380 381 382
        # 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"
383
              if Bugzilla->user->is_timetracker;
384 385
        }
    }
386

387
    $bug->send_changes($changes, $vars);
terry%netscape.com's avatar
terry%netscape.com committed
388 389
}

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

416
# End the response page.
417 418 419 420 421 422 423 424
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;