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/Component/Ast/NodeTraverser.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@
 namespace Hal\Component\Ast;
 
 if (PHP_VERSION_ID >= 70000) {
-    class_alias(Php7NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
+    class_alias(Php7NodeTraverser::class, __NAMESPACE__.'\\NodeTraverser');
 } else {
-    class_alias(Php5NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
+    class_alias(Php5NodeTraverser::class, __NAMESPACE__.'\\NodeTraverser');
 }

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(count($array) > 1) {
+if (count($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/Report/Html/template/violations.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php require __DIR__ . '/_header.php'; ?>
+<?php require __DIR__.'/_header.php'; ?>
 
 <?php
 $map = [
@@ -57,7 +57,7 @@ 
                     <tbody>
                     <?php foreach ($classes as $class) {
                         if (count($class['violations']) > 0) {
-                            $currentId = 'bloc-' . uniqid('', true);
+                            $currentId = 'bloc-'.uniqid('', true);
                             ?>
 
                             <tr>
@@ -108,7 +108,7 @@ 
                     <tbody>
                     <?php foreach ($packages as $package) {
                         if (count($package['violations']) > 0) {
-                            $currentId = 'bloc-' . uniqid();
+                            $currentId = 'bloc-'.uniqid();
                             ?>
 
                             <tr>
@@ -158,4 +158,4 @@ 
             }
         }
     </script>
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

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

Spacing +10 added lines, -10 removed lines

@@ -1,4 +1,4 @@ 
-<?php require __DIR__ . '/_header.php'; ?>
+<?php require __DIR__.'/_header.php'; ?>
     <div class="row">
         <div class="column">
             <div class="bloc bloc-number">
@@ -95,7 +95,7 @@ 
                             <tbody id="contentClassRank" class="clusterize-content">
                             <?php
                             $classesS = $classes;
-                            usort($classesS, function ($a, $b) {
+                            usort($classesS, function($a, $b) {
                                 return strcmp($b['pageRank'], $a['pageRank']);
                             });
                             //$classesS = array_slice($classesS, 0, 10);
@@ -137,7 +137,7 @@ 
                             <tbody id="contentPackages" class="clusterize-content">
                             <?php
                             $packages = isset($project['composer'], $project['composer']['packages']) ? $project['composer']['packages'] : [];
-                            usort($packages, function ($a, $b) {
+                            usort($packages, function($a, $b) {
                                 return strcmp($a->name, $b->name);
                             });
                             foreach ($packages as $package) { ?>
@@ -146,18 +146,18 @@ 
                                     <td><?php echo $package->required; ?></td>
                                     <?php if (0 !== count($packagesInstalled)) {?><td><?php echo $package->installed; ?></td><?php } ?>
                                     <td><?php echo $package->latest; ?></td>
-                                    <td><?php foreach($package->license as $license) { ?>
-                                            <a target="_blank" href="https://spdx.org/licenses/<?php echo $license;?>.html"><?php echo $license;?></a>
+                                    <td><?php foreach ($package->license as $license) { ?>
+                                            <a target="_blank" href="https://spdx.org/licenses/<?php echo $license; ?>.html"><?php echo $license; ?></a>
                                         <?php } ?>
                                     </td>
                                 </tr>
                             <?php } ?>
                             </tbody>
                         </table>
-                        <?php if(0 === count($packages)) { ?>
+                        <?php if (0 === count($packages)) { ?>
                             <div>No composer.json file found</div>
                         <?php } ?>
-                        <?php if(0 === count($packagesInstalled)) { ?>
+                        <?php if (0 === count($packagesInstalled)) { ?>
                             <div>No composer.lock file found</div>
                         <?php } ?>
                     </div>
@@ -168,7 +168,7 @@ 
         <div class="column">
             <div class="bloc bloc-number">
                 <div class="label">Licences of Composer dependencies</div>
-                <?php if(0 === count($packages)) { ?>
+                <?php if (0 === count($packages)) { ?>
                     <div>No composer.json file found</div>
                 <?php } ?>
                 <div id="svg-licenses"></div>
@@ -207,9 +207,9 @@ 
                     }
                 }
                 ?>
-                chartLicenses(<?php echo json_encode(array_values($json));?>);
+                chartLicenses(<?php echo json_encode(array_values($json)); ?>);
             }
         };
     </script>
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

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/Metric/System/Packages/Composer/Composer.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
         foreach ($rawRequirements as $requirement => $version) {
             $package = $packagist->get($requirement);
 
-            $packages[$requirement] = (object)[
+            $packages[$requirement] = (object) [
                 'name' => $requirement,
                 'required' => $version,
                 'installed' => isset($rawInstalled[$requirement]) ? $rawInstalled[$requirement] : null,
@@ -75,13 +75,13 @@ 
             if (!\preg_match('/composer(-dist)?\.json/', $filename)) {
                 continue;
             }
-            $composerJson = (object)\json_decode(\file_get_contents($filename));
+            $composerJson = (object) \json_decode(\file_get_contents($filename));
 
             if (!isset($composerJson->require)) {
                 continue;
             }
 
-            $rawRequirements[] = (array)$composerJson->require;
+            $rawRequirements[] = (array) $composerJson->require;
         }
 
         return \call_user_func_array('array_merge', $rawRequirements);
@@ -105,7 +105,7 @@ 
             if (false === \strpos($filename, 'composer.lock')) {
                 continue;
             }
-            $composerLockJson = (object)\json_decode(\file_get_contents($filename));
+            $composerLockJson = (object) \json_decode(\file_get_contents($filename));
 
             if (!isset($composerLockJson->packages)) {
                 continue;