Implement link generation.

Author: sebastianbergmann

PHP/CodeCoverage/Report/Node.php

@@ -80,6 +80,10 @@ abstract class PHP_CodeCoverage_Report_Node implements Countable
      */
     public function __construct($name, PHP_CodeCoverage_Report_Node $parent = NULL)
     {
+        if (substr($name, -1) == '/') {
+            $name = substr($name, 0, -1);
+        }
+
         $this->name   = $name;
         $this->parent = $parent;
     }

PHP/CodeCoverage/Report/Node/File.php

@@ -485,6 +485,8 @@ protected function processClasses(PHP_Token_Stream $tokens)
         $classes = $tokens->getClasses();
         unset($tokens);
 
+        $link = PHP_CodeCoverage_Util::nodeToId($this) . '.html#';
+
         foreach ($classes as $className => $class) {
             $this->classes[$className] = array(
               'methods'         => array(),
@@ -494,7 +496,8 @@ protected function processClasses(PHP_Token_Stream $tokens)
               'ccn'             => 0,
               'coverage'        => 0,
               'crap'            => 0,
-              'package'         => $class['package']
+              'package'         => $class['package'],
+              'link'            => $link . $class['startLine']
             );
 
             $this->startLines[$class['startLine']] = &$this->classes[$className];
@@ -509,7 +512,8 @@ protected function processClasses(PHP_Token_Stream $tokens)
                   'executedLines'   => 0,
                   'ccn'             => $method['ccn'],
                   'coverage'        => 0,
-                  'crap'            => 0
+                  'crap'            => 0,
+                  'link'            => $link . $method['startLine']
                 );
 
                 $this->startLines[$method['startLine']] = &$this->classes[$className]['methods'][$methodName];

PHP/CodeCoverage/Util.php

@@ -85,6 +85,11 @@ class PHP_CodeCoverage_Util
       'setUp', 'assertPreConditions', 'assertPostConditions', 'tearDown'
     );
 
+    /**
+     * @var array
+     */
+    protected static $ids = array();
+
     /**
      * Builds an array representation of the directory structure.
      *
@@ -476,6 +481,32 @@ public static function reducePaths(&$files)
         return substr($commonPath, 0, -1);
     }
 
+    /**
+     * @param  PHP_CodeCoverage_Report_Node $node
+     * @return string
+     */
+    public static function nodeToId(PHP_CodeCoverage_Report_Node $node)
+    {
+        if (!isset(self::$ids[$node->getPath()])) {
+            $parent = $node->getParent();
+
+            if ($parent === NULL) {
+                self::$ids[$node->getPath()] = 'index';
+            } else {
+                $parentId = self::nodeToId($parent);
+
+                if ($parentId == 'index') {
+                    self::$ids[$node->getPath()] = $node->getName();
+                } else {
+                    self::$ids[$node->getPath()] = $parentId . '_' .
+                                                   $node->getName();
+                }
+            }
+        }
+
+        return self::$ids[$node->getPath()];
+    }
+
     /**
      * @param  string $coveredElement
      * @return array

Tests/PHP/CodeCoverage/Report/FactoryTest.php

@@ -73,8 +73,8 @@ public function testSomething()
     {
         $root = $this->getCoverageForBankAccount()->getReport();
 
-        $this->assertEquals('/usr/local/src/code-coverage/Tests/_files/', $root->getName());
-        $this->assertEquals('/usr/local/src/code-coverage/Tests/_files/', $root->getPath());
+        $this->assertEquals('/usr/local/src/code-coverage/Tests/_files', $root->getName());
+        $this->assertEquals('/usr/local/src/code-coverage/Tests/_files', $root->getPath());
         $this->assertEquals(10, $root->getNumExecutableLines());
         $this->assertEquals(5, $root->getNumExecutedLines());
         $this->assertEquals(1, $root->getNumClasses());
@@ -103,7 +103,8 @@ public function testSomething()
                   'executedLines' => 1,
                   'ccn' => 1,
                   'coverage' => 100,
-                  'crap' => '1'
+                  'crap' => '1',
+                  'link' => 'BankAccount.php.html#6'
                 ),
                 'setBalance' => array(
                   'signature' => 'setBalance($balance)',
@@ -113,7 +114,8 @@ public function testSomething()
                   'executedLines' => 0,
                   'ccn' => 2,
                   'coverage' => 0,
-                  'crap' => 6
+                  'crap' => 6,
+                  'link' => 'BankAccount.php.html#11'
                 ),
                 'depositMoney' => array(
                   'signature' => 'depositMoney($balance)',
@@ -123,7 +125,8 @@ public function testSomething()
                   'executedLines' => 2,
                   'ccn' => 1,
                   'coverage' => 100,
-                  'crap' => '1'
+                  'crap' => '1',
+                  'link' => 'BankAccount.php.html#20'
                 ),
                 'withdrawMoney' => array(
                   'signature' => 'withdrawMoney($balance)',
@@ -133,7 +136,8 @@ public function testSomething()
                   'executedLines' => 2,
                   'ccn' => 1,
                   'coverage' => 100,
-                  'crap' => '1'
+                  'crap' => '1',
+                  'link' => 'BankAccount.php.html#27'
                 ),
               ),
               'startLine' => 2,
@@ -148,7 +152,8 @@ public function testSomething()
                 'category' => '',
                 'package' => '',
                 'subpackage' => ''
-              )
+              ),
+              'link' => 'BankAccount.php.html#2'
             )
           ),
           $root->getClasses()