simPod / graphql-php

src/Experimental/Executor/CoroutineExecutor.php

Spacing +19 added lines, -19 removed lines

@@ -184,7 +184,7 @@ 
         $this->collector = new Collector($this->schema, $this);
         $this->collector->initialize($this->documentNode, $this->operationName);
 
-        if (! empty($this->errors)) {
+        if (!empty($this->errors)) {
             return $this->promiseAdapter->createFulfilled($this->finishExecute(null, $this->errors));
         }
 
@@ -194,7 +194,7 @@ 
             $this->rawVariableValues ?: []
         );
 
-        if (! empty($errors)) {
+        if (!empty($errors)) {
             return $this->promiseAdapter->createFulfilled($this->finishExecute(null, $errors));
         }
 
@@ -215,11 +215,11 @@ 
             );
 
             $fieldDefinition = $this->findFieldDefinition($ctx);
-            if (! $fieldDefinition->getType() instanceof NonNull) {
+            if (!$fieldDefinition->getType() instanceof NonNull) {
                 $ctx->nullFence = [$shared->resultName];
             }
 
-            if ($this->collector->operation->operation === 'mutation' && ! $this->queue->isEmpty()) {
+            if ($this->collector->operation->operation === 'mutation' && !$this->queue->isEmpty()) {
                 $this->schedule->enqueue($ctx);
             } else {
                 $this->queue->enqueue(new Strand($this->spawn($ctx)));
@@ -229,7 +229,7 @@ 
         $this->run();
 
         if ($this->pending > 0) {
-            return $this->promiseAdapter->create(function (callable $resolve) {
+            return $this->promiseAdapter->create(function(callable $resolve) {
                 $this->doResolve = $resolve;
             });
         }
@@ -279,7 +279,7 @@ 
     private function run()
     {
         RUN:
-        while (! $this->queue->isEmpty()) {
+        while (!$this->queue->isEmpty()) {
             /** @var Strand $strand */
             $strand = $this->queue->dequeue();
 
@@ -309,20 +309,20 @@ 
                         // !!! increment pending before calling ->then() as it may invoke the callback right away
                         ++$this->pending;
 
-                        if (! $value instanceof Promise) {
+                        if (!$value instanceof Promise) {
                             $value = $this->promiseAdapter->convertThenable($value);
                         }
 
                         $this->promiseAdapter
                             ->then(
                                 $value,
-                                function ($value) use ($strand) {
+                                function($value) use ($strand) {
                                     $strand->success = true;
                                     $strand->value   = $value;
                                     $this->queue->enqueue($strand);
                                     $this->done();
                                 },
-                                function (Throwable $throwable) use ($strand) {
+                                function(Throwable $throwable) use ($strand) {
                                     $strand->success = false;
                                     $strand->value   = $throwable;
                                     $this->queue->enqueue($strand);
@@ -434,7 +434,7 @@ 
 
             $value = $resolve($ctx->value, $arguments, $this->contextValue, $ctx->resolveInfo);
 
-            if (! $this->completeValueFast($ctx, $returnType, $value, $ctx->path, $returnValue)) {
+            if (!$this->completeValueFast($ctx, $returnType, $value, $ctx->path, $returnValue)) {
                 $returnValue = yield $this->completeValue(
                     $ctx,
                     $returnType,
@@ -456,12 +456,12 @@ 
         if ($returnValue !== self::$undefined) {
             $ctx->result->{$ctx->shared->resultName} = $returnValue;
         } elseif ($ctx->resolveInfo !== null && $ctx->resolveInfo->returnType instanceof NonNull) { // !!! $ctx->resolveInfo might not have been initialized yet
-            $result =& $this->rootResult;
+            $result = & $this->rootResult;
             foreach ($ctx->nullFence ?? [] as $key) {
                 if (is_string($key)) {
-                    $result =& $result->{$key};
+                    $result = & $result->{$key};
                 } else {
-                    $result =& $result[$key];
+                    $result = & $result[$key];
                 }
             }
             $result = null;
@@ -503,7 +503,7 @@ 
             $type    = $type->getWrappedType();
         }
 
-        if (! $type instanceof LeafType) {
+        if (!$type instanceof LeafType) {
             return false;
         }
 
@@ -636,7 +636,7 @@ 
                 $ctx->resolveInfo->path = $itemPath;
 
                 try {
-                    if (! $this->completeValueFast($ctx, $itemType, $itemValue, $itemPath, $itemReturnValue)) {
+                    if (!$this->completeValueFast($ctx, $itemType, $itemValue, $itemPath, $itemReturnValue)) {
                         $itemReturnValue = yield $this->completeValue($ctx, $itemType, $itemValue, $itemPath, $nullFence);
                     }
                 } catch (Throwable $reason) {
@@ -727,7 +727,7 @@ 
 
                         $returnValue = self::$undefined;
                         goto CHECKED_RETURN;
-                    } elseif (! $objectType instanceof ObjectType) {
+                    } elseif (!$objectType instanceof ObjectType) {
                         $this->addError(Error::createLocatedError(
                             new InvariantViolation(sprintf(
                                 'Abstract type %s must resolve to an Object type at ' .
@@ -747,7 +747,7 @@ 
 
                         $returnValue = null;
                         goto CHECKED_RETURN;
-                    } elseif (! $this->schema->isPossibleType($type, $objectType)) {
+                    } elseif (!$this->schema->isPossibleType($type, $objectType)) {
                         $this->addError(Error::createLocatedError(
                             new InvariantViolation(sprintf(
                                 'Runtime Object type "%s" is not a possible type for "%s".',
@@ -799,7 +799,7 @@ 
                 if ($typeCheck !== null) {
                     // !!! $objectType->isTypeOf() might return promise, yield to resolve
                     $typeCheck = yield $typeCheck;
-                    if (! $typeCheck) {
+                    if (!$typeCheck) {
                         $this->addError(Error::createLocatedError(
                             sprintf('Expected value of type "%s" but got: %s.', $type->name, Utils::printSafe($value)),
                             $ctx->shared->fieldNodes,
@@ -950,7 +950,7 @@ 
         $selectedType = null;
         foreach ($possibleTypes as $type) {
             $typeCheck = yield $type->isTypeOf($value, $this->contextValue, $ctx->resolveInfo);
-            if ($selectedType !== null || ! $typeCheck) {
+            if ($selectedType !== null || !$typeCheck) {
                 continue;
             }
 

src/Experimental/Executor/Collector.php

Spacing +10 added lines, -10 removed lines

@@ -118,7 +118,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;
@@ -126,8 +126,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;
@@ -145,7 +145,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 */
@@ -194,7 +194,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] = [];
                 }
 
@@ -206,7 +206,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
@@ -219,7 +219,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
@@ -234,7 +234,7 @@ 
                         continue;
                     }
                 } elseif ($conditionType instanceof AbstractType) {
-                    if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                    if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                         continue;
                     }
                 }
@@ -244,7 +244,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
@@ -259,7 +259,7 @@ 
                             continue;
                         }
                     } elseif ($conditionType instanceof AbstractType) {
-                        if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                        if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                             continue;
                         }
                     }

src/Validator/Rules/ValuesOfCorrectType.php

Spacing +22 added lines, -22 removed lines

@@ -51,13 +51,13 @@ 
 
         return [
             NodeKind::FIELD        => [
-                'enter' => static function (FieldNode $node) use (&$fieldName) {
+                'enter' => static function(FieldNode $node) use (&$fieldName) {
                     $fieldName = $node->name->value;
                 },
             ],
-            NodeKind::NULL         => static function (NullValueNode $node) use ($context, &$fieldName) {
+            NodeKind::NULL         => static function(NullValueNode $node) use ($context, &$fieldName) {
                 $type = $context->getInputType();
-                if (! ($type instanceof NonNull)) {
+                if (!($type instanceof NonNull)) {
                     return;
                 }
 
@@ -68,21 +68,21 @@ 
                     )
                 );
             },
-            NodeKind::LST          => function (ListValueNode $node) use ($context, &$fieldName) {
+            NodeKind::LST          => function(ListValueNode $node) use ($context, &$fieldName) {
                 // Note: TypeInfo will traverse into a list's item type, so look to the
                 // parent input type to check if it is a list.
                 $type = Type::getNullableType($context->getParentInputType());
-                if (! $type instanceof ListOfType) {
+                if (!$type instanceof ListOfType) {
                     $this->isValidScalar($context, $node, $fieldName);
 
                     return Visitor::skipNode();
                 }
             },
-            NodeKind::OBJECT       => function (ObjectValueNode $node) use ($context, &$fieldName) {
+            NodeKind::OBJECT       => function(ObjectValueNode $node) use ($context, &$fieldName) {
                 // Note: TypeInfo will traverse into a list's item type, so look to the
                 // parent input type to check if it is a list.
                 $type = Type::getNamedType($context->getInputType());
-                if (! $type instanceof InputObjectType) {
+                if (!$type instanceof InputObjectType) {
                     $this->isValidScalar($context, $node, $fieldName);
 
                     return Visitor::skipNode();
@@ -93,7 +93,7 @@ 
                 $nodeFields   = iterator_to_array($node->fields);
                 $fieldNodeMap = array_combine(
                     array_map(
-                        static function ($field) {
+                        static function($field) {
                             return $field->name->value;
                         },
                         $nodeFields
@@ -102,7 +102,7 @@ 
                 );
                 foreach ($inputFields as $fieldName => $fieldDef) {
                     $fieldType = $fieldDef->getType();
-                    if (isset($fieldNodeMap[$fieldName]) || ! ($fieldType instanceof NonNull) || ($fieldDef->defaultValueExists())) {
+                    if (isset($fieldNodeMap[$fieldName]) || !($fieldType instanceof NonNull) || ($fieldDef->defaultValueExists())) {
                         continue;
                     }
 
@@ -114,11 +114,11 @@ 
                     );
                 }
             },
-            NodeKind::OBJECT_FIELD => static function (ObjectFieldNode $node) use ($context) {
+            NodeKind::OBJECT_FIELD => static function(ObjectFieldNode $node) use ($context) {
                 $parentType = Type::getNamedType($context->getParentInputType());
                 /** @var ScalarType|EnumType|InputObjectType|ListOfType|NonNull $fieldType */
                 $fieldType = $context->getInputType();
-                if ($fieldType || ! ($parentType instanceof InputObjectType)) {
+                if ($fieldType || !($parentType instanceof InputObjectType)) {
                     return;
                 }
 
@@ -126,7 +126,7 @@ 
                     $node->name->value,
                     array_keys($parentType->getFields())
                 );
-                $didYouMean  = $suggestions
+                $didYouMean = $suggestions
                     ? 'Did you mean ' . Utils::orList($suggestions) . '?'
                     : null;
 
@@ -137,11 +137,11 @@ 
                     )
                 );
             },
-            NodeKind::ENUM         => function (EnumValueNode $node) use ($context, &$fieldName) {
+            NodeKind::ENUM         => function(EnumValueNode $node) use ($context, &$fieldName) {
                 $type = Type::getNamedType($context->getInputType());
-                if (! $type instanceof EnumType) {
+                if (!$type instanceof EnumType) {
                     $this->isValidScalar($context, $node, $fieldName);
-                } elseif (! $type->getValue($node->value)) {
+                } elseif (!$type->getValue($node->value)) {
                     $context->reportError(
                         new Error(
                             self::getBadValueMessage(
@@ -156,16 +156,16 @@ 
                     );
                 }
             },
-            NodeKind::INT          => function (IntValueNode $node) use ($context, &$fieldName) {
+            NodeKind::INT          => function(IntValueNode $node) use ($context, &$fieldName) {
                 $this->isValidScalar($context, $node, $fieldName);
             },
-            NodeKind::FLOAT        => function (FloatValueNode $node) use ($context, &$fieldName) {
+            NodeKind::FLOAT        => function(FloatValueNode $node) use ($context, &$fieldName) {
                 $this->isValidScalar($context, $node, $fieldName);
             },
-            NodeKind::STRING       => function (StringValueNode $node) use ($context, &$fieldName) {
+            NodeKind::STRING       => function(StringValueNode $node) use ($context, &$fieldName) {
                 $this->isValidScalar($context, $node, $fieldName);
             },
-            NodeKind::BOOLEAN      => function (BooleanValueNode $node) use ($context, &$fieldName) {
+            NodeKind::BOOLEAN      => function(BooleanValueNode $node) use ($context, &$fieldName) {
                 $this->isValidScalar($context, $node, $fieldName);
             },
         ];
@@ -186,13 +186,13 @@ 
         /** @var ScalarType|EnumType|InputObjectType|ListOfType|NonNull $locationType */
         $locationType = $context->getInputType();
 
-        if (! $locationType) {
+        if (!$locationType) {
             return;
         }
 
         $type = Type::getNamedType($locationType);
 
-        if (! $type instanceof ScalarType) {
+        if (!$type instanceof ScalarType) {
             $context->reportError(
                 new Error(
                     self::getBadValueMessage(
@@ -239,7 +239,7 @@ 
             $suggestions = Utils::suggestionList(
                 Printer::doPrint($node),
                 array_map(
-                    static function (EnumValueDefinition $value) {
+                    static function(EnumValueDefinition $value) {
                         return $value->name;
                     },
                     $type->getValues()

src/Validator/Rules/QueryComplexity.php

Spacing +9 added lines, -9 removed lines

@@ -60,7 +60,7 @@ 
         return $this->invokeIfNeeded(
             $context,
             [
-                NodeKind::SELECTION_SET        => function (SelectionSetNode $selectionSet) use ($context) {
+                NodeKind::SELECTION_SET        => function(SelectionSetNode $selectionSet) use ($context) {
                     $this->fieldNodeAndDefs = $this->collectFieldASTsAndDefs(
                         $context,
                         $context->getParentType(),
@@ -69,16 +69,16 @@ 
                         $this->fieldNodeAndDefs
                     );
                 },
-                NodeKind::VARIABLE_DEFINITION  => function ($def) {
+                NodeKind::VARIABLE_DEFINITION  => function($def) {
                     $this->variableDefs[] = $def;
 
                     return Visitor::skipNode();
                 },
                 NodeKind::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinitionNode $operationDefinition) use ($context, &$complexity) {
+                    'leave' => function(OperationDefinitionNode $operationDefinition) use ($context, &$complexity) {
                         $errors = $context->getErrors();
 
-                        if (! empty($errors)) {
+                        if (!empty($errors)) {
                             return;
                         }
 
@@ -191,11 +191,11 @@ 
                 $this->variableDefs,
                 $this->getRawVariableValues()
             );
-            if (! empty($errors)) {
+            if (!empty($errors)) {
                 throw new Error(implode(
                     "\n\n",
                     array_map(
-                        static function ($error) {
+                        static function($error) {
                             return $error->getMessage();
                         },
                         $errors
@@ -207,7 +207,7 @@ 
                 /** @var bool $directiveArgsIf */
                 $directiveArgsIf = Values::getArgumentValues($directive, $directiveNode, $variableValues)['if'];
 
-                return ! $directiveArgsIf;
+                return !$directiveArgsIf;
             }
             if ($directiveNode->name->value === Directive::SKIP_NAME) {
                 $directive = Directive::skipDirective();
@@ -249,11 +249,11 @@ 
                 $rawVariableValues
             );
 
-            if (! empty($errors)) {
+            if (!empty($errors)) {
                 throw new Error(implode(
                     "\n\n",
                     array_map(
-                        static function ($error) {
+                        static function($error) {
                             return $error->getMessage();
                         },
                         $errors

src/Validator/Rules/LoneSchemaDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $schemaDefinitionsCount = 0;
 
         return [
-            NodeKind::SCHEMA_DEFINITION => static function (SchemaDefinitionNode $node) use ($alreadyDefined, $context, &$schemaDefinitionsCount) {
+            NodeKind::SCHEMA_DEFINITION => static function(SchemaDefinitionNode $node) use ($alreadyDefined, $context, &$schemaDefinitionsCount) {
                 if ($alreadyDefined !== false) {
                     $context->reportError(new Error(self::canNotDefineSchemaWithinExtensionMessage(), $node));
 

src/Validator/Rules/VariablesInAllowedPosition.php

Spacing +8 added lines, -8 removed lines

@@ -32,10 +32,10 @@ 
     {
         return [
             NodeKind::OPERATION_DEFINITION => [
-                'enter' => function () {
+                'enter' => function() {
                     $this->varDefMap = [];
                 },
-                'leave' => function (OperationDefinitionNode $operation) use ($context) {
+                'leave' => function(OperationDefinitionNode $operation) use ($context) {
                     $usages = $context->getRecursiveVariableUsages($operation);
 
                     foreach ($usages as $usage) {
@@ -57,7 +57,7 @@ 
                         $schema  = $context->getSchema();
                         $varType = TypeInfo::typeFromAST($schema, $varDef->type);
 
-                        if (! $varType || $this->allowedVariableUsage($schema, $varType, $varDef->defaultValue, $type, $defaultValue)) {
+                        if (!$varType || $this->allowedVariableUsage($schema, $varType, $varDef->defaultValue, $type, $defaultValue)) {
                             continue;
                         }
 
@@ -68,7 +68,7 @@ 
                     }
                 },
             ],
-            NodeKind::VARIABLE_DEFINITION  => function (VariableDefinitionNode $varDefNode) {
+            NodeKind::VARIABLE_DEFINITION  => function(VariableDefinitionNode $varDefNode) {
                 $this->varDefMap[$varDefNode->variable->name->value] = $varDefNode;
             },
         ];
@@ -100,10 +100,10 @@ 
      */
     private function allowedVariableUsage(Schema $schema, Type $varType, $varDefaultValue, Type $locationType, $locationDefaultValue) : bool
     {
-        if ($locationType instanceof NonNull && ! $varType instanceof NonNull) {
-            $hasNonNullVariableDefaultValue = $varDefaultValue && ! $varDefaultValue instanceof NullValueNode;
-            $hasLocationDefaultValue        = ! Utils::isInvalid($locationDefaultValue);
-            if (! $hasNonNullVariableDefaultValue && ! $hasLocationDefaultValue) {
+        if ($locationType instanceof NonNull && !$varType instanceof NonNull) {
+            $hasNonNullVariableDefaultValue = $varDefaultValue && !$varDefaultValue instanceof NullValueNode;
+            $hasLocationDefaultValue        = !Utils::isInvalid($locationDefaultValue);
+            if (!$hasNonNullVariableDefaultValue && !$hasLocationDefaultValue) {
                 return false;
             }
             $nullableLocationType = $locationType->getWrappedType();

src/Validator/Rules/UniqueDirectivesPerLocation.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
     public function getASTVisitor(ASTValidationContext $context)
     {
         return [
-            'enter' => static function (Node $node) use ($context) {
-                if (! isset($node->directives)) {
+            'enter' => static function(Node $node) use ($context) {
+                if (!isset($node->directives)) {
                     return;
                 }
 

src/Validator/Rules/ProvidedRequiredArguments.php

Spacing +6 added lines, -6 removed lines

@@ -19,10 +19,10 @@ 
     {
         return [
             NodeKind::FIELD     => [
-                'leave' => static function (FieldNode $fieldNode) use ($context) {
+                'leave' => static function(FieldNode $fieldNode) use ($context) {
                     $fieldDef = $context->getFieldDef();
 
-                    if (! $fieldDef) {
+                    if (!$fieldDef) {
                         return Visitor::skipNode();
                     }
                     $argNodes = $fieldNode->arguments ?: [];
@@ -33,7 +33,7 @@ 
                     }
                     foreach ($fieldDef->args as $argDef) {
                         $argNode = $argNodeMap[$argDef->name] ?? null;
-                        if ($argNode || (! ($argDef->getType() instanceof NonNull)) || $argDef->defaultValueExists()) {
+                        if ($argNode || (!($argDef->getType() instanceof NonNull)) || $argDef->defaultValueExists()) {
                             continue;
                         }
 
@@ -45,9 +45,9 @@ 
                 },
             ],
             NodeKind::DIRECTIVE => [
-                'leave' => static function (DirectiveNode $directiveNode) use ($context) {
+                'leave' => static function(DirectiveNode $directiveNode) use ($context) {
                     $directiveDef = $context->getDirective();
-                    if (! $directiveDef) {
+                    if (!$directiveDef) {
                         return Visitor::skipNode();
                     }
                     $argNodes   = $directiveNode->arguments ?: [];
@@ -58,7 +58,7 @@ 
 
                     foreach ($directiveDef->args as $argDef) {
                         $argNode = $argNodeMap[$argDef->name] ?? null;
-                        if ($argNode || (! ($argDef->getType() instanceof NonNull)) || $argDef->defaultValueExists()) {
+                        if ($argNode || (!($argDef->getType() instanceof NonNull)) || $argDef->defaultValueExists()) {
                             continue;
                         }
 

src/Validator/Rules/NoFragmentCycles.php

Spacing +5 added lines, -5 removed lines

@@ -41,10 +41,10 @@ 
         $this->spreadPathIndexByName = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => static function () {
+            NodeKind::OPERATION_DEFINITION => static function() {
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => function (FragmentDefinitionNode $node) use ($context) {
+            NodeKind::FRAGMENT_DEFINITION  => function(FragmentDefinitionNode $node) use ($context) {
                 $this->detectCycleRecursive($node, $context);
 
                 return Visitor::skipNode();
@@ -54,7 +54,7 @@ 
 
     private function detectCycleRecursive(FragmentDefinitionNode $fragment, ValidationContext $context)
     {
-        if (! empty($this->visitedFrags[$fragment->name->value])) {
+        if (!empty($this->visitedFrags[$fragment->name->value])) {
             return;
         }
 
@@ -82,7 +82,7 @@ 
                 }
             } else {
                 $cyclePath     = array_slice($this->spreadPath, $cycleIndex);
-                $fragmentNames = Utils::map(array_slice($cyclePath, 0, -1), static function ($s) {
+                $fragmentNames = Utils::map(array_slice($cyclePath, 0, -1), static function($s) {
                     return $s->name->value;
                 });
 
@@ -105,7 +105,7 @@ 
         return sprintf(
             'Cannot spread fragment "%s" within itself%s.',
             $fragName,
-            ! empty($spreadNames) ? ' via ' . implode(', ', $spreadNames) : ''
+            !empty($spreadNames) ? ' via ' . implode(', ', $spreadNames) : ''
         );
     }
 }