phpmetrics / PhpMetrics

build.php

Indentation +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
 chdir(__DIR__);
 
 if (!file_exists('vendor/autoload.php')) {
-  echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
-  exit(1);
+    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
+    exit(1);
 }
 
 $filename = 'build/phpmetrics.phar';

Spacing +7 added lines, -7 removed lines

@@ -2,7 +2,7 @@ 
 chdir(__DIR__);
 
 if (!file_exists('vendor/autoload.php')) {
-  echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
+  echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!'.PHP_EOL;
   exit(1);
 }
 
@@ -18,8 +18,8 @@ 
 
 $files = array_merge(rglob('*.php'), rglob('*.twig'), rglob('*.json'), rglob('*.pp'));
 $exclude = '!(.git)|(.svn)|(bin)|(tests)|(Tests)|(phpmetrics)!';
-foreach($files as $file) {
-    if(preg_match($exclude, $file)) continue;
+foreach ($files as $file) {
+    if (preg_match($exclude, $file)) continue;
     $path = str_replace(__DIR__.'/', '', $file);
     $phar->addFromString($path, file_get_contents($file));
 }
@@ -50,10 +50,10 @@ 
 
 chmod($filename, 0755);
 
-function rglob($pattern='*', $flags = 0, $path='')
+function rglob($pattern = '*', $flags = 0, $path = '')
 {
-    $paths=glob($path.'*', GLOB_MARK|GLOB_ONLYDIR|GLOB_NOSORT);
-    $files=glob($path.$pattern, $flags);
-    foreach ($paths as $path) { $files=array_merge($files,rglob($pattern, $flags, $path)); }
+    $paths = glob($path.'*', GLOB_MARK|GLOB_ONLYDIR|GLOB_NOSORT);
+    $files = glob($path.$pattern, $flags);
+    foreach ($paths as $path) { $files = array_merge($files, rglob($pattern, $flags, $path)); }
     return $files;
 }
\ No newline at end of file

compatibility.php

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@
 <?php
 // PHP 7
-defined('T_SPACESHIP') ||define('T_SPACESHIP', '<=>'); // 289
-defined('T_COALESCE') ||define('T_COALESCE', '??'); // 282
\ No newline at end of file
+defined('T_SPACESHIP') || define('T_SPACESHIP', '<=>'); // 289
+defined('T_COALESCE') || define('T_COALESCE', '??'); // 282
\ No newline at end of file

src/Hal/Application/Command/Job/Analyze/CardAndAgrestiAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
     public function execute(array $files) {
         $object = new FileSystemComplexity($this->classMap);
-        foreach($files as $filename) {
+        foreach ($files as $filename) {
             $result = $object->calculate($filename);
             $this->collection->get($filename)->setSystemComplexity($result);
         }

src/Hal/Application/Command/Job/Analyze/CouplingAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         // link between coupling and files
         $fileCoupling = new FileCoupling($this->classMap, $couplingMap);
-        foreach($files as $filename) {
+        foreach ($files as $filename) {
             $rCoupling = $fileCoupling->calculate($filename);
             $this->collection->get($filename)->setCoupling($rCoupling);
         }

src/Hal/Application/Command/Job/Analyze/FileAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
             ->setHalstead($rHalstead)
             ->setMaintainabilityIndex($rMaintainability);
 
-        if($this->withOOP) {
+        if ($this->withOOP) {
             $rOOP = $this->extractor->extract($filename);
             $this->classMap->push($filename, $rOOP);
             $resultSet->setOOP($rOOP);

src/Hal/Application/Command/Job/Analyze/LcomAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
     public function execute(array $files) {
         $fileCoupling = new FileLackOfCohesionOfMethods($this->classMap);
-        foreach($files as $filename) {
+        foreach ($files as $filename) {
             $result = $fileCoupling->calculate($filename);
             $this->collection->get($filename)->setLcom($result);
         }

src/Hal/Application/Command/Job/DoAggregatedAnalyze.php

Indentation +2 added lines, -2 removed lines

@@ -52,8 +52,8 @@
 
 
     /**
-    * @inheritdoc
-    */
+     * @inheritdoc
+     */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
 
         $this->output->write(str_pad("\x0DGrouping results by package/directory. This will take few minutes...", 80, "\x20"));

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function __construct(OutputInterface $output, Aggregator $aggregator)
     {
-        $this->aggregator= $aggregator;
+        $this->aggregator = $aggregator;
         $this->output = $output;
     }
 
@@ -66,12 +66,12 @@ 
         $bounds = new Bounds();
         $instability = new Instability();
 
-        foreach($groupedResults as $namespace => $results) {
+        foreach ($groupedResults as $namespace => $results) {
 
             // we filter aggregates to conserve only direct results
             $childs = new ResultCollection();
-            foreach($results as $r) {
-                if($namespace === dirname($r->getName())) {
+            foreach ($results as $r) {
+                if ($namespace === dirname($r->getName())) {
                     $childs->push($r);
                 }
             }

src/Hal/Application/Command/Job/DoAnalyze.php

Spacing +5 added lines, -5 removed lines

@@ -92,7 +92,7 @@ 
 
         $files = $this->finder->find($this->path);
 
-        if(0 == sizeof($files, COUNT_NORMAL)) {
+        if (0 == sizeof($files, COUNT_NORMAL)) {
             throw new \LogicException('No file found');
         }
 
@@ -116,12 +116,12 @@ 
             , $classMap
         );
 
-        foreach($files as $k => $filename) {
+        foreach ($files as $k => $filename) {
 
             $progress->advance();
 
             // Integrity
-            if(!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
+            if (!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
                 $this->output->writeln(sprintf('<error>file %s is not valid and has been skipped</error>', $filename));
                 unset($files[$k]);
                 continue;
@@ -130,7 +130,7 @@ 
             // Analyze
             try {
                 $resultSet = $fileAnalyzer->execute($filename);
-            } catch(NoTokenizableException $e) {
+            } catch (NoTokenizableException $e) {
                 $this->output->writeln(sprintf("<error>file %s has been skipped: \n%s</error>", $filename, $e->getMessage()));
                 unset($files[$k]);
                 continue;
@@ -151,7 +151,7 @@ 
         $progress->finish();
 
 
-        if($this->withOOP) {
+        if ($this->withOOP) {
             // COUPLING (should be done after parsing files)
             $this->output->write(str_pad("\x0DAnalyzing coupling. This will take few minutes...", 80, "\x20"));
             $couplingAnalyzer = new CouplingAnalyzer($classMap, $collection);

src/Hal/Application/Command/Job/Queue.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        foreach($this->jobs as $job) {
+        foreach ($this->jobs as $job) {
             $job->execute($collection, $aggregatedResults);
         }
     }