simPod / graphql-php

src/Server/Helper.php

Spacing +24 added lines, -24 removed lines

@@ -87,7 +87,7 @@ 
                     throw new RequestError('Could not parse JSON: ' . json_last_error_msg());
                 }
 
-                if (! is_array($bodyParams)) {
+                if (!is_array($bodyParams)) {
                     throw new RequestError(
                         'GraphQL Server expects JSON object or array, but got ' .
                         Utils::printSafeJson($bodyParams)
@@ -153,7 +153,7 @@ 
     public function validateOperationParams(OperationParams $params)
     {
         $errors = [];
-        if (! $params->query && ! $params->queryId) {
+        if (!$params->query && !$params->queryId) {
             $errors[] = new RequestError('GraphQL Request must include at least one of those two parameters: "query" or "queryId"');
         }
 
@@ -161,28 +161,28 @@ 
             $errors[] = new RequestError('GraphQL Request parameters "query" and "queryId" are mutually exclusive');
         }
 
-        if ($params->query !== null && (! is_string($params->query) || empty($params->query))) {
+        if ($params->query !== null && (!is_string($params->query) || empty($params->query))) {
             $errors[] = new RequestError(
                 'GraphQL Request parameter "query" must be string, but got ' .
                 Utils::printSafeJson($params->query)
             );
         }
 
-        if ($params->queryId !== null && (! is_string($params->queryId) || empty($params->queryId))) {
+        if ($params->queryId !== null && (!is_string($params->queryId) || empty($params->queryId))) {
             $errors[] = new RequestError(
                 'GraphQL Request parameter "queryId" must be string, but got ' .
                 Utils::printSafeJson($params->queryId)
             );
         }
 
-        if ($params->operation !== null && (! is_string($params->operation) || empty($params->operation))) {
+        if ($params->operation !== null && (!is_string($params->operation) || empty($params->operation))) {
             $errors[] = new RequestError(
                 'GraphQL Request parameter "operation" must be string, but got ' .
                 Utils::printSafeJson($params->operation)
             );
         }
 
-        if ($params->variables !== null && (! is_array($params->variables) || isset($params->variables[0]))) {
+        if ($params->variables !== null && (!is_array($params->variables) || isset($params->variables[0]))) {
             $errors[] = new RequestError(
                 'GraphQL Request parameter "variables" must be object or JSON string parsed to object, but got ' .
                 Utils::printSafeJson($params->getOriginalInput('variables'))
@@ -253,20 +253,20 @@ 
         $isBatch = false
     ) {
         try {
-            if (! $config->getSchema()) {
+            if (!$config->getSchema()) {
                 throw new InvariantViolation('Schema is required for the server');
             }
 
-            if ($isBatch && ! $config->getQueryBatching()) {
+            if ($isBatch && !$config->getQueryBatching()) {
                 throw new RequestError('Batched queries are not supported by this server');
             }
 
             $errors = $this->validateOperationParams($op);
 
-            if (! empty($errors)) {
+            if (!empty($errors)) {
                 $errors = Utils::map(
                     $errors,
-                    static function (RequestError $err) {
+                    static function(RequestError $err) {
                         return Error::createLocatedError($err, null, null);
                     }
                 );
@@ -280,7 +280,7 @@ 
                 ? $this->loadPersistedQuery($config, $op)
                 : $op->query;
 
-            if (! $doc instanceof DocumentNode) {
+            if (!$doc instanceof DocumentNode) {
                 $doc = Parser::parse($doc);
             }
 
@@ -315,7 +315,7 @@ 
             );
         }
 
-        $applyErrorHandling = static function (ExecutionResult $result) use ($config) {
+        $applyErrorHandling = static function(ExecutionResult $result) use ($config) {
             if ($config->getErrorsHandler()) {
                 $result->setErrorsHandler($config->getErrorsHandler());
             }
@@ -344,13 +344,13 @@ 
         // Load query if we got persisted query id:
         $loader = $config->getPersistentQueryLoader();
 
-        if (! $loader) {
+        if (!$loader) {
             throw new RequestError('Persisted queries are not supported by this server');
         }
 
         $source = $loader($operationParams->queryId, $operationParams);
 
-        if (! is_string($source) && ! $source instanceof DocumentNode) {
+        if (!is_string($source) && !$source instanceof DocumentNode) {
             throw new InvariantViolation(sprintf(
                 'Persistent query loader must return query string or instance of %s but got: %s',
                 DocumentNode::class,
@@ -378,7 +378,7 @@ 
         if (is_callable($validationRules)) {
             $validationRules = $validationRules($params, $doc, $operationType);
 
-            if (! is_array($validationRules)) {
+            if (!is_array($validationRules)) {
                 throw new InvariantViolation(sprintf(
                     'Expecting validation rules to be array or callable returning array, but got: %s',
                     Utils::printSafe($validationRules)
@@ -434,7 +434,7 @@ 
     public function sendResponse($result, $exitWhenDone = false)
     {
         if ($result instanceof Promise) {
-            $result->then(function ($actualResult) use ($exitWhenDone) {
+            $result->then(function($actualResult) use ($exitWhenDone) {
                 $this->doSendResponse($actualResult, $exitWhenDone);
             });
         } else {
@@ -482,8 +482,8 @@ 
         if (is_array($result) && isset($result[0])) {
             Utils::each(
                 $result,
-                static function ($executionResult, $index) {
-                    if (! $executionResult instanceof ExecutionResult) {
+                static 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',
                             ExecutionResult::class,
@@ -495,14 +495,14 @@ 
             );
             $httpStatus = 200;
         } else {
-            if (! $result instanceof ExecutionResult) {
+            if (!$result instanceof ExecutionResult) {
                 throw new InvariantViolation(sprintf(
                     'Expecting query result to be instance of %s but got %s',
                     ExecutionResult::class,
                     Utils::printSafe($result)
                 ));
             }
-            if ($result->data === null && ! empty($result->errors)) {
+            if ($result->data === null && !empty($result->errors)) {
                 $httpStatus = 400;
             } else {
                 $httpStatus = 200;
@@ -528,7 +528,7 @@ 
         } else {
             $contentType = $request->getHeader('content-type');
 
-            if (! isset($contentType[0])) {
+            if (!isset($contentType[0])) {
                 throw new RequestError('Missing "Content-Type" header');
             }
 
@@ -543,7 +543,7 @@ 
                     );
                 }
 
-                if (! is_array($bodyParams)) {
+                if (!is_array($bodyParams)) {
                     throw new RequestError(
                         'GraphQL Server expects JSON object or array, but got ' .
                         Utils::printSafeJson($bodyParams)
@@ -552,7 +552,7 @@ 
             } else {
                 $bodyParams = $request->getParsedBody();
 
-                if (! is_array($bodyParams)) {
+                if (!is_array($bodyParams)) {
                     throw new RequestError('Unexpected content type: ' . Utils::printSafeJson($contentType[0]));
                 }
             }
@@ -577,7 +577,7 @@ 
     public function toPsrResponse($result, ResponseInterface $response, StreamInterface $writableBodyStream)
     {
         if ($result instanceof Promise) {
-            return $result->then(function ($actualResult) use ($response, $writableBodyStream) {
+            return $result->then(function($actualResult) use ($response, $writableBodyStream) {
                 return $this->doConvertToPsrResponse($actualResult, $response, $writableBodyStream);
             });
         }

src/Language/Parser.php

Spacing +18 added lines, -18 removed lines

@@ -369,7 +369,7 @@ 
         $this->expect($openKind);
 
         $nodes = [];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -389,7 +389,7 @@ 
         $this->expect($openKind);
 
         $nodes = [$parseFn($this)];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -423,7 +423,7 @@ 
         return new DocumentNode([
             'definitions' => $this->many(
                 Token::SOF,
-                function () {
+                function() {
                     return $this->parseDefinition();
                 },
                 Token::EOF
@@ -550,7 +550,7 @@ 
         return $this->peek(Token::PAREN_L)
             ? $this->many(
                 Token::PAREN_L,
-                function () {
+                function() {
                     return $this->parseVariableDefinition();
                 },
                 Token::PAREN_R
@@ -601,7 +601,7 @@ 
             [
                 'selections' => $this->many(
                     Token::BRACE_L,
-                    function () {
+                    function() {
                         return $this->parseSelection();
                     },
                     Token::BRACE_R
@@ -656,10 +656,10 @@ 
     private function parseArguments(bool $isConst) : NodeList
     {
         $parseFn = $isConst
-            ? function () {
+            ? function() {
                 return $this->parseConstArgument();
             }
-            : function () {
+            : function() {
                 return $this->parseArgument();
             };
 
@@ -856,7 +856,7 @@ 
                 break;
 
             case Token::DOLLAR:
-                if (! $isConst) {
+                if (!$isConst) {
                     return $this->parseVariable();
                 }
                 break;
@@ -897,9 +897,9 @@ 
     private function parseArray(bool $isConst) : ListValueNode
     {
         $start   = $this->lexer->token;
-        $parseFn = $isConst ? function () {
+        $parseFn = $isConst ? function() {
             return $this->parseConstValue();
-        } : function () {
+        } : function() {
             return $this->parseVariableValue();
         };
 
@@ -916,7 +916,7 @@ 
         $start = $this->lexer->token;
         $this->expect(Token::BRACE_L);
         $fields = [];
-        while (! $this->skip(Token::BRACE_R)) {
+        while (!$this->skip(Token::BRACE_R)) {
             $fields[] = $this->parseObjectField($isConst);
         }
 
@@ -1090,7 +1090,7 @@ 
 
         $operationTypes = $this->many(
             Token::BRACE_L,
-            function () {
+            function() {
                 return $this->parseOperationTypeDefinition();
             },
             Token::BRACE_R
@@ -1180,7 +1180,7 @@ 
                 $types[] = $this->parseNamedType();
             } while ($this->skip(Token::AMP) ||
                 // Legacy support for the SDL?
-                (! empty($this->lexer->options['allowLegacySDLImplementsInterfaces']) && $this->peek(Token::NAME))
+                (!empty($this->lexer->options['allowLegacySDLImplementsInterfaces']) && $this->peek(Token::NAME))
             );
         }
 
@@ -1193,7 +1193,7 @@ 
     private function parseFieldsDefinition() : NodeList
     {
         // Legacy support for the SDL?
-        if (! empty($this->lexer->options['allowLegacySDLEmptyFields'])
+        if (!empty($this->lexer->options['allowLegacySDLEmptyFields'])
             && $this->peek(Token::BRACE_L)
             && $this->lexer->lookahead()->kind === Token::BRACE_R
         ) {
@@ -1207,7 +1207,7 @@ 
             $nodeList = $this->peek(Token::BRACE_L)
                 ? $this->many(
                     Token::BRACE_L,
-                    function () {
+                    function() {
                         return $this->parseFieldDefinition();
                     },
                     Token::BRACE_R
@@ -1250,7 +1250,7 @@ 
         $nodeList = $this->peek(Token::PAREN_L)
             ? $this->many(
                 Token::PAREN_L,
-                function () {
+                function() {
                     return $this->parseInputValueDefinition();
                 },
                 Token::PAREN_R
@@ -1382,7 +1382,7 @@ 
         $nodeList = $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseEnumValueDefinition();
                 },
                 Token::BRACE_R
@@ -1440,7 +1440,7 @@ 
         $nodeList = $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseInputValueDefinition();
                 },
                 Token::BRACE_R

src/Utils/TypeInfo.php

Spacing +9 added lines, -9 removed lines

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

src/Validator/Rules/KnownArgumentNames.php

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::ARGUMENT => static function (ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
+            NodeKind::ARGUMENT => static function(ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
                 $argDef = $context->getArgument();
                 if ($argDef !== null) {
                     return null;
@@ -48,7 +48,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $fieldDef->args
@@ -68,7 +68,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        static function ($arg) {
+                                        static function($arg) {
                                             return $arg->name;
                                         },
                                         $directive->args
@@ -89,7 +89,7 @@ 
     public static function unknownArgMessage($argName, $fieldName, $typeName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on field "%s" of type "%s".', $argName, $fieldName, $typeName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 
@@ -102,7 +102,7 @@ 
     public static function unknownDirectiveArgMessage($argName, $directiveName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on directive "@%s".', $argName, $directiveName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 

src/Validator/Rules/ProvidedRequiredArgumentsOnDirectives.php

Spacing +10 added lines, -10 removed lines

@@ -45,10 +45,10 @@ 
 
         foreach ($definedDirectives as $directive) {
             $requiredArgsMap[$directive->name] = Utils::keyMap(
-                array_filter($directive->args, static function (FieldArgument $arg) : bool {
-                    return $arg->getType() instanceof NonNull && ! isset($arg->defaultValue);
+                array_filter($directive->args, static function(FieldArgument $arg) : bool {
+                    return $arg->getType() instanceof NonNull && !isset($arg->defaultValue);
                 }),
-                static function (FieldArgument $arg) : string {
+                static function(FieldArgument $arg) : string {
                     return $arg->name;
                 }
             );
@@ -56,7 +56,7 @@ 
 
         $astDefinition = $context->getDocument()->definitions;
         foreach ($astDefinition as $def) {
-            if (! ($def instanceof DirectiveDefinitionNode)) {
+            if (!($def instanceof DirectiveDefinitionNode)) {
                 continue;
             }
 
@@ -70,32 +70,32 @@ 
 
             $requiredArgsMap[$def->name->value] = Utils::keyMap(
                 $arguments
-                    ? array_filter($arguments, static function (Node $argument) : bool {
+                    ? array_filter($arguments, static function(Node $argument) : bool {
                         return $argument instanceof NonNullTypeNode &&
                             (
-                                ! isset($argument->defaultValue) ||
+                                !isset($argument->defaultValue) ||
                                 $argument->defaultValue === null
                             );
                     })
                     : [],
-                static function (NamedTypeNode $argument) : string {
+                static function(NamedTypeNode $argument) : string {
                     return $argument->name->value;
                 }
             );
         }
 
         return [
-            NodeKind::DIRECTIVE => static function (DirectiveNode $directiveNode) use ($requiredArgsMap, $context) {
+            NodeKind::DIRECTIVE => static function(DirectiveNode $directiveNode) use ($requiredArgsMap, $context) {
                 $directiveName = $directiveNode->name->value;
                 $requiredArgs  = $requiredArgsMap[$directiveName] ?? null;
-                if (! $requiredArgs) {
+                if (!$requiredArgs) {
                     return null;
                 }
 
                 $argNodes   = $directiveNode->arguments ?: [];
                 $argNodeMap = Utils::keyMap(
                     $argNodes,
-                    static function (ArgumentNode $arg) : string {
+                    static function(ArgumentNode $arg) : string {
                         return $arg->name->value;
                     }
                 );