simPod / graphql-php

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

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $schemaDefinitionsCount = 0;
 
         return [
-            NodeKind::SCHEMA_DEFINITION => static function (SchemaDefinitionNode $node) use ($alreadyDefined, $context, &$schemaDefinitionsCount) {
+            NodeKind::SCHEMA_DEFINITION => static function(SchemaDefinitionNode $node) use ($alreadyDefined, $context, &$schemaDefinitionsCount) {
                 if ($alreadyDefined !== false) {
                     $context->reportError(new Error(self::canNotDefineSchemaWithinExtensionMessage(), $node));
 

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

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
     public function getASTVisitor(ASTValidationContext $context)
     {
         return [
-            'enter' => static function (Node $node) use ($context) {
-                if (! isset($node->directives)) {
+            'enter' => static function(Node $node) use ($context) {
+                if (!isset($node->directives)) {
                     return;
                 }
 

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

src/Validator/Rules/NoFragmentCycles.php

Spacing +5 added lines, -5 removed lines

@@ -41,10 +41,10 @@ 
         $this->spreadPathIndexByName = [];
 
         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) {
                 $this->detectCycleRecursive($node, $context);
 
                 return Visitor::skipNode();
@@ -54,7 +54,7 @@ 
 
     private function detectCycleRecursive(FragmentDefinitionNode $fragment, ValidationContext $context)
     {
-        if (! empty($this->visitedFrags[$fragment->name->value])) {
+        if (!empty($this->visitedFrags[$fragment->name->value])) {
             return;
         }
 
@@ -82,7 +82,7 @@ 
                 }
             } else {
                 $cyclePath     = array_slice($this->spreadPath, $cycleIndex);
-                $fragmentNames = Utils::map(array_slice($cyclePath, 0, -1), static function ($s) {
+                $fragmentNames = Utils::map(array_slice($cyclePath, 0, -1), static function($s) {
                     return $s->name->value;
                 });
 
@@ -105,7 +105,7 @@ 
         return sprintf(
             'Cannot spread fragment "%s" within itself%s.',
             $fragName,
-            ! empty($spreadNames) ? ' via ' . implode(', ', $spreadNames) : ''
+            !empty($spreadNames) ? ' via ' . implode(', ', $spreadNames) : ''
         );
     }
 }

src/Validator/Rules/OverlappingFieldsCanBeMerged.php

Spacing +16 added lines, -16 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));
     }
 
     /**
@@ -615,7 +615,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;
             }
 
@@ -631,7 +631,7 @@ 
                         $fields1[$i],
                         $fields2[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -665,7 +665,7 @@ 
         $comparedFragments[$fragmentName] = true;
 
         $fragment = $context->getFragment($fragmentName);
-        if (! $fragment) {
+        if (!$fragment) {
             return;
         }
 
@@ -765,7 +765,7 @@ 
 
         $fragment1 = $context->getFragment($fragmentName1);
         $fragment2 = $context->getFragment($fragmentName2);
-        if (! $fragment1 || ! $fragment2) {
+        if (!$fragment1 || !$fragment2) {
             return;
         }
 
@@ -838,7 +838,7 @@ 
             [
                 $responseName,
                 array_map(
-                    static function ($conflict) {
+                    static function($conflict) {
                         return $conflict[0];
                     },
                     $conflicts
@@ -846,14 +846,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]
@@ -880,7 +880,7 @@ 
     {
         if (is_array($reason)) {
             $tmp = array_map(
-                static function ($tmp) {
+                static function($tmp) {
                     [$responseName, $subReason] = $tmp;
 
                     $reasonMessage = self::reasonMessage($subReason);

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

Spacing +2 added lines, -2 removed lines

@@ -124,7 +124,7 @@ 
      */
     public static function allRules()
     {
-        if (! self::$initRules) {
+        if (!self::$initRules) {
             static::$rules     = array_merge(static::defaultRules(), self::securityRules(), self::$rules);
             static::$initRules = true;
         }
@@ -264,7 +264,7 @@ 
         return is_array($value)
             ? count(array_filter(
                 $value,
-                static function ($item) {
+                static function($item) {
                     return $item instanceof Throwable;
                 }
             )) === count($value)