process_bug.cgi 32.4 KB
Newer Older
1 2
#!/usr/bonsaitools/bin/perl -w
# -*- 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 <dave@intrec.com>
terry%netscape.com's avatar
terry%netscape.com committed
24

25 26 27
use diagnostics;
use strict;

28 29 30
my $UserInEditGroupSet = -1;
my $UserInCanConfirmGroupSet = -1;

31
require "CGI.pl";
32
use RelationSet;
33 34 35 36 37

# Shut up misguided -w warnings about "used only once":

use vars %::versions,
    %::components,
38
    %::COOKIE,
39
    %::MFORM,
40 41 42 43
    %::legal_keywords,
    %::legal_opsys,
    %::legal_platform,
    %::legal_priority,
44
    %::target_milestone,
45
    %::legal_severity;
46

47
my $whoid = confirm_login();
48

49 50
my $requiremilestone = 0;

51 52
print "Content-type: text/html\n\n";

53 54
PutHeader ("Bug processed");

55 56
GetVersionTable();

57 58 59 60
if ( Param("strictvaluechecks") ) {
    CheckFormFieldDefined(\%::FORM, 'product');
    CheckFormFieldDefined(\%::FORM, 'version');
    CheckFormFieldDefined(\%::FORM, 'component');
61 62 63 64 65

    # check if target milestone is defined - matthew@zeroknowledge.com
    if ( Param("usetargetmilestone") ) {
        CheckFormFieldDefined(\%::FORM, 'target_milestone');
    }
66 67
}

68
if ($::FORM{'product'} ne $::dontchange) {
69 70 71
    if ( Param("strictvaluechecks") ) {
        CheckFormField(\%::FORM, 'product', \@::legal_product);
    }
72
    my $prod = $::FORM{'product'};
73 74 75 76 77 78 79 80 81

    # note that when this script is called from buglist.cgi (rather
    # than show_bug.cgi), it's possible that the product will be changed
    # but that the version and/or component will be set to 
    # "--dont_change--" but still happen to be correct.  in this case,
    # the if statement will incorrectly trigger anyway.  this is a 
    # pretty weird case, and not terribly unreasonable behavior, but 
    # worthy of a comment, perhaps.
    #
82 83
    my $vok = lsearch($::versions{$prod}, $::FORM{'version'}) >= 0;
    my $cok = lsearch($::components{$prod}, $::FORM{'component'}) >= 0;
84 85 86 87 88 89 90 91 92

    my $mok = 1;   # so it won't affect the 'if' statement if milestones aren't used
    if ( Param("usetargetmilestone") ) {
       $mok = lsearch($::target_milestone{$prod}, $::FORM{'target_milestone'}) >= 0;
    }

    if (!$vok || !$cok || !$mok) {
        print "<H1>Changing product means changing version, target milestone and component.</H1>\n";
        print "You have chosen a new product, and now the version, target milestone and/or\n";
93
        print "component fields are not correct.  (Or, possibly, the bug did\n";
94 95
        print "not have a valid target milestone, component or version field in the first place.)\n";
        print "Anyway, please set the version, target milestone and component now.<p>\n";
96 97 98
        print "<form>\n";
        print "<table>\n";
        print "<tr>\n";
99
        print "<td align=right><b>Product:</b></td>\n";
100 101
        print "<td>$prod</td>\n";
        print "</tr><tr>\n";
102 103
        print "<td align=right><b>Version:</b></td>\n";
        print "<td>" . Version_element($::FORM{'version'}, $prod) . "</td>\n";
104
        print "</tr><tr>\n";
105 106 107 108 109 110 111

        if ( Param("usetargetmilestone") ) {
            print "<td align=right><b>Target Milestone:</b></td>\n";
            print "<td>" . Milestone_element($::FORM{'target_milestone'}, $prod) . "</td>\n";
            print "</tr><tr>\n";
        }

112 113
        print "<td align=right><b>Component:</b></td>\n";
        print "<td>" . Component_element($::FORM{'component'}, $prod) . "</td>\n";
114 115 116
        print "</tr>\n";
        print "</table>\n";
        foreach my $i (keys %::FORM) {
117
            if ($i ne 'version' && $i ne 'component' && $i ne 'target_milestone') {
118
                print "<input type=hidden name=$i value=\"" .
119
                value_quote($::FORM{$i}) . "\">\n";
terry%netscape.com's avatar
terry%netscape.com committed
120 121
            }
        }
122
        print "<input type=submit value=Commit>\n";
123 124
        print "</form>\n";
        print "</hr>\n";
125
        print "<a href=query.cgi>Cancel all this and go to the query page.</a>\n";
126
        PutFooter();
127
        exit;
terry%netscape.com's avatar
terry%netscape.com committed
128 129 130 131
    }
}


132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154
# Checks that the user is allowed to change the given field.  Actually, right
# now, the rules are pretty simple, and don't look at the field itself very
# much, but that could be enhanced.

