Passed
Pull Request — master (#321)
by Christoffer
02:39
created
src/Util/utils.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 
37 37
     return $count === 1
38 38
         ? $selected[0]
39
-        : \array_reduce($selected, function ($list, $item) use ($count, &$index) {
39
+        : \array_reduce($selected, function($list, $item) use ($count, &$index) {
40 40
             $list .= ($index > 0 && $index < ($count - 1) ? ', ' : '') . ($index === ($count - 1) ? ' or ' : '') .
41 41
                 $item;
42 42
             $index++;
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 
71 71
     $result = \array_keys($optionsByDistance);
72 72
 
73
-    \usort($result, function ($a, $b) use ($optionsByDistance) {
73
+    \usort($result, function($a, $b) use ($optionsByDistance) {
74 74
         return $optionsByDistance[$a] - $optionsByDistance[$b];
75 75
     });
76 76
 
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
  */
86 86
 function quotedOrList(array $items): string
87 87
 {
88
-    return orList(\array_map(function ($item) {
88
+    return orList(\array_map(function($item) {
89 89
         return '"' . $item . '"';
90 90
     }, $items));
91 91
 }
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
  */
99 99
 function arrayEvery(array $array, callable $fn): bool
100 100
 {
101
-    return \array_reduce($array, function ($result, $value) use ($fn) {
101
+    return \array_reduce($array, function($result, $value) use ($fn) {
102 102
         return $result && $fn($value);
103 103
     }, true);
104 104
 }
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
  */
111 111
 function arraySome(array $array, callable $fn)
112 112
 {
113
-    return \array_reduce($array, function ($result, $value) use ($fn) {
113
+    return \array_reduce($array, function($result, $value) use ($fn) {
114 114
         return $result || $fn($value);
115 115
     });
116 116
 }
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
  */
139 139
 function keyMap(array $array, callable $keyFn): array
140 140
 {
141
-    return \array_reduce($array, function ($map, $item) use ($keyFn) {
141
+    return \array_reduce($array, function($map, $item) use ($keyFn) {
142 142
         $map[$keyFn($item)] = $item;
143 143
         return $map;
144 144
     }, []);
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
  */
153 153
 function keyValueMap(array $array, callable $keyFn, callable $valFn): array
154 154
 {
155
-    return \array_reduce($array, function ($map, $item) use ($keyFn, $valFn) {
155
+    return \array_reduce($array, function($map, $item) use ($keyFn, $valFn) {
156 156
         $map[$keyFn($item)] = $valFn($item);
157 157
         return $map;
158 158
     }, []);
@@ -165,13 +165,13 @@  discard block
 block discarded – undo
165 165
 function promiseForMap(array $map): PromiseInterface
166 166
 {
167 167
     $keys             = \array_keys($map);
168
-    $promisesOrValues = \array_map(function ($name) use ($map) {
168
+    $promisesOrValues = \array_map(function($name) use ($map) {
169 169
         return $map[$name];
170 170
     }, $keys);
171 171
 
172
-    return \React\Promise\all($promisesOrValues)->then(function ($values) use ($keys) {
172
+    return \React\Promise\all($promisesOrValues)->then(function($values) use ($keys) {
173 173
         $i = 0;
174
-        return \array_reduce($values, function ($resolvedObject, $value) use ($keys, &$i) {
174
+        return \array_reduce($values, function($resolvedObject, $value) use ($keys, &$i) {
175 175
             $resolvedObject[$keys[$i++]] = $value;
176 176
             return $resolvedObject;
177 177
         }, []);
@@ -186,9 +186,9 @@  discard block
 block discarded – undo
186 186
  */
187 187
 function promiseReduce(array $values, callable $fn, $initial = null)
188 188
 {
189
-    return \array_reduce($values, function ($previous, $value) use ($fn) {
189
+    return \array_reduce($values, function($previous, $value) use ($fn) {
190 190
         return $previous instanceof PromiseInterface
191
-            ? $previous->then(function ($resolvedValue) use ($fn, $value) {
191
+            ? $previous->then(function($resolvedValue) use ($fn, $value) {
192 192
                 return $fn($resolvedValue, $value);
193 193
             })
194 194
             : $fn($previous, $value);
Please login to merge, or discard this patch.
src/Execution/ValuesResolver.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         }
58 58
 
59 59
         /** @var ArgumentNode[] $argumentNodeMap */
60
-        $argumentNodeMap = keyMap($argumentNodes, function (ArgumentNode $value) {
60
+        $argumentNodeMap = keyMap($argumentNodes, function(ArgumentNode $value) {
61 61
             return $value->getNameValue();
62 62
         });
63 63
 
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
         array $variableValues = []
141 141
     ): ?array {
142 142
         $directiveNode = $node->hasDirectives()
143
-            ? find($node->getDirectives(), function (NameAwareInterface $value) use ($directive) {
143
+            ? find($node->getDirectives(), function(NameAwareInterface $value) use ($directive) {
144 144
                 return $value->getNameValue() === $directive->getName();
145 145
             }) : null;
146 146
 
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
                 ]);
341 341
             }
342 342
             return new CoercedValue($parseResult);
343
-        } catch (InvalidTypeException|CoercingException $ex) {
343
+        } catch (InvalidTypeException | CoercingException $ex) {
344 344
             return new CoercedValue(null, [
345 345
                 $this->buildCoerceException(
346 346
                     sprintf('Expected type %s', (string)$type),
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
371 371
             return new CoercedValue($enumValue);
372 372
         }
373 373
 
374
-        $suggestions = suggestionList((string)$value, \array_map(function (EnumValue $enumValue) {
374
+        $suggestions = suggestionList((string)$value, \array_map(function(EnumValue $enumValue) {
375 375
             return $enumValue->getName();
376 376
         }, $type->getValues()));
377 377
 
Please login to merge, or discard this patch.
src/Execution/Execution.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         $data = $this->executeOperation($operationName, $context, $fieldCollector, $valuesResolver);
54 54
 
55 55
         if ($data instanceof PromiseInterface) {
56
-            $data->then(function ($resolvedData) use (&$data) {
56
+            $data->then(function($resolvedData) use (&$data) {
57 57
                 $data = $resolvedData;
58 58
             });
59 59
         }
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
         }
92 92
 
93 93
         if ($result instanceof PromiseInterface) {
94
-            return $result->then(null, function (ExecutionException $exception) use ($context) {
94
+            return $result->then(null, function(ExecutionException $exception) use ($context) {
95 95
                 $context->addError($exception);
96 96
                 return \React\Promise\resolve(null);
97 97
             });
Please login to merge, or discard this patch.
src/Execution/Strategy/SerialExecutionStrategy.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
     {
24 24
         return promiseReduce(
25 25
             \array_keys($fields),
26
-            function ($results, $fieldName) use ($parentType, $rootValue, $path, $fields) {
26
+            function($results, $fieldName) use ($parentType, $rootValue, $path, $fields) {
27 27
                 $fieldNodes  = $fields[$fieldName];
28 28
                 $fieldPath   = $path;
29 29
                 $fieldPath[] = $fieldName;
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
                 }
36 36
 
37 37
                 if ($result instanceof PromiseInterface) {
38
-                    return $result->then(function ($resolvedResult) use ($fieldName, $results) {
38
+                    return $result->then(function($resolvedResult) use ($fieldName, $results) {
39 39
                         $results[$fieldName] = $resolvedResult;
40 40
                         return $results;
41 41
                     });
Please login to merge, or discard this patch.
src/Execution/Strategy/AbstractExecutionStrategy.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
             );
253 253
 
254 254
             if ($result instanceof PromiseInterface) {
255
-                return $result->then(null, function ($exception) use ($fieldNode, $info) {
255
+                return $result->then(null, function($exception) use ($fieldNode, $info) {
256 256
                     return !$exception instanceof ExecutionException
257 257
                         ? $this->normalizeException($exception, [$fieldNode], $info->getPath())
258 258
                         : $exception;
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
     ) {
329 329
         try {
330 330
             $completed = $result instanceof PromiseInterface
331
-                ? $result->then(function ($resolvedResult) use ($returnType, $fieldNodes, $info, $path) {
331
+                ? $result->then(function($resolvedResult) use ($returnType, $fieldNodes, $info, $path) {
332 332
                     return $this->completeValue($returnType, $fieldNodes, $info, $path, $resolvedResult);
333 333
                 })
334 334
                 : $this->completeValue($returnType, $fieldNodes, $info, $path, $result);
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
             if ($completed instanceof PromiseInterface) {
337 337
                 // Note: we don't rely on a `catch` method, but we do expect "thenable"
338 338
                 // to take a second callback for the error case.
339
-                return $completed->then(null, function ($exception) use ($fieldNodes, $path, $returnType) {
339
+                return $completed->then(null, function($exception) use ($fieldNodes, $path, $returnType) {
340 340
                     $this->handleFieldError($exception, $fieldNodes, $path, $returnType);
341 341
                 });
342 342
             }
@@ -534,7 +534,7 @@  discard block
 block discarded – undo
534 534
             );
535 535
 
536 536
         if ($runtimeType instanceof PromiseInterface) {
537
-            return $runtimeType->then(function ($resolvedRuntimeType) use (
537
+            return $runtimeType->then(function($resolvedRuntimeType) use (
538 538
                 $returnType,
539 539
                 $fieldNodes,
540 540
                 $info,
@@ -588,7 +588,7 @@  discard block
 block discarded – undo
588 588
             $isTypeOf = $returnType->isTypeOf($result, $this->context->getContextValue(), $info);
589 589
 
590 590
             if ($isTypeOf instanceof PromiseInterface) {
591
-                return $isTypeOf->then(function ($resolvedIsTypeOf) use ($returnType, $result, $fieldNodes, $path) {
591
+                return $isTypeOf->then(function($resolvedIsTypeOf) use ($returnType, $result, $fieldNodes, $path) {
592 592
                     if (true === $resolvedIsTypeOf) {
593 593
                         return $this->executeSubFields($returnType, $fieldNodes, $path, $result);
594 594
                     }
@@ -877,7 +877,7 @@  discard block
 block discarded – undo
877 877
         }
878 878
 
879 879
         if (!empty($promises)) {
880
-            return \React\Promise\all($promises)->then(function ($resolvedPromises) use ($possibleTypes) {
880
+            return \React\Promise\all($promises)->then(function($resolvedPromises) use ($possibleTypes) {
881 881
                 foreach ($resolvedPromises as $index => $result) {
882 882
                     if (true === $result) {
883 883
                         return $possibleTypes[$index];
Please login to merge, or discard this patch.