simPod / graphql-php

src/Type/Definition/QueryPlan.php

Spacing +6 added lines, -6 removed lines

@@ -79,7 +79,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;
     }
@@ -94,7 +94,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;
     }
@@ -104,7 +104,7 @@ 
      */
     public function subFields(string $typename) : array
     {
-        if (! array_key_exists($typename, $this->types)) {
+        if (!array_key_exists($typename, $this->types)) {
             return [];
         }
 
@@ -120,7 +120,7 @@ 
         $implementors = [];
         /** @var FieldNode $fieldNode */
         foreach ($fieldNodes as $fieldNode) {
-            if (! $fieldNode->selectionSet) {
+            if (!$fieldNode->selectionSet) {
                 continue;
             }
 
@@ -228,7 +228,7 @@ 
      */
     private function mergeFields(Type $parentType, Type $type, array $fields, array $subfields, array &$implementors) : array
     {
-        if ($this->groupImplementorFields && $parentType instanceof AbstractType && ! $type instanceof AbstractType) {
+        if ($this->groupImplementorFields && $parentType instanceof AbstractType && !$type instanceof AbstractType) {
             $implementors[$type->name] = [
                 'type'   => $type,
                 'fields' => $this->arrayMergeDeep(
@@ -268,7 +268,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/Type/Definition/IDType.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
             || is_int($value)
             || (is_object($value) && method_exists($value, '__toString'));
 
-        if (! $canCast) {
+        if (!$canCast) {
             throw new Error('ID cannot represent value: ' . Utils::printSafe($value));
         }
 

src/Type/Definition/InputObjectType.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -73,7 +73,7 @@ 
                 ? call_user_func($fields)
                 : $fields;
 
-            if (! is_array($fields)) {
+            if (!is_array($fields)) {
                 throw new InvariantViolation(
                     sprintf('%s fields must be an array or a callable which returns such an array.', $this->name)
                 );
@@ -102,7 +102,7 @@ 
         parent::assertValid();
 
         Utils::invariant(
-            ! empty($this->getFields()),
+            !empty($this->getFields()),
             sprintf(
                 '%s fields must be an associative array with field names as keys or a callable which returns such an array.',
                 $this->name

src/Type/Definition/BooleanType.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      */
     public function parseLiteral(Node $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/Introspection.php

Spacing +55 added lines, -55 removed lines

@@ -61,7 +61,7 @@ 
             );
             $descriptions = $options;
         } else {
-            $descriptions = ! array_key_exists('descriptions', $options) || $options['descriptions'] === true;
+            $descriptions = !array_key_exists('descriptions', $options) || $options['descriptions'] === true;
         }
         $descriptionField = $descriptions ? 'description' : '';
 
@@ -186,7 +186,7 @@ 
 
     public static function _schema()
     {
-        if (! isset(self::$map['__Schema'])) {
+        if (!isset(self::$map['__Schema'])) {
             self::$map['__Schema'] = new ObjectType([
                 'name'            => '__Schema',
                 'isIntrospection' => true,
@@ -199,14 +199,14 @@ 
                     'types'            => [
                         'description' => 'A list of all types supported by this server.',
                         'type'        => new NonNull(new ListOfType(new NonNull(self::_type()))),
-                        'resolve'     => static function (Schema $schema) {
+                        'resolve'     => static function(Schema $schema) {
                             return array_values($schema->getTypeMap());
                         },
                     ],
                     'queryType'        => [
                         'description' => 'The type that query operations will be rooted at.',
                         'type'        => new NonNull(self::_type()),
-                        'resolve'     => static function (Schema $schema) {
+                        'resolve'     => static function(Schema $schema) {
                             return $schema->getQueryType();
                         },
                     ],
@@ -215,21 +215,21 @@ 
                             'If this server supports mutation, the type that ' .
                             'mutation operations will be rooted at.',
                         'type'        => self::_type(),
-                        'resolve'     => static function (Schema $schema) {
+                        'resolve'     => static function(Schema $schema) {
                             return $schema->getMutationType();
                         },
                     ],
                     'subscriptionType' => [
                         'description' => 'If this server support subscription, the type that subscription operations will be rooted at.',
                         'type'        => self::_type(),
-                        'resolve'     => static function (Schema $schema) {
+                        'resolve'     => static function(Schema $schema) {
                             return $schema->getSubscriptionType();
                         },
                     ],
                     'directives'       => [
                         'description' => 'A list of all directives supported by this server.',
                         'type'        => Type::nonNull(Type::listOf(Type::nonNull(self::_directive()))),
-                        'resolve'     => static function (Schema $schema) {
+                        'resolve'     => static function(Schema $schema) {
                             return $schema->getDirectives();
                         },
                     ],
@@ -242,7 +242,7 @@ 
 
     public static function _type()
     {
-        if (! isset(self::$map['__Type'])) {
+        if (!isset(self::$map['__Type'])) {
             self::$map['__Type'] = new ObjectType([
                 'name'            => '__Type',
                 'isIntrospection' => true,
@@ -256,11 +256,11 @@ 
                     'Object and Interface types provide the fields they describe. Abstract ' .
                     'types, Union and Interface, provide the Object types possible ' .
                     'at runtime. List and NonNull types compose other types.',
-                'fields'          => static function () {
+                'fields'          => static function() {
                     return [
                         'kind'          => [
                             'type'    => Type::nonNull(self::_typeKind()),
-                            'resolve' => static function (Type $type) {
+                            'resolve' => static function(Type $type) {
                                 switch (true) {
                                     case $type instanceof ListOfType:
                                         return TypeKind::LIST_KIND;
@@ -285,13 +285,13 @@ 
                         ],
                         'name'          => [
                             'type' => Type::string(),
-                            'resolve' => static function ($obj) {
+                            'resolve' => static function($obj) {
                                 return $obj->name;
                             },
                         ],
                         'description'   => [
                             'type' => Type::string(),
-                            'resolve' => static function ($obj) {
+                            'resolve' => static function($obj) {
                                 return $obj->description;
                             },
                         ],
@@ -300,15 +300,15 @@ 
                             'args'    => [
                                 'includeDeprecated' => ['type' => Type::boolean(), 'defaultValue' => false],
                             ],
-                            'resolve' => static function (Type $type, $args) {
+                            'resolve' => static function(Type $type, $args) {
                                 if ($type instanceof ObjectType || $type instanceof InterfaceType) {
                                     $fields = $type->getFields();
 
                                     if (empty($args['includeDeprecated'])) {
                                         $fields = array_filter(
                                             $fields,
-                                            static function (FieldDefinition $field) {
-                                                return ! $field->deprecationReason;
+                                            static function(FieldDefinition $field) {
+                                                return !$field->deprecationReason;
                                             }
                                         );
                                     }
@@ -321,7 +321,7 @@ 
                         ],
                         'interfaces'    => [
                             'type'    => Type::listOf(Type::nonNull(self::_type())),
-                            'resolve' => static function ($type) {
+                            'resolve' => static function($type) {
                                 if ($type instanceof ObjectType) {
                                     return $type->getInterfaces();
                                 }
@@ -331,7 +331,7 @@ 
                         ],
                         'possibleTypes' => [
                             'type'    => Type::listOf(Type::nonNull(self::_type())),
-                            'resolve' => static function ($type, $args, $context, ResolveInfo $info) {
+                            'resolve' => static function($type, $args, $context, ResolveInfo $info) {
                                 if ($type instanceof InterfaceType || $type instanceof UnionType) {
                                     return $info->schema->getPossibleTypes($type);
                                 }
@@ -344,15 +344,15 @@ 
                             'args'    => [
                                 'includeDeprecated' => ['type' => Type::boolean(), 'defaultValue' => false],
                             ],
-                            'resolve' => static function ($type, $args) {
+                            'resolve' => static function($type, $args) {
                                 if ($type instanceof EnumType) {
                                     $values = array_values($type->getValues());
 
                                     if (empty($args['includeDeprecated'])) {
                                         $values = array_filter(
                                             $values,
-                                            static function ($value) {
-                                                return ! $value->deprecationReason;
+                                            static function($value) {
+                                                return !$value->deprecationReason;
                                             }
                                         );
                                     }
@@ -365,7 +365,7 @@ 
                         ],
                         'inputFields'   => [
                             'type'    => Type::listOf(Type::nonNull(self::_inputValue())),
-                            'resolve' => static function ($type) {
+                            'resolve' => static function($type) {
                                 if ($type instanceof InputObjectType) {
                                     return array_values($type->getFields());
                                 }
@@ -375,7 +375,7 @@ 
                         ],
                         'ofType'        => [
                             'type'    => self::_type(),
-                            'resolve' => static function ($type) {
+                            'resolve' => static function($type) {
                                 if ($type instanceof WrappingType) {
                                     return $type->getWrappedType();
                                 }
@@ -393,7 +393,7 @@ 
 
     public static function _typeKind()
     {
-        if (! isset(self::$map['__TypeKind'])) {
+        if (!isset(self::$map['__TypeKind'])) {
             self::$map['__TypeKind'] = new EnumType([
                 'name'            => '__TypeKind',
                 'isIntrospection' => true,
@@ -440,48 +440,48 @@ 
 
     public static function _field()
     {
-        if (! isset(self::$map['__Field'])) {
+        if (!isset(self::$map['__Field'])) {
             self::$map['__Field'] = new ObjectType([
                 'name'            => '__Field',
                 'isIntrospection' => true,
                 'description'     =>
                     'Object and Interface types are described by a list of Fields, each of ' .
                     'which has a name, potentially a list of arguments, and a return type.',
-                'fields'          => static function () {
+                'fields'          => static function() {
                     return [
                         'name'              => [
                             'type' => Type::nonNull(Type::string()),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return $field->name;
                             },
                         ],
                         'description'       => [
                             'type' => Type::string(),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return $field->description;
                             },
                         ],
                         'args'              => [
                             'type'    => Type::nonNull(Type::listOf(Type::nonNull(self::_inputValue()))),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return empty($field->args) ? [] : $field->args;
                             },
                         ],
                         'type'              => [
                             'type'    => Type::nonNull(self::_type()),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return $field->getType();
                             },
                         ],
                         'isDeprecated'      => [
                             'type'    => Type::nonNull(Type::boolean()),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return (bool) $field->deprecationReason;
                             },
                         ],
                         'deprecationReason' => [
                             'type'    => Type::string(),
-                            'resolve' => static function (FieldDefinition $field) {
+                            'resolve' => static function(FieldDefinition $field) {
                                 return $field->deprecationReason;
                             },
                         ],
@@ -495,7 +495,7 @@ 
 
     public static function _inputValue()
     {
-        if (! isset(self::$map['__InputValue'])) {
+        if (!isset(self::$map['__InputValue'])) {
             self::$map['__InputValue'] = new ObjectType([
                 'name'            => '__InputValue',
                 'isIntrospection' => true,
@@ -503,11 +503,11 @@ 
                     'Arguments provided to Fields or Directives and the input fields of an ' .
                     'InputObject are represented as Input Values which describe their type ' .
                     'and optionally a default value.',
-                'fields'          => static function () {
+                'fields'          => static function() {
                     return [
                         'name'         => [
                             'type' => Type::nonNull(Type::string()),
-                            'resolve' => static function ($inputValue) {
+                            'resolve' => static function($inputValue) {
                                 /** @var FieldArgument|InputObjectField $inputValue */
                                 $inputValue = $inputValue;
 
@@ -516,7 +516,7 @@ 
                         ],
                         'description'  => [
                             'type' => Type::string(),
-                            'resolve' => static function ($inputValue) {
+                            'resolve' => static function($inputValue) {
                                 /** @var FieldArgument|InputObjectField $inputValue */
                                 $inputValue = $inputValue;
 
@@ -525,7 +525,7 @@ 
                         ],
                         'type'         => [
                             'type'    => Type::nonNull(self::_type()),
-                            'resolve' => static function ($value) {
+                            'resolve' => static function($value) {
                                 return method_exists($value, 'getType')
                                     ? $value->getType()
                                     : $value->type;
@@ -535,11 +535,11 @@ 
                             'type'        => Type::string(),
                             'description' =>
                                 'A GraphQL-formatted string representing the default value for this input value.',
-                            'resolve'     => static function ($inputValue) {
+                            'resolve'     => static function($inputValue) {
                                 /** @var FieldArgument|InputObjectField $inputValue */
                                 $inputValue = $inputValue;
 
-                                return ! $inputValue->defaultValueExists()
+                                return !$inputValue->defaultValueExists()
                                     ? null
                                     : Printer::doPrint(AST::astFromValue(
                                         $inputValue->defaultValue,
@@ -557,7 +557,7 @@ 
 
     public static function _enumValue()
     {
-        if (! isset(self::$map['__EnumValue'])) {
+        if (!isset(self::$map['__EnumValue'])) {
             self::$map['__EnumValue'] = new ObjectType([
                 'name'            => '__EnumValue',
                 'isIntrospection' => true,
@@ -568,25 +568,25 @@ 
                 'fields'          => [
                     'name'              => [
                         'type' => Type::nonNull(Type::string()),
-                        'resolve' => static function ($enumValue) {
+                        'resolve' => static function($enumValue) {
                             return $enumValue->name;
                         },
                     ],
                     'description'       => [
                         'type' => Type::string(),
-                        'resolve' => static function ($enumValue) {
+                        'resolve' => static function($enumValue) {
                             return $enumValue->description;
                         },
                     ],
                     'isDeprecated'      => [
                         'type'    => Type::nonNull(Type::boolean()),
-                        'resolve' => static function ($enumValue) {
+                        'resolve' => static function($enumValue) {
                             return (bool) $enumValue->deprecationReason;
                         },
                     ],
                     'deprecationReason' => [
                         'type' => Type::string(),
-                        'resolve' => static function ($enumValue) {
+                        'resolve' => static function($enumValue) {
                             return $enumValue->deprecationReason;
                         },
                     ],
@@ -599,7 +599,7 @@ 
 
     public static function _directive()
     {
-        if (! isset(self::$map['__Directive'])) {
+        if (!isset(self::$map['__Directive'])) {
             self::$map['__Directive'] = new ObjectType([
                 'name'            => '__Directive',
                 'isIntrospection' => true,
@@ -612,13 +612,13 @@ 
                 'fields'          => [
                     'name'        => [
                         'type'    => Type::nonNull(Type::string()),
-                        'resolve' => static function ($obj) {
+                        'resolve' => static function($obj) {
                             return $obj->name;
                         },
                     ],
                     'description' => [
                         'type' => Type::string(),
-                        'resolve' => static function ($obj) {
+                        'resolve' => static function($obj) {
                             return $obj->description;
                         },
                     ],
@@ -626,13 +626,13 @@ 
                         'type' => Type::nonNull(Type::listOf(Type::nonNull(
                             self::_directiveLocation()
                         ))),
-                        'resolve' => static function ($obj) {
+                        'resolve' => static function($obj) {
                             return $obj->locations;
                         },
                     ],
                     'args'        => [
                         'type'    => Type::nonNull(Type::listOf(Type::nonNull(self::_inputValue()))),
-                        'resolve' => static function (Directive $directive) {
+                        'resolve' => static function(Directive $directive) {
                             return $directive->args ?: [];
                         },
                     ],
@@ -645,7 +645,7 @@ 
 
     public static function _directiveLocation()
     {
-        if (! isset(self::$map['__DirectiveLocation'])) {
+        if (!isset(self::$map['__DirectiveLocation'])) {
             self::$map['__DirectiveLocation'] = new EnumType([
                 'name'            => '__DirectiveLocation',
                 'isIntrospection' => true,
@@ -739,13 +739,13 @@ 
 
     public static function schemaMetaFieldDef() : FieldDefinition
     {
-        if (! isset(self::$map[self::SCHEMA_FIELD_NAME])) {
+        if (!isset(self::$map[self::SCHEMA_FIELD_NAME])) {
             self::$map[self::SCHEMA_FIELD_NAME] = FieldDefinition::create([
                 'name'        => self::SCHEMA_FIELD_NAME,
                 'type'        => Type::nonNull(self::_schema()),
                 'description' => 'Access the current type schema of this server.',
                 'args'        => [],
-                'resolve'     => static function (
+                'resolve'     => static function(
                     $source,
                     $args,
                     $context,
@@ -761,7 +761,7 @@ 
 
     public static function typeMetaFieldDef() : FieldDefinition
     {
-        if (! isset(self::$map[self::TYPE_FIELD_NAME])) {
+        if (!isset(self::$map[self::TYPE_FIELD_NAME])) {
             self::$map[self::TYPE_FIELD_NAME] = FieldDefinition::create([
                 'name'        => self::TYPE_FIELD_NAME,
                 'type'        => self::_type(),
@@ -769,7 +769,7 @@ 
                 'args'        => [
                     ['name' => 'name', 'type' => Type::nonNull(Type::string())],
                 ],
-                'resolve'     => static function ($source, $args, $context, ResolveInfo $info) {
+                'resolve'     => static function($source, $args, $context, ResolveInfo $info) {
                     return $info->schema->getType($args['name']);
                 },
             ]);
@@ -780,13 +780,13 @@ 
 
     public static function typeNameMetaFieldDef() : FieldDefinition
     {
-        if (! isset(self::$map[self::TYPE_NAME_FIELD_NAME])) {
+        if (!isset(self::$map[self::TYPE_NAME_FIELD_NAME])) {
             self::$map[self::TYPE_NAME_FIELD_NAME] = FieldDefinition::create([
                 'name'        => self::TYPE_NAME_FIELD_NAME,
                 'type'        => Type::nonNull(Type::string()),
                 'description' => 'The name of the current Object type at runtime.',
                 'args'        => [],
-                'resolve'     => static function (
+                'resolve'     => static function(
                     $source,
                     $args,
                     $context,

src/Type/SchemaValidationContext.php

Spacing +40 added lines, -40 removed lines

@@ -74,12 +74,12 @@ 
     public function validateRootTypes()
     {
         $queryType = $this->schema->getQueryType();
-        if (! $queryType) {
+        if (!$queryType) {
             $this->reportError(
                 'Query root type must be provided.',
                 $this->schema->getAstNode()
             );
-        } elseif (! $queryType instanceof ObjectType) {
+        } elseif (!$queryType instanceof ObjectType) {
             $this->reportError(
                 'Query root type must be Object type, it cannot be ' . Utils::printSafe($queryType) . '.',
                 $this->getOperationTypeNode($queryType, 'query')
@@ -87,7 +87,7 @@ 
         }
 
         $mutationType = $this->schema->getMutationType();
-        if ($mutationType && ! $mutationType instanceof ObjectType) {
+        if ($mutationType && !$mutationType instanceof ObjectType) {
             $this->reportError(
                 'Mutation root type must be Object type if provided, it cannot be ' . Utils::printSafe($mutationType) . '.',
                 $this->getOperationTypeNode($mutationType, 'mutation')
@@ -95,7 +95,7 @@ 
         }
 
         $subscriptionType = $this->schema->getSubscriptionType();
-        if (! $subscriptionType || $subscriptionType instanceof ObjectType) {
+        if (!$subscriptionType || $subscriptionType instanceof ObjectType) {
             return;
         }
 
@@ -166,7 +166,7 @@ 
         $directives = $this->schema->getDirectives();
         foreach ($directives as $directive) {
             // Ensure all directives are in fact GraphQL directives.
-            if (! $directive instanceof Directive) {
+            if (!$directive instanceof Directive) {
                 $this->reportError(
                     'Expected directive but got: ' . Utils::printSafe($directive) . '.',
                     is_object($directive) ? $directive->astNode : null
@@ -222,7 +222,7 @@ 
 
             $nodes = Utils::map(
                 $directiveList,
-                static function (Directive $directive) {
+                static function(Directive $directive) {
                     return $directive->astNode;
                 }
             );
@@ -240,7 +240,7 @@ 
     {
         // Ensure names are valid, however introspection types opt out.
         $error = Utils::isValidNameError($node->name, $node->astNode);
-        if (! $error || Introspection::isIntrospectionType($node)) {
+        if (!$error || Introspection::isIntrospectionType($node)) {
             return;
         }
 
@@ -256,14 +256,14 @@ 
     {
         $subNodes = $this->getAllSubNodes(
             $directive,
-            static function ($directiveNode) {
+            static function($directiveNode) {
                 return $directiveNode->arguments;
             }
         );
 
         return Utils::filter(
             $subNodes,
-            static function ($argNode) use ($argName) {
+            static function($argNode) use ($argName) {
                 return $argNode->name->value === $argName;
             }
         );
@@ -286,7 +286,7 @@ 
         $typeMap = $this->schema->getTypeMap();
         foreach ($typeMap as $typeName => $type) {
             // Ensure all provided types are in fact GraphQL type.
-            if (! $type instanceof NamedType) {
+            if (!$type instanceof NamedType) {
                 $this->reportError(
                     'Expected GraphQL named type but got: ' . Utils::printSafe($type) . '.',
                     $type instanceof Type ? $type->astNode : null
@@ -378,11 +378,11 @@ 
             }
             $includes = Utils::some(
                 $schemaDirective->locations,
-                static function ($schemaLocation) use ($location) {
+                static function($schemaLocation) use ($location) {
                     return $schemaLocation === $location;
                 }
             );
-            if (! $includes) {
+            if (!$includes) {
                 $errorNodes = $schemaDirective->astNode
                     ? [$directive, $schemaDirective->astNode]
                     : [$directive];
@@ -416,7 +416,7 @@ 
         $fieldMap = $type->getFields();
 
         // Objects and Interfaces both must define one or more fields.
-        if (! $fieldMap) {
+        if (!$fieldMap) {
             $this->reportError(
                 sprintf('Type %s must define one or more fields.', $type->name),
                 $this->getAllNodes($type)
@@ -438,7 +438,7 @@ 
             }
 
             // Ensure the type is an output type
-            if (! Type::isOutputType($field->getType())) {
+            if (!Type::isOutputType($field->getType())) {
                 $this->reportError(
                     sprintf(
                         'The type of %s.%s must be Output Type but got: %s.',
@@ -472,7 +472,7 @@ 
                 $argNames[$argName] = true;
 
                 // Ensure the type is an input type
-                if (! Type::isInputType($arg->getType())) {
+                if (!Type::isInputType($arg->getType())) {
                     $this->reportError(
                         sprintf(
                             'The type of %s.%s(%s:) must be Input Type but got: %s.',
@@ -486,7 +486,7 @@ 
                 }
 
                 // Ensure argument definition directives are valid
-                if (! isset($arg->astNode, $arg->astNode->directives)) {
+                if (!isset($arg->astNode, $arg->astNode->directives)) {
                     continue;
                 }
 
@@ -497,7 +497,7 @@ 
             }
 
             // Ensure any directives are valid
-            if (! isset($field->astNode, $field->astNode->directives)) {
+            if (!isset($field->astNode, $field->astNode->directives)) {
                 continue;
             }
 
@@ -539,12 +539,12 @@ 
     {
         $result = new NodeList([]);
         foreach ($this->getAllNodes($obj) as $astNode) {
-            if (! $astNode) {
+            if (!$astNode) {
                 continue;
             }
 
             $subNodes = $getter($astNode);
-            if (! $subNodes) {
+            if (!$subNodes) {
                 continue;
             }
 
@@ -562,11 +562,11 @@ 
      */
     private function getAllFieldNodes($type, $fieldName)
     {
-        $subNodes = $this->getAllSubNodes($type, static function ($typeNode) {
+        $subNodes = $this->getAllSubNodes($type, static function($typeNode) {
             return $typeNode->fields;
         });
 
-        return Utils::filter($subNodes, static function ($fieldNode) use ($fieldName) {
+        return Utils::filter($subNodes, static function($fieldNode) use ($fieldName) {
             return $fieldNode->name->value === $fieldName;
         });
     }
@@ -653,7 +653,7 @@ 
     {
         $implementedTypeNames = [];
         foreach ($object->getInterfaces() as $iface) {
-            if (! $iface instanceof InterfaceType) {
+            if (!$iface instanceof InterfaceType) {
                 $this->reportError(
                     sprintf(
                         'Type %s must only implement Interface types, it cannot implement %s.',
@@ -683,7 +683,7 @@ 
      */
     private function getDirectives($object)
     {
-        return $this->getAllSubNodes($object, static function ($node) {
+        return $this->getAllSubNodes($object, static function($node) {
             return $node->directives;
         });
     }
@@ -707,11 +707,11 @@ 
      */
     private function getAllImplementsInterfaceNodes(ObjectType $type, $iface)
     {
-        $subNodes = $this->getAllSubNodes($type, static function ($typeNode) {
+        $subNodes = $this->getAllSubNodes($type, static function($typeNode) {
             return $typeNode->interfaces;
         });
 
-        return Utils::filter($subNodes, static function ($ifaceNode) use ($iface) {
+        return Utils::filter($subNodes, static function($ifaceNode) use ($iface) {
             return $ifaceNode->name->value === $iface->name;
         });
     }
@@ -731,7 +731,7 @@ 
                 : null;
 
             // Assert interface field exists on object.
-            if (! $objectField) {
+            if (!$objectField) {
                 $this->reportError(
                     sprintf(
                         'Interface field %s.%s expected but %s does not provide it.',
@@ -749,7 +749,7 @@ 
 
             // Assert interface field type is satisfied by object field type, by being
             // a valid subtype. (covariant)
-            if (! TypeComparators::isTypeSubTypeOf(
+            if (!TypeComparators::isTypeSubTypeOf(
                 $this->schema,
                 $objectField->getType(),
                 $ifaceField->getType()
@@ -785,7 +785,7 @@ 
                 }
 
                 // Assert interface field arg exists on object field.
-                if (! $objectArg) {
+                if (!$objectArg) {
                     $this->reportError(
                         sprintf(
                             'Interface field argument %s.%s(%s:) expected but %s.%s does not provide it.',
@@ -806,7 +806,7 @@ 
                 // Assert interface field arg type matches object field arg type.
                 // (invariant)
                 // TODO: change to contravariant?
-                if (! TypeComparators::isEqualType($ifaceArg->getType(), $objectArg->getType())) {
+                if (!TypeComparators::isEqualType($ifaceArg->getType(), $objectArg->getType())) {
                     $this->reportError(
                         sprintf(
                             'Interface field argument %s.%s(%s:) expects type %s but %s.%s(%s:) is type %s.',
@@ -840,7 +840,7 @@ 
                     }
                 }
 
-                if ($ifaceArg || ! ($objectArg->getType() instanceof NonNull)) {
+                if ($ifaceArg || !($objectArg->getType() instanceof NonNull)) {
                     continue;
                 }
 
@@ -867,7 +867,7 @@ 
     {
         $memberTypes = $union->getTypes();
 
-        if (! $memberTypes) {
+        if (!$memberTypes) {
             $this->reportError(
                 sprintf('Union type %s must define one or more member types.', $union->name),
                 $this->getAllNodes($union)
@@ -907,11 +907,11 @@ 
      */
     private function getUnionMemberTypeNodes(UnionType $union, $typeName)
     {
-        $subNodes = $this->getAllSubNodes($union, static function ($unionNode) {
+        $subNodes = $this->getAllSubNodes($union, static function($unionNode) {
             return $unionNode->types;
         });
 
-        return Utils::filter($subNodes, static function ($typeNode) use ($typeName) {
+        return Utils::filter($subNodes, static function($typeNode) use ($typeName) {
             return $typeNode->name->value === $typeName;
         });
     }
@@ -920,7 +920,7 @@ 
     {
         $enumValues = $enumType->getValues();
 
-        if (! $enumValues) {
+        if (!$enumValues) {
             $this->reportError(
                 sprintf('Enum type %s must define one or more values.', $enumType->name),
                 $this->getAllNodes($enumType)
@@ -949,7 +949,7 @@ 
             }
 
             // Ensure valid directives
-            if (! isset($enumValue->astNode, $enumValue->astNode->directives)) {
+            if (!isset($enumValue->astNode, $enumValue->astNode->directives)) {
                 continue;
             }
 
@@ -967,11 +967,11 @@ 
      */
     private function getEnumValueNodes(EnumType $enum, $valueName)
     {
-        $subNodes = $this->getAllSubNodes($enum, static function ($enumNode) {
+        $subNodes = $this->getAllSubNodes($enum, static function($enumNode) {
             return $enumNode->values;
         });
 
-        return Utils::filter($subNodes, static function ($valueNode) use ($valueName) {
+        return Utils::filter($subNodes, static function($valueNode) use ($valueName) {
             return $valueNode->name->value === $valueName;
         });
     }
@@ -980,7 +980,7 @@ 
     {
         $fieldMap = $inputObj->getFields();
 
-        if (! $fieldMap) {
+        if (!$fieldMap) {
             $this->reportError(
                 sprintf('Input Object type %s must define one or more fields.', $inputObj->name),
                 $this->getAllNodes($inputObj)
@@ -995,7 +995,7 @@ 
             // TODO: Ensure they are unique per field.
 
             // Ensure the type is an input type
-            if (! Type::isInputType($field->getType())) {
+            if (!Type::isInputType($field->getType())) {
                 $this->reportError(
                     sprintf(
                         'The type of %s.%s must be Input Type but got: %s.',
@@ -1008,7 +1008,7 @@ 
             }
 
             // Ensure valid directives
-            if (! isset($field->astNode, $field->astNode->directives)) {
+            if (!isset($field->astNode, $field->astNode->directives)) {
                 continue;
             }
 

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 instanceof Type
                             ? $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) {
                         if ($oldFieldType instanceof NamedType) {
                             $oldFieldTypeString = $oldFieldType->name;
                         } else {
@@ -356,7 +356,7 @@ 
         Type $newType
     ) {
         if ($oldType instanceof NamedType) {
-            if (! $newType instanceof NamedType) {
+            if (!$newType instanceof NamedType) {
                 return false;
             }
 
@@ -382,7 +382,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);
         }
 
@@ -405,7 +405,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 = [];
@@ -443,7 +443,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 = [];
@@ -485,9 +485,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;
             }
@@ -496,7 +496,7 @@ 
             $newTypeFields = $newType->getFields();
 
             foreach ($oldTypeFields as $fieldName => $oldField) {
-                if (! isset($newTypeFields[$fieldName])) {
+                if (!isset($newTypeFields[$fieldName])) {
                     continue;
                 }
 
@@ -504,7 +504,7 @@ 
                     $newArgs   = $newTypeFields[$fieldName]->args;
                     $newArgDef = Utils::find(
                         $newArgs,
-                        static function ($arg) use ($oldArgDef) {
+                        static function($arg) use ($oldArgDef) {
                             return $arg->name === $oldArgDef->name;
                         }
                     );
@@ -516,7 +516,7 @@ 
                         /** @var ScalarType|EnumType|InputObjectType|ListOfType|NonNull $oldArgType */
                         $oldArgType = $oldArgDef->getType();
                         $oldArgName = $oldArgDef->name;
-                        if (! $isSafe) {
+                        if (!$isSafe) {
                             $newArgType        = $newArgDef->getType();
                             $breakingChanges[] = [
                                 'type'        => self::BREAKING_CHANGE_ARG_CHANGED_KIND,
@@ -544,7 +544,7 @@ 
                         $oldArgs   = $oldTypeFields[$fieldName]->args;
                         $oldArgDef = Utils::find(
                             $oldArgs,
-                            static function ($arg) use ($newTypeFieldArgDef) {
+                            static function($arg) use ($newTypeFieldArgDef) {
                                 return $arg->name === $newTypeFieldArgDef->name;
                             }
                         );
@@ -590,7 +590,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;
             }
 
@@ -599,7 +599,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;
                     }
                 );
@@ -643,7 +643,7 @@ 
     {
         return Utils::keyMap(
             $schema->getDirectives(),
-            static function ($dir) {
+            static function($dir) {
                 return $dir->name;
             }
         );
@@ -655,7 +655,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -692,7 +692,7 @@ 
     {
         return Utils::keyMap(
             $directive->args ?: [],
-            static function ($arg) {
+            static function($arg) {
                 return $arg->name;
             }
         );
@@ -704,7 +704,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -712,7 +712,7 @@ 
                 $oldSchemaDirectiveMap[$newDirective->name],
                 $newDirective
             ) as $arg) {
-                if (! $arg->getType() instanceof NonNull) {
+                if (!$arg->getType() instanceof NonNull) {
                     continue;
                 }
                 $addedNonNullableArgs[] = [
@@ -756,7 +756,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -822,7 +822,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 = [];
@@ -857,7 +857,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;
             }
 
@@ -866,7 +866,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;
                     }
                 );
@@ -905,7 +905,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/SchemaPrinter.php

Spacing +25 added lines, -25 removed lines

@@ -51,11 +51,11 @@ 
     {
         return self::printFilteredSchema(
             $schema,
-            static function ($type) {
-                return ! Directive::isSpecifiedDirective($type);
+            static function($type) {
+                return !Directive::isSpecifiedDirective($type);
             },
-            static function ($type) {
-                return ! Type::isBuiltInType($type);
+            static function($type) {
+                return !Type::isBuiltInType($type);
             },
             $options
         );
@@ -68,7 +68,7 @@ 
     {
         $directives = array_filter(
             $schema->getDirectives(),
-            static function ($directive) use ($directiveFilter) {
+            static function($directive) use ($directiveFilter) {
                 return $directiveFilter($directive);
             }
         );
@@ -85,13 +85,13 @@ 
                     array_merge(
                         [self::printSchemaDefinition($schema)],
                         array_map(
-                            static function ($directive) use ($options) {
+                            static function($directive) use ($options) {
                                 return self::printDirective($directive, $options);
                             },
                             $directives
                         ),
                         array_map(
-                            static function ($type) use ($options) {
+                            static function($type) use ($options) {
                                 return self::printType($type, $options);
                             },
                             $types
@@ -154,7 +154,7 @@ 
 
         $subscriptionType = $schema->getSubscriptionType();
 
-        return ! $subscriptionType || $subscriptionType->name === 'Subscription';
+        return !$subscriptionType || $subscriptionType->name === 'Subscription';
     }
 
     private static function printDirective($directive, $options) : string
@@ -166,7 +166,7 @@ 
 
     private static function printDescription($options, $def, $indentation = '', $firstInBlock = true) : string
     {
-        if (! $def->description) {
+        if (!$def->description) {
             return '';
         }
         $lines = self::descriptionLines($def->description, 120 - strlen($indentation));
@@ -174,7 +174,7 @@ 
             return self::printDescriptionWithComments($lines, $indentation, $firstInBlock);
         }
 
-        $description = $indentation && ! $firstInBlock
+        $description = $indentation && !$firstInBlock
             ? "\n" . $indentation . '"""'
             : $indentation . '"""';
 
@@ -192,13 +192,13 @@ 
                 substr($lines[0], 0, 1) === ' ' ||
                 substr($lines[0], 0, 1) === '\t'
             );
-        if (! $hasLeadingSpace) {
+        if (!$hasLeadingSpace) {
             $description .= "\n";
         }
 
         $lineLength = count($lines);
         for ($i = 0; $i < $lineLength; $i++) {
-            if ($i !== 0 || ! $hasLeadingSpace) {
+            if ($i !== 0 || !$hasLeadingSpace) {
                 $description .= $indentation;
             }
             $description .= self::escapeQuote($lines[$i]) . "\n";
@@ -247,7 +247,7 @@ 
 
     private static function printDescriptionWithComments($lines, $indentation, $firstInBlock) : string
     {
-        $description = $indentation && ! $firstInBlock ? "\n" : '';
+        $description = $indentation && !$firstInBlock ? "\n" : '';
         foreach ($lines as $line) {
             if ($line === '') {
                 $description .= $indentation . "#\n";
@@ -266,14 +266,14 @@ 
 
     private static function printArgs($options, $args, $indentation = '') : string
     {
-        if (! $args) {
+        if (!$args) {
             return '';
         }
 
         // If every arg does not have a description, print them on one line.
         if (Utils::every(
             $args,
-            static function ($arg) {
+            static function($arg) {
                 return empty($arg->description);
             }
         )) {
@@ -285,8 +285,8 @@ 
             implode(
                 "\n",
                 array_map(
-                    static function ($arg, $i) use ($indentation, $options) {
-                        return self::printDescription($options, $arg, '  ' . $indentation, ! $i) . '  ' . $indentation .
+                    static function($arg, $i) use ($indentation, $options) {
+                        return self::printDescription($options, $arg, '  ' . $indentation, !$i) . '  ' . $indentation .
                             self::printInputValue($arg);
                     },
                     $args,
@@ -353,11 +353,11 @@ 
     private static function printObject(ObjectType $type, array $options) : string
     {
         $interfaces            = $type->getInterfaces();
-        $implementedInterfaces = ! empty($interfaces)
+        $implementedInterfaces = !empty($interfaces)
             ? ' implements ' . implode(
                 ' & ',
                 array_map(
-                    static function ($i) {
+                    static function($i) {
                         return $i->name;
                     },
                     $interfaces
@@ -379,8 +379,8 @@ 
         return implode(
             "\n",
             array_map(
-                static function ($f, $i) use ($options) {
-                    return self::printDescription($options, $f, '  ', ! $i) . '  ' .
+                static function($f, $i) use ($options) {
+                    return self::printDescription($options, $f, '  ', !$i) . '  ' .
                         $f->name . self::printArgs($options, $f->args, '  ') . ': ' .
                         (string) $f->getType() . self::printDeprecated($f);
                 },
@@ -439,8 +439,8 @@ 
         return implode(
             "\n",
             array_map(
-                static function ($value, $i) use ($options) {
-                    return self::printDescription($options, $value, '  ', ! $i) . '  ' .
+                static function($value, $i) use ($options) {
+                    return self::printDescription($options, $value, '  ', !$i) . '  ' .
                         $value->name . self::printDeprecated($value);
                 },
                 $values,
@@ -463,8 +463,8 @@ 
                 implode(
                     "\n",
                     array_map(
-                        static function ($f, $i) use ($options) {
-                            return self::printDescription($options, $f, '  ', ! $i) . '  ' . self::printInputValue($f);
+                        static function($f, $i) use ($options) {
+                            return self::printDescription($options, $f, '  ', !$i) . '  ' . self::printInputValue($f);
                         },
                         $fields,
                         array_keys($fields)

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;
                 }
             }