phpmetrics / PhpMetrics

src/Hal/Component/Output/CliOutput.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function writeln($message)
     {
-        $this->write(PHP_EOL . $message);
+        $this->write(PHP_EOL.$message);
         return $this;
     }
 
@@ -53,7 +53,7 @@ 
                     break;
             }
 
-            $message = $color . $message . "\033[0m";
+            $message = $color.$message."\033[0m";
         }
 
         $this->quietMode || file_put_contents('php://stdout', $message);
@@ -96,7 +96,7 @@ 
             return
                 0 >= version_compare(
                     '10.0.10586',
-                    PHP_WINDOWS_VERSION_MAJOR . '.' . PHP_WINDOWS_VERSION_MINOR . '.' . PHP_WINDOWS_VERSION_BUILD
+                    PHP_WINDOWS_VERSION_MAJOR.'.'.PHP_WINDOWS_VERSION_MINOR.'.'.PHP_WINDOWS_VERSION_BUILD
                 )
                 || false !== getenv('ANSICON')
                 || 'ON' === getenv('ConEmuANSI')

src/Hal/Application/Config/File/ConfigFileReaderJson.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
     private function resolvePath($path)
     {
         if (DIRECTORY_SEPARATOR !== $path[0]) {
-            $path = dirname($this->filename) . DIRECTORY_SEPARATOR . $path;
+            $path = dirname($this->filename).DIRECTORY_SEPARATOR.$path;
         }
 
         return $path;
@@ -78,7 +78,7 @@ 
 
         // Composer
         if (array_key_exists('composer', $jsonData)) {
-            $config->set('composer', (bool)$jsonData['composer']);
+            $config->set('composer', (bool) $jsonData['composer']);
         }
 
         // Search
@@ -111,7 +111,7 @@ 
         if (isset($jsonData['report']) && is_array($jsonData['report'])) {
             foreach ($jsonData['report'] as $reportType => $path) {
                 $path = $this->resolvePath($path);
-                $config->set('report-' . $reportType, $path);
+                $config->set('report-'.$reportType, $path);
             }
         }
     }

src/Hal/Metric/Class_/Coupling/ExternalsVisitor.php

Spacing +14 added lines, -14 removed lines

@@ -59,21 +59,21 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        $this->pushToDependencies($dependencies, (string)$interface);
-                        array_push($parents, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        $this->pushToDependencies($dependencies, (string) $interface);
+                        array_push($parents, (string) $interface);
                     }
                 } else {
-                    $this->pushToDependencies($dependencies, (string)$node->extends);
-                    array_push($parents, (string)$node->extends);
+                    $this->pushToDependencies($dependencies, (string) $node->extends);
+                    array_push($parents, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    $this->pushToDependencies($dependencies, (string)$interface);
-                    array_push($interfaces, (string)$interface);
+                    $this->pushToDependencies($dependencies, (string) $interface);
+                    array_push($interfaces, (string) $interface);
                 }
             }
 
@@ -82,7 +82,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            $this->pushToDependencies($dependencies, (string)$stmt->returnType);
+                            $this->pushToDependencies($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -90,13 +90,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                $this->pushToDependencies($dependencies, (string)$param->type);
+                                $this->pushToDependencies($dependencies, (string) $param->type);
                             }
                         }
                     }
 
                     // instantiations, static calls
