simPod / graphql-php

src/Type/Definition/FloatType.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             );
         }
 
-        if (! is_numeric($value) && $value !== true && $value !== false) {
+        if (!is_numeric($value) && $value !== true && $value !== false) {
             throw new Error(
                 'Float cannot represent non numeric value: ' .
                 Utils::printSafe($value)

src/Type/SchemaConfig.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     {
         $config = new static();
 
-        if (! empty($options)) {
+        if (!empty($options)) {
             if (isset($options['query'])) {
                 $config->setQuery($options['query']);
             }

src/Type/SchemaValidationContext.php

Spacing +22 added lines, -22 removed lines

@@ -64,12 +64,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')
@@ -77,7 +77,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')
@@ -85,7 +85,7 @@ 
         }
 
         $subscriptionType = $this->schema->getSubscriptionType();
-        if (! $subscriptionType || $subscriptionType instanceof ObjectType) {
+        if (!$subscriptionType || $subscriptionType instanceof ObjectType) {
             return;
         }
 
@@ -143,7 +143,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
@@ -198,7 +198,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;
         }
 
@@ -242,7 +242,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) . '.',
                     is_object($type) ? $type->astNode : null
@@ -282,7 +282,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->getAllObjectOrInterfaceNodes($type)
@@ -304,7 +304,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.',
@@ -379,7 +379,7 @@ 
         $fieldNodes = [];
         $astNodes   = $this->getAllObjectOrInterfaceNodes($type);
         foreach ($astNodes as $astNode) {
-            if (! $astNode || ! $astNode->fields) {
+            if (!$astNode || !$astNode->fields) {
                 continue;
             }
 
@@ -472,7 +472,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.',
@@ -516,7 +516,7 @@ 
         $astNodes        = $this->getAllObjectOrInterfaceNodes($type);
 
         foreach ($astNodes as $astNode) {
-            if (! $astNode || ! $astNode->interfaces) {
+            if (!$astNode || !$astNode->interfaces) {
                 continue;
             }
 
@@ -547,7 +547,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.',
@@ -562,7 +562,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()
@@ -598,7 +598,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.',
@@ -619,7 +619,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.',
@@ -653,7 +653,7 @@ 
                     }
                 }
 
-                if ($ifaceArg || ! ($objectArg->getType() instanceof NonNull)) {
+                if ($ifaceArg || !($objectArg->getType() instanceof NonNull)) {
                     continue;
                 }
 
@@ -680,7 +680,7 @@ 
     {
         $memberTypes = $union->getTypes();
 
-        if (! $memberTypes) {
+        if (!$memberTypes) {
             $this->reportError(
                 sprintf('Union type %s must define one or more member types.', $union->name),
                 $union->astNode
@@ -722,7 +722,7 @@ 
         if ($union->astNode && $union->astNode->types) {
             return array_filter(
                 $union->astNode->types,
-                function (NamedTypeNode $value) use ($typeName) {
+                function(NamedTypeNode $value) use ($typeName) {
                     return $value->name->value === $typeName;
                 }
             );
@@ -736,7 +736,7 @@ 
     {
         $enumValues = $enumType->getValues();
 
-        if (! $enumValues) {
+        if (!$enumValues) {
             $this->reportError(
                 sprintf('Enum type %s must define one or more values.', $enumType->name),
                 $enumType->astNode
@@ -777,7 +777,7 @@ 
         if ($enum->astNode && $enum->astNode->values) {
             return array_filter(
                 iterator_to_array($enum->astNode->values),
-                function (EnumValueDefinitionNode $value) use ($valueName) {
+                function(EnumValueDefinitionNode $value) use ($valueName) {
                     return $value->name->value === $valueName;
                 }
             );
@@ -791,7 +791,7 @@ 
     {
         $fieldMap = $inputObj->getFields();
 
-        if (! $fieldMap) {
+        if (!$fieldMap) {
             $this->reportError(
                 sprintf('Input Object type %s must define one or more fields.', $inputObj->name),
                 $inputObj->astNode

src/Type/Introspection.php

Spacing +47 added lines, -47 removed lines

@@ -56,7 +56,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' : '';
 
@@ -180,7 +180,7 @@ 
 
     public static function _schema()
     {
-        if (! isset(self::$map['__Schema'])) {
+        if (!isset(self::$map['__Schema'])) {
             self::$map['__Schema'] = new ObjectType([
                 'name'            => '__Schema',
                 'isIntrospection' => true,
@@ -193,14 +193,14 @@ 
                     'types'            => [
                         'description' => 'A list of all types supported by this server.',
                         'type'        => new NonNull(new ListOfType(new NonNull(self::_type()))),
-                        'resolve'     => function (Schema $schema) {
+                        'resolve'     => 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'     => function (Schema $schema) {
+                        'resolve'     => function(Schema $schema) {
                             return $schema->getQueryType();
                         },
                     ],
@@ -209,21 +209,21 @@ 
                             'If this server supports mutation, the type that ' .
                             'mutation operations will be rooted at.',
                         'type'        => self::_type(),
-                        'resolve'     => function (Schema $schema) {
+                        'resolve'     => 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'     => function (Schema $schema) {
+                        'resolve'     => 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'     => function (Schema $schema) {
+                        'resolve'     => function(Schema $schema) {
                             return $schema->getDirectives();
                         },
                     ],
@@ -236,7 +236,7 @@ 
 
     public static function _type()
     {
-        if (! isset(self::$map['__Type'])) {
+        if (!isset(self::$map['__Type'])) {
             self::$map['__Type'] = new ObjectType([
                 'name'            => '__Type',
                 'isIntrospection' => true,
@@ -250,11 +250,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'          => function () {
+                'fields'          => function() {
                     return [
                         'kind'          => [
                             'type'    => Type::nonNull(self::_typeKind()),
-                            'resolve' => function (Type $type) {
+                            'resolve' => function(Type $type) {
                                 switch (true) {
                                     case $type instanceof ListOfType:
                                         return TypeKind::LIST_KIND;
@@ -284,15 +284,15 @@ 
                             'args'    => [
                                 'includeDeprecated' => ['type' => Type::boolean(), 'defaultValue' => false],
                             ],
-                            'resolve' => function (Type $type, $args) {
+                            'resolve' => function(Type $type, $args) {
                                 if ($type instanceof ObjectType || $type instanceof InterfaceType) {
                                     $fields = $type->getFields();
 
                                     if (empty($args['includeDeprecated'])) {
                                         $fields = array_filter(
                                             $fields,
-                                            function (FieldDefinition $field) {
-                                                return ! $field->deprecationReason;
+                                            function(FieldDefinition $field) {
+                                                return !$field->deprecationReason;
                                             }
                                         );
                                     }
@@ -305,7 +305,7 @@ 
                         ],
                         'interfaces'    => [
                             'type'    => Type::listOf(Type::nonNull(self::_type())),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof ObjectType) {
                                     return $type->getInterfaces();
                                 }
@@ -315,7 +315,7 @@ 
                         ],
                         'possibleTypes' => [
                             'type'    => Type::listOf(Type::nonNull(self::_type())),
-                            'resolve' => function ($type, $args, $context, ResolveInfo $info) {
+                            'resolve' => function($type, $args, $context, ResolveInfo $info) {
                                 if ($type instanceof InterfaceType || $type instanceof UnionType) {
                                     return $info->schema->getPossibleTypes($type);
                                 }
@@ -328,15 +328,15 @@ 
                             'args'    => [
                                 'includeDeprecated' => ['type' => Type::boolean(), 'defaultValue' => false],
                             ],
-                            'resolve' => function ($type, $args) {
+                            'resolve' => function($type, $args) {
                                 if ($type instanceof EnumType) {
                                     $values = array_values($type->getValues());
 
                                     if (empty($args['includeDeprecated'])) {
                                         $values = array_filter(
                                             $values,
-                                            function ($value) {
-                                                return ! $value->deprecationReason;
+                                            function($value) {
+                                                return !$value->deprecationReason;
                                             }
                                         );
                                     }
@@ -349,7 +349,7 @@ 
                         ],
                         'inputFields'   => [
                             'type'    => Type::listOf(Type::nonNull(self::_inputValue())),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof InputObjectType) {
                                     return array_values($type->getFields());
                                 }
@@ -359,7 +359,7 @@ 
                         ],
                         'ofType'        => [
                             'type'    => self::_type(),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof WrappingType) {
                                     return $type->getWrappedType();
                                 }
@@ -377,7 +377,7 @@ 
 
     public static function _typeKind()
     {
-        if (! isset(self::$map['__TypeKind'])) {
+        if (!isset(self::$map['__TypeKind'])) {
             self::$map['__TypeKind'] = new EnumType([
                 'name'            => '__TypeKind',
                 'isIntrospection' => true,
@@ -424,33 +424,33 @@ 
 
     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'          => function () {
+                'fields'          => function() {
                     return [
                         'name'              => ['type' => Type::nonNull(Type::string())],
                         'description'       => ['type' => Type::string()],
                         'args'              => [
                             'type'    => Type::nonNull(Type::listOf(Type::nonNull(self::_inputValue()))),
-                            'resolve' => function (FieldDefinition $field) {
+                            'resolve' => function(FieldDefinition $field) {
                                 return empty($field->args) ? [] : $field->args;
                             },
                         ],
                         'type'              => [
                             'type'    => Type::nonNull(self::_type()),
-                            'resolve' => function (FieldDefinition $field) {
+                            'resolve' => function(FieldDefinition $field) {
                                 return $field->getType();
                             },
                         ],
                         'isDeprecated'      => [
                             'type'    => Type::nonNull(Type::boolean()),
-                            'resolve' => function (FieldDefinition $field) {
-                                return ! ! $field->deprecationReason;
+                            'resolve' => function(FieldDefinition $field) {
+                                return !!$field->deprecationReason;
                             },
                         ],
                         'deprecationReason' => [
@@ -466,7 +466,7 @@ 
 
     public static function _inputValue()
     {
-        if (! isset(self::$map['__InputValue'])) {
+        if (!isset(self::$map['__InputValue'])) {
             self::$map['__InputValue'] = new ObjectType([
                 'name'            => '__InputValue',
                 'isIntrospection' => true,
@@ -474,13 +474,13 @@ 
                     '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'          => function () {
+                'fields'          => function() {
                     return [
                         'name'         => ['type' => Type::nonNull(Type::string())],
                         'description'  => ['type' => Type::string()],
                         'type'         => [
                             'type'    => Type::nonNull(self::_type()),
-                            'resolve' => function ($value) {
+                            'resolve' => function($value) {
                                 return method_exists($value, 'getType') ? $value->getType() : $value->type;
                             },
                         ],
@@ -488,9 +488,9 @@ 
                             'type'        => Type::string(),
                             'description' =>
                                 'A GraphQL-formatted string representing the default value for this input value.',
-                            'resolve'     => function ($inputValue) {
+                            'resolve'     => function($inputValue) {
                                 /** @var FieldArgument|InputObjectField $inputValue */
-                                return ! $inputValue->defaultValueExists()
+                                return !$inputValue->defaultValueExists()
                                     ? null
                                     : Printer::doPrint(AST::astFromValue(
                                         $inputValue->defaultValue,
@@ -508,7 +508,7 @@ 
 
     public static function _enumValue()
     {
-        if (! isset(self::$map['__EnumValue'])) {
+        if (!isset(self::$map['__EnumValue'])) {
             self::$map['__EnumValue'] = new ObjectType([
                 'name'            => '__EnumValue',
                 'isIntrospection' => true,
@@ -521,8 +521,8 @@ 
                     'description'       => ['type' => Type::string()],
                     'isDeprecated'      => [
                         'type'    => Type::nonNull(Type::boolean()),
-                        'resolve' => function ($enumValue) {
-                            return ! ! $enumValue->deprecationReason;
+                        'resolve' => function($enumValue) {
+                            return !!$enumValue->deprecationReason;
                         },
                     ],
                     'deprecationReason' => [
@@ -537,7 +537,7 @@ 
 
     public static function _directive()
     {
-        if (! isset(self::$map['__Directive'])) {
+        if (!isset(self::$map['__Directive'])) {
             self::$map['__Directive'] = new ObjectType([
                 'name'            => '__Directive',
                 'isIntrospection' => true,
@@ -557,7 +557,7 @@ 
                     ],
                     'args'        => [
                         'type'    => Type::nonNull(Type::listOf(Type::nonNull(self::_inputValue()))),
-                        'resolve' => function (Directive $directive) {
+                        'resolve' => function(Directive $directive) {
                             return $directive->args ?: [];
                         },
                     ],
@@ -567,7 +567,7 @@ 
                     'onOperation' => [
                         'deprecationReason' => 'Use `locations`.',
                         'type'              => Type::nonNull(Type::boolean()),
-                        'resolve'           => function ($d) {
+                        'resolve'           => function($d) {
                             return in_array(DirectiveLocation::QUERY, $d->locations) ||
                                 in_array(DirectiveLocation::MUTATION, $d->locations) ||
                                 in_array(DirectiveLocation::SUBSCRIPTION, $d->locations);
@@ -576,7 +576,7 @@ 
                     'onFragment'  => [
                         'deprecationReason' => 'Use `locations`.',
                         'type'              => Type::nonNull(Type::boolean()),
-                        'resolve'           => function ($d) {
+                        'resolve'           => function($d) {
                             return in_array(DirectiveLocation::FRAGMENT_SPREAD, $d->locations) ||
                                 in_array(DirectiveLocation::INLINE_FRAGMENT, $d->locations) ||
                                 in_array(DirectiveLocation::FRAGMENT_DEFINITION, $d->locations);
@@ -585,7 +585,7 @@ 
                     'onField'     => [
                         'deprecationReason' => 'Use `locations`.',
                         'type'              => Type::nonNull(Type::boolean()),
-                        'resolve'           => function ($d) {
+                        'resolve'           => function($d) {
                             return in_array(DirectiveLocation::FIELD, $d->locations);
                         },
                     ],
@@ -598,7 +598,7 @@ 
 
     public static function _directiveLocation()
     {
-        if (! isset(self::$map['__DirectiveLocation'])) {
+        if (!isset(self::$map['__DirectiveLocation'])) {
             self::$map['__DirectiveLocation'] = new EnumType([
                 'name'            => '__DirectiveLocation',
                 'isIntrospection' => true,
@@ -688,13 +688,13 @@ 
 
     public static function schemaMetaFieldDef()
     {
-        if (! isset(self::$map['__schema'])) {
+        if (!isset(self::$map['__schema'])) {
             self::$map['__schema'] = FieldDefinition::create([
                 'name'        => '__schema',
                 'type'        => Type::nonNull(self::_schema()),
                 'description' => 'Access the current type schema of this server.',
                 'args'        => [],
-                'resolve'     => function (
+                'resolve'     => function(
                     $source,
                     $args,
                     $context,
@@ -710,7 +710,7 @@ 
 
     public static function typeMetaFieldDef()
     {
-        if (! isset(self::$map['__type'])) {
+        if (!isset(self::$map['__type'])) {
             self::$map['__type'] = FieldDefinition::create([
                 'name'        => '__type',
                 'type'        => self::_type(),
@@ -718,7 +718,7 @@ 
                 'args'        => [
                     ['name' => 'name', 'type' => Type::nonNull(Type::string())],
                 ],
-                'resolve'     => function ($source, $args, $context, ResolveInfo $info) {
+                'resolve'     => function($source, $args, $context, ResolveInfo $info) {
                     return $info->schema->getType($args['name']);
                 },
             ]);
@@ -729,13 +729,13 @@ 
 
     public static function typeNameMetaFieldDef()
     {
-        if (! isset(self::$map['__typename'])) {
+        if (!isset(self::$map['__typename'])) {
             self::$map['__typename'] = FieldDefinition::create([
                 'name'        => '__typename',
                 'type'        => Type::nonNull(Type::string()),
                 'description' => 'The name of the current Object type at runtime.',
                 'args'        => [],
-                'resolve'     => function (
+                'resolve'     => function(
                     $source,
                     $args,
                     $context,

src/Type/Schema.php

Spacing +12 added lines, -12 removed lines

@@ -101,11 +101,11 @@ 
                 Utils::getVariableType($config)
             );
             Utils::invariant(
-                ! $config->types || is_array($config->types) || is_callable($config->types),
+                !$config->types || is_array($config->types) || is_callable($config->types),
                 '"types" must be array or callable if provided but got: ' . Utils::getVariableType($config->types)
             );
             Utils::invariant(
-                ! $config->directives || is_array($config->directives),
+                !$config->directives || is_array($config->directives),
                 '"directives" must be Array if provided but got: ' . Utils::getVariableType($config->directives)
             );
         }
@@ -155,7 +155,7 @@ 
             $types = $types();
         }
 
-        if (! is_array($types) && ! $types instanceof \Traversable) {
+        if (!is_array($types) && !$types instanceof \Traversable) {
             throw new InvariantViolation(sprintf(
                 'Schema types callable must return array or instance of Traversable but got: %s',
                 Utils::getVariableType($types)
@@ -163,7 +163,7 @@ 
         }
 
         foreach ($types as $index => $type) {
-            if (! $type instanceof Type) {
+            if (!$type instanceof Type) {
                 throw new InvariantViolation(sprintf(
                     'Each entry of schema types must be instance of GraphQL\Type\Definition\Type but entry at %s is %s',
                     $index,
@@ -185,7 +185,7 @@ 
      */
     public function getTypeMap()
     {
-        if (! $this->fullyLoaded) {
+        if (!$this->fullyLoaded) {
             $this->resolvedTypes = $this->collectAllTypes();
             $this->fullyLoaded   = true;
         }
@@ -203,7 +203,7 @@ 
             $typeMap = TypeInfo::extractTypes($type, $typeMap);
         }
         foreach ($this->getDirectives() as $directive) {
-            if (! ($directive instanceof Directive)) {
+            if (!($directive instanceof Directive)) {
                 continue;
             }
 
@@ -281,9 +281,9 @@ 
      */
     public function getType($name)
     {
-        if (! isset($this->resolvedTypes[$name])) {
+        if (!isset($this->resolvedTypes[$name])) {
             $type = $this->loadType($name);
-            if (! $type) {
+            if (!$type) {
                 return null;
             }
             $this->resolvedTypes[$name] = $type;
@@ -300,13 +300,13 @@ 
     {
         $typeLoader = $this->config->typeLoader;
 
-        if (! $typeLoader) {
+        if (!$typeLoader) {
             return $this->defaultTypeLoader($typeName);
         }
 
         $type = $typeLoader($typeName);
 
-        if (! $type instanceof Type) {
+        if (!$type instanceof Type) {
             throw new InvariantViolation(
                 sprintf(
                     'Type loader is expected to return valid type "%s", but it returned %s',
@@ -362,7 +362,7 @@ 
             foreach ($this->getTypeMap() as $type) {
                 if ($type instanceof ObjectType) {
                     foreach ($type->getInterfaces() as $interface) {
-                        if (! ($interface instanceof InterfaceType)) {
+                        if (!($interface instanceof InterfaceType)) {
                             continue;
                         }
 
@@ -447,7 +447,7 @@ 
             $type->assertValid();
 
             // Make sure type loader returns the same instance as registered in other places of schema
-            if (! $this->config->typeLoader) {
+            if (!$this->config->typeLoader) {
                 continue;
             }
 

src/Type/EagerResolution.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 
         // Keep track of all possible types for abstract types
         foreach ($this->typeMap as $typeName => $type) {
-            if (! ($type instanceof ObjectType)) {
+            if (!($type instanceof ObjectType)) {
                 continue;
             }
 
@@ -61,7 +61,7 @@ 
      */
     public function resolvePossibleTypes(AbstractType $abstractType)
     {
-        if (! isset($this->typeMap[$abstractType->name])) {
+        if (!isset($this->typeMap[$abstractType->name])) {
             return [];
         }
 

src/Type/LazyResolution.php

Spacing +6 added lines, -6 removed lines

@@ -66,14 +66,14 @@ 
      */
     public function resolvePossibleTypes(AbstractType $type)
     {
-        if (! isset($this->possibleTypeMap[$type->name])) {
+        if (!isset($this->possibleTypeMap[$type->name])) {
             return [];
         }
-        if (! isset($this->loadedPossibleTypes[$type->name])) {
+        if (!isset($this->loadedPossibleTypes[$type->name])) {
             $tmp = [];
             foreach ($this->possibleTypeMap[$type->name] as $typeName => $true) {
                 $obj = $this->resolveType($typeName);
-                if (! $obj instanceof ObjectType) {
+                if (!$obj instanceof ObjectType) {
                     throw new InvariantViolation(
                         sprintf(
                             'Lazy Type Resolution Error: Implementation %s of interface %s is expected to be instance of ObjectType, but got %s',
@@ -96,12 +96,12 @@ 
      */
     public function resolveType($name)
     {
-        if (! isset($this->typeMap[$name])) {
+        if (!isset($this->typeMap[$name])) {
             return null;
         }
-        if (! isset($this->loadedTypes[$name])) {
+        if (!isset($this->loadedTypes[$name])) {
             $type = call_user_func($this->typeLoader, $name);
-            if (! $type instanceof Type && $type !== null) {
+            if (!$type instanceof Type && $type !== null) {
                 throw new InvariantViolation(
                     'Lazy Type Resolution Error: Expecting GraphQL Type instance, but got ' .
                     Utils::getVariableType($type)

src/Executor/Values.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
         if (isset($node->directives) && $node->directives instanceof NodeList) {
             $directiveNode = Utils::find(
                 $node->directives,
-                function (DirectiveNode $directive) use ($directiveDef) {
+                function(DirectiveNode $directive) use ($directiveDef) {
                     return $directive->name->value === $directiveDef->name;
                 }
             );
@@ -162,7 +162,7 @@ 
         /** @var ArgumentNode[] $argNodeMap */
         $argNodeMap = $argNodes ? Utils::keyMap(
             $argNodes,
-            function (ArgumentNode $arg) {
+            function(ArgumentNode $arg) {
                 return $arg->name->value;
             }
         ) : [];
@@ -172,7 +172,7 @@ 
             $argType      = $argDef->getType();
             $argumentNode = $argNodeMap[$name] ?? null;
 
-            if (! $argumentNode) {
+            if (!$argumentNode) {
                 if ($argDef->defaultValueExists()) {
                     $coercedValues[$name] = $argDef->defaultValue;
                 } elseif ($argType instanceof NonNull) {
@@ -242,7 +242,7 @@ 
 
         return $errors
             ? array_map(
-                function (Throwable $error) {
+                function(Throwable $error) {
                     return $error->getMessage();
                 },
                 $errors

src/Executor/Executor.php

Spacing +40 added lines, -42 removed lines

@@ -64,7 +64,7 @@ 
 
     private function __construct(ExecutionContext $context)
     {
-        if (! self::$UNDEFINED) {
+        if (!self::$UNDEFINED) {
             self::$UNDEFINED = Utils::undefined();
         }
 
@@ -212,10 +212,10 @@ 
         foreach ($documentNode->definitions as $definition) {
             switch ($definition->kind) {
                 case NodeKind::OPERATION_DEFINITION:
-                    if (! $operationName && $operation) {
+                    if (!$operationName && $operation) {
                         $hasMultipleAssumedOperations = true;
                     }
-                    if (! $operationName ||
+                    if (!$operationName ||
                         (isset($definition->name) && $definition->name->value === $operationName)) {
                         $operation = $definition;
                     }
@@ -226,7 +226,7 @@ 
             }
         }
 
-        if (! $operation) {
+        if (!$operation) {
             if ($operationName) {
                 $errors[] = new Error(sprintf('Unknown operation named "%s".', $operationName));
             } else {
@@ -302,7 +302,7 @@ 
     private function buildResponse($data)
     {
         if ($this->isPromise($data)) {
-            return $data->then(function ($resolved) {
+            return $data->then(function($resolved) {
                 return $this->buildResponse($resolved);
             });
         }
@@ -332,13 +332,12 @@ 
         // Similar to completeValueCatchingError.
         try {
             $result = $operation->operation === 'mutation' ?
-                $this->executeFieldsSerially($type, $rootValue, $path, $fields) :
-                $this->executeFields($type, $rootValue, $path, $fields);
+                $this->executeFieldsSerially($type, $rootValue, $path, $fields) : $this->executeFields($type, $rootValue, $path, $fields);
 
             if ($this->isPromise($result)) {
                 return $result->then(
                     null,
-                    function ($error) {
+                    function($error) {
                         $this->exeContext->addError($error);
                         return $this->exeContext->promises->createFulfilled(null);
                     }
@@ -363,7 +362,7 @@ 
         switch ($operation->operation) {
             case 'query':
                 $queryType = $schema->getQueryType();
-                if (! $queryType) {
+                if (!$queryType) {
                     throw new Error(
                         'Schema does not define the required query root type.',
                         [$operation]
@@ -373,7 +372,7 @@ 
                 return $queryType;
             case 'mutation':
                 $mutationType = $schema->getMutationType();
-                if (! $mutationType) {
+                if (!$mutationType) {
                     throw new Error(
                         'Schema is not configured for mutations.',
                         [$operation]
@@ -383,7 +382,7 @@ 
                 return $mutationType;
             case 'subscription':
                 $subscriptionType = $schema->getSubscriptionType();
-                if (! $subscriptionType) {
+                if (!$subscriptionType) {
                     throw new Error(
                         'Schema is not configured for subscriptions.',
                         [$operation]
@@ -422,18 +421,18 @@ 
         foreach ($selectionSet->selections as $selection) {
             switch ($selection->kind) {
                 case NodeKind::FIELD:
-                    if (! $this->shouldIncludeNode($selection)) {
+                    if (!$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $name = self::getFieldEntryKey($selection);
-                    if (! isset($fields[$name])) {
+                    if (!isset($fields[$name])) {
                         $fields[$name] = new \ArrayObject();
                     }
                     $fields[$name][] = $selection;
                     break;
                 case NodeKind::INLINE_FRAGMENT:
-                    if (! $this->shouldIncludeNode($selection) ||
-                        ! $this->doesFragmentConditionMatch($selection, $runtimeType)
+                    if (!$this->shouldIncludeNode($selection) ||
+                        !$this->doesFragmentConditionMatch($selection, $runtimeType)
                     ) {
                         break;
                     }
@@ -446,14 +445,14 @@ 
                     break;
                 case NodeKind::FRAGMENT_SPREAD:
                     $fragName = $selection->name->value;
-                    if (! empty($visitedFragmentNames[$fragName]) || ! $this->shouldIncludeNode($selection)) {
+                    if (!empty($visitedFragmentNames[$fragName]) || !$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $visitedFragmentNames[$fragName] = true;
 
                     /** @var FragmentDefinitionNode|null $fragment */
                     $fragment = $exeContext->fragments[$fragName] ?? null;
-                    if (! $fragment || ! $this->doesFragmentConditionMatch($fragment, $runtimeType)) {
+                    if (!$fragment || !$this->doesFragmentConditionMatch($fragment, $runtimeType)) {
                         break;
                     }
                     $this->collectFields(
@@ -556,7 +555,7 @@ 
     {
         $result = $this->promiseReduce(
             array_keys($fields->getArrayCopy()),
-            function ($results, $responseName) use ($path, $parentType, $sourceValue, $fields) {
+            function($results, $responseName) use ($path, $parentType, $sourceValue, $fields) {
                 $fieldNodes  = $fields[$responseName];
                 $fieldPath   = $path;
                 $fieldPath[] = $responseName;
@@ -566,7 +565,7 @@ 
                 }
                 $promise = $this->getPromise($result);
                 if ($promise) {
-                    return $promise->then(function ($resolvedResult) use ($responseName, $results) {
+                    return $promise->then(function($resolvedResult) use ($responseName, $results) {
                         $results[$responseName] = $resolvedResult;
                         return $results;
                     });
@@ -577,7 +576,7 @@ 
             []
         );
         if ($this->isPromise($result)) {
-            return $result->then(function ($resolvedResults) {
+            return $result->then(function($resolvedResults) {
                 return self::fixResultsIfEmptyArray($resolvedResults);
             });
         }
@@ -604,7 +603,7 @@ 
         $fieldName = $fieldNode->name->value;
         $fieldDef  = $this->getFieldDef($exeContext->schema, $parentType, $fieldName);
 
-        if (! $fieldDef) {
+        if (!$fieldDef) {
             return self::$UNDEFINED;
         }
 
@@ -770,7 +769,7 @@ 
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($exeContext) {
+                    function($error) use ($exeContext) {
                         $exeContext->addError($error);
 
                         return $this->exeContext->promises->createFulfilled(null);
@@ -813,11 +812,11 @@ 
                 $path,
                 $result
             );
-            $promise   = $this->getPromise($completed);
+            $promise = $this->getPromise($completed);
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($fieldNodes, $path) {
+                    function($error) use ($fieldNodes, $path) {
                         return $this->exeContext->promises->createRejected(Error::createLocatedError(
                             $error,
                             $fieldNodes,
@@ -874,7 +873,7 @@ 
 
         // If result is a Promise, apply-lift over completeValue.
         if ($promise) {
-            return $promise->then(function (&$resolved) use ($returnType, $fieldNodes, $info, $path) {
+            return $promise->then(function(&$resolved) use ($returnType, $fieldNodes, $info, $path) {
                 return $this->completeValue($returnType, $fieldNodes, $info, $path, $resolved);
             });
         }
@@ -974,7 +973,7 @@ 
         }
         if ($this->exeContext->promises->isThenable($value)) {
             $promise = $this->exeContext->promises->convertThenable($value);
-            if (! $promise instanceof Promise) {
+            if (!$promise instanceof Promise) {
                 throw new InvariantViolation(sprintf(
                     '%s::convertThenable is expected to return instance of GraphQL\Executor\Promise\Promise, got: %s',
                     get_class($this->exeContext->promises),
@@ -1002,10 +1001,10 @@ 
      */
     private function promiseReduce(array $values, \Closure $callback, $initialValue)
     {
-        return array_reduce($values, function ($previous, $value) use ($callback) {
+        return array_reduce($values, function($previous, $value) use ($callback) {
             $promise = $this->getPromise($previous);
             if ($promise) {
-                return $promise->then(function ($resolved) use ($callback, $value) {
+                return $promise->then(function($resolved) use ($callback, $value) {
                     return $callback($resolved, $value);
                 });
             }
@@ -1038,7 +1037,7 @@ 
             $fieldPath     = $path;
             $fieldPath[]   = $i++;
             $completedItem = $this->completeValueCatchingError($itemType, $fieldNodes, $info, $fieldPath, $item);
-            if (! $containsPromise && $this->getPromise($completedItem)) {
+            if (!$containsPromise && $this->getPromise($completedItem)) {
                 $containsPromise = true;
             }
             $completedItems[] = $completedItem;
@@ -1094,7 +1093,7 @@ 
 
         $promise = $this->getPromise($runtimeType);
         if ($promise) {
-            return $promise->then(function ($resolvedRuntimeType) use (
+            return $promise->then(function($resolvedRuntimeType) use (
                 $returnType,
                 $fieldNodes,
                 $info,
@@ -1188,9 +1187,9 @@ 
             }
         }
 
-        if (! empty($promisedIsTypeOfResults)) {
+        if (!empty($promisedIsTypeOfResults)) {
             return $this->exeContext->promises->all($promisedIsTypeOfResults)
-                ->then(function ($isTypeOfResults) use ($possibleTypes) {
+                ->then(function($isTypeOfResults) use ($possibleTypes) {
                     foreach ($isTypeOfResults as $index => $result) {
                         if ($result) {
                             return $possibleTypes[$index];
@@ -1223,14 +1222,14 @@ 
         if ($isTypeOf !== null) {
             $promise = $this->getPromise($isTypeOf);
             if ($promise) {
-                return $promise->then(function ($isTypeOfResult) use (
+                return $promise->then(function($isTypeOfResult) use (
                     $returnType,
                     $fieldNodes,
                     $info,
                     $path,
                     &$result
                 ) {
-                    if (! $isTypeOfResult) {
+                    if (!$isTypeOfResult) {
                         throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
                     }
 
@@ -1243,7 +1242,7 @@ 
                     );
                 });
             }
-            if (! $isTypeOf) {
+            if (!$isTypeOf) {
                 throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
             }
         }
@@ -1344,14 +1343,14 @@ 
             if ($result === self::$UNDEFINED) {
                 continue;
             }
-            if (! $containsPromise && $this->getPromise($result)) {
+            if (!$containsPromise && $this->getPromise($result)) {
                 $containsPromise = true;
             }
             $finalResults[$responseName] = $result;
         }
 
         // If there are no promises, we can just return the object
-        if (! $containsPromise) {
+        if (!$containsPromise) {
             return self::fixResultsIfEmptyArray($finalResults);
         }
 
@@ -1394,7 +1393,7 @@ 
 
         $promise = $this->exeContext->promises->all($valuesAndPromises);
 
-        return $promise->then(function ($values) use ($keys) {
+        return $promise->then(function($values) use ($keys) {
             $resolvedResults = [];
             foreach ($values as $i => $value) {
                 $resolvedResults[$keys[$i]] = $value;
@@ -1417,10 +1416,9 @@ 
         &$result
     ) {
         $runtimeType = is_string($runtimeTypeOrName) ?
-            $this->exeContext->schema->getType($runtimeTypeOrName) :
-            $runtimeTypeOrName;
+            $this->exeContext->schema->getType($runtimeTypeOrName) : $runtimeTypeOrName;
 
-        if (! $runtimeType instanceof ObjectType) {
+        if (!$runtimeType instanceof ObjectType) {
             throw new InvariantViolation(
                 sprintf(
                     'Abstract type %s must resolve to an Object type at ' .
@@ -1437,7 +1435,7 @@ 
             );
         }
 
-        if (! $this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
+        if (!$this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
             throw new InvariantViolation(
                 sprintf('Runtime Object type "%s" is not a possible type for "%s".', $runtimeType, $returnType)
             );