phpmetrics / PhpMetrics

src/Hal/Metric/Package/PackageDependencies.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 {
     public function calculate(Metrics $metrics)
     {
-        $classes = array_filter($metrics->all(), function (Metric $metric) {
+        $classes = array_filter($metrics->all(), function(Metric $metric) {
             return $metric instanceof ClassMetric || $metric instanceof InterfaceMetric;
         });
 
@@ -27,7 +27,7 @@ 
      */
     private function increaseDependencies(Metric $class, Metrics $metrics)
     {
-        if (! $class->has('package') || ! $class->has('externals')) {
+        if (!$class->has('package') || !$class->has('externals')) {
             return;
         }
         $incomingPackage = $metrics->get($class->get('package')); /* @var $incomingPackage PackageMetric */
@@ -56,6 +56,6 @@ 
         }
         $parts = explode('\\', $className);
         array_pop($parts);
-        return implode('\\', $parts) . '\\';
+        return implode('\\', $parts).'\\';
     }
 }

src/Hal/Metric/Package/PackageInstability.php

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
     public function calculate(Metrics $metrics)
     {
         /* @var $packages PackageMetric[] */
-        $packages = array_filter($metrics->all(), function ($metric) {
+        $packages = array_filter($metrics->all(), function($metric) {
             return $metric instanceof PackageMetric;
         });
 
@@ -28,7 +28,7 @@ 
         }
         // Set depending instabilities
         foreach ($packages as $eachPackage) {
-            $dependentInstabilities = array_map(function ($packageName) use ($instabilitiesByPackage) {
+            $dependentInstabilities = array_map(function($packageName) use ($instabilitiesByPackage) {
                 return isset($instabilitiesByPackage[$packageName]) ? $instabilitiesByPackage[$packageName] : null;
             }, $eachPackage->getOutgoingPackageDependencies());
             $dependentInstabilities = array_combine(

src/Hal/Metric/Helper/MetricClassNameGenerator.php

Spacing +1 added lines, -2 removed lines

@@ -17,7 +17,6 @@
     public static function getName(Node $node)
     {
         return ($node instanceof Node\Stmt\Class_ && $node->isAnonymous()) ?
-            'anonymous@' . spl_object_hash($node) :
-            $node->namespacedName->toString();
+            'anonymous@'.spl_object_hash($node) : $node->namespacedName->toString();
     }
 }

src/Hal/Report/Csv/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             return;
         }
         if (!file_exists(dirname($logFile)) || !is_writable(dirname($logFile))) {
-            throw new \RuntimeException('You don\'t have permissions to write CSV report in ' . $logFile);
+            throw new \RuntimeException('You don\'t have permissions to write CSV report in '.$logFile);
         }
 
         $availables = (new Registry())->allForStructures();

src/Hal/Report/Json/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,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);
         }
 
         file_put_contents($logFile, json_encode($metrics, JSON_PRETTY_PRINT));

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

Spacing +4 added lines, -4 removed lines

@@ -45,7 +45,7 @@ 
 
             $package = $packagist->get($requirement);
 
-            $packages[$requirement] = (object)array(
+            $packages[$requirement] = (object) array(
                 'name' => $requirement,
                 'required' => $version,
                 'installed' => isset($rawInstalled[$requirement]) ? $rawInstalled[$requirement] : null,
@@ -76,13 +76,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);
@@ -106,7 +106,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;

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 === sizeof($packages)) { ?>
+                <?php if (0 === sizeof($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/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);
         }