my $lastbugid = 0;
my $ownerid;
my $reporterid;
my $qacontactid;

sub CheckCanChangeField {
    my ($f, $bugid, $oldvalue, $newvalue) = (@_);
    if ($f eq "assigned_to" || $f eq "reporter" || $f eq "qa_contact") {
        if ($oldvalue =~ /^\d+$/) {
            if ($oldvalue == 0) {
                $oldvalue = "";
            } else {
                $oldvalue = DBID_to_name($oldvalue);
            }
        }
    }
    if ($oldvalue eq $newvalue) {
        return 1;
    }
155 156 157
    if (trim($oldvalue) eq trim($newvalue)) {
        return 1;
    }
158 159 160 161 162 163 164 165 166 167 168 169 170 171
    if ($f =~ /^longdesc/) {
        return 1;
    }
    if ($UserInEditGroupSet < 0) {
        $UserInEditGroupSet = UserInGroup("editbugs");
    }
    if ($UserInEditGroupSet) {
        return 1;
    }
    if ($lastbugid != $bugid) {
        SendSQL("SELECT reporter, assigned_to, qa_contact FROM bugs " .
                "WHERE bug_id = $bugid");
        ($reporterid, $ownerid, $qacontactid) = (FetchSQLData());
    }
172 173 174 175 176 177
    # Let reporter change bug status, even if they can't edit bugs.
    # If reporter can't re-open their bug they will just file a duplicate.
    # While we're at it, let them close their own bugs as well.
    if ( ($f eq "bug_status") && ($whoid eq $reporterid) ) {
        return 1;
    }
178 179
    if ($f eq "bug_status" && $newvalue ne $::unconfirmedstate &&
        IsOpenedState($newvalue)) {
180 181 182 183 184 185 186 187 188 189 190 191

        # Hmm.  They are trying to set this bug to some opened state
        # that isn't the UNCONFIRMED state.  Are they in the right
        # group?  Or, has it ever been confirmed?  If not, then this
        # isn't legal.

        if ($UserInCanConfirmGroupSet < 0) {
            $UserInCanConfirmGroupSet = UserInGroup("canconfirm");
        }
        if ($UserInCanConfirmGroupSet) {
            return 1;
        }
192 193 194 195
        SendSQL("SELECT everconfirmed FROM bugs WHERE bug_id = $bugid");
        my $everconfirmed = FetchOneColumn();
        if ($everconfirmed) {
            return 1;
196
        }
197 198 199
    } elsif ($reporterid eq $whoid || $ownerid eq $whoid ||
             $qacontactid eq $whoid) {
        return 1;
200 201 202 203
    }
    SendSQL("UNLOCK TABLES");
    $oldvalue = value_quote($oldvalue);
    $newvalue = value_quote($newvalue);
204
    print PuntTryAgain(qq{
205 206 207 208 209 210
Only the owner or submitter of the bug, or a sufficiently
empowered user, may make that change to the $f field.
<TABLE>
<TR><TH ALIGN="right">Old value:</TH><TD>$oldvalue</TD></TR>
<TR><TH ALIGN="right">New value:</TH><TD>$newvalue</TD></TR>
</TABLE>
211
});
212 213 214 215 216 217 218 219 220
    PutFooter();
    exit();
}


    
    


221 222
my @idlist;
if (defined $::FORM{'id'}) {
223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245

    # since this means that we were called from show_bug.cgi, now is a good
    # time to do a whole bunch of error checking that can't easily happen when
    # we've been called from buglist.cgi, because buglist.cgi only tweaks
    # values that have been changed instead of submitting all the new values.
    # (XXX those error checks need to happen too, but implementing them 
    # is more work in the current architecture of this script...)
    #
    if ( Param('strictvaluechecks') ) { 
        CheckFormField(\%::FORM, 'rep_platform', \@::legal_platform);
        CheckFormField(\%::FORM, 'priority', \@::legal_priority);
        CheckFormField(\%::FORM, 'bug_severity', \@::legal_severity);
        CheckFormField(\%::FORM, 'component', 
                       \@{$::components{$::FORM{'product'}}});
        CheckFormFieldDefined(\%::FORM, 'bug_file_loc');
        CheckFormFieldDefined(\%::FORM, 'short_desc');
        CheckFormField(\%::FORM, 'product', \@::legal_product);
        CheckFormField(\%::FORM, 'version', 
                       \@{$::versions{$::FORM{'product'}}});
        CheckFormField(\%::FORM, 'op_sys', \@::legal_opsys);
        CheckFormFieldDefined(\%::FORM, 'longdesclength');
        CheckPosInt($::FORM{'id'});
    }
246
    push @idlist, $::FORM{'id'};
terry%netscape.com's avatar
terry%netscape.com committed
247
} else {
248 249
    foreach my $i (keys %::FORM) {
        if ($i =~ /^id_/) {
250 251 252
            if ( Param('strictvaluechecks') ) { 
                CheckPosInt(substr($i, 3));
            }
253
            push @idlist, substr($i, 3);
terry%netscape.com's avatar
terry%netscape.com committed
254 255 256 257
        }
    }
}

