phpmetrics / PhpMetrics

src/Hal/Metric/Class_/Text/HalsteadVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -44,10 +44,10 @@ 
     {
         if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) {
             if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $classOrFunction = $this->metrics->get($name);
             } else {
-                $classOrFunction = new FunctionMetric((string)$node->name);
+                $classOrFunction = new FunctionMetric((string) $node->name);
                 $this->metrics->attach($classOrFunction);
             }
 
@@ -55,7 +55,7 @@ 
             $operands = [];
             $operators = [];
 
-            iterate_over_node($node, function ($node) use (&$operators, &$operands) {
+            iterate_over_node($node, function($node) use (&$operators, &$operands) {
                 if ($node instanceof Node\Expr\BinaryOp
                     || $node instanceof Node\Expr\AssignOp
                     || $node instanceof Stmt\If_

src/Hal/Metric/Class_/Text/LengthVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
     {
         if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) {
             if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $classOrFunction = $this->metrics->get($name);
             } else {
-                $classOrFunction = new FunctionMetric((string)$node->name);
+                $classOrFunction = new FunctionMetric((string) $node->name);
                 $this->metrics->attach($classOrFunction);
             }
 
@@ -57,7 +57,7 @@ 
             $code = preg_replace('!/\*.*?\*/!s', '', $code);
 
             // count and remove single line comments
-            $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function (array $matches) use (&$cloc) {
+            $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function(array $matches) use (&$cloc) {
                 if (isset($matches[2])) {
                     $cloc += 1;
                 }

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

Spacing +5 added lines, -5 removed lines

@@ -42,12 +42,12 @@ 
 
             foreach ($node->stmts as $stmt) {
                 if ($stmt instanceof Stmt\ClassMethod) {
-                    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;
@@ -63,7 +63,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/Package/PackageCollectingVisitor.php

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
     public function enterNode(Node $node)
     {
         if ($node instanceof Namespace_) {
-            $this->namespace = (string)$node->name;
+            $this->namespace = (string) $node->name;
         }
     }
 
@@ -42,17 +42,17 @@ 
                 $package = $matches[1];
             }
             if (preg_match('/^\s*\* @subpackage (.*)/m', $docBlockText, $matches)) {
-                $package = $package . '\\' . $matches[1];
+                $package = $package.'\\'.$matches[1];
             }
 
-            $packageName = $package . '\\';
-            if (! $packageMetric = $this->metrics->get($packageName)) {
+            $packageName = $package.'\\';
+            if (!$packageMetric = $this->metrics->get($packageName)) {
                 $packageMetric = new PackageMetric($packageName);
                 $this->metrics->attach($packageMetric);
             }
             /* @var PackageMetric $packageMetric */
-            $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node);
-            $elementName = (string)$elementName;
+            $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node);
+            $elementName = (string) $elementName;
             $packageMetric->addClass($elementName);
 
             $this->metrics->get($elementName)->set('package', $packageName);

src/Hal/Metric/PackageMetric.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
     public function addClass($name)
     {
         $elements = $this->get('classes');
-        $elements[] = (string)$name;
+        $elements[] = (string) $name;
         $this->set('classes', $elements);
     }
 
@@ -26,7 +26,7 @@ 
     public function setAbstraction($abstraction)
     {
         if ($abstraction !== null) {
-            $abstraction = (float)$abstraction;
+            $abstraction = (float) $abstraction;
         }
         $this->set('abstraction', $abstraction);
     }
@@ -41,7 +41,7 @@ 
     public function setInstability($instability)
     {
         if ($instability !== null) {
-            $instability = (float)$instability;
+            $instability = (float) $instability;
         }
         $this->set('instability', $instability);
     }
@@ -63,11 +63,11 @@ 
         }
         $classDependencies = $this->getOutgoingClassDependencies();
         $packageDependencies = $this->getOutgoingPackageDependencies();
-        if (! in_array($className, $classDependencies)) {
+        if (!in_array($className, $classDependencies)) {
             $classDependencies[] = $className;
             $this->set('outgoing_class_dependencies', $classDependencies);
         }
-        if (! in_array($packageName, $packageDependencies)) {
+        if (!in_array($packageName, $packageDependencies)) {
             $packageDependencies[] = $packageName;
             $this->set('outgoing_package_dependencies', $packageDependencies);
         }
@@ -96,11 +96,11 @@ 
         }
         $classDependencies = $this->getIncomingClassDependencies();
         $packageDependencies = $this->getIncomingPackageDependencies();
-        if (! in_array($className, $classDependencies)) {
+        if (!in_array($className, $classDependencies)) {
             $classDependencies[] = $className;
             $this->set('incoming_class_dependencies', $classDependencies);
         }
-        if (! in_array($packageName, $packageDependencies)) {
+        if (!in_array($packageName, $packageDependencies)) {
             $packageDependencies[] = $packageName;
             $this->set('incoming_package_dependencies', $packageDependencies);
         }

src/Hal/Component/Ast/NodeTraverser.php

Spacing +2 added lines, -2 removed lines

@@ -9,9 +9,9 @@
 namespace Hal\Component\Ast;
 
 if (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/Metric/Helper/RoleOfMethodDetector.php

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
                 'PhpParser\\Node\\Param',
                 'PhpParser\\Node\\Expr\\Variable',
                 'PhpParser\\Node\\Name',
-            ],[
+            ], [
                 // function setOk(?bool $ok): self { $this->isOk = $ok; return $this; }
                 'PhpParser\\Node\\Stmt\\ClassMethod',
                 'PhpParser\\Node\\Stmt\\Expression',
@@ -98,13 +98,13 @@ 
      */
     public function detects($node)
     {
-        if (! $node instanceof ClassMethod) {
+        if (!$node instanceof ClassMethod) {
             return null;
         }
 
         // build a fingerprint of the given method
         $fingerprintOfMethod = [];
-        iterate_over_node($node, function ($node) use (&$fingerprintOfMethod) {
+        iterate_over_node($node, function($node) use (&$fingerprintOfMethod) {
             // avoid identifier (php-parser:^4)
             if ($node instanceof Identifier) {
                 return;

src/Hal/Metric/System/Packages/Composer/Packagist.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 
         // get latest version
         $latest = '0.0.0';
-        foreach ((array)$json->package->versions as $version => $datas) {
+        foreach ((array) $json->package->versions as $version => $datas) {
             if ($version[0] === 'v') {
                 $version = substr($version, 1);
             }
@@ -45,7 +45,7 @@ 
                 $latest = $version;
                 $response->name = $package;
                 $response->latest = $version;
-                $response->license = (array)$datas->license;
+                $response->license = (array) $datas->license;
                 $response->homepage = $datas->homepage;
                 $response->time = $datas->time;
                 $response->zip = $datas->dist->url;

src/Hal/Report/Html/Reporter.php

Spacing +30 added lines, -30 removed lines

@@ -40,11 +40,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);
@@ -53,39 +53,39 @@ 
         }
 
         // 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);
         }
-        recurse_copy(PROJECT_DIR . '/templates/html_report/js', $logDir . '/js');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/css', $logDir . '/css');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/images', $logDir . '/images');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/fonts', $logDir . '/fonts');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/js', $logDir.'/js');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/css', $logDir.'/css');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/images', $logDir.'/images');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(PROJECT_DIR . '/templates/html_report/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(PROJECT_DIR.'/templates/html_report/git.php', $logDir.'/git.html', $consolidated, $history);
         }
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/junit.php', $logDir . '/junit.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/junit.php', $logDir.'/junit.html', $consolidated, $history);
 
         // js data
         file_put_contents(
@@ -99,8 +99,8 @@ 
 
         // json data
         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));
@@ -167,7 +167,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';