Fix sorting and collapsing of dump output

I thought the panel layout was used. It is not.
I forgot to apply variable sorting when switching dump implementations.

Fixes #771

Author: markstory

src/View/Helper/ToolbarHelper.php

@@ -72,6 +72,11 @@ public function dump($value)
             $restore = true;
             $debugger->setConfig('exportFormatter', HtmlFormatter::class);
         }
+
+        if ($this->sort && is_array($value)) {
+            ksort($value);
+        }
+
         $contents = Debugger::exportVar($value, 25);
         if ($restore) {
             $debugger->setConfig('exportFormatter', $exportFormatter);

templates/layout/panel.php

@@ -23,6 +23,7 @@
 use Cake\View\View;
 use DebugKit\View\Helper\ToolbarHelper;
 use stdClass;
+use SimpleXmlElement;
 
 /**
  * Class ToolbarHelperTestCase
@@ -78,10 +79,30 @@ public function testDumpCoerceHtml()
             Debugger::configInstance('exportFormatter'),
             'Should restore setting'
         );
+
         // Restore back to original value.
         Debugger::configInstance('exportFormatter', $restore);
     }
 
+    public function testDumpSorted()
+    {
+        $getValues = function ($el) {
+            return (string)$el;
+        };
+        $path = '//*[@class="cake-dbg-array-item"]/*[@class="cake-dbg-string"]';
+        $data = ['z' => 1, 'a' => 99, 'm' => 123];
+        $result = $this->Toolbar->dump($data);
+        $xml = new SimpleXmlElement($result);
+        $elements = $xml->xpath($path);
+        $this->assertSame(["'z'", "'a'", "'m'"], array_map($getValues, $elements));
+
+        $this->Toolbar->setSort(true);
+        $result = $this->Toolbar->dump($data);
+        $xml = new SimpleXmlElement($result);
+        $elements = $xml->xpath($path);
+        $this->assertSame(["'a'", "'m'", "'z'"], array_map($getValues, $elements));
+    }
+
     /**
      * Test makeNeatArray with basic types.
      *

tests/TestCase/View/Helper/ToolbarHelperTest.php

@@ -150,11 +150,14 @@ Toolbar.prototype = {
       // Slide panel into place - css transitions.
       _this.content.addClass('enabled');
       contentArea.html(response);
+      _this.bindVariableSort();
+      _this.bindDebugBlock();
+
       _this.bindNeatArray();
     });
   },
 
-  bindNeatArray: function() {
+  bindVariableSort: function() {
     var sortButton = this.content.find('.neat-array-sort');
     var _this = this;
     sortButton.click(function() {
@@ -165,7 +168,16 @@ Toolbar.prototype = {
       }
       _this.loadPanel(_this.currentPanel());
     });
+  },
+
+  bindDebugBlock: function () {
+    if (window.__cakeDebugBlockInit) {
+        window.__cakeDebugBlockInit();
+    }
+  },
 
+  bindNeatArray: function() {
+    var _this = this;
     var lists = this.content.find('.depth-0');
     lists.find('ul').hide()
       .parent().addClass('expandable collapsed');