phpmetrics / PhpMetrics

src/Hal/Metrics/Complexity/Component/Myer/Result.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      * @inheritdoc
      */
     public function asArray() {
-        return array (
+        return array(
             'loc' => $this->getLoc()
             ,'logicalLoc' => $this->getLogicalLoc()
         );

src/Hal/Metrics/Complexity/Structural/LCOM/Result.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      * @inheritdoc
      */
     public function asArray() {
-        return array (
+        return array(
             'loc' => $this->getLoc()
             ,'logicalLoc' => $this->getLogicalLoc()
         );

artifacts/phar/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

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
 $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;
+    if(preg_match($exclude, $file)) {
+        continue;
+    }
     $path = str_replace(__DIR__.'/', '', $file);
     $phar->addFromString($path, file_get_contents($file));
 }

src/Hal/Component/OOP/Extractor/Searcher.php

Spacing +15 added lines, -15 removed lines

@@ -30,9 +30,9 @@ 
     {
         $end = sizeof($tokens, COUNT_NORMAL);
         $value = '';
-        while($startingToken < $end) {
+        while ($startingToken < $end) {
             $token = $tokens[$startingToken];
-            if(in_array($token->getValue(), $delimiters)) {
+            if (in_array($token->getValue(), $delimiters)) {
                 return $value;
             }
             $value .= $token->getValue();
@@ -52,8 +52,8 @@ 
     public function getPrevious(&$startingToken, TokenCollection $tokens)
     {
         $p = $startingToken - 1;
-        for($i = $p ; $i > 0; $i--) {
-            if(T_WHITESPACE !== $tokens[$i]->getType()) {
+        for ($i = $p; $i > 0; $i--) {
+            if (T_WHITESPACE !== $tokens[$i]->getType()) {
                 return $tokens[$i];
             }
         }
@@ -88,19 +88,19 @@ 
         $openBrace = 0;
         $start = null;
         $len = sizeof($tokens);
-        for($i = $startingToken; $i < $len; $i++) {
+        for ($i = $startingToken; $i < $len; $i++) {
             $token = $tokens[$i];
-            if(T_STRING == $token->getType()) {
-                switch($token->getValue()) {
+            if (T_STRING == $token->getType()) {
+                switch ($token->getValue()) {
                     case '{':
                         $openBrace++;
-                        if(is_null($start)) {
+                        if (is_null($start)) {
                             $start = $startingToken = $i + 1;
                         }
                         break;
                     case '}':
                         $openBrace--;
-                        if($openBrace <= 0) {
+                        if ($openBrace <= 0) {
                             return $i;
                         }
                         break;
@@ -119,7 +119,7 @@ 
      */
     public function getExtendPosition(TokenCollection $tokens, $startingToken)
     {
-        for($i = $startingToken; $i > 0; $i--) {
+        for ($i = $startingToken; $i > 0; $i--) {
             $token = $tokens[$i];
             if ($token->getValue() === 'extends') {
                 return $i;
@@ -136,7 +136,7 @@ 
     public function getClassNamePosition(TokenCollection $tokens)
     {
         $len = sizeof($tokens);
-        for($i = 0; $i < $len; $i++) {
+        for ($i = 0; $i < $len; $i++) {
             $token = $tokens[$i];
             if ($token->getValue() === 'class') {
                 return $i;
@@ -155,8 +155,8 @@ 
      */
     public function getPositionOfPrevious($tokenType, $startingToken, TokenCollection $tokens)
     {
-        for($i = $startingToken; $i > 0; $i--) {
-            if($tokenType == $tokens->get($i)->getType()) {
+        for ($i = $startingToken; $i > 0; $i--) {
+            if ($tokenType == $tokens->get($i)->getType()) {
                 return $i;
             }
         }
@@ -174,8 +174,8 @@ 
     public function getPositionOfNext($tokenType, $startingToken, TokenCollection $tokens) 
     {
         $len = sizeof($tokens);
-        for($i = $startingToken; $i < $len; $i++) {
-            if($tokenType == $tokens->get($i)->getType()) {
+        for ($i = $startingToken; $i < $len; $i++) {
+            if ($tokenType == $tokens->get($i)->getType()) {
                 return $i;
             }
         }

src/Hal/Component/Token/Dumper.php

Spacing +3 added lines, -3 removed lines

@@ -26,13 +26,13 @@
     public function dump(TokenCollection $tokens, $start = 0, $len = 0)
     {
         $str = '';
-        if($len == 0) {
+        if ($len == 0) {
             $len = sizeof($tokens);
         }
-        if($start < 0) {
+        if ($start < 0) {
             $start = 0;
         }
-        for($i = $start; $i < $len; $i++) {
+        for ($i = $start; $i < $len; $i++) {
             $str .= sprintf("\n%s : %s", token_name($tokens[$i]->getType()), $tokens[$i]->getValue());
         }
         return $str;

src/Hal/Component/OOP/Extractor/Extractor.php

Spacing +9 added lines, -9 removed lines

@@ -48,7 +48,7 @@ 
 
         $this->tokenizer = $tokenizer;
         $this->searcher = new Searcher();
-        $this->result= new Result;
+        $this->result = new Result;
 
         $this->extractors = (object) array(
             'class' => new ClassExtractor($this->searcher)
@@ -80,9 +80,9 @@ 
         $endAnonymous = 0;
         $mainContextClass = null; // class containing a anonymous class
 
-        for($n = 0; $n < $len; $n++) {
+        for ($n = 0; $n < $len; $n++) {
 
-            if($mainContextClass && $n > $endAnonymous) {
+            if ($mainContextClass && $n > $endAnonymous) {
                 // anonymous class is finished. We back to parent class
                 // methods will be added to the main class now
                 $class = $mainContextClass;
@@ -91,7 +91,7 @@ 
 
             $token = $tokens[$n];
 
-            switch($token->getType()) {
+            switch ($token->getType()) {
 
                 case T_USE:
                     $alias = $this->extractors->alias->extract($n, $tokens);
@@ -131,7 +131,7 @@ 
                 case T_CLASS:
                     $i = $n;
                     // avoid MyClass::class syntax
-                    if($this->searcher->isPrecededBy(T_DOUBLE_COLON, $i, $tokens, 1)) {
+                    if ($this->searcher->isPrecededBy(T_DOUBLE_COLON, $i, $tokens, 1)) {
                         continue;
                     }
 
@@ -142,14 +142,14 @@ 
                     $result->pushClass($c);
 
                     // PHP 7 and inner classes
-                    if($c instanceof ReflectedAnonymousClass) {
+                    if ($c instanceof ReflectedAnonymousClass) {
                         // avoid to consider anonymous class as main class
                         $p = $n;
                         $endAnonymous = $this->searcher->getPositionOfClosingBrace($p, $tokens);
                         $mainContextClass = $class;
 
                         // add anonymous class in method
-                        if($method) {
+                        if ($method) {
                             $method->pushAnonymousClass($c);
                         }
                     }
@@ -157,10 +157,10 @@ 
                     break;
 
                 case T_FUNCTION:
-                    if($class) {
+                    if ($class) {
                         // avoid closure
                         $next = $tokens[$n + 1];
-                        if(T_WHITESPACE != $next->getType()) {
+                        if (T_WHITESPACE != $next->getType()) {
                             continue;
                         }
                         $method = $this->extractors->method->extract($n, $tokens, $class);

src/Hal/Application/Config/ConfigValidator.php

Unused Use Statements -4 removed lines

@@ -9,10 +9,6 @@
 
 namespace Hal\Application\Config;
 use Hal\Component\Chart\Graphviz;
-use Hal\Component\Config\Hydrator;
-use Hal\Component\Config\Loader;
-use Hal\Component\Config\Validator;
-use Symfony\Component\Console\Input\InputInterface;
 
 /**
  * Config checker

Spacing +2 added lines, -2 removed lines

@@ -27,9 +27,9 @@
     public function validates(Configuration $config)
     {
         // graphviz
-        if($config->getLogging()->getChart('bubbles')) {
+        if ($config->getLogging()->getChart('bubbles')) {
             $graphviz = new Graphviz();
-            if(!$graphviz->isAvailable()) {
+            if (!$graphviz->isAvailable()) {
                 throw new \RuntimeException('Graphviz not installed');
             }
         }

src/Hal/Component/Chart/Graphviz.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function getImage($dotContent) {
 
-        if(!$this->isAvailable()) {
+        if (!$this->isAvailable()) {
             throw new \RuntimeException('Graphviz not installed');
         }