From e3cfcc4a464388771087231404510ab46b649594 Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 16 Jan 2013 11:39:13 -0800 Subject: [PATCH] Sort options by key on "all config" Summary: Oops, missed this -- alphabetical is probably a better sort order than by-group-then-by-definition. Test Plan: Looked at alphabetical options. Reviewers: codeblock, btrahan Reviewed By: codeblock CC: aran Maniphest Tasks: T2255 Differential Revision: https://secure.phabricator.com/D4474 --- .../config/controller/PhabricatorConfigAllController.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/config/controller/PhabricatorConfigAllController.php b/src/applications/config/controller/PhabricatorConfigAllController.php index 8ae6f2098c..81b32c226b 100644 --- a/src/applications/config/controller/PhabricatorConfigAllController.php +++ b/src/applications/config/controller/PhabricatorConfigAllController.php @@ -9,6 +9,7 @@ final class PhabricatorConfigAllController $rows = array(); $options = PhabricatorApplicationConfigOptions::loadAllOptions(); + ksort($options); foreach ($options as $option) { $key = $option->getKey();