Added node:list sort order

Author: dantleech

CHANGELOG.md

@@ -8,6 +8,7 @@ dev-master
 ### Features
 
 - [profile:show] Added command to display current profile
+- [node:list] Sort properties by default, added option to specify
 
 ### Bug fixes
 

features/all/phpcr_node_list.feature

@@ -143,3 +143,19 @@ Feature: List properites and chidren of current nodeA
         """
         5 nodes in set
         """
+
+    Scenario: List with sort order ascending
+        When I execute the "node:list --sort=asc" command
+        Then the command should not fail
+
+    Scenario: List with sort order descending
+        When I execute the "node:list --sort=desc" command
+        Then the command should not fail
+
+    Scenario: List with no sorting
+        When I execute the "node:list --sort=none" command
+        Then the command should not fail
+
+    Scenario: List with invalid sorting
+        When I execute the "node:list --sort=foobar" command
+        Then the command should fail

src/PHPCR/Shell/Console/Command/Phpcr/NodeListCommand.php

@@ -23,6 +23,8 @@
 
 class NodeListCommand extends BasePhpcrCommand
 {
+    const SORT_OPTIONS = array('none', 'asc', 'desc');
+
     protected $formatter;
     protected $textHelper;
     protected $maxLevel;
@@ -38,6 +40,7 @@ protected function configure()
         $this->addOption('properties', null, InputOption::VALUE_NONE, 'List only the properties of this node');
         $this->addOption('level', 'L', InputOption::VALUE_REQUIRED, 'Depth of tree to show');
         $this->addOption('template', 't', InputOption::VALUE_NONE, 'Show template nodes and properties');
+        $this->addOption('sort', 's', InputOption::VALUE_REQUIRED, 'Sort properties, one of <comment>asc</comment>, <comment>desc</comment>, or <comment>none</comment>', 'asc');
         $this->setHelp(<<<HERE
 List both or one of the children and properties of this node.
 
@@ -61,6 +64,15 @@ public function execute(InputInterface $input, OutputInterface $output)
         $this->formatter = $this->get('helper.result_formatter');
         $this->textHelper = $this->get('helper.text');
         $this->maxLevel = $input->getOption('level');
+        $this->sort = strtolower($input->getOption('sort'));
+
+        if (!in_array($this->sort, self::SORT_OPTIONS)) {
+            throw new \InvalidArgumentException(sprintf(
+                'Sort must be one of "%s". "%s" given',
+                implode('", "', self::SORT_OPTIONS),
+                $this->sort
+            ));
+        }
 
         $this->showChildren = $input->getOption('children');
         $this->showProperties = $input->getOption('properties');
@@ -133,7 +145,7 @@ private function renderNode($currentNode, $table, $spacers = array(), $filter =
     private function renderChildren($currentNode, $table, $spacers, $filter = null)
     {
         $start = microtime(true);
-        $children = $currentNode->getNodes($filter ?: null);
+        $children = (array) $currentNode->getNodes($filter ?: null);
         $this->time += microtime(true) - $start;
 
         $nodeType = $currentNode->getPrimaryNodeType();
@@ -198,7 +210,8 @@ private function renderChildren($currentNode, $table, $spacers, $filter = null)
 
     private function renderProperties($currentNode, $table, $spacers, $filter = null)
     {
-        $properties = $currentNode->getProperties($filter ?: null);
+        $properties = (array) $currentNode->getProperties($filter ?: null);
+        $properties = $this->sort($properties);
 
         try {
             $primaryItem = $currentNode->getPrimaryItem();
@@ -248,4 +261,19 @@ private function renderProperties($currentNode, $table, $spacers, $filter = null
             }
         }
     }
+
+    private function sort($array)
+    {
+        switch ($this->sort) {
+            case 'asc':
+                ksort($array);
+                return $array;
+            case 'desc':
+                ksort($array);
+                $array = array_reverse($array);
+                return $array;
+        }
+
+        return $array;
+    }
 }