Initial work on issue sebastianbergmann#37.

Author: sebastianbergmann

PHP/CodeCoverage/Report/HTML/Renderer/File.php

@@ -291,14 +291,35 @@ protected function renderFunctionItems(array $functions, Text_Template $template
     protected function renderFunctionOrMethodItem(Text_Template $template, array $item, $indent = '')
     {
         $numTestedItems = $item['executedLines'] == $item['executableLines'] ? 1 : 0;
+        $visibility     = '';
+
+        if (isset($item['visibility'])) {
+            switch ($item['visibility']) {
+                case 'public': {
+                    $visibility = '+ ';
+                }
+                break;
+
+                case 'protected': {
+                    $visibility = '# ';
+                }
+                break;
+
+                case 'private': {
+                    $visibility = '- ';
+                }
+                break;
+            }
+        }
 
         return $this->renderItemTemplate(
           $template,
           array(
             'name'                         => sprintf(
-                                                '%s<a href="#%d">%s</a>',
+                                                '%s<a href="#%d">%s%s</a>',
                                                 $indent,
                                                 $item['startLine'],
+                                                $visibility,
                                                 htmlspecialchars($item['signature'])
                                               ),
             'numMethods'                   => 1,

PHP/CodeCoverage/Report/Node/File.php

@@ -571,7 +571,8 @@ protected function processClasses(PHP_Token_Stream $tokens)
                   'ccn'             => $method['ccn'],
                   'coverage'        => 0,
                   'crap'            => 0,
-                  'link'            => $link . $method['startLine']
+                  'link'            => $link . $method['startLine'],
+                  'visibility'      => $method['visibility']
                 );
 
                 $this->startLines[$method['startLine']] = &$this->classes[$className]['methods'][$methodName];

Tests/PHP/CodeCoverage/Report/FactoryTest.php

@@ -111,7 +111,8 @@ public function testSomething()
                   'ccn' => 1,
                   'coverage' => 100,
                   'crap' => '1',
-                  'link' => 'BankAccount.php.html#6'
+                  'link' => 'BankAccount.php.html#6',
+                  'visibility' => 'public'
                 ),
                 'setBalance' => array(
                   'signature' => 'setBalance($balance)',
@@ -122,7 +123,8 @@ public function testSomething()
                   'ccn' => 2,
                   'coverage' => 0,
                   'crap' => 6,
-                  'link' => 'BankAccount.php.html#11'
+                  'link' => 'BankAccount.php.html#11',
+                  'visibility' => 'protected'
                 ),
                 'depositMoney' => array(
                   'signature' => 'depositMoney($balance)',
@@ -133,7 +135,8 @@ public function testSomething()
                   'ccn' => 1,
                   'coverage' => 100,
                   'crap' => '1',
-                  'link' => 'BankAccount.php.html#20'
+                  'link' => 'BankAccount.php.html#20',
+                  'visibility' => 'public'
                 ),
                 'withdrawMoney' => array(
                   'signature' => 'withdrawMoney($balance)',
@@ -144,7 +147,8 @@ public function testSomething()
                   'ccn' => 1,
                   'coverage' => 100,
                   'crap' => '1',
-                  'link' => 'BankAccount.php.html#27'
+                  'link' => 'BankAccount.php.html#27',
+                  'visibility' => 'public'
                 ),
               ),
               'startLine' => 2,