simPod / graphql-php

src/Executor/ExecutionResult.php

Spacing +3 added lines, -3 removed lines

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

src/Server/OperationParams.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
 
         // Some parameters could be provided as serialized JSON.
         foreach (['extensions', 'variables'] as $param) {
-            if (! is_string($params[$param])) {
+            if (!is_string($params[$param])) {
                 continue;
             }
 

src/Type/Validation/InputObjectCircularRefs.php

Spacing +3 added lines, -3 removed lines

@@ -71,13 +71,13 @@ 
                 if ($fieldType instanceof InputObjectType) {
                     $this->fieldPath[] = $field;
 
-                    if (! isset($this->fieldPathIndexByTypeName[$fieldType->name])) {
+                    if (!isset($this->fieldPathIndexByTypeName[$fieldType->name])) {
                         $this->validate($fieldType);
                     } else {
                         $cycleIndex = $this->fieldPathIndexByTypeName[$fieldType->name];
                         $cyclePath  = array_slice($this->fieldPath, $cycleIndex);
                         $fieldNames = array_map(
-                            static function (InputObjectField $field) : string {
+                            static function(InputObjectField $field) : string {
                                 return $field->name;
                             },
                             $cyclePath
@@ -87,7 +87,7 @@ 
                             'Cannot reference Input Object "' . $fieldType->name . '" within itself '
                             . 'through a series of non-null fields: "' . implode('.', $fieldNames) . '".',
                             array_map(
-                                static function (InputObjectField $field) : ?InputValueDefinitionNode {
+                                static function(InputObjectField $field) : ?InputValueDefinitionNode {
                                     return $field->astNode;
                                 },
                                 $cyclePath

src/Type/Definition/BooleanType.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      */
     public function parseLiteral($valueNode, ?array $variables = null)
     {
-        if (! $valueNode instanceof BooleanValueNode) {
+        if (!$valueNode instanceof BooleanValueNode) {
             // Intentionally without message, as all information already in wrapped Exception
             throw new Exception();
         }

src/Type/Definition/QueryPlan.php

Spacing +5 added lines, -5 removed lines

@@ -72,7 +72,7 @@ 
 
     public function hasType(string $type) : bool
     {
-        return count(array_filter($this->getReferencedTypes(), static function (string $referencedType) use ($type) {
+        return count(array_filter($this->getReferencedTypes(), static function(string $referencedType) use ($type) {
                 return $type === $referencedType;
         })) > 0;
     }
@@ -87,7 +87,7 @@ 
 
     public function hasField(string $field) : bool
     {
-        return count(array_filter($this->getReferencedFields(), static function (string $referencedField) use ($field) {
+        return count(array_filter($this->getReferencedFields(), static function(string $referencedField) use ($field) {
             return $field === $referencedField;
         })) > 0;
     }
@@ -97,7 +97,7 @@ 
      */
     public function subFields(string $typename) : array
     {
-        if (! array_key_exists($typename, $this->types)) {
+        if (!array_key_exists($typename, $this->types)) {
             return [];
         }
 
@@ -112,7 +112,7 @@ 
         $queryPlan = [];
         /** @var FieldNode $fieldNode */
         foreach ($fieldNodes as $fieldNode) {
-            if (! $fieldNode->selectionSet) {
+            if (!$fieldNode->selectionSet) {
                 continue;
             }
 
@@ -225,7 +225,7 @@ 
 
         foreach ($array2 as $key => & $value) {
             if (is_numeric($key)) {
-                if (! in_array($value, $merged, true)) {
+                if (!in_array($value, $merged, true)) {
                     $merged[] = $value;
                 }
             } elseif (is_array($value) && isset($merged[$key]) && is_array($merged[$key])) {

src/Language/Printer.php

Spacing +48 added lines, -48 removed lines

@@ -99,19 +99,19 @@ 
             $ast,
             [
                 'leave' => [
-                    NodeKind::NAME => static function (NameNode $node) {
+                    NodeKind::NAME => static function(NameNode $node) {
                         return '' . $node->value;
                     },
 
-                    NodeKind::VARIABLE => static function (VariableNode $node) {
+                    NodeKind::VARIABLE => static function(VariableNode $node) {
                         return '$' . $node->name;
                     },
 
-                    NodeKind::DOCUMENT => function (DocumentNode $node) {
+                    NodeKind::DOCUMENT => function(DocumentNode $node) {
                         return $this->join($node->definitions, "\n\n") . "\n";
                     },
 
-                    NodeKind::OPERATION_DEFINITION => function (OperationDefinitionNode $node) {
+                    NodeKind::OPERATION_DEFINITION => function(OperationDefinitionNode $node) {
                         $op           = $node->operation;
                         $name         = $node->name;
                         $varDefs      = $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')');
@@ -120,20 +120,20 @@ 
 
                         // Anonymous queries with no directives or variable definitions can use
                         // the query short form.
-                        return ! $name && ! $directives && ! $varDefs && $op === 'query'
+                        return !$name && !$directives && !$varDefs && $op === 'query'
                             ? $selectionSet
                             : $this->join([$op, $this->join([$name, $varDefs]), $directives, $selectionSet], ' ');
                     },
 
-                    NodeKind::VARIABLE_DEFINITION => function (VariableDefinitionNode $node) {
+                    NodeKind::VARIABLE_DEFINITION => function(VariableDefinitionNode $node) {
                         return $node->variable . ': ' . $node->type . $this->wrap(' = ', $node->defaultValue);
                     },
 
-                    NodeKind::SELECTION_SET => function (SelectionSetNode $node) {
+                    NodeKind::SELECTION_SET => function(SelectionSetNode $node) {
                         return $this->block($node->selections);
                     },
 
-                    NodeKind::FIELD => function (FieldNode $node) {
+                    NodeKind::FIELD => function(FieldNode $node) {
                         return $this->join(
                             [
                                 $this->wrap('', $node->alias, ': ') . $node->name . $this->wrap(
@@ -148,15 +148,15 @@ 
                         );
                     },
 
-                    NodeKind::ARGUMENT => static function (ArgumentNode $node) {
+                    NodeKind::ARGUMENT => static function(ArgumentNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::FRAGMENT_SPREAD => function (FragmentSpreadNode $node) {
+                    NodeKind::FRAGMENT_SPREAD => function(FragmentSpreadNode $node) {
                         return '...' . $node->name . $this->wrap(' ', $this->join($node->directives, ' '));
                     },
 
-                    NodeKind::INLINE_FRAGMENT => function (InlineFragmentNode $node) {
+                    NodeKind::INLINE_FRAGMENT => function(InlineFragmentNode $node) {
                         return $this->join(
                             [
                                 '...',
@@ -168,7 +168,7 @@ 
                         );
                     },
 
-                    NodeKind::FRAGMENT_DEFINITION => function (FragmentDefinitionNode $node) {
+                    NodeKind::FRAGMENT_DEFINITION => function(FragmentDefinitionNode $node) {
                         // Note: fragment variable definitions are experimental and may be changed or removed in the future.
                         return sprintf('fragment %s', $node->name)
                             . $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')')
@@ -177,15 +177,15 @@ 
                             . $node->selectionSet;
                     },
 
-                    NodeKind::INT => static function (IntValueNode $node) {
+                    NodeKind::INT => static function(IntValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::FLOAT => static function (FloatValueNode $node) {
+                    NodeKind::FLOAT => static function(FloatValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::STRING => function (StringValueNode $node, $key) {
+                    NodeKind::STRING => function(StringValueNode $node, $key) {
                         if ($node->block) {
                             return $this->printBlockString($node->value, $key === 'description');
                         }
@@ -193,47 +193,47 @@ 
                         return json_encode($node->value);
                     },
 
-                    NodeKind::BOOLEAN => static function (BooleanValueNode $node) {
+                    NodeKind::BOOLEAN => static function(BooleanValueNode $node) {
                         return $node->value ? 'true' : 'false';
                     },
 
-                    NodeKind::NULL => static function (NullValueNode $node) {
+                    NodeKind::NULL => static function(NullValueNode $node) {
                         return 'null';
                     },
 
-                    NodeKind::ENUM => static function (EnumValueNode $node) {
+                    NodeKind::ENUM => static function(EnumValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::LST => function (ListValueNode $node) {
+                    NodeKind::LST => function(ListValueNode $node) {
                         return '[' . $this->join($node->values, ', ') . ']';
                     },
 
-                    NodeKind::OBJECT => function (ObjectValueNode $node) {
+                    NodeKind::OBJECT => function(ObjectValueNode $node) {
                         return '{' . $this->join($node->fields, ', ') . '}';
                     },
 
-                    NodeKind::OBJECT_FIELD => static function (ObjectFieldNode $node) {
+                    NodeKind::OBJECT_FIELD => static function(ObjectFieldNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::DIRECTIVE => function (DirectiveNode $node) {
+                    NodeKind::DIRECTIVE => function(DirectiveNode $node) {
                         return '@' . $node->name . $this->wrap('(', $this->join($node->arguments, ', '), ')');
                     },
 
-                    NodeKind::NAMED_TYPE => static function (NamedTypeNode $node) {
+                    NodeKind::NAMED_TYPE => static function(NamedTypeNode $node) {
                         return $node->name;
                     },
 
-                    NodeKind::LIST_TYPE => static function (ListTypeNode $node) {
+                    NodeKind::LIST_TYPE => static function(ListTypeNode $node) {
                         return '[' . $node->type . ']';
                     },
 
-                    NodeKind::NON_NULL_TYPE => static function (NonNullTypeNode $node) {
+                    NodeKind::NON_NULL_TYPE => static function(NonNullTypeNode $node) {
                         return $node->type . '!';
                     },
 
-                    NodeKind::SCHEMA_DEFINITION => function (SchemaDefinitionNode $def) {
+                    NodeKind::SCHEMA_DEFINITION => function(SchemaDefinitionNode $def) {
                         return $this->join(
                             [
                                 'schema',
@@ -244,15 +244,15 @@ 
                         );
                     },
 
-                    NodeKind::OPERATION_TYPE_DEFINITION => static function (OperationTypeDefinitionNode $def) {
+                    NodeKind::OPERATION_TYPE_DEFINITION => static function(OperationTypeDefinitionNode $def) {
                         return $def->operation . ': ' . $def->type;
                     },
 
-                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function (ScalarTypeDefinitionNode $def) {
+                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function(ScalarTypeDefinitionNode $def) {
                         return $this->join(['scalar', $def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function (ObjectTypeDefinitionNode $def) {
+                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function(ObjectTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'type',
@@ -265,8 +265,8 @@ 
                         );
                     }),
 
-                    NodeKind::FIELD_DEFINITION => $this->addDescription(function (FieldDefinitionNode $def) {
-                        $noIndent = Utils::every($def->arguments, static function (string $arg) {
+                    NodeKind::FIELD_DEFINITION => $this->addDescription(function(FieldDefinitionNode $def) {
+                        $noIndent = Utils::every($def->arguments, static function(string $arg) {
                             return strpos($arg, "\n") === false;
                         });
 
@@ -278,7 +278,7 @@ 
                             . $this->wrap(' ', $this->join($def->directives, ' '));
                     }),
 
-                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function (InputValueDefinitionNode $def) {
+                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function(InputValueDefinitionNode $def) {
                         return $this->join(
                             [
                                 $def->name . ': ' . $def->type,
@@ -290,7 +290,7 @@ 
                     }),
 
                     NodeKind::INTERFACE_TYPE_DEFINITION => $this->addDescription(
-                        function (InterfaceTypeDefinitionNode $def) {
+                        function(InterfaceTypeDefinitionNode $def) {
                             return $this->join(
                                 [
                                     'interface',
@@ -303,7 +303,7 @@ 
                         }
                     ),
 
-                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function (UnionTypeDefinitionNode $def) {
+                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function(UnionTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'union',
@@ -317,7 +317,7 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function (EnumTypeDefinitionNode $def) {
+                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function(EnumTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'enum',
@@ -329,11 +329,11 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function (EnumValueDefinitionNode $def) {
+                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function(EnumValueDefinitionNode $def) {
                         return $this->join([$def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function (
+                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function(
                         InputObjectTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -347,7 +347,7 @@ 
                         );
                     }),
 
-                    NodeKind::SCHEMA_EXTENSION => function (SchemaTypeExtensionNode $def) {
+                    NodeKind::SCHEMA_EXTENSION => function(SchemaTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend schema',
@@ -358,7 +358,7 @@ 
                         );
                     },
 
-                    NodeKind::SCALAR_TYPE_EXTENSION => function (ScalarTypeExtensionNode $def) {
+                    NodeKind::SCALAR_TYPE_EXTENSION => function(ScalarTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend scalar',
@@ -369,7 +369,7 @@ 
                         );
                     },
 
-                    NodeKind::OBJECT_TYPE_EXTENSION => function (ObjectTypeExtensionNode $def) {
+                    NodeKind::OBJECT_TYPE_EXTENSION => function(ObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend type',
@@ -382,7 +382,7 @@ 
                         );
                     },
 
-                    NodeKind::INTERFACE_TYPE_EXTENSION => function (InterfaceTypeExtensionNode $def) {
+                    NodeKind::INTERFACE_TYPE_EXTENSION => function(InterfaceTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend interface',
@@ -394,7 +394,7 @@ 
                         );
                     },
 
-                    NodeKind::UNION_TYPE_EXTENSION => function (UnionTypeExtensionNode $def) {
+                    NodeKind::UNION_TYPE_EXTENSION => function(UnionTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend union',
@@ -408,7 +408,7 @@ 
                         );
                     },
 
-                    NodeKind::ENUM_TYPE_EXTENSION => function (EnumTypeExtensionNode $def) {
+                    NodeKind::ENUM_TYPE_EXTENSION => function(EnumTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend enum',
@@ -420,7 +420,7 @@ 
                         );
                     },
 
-                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function (InputObjectTypeExtensionNode $def) {
+                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function(InputObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend input',
@@ -432,8 +432,8 @@ 
                         );
                     },
 
-                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function (DirectiveDefinitionNode $def) {
-                        $noIndent = Utils::every($def->arguments, static function (string $arg) {
+                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function(DirectiveDefinitionNode $def) {
+                        $noIndent = Utils::every($def->arguments, static function(string $arg) {
                             return strpos($arg, "\n") === false;
                         });
 
@@ -451,7 +451,7 @@ 
 
     public function addDescription(callable $cb)
     {
-        return function ($node) use ($cb) {
+        return function($node) use ($cb) {
             return $this->join([$node->description, $cb($node)], "\n");
         };
     }
@@ -498,7 +498,7 @@ 
                 $separator,
                 Utils::filter(
                     $maybeArray,
-                    static function ($x) {
+                    static function($x) {
                         return (bool) $x;
                     }
                 )

src/Utils/BreakingChangesFinder.php

Spacing +32 added lines, -32 removed lines

@@ -120,7 +120,7 @@ 
 
         $breakingChanges = [];
         foreach ($schemaATypeMap as $typeName => $schemaAType) {
-            if (! isset($schemaBTypeMap[$typeName])) {
+            if (!isset($schemaBTypeMap[$typeName])) {
                 continue;
             }
             $schemaBType = $schemaBTypeMap[$typeName];
@@ -190,9 +190,9 @@ 
         $breakingChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
-                ! ($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
-                ! ($newType instanceof $oldType)
+            if (!($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
+                !($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
+                !($newType instanceof $oldType)
             ) {
                 continue;
             }
@@ -201,7 +201,7 @@ 
             $newTypeFieldsDef = $newType->getFields();
             foreach ($oldTypeFieldsDef as $fieldName => $fieldDefinition) {
                 // Check if the field is missing on the type in the new schema.
-                if (! isset($newTypeFieldsDef[$fieldName])) {
+                if (!isset($newTypeFieldsDef[$fieldName])) {
                     $breakingChanges[] = [
                         'type'        => self::BREAKING_CHANGE_FIELD_REMOVED,
                         'description' => "${typeName}.${fieldName} was removed.",
@@ -213,7 +213,7 @@ 
                         $oldFieldType,
                         $newFieldType
                     );
-                    if (! $isSafe) {
+                    if (!$isSafe) {
                         $oldFieldTypeString = $oldFieldType instanceof NamedType
                             ? $oldFieldType->name
                             : $oldFieldType;
@@ -283,14 +283,14 @@ 
         $dangerousChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof InputObjectType) || ! ($newType instanceof InputObjectType)) {
+            if (!($oldType instanceof InputObjectType) || !($newType instanceof InputObjectType)) {
                 continue;
             }
 
             $oldTypeFieldsDef = $oldType->getFields();
             $newTypeFieldsDef = $newType->getFields();
             foreach (array_keys($oldTypeFieldsDef) as $fieldName) {
-                if (! isset($newTypeFieldsDef[$fieldName])) {
+                if (!isset($newTypeFieldsDef[$fieldName])) {
                     $breakingChanges[] = [
                         'type'        => self::BREAKING_CHANGE_FIELD_REMOVED,
                         'description' => "${typeName}.${fieldName} was removed.",
@@ -303,7 +303,7 @@ 
                         $oldFieldType,
                         $newFieldType
                     );
-                    if (! $isSafe) {
+                    if (!$isSafe) {
                         $oldFieldTypeString = $oldFieldType instanceof NamedType
                             ? $oldFieldType->name
                             : $oldFieldType;
@@ -374,7 +374,7 @@ 
                         $newType->getWrappedType()
                     )) ||
                 // moving from non-null to nullable of the same underlying type is safe
-                ! ($newType instanceof NonNull) &&
+                !($newType instanceof NonNull) &&
                 self::isChangeSafeForInputObjectFieldOrFieldArg($oldType->getWrappedType(), $newType);
         }
 
@@ -397,7 +397,7 @@ 
         $typesRemovedFromUnion = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof UnionType) || ! ($newType instanceof UnionType)) {
+            if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) {
                 continue;
             }
             $typeNamesInNewUnion = [];
@@ -435,7 +435,7 @@ 
         $valuesRemovedFromEnums = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof EnumType) || ! ($newType instanceof EnumType)) {
+            if (!($oldType instanceof EnumType) || !($newType instanceof EnumType)) {
                 continue;
             }
             $valuesInNewEnum = [];
@@ -477,9 +477,9 @@ 
 
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
-                ! ($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
-                ! ($newType instanceof $oldType)
+            if (!($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
+                !($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
+                !($newType instanceof $oldType)
             ) {
                 continue;
             }
@@ -488,7 +488,7 @@ 
             $newTypeFields = $newType->getFields();
 
             foreach ($oldTypeFields as $fieldName => $oldField) {
-                if (! isset($newTypeFields[$fieldName])) {
+                if (!isset($newTypeFields[$fieldName])) {
                     continue;
                 }
 
@@ -496,18 +496,18 @@ 
                     $newArgs   = $newTypeFields[$fieldName]->args;
                     $newArgDef = Utils::find(
                         $newArgs,
-                        static function ($arg) use ($oldArgDef) {
+                        static function($arg) use ($oldArgDef) {
                             return $arg->name === $oldArgDef->name;
                         }
                     );
                     if ($newArgDef !== null) {
-                        $isSafe     = self::isChangeSafeForInputObjectFieldOrFieldArg(
+                        $isSafe = self::isChangeSafeForInputObjectFieldOrFieldArg(
                             $oldArgDef->getType(),
                             $newArgDef->getType()
                         );
                         $oldArgType = $oldArgDef->getType();
                         $oldArgName = $oldArgDef->name;
-                        if (! $isSafe) {
+                        if (!$isSafe) {
                             $newArgType        = $newArgDef->getType();
                             $breakingChanges[] = [
                                 'type'        => self::BREAKING_CHANGE_ARG_CHANGED_KIND,
@@ -535,7 +535,7 @@ 
                         $oldArgs   = $oldTypeFields[$fieldName]->args;
                         $oldArgDef = Utils::find(
                             $oldArgs,
-                            static function ($arg) use ($newTypeFieldArgDef) {
+                            static function($arg) use ($newTypeFieldArgDef) {
                                 return $arg->name === $newTypeFieldArgDef->name;
                             }
                         );
@@ -581,7 +581,7 @@ 
 
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType) || ! ($newType instanceof ObjectType)) {
+            if (!($oldType instanceof ObjectType) || !($newType instanceof ObjectType)) {
                 continue;
             }
 
@@ -590,7 +590,7 @@ 
             foreach ($oldInterfaces as $oldInterface) {
                 $interface = Utils::find(
                     $newInterfaces,
-                    static function (InterfaceType $interface) use ($oldInterface) : bool {
+                    static function(InterfaceType $interface) use ($oldInterface) : bool {
                         return $interface->name === $oldInterface->name;
                     }
                 );
@@ -634,7 +634,7 @@ 
     {
         return Utils::keyMap(
             $schema->getDirectives(),
-            static function ($dir) {
+            static function($dir) {
                 return $dir->name;
             }
         );
@@ -646,7 +646,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -683,7 +683,7 @@ 
     {
         return Utils::keyMap(
             $directive->args ?: [],
-            static function ($arg) {
+            static function($arg) {
                 return $arg->name;
             }
         );
@@ -695,7 +695,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -703,7 +703,7 @@ 
                 $oldSchemaDirectiveMap[$newDirective->name],
                 $newDirective
             ) as $arg) {
-                if (! $arg->getType() instanceof NonNull) {
+                if (!$arg->getType() instanceof NonNull) {
                     continue;
                 }
                 $addedNonNullableArgs[] = [
@@ -747,7 +747,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -813,7 +813,7 @@ 
         $valuesAddedToEnums = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof EnumType) || ! ($newType instanceof EnumType)) {
+            if (!($oldType instanceof EnumType) || !($newType instanceof EnumType)) {
                 continue;
             }
             $valuesInOldEnum = [];
@@ -848,7 +848,7 @@ 
 
         foreach ($newTypeMap as $typeName => $newType) {
             $oldType = $oldTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType) || ! ($newType instanceof ObjectType)) {
+            if (!($oldType instanceof ObjectType) || !($newType instanceof ObjectType)) {
                 continue;
             }
 
@@ -857,7 +857,7 @@ 
             foreach ($newInterfaces as $newInterface) {
                 $interface = Utils::find(
                     $oldInterfaces,
-                    static function (InterfaceType $interface) use ($newInterface) : bool {
+                    static function(InterfaceType $interface) use ($newInterface) : bool {
                         return $interface->name === $newInterface->name;
                     }
                 );
@@ -896,7 +896,7 @@ 
         $typesAddedToUnion = [];
         foreach ($newTypeMap as $typeName => $newType) {
             $oldType = $oldTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof UnionType) || ! ($newType instanceof UnionType)) {
+            if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) {
                 continue;
             }
 

src/Utils/TypeInfo.php

Spacing +14 added lines, -14 removed lines

@@ -81,7 +81,7 @@ 
         $this->parentTypeStack = [];
         $this->inputTypeStack  = [];
         $this->fieldDefStack   = [];
-        if (! $initialType) {
+        if (!$initialType) {
             return;
         }
 
@@ -91,7 +91,7 @@ 
         if (Type::isCompositeType($initialType)) {
             $this->parentTypeStack[] = $initialType;
         }
-        if (! Type::isOutputType($initialType)) {
+        if (!Type::isOutputType($initialType)) {
             return;
         }
 
@@ -140,17 +140,17 @@ 
      */
     public static function extractTypes($type, ?array $typeMap = null)
     {
-        if (! $typeMap) {
+        if (!$typeMap) {
             $typeMap = [];
         }
-        if (! $type) {
+        if (!$type) {
             return $typeMap;
         }
 
         if ($type instanceof WrappingType) {
             return self::extractTypes($type->getWrappedType(true), $typeMap);
         }
-        if (! $type instanceof Type) {
+        if (!$type instanceof Type) {
             // Preserve these invalid types in map (at numeric index) to make them
             // detectable during $schema->validate()
             $i            = 0;
@@ -159,14 +159,14 @@ 
                 $alreadyInMap = $alreadyInMap || $typeMap[$i] === $type;
                 $i++;
             }
-            if (! $alreadyInMap) {
+            if (!$alreadyInMap) {
                 $typeMap[$i] = $type;
             }
 
             return $typeMap;
         }
 
-        if (! empty($typeMap[$type->name])) {
+        if (!empty($typeMap[$type->name])) {
             Utils::invariant(
                 $typeMap[$type->name] === $type,
                 sprintf('Schema must contain unique named types but contains multiple types named "%s" ', $type) .
@@ -187,9 +187,9 @@ 
         }
         if ($type instanceof ObjectType || $type instanceof InterfaceType) {
             foreach ($type->getFields() as $fieldName => $field) {
-                if (! empty($field->args)) {
+                if (!empty($field->args)) {
                     $fieldArgTypes = array_map(
-                        static function (FieldArgument $arg) {
+                        static function(FieldArgument $arg) {
                             return $arg->getType();
                         },
                         $field->args
@@ -320,7 +320,7 @@ 
                 if ($fieldOrDirective) {
                     $argDef = Utils::find(
                         $fieldOrDirective->args,
-                        static function ($arg) use ($node) {
+                        static function($arg) use ($node) {
                             return $arg->name === $node->name->value;
                         }
                     );
@@ -368,7 +368,7 @@ 
      */
     public function getType()
     {
-        if (! empty($this->typeStack)) {
+        if (!empty($this->typeStack)) {
             return $this->typeStack[count($this->typeStack) - 1];
         }
 
@@ -380,7 +380,7 @@ 
      */
     public function getParentType()
     {
-        if (! empty($this->parentTypeStack)) {
+        if (!empty($this->parentTypeStack)) {
             return $this->parentTypeStack[count($this->parentTypeStack) - 1];
         }
 
@@ -445,7 +445,7 @@ 
      */
     public function getFieldDef()
     {
-        if (! empty($this->fieldDefStack)) {
+        if (!empty($this->fieldDefStack)) {
             return $this->fieldDefStack[count($this->fieldDefStack) - 1];
         }
 
@@ -457,7 +457,7 @@ 
      */
     public function getInputType()
     {
-        if (! empty($this->inputTypeStack)) {
+        if (!empty($this->inputTypeStack)) {
             return $this->inputTypeStack[count($this->inputTypeStack) - 1];
         }
 

src/Utils/ASTDefinitionBuilder.php

Spacing +17 added lines, -17 removed lines

@@ -83,7 +83,7 @@ 
             'description' => $this->getDescription($directiveNode),
             'locations'   => Utils::map(
                 $directiveNode->locations,
-                static function ($node) {
+                static function($node) {
                     return $node->value;
                 }
             ),
@@ -113,7 +113,7 @@ 
     private function getLeadingCommentBlock($node)
     {
         $loc = $node->loc;
-        if (! $loc || ! $loc->startToken) {
+        if (!$loc || !$loc->startToken) {
             return null;
         }
         $comments = [];
@@ -136,10 +136,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.
@@ -198,7 +198,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) {
@@ -216,7 +216,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',
@@ -271,10 +271,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,
@@ -286,10 +286,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);
                 }
             )
@@ -333,7 +333,7 @@ 
             // validation with validateSchema() will produce more actionable results.
             return Utils::map(
                 $def->interfaces,
-                function ($iface) {
+                function($iface) {
                     return $this->buildType($iface);
                 }
             );
@@ -349,7 +349,7 @@ 
         return new InterfaceType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
             'astNode'     => $def,
@@ -364,10 +364,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),
@@ -391,7 +391,7 @@ 
             'types'       => $def->types
                 ? Utils::map(
                     $def->types,
-                    function ($typeNode) {
+                    function($typeNode) {
                         return $this->buildType($typeNode);
                     }
                 ) :
@@ -406,7 +406,7 @@ 
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
             'astNode'     => $def,
-            'serialize'   => static function ($value) {
+            'serialize'   => static function($value) {
                 return $value;
             },
         ]);
@@ -417,7 +417,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)
                     : [];