simPod / graphql-php

src/Executor/Promise/Adapter/SyncPromise.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
     public static function runQueue() : void
     {
         $q = self::$queue;
-        while ($q !== null && ! $q->isEmpty()) {
+        while ($q !== null && !$q->isEmpty()) {
             $task = $q->dequeue();
             $task();
         }
@@ -56,10 +56,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);
                         }
                     );
@@ -85,7 +85,7 @@ 
 
     public function reject($reason) : self
     {
-        if (! $reason instanceof Throwable) {
+        if (!$reason instanceof Throwable) {
             throw new Exception('SyncPromise::reject() has to be called with an instance of \Throwable');
         }
 
@@ -115,7 +115,7 @@ 
         );
 
         foreach ($this->waiting as $descriptor) {
-            self::getQueue()->enqueue(function () use ($descriptor) {
+            self::getQueue()->enqueue(function() use ($descriptor) {
                 /** @var self $promise */
                 [$promise, $onFulfilled, $onRejected] = $descriptor;
 

src/Executor/Values.php

Indentation +8 added lines, -8 removed lines

@@ -265,23 +265,23 @@
                 );
             } elseif ($hasValue) {
                 if ($argumentValueNode instanceof NullValueNode) {
-                  // If the explicit value `null` was provided, an entry in the coerced
-                  // values must exist as the value `null`.
+                    // If the explicit value `null` was provided, an entry in the coerced
+                    // values must exist as the value `null`.
                     $coercedValues[$name] = null;
                 } elseif ($argumentValueNode instanceof VariableNode) {
                     $variableName = $argumentValueNode->name->value;
                     Utils::invariant($variableValues !== null, 'Must exist for hasValue to be true.');
-                  // Note: This does no further checking that this variable is correct.
-                  // This assumes that this query has been validated and the variable
-                  // usage here is of the correct type.
+                    // Note: This does no further checking that this variable is correct.
+                    // This assumes that this query has been validated and the variable
+                    // usage here is of the correct type.
                     $coercedValues[$name] = $variableValues[$variableName] ?? null;
                 } else {
                     $valueNode    = $argumentValueNode;
                     $coercedValue = AST::valueFromAST($valueNode, $argType, $variableValues);
                     if (Utils::isInvalid($coercedValue)) {
-                      // Note: ValuesOfCorrectType validation should catch this before
-                      // execution. This is a runtime check to ensure execution does not
-                      // continue with an invalid argument value.
+                        // Note: ValuesOfCorrectType validation should catch this before
+                        // execution. This is a runtime check to ensure execution does not
+                        // continue with an invalid argument value.
                         throw new Error(
                             'Argument "' . $name . '" has invalid value ' . Printer::doPrint($valueNode) . '.',
                             [$argumentValueNode]

Spacing +8 added lines, -8 removed lines

@@ -67,7 +67,7 @@ 
             /** @var InputType|Type $varType */
             $varType = TypeInfo::typeFromAST($schema, $varDefNode->type);
 
-            if (! Type::isInputType($varType)) {
+            if (!Type::isInputType($varType)) {
                 // Must use input types for variables. This should be caught during
                 // validation, however is checked again here for safety.
                 $errors[] = new Error(
@@ -82,11 +82,11 @@ 
                 $hasValue = array_key_exists($varName, $inputs);
                 $value    = $hasValue ? $inputs[$varName] : Utils::undefined();
 
-                if (! $hasValue && $varDefNode->defaultValue) {
+                if (!$hasValue && $varDefNode->defaultValue) {
                     // If no value was provided to a variable with a default value,
                     // use the default value.
                     $coercedValues[$varName] = AST::valueFromAST($varDefNode->defaultValue, $varType);
-                } elseif ((! $hasValue || $value === null) && ($varType instanceof NonNull)) {
+                } elseif ((!$hasValue || $value === null) && ($varType instanceof NonNull)) {
                     // If no value or a nullish value was provided to a variable with a
                     // non-null type (required), produce an error.
                     $errors[] = new Error(
@@ -136,7 +136,7 @@ 
             }
         }
 
-        if (! empty($errors)) {
+        if (!empty($errors)) {
             return [$errors, null];
         }
 
@@ -160,7 +160,7 @@ 
         if (isset($node->directives) && $node->directives instanceof NodeList) {
             $directiveNode = Utils::find(
                 $node->directives,
-                static function (DirectiveNode $directive) use ($directiveDef) {
+                static function(DirectiveNode $directive) use ($directiveDef) {
                     return $directive->name->value === $directiveDef->name;
                 }
             );
@@ -233,11 +233,11 @@ 
                 $isNull   = $argumentValueNode instanceof NullValueNode;
             }
 
-            if (! $hasValue && $argumentDefinition->defaultValueExists()) {
+            if (!$hasValue && $argumentDefinition->defaultValueExists()) {
                 // If no argument was provided where the definition has a default value,
                 // use the default value.
                 $coercedValues[$name] = $argumentDefinition->defaultValue;
-            } elseif ((! $hasValue || $isNull) && ($argType instanceof NonNull)) {
+            } elseif ((!$hasValue || $isNull) && ($argType instanceof NonNull)) {
                 // If no argument or a null value was provided to an argument with a
                 // non-null type (required), produce a field error.
                 if ($isNull) {
@@ -327,7 +327,7 @@ 
 
         return $errors
             ? array_map(
-                static function (Throwable $error) {
+                static function(Throwable $error) {
                     return $error->getMessage();
                 },
                 $errors

src/Executor/ReferenceExecutor.php

Spacing +35 added lines, -35 removed lines

@@ -61,7 +61,7 @@ 
 
     private function __construct(ExecutionContext $context)
     {
-        if (! self::$UNDEFINED) {
+        if (!self::$UNDEFINED) {
             self::$UNDEFINED = Utils::undefined();
         }
         $this->exeContext    = $context;
@@ -176,7 +176,7 @@ 
                 $errors = array_merge($errors, $coercionErrors);
             }
         }
-        if (! empty($errors)) {
+        if (!empty($errors)) {
             return $errors;
         }
         Utils::invariant($operation, 'Has operation if no errors.');
@@ -222,7 +222,7 @@ 
     private function buildResponse($data)
     {
         if ($this->isPromise($data)) {
-            return $data->then(function ($resolved) {
+            return $data->then(function($resolved) {
                 return $this->buildResponse($resolved);
             });
         }
@@ -257,7 +257,7 @@ 
             if ($this->isPromise($result)) {
                 return $result->then(
                     null,
-                    function ($error) {
+                    function($error) {
                         if ($error instanceof Error) {
                             $this->exeContext->addError($error);
 
@@ -346,18 +346,18 @@ 
         foreach ($selectionSet->selections as $selection) {
             switch (true) {
                 case $selection instanceof FieldNode:
-                    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 $selection instanceof InlineFragmentNode:
-                    if (! $this->shouldIncludeNode($selection) ||
-                        ! $this->doesFragmentConditionMatch($selection, $runtimeType)
+                    if (!$this->shouldIncludeNode($selection) ||
+                        !$this->doesFragmentConditionMatch($selection, $runtimeType)
                     ) {
                         break;
                     }
@@ -370,13 +370,13 @@ 
                     break;
                 case $selection instanceof FragmentSpreadNode:
                     $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 === null || ! $this->doesFragmentConditionMatch($fragment, $runtimeType)) {
+                    if ($fragment === null || !$this->doesFragmentConditionMatch($fragment, $runtimeType)) {
                         break;
                     }
                     $this->collectFields(
@@ -417,7 +417,7 @@ 
             $variableValues
         );
 
-        return ! isset($include['if']) || $include['if'] !== false;
+        return !isset($include['if']) || $include['if'] !== false;
     }
 
     /**
@@ -468,7 +468,7 @@ 
     {
         $result = $this->promiseReduce(
             array_keys($fields->getArrayCopy()),
-            function ($results, $responseName) use ($path, $parentType, $rootValue, $fields) {
+            function($results, $responseName) use ($path, $parentType, $rootValue, $fields) {
                 $fieldNodes  = $fields[$responseName];
                 $fieldPath   = $path;
                 $fieldPath[] = $responseName;
@@ -478,7 +478,7 @@ 
                 }
                 $promise = $this->getPromise($result);
                 if ($promise !== null) {
-                    return $promise->then(static function ($resolvedResult) use ($responseName, $results) {
+                    return $promise->then(static function($resolvedResult) use ($responseName, $results) {
                         $results[$responseName] = $resolvedResult;
 
                         return $results;
@@ -492,7 +492,7 @@ 
         );
 
         if ($this->isPromise($result)) {
-            return $result->then(static function ($resolvedResults) {
+            return $result->then(static function($resolvedResults) {
                 return self::fixResultsIfEmptyArray($resolvedResults);
             });
         }
@@ -616,7 +616,7 @@ 
         try {
             // Build a map of arguments from the field.arguments AST, using the
             // variables scope to fulfill any variable references.
-            $args         = Values::getArgumentValues(
+            $args = Values::getArgumentValues(
                 $fieldDef,
                 $fieldNode,
                 $this->exeContext->variableValues
@@ -651,7 +651,7 @@ 
         try {
             $promise = $this->getPromise($result);
             if ($promise !== null) {
-                $completed = $promise->then(function (&$resolved) use ($returnType, $fieldNodes, $info, $path) {
+                $completed = $promise->then(function(&$resolved) use ($returnType, $fieldNodes, $info, $path) {
                     return $this->completeValue($returnType, $fieldNodes, $info, $path, $resolved);
                 });
             } else {
@@ -660,7 +660,7 @@ 
 
             $promise = $this->getPromise($completed);
             if ($promise !== null) {
-                return $promise->then(null, function ($error) use ($fieldNodes, $path, $returnType) {
+                return $promise->then(null, function($error) use ($fieldNodes, $path, $returnType) {
                     return $this->handleFieldError($error, $fieldNodes, $path, $returnType);
                 });
             }
@@ -819,7 +819,7 @@ 
         }
         if ($this->exeContext->promiseAdapter->isThenable($value)) {
             $promise = $this->exeContext->promiseAdapter->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->promiseAdapter),
@@ -849,10 +849,10 @@ 
     {
         return array_reduce(
             $values,
-            function ($previous, $value) use ($callback) {
+            function($previous, $value) use ($callback) {
                 $promise = $this->getPromise($previous);
                 if ($promise !== null) {
-                    return $promise->then(static function ($resolved) use ($callback, $value) {
+                    return $promise->then(static function($resolved) use ($callback, $value) {
                         return $callback($resolved, $value);
                     });
                 }
@@ -889,7 +889,7 @@ 
             $fieldPath[]   = $i++;
             $info->path    = $fieldPath;
             $completedItem = $this->completeValueCatchingError($itemType, $fieldNodes, $info, $fieldPath, $item);
-            if (! $containsPromise && $this->getPromise($completedItem) !== null) {
+            if (!$containsPromise && $this->getPromise($completedItem) !== null) {
                 $containsPromise = true;
             }
             $completedItems[] = $completedItem;
@@ -943,7 +943,7 @@ 
         }
         $promise = $this->getPromise($runtimeType);
         if ($promise !== null) {
-            return $promise->then(function ($resolvedRuntimeType) use (
+            return $promise->then(function($resolvedRuntimeType) use (
                 $returnType,
                 $fieldNodes,
                 $info,
@@ -1034,9 +1034,9 @@ 
                 return $type;
             }
         }
-        if (! empty($promisedIsTypeOfResults)) {
+        if (!empty($promisedIsTypeOfResults)) {
             return $this->exeContext->promiseAdapter->all($promisedIsTypeOfResults)
-                ->then(static function ($isTypeOfResults) use ($possibleTypes) {
+                ->then(static function($isTypeOfResults) use ($possibleTypes) {
                     foreach ($isTypeOfResults as $index => $result) {
                         if ($result) {
                             return $possibleTypes[$index];
@@ -1070,13 +1070,13 @@ 
         if ($isTypeOf !== null) {
             $promise = $this->getPromise($isTypeOf);
             if ($promise !== null) {
-                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);
                     }
 
@@ -1088,7 +1088,7 @@ 
                     );
                 });
             }
-            if (! $isTypeOf) {
+            if (!$isTypeOf) {
                 throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
             }
         }
@@ -1147,15 +1147,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(
@@ -1192,13 +1192,13 @@ 
             if ($result === self::$UNDEFINED) {
                 continue;
             }
-            if (! $containsPromise && $this->isPromise($result)) {
+            if (!$containsPromise && $this->isPromise($result)) {
                 $containsPromise = true;
             }
             $results[$responseName] = $result;
         }
         // If there are no promises, we can just return the object
-        if (! $containsPromise) {
+        if (!$containsPromise) {
             return self::fixResultsIfEmptyArray($results);
         }
 
@@ -1241,7 +1241,7 @@ 
         $valuesAndPromises = array_values($assoc);
         $promise           = $this->exeContext->promiseAdapter->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;
@@ -1267,7 +1267,7 @@ 
         $runtimeType = is_string($runtimeTypeOrName)
             ? $this->exeContext->schema->getType($runtimeTypeOrName)
             : $runtimeTypeOrName;
-        if (! $runtimeType instanceof ObjectType) {
+        if (!$runtimeType instanceof ObjectType) {
             throw new InvariantViolation(
                 sprintf(
                     'Abstract type %s must resolve to an Object type at ' .
@@ -1283,7 +1283,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/Type/Schema.php

Spacing +12 added lines, -12 removed lines

@@ -109,11 +109,11 @@ 
                 Utils::getVariableType($config)
             );
             Utils::invariant(
-                ! $config->types || is_array($config->types) || is_callable($config->types),
+                !$config->types || is_array($config->types) || is_callable($config->types),
                 '"types" must be array or callable if provided but got: ' . Utils::getVariableType($config->types)
             );
             Utils::invariant(
-                ! $config->directives || is_array($config->directives),
+                !$config->directives || is_array($config->directives),
                 '"directives" must be Array if provided but got: ' . Utils::getVariableType($config->directives)
             );
         }
@@ -165,7 +165,7 @@ 
             $types = $types();
         }
 
-        if (! is_array($types) && ! $types instanceof Traversable) {
+        if (!is_array($types) && !$types instanceof Traversable) {
             throw new InvariantViolation(sprintf(
                 'Schema types callable must return array or instance of Traversable but got: %s',
                 Utils::getVariableType($types)
@@ -173,7 +173,7 @@ 
         }
 
         foreach ($types as $index => $type) {
-            if (! $type instanceof Type) {
+            if (!$type instanceof Type) {
                 throw new InvariantViolation(sprintf(
                     'Each entry of schema types must be instance of GraphQL\Type\Definition\Type but entry at %s is %s',
                     $index,
@@ -196,7 +196,7 @@ 
      */
     public function getTypeMap()
     {
-        if (! $this->fullyLoaded) {
+        if (!$this->fullyLoaded) {
             $this->resolvedTypes = $this->collectAllTypes();
             $this->fullyLoaded   = true;
         }
@@ -214,7 +214,7 @@ 
             $typeMap = TypeInfo::extractTypes($type, $typeMap);
         }
         foreach ($this->getDirectives() as $directive) {
-            if (! ($directive instanceof Directive)) {
+            if (!($directive instanceof Directive)) {
                 continue;
             }
 
@@ -314,9 +314,9 @@ 
      */
     public function getType(string $name) : ?Type
     {
-        if (! isset($this->resolvedTypes[$name])) {
+        if (!isset($this->resolvedTypes[$name])) {
             $type = $this->loadType($name);
-            if (! $type) {
+            if (!$type) {
                 return null;
             }
             $this->resolvedTypes[$name] = $type;
@@ -334,13 +334,13 @@ 
     {
         $typeLoader = $this->config->typeLoader;
 
-        if (! $typeLoader) {
+        if (!$typeLoader) {
             return $this->defaultTypeLoader($typeName);
         }
 
         $type = $typeLoader($typeName);
 
-        if (! $type instanceof Type) {
+        if (!$type instanceof Type) {
             throw new InvariantViolation(
                 sprintf(
                     'Type loader is expected to return valid type "%s", but it returned %s',
@@ -395,7 +395,7 @@ 
             foreach ($this->getTypeMap() as $type) {
                 if ($type instanceof ObjectType) {
                     foreach ($type->getInterfaces() as $interface) {
-                        if (! ($interface instanceof InterfaceType)) {
+                        if (!($interface instanceof InterfaceType)) {
                             continue;
                         }
 
@@ -481,7 +481,7 @@ 
             $type->assertValid();
 
             // Make sure type loader returns the same instance as registered in other places of schema
-            if (! $this->config->typeLoader) {
+            if (!$this->config->typeLoader) {
                 continue;
             }
 

src/Type/Definition/IntType.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
     {
         $isInt = is_int($value) || (is_float($value) && floor($value) === $value);
 
-        if (! $isInt) {
+        if (!$isInt) {
             throw new Error(
                 'Int cannot represent non-integer value: ' .
                 Utils::printSafe($value)

src/Type/Definition/UnionType.php

Spacing +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -53,7 +53,7 @@ 
 
     public function isPossibleType(Type $type) : bool
     {
-        if (! $type instanceof ObjectType) {
+        if (!$type instanceof ObjectType) {
             return false;
         }
 
@@ -73,7 +73,7 @@ 
     public function getTypes()
     {
         if ($this->types === null) {
-            if (! isset($this->config['types'])) {
+            if (!isset($this->config['types'])) {
                 $types = null;
             } elseif (is_callable($this->config['types'])) {
                 $types = call_user_func($this->config['types']);
@@ -81,7 +81,7 @@ 
                 $types = $this->config['types'];
             }
 
-            if (! is_array($types)) {
+            if (!is_array($types)) {
                 throw new InvariantViolation(
                     sprintf(
                         'Must provide Array of types or a callable which returns such an array for Union %s',
@@ -122,7 +122,7 @@ 
     {
         parent::assertValid();
 
-        if (! isset($this->config['resolveType'])) {
+        if (!isset($this->config['resolveType'])) {
             return;
         }
 

src/Type/Definition/FloatType.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
     {
         $float = is_numeric($value) || is_bool($value) ? floatval($value) : null;
 
-        if ($float === null || ! is_finite($float)) {
+        if ($float === null || !is_finite($float)) {
             throw new Error(
                 'Float cannot represent non numeric value: ' .
                 Utils::printSafe($value)
@@ -56,7 +56,7 @@ 
     {
         $float = is_float($value) || is_int($value) ? floatval($value) : null;
 
-        if ($float === null || ! is_finite($float)) {
+        if ($float === null || !is_finite($float)) {
             throw new Error(
                 'Float cannot represent non numeric value: ' .
                 Utils::printSafe($value)

src/Type/Definition/StringType.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
             || (is_object($value) && method_exists($value, '__toString'))
             || $value === null;
 
-        if (! $canCast) {
+        if (!$canCast) {
             throw new Error(
                 'String cannot represent value: ' . Utils::printSafe($value)
             );
@@ -56,7 +56,7 @@ 
      */
     public function parseValue($value)
     {
-        if (! is_string($value)) {
+        if (!is_string($value)) {
             throw new Error(
                 'String cannot represent a non string value: ' . Utils::printSafe($value)
             );

src/Type/Definition/ObjectType.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -189,7 +189,7 @@ 
                 ? call_user_func($interfaces)
                 : $interfaces;
 
-            if ($interfaces !== null && ! is_array($interfaces)) {
+            if ($interfaces !== null && !is_array($interfaces)) {
                 throw new InvariantViolation(
                     sprintf('%s interfaces must be an Array or a callable which returns an Array.', $this->name)
                 );