-                    \iterate_over_node($stmt, function ($node) use (&$dependencies) {
+                    \iterate_over_node($stmt, function($node) use (&$dependencies) {
                         switch (true) {
                             case $node instanceof Node\Expr\New_:
                                 // new MyClass
@@ -115,13 +115,13 @@ 
                         foreach ($matches[1] as $check) {
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
-                                    if (((string)$use->getAlias()) === $check) {
-                                        $this->pushToDependencies($dependencies, (string)($use->name));
+                                    if (((string) $use->getAlias()) === $check) {
+                                        $this->pushToDependencies($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    $this->pushToDependencies($dependencies, (string)($use->name));
+                                    $this->pushToDependencies($dependencies, (string) ($use->name));
                                 }
                             }
                         }
@@ -141,6 +141,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

src/Hal/Metric/Class_/Structural/LcomVisitor.php

Spacing +5 added lines, -5 removed lines

@@ -52,12 +52,12 @@ 
                         continue;
                     }
 
-                    if (!$graph->has($stmt->name . '()')) {
-                        $graph->insert(new TreeNode($stmt->name . '()'));
+                    if (!$graph->has($stmt->name.'()')) {
+                        $graph->insert(new TreeNode($stmt->name.'()'));
                     }
-                    $from = $graph->get($stmt->name . '()');
+                    $from = $graph->get($stmt->name.'()');
 
-                    \iterate_over_node($stmt, function ($node) use ($from, &$graph) {
+                    \iterate_over_node($stmt, function($node) use ($from, &$graph) {
                         if ($node instanceof Node\Expr\PropertyFetch && isset($node->var->name) && $node->var->name == 'this') {
                             $name = getNameOfNode($node);
                             // use of attribute $this->xxx;
@@ -73,7 +73,7 @@ 
                             if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && getNameOfNode($node->var) === 'this') {
                                 // use of method call $this->xxx();
                                 // use of attribute $this->xxx;
-                                $name = getNameOfNode($node->name) . '()';
+                                $name = getNameOfNode($node->name).'()';
                                 if (!$graph->has($name)) {
                                     $graph->insert(new TreeNode($name));
                                 }

src/Hal/Metric/Class_/Complexity/CyclomaticComplexityVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
                     }
 
                     // iterate over children, recursively
-                    $cb = function ($node) use (&$cb) {
+                    $cb = function($node) use (&$cb) {
                         $ccn = 0;
 
                         foreach (get_object_vars($node) as $name => $member) {

src/Hal/Report/Cli/SearchReporter.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         );
 
         $config = $this->config->get('searches')->get($searchName)->getConfig();
-        if(!empty($foundSearch) && !empty($config->failIfFound) && true === $config->failIfFound) {
+        if (!empty($foundSearch) && !empty($config->failIfFound) && true === $config->failIfFound) {
             $title = sprintf(
                 '<error>[ERR] Found %d occurrences for search "%s"</error>',
                 sizeof($foundSearch),

src/Hal/Report/Violations/Xml/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
     public function generate(Metrics $metrics)
     {
-        if(!class_exists('\DOMDocument')) {
+        if (!class_exists('\DOMDocument')) {
             $this->output->writeln('<error>The DOM extension is not available. Please install it if you want to use the Xml Violations report.</error>');
             return;
         }

src/Hal/Report/Html/Reporter.php

Spacing +35 added lines, -35 removed lines

@@ -54,7 +54,7 @@ 
     {
         $this->config = $config;
         $this->output = $output;
-        $this->templateDir = __DIR__ . '/../../../../templates';
+        $this->templateDir = __DIR__.'/../../../../templates';
     }
 
 
@@ -79,11 +79,11 @@ 
         $consolidated = new Consolidated($metrics);
 
         // history of builds
-        $today = (object)[
+        $today = (object) [
             'avg' => $consolidated->getAvg(),
             'sum' => $consolidated->getSum()
         ];
-        $files = glob($logDir . '/js/history-*.json');
+        $files = glob($logDir.'/js/history-*.json');
         $next = count($files) + 1;
         $history = [];
         natsort($files);
@@ -92,45 +92,45 @@ 
         }
 
         // copy sources
-        if (!file_exists($logDir . '/js')) {
-            mkdir($logDir . '/js', 0755, true);
+        if (!file_exists($logDir.'/js')) {
+            mkdir($logDir.'/js', 0755, true);
         }
-        if (!file_exists($logDir . '/css')) {
-            mkdir($logDir . '/css', 0755, true);
+        if (!file_exists($logDir.'/css')) {
+            mkdir($logDir.'/css', 0755, true);
         }
-        if (!file_exists($logDir . '/images')) {
-            mkdir($logDir . '/images', 0755, true);
+        if (!file_exists($logDir.'/images')) {
+            mkdir($logDir.'/images', 0755, true);
         }
-        if (!file_exists($logDir . '/fonts')) {
-            mkdir($logDir . '/fonts', 0755, true);
+        if (!file_exists($logDir.'/fonts')) {
+            mkdir($logDir.'/fonts', 0755, true);
         }
 
         if (!is_writable($logDir)) {
             throw new RuntimeExceptionAlias(sprintf('Unable to write in the directory "%s"', $logDir));
         }
 
-        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);
 
         // js data
         file_put_contents(
@@ -144,8 +144,8 @@ 
 
         // json data
         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
@@ -166,7 +166,7 @@ 
 
         // consolidated by groups
         foreach ($consolidatedGroups as $name => $consolidated) {
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
             $this->currentGroup = $name;
             $this->assetPath = '../';
 
@@ -183,8 +183,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)
                 );
             }
         }
@@ -261,7 +261,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

templates/html_report/all.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 $fullwidth = true;
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
 
 <?php
 
@@ -37,4 +37,4 @@ 
             </div>
         </div>
     </div>
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>