phpmetrics / PhpMetrics

src/functions.php

Spacing +5 added lines, -5 removed lines

@@ -102,7 +102,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name) {
-        if(!property_exists($node, 'parts') || null === $node->parts) {
+        if (!property_exists($node, 'parts') || null === $node->parts) {
             return (string) $node->name;
         }
         return (string) implode($node->parts);
@@ -134,7 +134,7 @@ 
 
     // do not use isset here, we want to check if property is set
     if (property_exists($node, 'name') && null === $node->name) {
-        return 'anonymous@' . spl_object_hash($node);
+        return 'anonymous@'.spl_object_hash($node);
     }
 
     if (isset($node->name)) {
@@ -156,10 +156,10 @@ 
     }
     while (false !== ($file = readdir($dir))) {
         if (($file != '.') && ($file != '..')) {
-            if (is_dir($src . '/' . $file)) {
-                recurse_copy($src . '/' . $file, $dst . '/' . $file);
+            if (is_dir($src.'/'.$file)) {
+                recurse_copy($src.'/'.$file, $dst.'/'.$file);
             } else {
-                copy($src . '/' . $file, $dst . '/' . $file);
+                copy($src.'/'.$file, $dst.'/'.$file);
             }
         }
     }

src/Hal/Search/Search.php

Spacing +3 added lines, -3 removed lines

@@ -118,7 +118,7 @@ 
 
     private function matchExpectedName(Metric $metric, $expectedName)
     {
-        return preg_match('@' . $expectedName . '@i', $metric->getName());
+        return preg_match('@'.$expectedName.'@i', $metric->getName());
     }
 
     private function matchInstanceOf(Metric $metric, $instanceOf)
@@ -137,7 +137,7 @@ 
     {
         foreach ($usesClasses as $expectedClass) {
             foreach ((array) $metric->get('externals') as $use) {
-                if (preg_match('@' . $expectedClass . '@i', $use)) {
+                if (preg_match('@'.$expectedClass.'@i', $use)) {
                     return true;
                 }
             }
@@ -149,7 +149,7 @@ 
     private function matchesMetric(Metric $metric, $metricName, $metricValue)
     {
         if (!preg_match_all('!^([=><]*)([\d\.]+)!', $metricValue, $matches, PREG_SET_ORDER)) {
-            throw new \LogicException('Invalid search expression for key ' . $metricValue);
+            throw new \LogicException('Invalid search expression for key '.$metricValue);
         }
         list(, $operator, $expected) = $matches[0];
 

src/Hal/Application/Config/Validator.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
         }
         $groupsRaw = $config->get('groups');
 
-        $groups = array_map(static function (array $groupRaw) {
+        $groups = array_map(static function(array $groupRaw) {
             return new Group($groupRaw['name'], $groupRaw['match']);
         }, $groupsRaw);
         $config->set('groups', $groups);
@@ -69,7 +69,7 @@ 
         } else {
             // When PHP is not compiled with the filter extension, we need to do it manually
             $bool = $config->get('composer');
-            if( is_string($bool) ) {
+            if (is_string($bool)) {
                 $bool = strtolower($bool);
                 $bool = in_array($bool, ['true', '1', 'yes', 'on'], true);
             }

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

Spacing +2 added lines, -2 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;
@@ -118,7 +118,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/Component/Ast/NodeTraverser.php

Spacing +3 added lines, -3 removed lines

@@ -10,11 +10,11 @@
 namespace Hal\Component\Ast;
 
 if (PHP_VERSION_ID >= 80000) {
-    class_alias(Php8NodeTraverser::class, __NAMESPACE__ . '\\ActualNodeTraverser');
+    class_alias(Php8NodeTraverser::class, __NAMESPACE__.'\\ActualNodeTraverser');
 }elseif (PHP_VERSION_ID >= 70000) {
-    class_alias(Php7NodeTraverser::class, __NAMESPACE__ . '\\ActualNodeTraverser');
+    class_alias(Php7NodeTraverser::class, __NAMESPACE__.'\\ActualNodeTraverser');
 } else {
-    class_alias(Php5NodeTraverser::class, __NAMESPACE__ . '\\ActualNodeTraverser');
+    class_alias(Php5NodeTraverser::class, __NAMESPACE__.'\\ActualNodeTraverser');
 }
 
 /**

src/Hal/Component/Ast/ParserTraverserVisitorsAssigner.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
         // With nikic/php-parser >= v5, visitors are traversed in LIFO order.
         // With nikic/php-parser < v5, visitors are traversed in FIFO order.
 
-        if (! method_exists('PhpParser\ParserFactory', 'create')) {
+        if (!method_exists('PhpParser\ParserFactory', 'create')) {
             $visitors = array_reverse($visitors);
         }
 

src/Hal/Component/Ast/ParserFactoryBridge.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     public function create($kind = null)
     {
         if (!method_exists('PhpParser\ParserFactory', 'createForNewestSupportedVersion')) {
-            if(null === $kind) {
+            if (null === $kind) {
                 $kind = \PhpParser\ParserFactory::PREFER_PHP7;
             }
             return (new \PhpParser\ParserFactory())->create($kind);

src/Hal/Violation/Package/StableAbstractionsPrinciple.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof PackageMetric) {
+        if (!$metric instanceof PackageMetric) {
             return;
         }
         if (abs($metric->getDistance()) > sqrt(2) / 4) {

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';
     }
 
 
@@ -83,7 +83,7 @@ 
             '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,47 +92,47 @@ 
         }
 
         // 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));
         }
 
-        copy($this->templateDir . '/html_report/favicon.ico', $logDir . '/favicon.ico');
+        copy($this->templateDir.'/html_report/favicon.ico', $logDir.'/favicon.ico');
 
-        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(
@@ -146,8 +146,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
@@ -168,7 +168,7 @@ 
 
         // consolidated by groups
         foreach ($consolidatedGroups as $name => $consolidated) {
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
             $this->currentGroup = $name;
             $this->assetPath = '../';
 
@@ -185,8 +185,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)
                 );
             }
         }
@@ -263,7 +263,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';