revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 06:42:41 +01:00
Code Issues Releases Wiki Activity

Fix two "msort()" vs "msortv()" issues in "arc land"

Browse source 
Summary: See <https://github.com/phacility/arcanist/pull/242>. Ref T13303. A little more fallout turned up in `arc`.

Test Plan: `grep msort(`

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13303

Differential Revision: https://secure.phabricator.com/D20605
This commit is contained in:
epriestley 2019-06-20 14:40:20 -07:00
parent 1ef9409817
commit d92fa96366
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/workflow/ArcanistLandWorkflow.php
View file

@ -1465,7 +1465,7 @@ EOTEXT
$console->writeOut($message."\n\n");
$builds = msort($builds, 'getStatusSortVector');
$builds = msortv($builds, 'getStatusSortVector');
foreach ($builds as $build) {
$ansi_color = $build->getStatusANSIColor();
$status_name = $build->getStatusName();
@ -1557,7 +1557,7 @@ EOTEXT
$ongoing_builds = array();
$failed_builds = array();
$builds = msort($builds, 'getStatusSortVector');
$builds = msortv($builds, 'getStatusSortVector');
foreach ($builds as $build_ref) {
$plan = idx($plans, $build_ref->getBuildPlanPHID());
if (!$plan) {