Commit 6f274b8a authored by Max Kanat-Alexander's avatar Max Kanat-Alexander

Bug 577037: Make convert-workflow convert statuses in order, so that

IN_PROGRESS doesn't end up before CONFIRMED. r=LpSolit, a=LpSolit
parent b0b2c098
...@@ -54,16 +54,21 @@ END ...@@ -54,16 +54,21 @@ END
getc; getc;
my $dbh = Bugzilla->dbh; my $dbh = Bugzilla->dbh;
my %translation = ( # This is an array instead of a hash so that we can be sure that
NEW => 'CONFIRMED', # the translation happens in the right order. In particular, we
ASSIGNED => 'IN_PROGRESS', # want NEW to be renamed to CONFIRMED, instead of having REOPENED
REOPENED => 'CONFIRMED', # be the one that gets renamed.
CLOSED => 'VERIFIED', my @translation = (
[NEW => 'CONFIRMED'],
[ASSIGNED => 'IN_PROGRESS'],
[REOPENED => 'CONFIRMED'],
[CLOSED => 'VERIFIED'],
); );
my $status_field = Bugzilla::Field->check('bug_status'); my $status_field = Bugzilla::Field->check('bug_status');
$dbh->bz_start_transaction(); $dbh->bz_start_transaction();
while (my ($from, $to) = each %translation) { foreach my $pair (@translation) {
my ($from, $to) = @$pair;
print "Converting $from to $to...\n"; print "Converting $from to $to...\n";
$dbh->do('UPDATE bugs SET bug_status = ? WHERE bug_status = ?', $dbh->do('UPDATE bugs SET bug_status = ? WHERE bug_status = ?',
undef, $to, $from); undef, $to, $from);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment