phpmetrics / PhpMetrics

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';

src/Hal/Metric/Class_/Component/MaintainabilityIndexVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
             $name = getNameOfNode($node);
             $classOrFunction = $this->metrics->get($name);
 
-            if(null === $classOrFunction) {
-                throw new \LogicException('class or function ' . $name . ' not found in metrics');
+            if (null === $classOrFunction) {
+                throw new \LogicException('class or function '.$name.' not found in metrics');
             }
 
             if (null === $lloc = $classOrFunction->get('lloc')) {

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

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
         if (NodeTyper::isOrganizedLogicalClassStructure($node)) {
             $class = $this->metrics->get(getNameOfNode($node));
             if (null === $class) {
-                throw new \RuntimeException('Class metric not found for ' . getNameOfNode($node));
+                throw new \RuntimeException('Class metric not found for '.getNameOfNode($node));
             }
 
             $sy = $dc = $sc = [];
@@ -54,7 +54,7 @@ 
                     $fanout = [];
 
                     $parentNode = $node;
-                    iterate_over_node($node, function ($node) use (&$output, &$fanout, $parentNode) {
+                    iterate_over_node($node, function($node) use (&$output, &$fanout, $parentNode) {
                         switch (true) {
                             case $node instanceof Stmt\Return_:
                                 $output++;
@@ -64,14 +64,14 @@ 
                                 if ('static' === $class || 'self' === $class) {
                                     $class = getNameOfNode($parentNode);
                                 }
-                                $fanout[] = $class . '::' . getNameOfNode($node->name) . '()';
+                                $fanout[] = $class.'::'.getNameOfNode($node->name).'()';
                                 break;
                             case $node instanceof Node\Expr\MethodCall:
                                 $class = getNameOfNode($node->var);
                                 if ('this' === $class) {
                                     $class = getNameOfNode($parentNode);
                                 }
-                                $fanout[] = $class . '->' . getNameOfNode($node->name) . '()';
+                                $fanout[] = $class.'->'.getNameOfNode($node->name).'()';
                                 break;
                         }
                     });

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

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
             NodeTyper::isOrganizedLogicalClassStructure($node)
             || $node instanceof Stmt\Function_
         ) {
-            if (NodeTyper::isOrganizedLogicalClassStructure($node) ) {
+            if (NodeTyper::isOrganizedLogicalClassStructure($node)) {
                 $name = getNameOfNode($node);
                 $classOrFunction = $this->metrics->get($name);
             } else {
@@ -58,7 +58,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_/Complexity/CyclomaticComplexityVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
         if (NodeTyper::isOrganizedStructure($node)) {
             $class = $this->metrics->get(getNameOfNode($node));
             if ($class === null) {
-                throw new \RuntimeException('Class metric not found for ' . getNameOfNode($node));
+                throw new \RuntimeException('Class metric not found for '.getNameOfNode($node));
             }
 
             $ccn = 1;
@@ -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/Metric/Class_/ClassEnumVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             $roleDetector = new RoleOfMethodDetector();
             foreach ($node->stmts as $stmt) {
                 if ($stmt instanceof Stmt\ClassMethod) {
-                    $function = new FunctionMetric((string)$stmt->name);
+                    $function = new FunctionMetric((string) $stmt->name);
 
                     $role = $roleDetector->detects($stmt);
                     $function->set('role', $role);