From 0e612c910b416623b0412f2064e0942c425ccb2c Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Wed, 16 Jan 2013 10:51:08 -0800
Subject: [PATCH] Sort repositories in Diffusion by name, not creation order

Summary: Ref T2298. This seems like the least complicated reasonable order to implement.

Test Plan: Looked at repositories, saw them ordered by name.

Reviewers: vrana, btrahan, brennantaylor

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2298

Differential Revision: https://secure.phabricator.com/D4395
---
 .../diffusion/controller/DiffusionHomeController.php             | 1 +
 .../controller/PhabricatorRepositoryListController.php           | 1 +
 2 files changed, 2 insertions(+)

diff --git a/src/applications/diffusion/controller/DiffusionHomeController.php b/src/applications/diffusion/controller/DiffusionHomeController.php
index 96436d8c33..d59504f69a 100644
--- a/src/applications/diffusion/controller/DiffusionHomeController.php
+++ b/src/applications/diffusion/controller/DiffusionHomeController.php
@@ -51,6 +51,7 @@ final class DiffusionHomeController extends DiffusionController {
         unset($repositories[$key]);
       }
     }
+    $repositories = msort($repositories, 'getName');
 
     $repository_ids = mpull($repositories, 'getID');
     $summaries = array();
diff --git a/src/applications/repository/controller/PhabricatorRepositoryListController.php b/src/applications/repository/controller/PhabricatorRepositoryListController.php
index 484bda28b8..56921bd089 100644
--- a/src/applications/repository/controller/PhabricatorRepositoryListController.php
+++ b/src/applications/repository/controller/PhabricatorRepositoryListController.php
@@ -14,6 +14,7 @@ final class PhabricatorRepositoryListController
     $is_admin = $user->getIsAdmin();
 
     $repos = id(new PhabricatorRepository())->loadAll();
+    $repos = msort($repos, 'getName');
 
     $rows = array();
     foreach ($repos as $repo) {