webonyx / graphql-php

src/Utils/AST.php

Spacing +16 added lines, -16 removed lines

@@ -88,7 +88,7 @@ 
      */
     public static function fromArray(array $node) : Node
     {
-        if (! isset($node['kind']) || ! isset(NodeKind::$classMap[$node['kind']])) {
+        if (!isset($node['kind']) || !isset(NodeKind::$classMap[$node['kind']])) {
             throw new InvariantViolation('Unexpected node structure: ' . Utils::printSafeJson($node));
         }
 
@@ -176,7 +176,7 @@ 
                 $valuesNodes = [];
                 foreach ($value as $item) {
                     $itemNode = self::astFromValue($item, $itemType);
-                    if (! $itemNode) {
+                    if (!$itemNode) {
                         continue;
                     }
 
@@ -194,7 +194,7 @@ 
         if ($type instanceof InputObjectType) {
             $isArray     = is_array($value);
             $isArrayLike = $isArray || $value instanceof ArrayAccess;
-            if ($value === null || (! $isArrayLike && ! is_object($value))) {
+            if ($value === null || (!$isArrayLike && !is_object($value))) {
                 return null;
             }
             $fields     = $type->getFields();
@@ -218,13 +218,13 @@ 
                     $fieldExists = property_exists($value, $fieldName);
                 }
 
-                if (! $fieldExists) {
+                if (!$fieldExists) {
                     continue;
                 }
 
                 $fieldNode = self::astFromValue($fieldValue, $field->getType());
 
-                if (! $fieldNode) {
+                if (!$fieldNode) {
                     continue;
                 }
 
@@ -343,7 +343,7 @@ 
         if ($valueNode instanceof VariableNode) {
             $variableName = $valueNode->name->value;
 
-            if (! $variables || ! array_key_exists($variableName, $variables)) {
+            if (!$variables || !array_key_exists($variableName, $variables)) {
                 // No valid return value.
                 return $undefined;
             }
@@ -396,7 +396,7 @@ 
         }
 
         if ($type instanceof InputObjectType) {
-            if (! $valueNode instanceof ObjectValueNode) {
+            if (!$valueNode instanceof ObjectValueNode) {
                 // Invalid: intentionally return no value.
                 return $undefined;
             }
@@ -405,7 +405,7 @@ 
             $fields     = $type->getFields();
             $fieldNodes = Utils::keyMap(
                 $valueNode->fields,
-                static function ($field) {
+                static function($field) {
                     return $field->name->value;
                 }
             );
@@ -441,11 +441,11 @@ 
         }
 
         if ($type instanceof EnumType) {
-            if (! $valueNode instanceof EnumValueNode) {
+            if (!$valueNode instanceof EnumValueNode) {
                 return $undefined;
             }
             $enumValue = $type->getValue($valueNode->value);
-            if (! $enumValue) {
+            if (!$enumValue) {
                 return $undefined;
             }
 
@@ -478,7 +478,7 @@ 
     private static function isMissingVariable(ValueNode $valueNode, $variables)
     {
         return $valueNode instanceof VariableNode &&
-            (count($variables) === 0 || ! array_key_exists($valueNode->name->value, $variables));
+            (count($variables) === 0 || !array_key_exists($valueNode->name->value, $variables));
     }
 
     /**
@@ -521,7 +521,7 @@ 
                 return $valueNode->value;
             case $valueNode instanceof ListValueNode:
                 return array_map(
-                    static function ($node) use ($variables) {
+                    static function($node) use ($variables) {
                         return self::valueFromASTUntyped($node, $variables);
                     },
                     iterator_to_array($valueNode->values)
@@ -529,13 +529,13 @@ 
             case $valueNode instanceof ObjectValueNode:
                 return array_combine(
                     array_map(
-                        static function ($field) {
+                        static function($field) {
                             return $field->name->value;
                         },
                         iterator_to_array($valueNode->fields)
                     ),
                     array_map(
-                        static function ($field) use ($variables) {
+                        static function($field) use ($variables) {
                             return self::valueFromASTUntyped($field->value, $variables);
                         },
                         iterator_to_array($valueNode->fields)
@@ -595,11 +595,11 @@ 
     {
         if ($document->definitions) {
             foreach ($document->definitions as $def) {
-                if (! ($def instanceof OperationDefinitionNode)) {
+                if (!($def instanceof OperationDefinitionNode)) {
                     continue;
                 }
 
-                if (! $operationName || (isset($def->name->value) && $def->name->value === $operationName)) {
+                if (!$operationName || (isset($def->name->value) && $def->name->value === $operationName)) {
                     return $def->operation;
                 }
             }

src/Utils/ASTDefinitionBuilder.php

Spacing +18 added lines, -18 removed lines

@@ -81,7 +81,7 @@ 
             'description' => $this->getDescription($directiveNode),
             'locations'   => Utils::map(
                 $directiveNode->locations,
-                static function ($node) {
+                static function($node) {
                     return $node->value;
                 }
             ),
@@ -111,7 +111,7 @@ 
     private function getLeadingCommentBlock($node)
     {
         $loc = $node->loc;
-        if (! $loc || ! $loc->startToken) {
+        if (!$loc || !$loc->startToken) {
             return null;
         }
         $comments = [];
@@ -134,10 +134,10 @@ 
     {
         return Utils::keyValMap(
             $values,
-            static function ($value) {
+            static function($value) {
                 return $value->name->value;
             },
-            function ($value) {
+            function($value) {
                 // Note: While this could make assertions to get the correctly typed
                 // value, that would throw immediately while type system validation
                 // with validateSchema() will produce more actionable results.
@@ -201,7 +201,7 @@ 
      */
     private function internalBuildType($typeName, $typeNode = null)
     {
-        if (! isset($this->cache[$typeName])) {
+        if (!isset($this->cache[$typeName])) {
             if (isset($this->typeDefinitionsMap[$typeName])) {
                 $type = $this->makeSchemaDef($this->typeDefinitionsMap[$typeName]);
                 if ($this->typeConfigDecorator) {
@@ -219,7 +219,7 @@ 
                             $e
                         );
                     }
-                    if (! is_array($config) || isset($config[0])) {
+                    if (!is_array($config) || isset($config[0])) {
                         throw new Error(
                             sprintf(
                                 'Type config decorator passed to %s is expected to return an array, but got %s',
@@ -274,10 +274,10 @@ 
         return new ObjectType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
-            'interfaces'  => function () use ($def) {
+            'interfaces'  => function() use ($def) {
                 return $this->makeImplementedInterfaces($def);
             },
             'astNode'     => $def,
@@ -289,10 +289,10 @@ 
         return $def->fields
             ? Utils::keyValMap(
                 $def->fields,
-                static function ($field) {
+                static function($field) {
                     return $field->name->value;
                 },
-                function ($field) {
+                function($field) {
                     return $this->buildField($field);
                 }
             )
@@ -336,7 +336,7 @@ 
             // validation with validateSchema() will produce more actionable results.
             return Utils::map(
                 $def->interfaces,
-                function ($iface) {
+                function($iface) {
                     return $this->buildType($iface);
                 }
             );
@@ -352,7 +352,7 @@ 
         return new InterfaceType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
             'astNode'     => $def,
@@ -367,10 +367,10 @@ 
             'values'      => $def->values
                 ? Utils::keyValMap(
                     $def->values,
-                    static function ($enumValue) {
+                    static function($enumValue) {
                         return $enumValue->name->value;
                     },
-                    function ($enumValue) {
+                    function($enumValue) {
                         return [
                             'description'       => $this->getDescription($enumValue),
                             'deprecationReason' => $this->getDeprecationReason($enumValue),
@@ -392,10 +392,10 @@ 
             // values below, that would throw immediately while type system
             // validation with validateSchema() will produce more actionable results.
             'types'       => $def->types
-                ? function () use ($def) {
+                ? function() use ($def) {
                     return Utils::map(
                         $def->types,
-                        function ($typeNode) {
+                        function($typeNode) {
                             return $this->buildType($typeNode);
                         }
                     );
@@ -411,7 +411,7 @@ 
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
             'astNode'     => $def,
-            'serialize'   => static function ($value) {
+            'serialize'   => static function($value) {
                 return $value;
             },
         ]);
@@ -422,7 +422,7 @@ 
         return new InputObjectType([
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $def->fields
                     ? $this->makeInputValues($def->fields)
                     : [];

src/Validator/Rules/UniqueFragmentNames.php

Spacing +2 added lines, -2 removed lines

@@ -22,10 +22,10 @@
         $this->knownFragmentNames = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => static function () {
+            NodeKind::OPERATION_DEFINITION => static function() {
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => function (FragmentDefinitionNode $node) use ($context) {
+            NodeKind::FRAGMENT_DEFINITION  => function(FragmentDefinitionNode $node) use ($context) {
                 $fragmentName = $node->name->value;
                 if (empty($this->knownFragmentNames[$fragmentName])) {
                     $this->knownFragmentNames[$fragmentName] = $node->name;

src/Validator/Rules/QueryDepth.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             $context,
             [
                 NodeKind::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinitionNode $operationDefinition) use ($context) {
+                    'leave' => function(OperationDefinitionNode $operationDefinition) use ($context) {
                         $maxDepth = $this->fieldDepth($operationDefinition);
 
                         if ($maxDepth <= $this->getMaxQueryDepth()) {

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