Make methods private

Author: sebastianbergmann

src/Report/Html/Renderer/Dashboard.php

@@ -55,10 +55,19 @@ public function render(DirectoryNode $node, string $file): void
         $template->renderTo($file);
     }
 
+    protected function activeBreadcrumb(AbstractNode $node): string
+    {
+        return \sprintf(
+            '         <li class="breadcrumb-item"><a href="index.html">%s</a></li>' . "\n" .
+            '         <li class="breadcrumb-item active">(Dashboard)</li>' . "\n",
+            $node->name()
+        );
+    }
+
     /**
      * Returns the data for the Class/Method Complexity charts.
      */
-    protected function complexity(array $classes, string $baseLink): array
+    private function complexity(array $classes, string $baseLink): array
     {
         $result = ['class' => [], 'method' => []];
 
@@ -99,7 +108,7 @@ protected function complexity(array $classes, string $baseLink): array
     /**
      * Returns the data for the Class / Method Coverage Distribution chart.
      */
-    protected function coverageDistribution(array $classes): array
+    private function coverageDistribution(array $classes): array
     {
         $result = [
             'class' => [
@@ -165,7 +174,7 @@ protected function coverageDistribution(array $classes): array
     /**
      * Returns the classes / methods with insufficient coverage.
      */
-    protected function insufficientCoverage(array $classes, string $baseLink): array
+    private function insufficientCoverage(array $classes, string $baseLink): array
     {
         $leastTestedClasses = [];
         $leastTestedMethods = [];
@@ -219,7 +228,7 @@ protected function insufficientCoverage(array $classes, string $baseLink): array
     /**
      * Returns the project risks according to the CRAP index.
      */
-    protected function projectRisks(array $classes, string $baseLink): array
+    private function projectRisks(array $classes, string $baseLink): array
     {
         $classRisks  = [];
         $methodRisks = [];
@@ -270,13 +279,4 @@ protected function projectRisks(array $classes, string $baseLink): array
 
         return $result;
     }
-
-    protected function activeBreadcrumb(AbstractNode $node): string
-    {
-        return \sprintf(
-            '         <li class="breadcrumb-item"><a href="index.html">%s</a></li>' . "\n" .
-            '         <li class="breadcrumb-item active">(Dashboard)</li>' . "\n",
-            $node->name()
-        );
-    }
 }

src/Report/Html/Renderer/Directory.php

@@ -48,7 +48,7 @@ public function render(DirectoryNode $node, string $file): void
         $template->renderTo($file);
     }
 
-    protected function renderItem(Node $node, bool $total = false): string
+    private function renderItem(Node $node, bool $total = false): string
     {
         $data = [
             'numClasses'                      => $node->numberOfClassesAndTraits(),

src/Report/Html/Renderer/File.php

@@ -43,7 +43,7 @@ public function render(FileNode $node, string $file): void
         $template->renderTo($file);
     }
 
-    protected function renderItems(FileNode $node): string
+    private function renderItems(FileNode $node): string
     {
         $templateName = $this->templatePath . ($this->hasBranchCoverage ? 'file_item_branch.html' : 'file_item.html');
         $template     = new Template($templateName, '{{', '}}');
@@ -103,7 +103,7 @@ protected function renderItems(FileNode $node): string
         return $items;
     }
 
-    protected function renderTraitOrClassItems(array $items, Template $template, Template $methodItemTemplate): string
+    private function renderTraitOrClassItems(array $items, Template $template, Template $methodItemTemplate): string
     {
         $buffer = '';
 
@@ -207,7 +207,7 @@ protected function renderTraitOrClassItems(array $items, Template $template, Tem
         return $buffer;
     }
 
-    protected function renderFunctionItems(array $functions, Template $template): string
+    private function renderFunctionItems(array $functions, Template $template): string
     {
         if (empty($functions)) {
             return '';
@@ -225,7 +225,7 @@ protected function renderFunctionItems(array $functions, Template $template): st
         return $buffer;
     }
 
-    protected function renderFunctionOrMethodItem(Template $template, array $item, string $indent = ''): string
+    private function renderFunctionOrMethodItem(Template $template, array $item, string $indent = ''): string
     {
         $numMethods       = 0;
         $numTestedMethods = 0;
@@ -289,7 +289,7 @@ protected function renderFunctionOrMethodItem(Template $template, array $item, s
         );
     }
 
-    protected function renderSource(FileNode $node): string
+    private function renderSource(FileNode $node): string
     {
         $coverageData = $node->lineCoverageData();
         $testData     = $node->testData();
@@ -408,7 +408,7 @@ protected function renderSource(FileNode $node): string
     /**
      * @param string $file
      */
-    protected function loadFile($file): array
+    private function loadFile($file): array
     {
         $buffer              = \file_get_contents($file);
         $tokens              = \token_get_all($buffer);