simPod / graphql-php

src/Type/Definition/FieldDefinition.php

Spacing +6 added lines, -6 removed lines

@@ -90,7 +90,7 @@ 
         if (is_callable($fields)) {
             $fields = $fields();
         }
-        if (! is_array($fields)) {
+        if (!is_array($fields)) {
             throw new InvariantViolation(
                 sprintf('%s fields must be an array or a callable which returns such an array.', $type->name)
             );
@@ -98,8 +98,8 @@ 
         $map = [];
         foreach ($fields as $name => $field) {
             if (is_array($field)) {
-                if (! isset($field['name'])) {
-                    if (! is_string($name)) {
+                if (!isset($field['name'])) {
+                    if (!is_string($name)) {
                         throw new InvariantViolation(
                             sprintf(
                                 '%s fields must be an associative array with field names as keys or a function which returns such an array.',
@@ -110,7 +110,7 @@ 
 
                     $field['name'] = $name;
                 }
-                if (isset($field['args']) && ! is_array($field['args'])) {
+                if (isset($field['args']) && !is_array($field['args'])) {
                     throw new InvariantViolation(
                         sprintf('%s.%s args must be an array.', $type->name, $name)
                     );
@@ -119,7 +119,7 @@ 
             } elseif ($field instanceof self) {
                 $fieldDef = $field;
             } else {
-                if (! is_string($name) || ! $field) {
+                if (!is_string($name) || !$field) {
                     throw new InvariantViolation(
                         sprintf(
                             '%s.%s field config must be an array, but got: %s',
@@ -207,7 +207,7 @@ 
             throw new InvariantViolation(sprintf('%s.%s: %s', $parentType->name, $this->name, $e->getMessage()));
         }
         Utils::invariant(
-            ! isset($this->config['isDeprecated']),
+            !isset($this->config['isDeprecated']),
             sprintf(
                 '%s.%s should provide "deprecationReason" instead of "isDeprecated".',
                 $parentType->name,

src/Type/Definition/InputObjectType.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -67,7 +67,7 @@ 
             $fields       = $this->config['fields'] ?? [];
             $fields       = is_callable($fields) ? call_user_func($fields) : $fields;
 
-            if (! is_array($fields)) {
+            if (!is_array($fields)) {
                 throw new InvariantViolation(
                     sprintf('%s fields must be an array or a callable which returns such an array.', $this->name)
                 );
@@ -96,7 +96,7 @@ 
         parent::assertValid();
 
         Utils::invariant(
-            ! empty($this->getFields()),
+            !empty($this->getFields()),
             sprintf(
                 '%s fields must be an associative array with field names as keys or a callable which returns such an array.',
                 $this->name

src/Deferred.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     public static function runQueue()
     {
         $q = self::$queue;
-        while ($q && ! $q->isEmpty()) {
+        while ($q && !$q->isEmpty()) {
             /** @var self $dfd */
             $dfd = $q->dequeue();
             $dfd->run();

src/Executor/ReferenceExecutor.php

Spacing +44 added lines, -46 removed lines

@@ -62,7 +62,7 @@ 
 
     private function __construct(ExecutionContext $context)
     {
-        if (! self::$UNDEFINED) {
+        if (!self::$UNDEFINED) {
             self::$UNDEFINED = Utils::undefined();
         }
         $this->exeContext    = $context;
@@ -140,10 +140,10 @@ 
         foreach ($documentNode->definitions as $definition) {
             switch ($definition->kind) {
                 case NodeKind::OPERATION_DEFINITION:
-                    if (! $operationName && $operation) {
+                    if (!$operationName && $operation) {
                         $hasMultipleAssumedOperations = true;
                     }
-                    if (! $operationName ||
+                    if (!$operationName ||
                         (isset($definition->name) && $definition->name->value === $operationName)) {
                         $operation = $definition;
                     }
@@ -177,7 +177,7 @@ 
                 $errors = array_merge($errors, $coercionErrors);
             }
         }
-        if (! empty($errors)) {
+        if (!empty($errors)) {
             return $errors;
         }
         Utils::invariant($operation, 'Has operation if no errors.');
@@ -221,7 +221,7 @@ 
     private function buildResponse($data)
     {
         if ($this->isPromise($data)) {
-            return $data->then(function ($resolved) {
+            return $data->then(function($resolved) {
                 return $this->buildResponse($resolved);
             });
         }
@@ -250,12 +250,11 @@ 
         // Similar to completeValueCatchingError.
         try {
             $result = $operation->operation === 'mutation' ?
-                $this->executeFieldsSerially($type, $rootValue, $path, $fields) :
-                $this->executeFields($type, $rootValue, $path, $fields);
+                $this->executeFieldsSerially($type, $rootValue, $path, $fields) : $this->executeFields($type, $rootValue, $path, $fields);
             if ($this->isPromise($result)) {
                 return $result->then(
                     null,
-                    function ($error) {
+                    function($error) {
                         $this->exeContext->addError($error);
                         return $this->exeContext->promises->createFulfilled(null);
                     }
@@ -280,7 +279,7 @@ 
         switch ($operation->operation) {
             case 'query':
                 $queryType = $schema->getQueryType();
-                if (! $queryType) {
+                if (!$queryType) {
                     throw new Error(
                         'Schema does not define the required query root type.',
                         [$operation]
@@ -289,7 +288,7 @@ 
                 return $queryType;
             case 'mutation':
                 $mutationType = $schema->getMutationType();
-                if (! $mutationType) {
+                if (!$mutationType) {
                     throw new Error(
                         'Schema is not configured for mutations.',
                         [$operation]
@@ -298,7 +297,7 @@ 
                 return $mutationType;
             case 'subscription':
                 $subscriptionType = $schema->getSubscriptionType();
-                if (! $subscriptionType) {
+                if (!$subscriptionType) {
                     throw new Error(
                         'Schema is not configured for subscriptions.',
                         [$operation]
@@ -336,18 +335,18 @@ 
         foreach ($selectionSet->selections as $selection) {
             switch ($selection->kind) {
                 case NodeKind::FIELD:
-                    if (! $this->shouldIncludeNode($selection)) {
+                    if (!$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $name = self::getFieldEntryKey($selection);
-                    if (! isset($fields[$name])) {
+                    if (!isset($fields[$name])) {
                         $fields[$name] = new ArrayObject();
                     }
                     $fields[$name][] = $selection;
                     break;
                 case NodeKind::INLINE_FRAGMENT:
-                    if (! $this->shouldIncludeNode($selection) ||
-                        ! $this->doesFragmentConditionMatch($selection, $runtimeType)
+                    if (!$this->shouldIncludeNode($selection) ||
+                        !$this->doesFragmentConditionMatch($selection, $runtimeType)
                     ) {
                         break;
                     }
@@ -360,13 +359,13 @@ 
                     break;
                 case NodeKind::FRAGMENT_SPREAD:
                     $fragName = $selection->name->value;
-                    if (! empty($visitedFragmentNames[$fragName]) || ! $this->shouldIncludeNode($selection)) {
+                    if (!empty($visitedFragmentNames[$fragName]) || !$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $visitedFragmentNames[$fragName] = true;
                     /** @var FragmentDefinitionNode|null $fragment */
                     $fragment = $exeContext->fragments[$fragName] ?? null;
-                    if (! $fragment || ! $this->doesFragmentConditionMatch($fragment, $runtimeType)) {
+                    if (!$fragment || !$this->doesFragmentConditionMatch($fragment, $runtimeType)) {
                         break;
                     }
                     $this->collectFields(
@@ -462,7 +461,7 @@ 
     {
         $result = $this->promiseReduce(
             array_keys($fields->getArrayCopy()),
-            function ($results, $responseName) use ($path, $parentType, $sourceValue, $fields) {
+            function($results, $responseName) use ($path, $parentType, $sourceValue, $fields) {
                 $fieldNodes  = $fields[$responseName];
                 $fieldPath   = $path;
                 $fieldPath[] = $responseName;
@@ -472,7 +471,7 @@ 
                 }
                 $promise = $this->getPromise($result);
                 if ($promise) {
-                    return $promise->then(static function ($resolvedResult) use ($responseName, $results) {
+                    return $promise->then(static function($resolvedResult) use ($responseName, $results) {
                         $results[$responseName] = $resolvedResult;
                         return $results;
                     });
@@ -483,7 +482,7 @@ 
             []
         );
         if ($this->isPromise($result)) {
-            return $result->then(static function ($resolvedResults) {
+            return $result->then(static function($resolvedResults) {
                 return self::fixResultsIfEmptyArray($resolvedResults);
             });
         }
@@ -508,7 +507,7 @@ 
         $fieldNode  = $fieldNodes[0];
         $fieldName  = $fieldNode->name->value;
         $fieldDef   = $this->getFieldDef($exeContext->schema, $parentType, $fieldName);
-        if (! $fieldDef) {
+        if (!$fieldDef) {
             return self::$UNDEFINED;
         }
         $returnType = $fieldDef->getType();
@@ -657,11 +656,11 @@ 
                 $path,
                 $result
             );
-            $promise   = $this->getPromise($completed);
+            $promise = $this->getPromise($completed);
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($exeContext) {
+                    function($error) use ($exeContext) {
                         $exeContext->addError($error);
                         return $this->exeContext->promises->createFulfilled(null);
                     }
@@ -703,11 +702,11 @@ 
                 $path,
                 $result
             );
-            $promise   = $this->getPromise($completed);
+            $promise = $this->getPromise($completed);
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($fieldNodes, $path) {
+                    function($error) use ($fieldNodes, $path) {
                         return $this->exeContext->promises->createRejected(Error::createLocatedError(
                             $error,
                             $fieldNodes,
@@ -764,7 +763,7 @@ 
         $promise = $this->getPromise($result);
         // If result is a Promise, apply-lift over completeValue.
         if ($promise) {
-            return $promise->then(function (&$resolved) use ($returnType, $fieldNodes, $info, $path) {
+            return $promise->then(function(&$resolved) use ($returnType, $fieldNodes, $info, $path) {
                 return $this->completeValue($returnType, $fieldNodes, $info, $path, $resolved);
             });
         }
@@ -856,7 +855,7 @@ 
         }
         if ($this->exeContext->promises->isThenable($value)) {
             $promise = $this->exeContext->promises->convertThenable($value);
-            if (! $promise instanceof Promise) {
+            if (!$promise instanceof Promise) {
                 throw new InvariantViolation(sprintf(
                     '%s::convertThenable is expected to return instance of GraphQL\Executor\Promise\Promise, got: %s',
                     get_class($this->exeContext->promises),
@@ -884,10 +883,10 @@ 
     {
         return array_reduce(
             $values,
-            function ($previous, $value) use ($callback) {
+            function($previous, $value) use ($callback) {
                 $promise = $this->getPromise($previous);
                 if ($promise) {
-                    return $promise->then(static function ($resolved) use ($callback, $value) {
+                    return $promise->then(static function($resolved) use ($callback, $value) {
                         return $callback($resolved, $value);
                     });
                 }
@@ -923,7 +922,7 @@ 
             $fieldPath     = $path;
             $fieldPath[]   = $i++;
             $completedItem = $this->completeValueCatchingError($itemType, $fieldNodes, $info, $fieldPath, $item);
-            if (! $containsPromise && $this->getPromise($completedItem)) {
+            if (!$containsPromise && $this->getPromise($completedItem)) {
                 $containsPromise = true;
             }
             $completedItems[] = $completedItem;
@@ -980,7 +979,7 @@ 
         }
         $promise = $this->getPromise($runtimeType);
         if ($promise) {
-            return $promise->then(function ($resolvedRuntimeType) use (
+            return $promise->then(function($resolvedRuntimeType) use (
                 $returnType,
                 $fieldNodes,
                 $info,
@@ -1068,9 +1067,9 @@ 
                 return $type;
             }
         }
-        if (! empty($promisedIsTypeOfResults)) {
+        if (!empty($promisedIsTypeOfResults)) {
             return $this->exeContext->promises->all($promisedIsTypeOfResults)
-                ->then(static function ($isTypeOfResults) use ($possibleTypes) {
+                ->then(static function($isTypeOfResults) use ($possibleTypes) {
                     foreach ($isTypeOfResults as $index => $result) {
                         if ($result) {
                             return $possibleTypes[$index];
@@ -1102,13 +1101,13 @@ 
         if ($isTypeOf !== null) {
             $promise = $this->getPromise($isTypeOf);
             if ($promise) {
-                return $promise->then(function ($isTypeOfResult) use (
+                return $promise->then(function($isTypeOfResult) use (
                     $returnType,
                     $fieldNodes,
                     $path,
                     &$result
                 ) {
-                    if (! $isTypeOfResult) {
+                    if (!$isTypeOfResult) {
                         throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
                     }
                     return $this->collectAndExecuteSubfields(
@@ -1119,7 +1118,7 @@ 
                     );
                 });
             }
-            if (! $isTypeOf) {
+            if (!$isTypeOf) {
                 throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
             }
         }
@@ -1169,15 +1168,15 @@ 
 
     private function collectSubFields(ObjectType $returnType, $fieldNodes) : ArrayObject
     {
-        if (! isset($this->subFieldCache[$returnType])) {
+        if (!isset($this->subFieldCache[$returnType])) {
             $this->subFieldCache[$returnType] = new SplObjectStorage();
         }
-        if (! isset($this->subFieldCache[$returnType][$fieldNodes])) {
+        if (!isset($this->subFieldCache[$returnType][$fieldNodes])) {
             // Collect sub-fields to execute to complete this value.
             $subFieldNodes        = new ArrayObject();
             $visitedFragmentNames = new ArrayObject();
             foreach ($fieldNodes as $fieldNode) {
-                if (! isset($fieldNode->selectionSet)) {
+                if (!isset($fieldNode->selectionSet)) {
                     continue;
                 }
                 $subFieldNodes = $this->collectFields(
@@ -1213,13 +1212,13 @@ 
             if ($result === self::$UNDEFINED) {
                 continue;
             }
-            if (! $containsPromise && $this->getPromise($result)) {
+            if (!$containsPromise && $this->getPromise($result)) {
                 $containsPromise = true;
             }
             $finalResults[$responseName] = $result;
         }
         // If there are no promises, we can just return the object
-        if (! $containsPromise) {
+        if (!$containsPromise) {
             return self::fixResultsIfEmptyArray($finalResults);
         }
         // Otherwise, results is a map from field name to the result
@@ -1260,7 +1259,7 @@ 
         $keys              = array_keys($assoc);
         $valuesAndPromises = array_values($assoc);
         $promise           = $this->exeContext->promises->all($valuesAndPromises);
-        return $promise->then(static function ($values) use ($keys) {
+        return $promise->then(static function($values) use ($keys) {
             $resolvedResults = [];
             foreach ($values as $i => $value) {
                 $resolvedResults[$keys[$i]] = $value;
@@ -1283,9 +1282,8 @@ 
         &$result
     ) {
         $runtimeType = is_string($runtimeTypeOrName) ?
-            $this->exeContext->schema->getType($runtimeTypeOrName) :
-            $runtimeTypeOrName;
-        if (! $runtimeType instanceof ObjectType) {
+            $this->exeContext->schema->getType($runtimeTypeOrName) : $runtimeTypeOrName;
+        if (!$runtimeType instanceof ObjectType) {
             throw new InvariantViolation(
                 sprintf(
                     'Abstract type %s must resolve to an Object type at ' .
@@ -1301,7 +1299,7 @@ 
                 )
             );
         }
-        if (! $this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
+        if (!$this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
             throw new InvariantViolation(
                 sprintf('Runtime Object type "%s" is not a possible type for "%s".', $runtimeType, $returnType)
             );

src/Executor/ExecutionResult.php

Spacing +3 added lines, -3 removed lines

@@ -138,8 +138,8 @@ 
     {
         $result = [];
 
-        if (! empty($this->errors)) {
-            $errorsHandler = $this->errorsHandler ?: static function (array $errors, callable $formatter) {
+        if (!empty($this->errors)) {
+            $errorsHandler = $this->errorsHandler ?: static function(array $errors, callable $formatter) {
                 return array_map($formatter, $errors);
             };
 
@@ -153,7 +153,7 @@ 
             $result['data'] = $this->data;
         }
 
-        if (! empty($this->extensions)) {
+        if (!empty($this->extensions)) {
             $result['extensions'] = (array) $this->extensions;
         }
 

src/Executor/Promise/Promise.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      */
     public function __construct($adoptedPromise, PromiseAdapter $adapter)
     {
-        Utils::invariant(! $adoptedPromise instanceof self, 'Expecting promise from adapted system, got ' . self::class);
+        Utils::invariant(!$adoptedPromise instanceof self, 'Expecting promise from adapted system, got ' . self::class);
 
         $this->adapter        = $adapter;
         $this->adoptedPromise = $adoptedPromise;

src/Executor/Promise/Adapter/SyncPromise.php

Spacing +7 added lines, -7 removed lines

@@ -43,7 +43,7 @@ 
     public static function runQueue()
     {
         $q = self::$queue;
-        while ($q && ! $q->isEmpty()) {
+        while ($q && !$q->isEmpty()) {
             $task = $q->dequeue();
             $task();
         }
@@ -58,10 +58,10 @@ 
                 }
                 if (is_object($value) && method_exists($value, 'then')) {
                     $value->then(
-                        function ($resolvedValue) {
+                        function($resolvedValue) {
                             $this->resolve($resolvedValue);
                         },
-                        function ($reason) {
+                        function($reason) {
                             $this->reject($reason);
                         }
                     );
@@ -87,7 +87,7 @@ 
 
     public function reject($reason)
     {
-        if (! $reason instanceof Exception && ! $reason instanceof Throwable) {
+        if (!$reason instanceof Exception && !$reason instanceof Throwable) {
             throw new Exception('SyncPromise::reject() has to be called with an instance of \Throwable');
         }
 
@@ -117,7 +117,7 @@ 
         );
 
         foreach ($this->waiting as $descriptor) {
-            self::getQueue()->enqueue(function () use ($descriptor) {
+            self::getQueue()->enqueue(function() use ($descriptor) {
                 /** @var $promise self */
                 [$promise, $onFulfilled, $onRejected] = $descriptor;
 
@@ -154,10 +154,10 @@ 
 
     public function then(?callable $onFulfilled = null, ?callable $onRejected = null)
     {
-        if ($this->state === self::REJECTED && ! $onRejected) {
+        if ($this->state === self::REJECTED && !$onRejected) {
             return $this;
         }
-        if ($this->state === self::FULFILLED && ! $onFulfilled) {
+        if ($this->state === self::FULFILLED && !$onFulfilled) {
             return $this;
         }
         $tmp             = new self();

src/Executor/Promise/Adapter/SyncPromiseAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function convertThenable($thenable)
     {
-        if (! $thenable instanceof Deferred) {
+        if (!$thenable instanceof Deferred) {
             throw new InvariantViolation('Expected instance of GraphQL\Deferred, got ' . Utils::printSafe($thenable));
         }
 
@@ -113,7 +113,7 @@ 
             if ($promiseOrValue instanceof Promise) {
                 $result[$index] = null;
                 $promiseOrValue->then(
-                    static function ($value) use ($index, &$count, $total, &$result, $all) {
+                    static function($value) use ($index, &$count, $total, &$result, $all) {
                         $result[$index] = $value;
                         $count++;
                         if ($count < $total) {
@@ -148,7 +148,7 @@ 
         $promiseQueue = SyncPromise::getQueue();
 
         while ($promise->adoptedPromise->state === SyncPromise::PENDING &&
-            ! ($dfdQueue->isEmpty() && $promiseQueue->isEmpty())
+            !($dfdQueue->isEmpty() && $promiseQueue->isEmpty())
         ) {
             Deferred::runQueue();
             SyncPromise::runQueue();

src/Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -80,12 +80,12 @@
         // TODO: rework with generators when PHP minimum required version is changed to 5.5+
         $promisesOrValues = Utils::map(
             $promisesOrValues,
-            static function ($item) {
+            static function($item) {
                 return $item instanceof Promise ? $item->adoptedPromise : $item;
             }
         );
 
-        $promise = all($promisesOrValues)->then(static function ($values) use ($promisesOrValues) {
+        $promise = all($promisesOrValues)->then(static function($values) use ($promisesOrValues) {
             $orderedResults = [];
 
             foreach ($promisesOrValues as $key => $value) {