simPod / graphql-php

src/Utils/Value.php

Spacing +1 added lines, -1 removed lines

@@ -242,7 +242,7 @@
     private static function printPath(array $path = null) {
         $pathStr = '';
         $currentPath = $path;
-        while($currentPath) {
+        while ($currentPath) {
             $pathStr =
                 (is_string($currentPath['key'])
                     ? '.' . $currentPath['key']

src/Server/Helper.php

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
             $contentType = isset($_SERVER['CONTENT_TYPE']) ? $_SERVER['CONTENT_TYPE'] : null;
 
             if (stripos($contentType, 'application/graphql') !== false) {
-                $rawBody =  $readRawBodyFn ? $readRawBodyFn() : $this->readRawBody();
+                $rawBody = $readRawBodyFn ? $readRawBodyFn() : $this->readRawBody();
                 $bodyParams = ['query' => $rawBody ?: ''];
             } else if (stripos($contentType, 'application/json') !== false) {
                 $rawBody = $readRawBodyFn ? $readRawBodyFn() : $this->readRawBody();
@@ -270,7 +270,7 @@ 
             );
         }
 
-        $applyErrorHandling = function (ExecutionResult $result) use ($config) {
+        $applyErrorHandling = function(ExecutionResult $result) use ($config) {
             if ($config->getErrorsHandler()) {
                 $result->setErrorsHandler($config->getErrorsHandler());
             }
@@ -435,7 +435,7 @@ 
     private function resolveHttpStatus($result)
     {
         if (is_array($result) && isset($result[0])) {
-            Utils::each($result, function ($executionResult, $index) {
+            Utils::each($result, function($executionResult, $index) {
                 if (!$executionResult instanceof ExecutionResult) {
                     throw new InvariantViolation(sprintf(
                         "Expecting every entry of batched query result to be instance of %s but entry at position %d is %s",

src/Type/Introspection.php

Indentation +22 added lines, -22 removed lines

@@ -331,48 +331,48 @@
                         'description' => 'Location adjacent to an inline fragment.'
                     ],
                     'SCHEMA' => [
-                      'value' => DirectiveLocation::SCHEMA,
-                      'description' =>  'Location adjacent to a schema definition.'
+                        'value' => DirectiveLocation::SCHEMA,
+                        'description' =>  'Location adjacent to a schema definition.'
                     ],
                     'SCALAR' => [
-                      'value' => DirectiveLocation::SCALAR,
-                      'description' =>  'Location adjacent to a scalar definition.'
+                        'value' => DirectiveLocation::SCALAR,
+                        'description' =>  'Location adjacent to a scalar definition.'
                     ],
                     'OBJECT' => [
-                      'value' => DirectiveLocation::OBJECT,
-                      'description' =>  'Location adjacent to an object type definition.'
+                        'value' => DirectiveLocation::OBJECT,
+                        'description' =>  'Location adjacent to an object type definition.'
                     ],
                     'FIELD_DEFINITION' => [
-                      'value' => DirectiveLocation::FIELD_DEFINITION,
-                      'description' =>  'Location adjacent to a field definition.'
+                        'value' => DirectiveLocation::FIELD_DEFINITION,
+                        'description' =>  'Location adjacent to a field definition.'
                     ],
                     'ARGUMENT_DEFINITION' => [
-                      'value' => DirectiveLocation::ARGUMENT_DEFINITION,
-                      'description' =>  'Location adjacent to an argument definition.'
+                        'value' => DirectiveLocation::ARGUMENT_DEFINITION,
+                        'description' =>  'Location adjacent to an argument definition.'
                     ],
                     'INTERFACE' => [
-                      'value' => DirectiveLocation::IFACE,
-                      'description' =>  'Location adjacent to an interface definition.'
+                        'value' => DirectiveLocation::IFACE,
+                        'description' =>  'Location adjacent to an interface definition.'
                     ],
                     'UNION' => [
-                      'value' => DirectiveLocation::UNION,
-                      'description' =>  'Location adjacent to a union definition.'
+                        'value' => DirectiveLocation::UNION,
+                        'description' =>  'Location adjacent to a union definition.'
                     ],
                     'ENUM' => [
-                      'value' => DirectiveLocation::ENUM,
-                      'description' =>  'Location adjacent to an enum definition.'
+                        'value' => DirectiveLocation::ENUM,
+                        'description' =>  'Location adjacent to an enum definition.'
                     ],
                     'ENUM_VALUE' => [
-                      'value' => DirectiveLocation::ENUM_VALUE,
-                      'description' =>  'Location adjacent to an enum value definition.'
+                        'value' => DirectiveLocation::ENUM_VALUE,
+                        'description' =>  'Location adjacent to an enum value definition.'
                     ],
                     'INPUT_OBJECT' => [
-                      'value' => DirectiveLocation::INPUT_OBJECT,
-                      'description' =>  'Location adjacent to an input object type definition.'
+                        'value' => DirectiveLocation::INPUT_OBJECT,
+                        'description' =>  'Location adjacent to an input object type definition.'
                     ],
                     'INPUT_FIELD_DEFINITION' => [
-                      'value' => DirectiveLocation::INPUT_FIELD_DEFINITION,
-                      'description' =>  'Location adjacent to an input object field definition.'
+                        'value' => DirectiveLocation::INPUT_FIELD_DEFINITION,
+                        'description' =>  'Location adjacent to an input object field definition.'
                     ]
 
                 ]

Spacing +25 added lines, -25 removed lines

@@ -49,7 +49,7 @@ 
     {
         if (is_bool($options)) {
             trigger_error(
-                'Calling Introspection::getIntrospectionQuery(boolean) is deprecated. '.
+                'Calling Introspection::getIntrospectionQuery(boolean) is deprecated. ' .
                 'Please use Introspection::getIntrospectionQuery(["descriptions" => boolean]).',
                 E_USER_DEPRECATED
             );
@@ -192,14 +192,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();
                         }
                     ],
@@ -208,14 +208,14 @@ 
                             '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();
                         },
                     ],
@@ -254,7 +254,7 @@ 
                     ],
                     'args' => [
                         'type' => Type::nonNull(Type::listOf(Type::nonNull(self::_inputValue()))),
-                        'resolve' => function (Directive $directive) {
+                        'resolve' => function(Directive $directive) {
                             return $directive->args ?: [];
                         }
                     ],
@@ -390,7 +390,7 @@ 
                 'description' =>
                     'The fundamental unit of any GraphQL Schema is the type. There are ' .
                     'many kinds of types in GraphQL as represented by the `__TypeKind` enum.' .
-                    "\n\n".
+                    "\n\n" .
                     'Depending on the kind of a type, certain fields describe ' .
                     'information about that type. Scalar types provide no information ' .
                     'beyond a name and description, while Enum types provide their values. ' .
@@ -401,7 +401,7 @@ 
                     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;
@@ -431,12 +431,12 @@ 
                             '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) {
+                                        $fields = array_filter($fields, function(FieldDefinition $field) {
                                             return !$field->deprecationReason;
                                         });
                                     }
@@ -447,7 +447,7 @@ 
                         ],
                         'interfaces' => [
                             'type' => Type::listOf(Type::nonNull(self::_type())),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof ObjectType) {
                                     return $type->getInterfaces();
                                 }
@@ -456,7 +456,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);
                                 }
@@ -468,12 +468,12 @@ 
                             '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) {
+                                        $values = array_filter($values, function($value) {
                                             return !$value->deprecationReason;
                                         });
                                     }
@@ -485,7 +485,7 @@ 
                         ],
                         'inputFields' => [
                             'type' => Type::listOf(Type::nonNull(self::_inputValue())),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof InputObjectType) {
                                     return array_values($type->getFields());
                                 }
@@ -494,7 +494,7 @@ 
                         ],
                         'ofType' => [
                             'type' => self::_type(),
-                            'resolve' => function ($type) {
+                            'resolve' => function($type) {
                                 if ($type instanceof WrappingType) {
                                     return $type->getWrappedType();
                                 }
@@ -524,19 +524,19 @@ 
                         '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) {
+                            'resolve' => function(FieldDefinition $field) {
                                 return !!$field->deprecationReason;
                             }
                         ],
@@ -566,7 +566,7 @@ 
                         '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;
                             }
                         ],
@@ -574,7 +574,7 @@ 
                             '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()
                                     ? null
@@ -603,7 +603,7 @@ 
                     'description' => ['type' => Type::string()],
                     'isDeprecated' => [
                         'type' => Type::nonNull(Type::boolean()),
-                        'resolve' => function ($enumValue) {
+                        'resolve' => function($enumValue) {
                             return !!$enumValue->deprecationReason;
                         }
                     ],
@@ -670,7 +670,7 @@ 
                 'type' => Type::nonNull(self::_schema()),
                 'description' => 'Access the current type schema of this server.',
                 'args' => [],
-                'resolve' => function (
+                'resolve' => function(
                     $source,
                     $args,
                     $context,
@@ -693,7 +693,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']);
                 }
             ]);
@@ -709,7 +709,7 @@ 
                 '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/SchemaValidationContext.php

Spacing +24 added lines, -24 removed lines

@@ -98,7 +98,7 @@ 
         if ($astNode instanceof SchemaDefinitionNode) {
             $operationTypeNode = null;
 
-            foreach($astNode->operationTypes as $operationType) {
+            foreach ($astNode->operationTypes as $operationType) {
                 if ($operationType->operation === $operation) {
                     $operationTypeNode = $operationType;
                     break;
@@ -112,7 +112,7 @@ 
     public function validateDirectives()
     {
         $directives = $this->schema->getDirectives();
-        foreach($directives as $directive) {
+        foreach ($directives as $directive) {
             // Ensure all directives are in fact GraphQL directives.
             if (!$directive instanceof Directive) {
                 $this->reportError(
@@ -171,7 +171,7 @@ 
     public function validateTypes()
     {
         $typeMap = $this->schema->getTypeMap();
-        foreach($typeMap as $typeName => $type) {
+        foreach ($typeMap as $typeName => $type) {
             // Ensure all provided types are in fact GraphQL type.
             if (!$type instanceof NamedType) {
                 $this->reportError(
@@ -244,7 +244,7 @@ 
 
             // Ensure the arguments are valid
             $argNames = [];
-            foreach($field->args as $arg) {
+            foreach ($field->args as $arg) {
                 $argName = $arg->name;
 
                 // Ensure they are named correctly.
@@ -263,7 +263,7 @@ 
                 if (!Type::isInputType($arg->getType())) {
                     $this->reportError(
                         "The type of {$type->name}.{$fieldName}({$argName}:) must be Input " .
-                        'Type but got: '. Utils::printSafe($arg->getType()) . '.',
+                        'Type but got: ' . Utils::printSafe($arg->getType()) . '.',
                         $this->getFieldArgTypeNode($type, $fieldName, $argName)
                     );
                 }
@@ -273,7 +273,7 @@ 
 
     private function validateObjectInterfaces(ObjectType $object) {
         $implementedTypeNames = [];
-        foreach($object->getInterfaces() as $iface) {
+        foreach ($object->getInterfaces() as $iface) {
             if (isset($implementedTypeNames[$iface->name])) {
                 $this->reportError(
                     "Type {$object->name} can only implement {$iface->name} once.",
@@ -295,7 +295,7 @@ 
         if (!$iface instanceof InterfaceType) {
             $this->reportError(
                 "Type {$object->name} must only implement Interface types, " .
-                "it cannot implement ". Utils::printSafe($iface) . ".",
+                "it cannot implement " . Utils::printSafe($iface) . ".",
                 $this->getImplementsInterfaceNode($object, $iface)
             );
             return;
@@ -330,7 +330,7 @@ 
                 )
             ) {
                 $this->reportError(
-                    "Interface field {$iface->name}.{$fieldName} expects type ".
+                    "Interface field {$iface->name}.{$fieldName} expects type " .
                     "{$ifaceField->getType()} but {$object->name}.{$fieldName} " .
                     "is type " . Utils::printSafe($objectField->getType()) . ".",
                     [
@@ -341,11 +341,11 @@ 
             }
 
             // Assert each interface field arg is implemented.
-            foreach($ifaceField->args as $ifaceArg) {
+            foreach ($ifaceField->args as $ifaceArg) {
                 $argName = $ifaceArg->name;
                 $objectArg = null;
 
-                foreach($objectField->args as $arg) {
+                foreach ($objectField->args as $arg) {
                     if ($arg->name === $argName) {
                         $objectArg = $arg;
                         break;
@@ -370,7 +370,7 @@ 
                 // TODO: change to contravariant?
                 if (!TypeComparators::isEqualType($ifaceArg->getType(), $objectArg->getType())) {
                     $this->reportError(
-                        "Interface field argument {$iface->name}.{$fieldName}({$argName}:) ".
+                        "Interface field argument {$iface->name}.{$fieldName}({$argName}:) " .
                         "expects type " . Utils::printSafe($ifaceArg->getType()) . " but " .
                         "{$object->name}.{$fieldName}({$argName}:) is type " .
                         Utils::printSafe($objectArg->getType()) . ".",
@@ -385,11 +385,11 @@ 
             }
 
             // Assert additional arguments must not be required.
-            foreach($objectField->args as $objectArg) {
+            foreach ($objectField->args as $objectArg) {
                 $argName = $objectArg->name;
                 $ifaceArg = null;
 
-                foreach($ifaceField->args as $arg) {
+                foreach ($ifaceField->args as $arg) {
                     if ($arg->name === $argName) {
                         $ifaceArg = $arg;
                         break;
@@ -424,10 +424,10 @@ 
 
         $includedTypeNames = [];
 
-        foreach($memberTypes as $memberType) {
+        foreach ($memberTypes as $memberType) {
             if (isset($includedTypeNames[$memberType->name])) {
                 $this->reportError(
-                    "Union type {$union->name} can only include type ".
+                    "Union type {$union->name} can only include type " .
                     "{$memberType->name} once.",
                     $this->getUnionMemberTypeNodes($union, $memberType->name)
                 );
@@ -436,7 +436,7 @@ 
             $includedTypeNames[$memberType->name] = true;
             if (!$memberType instanceof ObjectType) {
                 $this->reportError(
-                    "Union type {$union->name} can only include Object types, ".
+                    "Union type {$union->name} can only include Object types, " .
                     "it cannot include " . Utils::printSafe($memberType) . ".",
                     $this->getUnionMemberTypeNodes($union, Utils::printSafe($memberType))
                 );
@@ -455,7 +455,7 @@ 
             );
         }
 
-        foreach($enumValues as $enumValue) {
+        foreach ($enumValues as $enumValue) {
             $valueName = $enumValue->name;
 
             // Ensure no duplicates
@@ -541,9 +541,9 @@ 
         $implementsNodes = [];
         $astNodes = $this->getAllObjectOrInterfaceNodes($type);
 
-        foreach($astNodes as $astNode) {
+        foreach ($astNodes as $astNode) {
             if ($astNode && $astNode->interfaces) {
-                foreach($astNode->interfaces as $node) {
+                foreach ($astNode->interfaces as $node) {
                     if ($node->name->value === $iface->name) {
                         $implementsNodes[] = $node;
                     }
@@ -574,9 +574,9 @@ 
     {
         $fieldNodes = [];
         $astNodes = $this->getAllObjectOrInterfaceNodes($type);
-        foreach($astNodes as $astNode) {
+        foreach ($astNodes as $astNode) {
             if ($astNode && $astNode->fields) {
-                foreach($astNode->fields as $node) {
+                foreach ($astNode->fields as $node) {
                     if ($node->name->value === $fieldName) {
                         $fieldNodes[] = $node;
                     }
@@ -651,7 +651,7 @@ 
         $argNodes = [];
         $directiveNode = $directive->astNode;
         if ($directiveNode && $directiveNode->arguments) {
-            foreach($directiveNode->arguments as $node) {
+            foreach ($directiveNode->arguments as $node) {
                 if ($node->name->value === $argName) {
                     $argNodes[] = $node;
                 }
@@ -682,7 +682,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;
                 }
             );
@@ -701,7 +701,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;
                 }
             );

src/Type/Schema.php

Spacing +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
                 if (isset($this->resolvedTypes[$type->name])) {
                     Utils::invariant(
                         $type === $this->resolvedTypes[$type->name],
-                        "Schema must contain unique named types but contains multiple types named \"$type\" ".
+                        "Schema must contain unique named types but contains multiple types named \"$type\" " .
                         "(see http://webonyx.github.io/graphql-php/type-system/#type-registry)."
                     );
                 }
@@ -443,7 +443,7 @@ 
         $internalTypes = Type::getInternalTypes() + Introspection::getTypes();
         foreach ($this->getTypeMap() as $name => $type) {
             if (isset($internalTypes[$name])) {
-                continue ;
+                continue;
             }
 
             $type->assertValid();
@@ -452,7 +452,7 @@ 
             if ($this->config->typeLoader) {
                 Utils::invariant(
                     $this->loadType($name) === $type,
-                    "Type loader returns different instance for {$name} than field/argument definitions. ".
+                    "Type loader returns different instance for {$name} than field/argument definitions. " .
                     'Make sure you always return the same instance for the same type name.'
                 );
             }

src/Validator/Rules/NoUnusedFragments.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
                         if (empty($fragmentNameUsed[$fragName])) {
                             $context->reportError(new Error(
                                 self::unusedFragMessage($fragName),
-                                [ $fragmentDef ]
+                                [$fragmentDef]
                             ));
                         }
                     }

src/Validator/Rules/ExecutableDefinitions.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::DOCUMENT => function (DocumentNode $node) use ($context) {
+            NodeKind::DOCUMENT => function(DocumentNode $node) use ($context) {
                 /** @var Node $definition */
                 foreach ($node->definitions as $definition) {
                     if (

src/Validator/Rules/DisableIntrospection.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         return $this->invokeIfNeeded(
             $context,
             [
-                NodeKind::FIELD => function (FieldNode $node) use ($context) {
+                NodeKind::FIELD => function(FieldNode $node) use ($context) {
                     if ($node->name->value === '__type' || $node->name->value === '__schema') {
                         $context->reportError(new Error(
                             static::introspectionDisabledMessage(),

src/Validator/Rules/UniqueOperationNames.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     static function duplicateOperationNameMessage($operationName)
     {
-      return "There can be only one operation named \"$operationName\".";
+        return "There can be only one operation named \"$operationName\".";
     }
 
     public $knownOperationNames;

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
                     if (!empty($this->knownOperationNames[$operationName->value])) {
                         $context->reportError(new Error(
                             self::duplicateOperationNameMessage($operationName->value),
-                            [ $this->knownOperationNames[$operationName->value], $operationName ]
+                            [$this->knownOperationNames[$operationName->value], $operationName]
                         ));
                     } else {
                         $this->knownOperationNames[$operationName->value] = $operationName;