Passed
Pull Request — master (#1487)
by Martin Poirier
03:02
created
src/Metadata/Driver/TypedPropertiesDriver.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
      */
124 124
     private function shouldTypeHint(?ReflectionType $reflectionType): bool
125 125
     {
126
-        if (!$reflectionType instanceof ReflectionNamedType) {
126
+        if ( ! $reflectionType instanceof ReflectionNamedType) {
127 127
             return false;
128 128
         }
129 129
 
Please login to merge, or discard this patch.
src/Metadata/Driver/DocBlockDriver/DocBlockTypeResolver.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -108,25 +108,25 @@  discard block
 block discarded – undo
108 108
         if ($type instanceof ArrayTypeNode) {
109 109
             $resolvedType = $this->resolveTypeFromTypeNode($type->type, $reflector);
110 110
 
111
-            return 'array<' . $resolvedType . '>';
111
+            return 'array<'.$resolvedType.'>';
112 112
         }
113 113
 
114 114
         // Generic array syntax: array<Product> | array<\Foo\Bar\Product> | array<int,Product>
115 115
         if ($type instanceof GenericTypeNode) {
116 116
             if ($this->isSimpleType($type->type, 'array')) {
117
-                $resolvedTypes = array_map(function (TypeNode $node) use ($reflector) {
117
+                $resolvedTypes = array_map(function(TypeNode $node) use ($reflector) {
118 118
                     return $this->resolveTypeFromTypeNode($node, $reflector);
119 119
                 }, $type->genericTypes);
120 120
 
121
-                return 'array<' . implode(',', $resolvedTypes) . '>';
121
+                return 'array<'.implode(',', $resolvedTypes).'>';
122 122
             }
123 123
 
124 124
             if ($this->isSimpleType($type->type, 'list')) {
125
-                $resolvedTypes = array_map(function (TypeNode $node) use ($reflector) {
125
+                $resolvedTypes = array_map(function(TypeNode $node) use ($reflector) {
126 126
                     return $this->resolveTypeFromTypeNode($node, $reflector);
127 127
                 }, $type->genericTypes);
128 128
 
129
-                return 'array<int, ' . implode(',', $resolvedTypes) . '>';
129
+                return 'array<int, '.implode(',', $resolvedTypes).'>';
130 130
             }
131 131
 
132 132
             throw new \InvalidArgumentException(sprintf("Can't use non-array generic type %s for collection in %s:%s", (string) $type->type, $reflector->getDeclaringClass()->getName(), $reflector->getName()));
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
             return [];
150 150
         }
151 151
 
152
-        return array_merge(...array_map(static function ($node) {
152
+        return array_merge(...array_map(static function($node) {
153 153
             if ($node->type instanceof UnionTypeNode) {
154 154
                 return $node->type->types;
155 155
             }
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
      */
194 194
     private function filterNullFromTypes(array $types): array
195 195
     {
196
-        return array_values(array_filter(array_map(function (TypeNode $node) {
196
+        return array_values(array_filter(array_map(function(TypeNode $node) {
197 197
             return $this->isNullType($node) ? null : $node;
198 198
         }, $types)));
199 199
     }
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
      */
237 237
     private function resolveTypeFromTypeNode(TypeNode $typeNode, $reflector): string
238 238
     {
239
-        if (!($typeNode instanceof IdentifierTypeNode)) {
239
+        if ( ! ($typeNode instanceof IdentifierTypeNode)) {
240 240
             throw new \InvalidArgumentException(sprintf("Can't use unsupported type %s for collection in %s:%s", (string) $typeNode, $reflector->getDeclaringClass()->getName(), $reflector->getName()));
241 241
         }
242 242
 
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
             return $typeHint;
253 253
         }
254 254
 
255
-        $expandedClassName = $declaringClass->getNamespaceName() . '\\' . $typeHint;
255
+        $expandedClassName = $declaringClass->getNamespaceName().'\\'.$typeHint;
256 256
         if ($this->isClassOrInterface($expandedClassName)) {
257 257
             return $expandedClassName;
258 258
         }
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 
287 287
     private function endsWith(string $statementClassToCheck, string $typeHintToSearchFor): bool
288 288
     {
289
-        $typeHintToSearchFor = '\\' . $typeHintToSearchFor;
289
+        $typeHintToSearchFor = '\\'.$typeHintToSearchFor;
290 290
 
291 291
         return substr($statementClassToCheck, -strlen($typeHintToSearchFor)) === $typeHintToSearchFor;
292 292
     }
@@ -307,7 +307,7 @@  discard block
 block discarded – undo
307 307
         preg_match_all(self::GROUP_USE_STATEMENTS_REGEX, $classContents, $foundGroupUseStatements);
308 308
         for ($useStatementIndex = 0; $useStatementIndex < count($foundGroupUseStatements[0]); $useStatementIndex++) {
309 309
             foreach (explode(',', $foundGroupUseStatements[2][$useStatementIndex]) as $singleUseStatement) {
310
-                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]) . trim($singleUseStatement);
310
+                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]).trim($singleUseStatement);
311 311
             }
312 312
         }
313 313
 
@@ -346,7 +346,7 @@  discard block
 block discarded – undo
346 346
      */
347 347
     private function resolveType(string $typeHint, $reflector): string
348 348
     {
349
-        if (!$this->hasGlobalNamespacePrefix($typeHint) && !$this->isPrimitiveType($typeHint)) {
349
+        if ( ! $this->hasGlobalNamespacePrefix($typeHint) && ! $this->isPrimitiveType($typeHint)) {
350 350
             $typeHint = $this->expandClassNameUsingUseStatements($typeHint, $this->getDeclaringClassOrTrait($reflector), $reflector);
351 351
         }
352 352
 
@@ -364,15 +364,15 @@  discard block
 block discarded – undo
364 364
     private function resolveTypeFromDocblock($reflector): array
365 365
     {
366 366
         $docComment = $reflector->getDocComment();
367
-        if (!$docComment && PHP_VERSION_ID >= 80000 && $reflector instanceof \ReflectionProperty && $reflector->isPromoted()) {
367
+        if ( ! $docComment && PHP_VERSION_ID >= 80000 && $reflector instanceof \ReflectionProperty && $reflector->isPromoted()) {
368 368
             $constructor = $reflector->getDeclaringClass()->getConstructor();
369
-            if (!$constructor) {
369
+            if ( ! $constructor) {
370 370
                 return [];
371 371
             }
372 372
 
373 373
             $docComment = $constructor->getDocComment();
374 374
 
375
-            if (!$docComment) {
375
+            if ( ! $docComment) {
376 376
                 return [];
377 377
             }
378 378
 
@@ -382,7 +382,7 @@  discard block
 block discarded – undo
382 382
             return $this->flattenParamTagValueTypes($reflector->getName(), $phpDocNode->getParamTagValues());
383 383
         }
384 384
 
385
-        if (!$docComment) {
385
+        if ( ! $docComment) {
386 386
             return [];
387 387
         }
388 388
 
@@ -426,7 +426,7 @@  discard block
 block discarded – undo
426 426
 
427 427
                     return sprintf('array<%s>', implode(
428 428
                         ',',
429
-                        array_map(static function (string $type) use ($reflector, $self) {
429
+                        array_map(static function(string $type) use ($reflector, $self) {
430 430
                             return $self->resolveType(trim($type), $reflector);
431 431
                         }, $types)
432 432
                     ));
Please login to merge, or discard this patch.