Fixed sorting of cached scripts

Author: PeeHaa

src/OpCacheGUI/OpCache/Status.php

@@ -260,6 +260,21 @@ public function getCachedScripts()
             ];
         }
 
+        usort($scripts, [$this, 'sortCachedScripts']);
+
         return $scripts;
     }
+
+    /**
+     * Sorts the lists of cached scripts
+     *
+     * @param array $a Array to compare
+     * @param array $b Array to compare
+     *
+     * @return int The direction of the sort
+     */
+    private function sortCachedScripts(array $a, array $b)
+    {
+        return strcmp($a['full_path'], $b['full_path']);
+    }
 }

test/Unit/OpCache/StatusTest.php

@@ -319,6 +319,7 @@ public function testGetCachedScriptsEmpty()
     /**
      * @covers OpCacheGUI\OpCache\Status::__construct
      * @covers OpCacheGUI\OpCache\Status::getCachedScripts
+     * @covers OpCacheGUI\OpCache\Status::sortCachedScripts
      */
     public function testGetCachedScriptsFilled()
     {
@@ -328,6 +329,13 @@ public function testGetCachedScriptsFilled()
         $status = new Status($formatter, $this->statusData);
 
         $data = [
+            [
+                'full_path'           => '/var/www/vhosts/NoTimeStamp/src/Psr/Autoloader.php',
+                'hits'                => 12876,
+                'memory_consumption'  => '6KB',
+                'last_used_timestamp' => '14:25:15 09-10-2014',
+                'timestamp'           => 'N/A',
+            ],
             [
                 'full_path'           => '/var/www/vhosts/OpcacheGUI/src/OpCacheGUI/Network/Request.php',
                 'hits'                => 1,
@@ -342,34 +350,27 @@ public function testGetCachedScriptsFilled()
                 'last_used_timestamp' => '14:08:35 09-10-2014',
                 'timestamp'           => '16:20:53 07-10-2014',
             ],
-            [
-                'full_path'           => '/var/www/vhosts/SomeOtherProject/src/Foo.php',
-                'hits'                => 19,
-                'memory_consumption'  => '3KB',
-                'last_used_timestamp' => '15:31:55 09-10-2014',
-                'timestamp'           => '20:07:33 08-10-2014',
-            ],
             [
                 'full_path'           => '/var/www/vhosts/RedTube/template/humiliation/germany-vs-brazil.phtml',
                 'hits'                => 71,
                 'memory_consumption'  => '4KB',
                 'last_used_timestamp' => '14:25:15 09-10-2014',
                 'timestamp'           => '20:01:15 08-10-2014',
             ],
+            [
+                'full_path'           => '/var/www/vhosts/SomeOtherProject/src/Foo.php',
+                'hits'                => 19,
+                'memory_consumption'  => '3KB',
+                'last_used_timestamp' => '15:31:55 09-10-2014',
+                'timestamp'           => '20:07:33 08-10-2014',
+            ],
             [
                 'full_path'           => '/var/www/vhosts/SomeOtherProject/src/Psr/Autoloader.php',
                 'hits'                => 32,
                 'memory_consumption'  => '5KB',
                 'last_used_timestamp' => '14:25:15 09-10-2014',
                 'timestamp'           => '20:01:15 08-10-2014',
             ],
-            [
-                'full_path'           => '/var/www/vhosts/NoTimeStamp/src/Psr/Autoloader.php',
-                'hits'                => 12876,
-                'memory_consumption'  => '6KB',
-                'last_used_timestamp' => '14:25:15 09-10-2014',
-                'timestamp'           => 'N/A',
-            ],
         ];
 
         $this->assertSame($data, $status->getCachedScripts());