Closes sebastianbergmann#287

Author: sebastianbergmann

src/CodeCoverage/Report/HTML/Renderer/Dashboard.php

@@ -62,17 +62,19 @@ class PHP_CodeCoverage_Report_HTML_Renderer_Dashboard extends PHP_CodeCoverage_R
      */
     public function render(PHP_CodeCoverage_Report_Node_Directory $node, $file)
     {
-        $classes  = $node->getClassesAndTraits();
+        $classes = $node->getClassesAndTraits();
+        $isRoot  = $node->getParent() === null;
+
         $template = new Text_Template(
             $this->templatePath . 'dashboard.html', '{{', '}}'
         );
 
         $this->setCommonTemplateVariables($template, $node);
 
-        $complexity           = $this->complexity($classes);
-        $coverageDistribution = $this->coverageDistribution($classes);
-        $insufficientCoverage = $this->insufficientCoverage($classes);
-        $projectRisks         = $this->projectRisks($classes);
+        $complexity           = $this->complexity($classes, $isRoot);
+        $coverageDistribution = $this->coverageDistribution($classes, $isRoot);
+        $insufficientCoverage = $this->insufficientCoverage($classes, $isRoot);
+        $projectRisks         = $this->projectRisks($classes, $isRoot);
 
         $template->setVar(
             array(
@@ -93,10 +95,11 @@ public function render(PHP_CodeCoverage_Report_Node_Directory $node, $file)
     /**
      * Returns the data for the Class/Method Complexity charts.
      *
-     * @param  array $classes
+     * @param  array   $classes
+     * @param  boolean $isRoot
      * @return array
      */
-    protected function complexity(array $classes)
+    protected function complexity(array $classes, $isRoot)
     {
         $result = array('class' => array(), 'method' => array());
 
@@ -111,7 +114,7 @@ protected function complexity(array $classes)
                     $method['ccn'],
                     sprintf(
                         '<a href="%s">%s</a>',
-                        $method['link'],
+                        $this->rebaseLink($method['link'], $isRoot),
                         $methodName
                     )
                 );
@@ -122,7 +125,7 @@ protected function complexity(array $classes)
                 $class['ccn'],
                 sprintf(
                     '<a href="%s">%s</a>',
-                    $class['link'],
+                    $this->rebaseLink($class['link'], $isRoot),
                     $className
                 )
             );
@@ -137,10 +140,11 @@ protected function complexity(array $classes)
     /**
      * Returns the data for the Class / Method Coverage Distribution chart.
      *
-     * @param  array $classes
+     * @param  array   $classes
+     * @param  boolean $isRoot
      * @return array
      */
-    protected function coverageDistribution(array $classes)
+    protected function coverageDistribution(array $classes, $isRoot)
     {
         $result = array(
             'class' => array(
@@ -206,10 +210,11 @@ protected function coverageDistribution(array $classes)
     /**
      * Returns the classes / methods with insufficient coverage.
      *
-     * @param  array $classes
+     * @param  array   $classes
+     * @param  boolean $isRoot
      * @return array
      */
-    protected function insufficientCoverage(array $classes)
+    protected function insufficientCoverage(array $classes, $isRoot)
     {
         $leastTestedClasses = array();
         $leastTestedMethods = array();
@@ -239,7 +244,7 @@ protected function insufficientCoverage(array $classes)
         foreach ($leastTestedClasses as $className => $coverage) {
             $result['class'] .= sprintf(
                 '       <tr><td><a href="%s">%s</a></td><td class="text-right">%d%%</td></tr>' . "\n",
-                $classes[$className]['link'],
+                $this->rebaseLink($classes[$className]['link'], $isRoot),
                 $className,
                 $coverage
             );
@@ -250,7 +255,7 @@ protected function insufficientCoverage(array $classes)
 
             $result['method'] .= sprintf(
                 '       <tr><td><a href="%s"><abbr title="%s">%s</a></a></td><td class="text-right">%d%%</td></tr>' . "\n",
-                $classes[$class]['methods'][$method]['link'],
+                $this->rebaseLink($classes[$class]['methods'][$method]['link'], $isRoot),
                 $methodName,
                 $method,
                 $coverage
@@ -263,10 +268,11 @@ protected function insufficientCoverage(array $classes)
     /**
      * Returns the project risks according to the CRAP index.
      *
-     * @param  array $classes
+     * @param  array   $classes
+     * @param  boolean $isRoot
      * @return array
      */
-    protected function projectRisks(array $classes)
+    protected function projectRisks(array $classes, $isRoot)
     {
         $classRisks  = array();
         $methodRisks = array();
@@ -298,7 +304,7 @@ protected function projectRisks(array $classes)
         foreach ($classRisks as $className => $crap) {
             $result['class'] .= sprintf(
                 '       <tr><td><a href="%s">%s</a></td><td class="text-right">%d</td></tr>' . "\n",
-                $classes[$className]['link'],
+                $this->rebaseLink($classes[$className]['link'], $isRoot),
                 $className,
                 $crap
             );
@@ -309,7 +315,7 @@ protected function projectRisks(array $classes)
 
             $result['method'] .= sprintf(
                 '       <tr><td><a href="%s"><abbr title="%s">%s</abbr></a></td><td class="text-right">%d</td></tr>' . "\n",
-                $classes[$class]['methods'][$method]['link'],
+                $this->rebaseLink($classes[$class]['methods'][$method]['link'], $isRoot),
                 $methodName,
                 $method,
                 $crap
@@ -327,4 +333,15 @@ protected function getActiveBreadcrumb(PHP_CodeCoverage_Report_Node $node)
             $node->getName()
         );
     }
+
+    /**
+     * @param  string  $link
+     * @param  boolean $isRoot
+     * @return string
+     * @since  Method available since Release 2.0.12
+     */
+    private function rebaseLink($link, $isRoot)
+    {
+        return $isRoot ? $link : substr($link, strpos($link, '/') + 1);
+    }
 }