webonyx / graphql-php

src/Validator/Rules/FieldsOnCorrectType.php

Spacing +6 added lines, -6 removed lines

@@ -23,9 +23,9 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::FIELD => function (FieldNode $node) use ($context) {
+            NodeKind::FIELD => function(FieldNode $node) use ($context) {
                 $type = $context->getParentType();
-                if (! $type) {
+                if (!$type) {
                     return;
                 }
 
@@ -85,19 +85,19 @@ 
 
             foreach ($schema->getPossibleTypes($type) as $possibleType) {
                 $fields = $possibleType->getFields();
-                if (! isset($fields[$fieldName])) {
+                if (!isset($fields[$fieldName])) {
                     continue;
                 }
                 // This object type defines this field.
                 $suggestedObjectTypes[] = $possibleType->name;
                 foreach ($possibleType->getInterfaces() as $possibleInterface) {
                     $fields = $possibleInterface->getFields();
-                    if (! isset($fields[$fieldName])) {
+                    if (!isset($fields[$fieldName])) {
                         continue;
                     }
                     // This interface type defines this field.
                     $interfaceUsageCount[$possibleInterface->name] =
-                        ! isset($interfaceUsageCount[$possibleInterface->name])
+                        !isset($interfaceUsageCount[$possibleInterface->name])
                             ? 0
                             : $interfaceUsageCount[$possibleInterface->name] + 1;
                 }
@@ -156,7 +156,7 @@ 
             $suggestions = Utils::quotedOrList($suggestedTypeNames);
 
             $message .= sprintf(' Did you mean to use an inline fragment on %s?', $suggestions);
-        } elseif (! empty($suggestedFieldNames)) {
+        } elseif (!empty($suggestedFieldNames)) {
             $suggestions = Utils::quotedOrList($suggestedFieldNames);
 
             $message .= sprintf(' Did you mean %s?', $suggestions);

src/Validator/Rules/DisableIntrospection.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         return $this->invokeIfNeeded(
             $context,
             [
-                NodeKind::FIELD => static function (FieldNode $node) use ($context) {
+                NodeKind::FIELD => static function(FieldNode $node) use ($context) {
                     if ($node->name->value !== '__type' && $node->name->value !== '__schema') {
                         return;
                     }

src/Validator/Rules/NoUnusedFragments.php

Spacing +4 added lines, -4 removed lines

@@ -26,18 +26,18 @@ 
         $this->fragmentDefs  = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => function ($node) {
+            NodeKind::OPERATION_DEFINITION => function($node) {
                 $this->operationDefs[] = $node;
 
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => function (FragmentDefinitionNode $def) {
+            NodeKind::FRAGMENT_DEFINITION  => function(FragmentDefinitionNode $def) {
                 $this->fragmentDefs[] = $def;
 
                 return Visitor::skipNode();
             },
             NodeKind::DOCUMENT             => [
-                'leave' => function () use ($context) {
+                'leave' => function() use ($context) {
                     $fragmentNameUsed = [];
 
                     foreach ($this->operationDefs as $operation) {
@@ -48,7 +48,7 @@ 
 
                     foreach ($this->fragmentDefs as $fragmentDef) {
                         $fragName = $fragmentDef->name->value;
-                        if (! empty($fragmentNameUsed[$fragName])) {
+                        if (!empty($fragmentNameUsed[$fragName])) {
                             continue;
                         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -40,18 +40,18 @@
 
         return [
             NodeKind::OBJECT       => [
-                'enter' => function () {
+                'enter' => function() {
                     $this->knownNameStack[] = $this->knownNames;
                     $this->knownNames       = [];
                 },
-                'leave' => function () {
+                'leave' => function() {
                     $this->knownNames = array_pop($this->knownNameStack);
                 },
             ],
-            NodeKind::OBJECT_FIELD => function (ObjectFieldNode $node) use ($context) {
+            NodeKind::OBJECT_FIELD => function(ObjectFieldNode $node) use ($context) {
                 $fieldName = $node->name->value;
 
-                if (! empty($this->knownNames[$fieldName])) {
+                if (!empty($this->knownNames[$fieldName])) {
                     $context->reportError(new Error(
                         self::duplicateInputFieldMessage($fieldName),
                         [$this->knownNames[$fieldName], $node->name]

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