phpmetrics / PhpMetrics

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");
     }
 
     /**

src/Hal/Report/Html/Reporter.php

Spacing +30 added lines, -30 removed lines

@@ -57,7 +57,7 @@ 
     {
         $this->config = $config;
         $this->output = $output;
-        $this->templateDir = __DIR__ . '/../../../../templates';
+        $this->templateDir = __DIR__.'/../../../../templates';
     }
 
     /**
@@ -82,52 +82,52 @@ 
         }
         $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);
         }
 
-        recurse_copy($this->templateDir . '/html_report/js', $logDir . '/js');
-        recurse_copy($this->templateDir . '/html_report/css', $logDir . '/css');
-        recurse_copy($this->templateDir . '/html_report/images', $logDir . '/images');
-        recurse_copy($this->templateDir . '/html_report/fonts', $logDir . '/fonts');
+        recurse_copy($this->templateDir.'/html_report/js', $logDir.'/js');
+        recurse_copy($this->templateDir.'/html_report/css', $logDir.'/css');
+        recurse_copy($this->templateDir.'/html_report/images', $logDir.'/images');
+        recurse_copy($this->templateDir.'/html_report/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage($this->templateDir . '/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/composer.php', $logDir . '/composer.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/composer.php', $logDir.'/composer.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage($this->templateDir . '/html_report/git.php', $logDir . '/git.html', $consolidated, $consolidatedGroups, $history);
+            $this->renderPage($this->templateDir.'/html_report/git.php', $logDir.'/git.html', $consolidated, $consolidatedGroups, $history);
         }
-        $this->renderPage($this->templateDir . '/html_report/junit.php', $logDir . '/junit.html', $consolidated, $consolidatedGroups, $history);
+        $this->renderPage($this->templateDir.'/html_report/junit.php', $logDir.'/junit.html', $consolidated, $consolidatedGroups, $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 . '/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         // HTML files to generate
@@ -148,7 +148,7 @@ 
 
         // consolidated by groups
         foreach ($consolidatedGroups as $name => $consolidated) {
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
             $this->currentGroup = $name;
             $this->assetPath = '../';
 
@@ -165,8 +165,8 @@ 
                 );
 
                 file_put_contents(
-                    $outDir . '/classes.js',
-                    'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+                    $outDir.'/classes.js',
+                    'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
                 );
             }
         }
@@ -238,7 +238,7 @@ 
         $goodOrBad = 'neutral';
         if ($newValue > $oldValue) {
             $r = 'gt';
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
             $goodOrBad = $lowIsBetter ? 'bad' : $goodOrBad;
             $goodOrBad = $highIsBetter ? 'good' : $goodOrBad;
         } elseif ($newValue < $oldValue) {