phpmetrics / PhpMetrics

src/Hal/Report/Html/Reporter.php

Spacing +26 added lines, -26 removed lines

@@ -44,11 +44,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 = sizeof($files) + 1;
         $history = [];
         natsort($files);
@@ -57,39 +57,39 @@ 
         }
 
         // copy sources
-        if(!file_exists($logDir . '/js')) {
+        if (!file_exists($logDir.'/js')) {
             mkdir($logDir.'/js', 0755, true);
         }
-        if(!file_exists($logDir . '/css')) {
+        if (!file_exists($logDir.'/css')) {
             mkdir($logDir.'/css', 0755, true);
         }
-        if(!file_exists($logDir . '/images')) {
+        if (!file_exists($logDir.'/images')) {
             mkdir($logDir.'/images', 0755, true);
         }
-        if(!file_exists($logDir . '/fonts')) {
+        if (!file_exists($logDir.'/fonts')) {
             mkdir($logDir.'/fonts', 0755, true);
         }
-        recurse_copy(__DIR__ . '/template/js', $logDir . '/js');
-        recurse_copy(__DIR__ . '/template/css', $logDir . '/css');
-        recurse_copy(__DIR__ . '/template/images', $logDir . '/images');
-        recurse_copy(__DIR__ . '/template/fonts', $logDir . '/fonts');
+        recurse_copy(__DIR__.'/template/js', $logDir.'/js');
+        recurse_copy(__DIR__.'/template/css', $logDir.'/css');
+        recurse_copy(__DIR__.'/template/images', $logDir.'/images');
+        recurse_copy(__DIR__.'/template/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage(__DIR__ . '/template/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(__DIR__ . '/template/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(__DIR__.'/template/git.php', $logDir.'/git.html', $consolidated, $history);
         }
-        $this->renderPage(__DIR__ . '/template/junit.php', $logDir . '/junit.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/junit.php', $logDir.'/junit.html', $consolidated, $history);
 
         // js data
         file_put_contents(
@@ -103,8 +103,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));
@@ -173,7 +173,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

src/Hal/Violation/Package/StableAbstractionsPrinciple.php

Spacing +2 added lines, -2 removed lines

@@ -18,10 +18,10 @@
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof PackageMetric) {
+        if (!$metric instanceof PackageMetric) {
             return;
         }
-        if (abs($metric->getDistance()) > sqrt(2)/4) {
+        if (abs($metric->getDistance()) > sqrt(2) / 4) {
             $this->metric = $metric;
             $metric->get('violations')->add($this);
         }

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

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,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);
             }

src/Hal/Report/Html/template/loc.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php require __DIR__ . '/_header.php'; ?>
+<?php require __DIR__.'/_header.php'; ?>
 
 
 <?php
@@ -19,10 +19,10 @@ 
 
 // 2. percentile map
 $json = [];
-if(sizeof($array) > 1) {
+if (sizeof($array) > 1) {
     $range = range(0.5, 1, .05);
     foreach ($range as $percentile) {
-        $json[] = (object)[
+        $json[] = (object) [
             'lloc' => percentile($array, $percentile),
             'percentile' => round($percentile * 100),
         ];
@@ -74,7 +74,7 @@ 
 </div>
 
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>
 
 
 <script>

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

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
                 if ($stmt instanceof Stmt\ClassMethod) {
 
                     // 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_/Coupling/ExternalsVisitor.php

Spacing +11 added lines, -11 removed lines

@@ -63,20 +63,20 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        array_push($dependencies, (string)$interface);
-                        array_push($parents, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        array_push($dependencies, (string) $interface);
+                        array_push($parents, (string) $interface);
                     }
                 } else {
-                    array_push($dependencies, (string)$node->extends);
-                    array_push($parents, (string)$node->extends);
+                    array_push($dependencies, (string) $node->extends);
+                    array_push($parents, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    array_push($dependencies, (string)$interface);
+                    array_push($dependencies, (string) $interface);
                 }
             }
 
@@ -85,7 +85,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            array_push($dependencies, (string)$stmt->returnType);
+                            array_push($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -93,13 +93,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                array_push($dependencies, (string)$param->type);
+                                array_push($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
@@ -119,12 +119,12 @@ 
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
                                     if (((string) $use->getAlias()) === $check) {
-                                        array_push($dependencies, (string)($use->name));
+                                        array_push($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    array_push($dependencies, (string)($use->name));
+                                    array_push($dependencies, (string) ($use->name));
                                 }
                             }
                         }

src/Hal/Component/Ast/NodeTraverser.php

Spacing +2 added lines, -2 removed lines

@@ -28,9 +28,9 @@
 
     public function __construct(Mother $traverser, $stopCondition = null)
     {
-        if(null === $stopCondition) {
+        if (null === $stopCondition) {
             $stopCondition = function($node) {
-                if($node instanceof Node\Stmt\Class_ || $node instanceof Node\Stmt\Interface_) {
+                if ($node instanceof Node\Stmt\Class_ || $node instanceof Node\Stmt\Interface_) {
                     return false;
                 }
 

src/Hal/Metric/Helper/RoleOfMethodDetector.php

Spacing +2 added lines, -2 removed lines

@@ -81,13 +81,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) {