goaop / parser-reflection

src/Traits/ReflectionClassLikeTrait.php

Doc Comments +3 added lines

@@ -312,6 +312,9 @@
         return new ReflectionExtension($extName);
     }
 
+    /**
+     * @return string
+     */
     public function getExtensionName()
     {
         if (!$this->classLikeNode) {

Spacing +15 added lines, -15 removed lines

@@ -134,17 +134,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),
@@ -168,8 +168,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(),
@@ -210,7 +210,7 @@ 
     {
         if (!isset($this->constants)) {
             if ($this->classLikeNode) {
-                $this->constants = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance) {
+                $this->constants = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance) {
                     $result += $instance->getConstants();
                 });
                 $this->collectSelfConstants();
@@ -256,7 +256,7 @@ 
         $staticOrder   = [true, false];
         foreach ($staticOrder as $shouldBeStatic) {
             foreach ($properties as $property) {
-                $isStaticProperty     = $property->isStatic();
+                $isStaticProperty = $property->isStatic();
                 if ($shouldBeStatic !== $isStaticProperty) {
                     continue;
                 }
@@ -347,7 +347,7 @@ 
     {
         if (!isset($this->interfaceClasses)) {
             if ($this->classLikeNode) {
-                $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;
                     }
@@ -394,7 +394,7 @@ 
         if (!isset($this->methods)) {
             if ($this->classLikeNode) {
                 $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()) {
@@ -479,9 +479,9 @@ 
      */
     public function getName()
     {
-        $namespaceName = $this->namespaceName ? $this->namespaceName . '\\' : '';
+        $namespaceName = $this->namespaceName ? $this->namespaceName.'\\' : '';
 
-        return $namespaceName . $this->getShortName();
+        return $namespaceName.$this->getShortName();
     }
 
     /**
@@ -535,7 +535,7 @@ 
         if (!isset($this->properties)) {
             if ($this->classLikeNode) {
                 $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()) {
@@ -634,7 +634,7 @@ 
                         break;
                     }
                 }
-                $aliases[$adaptation->newName] = $traitName . '::'. $methodName;
+                $aliases[$adaptation->newName] = $traitName.'::'.$methodName;
             }
         }
 

src/Traits/ReflectionFunctionLikeTrait.php

Doc Comments +4 added lines, -1 removed lines

@@ -106,6 +106,9 @@ 
         return new ReflectionExtension($extName);
     }
 
+    /**
+     * @return string
+     */
     public function getExtensionName()
     {
         if (!$this->functionLikeNode) {
@@ -223,7 +226,7 @@ 
     /**
      * Gets the specified return type of a function
      *
-     * @return \ReflectionType
+     * @return null|ReflectionType
      *
      * @link http://php.net/manual/en/reflectionfunctionabstract.getreturntype.php
      */

Spacing +5 added lines, -5 removed lines

@@ -138,7 +138,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;
@@ -229,7 +229,7 @@ 
      */
     public function getReturnType()
     {
-        $isBuiltin  = false;
+        $isBuiltin = false;
         if ($this->functionLikeNode) {
             $returnType = $this->functionLikeNode->getReturnType();
             $isNullable = $returnType instanceof NullableType;
@@ -255,7 +255,7 @@ 
             }
             $isNullable = $nativeType->allowsNull();
             $isBuiltin = $nativeType->isBuiltin();
-            $returnType = (string)$nativeType;
+            $returnType = (string) $nativeType;
         }
 
         return new ReflectionType($returnType, $isNullable, $isBuiltin);
@@ -482,9 +482,9 @@ 
         }
         if (method_exists($orig, 'hasType') && $orig->hasType()) {
             $typeRef = $orig->getType();
-            $stringType = ltrim((string)$typeRef, '?'); // ltrim() is precautionary.
+            $stringType = ltrim((string) $typeRef, '?'); // ltrim() is precautionary.
             if (PHP_VERSION_ID >= 70100 && $typeRef->allowsNull()) {
-                $stringType = '?' . $stringType;
+                $stringType = '?'.$stringType;
                 $nullableImplied = true;
             }
             $builder->setTypeHint($stringType);

src/ValueResolver/NodeExpressionResolver.php

Spacing +7 added lines, -7 removed lines

@@ -141,7 +141,7 @@ 
     protected function resolveScalarMagicConstMethod()
     {
         if ($this->context instanceof \ReflectionMethod) {
-            $fullName = $this->context->getDeclaringClass()->name . '::' . $this->context->getShortName();
+            $fullName = $this->context->getDeclaringClass()->name.'::'.$this->context->getShortName();
 
             return $fullName;
         }
@@ -228,13 +228,13 @@ 
 
         if (!$isFQNConstant) {
             if (method_exists($this->context, 'getFileName')) {
-                $fileName      = $this->context->getFileName();
+                $fileName = $this->context->getFileName();
                 if ($fileName !== false) {
                     $namespaceName = $this->resolveScalarMagicConstNamespace();
                     $fileNamespace = new ReflectionFileNamespace($fileName, $namespaceName);
                     if ($fileNamespace->hasConstant($constantName)) {
                         $constantValue = $fileNamespace->getConstant($constantName);
-                        $constantName  = $fileNamespace->getName() . '\\' . $constantName;
+                        $constantName  = $fileNamespace->getName().'\\'.$constantName;
                         $isResolved    = true;
                     }
                 }
@@ -262,7 +262,7 @@ 
                 $reason = 'Unable';
                 if ($classToReflect instanceof Expr) {
                     $methodName = $this->getDispatchMethodFor($classToReflect);
-                    $reason = "Method " . __CLASS__ . "::{$methodName}() not found trying";
+                    $reason = "Method ".__CLASS__."::{$methodName}() not found trying";
                 }
                 throw new ReflectionException("$reason to resolve class constant.");
             }
@@ -279,7 +279,7 @@ 
         }
 
         $this->isConstant = true;
-        $this->constantName = (string)$classToReflect . '::' . $constantName;
+        $this->constantName = (string) $classToReflect.'::'.$constantName;
 
         return $refClass->getConstant($constantName);
     }
@@ -363,7 +363,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)
@@ -447,7 +447,7 @@ 
     private function getDispatchMethodFor(Node $node)
     {
         $nodeType = $node->getType();
-        return 'resolve' . str_replace('_', '', $nodeType);
+        return 'resolve'.str_replace('_', '', $nodeType);
     }
 
     /**

src/ReflectionType.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
             'int'  => 'integer',
             'bool' => 'boolean'
         ];
-        $displayType = (string)$type;
+        $displayType = (string) $type;
         if (isset($typeMap[$displayType])) {
             $displayType = $typeMap[$displayType];
         }
@@ -108,7 +108,7 @@ 
     public function wasIncluded()
     {
         return
-            $this->isBuiltin()                   ||
+            $this->isBuiltin() ||
             interface_exists($this->type, false) ||
             trait_exists($this->type, false) ||
             class_exists($this->type, false);

src/ReflectionParameter.php

Doc Comments +1 added lines, -1 removed lines

@@ -222,7 +222,7 @@
 
     /**
      * {@inheritDoc}
-     * @return ReflectionClass|null  The class of the method that declared the
+     * @return \ReflectionClass|null  The class of the method that declared the
      *                                   parameter, if any.
      */
     public function getDeclaringClass()

Spacing +5 added lines, -5 removed lines

@@ -138,11 +138,11 @@ 
         if ($hasDefaultValue) {
             $defaultValue = $this->getDefaultValue();
             if (is_string($defaultValue) && strlen($defaultValue) > 15) {
-                $defaultValue = substr($defaultValue, 0, 15) . '...';
+                $defaultValue = substr($defaultValue, 0, 15).'...';
             }
             /* @see https://3v4l.org/DJOEb for behaviour changes */
             if (is_double($defaultValue) && fmod($defaultValue, 1.0) === 0.0) {
-                $defaultValue = (int)$defaultValue;
+                $defaultValue = (int) $defaultValue;
             }
 
             $defaultValue = str_replace('\\\\', '\\', var_export($defaultValue, true));
@@ -152,11 +152,11 @@ 
             'Parameter #%d [ %s %s%s%s$%s%s ]',
             $this->parameterIndex,
             $isOptional ? '<optional>' : '<required>',
-            $parameterType ? ReflectionType::convertToDisplayType($parameterType) . ' ' : '',
+            $parameterType ? ReflectionType::convertToDisplayType($parameterType).' ' : '',
             $this->isVariadic() ? '...' : '',
             $this->isPassedByReference() ? '&' : '',
             $this->getName(),
-            ($isOptional && $hasDefaultValue) ? (' = ' . $defaultValue) : ''
+            ($isOptional && $hasDefaultValue) ? (' = '.$defaultValue) : ''
         );
     }
 
@@ -215,7 +215,7 @@ 
 
                 throw new ReflectionException("Can not resolve a class name for parameter");
             }
-            $className   = $parameterType->toString();
+            $className = $parameterType->toString();
 
             return new ReflectionClass($className);
         }

src/ReflectionFunction.php

Spacing +4 added lines, -4 removed lines

@@ -154,7 +154,7 @@ 
             $origin = 'internal';
             $phpExt = $this->getExtension();
             if ($phpExt) {
-                $origin .= ':' . $phpExt->getName();
+                $origin .= ':'.$phpExt->getName();
             }
         } else {
             $source = sprintf("\n  @@ %s %d - %d", $this->getFileName(), $this->getStartLine(), $this->getEndLine());
@@ -172,15 +172,15 @@ 
 
         return sprintf(
             $reflectionFormat,
-            $this->getDocComment() ? $this->getDocComment() . "\n" : '',
+            $this->getDocComment() ? $this->getDocComment()."\n" : '',
             $origin,
             $this->getName(),
             $source,
             count($this->getParameters()),
             array_reduce(
                 $this->getParameters(),
-                (function ($str, ReflectionParameter $param) {
-                    return $str . "\n    " . $param;
+                (function($str, ReflectionParameter $param) {
+                    return $str."\n    ".$param;
                 }),
                 ''
             ),

src/NodeVisitor/BuiltinTypeFixer.php

Doc Comments +6 added lines

@@ -189,6 +189,9 @@ 
         $node->returnType = $this->fixType($node->returnType, self::RETURN_TYPES);
     }
 
+    /**
+     * @param integer $contextType
+     */
     private function fixType($node, $contextType)
     {
         $typeAsString = $this->getTypeAsString($node);
@@ -232,6 +235,9 @@ 
         return strval($node);
     }
 
+    /**
+     * @param string $typeName
+     */
     private function isTypeDefinedForContext($typeName, $contextType)
     {
         return

Spacing +20 added lines, -20 removed lines

@@ -43,7 +43,7 @@ 
             }
             $numericIndexCount = count(array_filter(
                 $options['supportedBuiltinTypeHints'],
-                (function ($val) {
+                (function($val) {
                     return preg_match('/^0*(0|[1-9]\\d*)$/', $val);
                 }),
                 ARRAY_FILTER_USE_KEY
@@ -53,24 +53,24 @@ 
                 $typeHintName = $key;
                 $validFor     = $value;
                 if (preg_match('/^0*(0|[1-9]\\d*)$/', $key) &&
-                    (intval($key) >= 0)                     &&
-                    (intval($key) <  $numericIndexCount)
+                    (intval($key) >= 0) &&
+                    (intval($key) < $numericIndexCount)
                 ) {
                     $numericIndex = true;
                     $typeHintName = $value;
-                    $validFor     = self::PARAMETER_TYPES|self::RETURN_TYPES;
+                    $validFor     = self::PARAMETER_TYPES | self::RETURN_TYPES;
                 }
                 if (!is_string($typeHintName) || !preg_match('/^\\w(?<!\\d)\\w*$/', $typeHintName)) {
                     throw new \InvalidArgumentException(
                         sprintf(
-                            "Option 'supportedBuiltinTypeHints's element %s " .
+                            "Option 'supportedBuiltinTypeHints's element %s ".
                                 "isn't a valid typehint string.",
                             var_export($typeHintName, true)
                         )
                     );
-                } elseif (!is_scalar($validFor)                                                                   ||
-                    (strval($validFor) != strval(intval($validFor)))                                        ||
-                    (intval($validFor) != (intval($validFor) & (self::PARAMETER_TYPES|self::RETURN_TYPES))) ||
+                } elseif (!is_scalar($validFor) ||
+                    (strval($validFor) != strval(intval($validFor))) ||
+                    (intval($validFor) != (intval($validFor) & (self::PARAMETER_TYPES | self::RETURN_TYPES))) ||
                     (intval($validFor) == 0)
                 ) {
                     throw new \InvalidArgumentException(
@@ -88,7 +88,7 @@ 
                         )
                     );
                 } else {
-                    $this->supportedBuiltinTypeHints[$typeHintName] = intval($validFor) & (self::PARAMETER_TYPES|self::RETURN_TYPES);
+                    $this->supportedBuiltinTypeHints[$typeHintName] = intval($validFor) & (self::PARAMETER_TYPES | self::RETURN_TYPES);
                 }
             }
         } else {
@@ -126,31 +126,31 @@ 
             $builtInTypeNames = [
                 'array'    => [
                     'introduced_version' => 50100,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'callable' => [
                     'introduced_version' => 50400,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'bool'     => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'float'    => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'int'      => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'string'   => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'iterable' => [
                     'introduced_version' => 70100,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'void'     => [
                     'introduced_version' => 70100,
@@ -158,7 +158,7 @@ 
                 ],
                 'object'   => [
                     'introduced_version' => 70200,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
             ];
             foreach ($builtInTypeNames as $typeHintName => $valid) {
@@ -172,7 +172,7 @@ 
     public function enterNode(Node $node)
     {
         if (
-            ($node instanceof Stmt\Function_)   ||
+            ($node instanceof Stmt\Function_) ||
             ($node instanceof Stmt\ClassMethod) ||
             ($node instanceof Expr\Closure)
         ) {
@@ -224,10 +224,10 @@ 
         }
         if ($node instanceof Node\NullableType) {
             // This *SHOULD* never be called, but correct behavior shorter than Exception.
-            return '?' . $this->getTypeAsString($node->type);
+            return '?'.$this->getTypeAsString($node->type);
         }
         if ($node instanceof Name) {
-            return ($node->isFullyQualified() ? '\\' : '') . $node->toString();
+            return ($node->isFullyQualified() ? '\\' : '').$node->toString();
         }
         return strval($node);
     }