webonyx / graphql-php

src/Language/Parser.php

Spacing +18 added lines, -18 removed lines

@@ -369,7 +369,7 @@ 
         $this->expect($openKind);
 
         $nodes = [];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -389,7 +389,7 @@ 
         $this->expect($openKind);
 
         $nodes = [$parseFn($this)];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -423,7 +423,7 @@ 
         return new DocumentNode([
             'definitions' => $this->many(
                 Token::SOF,
-                function () {
+                function() {
                     return $this->parseDefinition();
                 },
                 Token::EOF
@@ -550,7 +550,7 @@ 
         return $this->peek(Token::PAREN_L)
             ? $this->many(
                 Token::PAREN_L,
-                function () {
+                function() {
                     return $this->parseVariableDefinition();
                 },
                 Token::PAREN_R
@@ -601,7 +601,7 @@ 
             [
                 'selections' => $this->many(
                     Token::BRACE_L,
-                    function () {
+                    function() {
                         return $this->parseSelection();
                     },
                     Token::BRACE_R
@@ -656,10 +656,10 @@ 
     private function parseArguments(bool $isConst) : NodeList
     {
         $parseFn = $isConst
-            ? function () {
+            ? function() {
                 return $this->parseConstArgument();
             }
-            : function () {
+            : function() {
                 return $this->parseArgument();
             };
 
@@ -856,7 +856,7 @@ 
                 break;
 
             case Token::DOLLAR:
-                if (! $isConst) {
+                if (!$isConst) {
                     return $this->parseVariable();
                 }
                 break;
@@ -897,9 +897,9 @@ 
     private function parseArray(bool $isConst) : ListValueNode
     {
         $start   = $this->lexer->token;
-        $parseFn = $isConst ? function () {
+        $parseFn = $isConst ? function() {
             return $this->parseConstValue();
-        } : function () {
+        } : function() {
             return $this->parseVariableValue();
         };
 
@@ -916,7 +916,7 @@ 
         $start = $this->lexer->token;
         $this->expect(Token::BRACE_L);
         $fields = [];
-        while (! $this->skip(Token::BRACE_R)) {
+        while (!$this->skip(Token::BRACE_R)) {
             $fields[] = $this->parseObjectField($isConst);
         }
 
@@ -1090,7 +1090,7 @@ 
 
         $operationTypes = $this->many(
             Token::BRACE_L,
-            function () {
+            function() {
                 return $this->parseOperationTypeDefinition();
             },
             Token::BRACE_R
@@ -1180,7 +1180,7 @@ 
                 $types[] = $this->parseNamedType();
             } while ($this->skip(Token::AMP) ||
                 // Legacy support for the SDL?
-                (! empty($this->lexer->options['allowLegacySDLImplementsInterfaces']) && $this->peek(Token::NAME))
+                (!empty($this->lexer->options['allowLegacySDLImplementsInterfaces']) && $this->peek(Token::NAME))
             );
         }
 
@@ -1193,7 +1193,7 @@ 
     private function parseFieldsDefinition() : NodeList
     {
         // Legacy support for the SDL?
-        if (! empty($this->lexer->options['allowLegacySDLEmptyFields'])
+        if (!empty($this->lexer->options['allowLegacySDLEmptyFields'])
             && $this->peek(Token::BRACE_L)
             && $this->lexer->lookahead()->kind === Token::BRACE_R
         ) {
@@ -1207,7 +1207,7 @@ 
             $nodeList = $this->peek(Token::BRACE_L)
                 ? $this->many(
                     Token::BRACE_L,
-                    function () {
+                    function() {
                         return $this->parseFieldDefinition();
                     },
                     Token::BRACE_R
@@ -1250,7 +1250,7 @@ 
         $nodeList = $this->peek(Token::PAREN_L)
             ? $this->many(
                 Token::PAREN_L,
-                function () {
+                function() {
                     return $this->parseInputValueDefinition();
                 },
                 Token::PAREN_R
@@ -1382,7 +1382,7 @@ 
         $nodeList = $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseEnumValueDefinition();
                 },
                 Token::BRACE_R
@@ -1440,7 +1440,7 @@ 
         $nodeList = $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseInputValueDefinition();
                 },
                 Token::BRACE_R

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) {

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/Promise/Adapter/SyncPromiseAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,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));
         }
 
@@ -110,7 +110,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) {
@@ -145,7 +145,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/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/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'] = $this->extensions;
         }
 

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/GraphQL.php

Spacing +3 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
                 $queryComplexity->setRawVariableValues($variableValues);
             } else {
                 foreach ($validationRules as $rule) {
-                    if (! ($rule instanceof QueryComplexity)) {
+                    if (!($rule instanceof QueryComplexity)) {
                         continue;
                     }
 
@@ -151,7 +151,7 @@ 
 
             $validationErrors = DocumentValidator::validate($schema, $documentNode, $validationRules);
 
-            if (! empty($validationErrors)) {
+            if (!empty($validationErrors)) {
                 return $promiseAdapter->createFulfilled(
                     new ExecutionResult(null, $validationErrors)
                 );
@@ -213,7 +213,7 @@ 
         if ($promiseAdapter instanceof SyncPromiseAdapter) {
             $result = $promiseAdapter->wait($result)->toArray();
         } else {
-            $result = $result->then(static function (ExecutionResult $r) {
+            $result = $result->then(static function(ExecutionResult $r) {
                 return $r->toArray();
             });
         }

src/Experimental/Executor/Collector.php

Spacing +10 added lines, -10 removed lines

@@ -127,7 +127,7 @@ 
             $fieldName = $fieldNode->name->value;
 
             $argumentValueMap = null;
-            if (! empty($fieldNode->arguments)) {
+            if (!empty($fieldNode->arguments)) {
                 foreach ($fieldNode->arguments as $argumentNode) {
                     $argumentValueMap                             = $argumentValueMap ?? [];
                     $argumentValueMap[$argumentNode->name->value] = $argumentNode->value;
@@ -135,8 +135,8 @@ 
             }
 
             if ($fieldName !== Introspection::TYPE_NAME_FIELD_NAME &&
-                ! ($runtimeType === $this->schema->getQueryType() && ($fieldName === Introspection::SCHEMA_FIELD_NAME || $fieldName === Introspection::TYPE_FIELD_NAME)) &&
-                ! $runtimeType->hasField($fieldName)
+                !($runtimeType === $this->schema->getQueryType() && ($fieldName === Introspection::SCHEMA_FIELD_NAME || $fieldName === Introspection::TYPE_FIELD_NAME)) &&
+                !$runtimeType->hasField($fieldName)
             ) {
                 // do not emit error
                 continue;
@@ -154,7 +154,7 @@ 
 
         foreach ($selectionSet->selections as $selection) {
             /** @var FieldNode|FragmentSpreadNode|InlineFragmentNode $selection */
-            if (! empty($selection->directives)) {
+            if (!empty($selection->directives)) {
                 foreach ($selection->directives as $directiveNode) {
                     if ($directiveNode->name->value === Directive::SKIP_NAME) {
                         /** @var VariableNode|NullValueNode|IntValueNode|FloatValueNode|StringValueNode|BooleanValueNode|EnumValueNode|ListValueNode|ObjectValueNode|null $condition */
@@ -203,7 +203,7 @@ 
             if ($selection instanceof FieldNode) {
                 $resultName = $selection->alias === null ? $selection->name->value : $selection->alias->value;
 
-                if (! isset($this->fields[$resultName])) {
+                if (!isset($this->fields[$resultName])) {
                     $this->fields[$resultName] = [];
                 }
 
@@ -215,7 +215,7 @@ 
                     continue;
                 }
 
-                if (! isset($this->fragments[$fragmentName])) {
+                if (!isset($this->fragments[$fragmentName])) {
                     $this->runtime->addError(new Error(
                         sprintf('Fragment "%s" does not exist.', $fragmentName),
                         $selection
@@ -228,7 +228,7 @@ 
                 $fragmentDefinition = $this->fragments[$fragmentName];
                 $conditionTypeName  = $fragmentDefinition->typeCondition->name->value;
 
-                if (! $this->schema->hasType($conditionTypeName)) {
+                if (!$this->schema->hasType($conditionTypeName)) {
                     $this->runtime->addError(new Error(
                         sprintf('Cannot spread fragment "%s", type "%s" does not exist.', $fragmentName, $conditionTypeName),
                         $selection
@@ -243,7 +243,7 @@ 
                         continue;
                     }
                 } elseif ($conditionType instanceof AbstractType) {
-                    if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                    if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                         continue;
                     }
                 }
@@ -253,7 +253,7 @@ 
                 if ($selection->typeCondition !== null) {
                     $conditionTypeName = $selection->typeCondition->name->value;
 
-                    if (! $this->schema->hasType($conditionTypeName)) {
+                    if (!$this->schema->hasType($conditionTypeName)) {
                         $this->runtime->addError(new Error(
                             sprintf('Cannot spread inline fragment, type "%s" does not exist.', $conditionTypeName),
                             $selection
@@ -268,7 +268,7 @@ 
                             continue;
                         }
                     } elseif ($conditionType instanceof AbstractType) {
-                        if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                        if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                             continue;
                         }
                     }