258 259 260 261 262 263 264 265 266 267 268 269
my $action  = '';
if (defined $::FORM{action}) {
  $action  = trim($::FORM{action});
}
if ($action eq Param("move-button-text")) {
  $::FORM{'buglist'} = join (":", @idlist);
  do "move.pl" || die "Error executing move.cgi: $!";
  PutFooter();
  exit;
}


270 271
if (!defined $::FORM{'who'}) {
    $::FORM{'who'} = $::COOKIE{'Bugzilla_login'};
terry%netscape.com's avatar
terry%netscape.com committed
272 273
}

274 275
# the common updates to all bugs in @idlist start here
#
276 277 278 279 280
print "<TITLE>Update Bug " . join(" ", @idlist) . "</TITLE>\n";
if (defined $::FORM{'id'}) {
    navigation_header();
}
print "<HR>\n";
281 282 283 284 285 286 287
$::query = "update bugs\nset";
$::comma = "";
umask(0);

sub DoComma {
    $::query .= "$::comma\n    ";
    $::comma = ",";
terry%netscape.com's avatar
terry%netscape.com committed
288 289
}

290 291 292 293 294 295 296 297 298 299 300 301 302 303
sub DoConfirm {
    if ($UserInEditGroupSet < 0) {
        $UserInEditGroupSet = UserInGroup("editbugs");
    }
    if ($UserInCanConfirmGroupSet < 0) {
        $UserInCanConfirmGroupSet = UserInGroup("canconfirm");
    }
    if ($UserInEditGroupSet || $UserInCanConfirmGroupSet) {
        DoComma();
        $::query .= "everconfirmed = 1";
    }
}


304 305 306 307
sub ChangeStatus {
    my ($str) = (@_);
    if ($str ne $::dontchange) {
        DoComma();
308 309 310 311 312
        if (IsOpenedState($str)) {
            $::query .= "bug_status = IF(everconfirmed = 1, '$str', '$::unconfirmedstate')";
        } else {
            $::query .= "bug_status = '$str'";
        }
313 314 315
        $::FORM{'bug_status'} = $str; # Used later for call to
                                      # CheckCanChangeField to make sure this
                                      # is really kosher.
terry%netscape.com's avatar
terry%netscape.com committed
316 317 318
    }
}

319 320 321 322 323
sub ChangeResolution {
    my ($str) = (@_);
    if ($str ne $::dontchange) {
        DoComma();
        $::query .= "resolution = '$str'";
terry%netscape.com's avatar
terry%netscape.com committed
324 325 326
    }
}

327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342
#
# This function checks if there is a comment required for a specific
# function and tests, if the comment was given.
# If comments are required for functions  is defined by params.
#
sub CheckonComment( $ ) {
    my ($function) = (@_);
    
    # Param is 1 if comment should be added !
    my $ret = Param( "commenton" . $function );

    # Allow without comment in case of undefined Params.
    $ret = 0 unless ( defined( $ret ));

    if( $ret ) {
        if (!defined $::FORM{'comment'} || $::FORM{'comment'} =~ /^\s*$/) {
343
            # No comment - sorry, action not allowed !
344 345 346
            PuntTryAgain("You have to specify a <b>comment</b> on this " .
                         "change.  Please give some words " .
                         "on the reason for your change.");
347 348 349 350 351 352 353
        } else {
            $ret = 0;
        }
    }
    return( ! $ret ); # Return val has to be inverted
}

terry%netscape.com's avatar
terry%netscape.com committed
354

355 356 357 358 359 360 361 362 363 364 365 366 367 368 369
my $foundbit = 0;
foreach my $b (grep(/^bit-\d*$/, keys %::FORM)) {
    if (!$foundbit) {
        $foundbit = 1;
        DoComma();
        $::query .= "groupset = 0";
    }
    if ($::FORM{$b}) {
        my $v = substr($b, 4);
        $::query .= "+ $v";     # Carefully written so that the math is
                                # done by MySQL, which can handle 64-bit math,
                                # and not by Perl, which I *think* can not.
    }
}

370
foreach my $field ("rep_platform", "priority", "bug_severity",          
371
                   "summary", "component", "bug_file_loc", "short_desc",
372
                   "product", "version", "op_sys",
373
                   "target_milestone", "status_whiteboard") {
374 375 376
    if (defined $::FORM{$field}) {
        if ($::FORM{$field} ne $::dontchange) {
            DoComma();
377
            $::query .= "$field = " . SqlQuote(trim($::FORM{$field}));
terry%netscape.com's avatar
terry%netscape.com committed
378 379 380 381
        }
    }
}

382

