simPod / graphql-php

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);
     }
 
     /**

src/Validator/Rules/ProvidedNonNullArguments.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)) {
+                        if ($argNode || !($argDef->getType() instanceof NonNull)) {
                             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)) {
+                        if ($argNode || !($argDef->getType() instanceof NonNull)) {
                             continue;
                         }
 

src/Validator/Rules/ProvidedRequiredArgumentsOnDirectives.php

Spacing +10 added lines, -10 removed lines

@@ -42,10 +42,10 @@ 
 
         foreach ($definedDirectives as $directive) {
             $requiredArgsMap[$directive->name] = Utils::keyMap(
-                array_filter($directive->args, static function (FieldArgument $arg) : bool {
-                    return $arg->getType() instanceof NonNull && ! isset($arg->defaultValue);
+                array_filter($directive->args, static function(FieldArgument $arg) : bool {
+                    return $arg->getType() instanceof NonNull && !isset($arg->defaultValue);
                 }),
-                static function (FieldArgument $arg) : string {
+                static function(FieldArgument $arg) : string {
                     return $arg->name;
                 }
             );
@@ -53,7 +53,7 @@ 
 
         $astDefinition = $context->getDocument()->definitions;
         foreach ($astDefinition as $def) {
-            if (! ($def instanceof DirectiveDefinitionNode)) {
+            if (!($def instanceof DirectiveDefinitionNode)) {
                 continue;
             }
 
@@ -66,31 +66,31 @@ 
             }
 
             $requiredArgsMap[$def->name->value] = Utils::keyMap(
-                $arguments ? array_filter($arguments, static function (Node $argument) : bool {
+                $arguments ? array_filter($arguments, static function(Node $argument) : bool {
                     return $argument instanceof NonNullTypeNode &&
                         (
-                            ! isset($argument->defaultValue) ||
+                            !isset($argument->defaultValue) ||
                             $argument->defaultValue === null
                         );
                 }) : [],
-                static function (NamedTypeNode $argument) : string {
+                static function(NamedTypeNode $argument) : string {
                     return $argument->name->value;
                 }
             );
         }
 
         return [
-            NodeKind::DIRECTIVE => static function (DirectiveNode $directiveNode) use ($requiredArgsMap, $context) {
+            NodeKind::DIRECTIVE => static function(DirectiveNode $directiveNode) use ($requiredArgsMap, $context) {
                 $directiveName = $directiveNode->name->value;
                 $requiredArgs  = $requiredArgsMap[$directiveName] ?? null;
-                if (! $requiredArgs) {
+                if (!$requiredArgs) {
                     return;
                 }
 
                 $argNodes   = $directiveNode->arguments ?: [];
                 $argNodeMap = Utils::keyMap(
                     $argNodes,
-                    static function (ArgumentNode $arg) : string {
+                    static function(ArgumentNode $arg) : string {
                         return $arg->name->value;
                     }
                 );

src/Validator/Rules/KnownArgumentNamesOnDirectives.php

Spacing +5 added lines, -5 removed lines

@@ -38,7 +38,7 @@ 
 
         foreach ($definedDirectives as $directive) {
             $directiveArgs[$directive->name] = array_map(
-                static function (FieldArgument $arg) : string {
+                static function(FieldArgument $arg) : string {
                     return $arg->name;
                 },
                 $directive->args
@@ -47,7 +47,7 @@ 
 
         $astDefinitions = $context->getDocument()->definitions;
         foreach ($astDefinitions as $def) {
-            if (! ($def instanceof DirectiveDefinitionNode)) {
+            if (!($def instanceof DirectiveDefinitionNode)) {
                 continue;
             }
 
@@ -59,7 +59,7 @@ 
                     $arguments = iterator_to_array($arguments->getIterator());
                 }
 
-                $directiveArgs[$name] = array_map(static function (InputValueDefinitionNode $arg) : string {
+                $directiveArgs[$name] = array_map(static function(InputValueDefinitionNode $arg) : string {
                     return $arg->name->value;
                 }, $arguments);
             } else {
@@ -68,11 +68,11 @@ 
         }
 
         return [
-            NodeKind::DIRECTIVE => static function (DirectiveNode $directiveNode) use ($directiveArgs, $context) {
+            NodeKind::DIRECTIVE => static function(DirectiveNode $directiveNode) use ($directiveArgs, $context) {
                 $directiveName = $directiveNode->name->value;
                 $knownArgs     = $directiveArgs[$directiveName] ?? null;
 
-                if ($directiveNode->arguments === null || ! $knownArgs) {
+                if ($directiveNode->arguments === null || !$knownArgs) {
                     return;
                 }
 

src/Validator/Rules/ExecutableDefinitions.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::DOCUMENT => static function (DocumentNode $node) use ($context) {
+            NodeKind::DOCUMENT => static function(DocumentNode $node) use ($context) {
                 /** @var Node $definition */
                 foreach ($node->definitions as $definition) {
                     if ($definition instanceof OperationDefinitionNode ||

src/Validator/Rules/VariablesDefaultValueAllowed.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::VARIABLE_DEFINITION => static function (VariableDefinitionNode $node) use ($context) {
+            NodeKind::VARIABLE_DEFINITION => static function(VariableDefinitionNode $node) use ($context) {
                 $name         = $node->variable->name->value;
                 $defaultValue = $node->defaultValue;
                 $type         = $context->getInputType();
@@ -44,10 +44,10 @@ 
 
                 return Visitor::skipNode();
             },
-            NodeKind::SELECTION_SET       => static function (SelectionSetNode $node) {
+            NodeKind::SELECTION_SET       => static function(SelectionSetNode $node) {
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION => static function (FragmentDefinitionNode $node) {
+            NodeKind::FRAGMENT_DEFINITION => static function(FragmentDefinitionNode $node) {
                 return Visitor::skipNode();
             },
         ];

src/Validator/Rules/KnownFragmentNames.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::FRAGMENT_SPREAD => static function (FragmentSpreadNode $node) use ($context) {
+            NodeKind::FRAGMENT_SPREAD => static function(FragmentSpreadNode $node) use ($context) {
                 $fragmentName = $node->name->value;
                 $fragment     = $context->getFragment($fragmentName);
                 if ($fragment) {