simPod / graphql-php

src/Language/Lexer.php

Spacing +1 added lines, -1 removed lines

@@ -495,7 +495,7 @@
                     case 117:
                         $position = $this->position;
                         [$hex]    = $this->readChars(4, true);
-                        if (! preg_match('/[0-9a-fA-F]{4}/', $hex)) {
+                        if (!preg_match('/[0-9a-fA-F]{4}/', $hex)) {
                             throw new SyntaxError(
                                 $this->source,
                                 $position - 1,

src/Language/Printer.php

Spacing +48 added lines, -48 removed lines

@@ -97,19 +97,19 @@ 
             $ast,
             [
                 'leave' => [
-                    NodeKind::NAME => static function (Node $node) {
+                    NodeKind::NAME => static function(Node $node) {
                         return '' . $node->value;
                     },
 
-                    NodeKind::VARIABLE => static function ($node) {
+                    NodeKind::VARIABLE => static function($node) {
                         return '$' . $node->name;
                     },
 
-                    NodeKind::DOCUMENT => function (DocumentNode $node) {
+                    NodeKind::DOCUMENT => function(DocumentNode $node) {
                         return $this->join($node->definitions, "\n\n") . "\n";
                     },
 
-                    NodeKind::OPERATION_DEFINITION => function (OperationDefinitionNode $node) {
+                    NodeKind::OPERATION_DEFINITION => function(OperationDefinitionNode $node) {
                         $op           = $node->operation;
                         $name         = $node->name;
                         $varDefs      = $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')');
@@ -117,20 +117,20 @@ 
                         $selectionSet = $node->selectionSet;
                         // Anonymous queries with no directives or variable definitions can use
                         // the query short form.
-                        return ! $name && ! $directives && ! $varDefs && $op === 'query'
+                        return !$name && !$directives && !$varDefs && $op === 'query'
                             ? $selectionSet
                             : $this->join([$op, $this->join([$name, $varDefs]), $directives, $selectionSet], ' ');
                     },
 
-                    NodeKind::VARIABLE_DEFINITION => function (VariableDefinitionNode $node) {
+                    NodeKind::VARIABLE_DEFINITION => function(VariableDefinitionNode $node) {
                         return $node->variable . ': ' . $node->type . $this->wrap(' = ', $node->defaultValue);
                     },
 
-                    NodeKind::SELECTION_SET => function (SelectionSetNode $node) {
+                    NodeKind::SELECTION_SET => function(SelectionSetNode $node) {
                         return $this->block($node->selections);
                     },
 
-                    NodeKind::FIELD => function (FieldNode $node) {
+                    NodeKind::FIELD => function(FieldNode $node) {
                         return $this->join(
                             [
                                 $this->wrap('', $node->alias, ': ') . $node->name . $this->wrap(
@@ -145,15 +145,15 @@ 
                         );
                     },
 
-                    NodeKind::ARGUMENT => static function (ArgumentNode $node) {
+                    NodeKind::ARGUMENT => static function(ArgumentNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::FRAGMENT_SPREAD => function (FragmentSpreadNode $node) {
+                    NodeKind::FRAGMENT_SPREAD => function(FragmentSpreadNode $node) {
                         return '...' . $node->name . $this->wrap(' ', $this->join($node->directives, ' '));
                     },
 
-                    NodeKind::INLINE_FRAGMENT => function (InlineFragmentNode $node) {
+                    NodeKind::INLINE_FRAGMENT => function(InlineFragmentNode $node) {
                         return $this->join(
                             [
                                 '...',
@@ -165,7 +165,7 @@ 
                         );
                     },
 
-                    NodeKind::FRAGMENT_DEFINITION => function (FragmentDefinitionNode $node) {
+                    NodeKind::FRAGMENT_DEFINITION => function(FragmentDefinitionNode $node) {
                         // Note: fragment variable definitions are experimental and may be changed or removed in the future.
                         return sprintf('fragment %s', $node->name)
                             . $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')')
@@ -174,15 +174,15 @@ 
                             . $node->selectionSet;
                     },
 
-                    NodeKind::INT => static function (IntValueNode $node) {
+                    NodeKind::INT => static function(IntValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::FLOAT => static function (FloatValueNode $node) {
+                    NodeKind::FLOAT => static function(FloatValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::STRING => function (StringValueNode $node, $key) {
+                    NodeKind::STRING => function(StringValueNode $node, $key) {
                         if ($node->block) {
                             return $this->printBlockString($node->value, $key === 'description');
                         }
@@ -190,47 +190,47 @@ 
                         return json_encode($node->value);
                     },
 
-                    NodeKind::BOOLEAN => static function (BooleanValueNode $node) {
+                    NodeKind::BOOLEAN => static function(BooleanValueNode $node) {
                         return $node->value ? 'true' : 'false';
                     },
 
-                    NodeKind::NULL => static function (NullValueNode $node) {
+                    NodeKind::NULL => static function(NullValueNode $node) {
                         return 'null';
                     },
 
-                    NodeKind::ENUM => static function (EnumValueNode $node) {
+                    NodeKind::ENUM => static function(EnumValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::LST => function (ListValueNode $node) {
+                    NodeKind::LST => function(ListValueNode $node) {
                         return '[' . $this->join($node->values, ', ') . ']';
                     },
 
-                    NodeKind::OBJECT => function (ObjectValueNode $node) {
+                    NodeKind::OBJECT => function(ObjectValueNode $node) {
                         return '{' . $this->join($node->fields, ', ') . '}';
                     },
 
-                    NodeKind::OBJECT_FIELD => static function (ObjectFieldNode $node) {
+                    NodeKind::OBJECT_FIELD => static function(ObjectFieldNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::DIRECTIVE => function (DirectiveNode $node) {
+                    NodeKind::DIRECTIVE => function(DirectiveNode $node) {
                         return '@' . $node->name . $this->wrap('(', $this->join($node->arguments, ', '), ')');
                     },
 
-                    NodeKind::NAMED_TYPE => static function (NamedTypeNode $node) {
+                    NodeKind::NAMED_TYPE => static function(NamedTypeNode $node) {
                         return $node->name;
                     },
 
-                    NodeKind::LIST_TYPE => static function (ListTypeNode $node) {
+                    NodeKind::LIST_TYPE => static function(ListTypeNode $node) {
                         return '[' . $node->type . ']';
                     },
 
-                    NodeKind::NON_NULL_TYPE => static function (NonNullTypeNode $node) {
+                    NodeKind::NON_NULL_TYPE => static function(NonNullTypeNode $node) {
                         return $node->type . '!';
                     },
 
-                    NodeKind::SCHEMA_DEFINITION => function (SchemaDefinitionNode $def) {
+                    NodeKind::SCHEMA_DEFINITION => function(SchemaDefinitionNode $def) {
                         return $this->join(
                             [
                                 'schema',
@@ -241,15 +241,15 @@ 
                         );
                     },
 
-                    NodeKind::OPERATION_TYPE_DEFINITION => static function (OperationTypeDefinitionNode $def) {
+                    NodeKind::OPERATION_TYPE_DEFINITION => static function(OperationTypeDefinitionNode $def) {
                         return $def->operation . ': ' . $def->type;
                     },
 
-                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function (ScalarTypeDefinitionNode $def) {
+                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function(ScalarTypeDefinitionNode $def) {
                         return $this->join(['scalar', $def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function (ObjectTypeDefinitionNode $def) {
+                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function(ObjectTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'type',
@@ -262,8 +262,8 @@ 
                         );
                     }),
 
-                    NodeKind::FIELD_DEFINITION => $this->addDescription(function (FieldDefinitionNode $def) {
-                        $noIndent = Utils::every($def->arguments, static function (string $arg) {
+                    NodeKind::FIELD_DEFINITION => $this->addDescription(function(FieldDefinitionNode $def) {
+                        $noIndent = Utils::every($def->arguments, static function(string $arg) {
                             return strpos($arg, "\n") === false;
                         });
 
@@ -275,7 +275,7 @@ 
                             . $this->wrap(' ', $this->join($def->directives, ' '));
                     }),
 
-                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function (InputValueDefinitionNode $def) {
+                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function(InputValueDefinitionNode $def) {
                         return $this->join(
                             [
                                 $def->name . ': ' . $def->type,
@@ -287,7 +287,7 @@ 
                     }),
 
                     NodeKind::INTERFACE_TYPE_DEFINITION => $this->addDescription(
-                        function (InterfaceTypeDefinitionNode $def) {
+                        function(InterfaceTypeDefinitionNode $def) {
                             return $this->join(
                                 [
                                     'interface',
@@ -300,7 +300,7 @@ 
                         }
                     ),
 
-                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function (UnionTypeDefinitionNode $def) {
+                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function(UnionTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'union',
@@ -314,7 +314,7 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function (EnumTypeDefinitionNode $def) {
+                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function(EnumTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'enum',
@@ -326,11 +326,11 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function (EnumValueDefinitionNode $def) {
+                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function(EnumValueDefinitionNode $def) {
                         return $this->join([$def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function (
+                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function(
                         InputObjectTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -344,7 +344,7 @@ 
                         );
                     }),
 
-                    NodeKind::SCHEMA_EXTENSION => function (SchemaTypeExtensionNode $def) {
+                    NodeKind::SCHEMA_EXTENSION => function(SchemaTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend schema',
@@ -355,7 +355,7 @@ 
                         );
                     },
 
-                    NodeKind::SCALAR_TYPE_EXTENSION => function (ScalarTypeExtensionNode $def) {
+                    NodeKind::SCALAR_TYPE_EXTENSION => function(ScalarTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend scalar',
@@ -366,7 +366,7 @@ 
                         );
                     },
 
-                    NodeKind::OBJECT_TYPE_EXTENSION => function (ObjectTypeExtensionNode $def) {
+                    NodeKind::OBJECT_TYPE_EXTENSION => function(ObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend type',
@@ -379,7 +379,7 @@ 
                         );
                     },
 
-                    NodeKind::INTERFACE_TYPE_EXTENSION => function (InterfaceTypeExtensionNode $def) {
+                    NodeKind::INTERFACE_TYPE_EXTENSION => function(InterfaceTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend interface',
@@ -391,7 +391,7 @@ 
                         );
                     },
 
-                    NodeKind::UNION_TYPE_EXTENSION => function (UnionTypeExtensionNode $def) {
+                    NodeKind::UNION_TYPE_EXTENSION => function(UnionTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend union',
@@ -405,7 +405,7 @@ 
                         );
                     },
 
-                    NodeKind::ENUM_TYPE_EXTENSION => function (EnumTypeExtensionNode $def) {
+                    NodeKind::ENUM_TYPE_EXTENSION => function(EnumTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend enum',
@@ -417,7 +417,7 @@ 
                         );
                     },
 
-                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function (InputObjectTypeExtensionNode $def) {
+                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function(InputObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend input',
@@ -429,8 +429,8 @@ 
                         );
                     },
 
-                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function (DirectiveDefinitionNode $def) {
-                        $noIndent = Utils::every($def->arguments, static function (string $arg) {
+                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function(DirectiveDefinitionNode $def) {
+                        $noIndent = Utils::every($def->arguments, static function(string $arg) {
                             return strpos($arg, "\n") === false;
                         });
 
@@ -448,7 +448,7 @@ 
 
     public function addDescription(callable $cb)
     {
-        return function ($node) use ($cb) {
+        return function($node) use ($cb) {
             return $this->join([$node->description, $cb($node)], "\n");
         };
     }
@@ -495,7 +495,7 @@ 
                 $separator,
                 Utils::filter(
                     $maybeArray,
-                    static function ($x) {
+                    static function($x) {
                         return (bool) $x;
                     }
                 )

src/Language/Visitor.php

Spacing +14 added lines, -14 removed lines

@@ -208,7 +208,7 @@ 
             $isEdited  = $isLeaving && count($edits) !== 0;
 
             if ($isLeaving) {
-                $key    = ! $ancestors ? $UNDEFINED : $path[count($path) - 1];
+                $key    = !$ancestors ? $UNDEFINED : $path[count($path) - 1];
                 $node   = $parent;
                 $parent = array_pop($ancestors);
 
@@ -262,8 +262,8 @@ 
             }
 
             $result = null;
-            if (! $node instanceof NodeList && ! is_array($node)) {
-                if (! ($node instanceof Node)) {
+            if (!$node instanceof NodeList && !is_array($node)) {
+                if (!($node instanceof Node)) {
                     throw new Exception('Invalid AST Node: ' . json_encode($node));
                 }
 
@@ -278,7 +278,7 @@ 
                             if ($result->doBreak) {
                                 break;
                             }
-                            if (! $isLeaving && $result->doContinue) {
+                            if (!$isLeaving && $result->doContinue) {
                                 array_pop($path);
                                 continue;
                             }
@@ -290,8 +290,8 @@ 
                         }
 
                         $edits[] = [$key, $editValue];
-                        if (! $isLeaving) {
-                            if (! ($editValue instanceof Node)) {
+                        if (!$isLeaving) {
+                            if (!($editValue instanceof Node)) {
                                 array_pop($path);
                                 continue;
                             }
@@ -309,7 +309,7 @@ 
             if ($isLeaving) {
                 array_pop($path);
             } else {
-                $stack   = [
+                $stack = [
                     'inArray' => $inArray,
                     'index'   => $index,
                     'keys'    => $keys,
@@ -391,9 +391,9 @@ 
         $skipping      = new SplFixedArray($visitorsCount);
 
         return [
-            'enter' => static function (Node $node) use ($visitors, $skipping, $visitorsCount) {
+            'enter' => static function(Node $node) use ($visitors, $skipping, $visitorsCount) {
                 for ($i = 0; $i < $visitorsCount; $i++) {
-                    if (! empty($skipping[$i])) {
+                    if (!empty($skipping[$i])) {
                         continue;
                     }
 
@@ -403,7 +403,7 @@ 
                         false
                     );
 
-                    if (! $fn) {
+                    if (!$fn) {
                         continue;
                     }
 
@@ -422,7 +422,7 @@ 
                     }
                 }
             },
-            'leave' => static function (Node $node) use ($visitors, $skipping, $visitorsCount) {
+            'leave' => static function(Node $node) use ($visitors, $skipping, $visitorsCount) {
                 for ($i = 0; $i < $visitorsCount; $i++) {
                     if (empty($skipping[$i])) {
                         $fn = self::getVisitFn(
@@ -458,7 +458,7 @@ 
     public static function visitWithTypeInfo(TypeInfo $typeInfo, $visitor)
     {
         return [
-            'enter' => static function (Node $node) use ($typeInfo, $visitor) {
+            'enter' => static function(Node $node) use ($typeInfo, $visitor) {
                 $typeInfo->enter($node);
                 $fn = self::getVisitFn($visitor, $node->kind, false);
 
@@ -476,7 +476,7 @@ 
 
                 return null;
             },
-            'leave' => static function (Node $node) use ($typeInfo, $visitor) {
+            'leave' => static function(Node $node) use ($typeInfo, $visitor) {
                 $fn     = self::getVisitFn($visitor, $node->kind, true);
                 $result = $fn ? call_user_func_array($fn, func_get_args()) : null;
                 $typeInfo->leave($node);
@@ -501,7 +501,7 @@ 
 
         $kindVisitor = $visitor[$kind] ?? null;
 
-        if (! $isLeaving && is_callable($kindVisitor)) {
+        if (!$isLeaving && is_callable($kindVisitor)) {
             // { Kind() {} }
             return $kindVisitor;
         }

src/Validator/ValidationContext.php

Spacing +7 added lines, -7 removed lines

@@ -128,10 +128,10 @@ 
                 Visitor::visitWithTypeInfo(
                     $typeInfo,
                     [
-                        NodeKind::VARIABLE_DEFINITION => static function () {
+                        NodeKind::VARIABLE_DEFINITION => static function() {
                             return false;
                         },
-                        NodeKind::VARIABLE            => static function (VariableNode $variable) use (
+                        NodeKind::VARIABLE            => static function(VariableNode $variable) use (
                             &$newUsages,
                             $typeInfo
                         ) {
@@ -158,19 +158,19 @@ 
             $fragments      = [];
             $collectedNames = [];
             $nodesToVisit   = [$operation];
-            while (! empty($nodesToVisit)) {
+            while (!empty($nodesToVisit)) {
                 $node    = array_pop($nodesToVisit);
                 $spreads = $this->getFragmentSpreads($node);
                 foreach ($spreads as $spread) {
                     $fragName = $spread->name->value;
 
-                    if (! empty($collectedNames[$fragName])) {
+                    if (!empty($collectedNames[$fragName])) {
                         continue;
                     }
 
                     $collectedNames[$fragName] = true;
                     $fragment                  = $this->getFragment($fragName);
-                    if (! $fragment) {
+                    if (!$fragment) {
                         continue;
                     }
 
@@ -194,7 +194,7 @@ 
             $spreads = [];
             /** @var SelectionSetNode[] $setsToVisit */
             $setsToVisit = [$node->selectionSet];
-            while (! empty($setsToVisit)) {
+            while (!empty($setsToVisit)) {
                 $set = array_pop($setsToVisit);
 
                 for ($i = 0, $selectionCount = count($set->selections); $i < $selectionCount; $i++) {
@@ -220,7 +220,7 @@ 
     public function getFragment($name)
     {
         $fragments = $this->fragments;
-        if (! $fragments) {
+        if (!$fragments) {
             $fragments = [];
             foreach ($this->getDocument()->definitions as $statement) {
                 if ($statement->kind !== NodeKind::FRAGMENT_DEFINITION) {

src/Validator/DocumentValidator.php

Spacing +3 added lines, -3 removed lines

@@ -127,7 +127,7 @@ 
      */
     public static function allRules()
     {
-        if (! self::$initRules) {
+        if (!self::$initRules) {
             static::$rules     = array_merge(static::defaultRules(), self::securityRules(), self::$rules);
             static::$initRules = true;
         }
@@ -268,7 +268,7 @@ 
         return is_array($value)
             ? count(array_filter(
                 $value,
-                static function ($item) {
+                static function($item) {
                     return $item instanceof Exception || $item instanceof Throwable;
                 }
             )) === count($value)
@@ -316,7 +316,7 @@ 
             throw new Error(
                 implode(
                     "\n\n",
-                    array_map(static function (Error $error) : string {
+                    array_map(static function(Error $error) : string {
                         return $error->message;
                     }, $errors)
                 )

src/Validator/Rules/KnownArgumentNames.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::ARGUMENT => static function (ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
+            NodeKind::ARGUMENT => static function(ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
                 /** @var NodeList|Node[] $ancestors */
                 $argDef = $context->getArgument();
                 if ($argDef !== null) {
@@ -46,7 +46,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $fieldDef->args
@@ -66,7 +66,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $directive->args
@@ -87,7 +87,7 @@ 
     public static function unknownArgMessage($argName, $fieldName, $typeName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on field "%s" of type "%s".', $argName, $fieldName, $typeName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 
@@ -100,7 +100,7 @@ 
     public static function unknownDirectiveArgMessage($argName, $directiveName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on directive "@%s".', $argName, $directiveName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 

src/Validator/Rules/NoUndefinedVariables.php

Spacing +4 added lines, -4 removed lines

@@ -25,17 +25,17 @@ 
 
         return [
             NodeKind::OPERATION_DEFINITION => [
-                'enter' => static function () use (&$variableNameDefined) {
+                'enter' => static function() use (&$variableNameDefined) {
                     $variableNameDefined = [];
                 },
-                'leave' => static function (OperationDefinitionNode $operation) use (&$variableNameDefined, $context) {
+                'leave' => static function(OperationDefinitionNode $operation) use (&$variableNameDefined, $context) {
                     $usages = $context->getRecursiveVariableUsages($operation);
 
                     foreach ($usages as $usage) {
                         $node    = $usage['node'];
                         $varName = $node->name->value;
 
-                        if (! empty($variableNameDefined[$varName])) {
+                        if (!empty($variableNameDefined[$varName])) {
                             continue;
                         }
 
@@ -49,7 +49,7 @@ 
                     }
                 },
             ],
-            NodeKind::VARIABLE_DEFINITION  => static function (VariableDefinitionNode $def) use (&$variableNameDefined) {
+            NodeKind::VARIABLE_DEFINITION  => static function(VariableDefinitionNode $def) use (&$variableNameDefined) {
                 $variableNameDefined[$def->variable->name->value] = true;
             },
         ];

src/Validator/Rules/OverlappingFieldsCanBeMerged.php

Spacing +20 added lines, -24 removed lines

@@ -60,7 +60,7 @@ 
         $this->cachedFieldsAndFragmentNames = new SplObjectStorage();
 
         return [
-            NodeKind::SELECTION_SET => function (SelectionSetNode $selectionSet) use ($context) {
+            NodeKind::SELECTION_SET => function(SelectionSetNode $selectionSet) use ($context) {
                 $conflicts = $this->findConflictsWithinSelectionSet(
                     $context,
                     $context->getParentType(),
@@ -259,7 +259,7 @@ 
                     }
                     $responseName = $selection->alias ? $selection->alias->value : $fieldName;
 
-                    if (! isset($astAndDefs[$responseName])) {
+                    if (!isset($astAndDefs[$responseName])) {
                         $astAndDefs[$responseName] = [];
                     }
                     $astAndDefs[$responseName][] = [$parentType, $selection, $fieldDef];
@@ -318,7 +318,7 @@ 
                         $fields[$i],
                         $fields[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -369,7 +369,7 @@ 
         $type1 = $def1 === null ? null : $def1->getType();
         $type2 = $def2 === null ? null : $def2->getType();
 
-        if (! $areMutuallyExclusive) {
+        if (!$areMutuallyExclusive) {
             // Two aliases must refer to the same field.
             $name1 = $ast1->name->value;
             $name2 = $ast2->name->value;
@@ -381,7 +381,7 @@ 
                 ];
             }
 
-            if (! $this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
+            if (!$this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
                 return [
                     [$responseName, 'they have differing arguments'],
                     [$ast1],
@@ -443,11 +443,11 @@ 
                     break;
                 }
             }
-            if (! $argument2) {
+            if (!$argument2) {
                 return false;
             }
 
-            if (! $this->sameValue($argument1->value, $argument2->value)) {
+            if (!$this->sameValue($argument1->value, $argument2->value)) {
                 return false;
             }
         }
@@ -460,7 +460,7 @@ 
      */
     private function sameValue(Node $value1, Node $value2)
     {
-        return (! $value1 && ! $value2) || (Printer::doPrint($value1) === Printer::doPrint($value2));
+        return (!$value1 && !$value2) || (Printer::doPrint($value1) === Printer::doPrint($value2));
     }
 
     /**
@@ -474,23 +474,19 @@ 
     {
         if ($type1 instanceof ListOfType) {
             return $type2 instanceof ListOfType ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type2 instanceof ListOfType) {
             return $type1 instanceof ListOfType ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type1 instanceof NonNull) {
             return $type2 instanceof NonNull ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type2 instanceof NonNull) {
             return $type1 instanceof NonNull ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if (Type::isLeafType($type1) || Type::isLeafType($type2)) {
             return $type1 !== $type2;
@@ -617,7 +613,7 @@ 
         // maps, each field from the first field map must be compared to every field
         // in the second field map to find potential conflicts.
         foreach ($fieldMap1 as $responseName => $fields1) {
-            if (! isset($fieldMap2[$responseName])) {
+            if (!isset($fieldMap2[$responseName])) {
                 continue;
             }
 
@@ -633,7 +629,7 @@ 
                         $fields1[$i],
                         $fields2[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -667,7 +663,7 @@ 
         $comparedFragments[$fragmentName] = true;
 
         $fragment = $context->getFragment($fragmentName);
-        if (! $fragment) {
+        if (!$fragment) {
             return;
         }
 
@@ -767,7 +763,7 @@ 
 
         $fragment1 = $context->getFragment($fragmentName1);
         $fragment2 = $context->getFragment($fragmentName2);
-        if (! $fragment1 || ! $fragment2) {
+        if (!$fragment1 || !$fragment2) {
             return;
         }
 
@@ -840,7 +836,7 @@ 
             [
                 $responseName,
                 array_map(
-                    static function ($conflict) {
+                    static function($conflict) {
                         return $conflict[0];
                     },
                     $conflicts
@@ -848,14 +844,14 @@ 
             ],
             array_reduce(
                 $conflicts,
-                static function ($allFields, $conflict) {
+                static function($allFields, $conflict) {
                     return array_merge($allFields, $conflict[1]);
                 },
                 [$ast1]
             ),
             array_reduce(
                 $conflicts,
-                static function ($allFields, $conflict) {
+                static function($allFields, $conflict) {
                     return array_merge($allFields, $conflict[2]);
                 },
                 [$ast2]
@@ -882,7 +878,7 @@ 
     {
         if (is_array($reason)) {
             $tmp = array_map(
-                static function ($tmp) {
+                static function($tmp) {
                     [$responseName, $subReason] = $tmp;
 
                     $reasonMessage = self::reasonMessage($subReason);

src/Validator/Rules/VariablesInAllowedPosition.php

Spacing +5 added lines, -5 removed lines

@@ -24,10 +24,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) {
@@ -48,7 +48,7 @@ 
                         $schema  = $context->getSchema();
                         $varType = TypeInfo::typeFromAST($schema, $varDef->type);
 
-                        if (! $varType || TypeComparators::isTypeSubTypeOf(
+                        if (!$varType || TypeComparators::isTypeSubTypeOf(
                             $schema,
                             $this->effectiveType($varType, $varDef),
                             $type
@@ -63,7 +63,7 @@ 
                     }
                 },
             ],
-            NodeKind::VARIABLE_DEFINITION  => function (VariableDefinitionNode $varDefNode) {
+            NodeKind::VARIABLE_DEFINITION  => function(VariableDefinitionNode $varDefNode) {
                 $this->varDefMap[$varDefNode->variable->name->value] = $varDefNode;
             },
         ];
@@ -71,7 +71,7 @@ 
 
     private function effectiveType($varType, $varDef)
     {
-        return ! $varDef->defaultValue || $varType instanceof NonNull ? $varType : new NonNull($varType);
+        return !$varDef->defaultValue || $varType instanceof NonNull ? $varType : new NonNull($varType);
     }
 
     /**