simPod / 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;
                 }
 
@@ -84,19 +84,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;
                 }
@@ -153,7 +153,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/QueryDepth.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,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/NoFragmentCycles.php

Spacing +5 added lines, -5 removed lines

@@ -43,11 +43,11 @@ 
         $this->spreadPathIndexByName = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => function () {
+            NodeKind::OPERATION_DEFINITION => function() {
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => function (FragmentDefinitionNode $node) use ($context) {
-                if (! isset($this->visitedFrags[$node->name->value])) {
+            NodeKind::FRAGMENT_DEFINITION  => function(FragmentDefinitionNode $node) use ($context) {
+                if (!isset($this->visitedFrags[$node->name->value])) {
                     $this->detectCycleRecursive($node, $context);
                 }
 
@@ -98,7 +98,7 @@ 
                         $spreadName,
                         Utils::map(
                             $cyclePath,
-                            function ($s) {
+                            function($s) {
                                 return $s->name->value;
                             }
                         )
@@ -119,7 +119,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 +20 added lines, -24 removed lines

@@ -58,7 +58,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(),
@@ -256,7 +256,7 @@ 
                     }
                     $responseName = $selection->alias ? $selection->alias->value : $fieldName;
 
-                    if (! isset($astAndDefs[$responseName])) {
+                    if (!isset($astAndDefs[$responseName])) {
                         $astAndDefs[$responseName] = [];
                     }
                     $astAndDefs[$responseName][] = [$parentType, $selection, $fieldDef];
@@ -315,7 +315,7 @@ 
                         $fields[$i],
                         $fields[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -365,7 +365,7 @@ 
         $type1 = $def1 ? $def1->getType() : null;
         $type2 = $def2 ? $def2->getType() : null;
 
-        if (! $areMutuallyExclusive) {
+        if (!$areMutuallyExclusive) {
             // Two aliases must refer to the same field.
             $name1 = $ast1->name->value;
             $name2 = $ast2->name->value;
@@ -377,7 +377,7 @@ 
                 ];
             }
 
-            if (! $this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
+            if (!$this->sameArguments($ast1->arguments ?: [], $ast2->arguments ?: [])) {
                 return [
                     [$responseName, 'they have differing arguments'],
                     [$ast1],
@@ -439,11 +439,11 @@ 
                     break;
                 }
             }
-            if (! $argument2) {
+            if (!$argument2) {
                 return false;
             }
 
-            if (! $this->sameValue($argument1->value, $argument2->value)) {
+            if (!$this->sameValue($argument1->value, $argument2->value)) {
                 return false;
             }
         }
@@ -456,7 +456,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));
     }
 
     /**
@@ -470,23 +470,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;
@@ -612,7 +608,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;
             }
 
@@ -628,7 +624,7 @@ 
                         $fields1[$i],
                         $fields2[$j]
                     );
-                    if (! $conflict) {
+                    if (!$conflict) {
                         continue;
                     }
 
@@ -662,7 +658,7 @@ 
         $comparedFragments[$fragmentName] = true;
 
         $fragment = $context->getFragment($fragmentName);
-        if (! $fragment) {
+        if (!$fragment) {
             return;
         }
 
@@ -762,7 +758,7 @@ 
 
         $fragment1 = $context->getFragment($fragmentName1);
         $fragment2 = $context->getFragment($fragmentName2);
-        if (! $fragment1 || ! $fragment2) {
+        if (!$fragment1 || !$fragment2) {
             return;
         }
 
@@ -834,7 +830,7 @@ 
             [
                 $responseName,
                 array_map(
-                    function ($conflict) {
+                    function($conflict) {
                         return $conflict[0];
                     },
                     $conflicts
@@ -842,14 +838,14 @@ 
             ],
             array_reduce(
                 $conflicts,
-                function ($allFields, $conflict) {
+                function($allFields, $conflict) {
                     return array_merge($allFields, $conflict[1]);
                 },
                 [$ast1]
             ),
             array_reduce(
                 $conflicts,
-                function ($allFields, $conflict) {
+                function($allFields, $conflict) {
                     return array_merge($allFields, $conflict[2]);
                 },
                 [$ast2]
@@ -876,7 +872,7 @@ 
     {
         if (is_array($reason)) {
             $tmp = array_map(
-                function ($tmp) {
+                function($tmp) {
                     [$responseName, $subReason] = $tmp;
 
                     $reasonMessage = self::reasonMessage($subReason);

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

Spacing +21 added lines, -21 removed lines

@@ -45,9 +45,9 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::NULL         => function (NullValueNode $node) use ($context) {
+            NodeKind::NULL         => function(NullValueNode $node) use ($context) {
                 $type = $context->getInputType();
-                if (! ($type instanceof NonNull)) {
+                if (!($type instanceof NonNull)) {
                     return;
                 }
 
@@ -58,21 +58,21 @@ 
                     )
                 );
             },
-            NodeKind::LST          => function (ListValueNode $node) use ($context) {
+            NodeKind::LST          => function(ListValueNode $node) use ($context) {
                 // 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);
 
                     return Visitor::skipNode();
                 }
             },
-            NodeKind::OBJECT       => function (ObjectValueNode $node) use ($context) {
+            NodeKind::OBJECT       => function(ObjectValueNode $node) use ($context) {
                 // 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);
 
                     return Visitor::skipNode();
@@ -82,7 +82,7 @@ 
                 $nodeFields   = iterator_to_array($node->fields);
                 $fieldNodeMap = array_combine(
                     array_map(
-                        function ($field) {
+                        function($field) {
                             return $field->name->value;
                         },
                         $nodeFields
@@ -91,7 +91,7 @@ 
                 );
                 foreach ($inputFields as $fieldName => $fieldDef) {
                     $fieldType = $fieldDef->getType();
-                    if (isset($fieldNodeMap[$fieldName]) || ! ($fieldType instanceof NonNull)) {
+                    if (isset($fieldNodeMap[$fieldName]) || !($fieldType instanceof NonNull)) {
                         continue;
                     }
 
@@ -103,10 +103,10 @@ 
                     );
                 }
             },
-            NodeKind::OBJECT_FIELD => function (ObjectFieldNode $node) use ($context) {
+            NodeKind::OBJECT_FIELD => function(ObjectFieldNode $node) use ($context) {
                 $parentType = Type::getNamedType($context->getParentInputType());
                 $fieldType  = $context->getInputType();
-                if ($fieldType || ! ($parentType instanceof InputObjectType)) {
+                if ($fieldType || !($parentType instanceof InputObjectType)) {
                     return;
                 }
 
@@ -114,7 +114,7 @@ 
                     $node->name->value,
                     array_keys($parentType->getFields())
                 );
-                $didYouMean  = $suggestions
+                $didYouMean = $suggestions
                     ? 'Did you mean ' . Utils::orList($suggestions) . '?'
                     : null;
 
@@ -125,11 +125,11 @@ 
                     )
                 );
             },
-            NodeKind::ENUM         => function (EnumValueNode $node) use ($context) {
+            NodeKind::ENUM         => function(EnumValueNode $node) use ($context) {
                 $type = Type::getNamedType($context->getInputType());
-                if (! $type instanceof EnumType) {
+                if (!$type instanceof EnumType) {
                     $this->isValidScalar($context, $node);
-                } elseif (! $type->getValue($node->value)) {
+                } elseif (!$type->getValue($node->value)) {
                     $context->reportError(
                         new Error(
                             self::badValueMessage(
@@ -142,16 +142,16 @@ 
                     );
                 }
             },
-            NodeKind::INT          => function (IntValueNode $node) use ($context) {
+            NodeKind::INT          => function(IntValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::FLOAT        => function (FloatValueNode $node) use ($context) {
+            NodeKind::FLOAT        => function(FloatValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::STRING       => function (StringValueNode $node) use ($context) {
+            NodeKind::STRING       => function(StringValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::BOOLEAN      => function (BooleanValueNode $node) use ($context) {
+            NodeKind::BOOLEAN      => function(BooleanValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
         ];
@@ -168,13 +168,13 @@ 
         // Report any error at the full type expected by the location.
         $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::badValueMessage(
@@ -234,7 +234,7 @@ 
             $suggestions = Utils::suggestionList(
                 Printer::doPrint($node),
                 array_map(
-                    function (EnumValueDefinition $value) {
+                    function(EnumValueDefinition $value) {
                         return $value->name;
                     },
                     $type->getValues()

src/Validator/Rules/ProvidedNonNullArguments.php

Spacing +6 added lines, -6 removed lines

@@ -19,10 +19,10 @@ 
     {
         return [
             NodeKind::FIELD     => [
-                'leave' => function (FieldNode $fieldNode) use ($context) {
+                'leave' => 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' => function (DirectiveNode $directiveNode) use ($context) {
+                'leave' => 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/DocumentValidator.php

Spacing +2 added lines, -2 removed lines

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

src/Executor/ExecutionResult.php

Spacing +3 added lines, -3 removed lines

@@ -133,8 +133,8 @@ 
     {
         $result = [];
 
-        if (! empty($this->errors)) {
-            $errorsHandler = $this->errorsHandler ?: function (array $errors, callable $formatter) {
+        if (!empty($this->errors)) {
+            $errorsHandler = $this->errorsHandler ?: function(array $errors, callable $formatter) {
                 return array_map($formatter, $errors);
             };
 
@@ -148,7 +148,7 @@ 
             $result['data'] = $this->data;
         }
 
-        if (! empty($this->extensions)) {
+        if (!empty($this->extensions)) {
             $result['extensions'] = (array) $this->extensions;
         }