383 384
if (defined $::FORM{'qa_contact'}) {
    my $name = trim($::FORM{'qa_contact'});
terry%netscape.com's avatar
terry%netscape.com committed
385
    if ($name ne $::dontchange) {
386 387 388 389 390 391 392 393 394
        my $id = 0;
        if ($name ne "") {
            $id = DBNameToIdAndCheck($name);
        }
        DoComma();
        $::query .= "qa_contact = $id";
    }
}

395 396 397

ConnectToDatabase();

398 399 400
my $formCcSet = new RelationSet;
my $origCcSet = new RelationSet;
my $origCcString;
401 402

# We make sure to check out the CC list before we actually start touching any
403 404 405 406
# bugs.  mergeFromString() ultimately searches the database using a quoted
# form of the data it gets from $::FORM{'cc'}, so anything bogus from a 
# security standpoint should trigger an abort there.
#
407
if (defined $::FORM{'newcc'} && defined $::FORM{'id'}) {
408
    $origCcSet->mergeFromDB("select who from cc where bug_id = $::FORM{'id'}");
409
    $formCcSet->mergeFromDB("select who from cc where bug_id = $::FORM{'id'}");
410
    $origCcString = $origCcSet->toString();  # cache a copy of the string vers
411 412 413 414
    if ((exists $::FORM{'removecc'}) && (exists $::FORM{'cc'})) {
      $formCcSet->removeItemsInArray(@{$::MFORM{'cc'}});
    }
    $formCcSet->mergeFromString($::FORM{'newcc'});
415 416
}

417 418 419
if ( Param('strictvaluechecks') ) {
    CheckFormFieldDefined(\%::FORM, 'knob');
}
420 421 422 423
SWITCH: for ($::FORM{'knob'}) {
    /^none$/ && do {
        last SWITCH;
    };
424 425 426 427 428
    /^confirm$/ && CheckonComment( "confirm" ) && do {
        DoConfirm();
        ChangeStatus('NEW');
        last SWITCH;
    };
429
    /^accept$/ && CheckonComment( "accept" ) && do {
430
        DoConfirm();
431
        ChangeStatus('ASSIGNED');
432 433 434 435 436
        if (Param("musthavemilestoneonaccept")) {
            if (Param("usetargetmilestone")) {
                $requiremilestone = 1;
            }
        }
437 438
        last SWITCH;
    };
439
    /^clearresolution$/ && CheckonComment( "clearresolution" ) && do {
440 441 442
        ChangeResolution('');
        last SWITCH;
    };
443
    /^resolve$/ && CheckonComment( "resolve" ) && do {
444 445 446 447
        ChangeStatus('RESOLVED');
        ChangeResolution($::FORM{'resolution'});
        last SWITCH;
    };
448
    /^reassign$/ && CheckonComment( "reassign" ) && do {
449 450 451
        if ($::FORM{'andconfirm'}) {
            DoConfirm();
        }
452 453
        ChangeStatus('NEW');
        DoComma();
454 455 456
        if ( Param("strictvaluechecks") ) {
          if ( !defined$::FORM{'assigned_to'} ||
               trim($::FORM{'assigned_to'}) eq "") {
457 458 459 460
            PuntTryAgain("You cannot reassign to a bug to nobody.  Unless " .
                         "you intentionally cleared out the " .
                         "\"Reassign bug to\" field, " .
                         Param("browserbugmessage"));
461 462
          }
        }
463 464 465 466
        my $newid = DBNameToIdAndCheck($::FORM{'assigned_to'});
        $::query .= "assigned_to = $newid";
        last SWITCH;
    };
467
    /^reassignbycomponent$/  && CheckonComment( "reassignbycomponent" ) && do {
468
        if ($::FORM{'product'} eq $::dontchange) {
469 470
            PuntTryAgain("You must specify a product to help determine the " .
                         "new owner of these bugs.");
471
        }
472
        if ($::FORM{'component'} eq $::dontchange) {
473 474
            PuntTryAgain("You must specify a component whose owner should " .
                         "get assigned these bugs.");
terry%netscape.com's avatar
terry%netscape.com committed
475
        }
476 477 478
        if ($::FORM{'compconfirm'}) {
            DoConfirm();
        }
479 480
        ChangeStatus('NEW');
        SendSQL("select initialowner from components where program=" .
481 482
                SqlQuote($::FORM{'product'}) . " and value=" .
                SqlQuote($::FORM{'component'}));
483 484 485 486
        my $newname = FetchOneColumn();
        my $newid = DBNameToIdAndCheck($newname, 1);
        DoComma();
        $::query .= "assigned_to = $newid";
487 488 489 490 491 492 493 494 495 496 497
        if (Param("useqacontact")) {
            SendSQL("select initialqacontact from components where program=" .
                    SqlQuote($::FORM{'product'}) .
                    " and value=" . SqlQuote($::FORM{'component'}));
            my $qacontact = FetchOneColumn();
            if (defined $qacontact && $qacontact ne "") {
                my $newqa = DBNameToIdAndCheck($qacontact, 1);
                DoComma();
                $::query .= "qa_contact = $newqa";
            }
        }
498 499
        last SWITCH;
    };   
500
    /^reopen$/  && CheckonComment( "reopen" ) && do {
501
        ChangeStatus('REOPENED');
502
        ChangeResolution('');
503 504
        last SWITCH;
    };
505
    /^verify$/ && CheckonComment( "verify" ) && do {
506 507 508
        ChangeStatus('VERIFIED');
        last SWITCH;
    };
509
    /^close$/ && CheckonComment( "close" ) && do {
510 511 512
        ChangeStatus('CLOSED');
        last SWITCH;
    };
513
    /^duplicate$/ && CheckonComment( "duplicate" ) && do {
514 515
        ChangeStatus('RESOLVED');
        ChangeResolution('DUPLICATE');
516 517 518
        if ( Param('strictvaluechecks') ) {
            CheckFormFieldDefined(\%::FORM,'dup_id');
        }
519
        my $num = trim($::FORM{'dup_id'});
520 521 522
        SendSQL("SELECT bug_id FROM bugs WHERE bug_id = " . SqlQuote($num));
        $num = FetchOneColumn();
        if (!$num) {
523
            PuntTryAgain("You must specify a bug number of which this bug " .
524
                         "is a duplicate or a valid bug number.  The bug has not been changed.")
terry%netscape.com's avatar
terry%netscape.com committed
525
        }
526
        if (!defined($::FORM{'id'}) || $num == $::FORM{'id'}) {
527 528 529
            PuntTryAgain("Nice try, $::FORM{'who'}.  But it doesn't really ".
                         "make sense to mark a bug as a duplicate of " .
                         "itself, does it?");
terry%netscape.com's avatar
terry%netscape.com committed
530
        }
531 532 533 534 535 536 537
        my $checkid = trim($::FORM{'id'});
        SendSQL("SELECT bug_id FROM bugs where bug_id = " .  SqlQuote($checkid));
        $checkid = FetchOneColumn();
        if (!$checkid) {
            PuntTryAgain("The bug id $::FORM{'id'} is invalid. Please reload this bug ".
                         "and try again.");
        }
538
        AppendComment($num, $::FORM{'who'}, "*** Bug $::FORM{'id'} has been marked as a duplicate of this bug. ***");
539 540 541
        if ( Param('strictvaluechecks') ) {
          CheckFormFieldDefined(\%::FORM,'comment');
        }
542
        $::FORM{'comment'} .= "\n\n*** This bug has been marked as a duplicate of $num ***";
543

544
        print "<TABLE BORDER=1><TD><H2>Notation added to bug $num</H2>\n";
545
        system("./processmail", $num, $::FORM{'who'});
546
        print "<TD><A HREF=\"show_bug.cgi?id=$num\">Go To BUG# $num</A></TABLE>\n";
547

548 549 550 551
        last SWITCH;
    };
    # default
    print "Unknown action $::FORM{'knob'}!\n";
552
    PutFooter();
553
    exit;
terry%netscape.com's avatar
terry%netscape.com committed
554 555 556
}


