phpmetrics / PhpMetrics

src/Hal/Component/File/Finder.php

Spacing +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
     public function find($path)
     {
         $files = array();
-        if(is_dir($path)) {
+        if (is_dir($path)) {
             $path = rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR;
             $directory = new RecursiveDirectoryIterator($path, $this->flags);
             $iterator = new RecursiveIteratorIterator($directory);
@@ -76,7 +76,7 @@ 
             $filterRegex = sprintf(
                 '`^%s%s$`',
                 !empty($this->excludedDirs) ? '((?!'.$this->excludedDirs.').)+' : '.+',
-                '\.(' . $this->extensions . ')'
+                '\.('.$this->extensions.')'
             );
 
             $filteredIterator = new RegexIterator(
@@ -85,10 +85,10 @@ 
                 \RecursiveRegexIterator::GET_MATCH
             );
 
-            foreach($filteredIterator as $file) {
+            foreach ($filteredIterator as $file) {
                 $files[] = $file[0];
             }
-        } elseif(is_file($path)) {
+        } elseif (is_file($path)) {
             $files = array($path);
         }
         return $files;

src/Hal/Component/File/SyntaxChecker.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         $php = 'php';
 
         if (0 >= version_compare('5.4.0', PHP_VERSION)) {
-           $php = PHP_BINARY;
+            $php = PHP_BINARY;
         }
 
         $output = shell_exec(sprintf('"%s" -l %s 2>&1', $php, escapeshellarg($filename)));

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * @return int
      */
     public function isCorrect($filename) {
-        if(1 === version_compare('5.0.4', PHP_VERSION)) {
+        if (1 === version_compare('5.0.4', PHP_VERSION)) {
             return php_check_syntax($filename);
         }
 

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

Spacing +2 added lines, -2 removed lines

@@ -43,9 +43,9 @@
     {
         $alias = $real = null;
         $expression = $this->searcher->getUnder(array(';'), $n, $tokens);
-        if(preg_match('!use\s+(.*)\s+as\s+(.*)!i', $expression, $matches)) {
+        if (preg_match('!use\s+(.*)\s+as\s+(.*)!i', $expression, $matches)) {
             list(, $real, $alias) = $matches;
-        } else if(preg_match('!use\s+([^\s\(]+)\s*!i', $expression, $matches)) {
+        } else if (preg_match('!use\s+([^\s\(]+)\s*!i', $expression, $matches)) {
             list(, $real) = $matches;
             $alias = $real;
         }

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

Spacing +6 added lines, -6 removed lines

@@ -52,12 +52,12 @@ 
 
         $token = $tokens[$n];
         $call = null;
-        switch($token->getType()) {
+        switch ($token->getType()) {
             case T_PAAMAYIM_NEKUDOTAYIM:
                 $prev = $n - 1;
                 $value = $this->searcher->getUnder(array('::'), $prev, $tokens);
 
-                switch(true) {
+                switch (true) {
                     case $value == 'parent' && $this->currentClass:
                         return $this->currentClass->getParent();
 
@@ -66,10 +66,10 @@ 
                         $extendPosition = $this->searcher->getExtendPosition($tokens, $n);
                         return $this->searcher->getFollowingName($extendPosition, $tokens);
 
-                    case ($value == 'static' ||$value === 'self')&& $this->currentClass:
+                    case ($value == 'static' || $value === 'self') && $this->currentClass:
                         return $this->currentClass->getFullname();
 
-                    case ($value == 'static' ||$value === 'self'):
+                    case ($value == 'static' || $value === 'self'):
                         $extendPosition = $this->searcher->getClassNamePosition($tokens);
                         return $this->searcher->getFollowingName($extendPosition, $tokens);
 
@@ -79,12 +79,12 @@ 
 
             case T_NEW:
                 $call = $this->searcher->getFollowingName($n, $tokens);
-                if(preg_match('!^(\w+)!', $call, $matches)) { // fixes PHP 5.4:    (new MyClass)->foo()
+                if (preg_match('!^(\w+)!', $call, $matches)) { // fixes PHP 5.4:    (new MyClass)->foo()
                     $call = $matches[1];
                 }
                 break;
         }
-        if(null === $call) {
+        if (null === $call) {
             throw new \LogicException('Classname of call not found');
         }
         return $call;

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

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     {
         // is PHP7 ?
         $previous = $tokens->get($n - 2);
-        if($previous && T_NEW === $previous->getType()) {
+        if ($previous && T_NEW === $previous->getType()) {
             // anonymous class
             $class = new ReflectedAnonymousClass($this->namespace, 'class@anonymous');
             return $class;

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

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * @return $this
      */
     public function push($filename, Result $result) {
-        $this->classes[$filename] = $result;;
+        $this->classes[$filename] = $result; ;
         return $this;
     }
 

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

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
     public function asArray() {
 
         $nom = 0;
-        foreach($this->getClasses() as $class) {
+        foreach ($this->getClasses() as $class) {
             $nom += sizeof($class->getMethods(), COUNT_NORMAL);
         }
 
@@ -71,8 +71,8 @@ 
      */
     public function getAbstractClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class->isAbstract() ||$class instanceof ReflectedInterface) {
+        foreach ($this->getClasses() as $class) {
+            if ($class->isAbstract() || $class instanceof ReflectedInterface) {
                 array_push($result, $class);
             }
         }
@@ -86,8 +86,8 @@ 
      */
     public function getAnonymousClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class instanceof ReflectedClass\ReflectedAnonymousClass) {
+        foreach ($this->getClasses() as $class) {
+            if ($class instanceof ReflectedClass\ReflectedAnonymousClass) {
                 array_push($result, $class);
             }
         }
@@ -101,8 +101,8 @@ 
      */
     public function getInterfaces() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if($class instanceof ReflectedInterface) {
+        foreach ($this->getClasses() as $class) {
+            if ($class instanceof ReflectedInterface) {
                 array_push($result, $class);
             }
         }
@@ -116,8 +116,8 @@ 
      */
     public function getConcreteClasses() {
         $result = array();
-        foreach($this->getClasses() as $class) {
-            if(!$class->isAbstract() &&!$class instanceof ReflectedInterface  && !$class instanceof ReflectedClass\ReflectedAnonymousClass) {
+        foreach ($this->getClasses() as $class) {
+            if (!$class->isAbstract() && !$class instanceof ReflectedInterface && !$class instanceof ReflectedClass\ReflectedAnonymousClass) {
                 array_push($result, $class);
             }
         }

src/Hal/Component/OOP/Reflected/ReflectedClass.php

Spacing +5 added lines, -5 removed lines

@@ -136,11 +136,11 @@ 
     public function getDependencies()
     {
         $dependencies = array();
-        foreach($this->getMethods() as $method) {
+        foreach ($this->getMethods() as $method) {
             $dependencies = array_merge($dependencies, $method->getDependencies());
         }
-        foreach($dependencies as &$name) {
-            if(in_array($name, array('self', 'static'))) {
+        foreach ($dependencies as &$name) {
+            if (in_array($name, array('self', 'static'))) {
                 $name = $this->getFullname();
             }
             $name = $this->nameResolver->resolve($name, $this->namespace);
@@ -207,7 +207,7 @@ 
     public function getInterfaces()
     {
         $resolvedInterfaces = array();
-        foreach($this->interfaces as $interface) {
+        foreach ($this->interfaces as $interface) {
             array_push($resolvedInterfaces, $this->nameResolver->resolve($interface, $this->namespace));
         }
         return $resolvedInterfaces;
@@ -230,7 +230,7 @@ 
      */
     public function getAnonymousClasses() {
         $result = array();
-        foreach($this->methods as $method) {
+        foreach ($this->methods as $method) {
             $result = array_merge($result, $method->getAnonymousClasses());
         }
         return $result;

src/Hal/Component/OOP/Reflected/ReflectedMethod.php

Spacing +7 added lines, -7 removed lines

@@ -189,10 +189,10 @@ 
         // on read : compare with aliases. We cannot make it in pushDependency() => aliases aren't yet known
         $result = array();
         $resolver = new TypeResolver();
-        foreach($this->returns as $return) {
+        foreach ($this->returns as $return) {
             $type = $this->nameResolver->resolve($return->getType(), null);
 
-            if("\\" !== $type[0] &&!$resolver->isNative($type)) {
+            if ("\\" !== $type[0] && !$resolver->isNative($type)) {
                 $type = $this->namespace.'\\'.$type;
             }
 
@@ -233,21 +233,21 @@ 
     {
         // on read : compare with aliases. We cannot make it in pushDependency() => aliases aren't yet known
         $dependencies = array();
-        foreach($this->dependencies as $name) {
+        foreach ($this->dependencies as $name) {
             array_push($dependencies, $this->nameResolver->resolve($name, null));
         }
 
         // returned values
         $resolver = new TypeResolver();
-        foreach($this->returns as $return) {
+        foreach ($this->returns as $return) {
             $name = $return->getType();
-            if(!$resolver->isNative($name)) {
+            if (!$resolver->isNative($name)) {
                 array_push($dependencies, $this->nameResolver->resolve($name, null));
             }
         }
 
         // anonymous classes in method (inner class)
-        foreach($this->anonymousClasses as $c) {
+        foreach ($this->anonymousClasses as $c) {
             array_push($dependencies, $c->getParent());
             $dependencies = array_merge($dependencies, $c->getDependencies());
         }
@@ -379,7 +379,7 @@ 
     public function getInstanciedClasses()
     {
         $classes = array();
-        foreach($this->instanciedClasses as $name) {
+        foreach ($this->instanciedClasses as $name) {
             array_push($classes, $this->nameResolver->resolve($name, null));
         }
         return $classes;

Doc Comments +4 added lines, -4 removed lines

@@ -163,7 +163,7 @@ 
     }
 
     /**
-     * @return TokenCollection
+     * @return \Hal\Component\Token\TokenCollection
      */
     public function getTokens()
     {
@@ -365,7 +365,7 @@ 
     }
 
     /**
-     * @param $class
+     * @param string|null $class
      * @return $this
      */
     public function pushInstanciedClass($class) {
@@ -386,7 +386,7 @@ 
     }
 
     /**
-     * @param $call
+     * @param string $call
      * @return $this
      */
     public function pushInternalCall($call)
@@ -404,7 +404,7 @@ 
     }
 
     /**
-     * @param $call
+     * @param string $call
      * @return $this
      */
     public function pushExternalCall($call)