goaop / parser-reflection

src/ReflectionFunction.php

Doc Comments +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
     /**
      * Initializes reflection instance for given AST-node
      *
-     * @param string|\Closure $functionName The name of the function to reflect or a closure.
-     * @param Function_|null  $functionNode Function node AST
+     * @param string $functionName The name of the function to reflect or a closure.
+     * @param Function_  $functionNode Function node AST
      */
     public function __construct($functionName, Function_ $functionNode)
     {

Spacing +3 added lines, -3 removed lines

@@ -106,14 +106,14 @@
 
         return sprintf(
             $reflectionFormat,
-            $this->getDocComment() ? $this->getDocComment() . "\n" : '',
+            $this->getDocComment() ? $this->getDocComment()."\n" : '',
             $this->getName(),
             $this->getFileName(),
             $this->getStartLine(),
             $this->getEndLine(),
             count($this->getParameters()),
-            array_reduce($this->getParameters(), function ($str, ReflectionParameter $param) {
-                return $str . "\n    " . $param;
+            array_reduce($this->getParameters(), function($str, ReflectionParameter $param) {
+                return $str."\n    ".$param;
             }, '')
         );
     }

src/Traits/ReflectionClassLikeTrait.php

Doc Comments +3 added lines

@@ -306,6 +306,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $name
      */
     public function getMethod($name)
     {
@@ -499,6 +500,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $name
      */
     public function hasMethod($name)
     {
@@ -529,6 +531,7 @@ 
 
     /**
      * {@inheritDoc}
+     * @param string $interfaceName
      */
     public function implementsInterface($interfaceName)
     {

Spacing +15 added lines, -15 removed lines

@@ -133,17 +133,17 @@ 
             }
         }
 
-        $buildString = function (array $items, $indentLevel = 4) {
+        $buildString = function(array $items, $indentLevel = 4) {
             if (!count($items)) {
                 return '';
             }
-            $indent = "\n" . str_repeat(' ', $indentLevel);
-            return $indent . implode($indent, explode("\n", implode("\n", $items)));
+            $indent = "\n".str_repeat(' ', $indentLevel);
+            return $indent.implode($indent, explode("\n", implode("\n", $items)));
         };
-        $buildConstants = function (array $items, $indentLevel = 4) {
+        $buildConstants = function(array $items, $indentLevel = 4) {
             $str = '';
             foreach ($items as $name => $value) {
-                $str .= "\n" . str_repeat(' ', $indentLevel);
+                $str .= "\n".str_repeat(' ', $indentLevel);
                 $str .= sprintf(
                     'Constant [ %s %s ] { %s }',
                     gettype($value),
@@ -167,8 +167,8 @@ 
             ($isObject ? 'Object of class' : 'Class'),
             $modifiers,
             $this->getName(),
-            false !== $parentClass ? (' extends ' . $parentClass->getName()) : '',
-            $interfaceNames ? (' implements ' . implode(', ', $interfaceNames)) : '',
+            false !== $parentClass ? (' extends '.$parentClass->getName()) : '',
+            $interfaceNames ? (' implements '.implode(', ', $interfaceNames)) : '',
             $this->getFileName(),
             $this->getStartLine(),
             $this->getEndLine(),
@@ -208,7 +208,7 @@ 
     public function getConstants()
     {
         if (!isset($this->constants)) {
-            $this->constants = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance) {
+            $this->constants = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance) {
                 $result += $instance->getConstants();
             });
             $this->collectSelfConstants();
@@ -245,7 +245,7 @@ 
         $staticOrder   = [true, false];
         foreach ($staticOrder as $shouldBeStatic) {
             foreach ($properties as $property) {
-                $isStaticProperty     = $property->isStatic();
+                $isStaticProperty = $property->isStatic();
                 if ($shouldBeStatic !== $isStaticProperty) {
                     continue;
                 }
@@ -309,7 +309,7 @@ 
     public function getInterfaces()
     {
         if (!isset($this->interfaceClasses)) {
-            $this->interfaceClasses = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance) {
+            $this->interfaceClasses = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance) {
                 if ($instance->isInterface()) {
                     $result[$instance->name] = $instance;
                 }
@@ -346,7 +346,7 @@ 
     {
         if (!isset($this->methods)) {
             $directMethods = ReflectionMethod::collectFromClassNode($this->classLikeNode, $this);
-            $parentMethods = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance, $isParent) {
+            $parentMethods = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance, $isParent) {
                 $reflectionMethods = [];
                 foreach ($instance->getMethods() as $reflectionMethod) {
                     if (!$isParent || !$reflectionMethod->isPrivate()) {
@@ -416,9 +416,9 @@ 
      */
     public function getName()
     {
-        $namespaceName = $this->namespaceName ? $this->namespaceName . '\\' : '';
+        $namespaceName = $this->namespaceName ? $this->namespaceName.'\\' : '';
 
-        return $namespaceName . $this->getShortName();
+        return $namespaceName.$this->getShortName();
     }
 
     /**
@@ -462,7 +462,7 @@ 
     {
         if (!isset($this->properties)) {
             $directProperties = ReflectionProperty::collectFromClassNode($this->classLikeNode, $this->getName());
-            $parentProperties = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance, $isParent) {
+            $parentProperties = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance, $isParent) {
                 $reflectionProperties = [];
                 foreach ($instance->getProperties() as $reflectionProperty) {
                     if (!$isParent || !$reflectionProperty->isPrivate()) {
@@ -542,7 +542,7 @@ 
                         break;
                     }
                 }
-                $aliases[$adaptation->newName] = $traitName . '::'. $methodName;
+                $aliases[$adaptation->newName] = $traitName.'::'.$methodName;
             }
         }
 

src/ReflectionProperty.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
         Property $propertyType = null,
         PropertyProperty $propertyNode = null)
     {
-        $this->className    = $className;
+        $this->className = $className;
         if (!$propertyType || !$propertyNode) {
             list ($propertyType, $propertyNode) = ReflectionEngine::parseClassProperty($className, $propertyName);
         }

src/ReflectionClass.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace Go\ParserReflection;
 
 use Go\ParserReflection\Traits\ReflectionClassLikeTrait;
-use PhpParser\Node\Name;
 use PhpParser\Node\Stmt\ClassLike;
 use ReflectionObject as InternalReflectionObject;
 

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     /**
      * Initializes reflection instance
      *
-     * @param string|object $argument Class name or instance of object
+     * @param string|null $argument Class name or instance of object
      * @param ClassLike $classLikeNode AST node for class
      */
     public function __construct($argument, ClassLike $classLikeNode = null)

src/Traits/ReflectionFunctionLikeTrait.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
         if ($this->functionLikeNode instanceof Function_ || $this->functionLikeNode instanceof ClassMethod) {
             $functionName = $this->functionLikeNode->name;
 
-            return $this->namespaceName ? $this->namespaceName . '\\' . $functionName : $functionName;
+            return $this->namespaceName ? $this->namespaceName.'\\'.$functionName : $functionName;
         }
 
         return false;

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
     /**
      * Gets the specified return type of a function
      *
-     * @return \ReflectionType
+     * @return null|ReflectionType
      *
      * @link http://php.net/manual/en/reflectionfunctionabstract.getreturntype.php
      */

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use Go\ParserReflection\NodeVisitor\GeneratorDetector;
 use Go\ParserReflection\NodeVisitor\StaticVariablesCollector;
-use Go\ParserReflection\ReflectionFileNamespace;
 use Go\ParserReflection\ReflectionParameter;
 use Go\ParserReflection\ReflectionType;
 use PhpParser\Node\Expr\Closure;

src/ReflectionMethod.php

Spacing +2 added lines, -2 removed lines

@@ -92,12 +92,12 @@
         $paramString = '';
         $identation  = str_repeat(' ', 4);
         foreach ($methodParameters as $methodParameter) {
-            $paramString .= "\n{$identation}" . $methodParameter;
+            $paramString .= "\n{$identation}".$methodParameter;
         }
 
         return sprintf(
             "%sMethod [ <user%s%s%s>%s%s%s %s method %s ] {\n  @@ %s %d - %d{$paramFormat}\n}\n",
-            $this->getDocComment() ? $this->getDocComment() . "\n" : '',
+            $this->getDocComment() ? $this->getDocComment()."\n" : '',
             $prototype ? ", overwrites {$prototypeClass}, prototype {$prototypeClass}" : '',
             $this->isConstructor() ? ', ctor' : '',
             $this->isDestructor() ? ', dtor' : '',

src/Instrument/PathResolver.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 
         $isRelative = !$pathScheme && ($path[0] !== '/') && ($path[1] !== ':');
         if ($isRelative) {
-            $path = getcwd() . DIRECTORY_SEPARATOR . $path;
+            $path = getcwd().DIRECTORY_SEPARATOR.$path;
         }
 
         // resolve path parts (single dot, double dot and double delimiters)

src/ValueResolver/NodeExpressionResolver.php

Doc Comments +3 added lines

@@ -159,6 +159,9 @@
         return '';
     }
 
+    /**
+     * @return string
+     */
     protected function resolveScalarMagicConstNamespace()
     {
         if (method_exists($this->context, 'getNamespaceName')) {

Spacing +5 added lines, -5 removed lines

@@ -113,7 +113,7 @@ 
             ++$this->nodeLevel;
 
             $nodeType   = $node->getType();
-            $methodName = 'resolve' . str_replace('_', '', $nodeType);
+            $methodName = 'resolve'.str_replace('_', '', $nodeType);
             if (method_exists($this, $methodName)) {
                 $value = $this->$methodName($node);
             }
@@ -142,7 +142,7 @@ 
     protected function resolveScalarMagicConstMethod()
     {
         if ($this->context instanceof \ReflectionMethod) {
-            $fullName = $this->context->getDeclaringClass()->getName() . '::' . $this->context->getShortName();
+            $fullName = $this->context->getDeclaringClass()->getName().'::'.$this->context->getShortName();
 
             return $fullName;
         }
@@ -236,7 +236,7 @@ 
                 $fileNamespace = new ReflectionFileNamespace($fileName, $namespaceName);
                 if ($fileNamespace->hasConstant($constantName)) {
                     $constantValue = $fileNamespace->getConstant($constantName);
-                    $constantName  = $fileNamespace->getName() . '\\' . $constantName;
+                    $constantName  = $fileNamespace->getName().'\\'.$constantName;
                     $isResolved    = true;
                 }
             }
@@ -265,7 +265,7 @@ 
         }
 
         $this->isConstant = true;
-        $this->constantName = (string)$node->class . '::' . $constantName;
+        $this->constantName = (string) $node->class.'::'.$constantName;
 
         return $refClass->getConstant($constantName);
     }
@@ -344,7 +344,7 @@ 
 
     protected function resolveExprBinaryOpConcat(Expr\BinaryOp\Concat $node)
     {
-        return $this->resolve($node->left) . $this->resolve($node->right);
+        return $this->resolve($node->left).$this->resolve($node->right);
     }
 
     protected function resolveExprTernary(Expr\Ternary $node)

src/ReflectionFileNamespace.php

Spacing +3 added lines, -3 removed lines

@@ -319,7 +319,7 @@ 
         foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
             if ($namespaceLevelNode instanceof ClassLike) {
                 $classShortName = $namespaceLevelNode->name;
-                $className = $namespaceName ? $namespaceName .'\\' . $classShortName : $classShortName;
+                $className = $namespaceName ? $namespaceName.'\\'.$classShortName : $classShortName;
 
                 $namespaceLevelNode->setAttribute('fileName', $this->fileName);
                 $classes[$className] = new ReflectionClass($className, $namespaceLevelNode);
@@ -343,7 +343,7 @@ 
         foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
             if ($namespaceLevelNode instanceof Function_) {
                 $funcShortName = $namespaceLevelNode->name;
-                $functionName  = $namespaceName ? $namespaceName .'\\' . $funcShortName : $funcShortName;
+                $functionName  = $namespaceName ? $namespaceName.'\\'.$funcShortName : $funcShortName;
 
                 $namespaceLevelNode->setAttribute('fileName', $this->fileName);
                 $functions[$funcShortName] = new ReflectionFunction($functionName, $namespaceLevelNode);
@@ -382,7 +382,7 @@ 
             foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
                 if ($namespaceLevelNode instanceof FuncCall
                     && $namespaceLevelNode->name instanceof Name
-                    && (string)$namespaceLevelNode->name === 'define'
+                    && (string) $namespaceLevelNode->name === 'define'
                 ) {
                     $expressionSolver->process($namespaceLevelNode->args[0]->value);
                     $constantName = $expressionSolver->getValue();