Passed
Pull Request — master (#164)
by Christoffer
02:28
created
src/Execution/ExecutionStrategy.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
         if ($isContainsPromise) {
252 252
             $keys    = array_keys($finalResults);
253 253
             $promise = \React\Promise\all(array_values($finalResults));
254
-            $promise->then(function ($values) use ($keys, &$finalResults) {
254
+            $promise->then(function($values) use ($keys, &$finalResults) {
255 255
                 foreach ($values as $i => $value) {
256 256
                     $finalResults[$keys[$i]] = $value;
257 257
                 }
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
 
286 286
         $promise = new \React\Promise\FulfilledPromise([]);
287 287
 
288
-        $resolve = function ($results, $fieldName, $path, $objectType, $rootValue, $fieldNodes) {
288
+        $resolve = function($results, $fieldName, $path, $objectType, $rootValue, $fieldNodes) {
289 289
             $fieldPath   = $path;
290 290
             $fieldPath[] = $fieldName;
291 291
             try {
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
 
297 297
             if ($this->isPromise($result)) {
298 298
                 /** @var ExtendedPromiseInterface $result */
299
-                return $result->then(function ($resolvedResult) use ($fieldName, $results) {
299
+                return $result->then(function($resolvedResult) use ($fieldName, $results) {
300 300
                     $results[$fieldName] = $resolvedResult;
301 301
                     return $results;
302 302
                 });
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
         };
309 309
 
310 310
         foreach ($fields as $fieldName => $fieldNodes) {
311
-            $promise = $promise->then(function ($resolvedResults) use (
311
+            $promise = $promise->then(function($resolvedResults) use (
312 312
                 $resolve,
313 313
                 $fieldName,
314 314
                 $path,
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
             });
321 321
         }
322 322
 
323
-        $promise->then(function ($resolvedResults) use (&$finalResults) {
323
+        $promise->then(function($resolvedResults) use (&$finalResults) {
324 324
             $finalResults = $resolvedResults ?? [];
325 325
         });
326 326
 
@@ -504,7 +504,7 @@  discard block
 block discarded – undo
504 504
             if ($this->isPromise($completed)) {
505 505
                 $context = $this->context;
506 506
                 /** @var ExtendedPromiseInterface $completed */
507
-                return $completed->then(null, function ($error) use ($context, $fieldNodes, $path) {
507
+                return $completed->then(null, function($error) use ($context, $fieldNodes, $path) {
508 508
                     //@TODO Handle $error better
509 509
                     if ($error instanceof \Exception) {
510 510
                         $context->addError($this->buildLocatedError($error, $fieldNodes, $path));
@@ -579,7 +579,7 @@  discard block
 block discarded – undo
579 579
     ) {
580 580
         if ($this->isPromise($result)) {
581 581
             /** @var ExtendedPromiseInterface $result */
582
-            return $result->then(function (&$value) use ($returnType, $fieldNodes, $info, $path) {
582
+            return $result->then(function(&$value) use ($returnType, $fieldNodes, $info, $path) {
583 583
                 return $this->completeValue($returnType, $fieldNodes, $info, $path, $value);
584 584
             });
585 585
         }
@@ -667,7 +667,7 @@  discard block
 block discarded – undo
667 667
 
668 668
         if ($this->isPromise($runtimeType)) {
669 669
             /** @var ExtendedPromiseInterface $runtimeType */
670
-            return $runtimeType->then(function ($resolvedRuntimeType) use (
670
+            return $runtimeType->then(function($resolvedRuntimeType) use (
671 671
                 $returnType,
672 672
                 $fieldNodes,
673 673
                 $info,
@@ -792,7 +792,7 @@  discard block
 block discarded – undo
792 792
 
793 793
         if (!empty($promisedIsTypeOfResults)) {
794 794
             return \React\Promise\all($promisedIsTypeOfResults)
795
-                ->then(function ($isTypeOfResults) use ($possibleTypes) {
795
+                ->then(function($isTypeOfResults) use ($possibleTypes) {
796 796
                     foreach ($isTypeOfResults as $index => $result) {
797 797
                         if ($result) {
798 798
                             return $possibleTypes[$index];
Please login to merge, or discard this patch.
src/Language/utils.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@
 block discarded – undo
146 146
  */
147 147
 function locationsShorthandToArray(array $locations): array
148 148
 {
149
-    return array_map(function ($shorthand) {
149
+    return array_map(function($shorthand) {
150 150
         return locationShorthandToArray($shorthand);
151 151
     }, $locations);
152 152
 }
Please login to merge, or discard this patch.
src/Execution/ValuesHelper.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
             return $coercedValues;
63 63
         }
64 64
 
65
-        $argumentNodeMap = keyMap($argumentNodes, function (ArgumentNode $value) {
65
+        $argumentNodeMap = keyMap($argumentNodes, function(ArgumentNode $value) {
66 66
             return $value->getNameValue();
67 67
         });
68 68
 
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
         array $variableValues = []
134 134
     ): ?array {
135 135
         $directiveNode = $node->hasDirectives()
136
-            ? find($node->getDirectives(), function (NameAwareInterface $value) use ($directive) {
136
+            ? find($node->getDirectives(), function(NameAwareInterface $value) use ($directive) {
137 137
                 return $value->getNameValue() === $directive->getName();
138 138
             }) : null;
139 139
 
Please login to merge, or discard this patch.
src/Util/ValueHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -168,7 +168,7 @@
 block discarded – undo
168 168
         $resolvedValues = [];
169 169
 
170 170
         /** @var ObjectFieldNode[] $fieldNodes */
171
-        $fieldNodes = keyMap($node->getFields(), function (ObjectFieldNode $value) {
171
+        $fieldNodes = keyMap($node->getFields(), function(ObjectFieldNode $value) {
172 172
             return $value->getNameValue();
173 173
         });
174 174
 
Please login to merge, or discard this patch.
src/Util/utils.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
     return $count === 1
34 34
         ? $selected[0]
35
-        : \array_reduce($selected, function ($list, $item) use ($count, &$index) {
35
+        : \array_reduce($selected, function($list, $item) use ($count, &$index) {
36 36
             $list .= ($index > 0 && $index < ($count - 1) ? ', ' : '') . ($index === ($count - 1) ? ' or ' : '') .
37 37
                 $item;
38 38
             $index++;
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 
67 67
     $result = \array_keys($optionsByDistance);
68 68
 
69
-    \usort($result, function ($a, $b) use ($optionsByDistance) {
69
+    \usort($result, function($a, $b) use ($optionsByDistance) {
70 70
         return $optionsByDistance[$a] - $optionsByDistance[$b];
71 71
     });
72 72
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
  */
82 82
 function quotedOrList(array $items): string
83 83
 {
84
-    return orList(array_map(function ($item) {
84
+    return orList(array_map(function($item) {
85 85
         return '"' . $item . '"';
86 86
     }, $items));
87 87
 }
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
  */
96 96
 function arrayEvery(array $array, callable $fn): bool
97 97
 {
98
-    return array_reduce($array, function ($result, $value) use ($fn) {
98
+    return array_reduce($array, function($result, $value) use ($fn) {
99 99
         return $result && $fn($value);
100 100
     }, true);
101 101
 }
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
  */
108 108
 function arraySome(array $array, callable $fn)
109 109
 {
110
-    return array_reduce($array, function ($result, $value) use ($fn) {
110
+    return array_reduce($array, function($result, $value) use ($fn) {
111 111
         return $result || $fn($value);
112 112
     });
113 113
 }
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
  */
136 136
 function keyMap(array $array, callable $keyFn): array
137 137
 {
138
-    return array_reduce($array, function ($map, $item) use ($keyFn) {
138
+    return array_reduce($array, function($map, $item) use ($keyFn) {
139 139
         $map[$keyFn($item)] = $item;
140 140
         return $map;
141 141
     }, []);
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
  */
150 150
 function keyValueMap(array $array, callable $keyFn, callable $valFn): array
151 151
 {
152
-    return array_reduce($array, function ($map, $item) use ($keyFn, $valFn) {
152
+    return array_reduce($array, function($map, $item) use ($keyFn, $valFn) {
153 153
         $map[$keyFn($item)] = $valFn($item);
154 154
         return $map;
155 155
     }, []);
Please login to merge, or discard this patch.
src/Util/TypeHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
                 // If both types are abstract, then determine if there is any intersection
93 93
                 // between possible concrete types of each.
94 94
                 return arraySome($schema->getPossibleTypes($typeA),
95
-                    function (TypeInterface $type) use ($schema, $typeB) {
95
+                    function(TypeInterface $type) use ($schema, $typeB) {
96 96
                         return $schema->isPossibleType($typeB, $type);
97 97
                     });
98 98
             }
Please login to merge, or discard this patch.
src/Schema/Extension/ExtensionContext.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
     public function getExtendedTypes(): array
250 250
     {
251 251
         return \array_merge(
252
-            \array_map(function ($type) {
252
+            \array_map(function($type) {
253 253
                 return $this->getExtendedType($type);
254 254
             }, \array_values($this->schema->getTypeMap())),
255 255
             $this->definitionBuilder->buildTypes(\array_values($this->typeDefinitionMap))
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
 
269 269
         return \array_merge(
270 270
             $existingDirectives,
271
-            \array_map(function (DirectiveDefinitionNode $node) {
271
+            \array_map(function(DirectiveDefinitionNode $node) {
272 272
                 return $this->definitionBuilder->buildDirective($node);
273 273
             }, $this->directiveDefinitions)
274 274
         );
@@ -395,10 +395,10 @@  discard block
 block discarded – undo
395 395
         return GraphQLObjectType([
396 396
             'name'              => $typeName,
397 397
             'description'       => $type->getDescription(),
398
-            'interfaces'        => function () use ($type) {
398
+            'interfaces'        => function() use ($type) {
399 399
                 return $this->extendImplementedInterfaces($type);
400 400
             },
401
-            'fields'            => function () use ($type) {
401
+            'fields'            => function() use ($type) {
402 402
                 return $this->extendFieldMap($type);
403 403
             },
404 404
             'astNode'           => $type->getAstNode(),
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
425 425
         return GraphQLInterfaceType([
426 426
             'name'              => $typeName,
427 427
             'description'       => $type->getDescription(),
428
-            'fields'            => function () use ($type) {
428
+            'fields'            => function() use ($type) {
429 429
                 return $this->extendFieldMap($type);
430 430
             },
431 431
             'astNode'           => $type->getAstNode(),
@@ -444,7 +444,7 @@  discard block
 block discarded – undo
444 444
         return GraphQLUnionType([
445 445
             'name'        => $type->getName(),
446 446
             'description' => $type->getDescription(),
447
-            'types'       => \array_map(function ($unionType) {
447
+            'types'       => \array_map(function($unionType) {
448 448
                 return $this->getExtendedType($unionType);
449 449
             }, $type->getTypes()),
450 450
             'astNode'     => $type->getAstNode(),
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
      */
460 460
     protected function extendImplementedInterfaces(ObjectType $type): array
461 461
     {
462
-        $interfaces = \array_map(function (InterfaceType $interface) {
462
+        $interfaces = \array_map(function(InterfaceType $interface) {
463 463
             return $this->getExtendedType($interface);
464 464
         }, $type->getInterfaces());
465 465
 
@@ -501,7 +501,7 @@  discard block
 block discarded – undo
501 501
                 'description'       => $field->getDescription(),
502 502
                 'deprecationReason' => $field->getDeprecationReason(),
503 503
                 'type'              => $this->extendFieldType($field->getType()),
504
-                'args'              => keyMap($field->getArguments(), function (Argument $argument) {
504
+                'args'              => keyMap($field->getArguments(), function(Argument $argument) {
505 505
                     return $argument->getName();
506 506
                 }),
507 507
                 'astNode'           => $field->getAstNode(),
Please login to merge, or discard this patch.
src/Schema/Building/BuilderContext.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
      */
120 120
     public function buildTypes(): array
121 121
     {
122
-        return \array_map(function (TypeDefinitionNodeInterface $definition) {
122
+        return \array_map(function(TypeDefinitionNodeInterface $definition) {
123 123
             return $this->definitionBuilder->buildType($definition);
124 124
         }, \array_values($this->typeDefinitionMap));
125 125
     }
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
      */
130 130
     public function buildDirectives(): array
131 131
     {
132
-        $directives = \array_map(function (DirectiveDefinitionNode $definition) {
132
+        $directives = \array_map(function(DirectiveDefinitionNode $definition) {
133 133
             return $this->definitionBuilder->buildDirective($definition);
134 134
         }, $this->directiveDefinitions);
135 135
 
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
         ];
141 141
 
142 142
         foreach ($specifiedDirectivesMap as $name => $directive) {
143
-            if (!arraySome($directives, function (Directive $directive) use ($name) {
143
+            if (!arraySome($directives, function(Directive $directive) use ($name) {
144 144
                 return $directive->getName() === $name;
145 145
             })) {
146 146
                 $directives[] = $directive;
Please login to merge, or discard this patch.
src/Schema/Schema.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
      */
127 127
     public function getDirective(string $name): ?Directive
128 128
     {
129
-        return find($this->directives, function (Directive $directive) use ($name) {
129
+        return find($this->directives, function(Directive $directive) use ($name) {
130 130
             return $directive->getName() === $name;
131 131
         });
132 132
     }
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
             );
180 180
 
181 181
             $this->possibleTypesMap[$abstractTypeName] = \array_reduce($possibleTypes,
182
-                function (array $map, TypeInterface $type) {
182
+                function(array $map, TypeInterface $type) {
183 183
                     /** @var NameAwareInterface $type */
184 184
                     $map[$type->getName()] = true;
185 185
                     return $map;
@@ -384,7 +384,7 @@  discard block
 block discarded – undo
384 384
         if ($type instanceof ObjectType || $type instanceof InterfaceType) {
385 385
             foreach ($type->getFields() as $field) {
386 386
                 if ($field->hasArguments()) {
387
-                    $fieldArgTypes = \array_map(function (Argument $argument) {
387
+                    $fieldArgTypes = \array_map(function(Argument $argument) {
388 388
                         return $argument->getType();
389 389
                     }, $field->getArguments());
390 390
 
@@ -418,7 +418,7 @@  discard block
 block discarded – undo
418 418
             return $map;
419 419
         }
420 420
 
421
-        return \array_reduce($directive->getArguments(), function ($map, Argument $argument) {
421
+        return \array_reduce($directive->getArguments(), function($map, Argument $argument) {
422 422
             return $this->typeMapReducer($map, $argument->getType());
423 423
         }, $map);
424 424
     }
Please login to merge, or discard this patch.