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)

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;

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

phpmetrics/src/Hal/Metric/System/Changes/GitChanges.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         $file = current($this->files);
         $realpath = realpath(dirname($file));
         if ($realpath === false) {
-            throw new ShouldNotHappenException('Cannot get realpath of ' . dirname($file));
+            throw new ShouldNotHappenException('Cannot get realpath of '.dirname($file));
         }
         $command = sprintf(
             "cd %s && %s log --format='* %%at\t%%cn' --numstat -n 500",

phpmetrics/src/Hal/Metric/System/UnitTesting/UnitTesting.php

Spacing +5 added lines, -5 removed lines

@@ -76,7 +76,7 @@ 
         $domList = $xpath->query('//testsuite[@file]');
         if ($domList !== false) {
             foreach ($domList as $suite) {
-                array_push($testsuites, (object)[
+                array_push($testsuites, (object) [
                     'file' => $suite->getAttribute('file'),
                     'name' => $suite->getAttribute('name'),
                     'assertions' => $suite->getAttribute('assertions'),
@@ -108,7 +108,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,
@@ -128,7 +128,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);
@@ -150,10 +150,10 @@ 
             if ($metric === null) {
                 throw new MetricNullException($suite->name, self::class);
             }
-            $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,