archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Correct flipped transaction constants in "Closed Date" migration

Browse Source 
Summary: These transaction constants are flipped, which can produce the wrong result in some cases.

Test Plan: `./bin/storage upgrade -f --apply phabricator:20180208.maniphest.02.populate.php`

Differential Revision: https://secure.phabricator.com/D19054
This commit is contained in:
epriestley
2018-02-10 06:09:22 -08:00
parent c64aae052f
commit ffc5c95c2f
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
resources/sql/autopatches/20180208.maniphest.02.populate.php
View File

@@ -20,8 +20,8 @@ foreach (new LiskMigrationIterator($table) as $task) {
// where the task was closed. A merge also counts as a close, even though
// it doesn't currently produce a separate transaction.
$type_merge = ManiphestTaskStatusTransaction::TRANSACTIONTYPE;
$type_status = ManiphestTaskMergedIntoTransaction::TRANSACTIONTYPE;
$type_status = ManiphestTaskStatusTransaction::TRANSACTIONTYPE;
$type_merge = ManiphestTaskMergedIntoTransaction::TRANSACTIONTYPE;
$xactions = id(new ManiphestTransactionQuery())
->setViewer($viewer)