phpmetrics / PhpMetrics

src/Hal/Violation/Class_/TooComplexClassCode.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof ClassMetric) {
+        if (!$metric instanceof ClassMetric) {
             return;
         }
 

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

Doc Comments +3 added lines

@@ -136,6 +136,9 @@
         }
     }
 
+    /**
+     * @param string|null $dependency
+     */
     private function pushToDependencies(array &$dependencies, $dependency)
     {
         $lowercase = strtolower($dependency);

Spacing +13 added lines, -13 removed lines

@@ -58,20 +58,20 @@ 
             // 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);
+                    $this->pushToDependencies($dependencies, (string) $interface);
                 }
             }
 
@@ -80,7 +80,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);
                         }
                     }
 
@@ -88,13 +88,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
@@ -113,13 +113,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));
                                 }
                             }
                         }
@@ -138,6 +138,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

src/Hal/Metric/System/UnitTesting/UnitTesting.php

Spacing +5 added lines, -5 removed lines

@@ -72,7 +72,7 @@ 
 
         // JUNIT format
         foreach ($xpath->query('//testsuite[@file]') as $suite) {
-            array_push($testsuites, (object)[
+            array_push($testsuites, (object) [
                 'file' => $suite->getAttribute('file'),
                 'name' => $suite->getAttribute('name'),
                 'assertions' => $suite->getAttribute('assertions'),
@@ -101,7 +101,7 @@ 
                 $assertions = $case === $suite->firstChild->nextSibling ? $suite->getAttribute('assertions') : 0;
             }
 
-            $testsuites[$case->getAttribute('class')] = (object)[
+            $testsuites[$case->getAttribute('class')] = (object) [
                 'file' => $case->getAttribute('file'),
                 'name' => $case->getAttribute('class'),
                 'assertions' => $assertions,
@@ -120,7 +120,7 @@ 
             $traverser->addVisitor(new ExternalsVisitor($metricsOfUnitTest));
 
             if (!file_exists($suite->file) || !is_readable($suite->file)) {
-                throw new \LogicException('Cannot find source file referenced in testsuite: ' . $suite->file);
+                throw new \LogicException('Cannot find source file referenced in testsuite: '.$suite->file);
             }
 
             $code = file_get_contents($suite->file);
@@ -133,10 +133,10 @@ 
 
             // list of externals sources of unit test
             $metric = $metricsOfUnitTest->get($suite->name);
-            $externals = (array)$metric->get('externals');
+            $externals = (array) $metric->get('externals');
 
             // global stats for each test
-            $infoAboutTests[$suite->name] = (object)[
+            $infoAboutTests[$suite->name] = (object) [
                 'nbExternals' => count(array_unique($externals)),
                 'externals' => array_unique($externals),
                 'filename' => $suite->file,

src/Hal/Component/File/Finder.php

Spacing +3 added lines, -3 removed lines

@@ -71,15 +71,15 @@
         $files = [];
         foreach ($paths as $path) {
             if (is_dir($path)) {
-                $path = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
+                $path = rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR;
                 $directory = new RecursiveDirectoryIterator($path, $this->flags);
                 $iterator = new RecursiveIteratorIterator($directory);
 
                 $filterRegex = sprintf(
                     '`^%s%s%s$`',
                     preg_quote($path, '`'),
-                    !empty($this->excludedDirs) ? '((?!' . implode('|', array_map('preg_quote', $this->excludedDirs)) . ').)+' : '.+',
-                    '\.(' . implode('|', $this->extensions) . ')'
+                    !empty($this->excludedDirs) ? '((?!'.implode('|', array_map('preg_quote', $this->excludedDirs)).').)+' : '.+',
+                    '\.('.implode('|', $this->extensions).')'
                 );
 
                 $filteredIterator = new RegexIterator(

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

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
             foreach ($node->stmts as $stmt) {
                 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/Application/Application.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $output = new CliOutput();
 
         // issues and debug
-        $issuer = (new Issuer($output));//->enable();
+        $issuer = (new Issuer($output)); //->enable();
 
         // config
         $config = (new Parser())->parse($argv);

src/Hal/Component/Output/ProgressBar.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,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/Component/Ast/Traverser.php

Spacing +1 added lines, -1 removed lines

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

src/Hal/Metric/Class_/ClassEnumVisitor.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
                 $class->set('interface', true);
                 $class->set('abstract', true);
             } else {
-                $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node));
+                $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node));
                 $class = new ClassMetric($name);
                 $class->set('interface', false);
                 $class->set('abstract', $node instanceof Stmt\Trait_ || $node->isAbstract());
@@ -50,7 +50,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);