phpmetrics / PhpMetrics

templates/html_report/complexity.php

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
 
     <div class="row">
         <div class="column">
@@ -75,7 +75,7 @@ 
                             <td><span class="path"><?php echo $class['name']; ?></span></td>
                             <?php foreach (['wmc', 'ccn', 'ccnMethodMax', 'relativeSystemComplexity', 'relativeDataComplexity', 'relativeStructuralComplexity', 'bugs', 'kanDefect'] as $attribute) {?>
                                 <td>
-                                    <span class="badge" <?php echo gradientStyleFor($classes, $attribute, $class[$attribute]);?>>
+                                    <span class="badge" <?php echo gradientStyleFor($classes, $attribute, $class[$attribute]); ?>>
                                     <?php echo isset($class[$attribute]) ? $class[$attribute] : ''; ?>
                                     </span>
                                 </td>
@@ -91,4 +91,4 @@ 
     </div>
 
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

src/Hal/Component/File/Finder.php

Spacing +3 added lines, -3 removed lines

@@ -59,15 +59,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);
                 $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/Report/Json/SummaryReporter.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
             return;
         }
         if (!file_exists(dirname($logFile)) || !is_writable(dirname($logFile))) {
-            throw new \RuntimeException('You don\'t have permissions to write JSON report in ' . $logFile);
+            throw new \RuntimeException('You don\'t have permissions to write JSON report in '.$logFile);
         }
 
         $summaryWriter = new SummaryWriter($metrics, new Consolidated($metrics), $this->config);

src/Hal/Search/SearchesValidator.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
             $registry = new Registry();
             $allowedKeys = array_merge($allowedKeys, $registry->allForStructures());
 
-            $diff = array_diff(array_keys((array)$config), $allowedKeys);
+            $diff = array_diff(array_keys((array) $config), $allowedKeys);
             if (count($diff) > 0) {
                 throw new ConfigException(
                     sprintf(

src/Hal/Search/Search.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
     public function __construct($name, array $config)
     {
         $this->name = $name;
-        $this->config = (object)$config;
+        $this->config = (object) $config;
     }
 
     /**
@@ -47,7 +47,7 @@ 
 
     public function matches(Metric $metric)
     {
-        $configAsArray = (array)$this->config;
+        $configAsArray = (array) $this->config;
 
         if (!empty($this->config->type)) {
             // Does the search concerns type ?
@@ -118,14 +118,14 @@ 
 
     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)
     {
         foreach ($instanceOf as $expectedInterface) {
             $expectedInterface = ltrim($expectedInterface, '\\');
-            if (!in_array($expectedInterface, (array)$metric->get('implements'))) {
+            if (!in_array($expectedInterface, (array) $metric->get('implements'))) {
                 return false;
             }
         }
@@ -136,8 +136,8 @@ 
     private function usesClasses(Metric $metric, $usesClasses)
     {
         foreach ($usesClasses as $expectedClass) {
-            foreach ((array)$metric->get('externals') as $use) {
-                if (preg_match('@' . $expectedClass . '@i', $use)) {
+            foreach ((array) $metric->get('externals') as $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/Component/Output/CliOutput.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function writeln($message)
     {
-        $this->write(PHP_EOL . $message);
+        $this->write(PHP_EOL.$message);
         return $this;
     }
 
@@ -53,7 +53,7 @@ 
                     break;
             }
 
-            $message = $color . $message . "\033[0m";
+            $message = $color.$message."\033[0m";
         }
 
         $this->quietMode || file_put_contents('php://stdout', $message);
@@ -96,7 +96,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/Application/Config/File/ConfigFileReaderJson.php

Spacing +3 added lines, -3 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;
@@ -78,7 +78,7 @@ 
 
         // Composer
         if (array_key_exists('composer', $jsonData)) {
-            $config->set('composer', (bool)$jsonData['composer']);
+            $config->set('composer', (bool) $jsonData['composer']);
         }
 
         // Search
@@ -111,7 +111,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/Metric/Class_/Coupling/ExternalsVisitor.php

Spacing +14 added lines, -14 removed lines

@@ -59,21 +59,21 @@ 
             // 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);
-                    array_push($interfaces, (string)$interface);
+                    $this->pushToDependencies($dependencies, (string) $interface);
+                    array_push($interfaces, (string) $interface);
                 }
             }
 
@@ -82,7 +82,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);
                         }
                     }
 
@@ -90,13 +90,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
@@ -115,13 +115,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));
                                 }
                             }
                         }
@@ -141,6 +141,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

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

Spacing +5 added lines, -5 removed lines

@@ -52,12 +52,12 @@ 
                         continue;
                     }
 
-                    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;
@@ -73,7 +73,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));
                                 }