From 1c04532852a3807cd1cc672e5d46eab5649f746c Mon Sep 17 00:00:00 2001 From: Bob Trahan Date: Tue, 30 Dec 2014 14:28:26 -0800 Subject: [PATCH] Owners - sort repositories alphabetically in path chooser Summary: Fixes T6834. Without this asort() call, repositories are ordered by ID, which is arbitrary to the user. This sorts them alphabetically by callsign. Test Plan: edited a package and saw repos sorted by call sign Reviewers: epriestley, chad Reviewed By: chad Subscribers: Korvin, epriestley Maniphest Tasks: T6834 Differential Revision: https://secure.phabricator.com/D11095 --- .../owners/controller/PhabricatorOwnersEditController.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/owners/controller/PhabricatorOwnersEditController.php b/src/applications/owners/controller/PhabricatorOwnersEditController.php index 6e1163ddc0..5ffdec2adb 100644 --- a/src/applications/owners/controller/PhabricatorOwnersEditController.php +++ b/src/applications/owners/controller/PhabricatorOwnersEditController.php @@ -145,6 +145,7 @@ final class PhabricatorOwnersEditController } $repos = mpull($repos, 'getCallsign', 'getPHID'); + asort($repos); $template = new AphrontTypeaheadTemplateView(); $template = $template->render();