webonyx / graphql-php

src/Validator/Rules/UniqueOperationNames.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
         $this->knownOperationNames = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => function (OperationDefinitionNode $node) use ($context) {
+            NodeKind::OPERATION_DEFINITION => function(OperationDefinitionNode $node) use ($context) {
                 $operationName = $node->name;
 
                 if ($operationName) {
@@ -38,7 +38,7 @@ 
 
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => static function () {
+            NodeKind::FRAGMENT_DEFINITION  => static function() {
                 return Visitor::skipNode();
             },
         ];

src/Validator/Rules/KnownDirectives.php

Spacing +5 added lines, -5 removed lines

@@ -72,12 +72,12 @@ 
         $astDefinition = $context->getDocument()->definitions;
 
         foreach ($astDefinition as $def) {
-            if (! ($def instanceof DirectiveDefinitionNode)) {
+            if (!($def instanceof DirectiveDefinitionNode)) {
                 continue;
             }
 
             $locationsMap[$def->name->value] = array_map(
-                static function ($name) {
+                static function($name) {
                     return $name->value;
                 },
                 $def->locations
@@ -85,7 +85,7 @@ 
         }
 
         return [
-            NodeKind::DIRECTIVE => function (
+            NodeKind::DIRECTIVE => function(
                 DirectiveNode $node,
                 $key,
                 $parent,
@@ -98,7 +98,7 @@ 
                 $name      = $node->name->value;
                 $locations = $locationsMap[$name] ?? null;
 
-                if (! $locations) {
+                if (!$locations) {
                     $context->reportError(new Error(
                         self::unknownDirectiveMessage($name),
                         [$node]
@@ -109,7 +109,7 @@ 
 
                 $candidateLocation = $this->getDirectiveLocationForASTPath($ancestors);
 
-                if (! $candidateLocation || in_array($candidateLocation, $locations, true)) {
+                if (!$candidateLocation || in_array($candidateLocation, $locations, true)) {
                     return;
                 }
                 $context->reportError(

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/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 FragmentDefinitionNode|OperationDefinitionNode|TypeSystemDefinitionNode $definition */
                 foreach ($node->definitions as $definition) {
                     if ($definition instanceof OperationDefinitionNode ||

src/Validator/Rules/QuerySecurityRule.php

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
     protected function invokeIfNeeded(ValidationContext $context, array $validators)
     {
         // is disabled?
-        if (! $this->isEnabled()) {
+        if (!$this->isEnabled()) {
             return [];
         }
 
@@ -80,7 +80,7 @@ 
         // Importantly this does not include inline fragments.
         $definitions = $context->getDocument()->definitions;
         foreach ($definitions as $node) {
-            if (! ($node instanceof FragmentDefinitionNode)) {
+            if (!($node instanceof FragmentDefinitionNode)) {
                 continue;
             }
 
@@ -134,7 +134,7 @@ 
                         }
                     }
                     $responseName = $this->getFieldName($selection);
-                    if (! isset($_astAndDefs[$responseName])) {
+                    if (!isset($_astAndDefs[$responseName])) {
                         $_astAndDefs[$responseName] = new ArrayObject();
                     }
                     // create field context

src/Validator/Rules/KnownArgumentNames.php

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,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) {
                 $argDef = $context->getArgument();
                 if ($argDef !== null) {
                     return null;
@@ -48,7 +48,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $fieldDef->args
@@ -68,7 +68,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $directive->args
@@ -89,7 +89,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));
         }
 
@@ -102,7 +102,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/OverlappingFieldsCanBeMerged.php

Spacing +16 added lines, -16 removed lines

@@ -59,7 +59,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(),
@@ -258,7 +258,7 @@ 
                     }
                     $responseName = $selection->alias ? $selection->alias->value : $fieldName;
 
-                    if (! isset($astAndDefs[$responseName])) {
+                    if (!isset($astAndDefs[$responseName])) {
                         $astAndDefs[$responseName] = [];
                     }
                     $astAndDefs[$responseName][] = [$parentType, $selection, $fieldDef];
@@ -317,7 +317,7 @@ 
                         $fields[$i],
                         $fields[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -368,7 +368,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;
@@ -380,7 +380,7 @@ 
                 ];
             }
 
-            if (! $this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
+            if (!$this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
                 return [
                     [$responseName, 'they have differing arguments'],
                     [$ast1],
@@ -442,11 +442,11 @@ 
                     break;
                 }
             }
-            if (! $argument2) {
+            if (!$argument2) {
                 return false;
             }
 
-            if (! $this->sameValue($argument1->value, $argument2->value)) {
+            if (!$this->sameValue($argument1->value, $argument2->value)) {
                 return false;
             }
         }
@@ -459,7 +459,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));
     }
 
     /**
@@ -614,7 +614,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;
             }
 
@@ -630,7 +630,7 @@ 
                         $fields1[$i],
                         $fields2[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -664,7 +664,7 @@ 
         $comparedFragments[$fragmentName] = true;
 
         $fragment = $context->getFragment($fragmentName);
-        if (! $fragment) {
+        if (!$fragment) {
             return;
         }
 
@@ -764,7 +764,7 @@ 
 
         $fragment1 = $context->getFragment($fragmentName1);
         $fragment2 = $context->getFragment($fragmentName2);
-        if (! $fragment1 || ! $fragment2) {
+        if (!$fragment1 || !$fragment2) {
             return;
         }
 
@@ -837,7 +837,7 @@ 
             [
                 $responseName,
                 array_map(
-                    static function ($conflict) {
+                    static function($conflict) {
                         return $conflict[0];
                     },
                     $conflicts
@@ -845,14 +845,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]
@@ -879,7 +879,7 @@ 
     {
         if (is_array($reason)) {
             $tmp = array_map(
-                static function ($tmp) {
+                static function($tmp) {
                     [$responseName, $subReason] = $tmp;
 
                     $reasonMessage = self::reasonMessage($subReason);

src/Validator/Rules/FragmentsOnCompositeTypes.php

Spacing +5 added lines, -5 removed lines

@@ -19,13 +19,13 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::INLINE_FRAGMENT     => static function (InlineFragmentNode $node) use ($context) {
-                if (! $node->typeCondition) {
+            NodeKind::INLINE_FRAGMENT     => static function(InlineFragmentNode $node) use ($context) {
+                if (!$node->typeCondition) {
                     return;
                 }
 
                 $type = TypeInfo::typeFromAST($context->getSchema(), $node->typeCondition);
-                if (! $type || Type::isCompositeType($type)) {
+                if (!$type || Type::isCompositeType($type)) {
                     return;
                 }
 
@@ -34,10 +34,10 @@ 
                     [$node->typeCondition]
                 ));
             },
-            NodeKind::FRAGMENT_DEFINITION => static function (FragmentDefinitionNode $node) use ($context) {
+            NodeKind::FRAGMENT_DEFINITION => static function(FragmentDefinitionNode $node) use ($context) {
                 $type = TypeInfo::typeFromAST($context->getSchema(), $node->typeCondition);
 
-                if (! $type || Type::isCompositeType($type)) {
+                if (!$type || Type::isCompositeType($type)) {
                     return;
                 }