phpmetrics / PhpMetrics

src/Hal/Report/Html/Reporter.php

Doc Comments +2 added lines, -2 removed lines

@@ -116,8 +116,8 @@
     }
 
     /**
-     * @param $source
-     * @param $destination
+     * @param string $source
+     * @param string $destination
      * @param Consolidated $consolidated
      * @param $history
      * @return $this

Spacing +21 added lines, -21 removed lines

@@ -51,46 +51,46 @@ 
 
         $consolidated = new Consolidated($metrics);
 
-        $files = glob($logDir . '/js/history-*.json');
+        $files = glob($logDir.'/js/history-*.json');
         natsort($files);
-        $history = array_map(static function ($filename) {
+        $history = array_map(static function($filename) {
             return json_decode(file_get_contents($filename));
         }, $files);
 
         foreach (['js', 'css', 'images', 'fonts'] as $subFolder) {
-            $folder = $logDir . '/' . $subFolder;
+            $folder = $logDir.'/'.$subFolder;
             if (!file_exists($folder)) {
                 mkdir($folder, 0755, true);
             }
-            recurse_copy(__DIR__ . '/template/' . $subFolder, $folder);
+            recurse_copy(__DIR__.'/template/'.$subFolder, $folder);
         }
 
         // render dynamic pages
-        $this->renderPage(__DIR__ . '/template/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(__DIR__ . '/template/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(__DIR__.'/template/git.php', $logDir.'/git.html', $consolidated, $history);
         }
-        $this->renderPage(__DIR__ . '/template/junit.php', $logDir . '/junit.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/junit.php', $logDir.'/junit.html', $consolidated, $history);
 
-        $today = (object)['avg' => $consolidated->getAvg(), 'sum' => $consolidated->getSum()];
+        $today = (object) ['avg' => $consolidated->getAvg(), 'sum' => $consolidated->getSum()];
         $encodedToday = json_encode($today, JSON_PRETTY_PRINT);
         $next = count($history) + 1;
         file_put_contents(sprintf('%s/js/history-%d.json', $logDir, $next), $encodedToday);
         file_put_contents(sprintf('%s/js/latest.json', $logDir), $encodedToday);
 
         file_put_contents(
-            $logDir . '/js/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/js/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         $this->output->writeln(sprintf('HTML report generated in "%s" directory', $logDir));
@@ -156,7 +156,7 @@ 
         $goodOrBad = 'neutral';
         if ($newValue > $oldValue) {
             $r = 'gt';
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
             $goodOrBad = $lowIsBetter ? 'bad' : $goodOrBad;
             $goodOrBad = $highIsBetter ? 'good' : $goodOrBad;
         } elseif ($newValue < $oldValue) {

src/Hal/Report/Csv/Reporter.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 
         $logDir = dirname($logFile);
         if (!file_exists($logDir) || !is_writable($logDir)) {
-            throw new RuntimeException('You do not have permissions to write CSV report in ' . $logFile);
+            throw new RuntimeException('You do not have permissions to write CSV report in '.$logFile);
         }
 
         $availableMetrics = (new Registry())->allForStructures();
@@ -59,9 +59,9 @@ 
             if (!$metric instanceof ClassMetric) {
                 continue;
             }
-            $row = array_map(static function ($key) use ($metric) {
+            $row = array_map(static function($key) use ($metric) {
                 $data = $metric->get($key);
-                return (!is_scalar($data)) ? 'N/A': $data;
+                return (!is_scalar($data)) ? 'N/A' : $data;
             }, $availableMetrics);
 
             fputcsv($logPointer, $row);

src/Hal/Report/Json/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 
         $logDir = dirname($logFile);
         if (!file_exists($logDir) || !is_writable($logDir)) {
-            throw new RuntimeException('You do not have permissions to write JSON report in ' . $logFile);
+            throw new RuntimeException('You do not have permissions to write JSON report in '.$logFile);
         }
 
         file_put_contents($logFile, json_encode($metrics, JSON_PRETTY_PRINT));

src/Hal/Report/Cli/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
             $out .= $this->reportUnitTesting($metrics->get('unitTesting'));
         }
 
-        $this->output->write($out . "\n");
+        $this->output->write($out."\n");
     }
 
     /**