phpmetrics / PhpMetrics

phpmetrics/src/Hal/Component/Issue/Issuer.php

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
     }
 
     /**
-     * @param int|string $status
+     * @param integer $status
      *
      * @return void
      */

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

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 
     /**
      * Returns the installed packages from the composer.lock file.
-     * @param mixed[] $rootPackageRequirements List of requirements to match installed packages only with requirements.
+     * @param integer[] $rootPackageRequirements List of requirements to match installed packages only with requirements.
      * @return mixed[]
      */
     protected function getComposerLockInstalled($rootPackageRequirements)

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Hal\Metric\System\Packages\Composer;
 
 use Hal\Application\Config\Config;
-use Hal\Application\Config\ConfigException;
 use Hal\Component\File\Finder;
 use Hal\Metric\Metrics;
 use Hal\Metric\ProjectMetric;

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
         }
 
         // exclude extensions
-        $packages = array_filter($packages, function ($package) {
+        $packages = array_filter($packages, function($package) {
             return !preg_match('!(^php$|^ext\-)!', $package->name);
         });
 
@@ -82,13 +82,13 @@ 
             if ($fileContent === false) {
                 throw new ShouldNotHappenException('Get composer content return false');
             }
-            $composerJson = (object)\json_decode($fileContent);
+            $composerJson = (object) \json_decode($fileContent);
 
             if (!isset($composerJson->require)) {
                 continue;
             }
 
-            $rawRequirements[] = (array)$composerJson->require;
+            $rawRequirements[] = (array) $composerJson->require;
         }
 
         return \call_user_func_array('array_merge', $rawRequirements);
@@ -116,7 +116,7 @@ 
             if ($fileContent === false) {
                 throw new ShouldNotHappenException('Get composer lock content return false');
             }
-            $composerLockJson = (object)\json_decode($fileContent);
+            $composerLockJson = (object) \json_decode($fileContent);
 
             if (!isset($composerLockJson->packages)) {
                 continue;

phpmetrics/tests/Component/Tree/GraphDeduplicatedTest.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Test;
 
-use Hal\Component\Tree\Graph;
 use Hal\Component\Tree\GraphDeduplicated;
 use Hal\Component\Tree\Node;
 

phpmetrics/tests/Component/Tree/Operator/SizeOfTreeTest.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Test;
 
 use Hal\Component\Tree\Graph;
-use Hal\Component\Tree\GraphFactory;
 use Hal\Component\Tree\Node;
 use Hal\Component\Tree\Operator\SizeOfTree;
 

src/Console/CliInput.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
             if (trim($item) === '') {
                 continue;
             }
-            $this->exts[] = '*.' . ltrim(trim($item), '*.');
+            $this->exts[] = '*.'.ltrim(trim($item), '*.');
         }
 
         if (empty($this->exts)) {

src/Parser/PhpParser.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
         try {
             $stmt = $this->parser->parse($file->getContents());
         } catch (Error $exc) {
-            throw new ParserException($file->getRelativePathname() . ': ' . $exc->getMessage());
+            throw new ParserException($file->getRelativePathname().': '.$exc->getMessage());
         }
 
         if ($stmt === null) {
-            throw new ParserException('Parse function return null when parsing ' . $file->getRelativePathname());
+            throw new ParserException('Parse function return null when parsing '.$file->getRelativePathname());
         }
 
         $this->traverser->traverse($stmt);

src/DiFactory.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      */
     public static function container(array $rules = []) : ContainerInterface
     {
-        $dirules = Json::decode(FileSystem::read(__DIR__ . '/dirules.json'), Json::FORCE_ARRAY);
+        $dirules = Json::decode(FileSystem::read(__DIR__.'/dirules.json'), Json::FORCE_ARRAY);
         $rlist = new DiRuleList();
         $rlist = $rlist->addRules($dirules);
         $rlist = $rlist->addRules($rules);

phpmetrics/src/Hal/Violation/Package/StableDependenciesPrinciple.php

Spacing +4 added lines, -4 removed lines

@@ -22,13 +22,13 @@ 
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof PackageMetric) {
+        if (!$metric instanceof PackageMetric) {
             return;
         }
         $instability = $metric->getInstability();
         $violatingInstabilities = array_filter(
             $metric->getDependentInstabilities(),
-            function ($otherInstability) use ($instability) {
+            function($otherInstability) use ($instability) {
                 return $otherInstability >= $instability;
             }
         );
@@ -57,8 +57,8 @@ 
 
         $count = count($this->violatingInstabilities);
         $thisInstability = round($value, 3);
-        $packages = implode("\n* ", array_map(function ($name, $instability) {
-            $name = $name === '\\' ? 'global' : substr((string)$name, 0, -1);
+        $packages = implode("\n* ", array_map(function($name, $instability) {
+            $name = $name === '\\' ? 'global' : substr((string) $name, 0, -1);
             $instability = round($instability, 3);
             return "$name ($instability)";
         }, array_keys($this->violatingInstabilities), $this->violatingInstabilities));

phpmetrics/src/Hal/Violation/Package/StableAbstractionsPrinciple.php

Spacing +2 added lines, -2 removed lines

@@ -19,14 +19,14 @@
 
     public function apply(Metric $metric)
     {
-        if (! $metric instanceof PackageMetric) {
+        if (!$metric instanceof PackageMetric) {
             return;
         }
         $distance = $metric->getDistance();
         if ($distance === null) {
             throw new ShouldNotHappenException('Distance is null');
         }
-        if (abs($distance) > sqrt(2)/4) {
+        if (abs($distance) > sqrt(2) / 4) {
             $this->metric = $metric;
             $metric->get('violations')->add($this);
         }