557
if ($#idlist < 0) {
558
    PuntTryAgain("You apparently didn't choose any bugs to modify.");
terry%netscape.com's avatar
terry%netscape.com committed
559 560
}

561 562 563 564 565

my @keywordlist;
my %keywordseen;

if ($::FORM{'keywords'}) {
566 567 568 569
    foreach my $keyword (split(/[\s,]+/, $::FORM{'keywords'})) {
        if ($keyword eq '') {
            next;
        }
570
        my $i = GetKeywordIdFromName($keyword);
571
        if (!$i) {
572 573 574 575
            PuntTryAgain("Unknown keyword named <code>$keyword</code>. " .
                         "<P>The legal keyword names are " .
                         "<A HREF=describekeywords.cgi>" .
                         "listed here</A>.");
576 577 578 579 580 581 582 583
        }
        if (!$keywordseen{$i}) {
            push(@keywordlist, $i);
            $keywordseen{$i} = 1;
        }
    }
}

584 585 586
my $keywordaction = $::FORM{'keywordaction'} || "makeexact";

if ($::comma eq "" && 0 == @keywordlist && $keywordaction ne "makeexact") {
587
    if (!defined $::FORM{'comment'} || $::FORM{'comment'} =~ /^\s*$/) {
588 589
        PuntTryAgain("Um, you apparently did not change anything on the " .
                     "selected bugs.");
terry%netscape.com's avatar
terry%netscape.com committed
590 591 592
    }
}

593
my $basequery = $::query;
594
my $delta_ts;
terry%netscape.com's avatar
terry%netscape.com committed
595

596

597 598
sub SnapShotBug {
    my ($id) = (@_);
599
    SendSQL("select delta_ts, " . join(',', @::log_columns) .
600
            " from bugs where bug_id = $id");
601 602
    my @row = FetchSQLData();
    $delta_ts = shift @row;
603

604
    return @row;
terry%netscape.com's avatar
terry%netscape.com committed
605 606 607
}


