phpmetrics / PhpMetrics

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/Metric/Class_/Component/MaintainabilityIndexVisitor.php

Unused Use Statements -2 removed lines

@@ -1,9 +1,7 @@
 <?php
 namespace Hal\Metric\Class_\Component;
 
-use Hal\Metric\FunctionMetric;
 use Hal\Metric\Metrics;
-use Hoa\Ruler\Model\Bag\Scalar;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 use PhpParser\NodeVisitorAbstract;

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);
         }

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

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
         // get latest version
         $latest = '0.0.0';
-        foreach ((array)$json->package->versions as $version => $datas) {
+        foreach ((array) $json->package->versions as $version => $datas) {
             if ($version[0] === 'v') {
                 $version = substr($version, 1);
             }

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(sizeof($array) > 1) {
+if (sizeof($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/Metric/Class_/Complexity/CyclomaticComplexityVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
                 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/Class_/Coupling/ExternalsVisitor.php

Spacing +11 added lines, -11 removed lines

@@ -63,20 +63,20 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        array_push($dependencies, (string)$interface);
-                        array_push($parents, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        array_push($dependencies, (string) $interface);
+                        array_push($parents, (string) $interface);
                     }
                 } else {
-                    array_push($dependencies, (string)$node->extends);
-                    array_push($parents, (string)$node->extends);
+                    array_push($dependencies, (string) $node->extends);
+                    array_push($parents, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    array_push($dependencies, (string)$interface);
+                    array_push($dependencies, (string) $interface);
                 }
             }
 
@@ -85,7 +85,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            array_push($dependencies, (string)$stmt->returnType);
+                            array_push($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -93,13 +93,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                array_push($dependencies, (string)$param->type);
+                                array_push($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
@@ -119,12 +119,12 @@ 
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
                                     if (((string) $use->getAlias()) === $check) {
-                                        array_push($dependencies, (string)($use->name));
+                                        array_push($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    array_push($dependencies, (string)($use->name));
+                                    array_push($dependencies, (string) ($use->name));
                                 }
                             }
                         }

src/Hal/Component/Ast/NodeTraverser.php

Doc Comments +7 added lines, -1 removed lines

@@ -44,7 +44,7 @@ 
 
     /**
      * @param array $nodes
-     * @param array|NodeVisitor[] $visitors
+     * @param NodeVisitor[] $visitors
      * @return array
      */
     public function traverseArray(array $nodes, array $visitors) {
@@ -101,6 +101,9 @@ 
         /** @var Traverser */
         private $traverser;
 
+        /**
+         * @param \Closure $stopCondition
+         */
         public function __construct($cloneNodes = false, $stopCondition = null)
         {
             parent::__construct();
@@ -123,6 +126,9 @@ 
         /** @var Traverser */
         private $traverser;
 
+        /**
+         * @param \Closure $stopCondition
+         */
         public function __construct($cloneNodes = false, $stopCondition = null)
         {
             parent::__construct();

Spacing +2 added lines, -2 removed lines

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