Completed
Pull Request — master (#81)
by Loren
02:51
created
src/ReflectionFunction.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -106,14 +106,14 @@
 block discarded – undo
106 106
 
107 107
         return sprintf(
108 108
             $reflectionFormat,
109
-            $this->getDocComment() ? $this->getDocComment() . "\n" : '',
109
+            $this->getDocComment() ? $this->getDocComment()."\n" : '',
110 110
             $this->getName(),
111 111
             $this->getFileName(),
112 112
             $this->getStartLine(),
113 113
             $this->getEndLine(),
114 114
             count($this->getParameters()),
115
-            array_reduce($this->getParameters(), function ($str, ReflectionParameter $param) {
116
-                return $str . "\n    " . $param;
115
+            array_reduce($this->getParameters(), function($str, ReflectionParameter $param) {
116
+                return $str."\n    ".$param;
117 117
             }, '')
118 118
         );
119 119
     }
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     /**
33 33
      * Initializes reflection instance for given AST-node
34 34
      *
35
-     * @param string|\Closure $functionName The name of the function to reflect or a closure.
35
+     * @param string $functionName The name of the function to reflect or a closure.
36 36
      * @param Function_|null  $functionNode Function node AST
37 37
      */
38 38
     public function __construct($functionName, Function_ $functionNode = null)
Please login to merge, or discard this patch.
src/Instrument/PathResolver.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
 
51 51
         $isRelative = !$pathScheme && ($path[0] !== '/') && ($path[1] !== ':');
52 52
         if ($isRelative) {
53
-            $path = getcwd() . DIRECTORY_SEPARATOR . $path;
53
+            $path = getcwd().DIRECTORY_SEPARATOR.$path;
54 54
         }
55 55
 
56 56
         // resolve path parts (single dot, double dot and double delimiters)
Please login to merge, or discard this patch.
src/ReflectionClass.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
     /**
30 30
      * Initializes reflection instance
31 31
      *
32
-     * @param string|object $argument Class name or instance of object
32
+     * @param string|null $argument Class name or instance of object
33 33
      * @param ClassLike $classLikeNode AST node for class
34 34
      */
35 35
     public function __construct($argument, ClassLike $classLikeNode = null)
Please login to merge, or discard this patch.
src/bootstrap.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,4 +17,4 @@
 block discarded – undo
17 17
  */
18 18
 ReflectionEngine::init(new ComposerLocator());
19 19
 
20
-require(__DIR__ . '/polyfill.php');
20
+require(__DIR__.'/polyfill.php');
Please login to merge, or discard this patch.
src/ReflectionFileNamespace.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
         $comments   = $this->namespaceNode->getAttribute('comments');
181 181
 
182 182
         if ($comments) {
183
-            $docComment = (string)$comments[0];
183
+            $docComment = (string) $comments[0];
184 184
         }
185 185
 
186 186
         return $docComment;
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
327 327
         foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
328 328
             if ($namespaceLevelNode instanceof ClassLike) {
329 329
                 $classShortName = $namespaceLevelNode->name;
330
-                $className = $namespaceName ? $namespaceName .'\\' . $classShortName : $classShortName;
330
+                $className = $namespaceName ? $namespaceName.'\\'.$classShortName : $classShortName;
331 331
 
332 332
                 $namespaceLevelNode->setAttribute('fileName', $this->fileName);
333 333
                 $classes[$className] = new ReflectionClass($className, $namespaceLevelNode);
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
         foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
352 352
             if ($namespaceLevelNode instanceof Function_) {
353 353
                 $funcShortName = $namespaceLevelNode->name;
354
-                $functionName  = $namespaceName ? $namespaceName .'\\' . $funcShortName : $funcShortName;
354
+                $functionName  = $namespaceName ? $namespaceName.'\\'.$funcShortName : $funcShortName;
355 355
 
356 356
                 $namespaceLevelNode->setAttribute('fileName', $this->fileName);
357 357
                 $functions[$funcShortName] = new ReflectionFunction($functionName, $namespaceLevelNode);
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
390 390
             foreach ($this->namespaceNode->stmts as $namespaceLevelNode) {
391 391
                 if ($namespaceLevelNode instanceof FuncCall
392 392
                     && $namespaceLevelNode->name instanceof Name
393
-                    && (string)$namespaceLevelNode->name === 'define'
393
+                    && (string) $namespaceLevelNode->name === 'define'
394 394
                 ) {
395 395
                     $expressionSolver->process($namespaceLevelNode->args[0]->value);
396 396
                     $constantName = $expressionSolver->getValue();
Please login to merge, or discard this patch.
src/ReflectionParameter.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -128,11 +128,11 @@  discard block
 block discarded – undo
128 128
         if ($hasDefaultValue) {
129 129
             $defaultValue = $this->getDefaultValue();
130 130
             if (is_string($defaultValue) && strlen($defaultValue) > 15) {
131
-                $defaultValue = substr($defaultValue, 0, 15) . '...';
131
+                $defaultValue = substr($defaultValue, 0, 15).'...';
132 132
             }
133 133
             /* @see https://3v4l.org/DJOEb for behaviour changes */
134 134
             if (is_double($defaultValue) && fmod($defaultValue, 1.0) === 0.0) {
135
-                $defaultValue = (int)$defaultValue;
135
+                $defaultValue = (int) $defaultValue;
136 136
             }
137 137
 
138 138
             $defaultValue = str_replace('\\\\', '\\', var_export($defaultValue, true));
@@ -142,11 +142,11 @@  discard block
 block discarded – undo
142 142
             'Parameter #%d [ %s %s%s%s$%s%s ]',
143 143
             $this->parameterIndex,
144 144
             $isOptional ? '<optional>' : '<required>',
145
-            $parameterType ? ReflectionType::convertToDisplayType($parameterType) . ' ' : '',
145
+            $parameterType ? ReflectionType::convertToDisplayType($parameterType).' ' : '',
146 146
             $this->isVariadic() ? '...' : '',
147 147
             $this->isPassedByReference() ? '&' : '',
148 148
             $this->getName(),
149
-            ($isOptional && $hasDefaultValue) ? (' = ' . $defaultValue) : ''
149
+            ($isOptional && $hasDefaultValue) ? (' = '.$defaultValue) : ''
150 150
         );
151 151
     }
152 152
 
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
 
197 197
                 throw new ReflectionException("Can not resolve a class name for parameter");
198 198
             }
199
-            $className   = $parameterType->toString();
199
+            $className = $parameterType->toString();
200 200
             $classOrInterfaceExists = class_exists($className, false) || interface_exists($className, false);
201 201
 
202 202
             return $classOrInterfaceExists ? new \ReflectionClass($className) : new ReflectionClass($className);
Please login to merge, or discard this patch.
src/ReflectionExtension.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
     /**
31 31
      * Has extension been loaded by PHP.
32 32
      *
33
-     * @return true
33
+     * @return boolean
34 34
      *     Enabled extensions are always loaded.
35 35
      */
36 36
     public function wasIncluded()
Please login to merge, or discard this patch.
src/Traits/ReflectionClassLikeTrait.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -312,6 +312,9 @@
 block discarded – undo
312 312
         return new ReflectionExtension($extName);
313 313
     }
314 314
 
315
+    /**
316
+     * @return string
317
+     */
315 318
     public function getExtensionName()
316 319
     {
317 320
         if (!$this->classLikeNode) {
Please login to merge, or discard this patch.
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -134,17 +134,17 @@  discard block
 block discarded – undo
134 134
             }
135 135
         }
136 136
 
137
-        $buildString = function (array $items, $indentLevel = 4) {
137
+        $buildString = function(array $items, $indentLevel = 4) {
138 138
             if (!count($items)) {
139 139
                 return '';
140 140
             }
141
-            $indent = "\n" . str_repeat(' ', $indentLevel);
142
-            return $indent . implode($indent, explode("\n", implode("\n", $items)));
141
+            $indent = "\n".str_repeat(' ', $indentLevel);
142
+            return $indent.implode($indent, explode("\n", implode("\n", $items)));
143 143
         };
144
-        $buildConstants = function (array $items, $indentLevel = 4) {
144
+        $buildConstants = function(array $items, $indentLevel = 4) {
145 145
             $str = '';
146 146
             foreach ($items as $name => $value) {
147
-                $str .= "\n" . str_repeat(' ', $indentLevel);
147
+                $str .= "\n".str_repeat(' ', $indentLevel);
148 148
                 $str .= sprintf(
149 149
                     'Constant [ %s %s ] { %s }',
150 150
                     gettype($value),
@@ -168,8 +168,8 @@  discard block
 block discarded – undo
168 168
             ($isObject ? 'Object of class' : 'Class'),
169 169
             $modifiers,
170 170
             $this->getName(),
171
-            false !== $parentClass ? (' extends ' . $parentClass->getName()) : '',
172
-            $interfaceNames ? (' implements ' . implode(', ', $interfaceNames)) : '',
171
+            false !== $parentClass ? (' extends '.$parentClass->getName()) : '',
172
+            $interfaceNames ? (' implements '.implode(', ', $interfaceNames)) : '',
173 173
             $this->getFileName(),
174 174
             $this->getStartLine(),
175 175
             $this->getEndLine(),
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
     {
211 211
         if (!isset($this->constants)) {
212 212
             if ($this->classLikeNode) {
213
-                $this->constants = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance) {
213
+                $this->constants = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance) {
214 214
                     $result += $instance->getConstants();
215 215
                 });
216 216
                 $this->collectSelfConstants();
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
         $staticOrder   = [true, false];
257 257
         foreach ($staticOrder as $shouldBeStatic) {
258 258
             foreach ($properties as $property) {
259
-                $isStaticProperty     = $property->isStatic();
259
+                $isStaticProperty = $property->isStatic();
260 260
                 if ($shouldBeStatic !== $isStaticProperty) {
261 261
                     continue;
262 262
                 }
@@ -347,7 +347,7 @@  discard block
 block discarded – undo
347 347
     {
348 348
         if (!isset($this->interfaceClasses)) {
349 349
             if ($this->classLikeNode) {
350
-                $this->interfaceClasses = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance) {
350
+                $this->interfaceClasses = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance) {
351 351
                     if ($instance->isInterface()) {
352 352
                         $result[$instance->name] = $instance;
353 353
                     }
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
         if (!isset($this->methods)) {
395 395
             if ($this->classLikeNode) {
396 396
                 $directMethods = ReflectionMethod::collectFromClassNode($this->classLikeNode, $this);
397
-                $parentMethods = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance, $isParent) {
397
+                $parentMethods = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance, $isParent) {
398 398
                     $reflectionMethods = [];
399 399
                     foreach ($instance->getMethods() as $reflectionMethod) {
400 400
                         if (!$isParent || !$reflectionMethod->isPrivate()) {
@@ -477,9 +477,9 @@  discard block
 block discarded – undo
477 477
      */
478 478
     public function getName()
479 479
     {
480
-        $namespaceName = $this->namespaceName ? $this->namespaceName . '\\' : '';
480
+        $namespaceName = $this->namespaceName ? $this->namespaceName.'\\' : '';
481 481
 
482
-        return $namespaceName . $this->getShortName();
482
+        return $namespaceName.$this->getShortName();
483 483
     }
484 484
 
485 485
     /**
@@ -533,7 +533,7 @@  discard block
 block discarded – undo
533 533
         if (!isset($this->properties)) {
534 534
             if ($this->classLikeNode) {
535 535
                 $directProperties = ReflectionProperty::collectFromClassNode($this->classLikeNode, $this->getName());
536
-                $parentProperties = $this->recursiveCollect(function (array &$result, \ReflectionClass $instance, $isParent) {
536
+                $parentProperties = $this->recursiveCollect(function(array &$result, \ReflectionClass $instance, $isParent) {
537 537
                     $reflectionProperties = [];
538 538
                     foreach ($instance->getProperties() as $reflectionProperty) {
539 539
                         if (!$isParent || !$reflectionProperty->isPrivate()) {
@@ -630,7 +630,7 @@  discard block
 block discarded – undo
630 630
                         break;
631 631
                     }
632 632
                 }
633
-                $aliases[$adaptation->newName] = $traitName . '::'. $methodName;
633
+                $aliases[$adaptation->newName] = $traitName.'::'.$methodName;
634 634
             }
635 635
         }
636 636
 
@@ -1090,7 +1090,7 @@  discard block
 block discarded – undo
1090 1090
     {
1091 1091
         return
1092 1092
             interface_exists($this->getName(), false) ||
1093
-            trait_exists($this->getName(),     false) ||
1094
-            class_exists($this->getName(),     false);
1093
+            trait_exists($this->getName(), false) ||
1094
+            class_exists($this->getName(), false);
1095 1095
     }
1096 1096
 }
Please login to merge, or discard this patch.
src/Traits/ReflectionFunctionLikeTrait.php 2 patches
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -106,6 +106,9 @@  discard block
 block discarded – undo
106 106
         return new ReflectionExtension($extName);
107 107
     }
108 108
 
109
+    /**
110
+     * @return string
111
+     */
109 112
     public function getExtensionName()
110 113
     {
111 114
         if (!$this->functionLikeNode) {
@@ -223,7 +226,7 @@  discard block
 block discarded – undo
223 226
     /**
224 227
      * Gets the specified return type of a function
225 228
      *
226
-     * @return \ReflectionType
229
+     * @return null|ReflectionType
227 230
      *
228 231
      * @link http://php.net/manual/en/reflectionfunctionabstract.getreturntype.php
229 232
      */
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
         if ($this->functionLikeNode instanceof Function_ || $this->functionLikeNode instanceof ClassMethod) {
139 139
             $functionName = $this->functionLikeNode->name;
140 140
 
141
-            return $this->namespaceName ? $this->namespaceName . '\\' . $functionName : $functionName;
141
+            return $this->namespaceName ? $this->namespaceName.'\\'.$functionName : $functionName;
142 142
         }
143 143
 
144 144
         return false;
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
      */
230 230
     public function getReturnType()
231 231
     {
232
-        $isBuiltin  = false;
232
+        $isBuiltin = false;
233 233
         if ($this->functionLikeNode) {
234 234
             $returnType = $this->functionLikeNode->getReturnType();
235 235
             $isNullable = $returnType instanceof NullableType;
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
             }
256 256
             $isNullable = $nativeType->allowsNull();
257 257
             $isBuiltin = $nativeType->isBuiltin();
258
-            $returnType = (string)$nativeType;
258
+            $returnType = (string) $nativeType;
259 259
         }
260 260
 
261 261
         return new ReflectionType($returnType, $isNullable, $isBuiltin);
@@ -482,9 +482,9 @@  discard block
 block discarded – undo
482 482
         }
483 483
         if (method_exists($orig, 'hasType') && $orig->hasType()) {
484 484
             $typeRef = $orig->getType();
485
-            $stringType = ltrim((string)$typeRef, '?'); // ltrim() is precautionary.
485
+            $stringType = ltrim((string) $typeRef, '?'); // ltrim() is precautionary.
486 486
             if (PHP_VERSION_ID >= 70100 && $typeRef->allowsNull()) {
487
-                $stringType = '?' . $stringType;
487
+                $stringType = '?'.$stringType;
488 488
                 $nullableImplied = true;
489 489
             }
490 490
             $builder->setTypeHint($stringType);
Please login to merge, or discard this patch.