608 609 610 611 612 613 614 615 616 617 618 619 620 621 622 623 624
sub SnapShotDeps {
    my ($i, $target, $me) = (@_);
    SendSQL("select $target from dependencies where $me = $i order by $target");
    my @list;
    while (MoreSQLData()) {
        push(@list, FetchOneColumn());
    }
    return join(',', @list);
}


my $timestamp;

sub LogDependencyActivity {
    my ($i, $oldstr, $target, $me) = (@_);
    my $newstr = SnapShotDeps($i, $target, $me);
    if ($oldstr ne $newstr) {
625 626 627 628
        my $fieldid = GetFieldID($target);
        SendSQL("INSERT INTO bugs_activity " .
                "(bug_id,who,bug_when,fieldid,oldvalue,newvalue) VALUES " .
                "($i,$whoid,$timestamp,$fieldid,'$oldstr','$newstr')");
629 630 631 632 633
        return 1;
    }
    return 0;
}

634 635 636 637 638 639
delete $::FORM{'resolution'};   # Make sure we don't test the resolution
                                # against our permissions; we've already done
                                # that kind of testing, and this form field
                                # is actually usually not used.


640
# this loop iterates once for each bug to be processed (eg when this script
641
# is called with multiple bugs selected from buglist.cgi instead of
642 643
# show_bug.cgi).
#
644
foreach my $id (@idlist) {
645
    my %dependencychanged;
646 647
    my $write = "WRITE";        # Might want to make a param to control
                                # whether we do LOW_PRIORITY ...
648 649 650
    SendSQL("LOCK TABLES bugs $write, bugs_activity $write, cc $write, " .
            "profiles $write, dependencies $write, votes $write, " .
            "keywords $write, longdescs $write, fielddefs $write, " .
651
            "keyworddefs READ, groups READ, attachments READ, products READ");
652
    my @oldvalues = SnapShotBug($id);
653
    my %oldhash;
654 655
    my $i = 0;
    foreach my $col (@::log_columns) {
656
        $oldhash{$col} = $oldvalues[$i];
657 658 659 660 661
        if (exists $::FORM{$col}) {
            CheckCanChangeField($col, $id, $oldvalues[$i], $::FORM{$col});
        }
        $i++;
    }
662 663 664 665 666 667 668 669 670 671 672 673 674 675 676
    if ($requiremilestone) {
        my $value = $::FORM{'target_milestone'};
        if (!defined $value || $value eq $::dontchange) {
            $value = $oldhash{'target_milestone'};
        }
        SendSQL("SELECT defaultmilestone FROM products WHERE product = " .
                SqlQuote($oldhash{'product'}));
        if ($value eq FetchOneColumn()) {
            SendSQL("UNLOCK TABLES");
            PuntTryAgain("You must determine a target milestone for bug $id " .
                         "if you are going to accept it.  (Part of " .
                         "accepting a bug is giving an estimate of when it " .
                         "will be fixed.)");
        }
    }   
677 678 679 680 681 682 683 684
    if (defined $::FORM{'delta_ts'} && $::FORM{'delta_ts'} ne $delta_ts) {
        print "
<H1>Mid-air collision detected!</H1>
Someone else has made changes to this bug at the same time you were trying to.
The changes made were:
<p>
";
        DumpBugActivity($id, $delta_ts);
685
        my $longdesc = GetLongDescriptionAsHTML($id);
686
        my $longchanged = 0;
687

688 689
        if (length($longdesc) > $::FORM{'longdesclength'}) {
            $longchanged = 1;
690 691 692
            print "<P>Added text to the long description:<blockquote>";
            print substr($longdesc, $::FORM{'longdesclength'});
            print "</blockquote>\n";
693 694 695 696 697 698 699 700 701 702
        }
        SendSQL("unlock tables");
        print "You have the following choices: <ul>\n";
        $::FORM{'delta_ts'} = $delta_ts;
        print "<li><form method=post>";
        foreach my $i (keys %::FORM) {
            my $value = value_quote($::FORM{$i});
            print qq{<input type=hidden name="$i" value="$value">\n};
        }
        print qq{<input type=submit value="Submit my changes anyway">\n};
703
        print " This will cause all of the above changes to be overwritten";
704 705 706
        if ($longchanged) {
            print ", except for the changes to the description";
        }
707
        print qq{.</form>\n<li><a href="show_bug.cgi?id=$id">Throw away my changes, and go revisit bug $id</a></ul>\n};
708
        PutFooter();
709 710 711
        exit;
    }
        
712 713 714 715 716 717 718 719 720 721 722 723 724 725 726 727
    my %deps;
    if (defined $::FORM{'dependson'}) {
        my $me = "blocked";
        my $target = "dependson";
        for (1..2) {
            $deps{$target} = [];
            my %seen;
            foreach my $i (split('[\s,]+', $::FORM{$target})) {
                if ($i eq "") {
                    next;

                }
                SendSQL("select bug_id from bugs where bug_id = " .
                        SqlQuote($i));
                my $comp = FetchOneColumn();
                if ($comp ne $i) {
728
                    PuntTryAgain("$i is not a legal bug number");
729 730 731 732 733 734 735 736 737 738 739 740 741
                }
                if (!exists $seen{$i}) {
                    push(@{$deps{$target}}, $i);
                    $seen{$i} = 1;
                }
            }
            my @stack = @{$deps{$target}};
            while (@stack) {
                my $i = shift @stack;
                SendSQL("select $target from dependencies where $me = $i");
                while (MoreSQLData()) {
                    my $t = FetchOneColumn();
                    if ($t == $id) {
742 743 744 745
                        PuntTryAgain("Dependency loop detected!<P>" .
                                     "The change you are making to " .
                                     "dependencies has caused a circular " .
                                     "dependency chain.");
746 747 748 749 750 751 752 753
                    }
                    if (!exists $seen{$t}) {
                        push @stack, $t;
                        $seen{$t} = 1;
                    }
                }
            }
                        
754

755 756 757 758 759
            my $tmp = $me;
            $me = $target;
            $target = $tmp;
        }
    }
760

761 762 763 764 765 766
    if (@::legal_keywords) {
        # There are three kinds of "keywordsaction": makeexact, add, delete.
        # For makeexact, we delete everything, and then add our things.
        # For add, we delete things we're adding (to make sure we don't
        # end up having them twice), and then we add them.
        # For delete, we just delete things on the list.
767
        my $changed = 0;
768 769
        if ($keywordaction eq "makeexact") {
            SendSQL("DELETE FROM keywords WHERE bug_id = $id");
770
            $changed = 1;
771 772 773 774 775
        }
        foreach my $keyword (@keywordlist) {
            if ($keywordaction ne "makeexact") {
                SendSQL("DELETE FROM keywords
                         WHERE bug_id = $id AND keywordid = $keyword");
776
                $changed = 1;
777 778 779 780
            }
            if ($keywordaction ne "delete") {
                SendSQL("INSERT INTO keywords 
                         (bug_id, keywordid) VALUES ($id, $keyword)");
781 782 783 784 785 786 787 788 789 790 791 792
                $changed = 1;
            }
        }
        if ($changed) {
            SendSQL("SELECT keyworddefs.name 
                     FROM keyworddefs, keywords
                     WHERE keywords.bug_id = $id
                         AND keyworddefs.id = keywords.keywordid
                     ORDER BY keyworddefs.name");
            my @list;
            while (MoreSQLData()) {
                push(@list, FetchOneColumn());
793
            }
794 795 796
            SendSQL("UPDATE bugs SET keywords = " .
                    SqlQuote(join(', ', @list)) .
                    " WHERE bug_id = $id");
797 798 799
        }
    }

800
    my $query = "$basequery\nwhere bug_id = $id";
terry%netscape.com's avatar
terry%netscape.com committed
801
    
802 803 804 805
# print "<PRE>$query</PRE>\n";

    if ($::comma ne "") {
        SendSQL($query);
806 807 808
        SendSQL("select delta_ts from bugs where bug_id = $id");
    } else {
        SendSQL("select now()");
terry%netscape.com's avatar
terry%netscape.com committed
809
    }
810
    $timestamp = FetchOneColumn();
terry%netscape.com's avatar
terry%netscape.com committed
811
    
812 813
    if (defined $::FORM{'comment'}) {
        AppendComment($id, $::FORM{'who'}, $::FORM{'comment'});
terry%netscape.com's avatar
terry%netscape.com committed
814 815
    }
    
816
    if (defined $::FORM{'newcc'} && defined $::FORM{'id'}
817 818 819 820 821 822 823 824 825 826 827 828
        && ! $origCcSet->isEqual($formCcSet) ) {

        # update the database to look like the form
        #
        my @CCDELTAS = $origCcSet->generateSqlDeltas($formCcSet, "cc", 
                                                     "bug_id", $::FORM{'id'},
                                                     "who");
        $CCDELTAS[0] eq "" || SendSQL($CCDELTAS[0]);
        $CCDELTAS[1] eq "" || SendSQL($CCDELTAS[1]);

        my $col = GetFieldID('cc');
        my $origq = SqlQuote($origCcString);
829
        my $newq = SqlQuote($formCcSet->toString());
830 831 832 833 834
        SendSQL("INSERT INTO bugs_activity " . 
                "(bug_id,who,bug_when,fieldid,oldvalue,newvalue) VALUES " . 
                "($id,$whoid,'$timestamp',$col,$origq,$newq)");
    }
  
835 836 837 838 839 840 841 842 843 844 845 846

    if (defined $::FORM{'dependson'}) {
        my $me = "blocked";
        my $target = "dependson";
        for (1..2) {
            SendSQL("select $target from dependencies where $me = $id order by $target");
            my %snapshot;
            my @oldlist;
            while (MoreSQLData()) {
                push(@oldlist, FetchOneColumn());
            }
            my @newlist = sort {$a <=> $b} @{$deps{$target}};
847 848
            @dependencychanged{@oldlist} = 1;
            @dependencychanged{@newlist} = 1;
849 850 851 852 853 854 855 856 857 858 859 860 861 862 863 864 865 866 867 868 869 870 871 872 873 874 875 876

            while (0 < @oldlist || 0 < @newlist) {
                if (@oldlist == 0 || (@newlist > 0 &&
                                      $oldlist[0] > $newlist[0])) {
                    $snapshot{$newlist[0]} = SnapShotDeps($newlist[0], $me,
                                                          $target);
                    shift @newlist;
                } elsif (@newlist == 0 || (@oldlist > 0 &&
                                           $newlist[0] > $oldlist[0])) {
                    $snapshot{$oldlist[0]} = SnapShotDeps($oldlist[0], $me,
                                                          $target);
                    shift @oldlist;
                } else {
                    if ($oldlist[0] != $newlist[0]) {
                        die "Error in list comparing code";
                    }
                    shift @oldlist;
                    shift @newlist;
                }
            }
            my @keys = keys(%snapshot);
            if (@keys) {
                my $oldsnap = SnapShotDeps($id, $target, $me);
                SendSQL("delete from dependencies where $me = $id");
                foreach my $i (@{$deps{$target}}) {
                    SendSQL("insert into dependencies ($me, $target) values ($id, $i)");
                }
                foreach my $k (@keys) {
877
                    LogDependencyActivity($k, $snapshot{$k}, $me, $target);
878 879 880 881 882 883 884 885 886 887 888
                }
                LogDependencyActivity($id, $oldsnap, $target, $me);
            }

            my $tmp = $me;
            $me = $target;
            $target = $tmp;
        }
    }


889 890 891 892
    # get a snapshot of the newly set values out of the database, 
    # and then generate any necessary bug activity entries by seeing 
    # what has changed since before we wrote out the new values.
    #
893
    my @newvalues = SnapShotBug($id);
894

895
    foreach my $c (@::log_columns) {
896 897
        my $col = $c;           # We modify it, don't want to modify array
                                # values in place.
898 899
        my $old = shift @oldvalues;
        my $new = shift @newvalues;
900 901 902 903 904 905
        if (!defined $old) {
            $old = "";
        }
        if (!defined $new) {
            $new = "";
        }
906
        if ($old ne $new) {
907
            if ($col eq 'assigned_to' || $col eq 'qa_contact') {
908 909
                $old = ($old) ? DBID_to_name($old) : "";
                $new = ($new) ? DBID_to_name($new) : "";
910 911 912
                $origCcString .= ",$old"; # make sure to send mail to people
                                          # if they are going to no longer get
                                          # updates about this bug.
terry%netscape.com's avatar
terry%netscape.com committed
913
            }
914
            if ($col eq 'product') {
915
                RemoveVotes($id, 0,
916 917
                            "This bug has been moved to a different product");
            }
918
            $col = GetFieldID($col);
919 920
            $old = SqlQuote($old);
            $new = SqlQuote($new);
921
            my $q = "insert into bugs_activity (bug_id,who,bug_when,fieldid,oldvalue,newvalue) values ($id,$whoid,'$timestamp',$col,$old,$new)";
922
            # puts "<pre>$q</pre>"
923
            SendSQL($q);
terry%netscape.com's avatar
terry%netscape.com committed
924 925 926
        }
    }
    
927
    print "<TABLE BORDER=1><TD><H2>Changes to bug $id submitted</H2>\n";
928
    SendSQL("unlock tables");
929
    system("./processmail", "-forcecc", $origCcString, $id, $::FORM{'who'});
930
    print "<TD><A HREF=\"show_bug.cgi?id=$id\">Back To BUG# $id</A></TABLE>\n";
931 932

    foreach my $k (keys(%dependencychanged)) {
933
        print "<TABLE BORDER=1><TD><H2>Checking for dependency changes on bug $k</H2>\n";
934
        system("./processmail", $k, $::FORM{'who'});
935 936 937
        print "<TD><A HREF=\"show_bug.cgi?id=$k\">Go To BUG# $k</A></TABLE>\n";
    }

terry%netscape.com's avatar
terry%netscape.com committed
938 939
}

940
if (defined $::next_bug) {
941
    print("<P>The next bug in your list is:\n");
942 943
    $::FORM{'id'} = $::next_bug;
    print "<HR>\n";
terry%netscape.com's avatar
terry%netscape.com committed
944

945
    navigation_header();
946
    do "bug_form.pl";
terry%netscape.com's avatar
terry%netscape.com committed
947
} else {
948
    navigation_header();
949
    PutFooter();
terry%netscape.com's avatar
terry%netscape.com committed
950
}