Reorder methods.

Author: sebastianbergmann

PHP/CodeCoverage/Report/HTML.php

@@ -147,6 +147,53 @@ public function process(PHP_CodeCoverage $coverage, $target)
         $this->copyFiles($target);
     }
 
+    /**
+     * @param PHP_CodeCoverage_Report_HTML_Node_Directory $root
+     * @param array                                       $items
+     * @param array                                       $files
+     */
+    protected function addItems(PHP_CodeCoverage_Report_HTML_Node_Directory $root, array $items, array $files)
+    {
+        foreach ($items as $key => $value) {
+            if (substr($key, -2) == '/f') {
+                try {
+                    $file = $root->addFile(
+                      substr($key, 0, -2),
+                      $value,
+                      $this->options['yui'],
+                      $this->options['highlight']
+                    );
+                }
+
+                catch (RuntimeException $e) {
+                    continue;
+                }
+            } else {
+                $child = $root->addDirectory($key);
+                $this->addItems($child, $value, $files);
+            }
+        }
+    }
+
+    /**
+     * Returns the classes.
+     *
+     * @param  PHP_CodeCoverage_Report_HTML_Node_Directory $root
+     * @return array
+     */
+    protected function classes(PHP_CodeCoverage_Report_HTML_Node_Directory $root)
+    {
+        $classes = array();
+
+        foreach ($root as $node) {
+            $classes = array_merge($classes, $node->getClasses());
+        }
+
+        ksort($classes);
+
+        return $classes;
+    }
+
     /**
      * Renders the Class Coverage Distribution chart.
      *
@@ -232,42 +279,26 @@ protected function classCoverageDistributionChart(array $classes, $target)
     }
 
     /**
-     * Returns the classes.
-     *
-     * @param  PHP_CodeCoverage_Report_HTML_Node_Directory $root
-     * @return array
-     */
-    protected function classes(PHP_CodeCoverage_Report_HTML_Node_Directory $root)
-    {
-        $classes = array();
-
-        foreach ($root as $node) {
-            $classes = array_merge($classes, $node->getClasses());
-        }
-
-        ksort($classes);
-
-        return $classes;
-    }
-
-    /**
-     * Returns the top project risks according to the CRAP index.
-     *
-     * @param  array   $classes
-     * @param  integer $max
-     * @return array
+     * @param string $target
      */
-    protected function topProjectRisks(array $classes, $max = 20)
+    protected function copyFiles($target)
     {
-        $risks = array();
+        $files = array(
+          'butter.png',
+          'chameleon.png',
+          'close12_1.gif',
+          'container.css',
+          'container-min.js',
+          'glass.png',
+          'scarlet_red.png',
+          'snow.png',
+          'style.css',
+          'yahoo-dom-event.js'
+        );
 
-        foreach ($classes as $className => $class) {
-            $risks[$className] = $class['crap'];
+        foreach ($files as $file) {
+            copy(self::$templatePath . $file, $target . $file);
         }
-
-        asort($risks);
-
-        return array_reverse(array_slice($risks, 0, max($max, count($risks))));
     }
 
     /**
@@ -301,53 +332,22 @@ protected function leastTestedMethods(array $classes, $max = 20)
     }
 
     /**
-     * @param PHP_CodeCoverage_Report_HTML_Node_Directory $root
-     * @param array                                       $items
-     * @param array                                       $files
+     * Returns the top project risks according to the CRAP index.
+     *
+     * @param  array   $classes
+     * @param  integer $max
+     * @return array
      */
-    protected function addItems(PHP_CodeCoverage_Report_HTML_Node_Directory $root, array $items, array $files)
+    protected function topProjectRisks(array $classes, $max = 20)
     {
-        foreach ($items as $key => $value) {
-            if (substr($key, -2) == '/f') {
-                try {
-                    $file = $root->addFile(
-                      substr($key, 0, -2),
-                      $value,
-                      $this->options['yui'],
-                      $this->options['highlight']
-                    );
-                }
+        $risks = array();
 
-                catch (RuntimeException $e) {
-                    continue;
-                }
-            } else {
-                $child = $root->addDirectory($key);
-                $this->addItems($child, $value, $files);
-            }
+        foreach ($classes as $className => $class) {
+            $risks[$className] = $class['crap'];
         }
-    }
 
-    /**
-     * @param string $target
-     */
-    protected function copyFiles($target)
-    {
-        $files = array(
-          'butter.png',
-          'chameleon.png',
-          'close12_1.gif',
-          'container.css',
-          'container-min.js',
-          'glass.png',
-          'scarlet_red.png',
-          'snow.png',
-          'style.css',
-          'yahoo-dom-event.js'
-        );
+        asort($risks);
 
-        foreach ($files as $file) {
-            copy(self::$templatePath . $file, $target . $file);
-        }
+        return array_reverse(array_slice($risks, 0, max($max, count($risks))));
     }
 }