goaop / parser-reflection

src/ReflectionExtension.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     /**
      * Has extension been loaded by PHP.
      *
-     * @return true
+     * @return boolean
      *     Enabled extensions are always loaded.
      */
     public function wasIncluded()

src/ReflectionFunction.php

Doc Comments +1 added lines, -1 removed lines

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

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 +17 added lines, -17 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()) {
@@ -477,9 +477,9 @@ 
      */
     public function getName()
     {
-        $namespaceName = $this->namespaceName ? $this->namespaceName . '\\' : '';
+        $namespaceName = $this->namespaceName ? $this->namespaceName.'\\' : '';
 
-        return $namespaceName . $this->getShortName();
+        return $namespaceName.$this->getShortName();
     }
 
     /**
@@ -533,7 +533,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()) {
@@ -630,7 +630,7 @@ 
                         break;
                     }
                 }
-                $aliases[$adaptation->newName] = $traitName . '::'. $methodName;
+                $aliases[$adaptation->newName] = $traitName.'::'.$methodName;
             }
         }
 
@@ -1090,7 +1090,7 @@ 
     {
         return
             interface_exists($this->getName(), false) ||
-            trait_exists(    $this->getName(), false) ||
-            class_exists(    $this->getName(), false);
+            trait_exists($this->getName(), false) ||
+            class_exists($this->getName(), false);
     }
 }

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 +8 added lines, -8 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);
@@ -505,9 +505,9 @@ 
             $fakeParamNode->setAttribute('prohibit_null', true);
         }
         return new ReflectionParameter(
-            $this->getName(),     // Calling function name:   Unused.
-            $orig->getName(),     // Parameter variable name: Unused.
-            $fakeParamNode,       // Synthetic parse node.
+            $this->getName(), // Calling function name:   Unused.
+            $orig->getName(), // Parameter variable name: Unused.
+            $fakeParamNode, // Synthetic parse node.
             $orig->getPosition(), // Parameter index.
             $this                 // Function or method being described.
         );

src/ReflectionMethod.php

Spacing +4 added lines, -4 removed lines

@@ -133,12 +133,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}{$returnFormat}\n}\n",
-            $this->getDocComment() ? $this->getDocComment() . "\n" : '',
+            $this->getDocComment() ? $this->getDocComment()."\n" : '',
             $prototype ? ", overwrites {$prototypeClass}, prototype {$prototypeClass}" : '',
             $this->isConstructor() ? ', ctor' : '',
             $this->isDestructor() ? ', dtor' : '',
@@ -410,7 +410,7 @@ 
     {
         return
             interface_exists($this->className, false) ||
-            trait_exists(    $this->className, false) ||
-            class_exists(    $this->className, false);
+            trait_exists($this->className, false) ||
+            class_exists($this->className, false);
     }
 }

src/ReflectionProperty.php

Spacing +2 added lines, -2 removed lines

@@ -351,7 +351,7 @@
     {
         return
             interface_exists($this->className, false) ||
-            trait_exists(    $this->className, false) ||
-            class_exists(    $this->className, false);
+            trait_exists($this->className, false) ||
+            class_exists($this->className, false);
     }
 }

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 +4 added lines, -4 removed lines

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