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">
@@ -74,7 +74,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>
@@ -90,4 +90,4 @@ 
     </div>
 
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

src/functions.php

Spacing +11 added lines, -11 removed lines

@@ -86,7 +86,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name\FullyQualified) {
-        return (string)$node;
+        return (string) $node;
     }
     if ($node instanceof \PhpParser\Node\Expr\New_) {
         return getNameOfNode($node->class);
@@ -97,7 +97,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name) {
-        return (string)implode($node->parts);
+        return (string) implode($node->parts);
     }
 
     if (isset($node->name) && $node->name instanceof \PhpParser\Node\Expr\Variable) {
@@ -125,11 +125,11 @@ 
     }
 
     if (isset($node->name) && null === $node->name) {
-        return 'anonymous@' . spl_object_hash($node);
+        return 'anonymous@'.spl_object_hash($node);
     }
 
     if (isset($node->name)) {
-        return (string)$node->name;
+        return (string) $node->name;
     }
 
     return null;
@@ -147,10 +147,10 @@ 
     }
     while (false !== ($file = readdir($dir))) {
         if (($file != '.') && ($file != '..')) {
-            if (is_dir($src . '/' . $file)) {
-                recurse_copy($src . '/' . $file, $dst . '/' . $file);
+            if (is_dir($src.'/'.$file)) {
+                recurse_copy($src.'/'.$file, $dst.'/'.$file);
             } else {
-                copy($src . '/' . $file, $dst . '/' . $file);
+                copy($src.'/'.$file, $dst.'/'.$file);
             }
         }
     }
@@ -175,16 +175,16 @@ 
 {
     // memory cache
     static $caches;
-    if(null === $caches) {
+    if (null === $caches) {
         $caches = [];
     }
 
-    if(!isset($caches[$attribute])) {
+    if (!isset($caches[$attribute])) {
         // avoid to iterate over array too many times
         $max = 0;
         $min = 1;
-        foreach($array as $item) {
-            if(!isset($item[$attribute])) {
+        foreach ($array as $item) {
+            if (!isset($item[$attribute])) {
                 continue;
             }
 

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

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
         }
 
         // exclude extensions
-        $packages = array_filter($packages, function ($package) {
+        $packages = array_filter($packages, function($package) {
             return !preg_match('!(^php$|^ext\-)!', $package->name);
         });
 
@@ -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;

templates/html_report/composer.php

Spacing +4 added lines, -4 removed lines

@@ -1,13 +1,13 @@ 
 <?php
-require __DIR__ . '/_header.php';
+require __DIR__.'/_header.php';
 
 $packages = isset($project['composer']['packages']) ? $project['composer']['packages'] : [];
-usort($packages, function ($a, $b) {
+usort($packages, function($a, $b) {
     return strcmp($a->name, $b->name);
 });
 if ([] === $packages) {
     echo '<div class="row"><div class="column"><div class="bloc">No composer.json file found in this project</div></div></div>';
-    require __DIR__ . '/_footer.php';
+    require __DIR__.'/_footer.php';
     return;
 }
 ?>
@@ -95,4 +95,4 @@ 
     </div>
 </div>
 
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

templates/html_report/_header.php

Spacing +2 added lines, -2 removed lines

@@ -57,10 +57,10 @@
         <?php if ([] !== $this->groups) {?>
         <div class="content-full group-tabs">
             <ul class="tabs">
-                <li class="<?php if(!$this->currentGroup) { echo 'active'; }?>"><a href="<?php echo $this->assetPath;?>index.html">All</a></li>
+                <li class="<?php if (!$this->currentGroup) { echo 'active'; }?>"><a href="<?php echo $this->assetPath; ?>index.html">All</a></li>
                 <?php foreach ($this->groups as $group) { ?>
                     <li class="<?php if ($group->getName() === $this->currentGroup) { echo 'active'; }?> ">
-                        <a href="<?php echo $this->assetPath . $group->getName();?>/index.html"><?php echo $group->getName();?></a>
+                        <a href="<?php echo $this->assetPath.$group->getName(); ?>/index.html"><?php echo $group->getName(); ?></a>
                     </li>
                 <?php } ?>
             </ul>

templates/html_report/index.php

Spacing +10 added lines, -10 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
     <div class="row">
         <div class="column">
             <div class="bloc bloc-number">
@@ -51,7 +51,7 @@ 
                 <div class="label">
                     <a href="junit.html">Assertions in tests</a>
                 </div>
-                <?php if(isset($project['unitTesting'])) { ?>
+                <?php if (isset($project['unitTesting'])) { ?>
                     <div class="number">
                         <?php echo $project['unitTesting']['assertions']; ?>
                     </div>
@@ -147,14 +147,14 @@ 
                             <tbody id="contentClassRank" class="clusterize-content">
                             <?php
                             $classesS = $classes;
-                            usort($classesS, static function ($a, $b) {
+                            usort($classesS, static function($a, $b) {
                                 return strcmp($b['pageRank'], $a['pageRank']);
                             });
                             //$classesS = array_slice($classesS, 0, 10);
                             foreach ($classesS as $class) { ?>
                                 <tr>
                                     <td>
-                                        <span class="badge" <?php echo gradientStyleFor($classes, 'pageRank', $class['pageRank']);?>);">
+                                        <span class="badge" <?php echo gradientStyleFor($classes, 'pageRank', $class['pageRank']); ?>);">
                                         <?php echo $class['pageRank']; ?>
                                     </td>
                                     </td>
@@ -166,8 +166,8 @@ 
                                             $badgeTitleMI = 'Maintainability Index';
                                             $mi = isset($class['mi']) ? $class['mi'] : '';
                                         ?>
-                                        <span class="badge" title="<?php echo $badgeTitleMI;?>"><?php echo $mi;?></span>
-                                        <span class="badge" title="<?php echo $badgeTitleMIWOC;?>"><?php echo $mIwoC;?></span>
+                                        <span class="badge" title="<?php echo $badgeTitleMI; ?>"><?php echo $mi; ?></span>
+                                        <span class="badge" title="<?php echo $badgeTitleMIWOC; ?>"><?php echo $mIwoC; ?></span>
                                     </td>
                                 </tr>
                             <?php } ?>
@@ -180,7 +180,7 @@ 
     </div>
 
     <div class="row">
-        <?php if($this->isHomePage()) {?>
+        <?php if ($this->isHomePage()) {?>
             <div class="column">
                 <div class="bloc bloc-number">
                     <div class="label">
@@ -193,7 +193,7 @@ 
                         <div class="help number-alternate"><div class="help-inner">No composer.json file found</div></div>
                     <?php } else {?>
                         <div class="number">
-                            <?php echo count($packages);?> dependencies
+                            <?php echo count($packages); ?> dependencies
                         </div>
                     <?php } ?>
 
@@ -235,9 +235,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 +35 added lines, -35 removed lines

@@ -53,7 +53,7 @@ 
     {
         $this->config = $config;
         $this->output = $output;
-        $this->templateDir = __DIR__ . '/../../../../templates';
+        $this->templateDir = __DIR__.'/../../../../templates';
     }
 
 
@@ -78,11 +78,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 = count($files) + 1;
         $history = [];
         natsort($files);
@@ -91,40 +91,40 @@ 
         }
 
         // copy sources
-        if (!file_exists($logDir . '/js')) {
-            mkdir($logDir . '/js', 0755, true);
+        if (!file_exists($logDir.'/js')) {
+            mkdir($logDir.'/js', 0755, true);
         }
-        if (!file_exists($logDir . '/css')) {
-            mkdir($logDir . '/css', 0755, true);
+        if (!file_exists($logDir.'/css')) {
+            mkdir($logDir.'/css', 0755, true);
         }
-        if (!file_exists($logDir . '/images')) {
-            mkdir($logDir . '/images', 0755, true);
+        if (!file_exists($logDir.'/images')) {
+            mkdir($logDir.'/images', 0755, true);
         }
-        if (!file_exists($logDir . '/fonts')) {
-            mkdir($logDir . '/fonts', 0755, true);
+        if (!file_exists($logDir.'/fonts')) {
+            mkdir($logDir.'/fonts', 0755, true);
         }
-        recurse_copy($this->templateDir . '/html_report/js', $logDir . '/js');
-        recurse_copy($this->templateDir . '/html_report/css', $logDir . '/css');
-        recurse_copy($this->templateDir . '/html_report/images', $logDir . '/images');
-        recurse_copy($this->templateDir . '/html_report/fonts', $logDir . '/fonts');
+        recurse_copy($this->templateDir.'/html_report/js', $logDir.'/js');
+        recurse_copy($this->templateDir.'/html_report/css', $logDir.'/css');
+        recurse_copy($this->templateDir.'/html_report/images', $logDir.'/images');
+        recurse_copy($this->templateDir.'/html_report/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage($this->templateDir . '/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/composer.php', $logDir . '/composer.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/composer.php', $logDir.'/composer.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage($this->templateDir . '/html_report/git.php', $logDir . '/git.html', $consolidated, $consolidatedGroups, $history);
+            $this->renderPage($this->templateDir.'/html_report/git.php', $logDir.'/git.html', $consolidated, $consolidatedGroups, $history);
         }
-        $this->renderPage($this->templateDir . '/html_report/junit.php', $logDir . '/junit.html', $consolidated, $consolidatedGroups, $history);
+        $this->renderPage($this->templateDir.'/html_report/junit.php', $logDir.'/junit.html', $consolidated, $consolidatedGroups, $history);
 
         // js data
         file_put_contents(
@@ -138,8 +138,8 @@ 
 
         // json data
         file_put_contents(
-            $logDir . '/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         // HTML files to generate
@@ -160,7 +160,7 @@ 
 
         // consolidated by groups
         foreach ($consolidatedGroups as $name => $consolidated) {
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
             $this->currentGroup = $name;
             $this->assetPath = '../';
 
@@ -177,8 +177,8 @@ 
                 );
 
                 file_put_contents(
-                    $outDir . '/classes.js',
-                    'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+                    $outDir.'/classes.js',
+                    'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
                 );
             }
         }
@@ -251,7 +251,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

src/Hal/Metric/Consolidated.php

Spacing +4 added lines, -4 removed lines

@@ -66,13 +66,13 @@ 
         }
 
         // sums
-        $sum = (object)[
+        $sum = (object) [
             'loc' => 0,
             'cloc' => 0,
             'lloc' => 0,
             'nbMethods' => 0,
         ];
-        $avg = (object)[
+        $avg = (object) [
             'wmc' => [],
             'ccn' => [],
             'bugs' => [],
@@ -107,7 +107,7 @@ 
 
         foreach ($avg as &$a) {
             if (count($a) > 0) {
-                $a = round(array_sum((array)$a) / count($a), 2);
+                $a = round(array_sum((array) $a) / count($a), 2);
             } else {
                 $a = 0;
             }
@@ -166,7 +166,7 @@ 
                 $violations[$name]++;
             }
         }
-        $sum->violations = (object)$violations;
+        $sum->violations = (object) $violations;
 
         $this->avg = $avg;
         $this->sum = $sum;

src/Hal/Application/Config/Validator.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
         }
         $groupsRaw = $config->get('groups');
 
-        $groups = array_map(static function (array $groupRaw) {
+        $groups = array_map(static function(array $groupRaw) {
             return new Group($groupRaw['name'], $groupRaw['match']);
         }, $groupsRaw);
         $config->set('groups', $groups);