simPod / graphql-php

src/Server/OperationParams.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
 
         if (is_string($params['variables'])) {
             $tmp = json_decode($params['variables'], true);
-            if (! json_last_error()) {
+            if (!json_last_error()) {
                 $params['variables'] = $tmp;
             }
         }

src/Server/Helper.php

Spacing +24 added lines, -24 removed lines

@@ -80,7 +80,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)
@@ -142,7 +142,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"');
         }
 
@@ -150,28 +150,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'))
@@ -239,20 +239,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,
-                    function (RequestError $err) {
+                    function(RequestError $err) {
                         return Error::createLocatedError($err, null, null);
                     }
                 );
@@ -264,7 +264,7 @@ 
 
             $doc = $op->queryId ? $this->loadPersistedQuery($config, $op) : $op->query;
 
-            if (! $doc instanceof DocumentNode) {
+            if (!$doc instanceof DocumentNode) {
                 $doc = Parser::parse($doc);
             }
 
@@ -294,7 +294,7 @@ 
             );
         }
 
-        $applyErrorHandling = function (ExecutionResult $result) use ($config) {
+        $applyErrorHandling = function(ExecutionResult $result) use ($config) {
             if ($config->getErrorsHandler()) {
                 $result->setErrorsHandler($config->getErrorsHandler());
             }
@@ -322,13 +322,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,
@@ -355,7 +355,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)
@@ -410,7 +410,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 {
@@ -457,8 +457,8 @@ 
         if (is_array($result) && isset($result[0])) {
             Utils::each(
                 $result,
-                function ($executionResult, $index) {
-                    if (! $executionResult instanceof ExecutionResult) {
+                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,
@@ -470,14 +470,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;
@@ -501,7 +501,7 @@ 
         } else {
             $contentType = $request->getHeader('content-type');
 
-            if (! isset($contentType[0])) {
+            if (!isset($contentType[0])) {
                 throw new RequestError('Missing "Content-Type" header');
             }
 
@@ -516,7 +516,7 @@ 
                     );
                 }
 
-                if (! is_array($bodyParams)) {
+                if (!is_array($bodyParams)) {
                     throw new RequestError(
                         'GraphQL Server expects JSON object or array, but got ' .
                         Utils::printSafeJson($bodyParams)
@@ -525,7 +525,7 @@ 
             } else {
                 $bodyParams = $request->getParsedBody();
 
-                if (! is_array($bodyParams)) {
+                if (!is_array($bodyParams)) {
                     throw new RequestError('Unexpected content type: ' . Utils::printSafeJson($contentType[0]));
                 }
             }
@@ -548,7 +548,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/Server/ServerConfig.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $instance = new static();
         foreach ($config as $key => $value) {
             $method = 'set' . ucfirst($key);
-            if (! method_exists($instance, $method)) {
+            if (!method_exists($instance, $method)) {
                 throw new InvariantViolation(sprintf('Unknown server config option "%s"', $key));
             }
             $instance->$method($value);
@@ -155,7 +155,7 @@ 
      */
     public function setValidationRules($validationRules)
     {
-        if (! is_callable($validationRules) && ! is_array($validationRules) && $validationRules !== null) {
+        if (!is_callable($validationRules) && !is_array($validationRules) && $validationRules !== null) {
             throw new InvariantViolation(
                 'Server config expects array of validation rules or callable returning such array, but got ' .
                 Utils::printSafe($validationRules)

src/Error/FormattedError.php

Spacing +18 added lines, -18 removed lines

@@ -65,7 +65,7 @@ 
         if ($error->nodes) {
             /** @var Node $node */
             foreach ($error->nodes as $node) {
-                if (! $node->loc) {
+                if (!$node->loc) {
                     continue;
                 }
 
@@ -85,7 +85,7 @@ 
             }
         }
 
-        return ! $printedLocations
+        return !$printedLocations
             ? $error->getMessage()
             : implode("\n\n", array_merge([$error->getMessage()], $printedLocations)) . "\n";
     }
@@ -189,17 +189,17 @@ 
         if ($e instanceof Error) {
             $locations = Utils::map(
                 $e->getLocations(),
-                function (SourceLocation $loc) {
+                function(SourceLocation $loc) {
                     return $loc->toSerializableArray();
                 }
             );
-            if (! empty($locations)) {
+            if (!empty($locations)) {
                 $formattedError['locations'] = $locations;
             }
-            if (! empty($e->path)) {
+            if (!empty($e->path)) {
                 $formattedError['path'] = $e->path;
             }
-            if (! empty($e->getExtensions())) {
+            if (!empty($e->getExtensions())) {
                 $formattedError['extensions'] = $e->getExtensions();
             }
         }
@@ -223,7 +223,7 @@ 
      */
     public static function addDebugEntries(array $formattedError, $e, $debug)
     {
-        if (! $debug) {
+        if (!$debug) {
             return $formattedError;
         }
 
@@ -236,7 +236,7 @@ 
         $debug = (int) $debug;
 
         if ($debug & Debug::RETHROW_INTERNAL_EXCEPTIONS) {
-            if (! $e instanceof Error) {
+            if (!$e instanceof Error) {
                 throw $e;
             }
 
@@ -245,7 +245,7 @@ 
             }
         }
 
-        $isInternal = ! $e instanceof ClientAware || ! $e->isClientSafe();
+        $isInternal = !$e instanceof ClientAware || !$e->isClientSafe();
 
         if (($debug & Debug::INCLUDE_DEBUG_MESSAGE) && $isInternal) {
             // Displaying debugMessage as a first entry:
@@ -260,9 +260,9 @@ 
                 ];
             }
 
-            $isTrivial = $e instanceof Error && ! $e->getPrevious();
+            $isTrivial = $e instanceof Error && !$e->getPrevious();
 
-            if (! $isTrivial) {
+            if (!$isTrivial) {
                 $debugging               = $e->getPrevious() ?: $e;
                 $formattedError['trace'] = static::toSafeTrace($debugging);
             }
@@ -280,11 +280,11 @@ 
      */
     public static function prepareFormatter(?callable $formatter = null, $debug)
     {
-        $formatter = $formatter ?: function ($e) {
+        $formatter = $formatter ?: function($e) {
             return FormattedError::createFromException($e);
         };
         if ($debug) {
-            $formatter = function ($e) use ($formatter, $debug) {
+            $formatter = function($e) use ($formatter, $debug) {
                 return FormattedError::addDebugEntries($formatter($e), $e, $debug);
             };
         }
@@ -307,18 +307,18 @@ 
             // Remove invariant entries as they don't provide much value:
             ($trace[0]['class'] . '::' . $trace[0]['function'] === 'GraphQL\Utils\Utils::invariant')) {
             array_shift($trace);
-        } elseif (! isset($trace[0]['file'])) {
+        } elseif (!isset($trace[0]['file'])) {
             // Remove root call as it's likely error handler trace:
             array_shift($trace);
         }
 
         return array_map(
-            function ($err) {
+            function($err) {
                 $safeErr = array_intersect_key($err, ['file' => true, 'line' => true]);
 
                 if (isset($err['function'])) {
                     $func    = $err['function'];
-                    $args    = ! empty($err['args']) ? array_map([__CLASS__, 'printVar'], $err['args']) : [];
+                    $args    = !empty($err['args']) ? array_map([__CLASS__, 'printVar'], $err['args']) : [];
                     $funcStr = $func . '(' . implode(', ', $args) . ')';
 
                     if (isset($err['class'])) {
@@ -384,9 +384,9 @@ 
     {
         $formatted = ['message' => $error];
 
-        if (! empty($locations)) {
+        if (!empty($locations)) {
             $formatted['locations'] = array_map(
-                function ($loc) {
+                function($loc) {
                     return $loc->toArray();
                 },
                 $locations

src/Error/Error.php

Spacing +11 added lines, -11 removed lines

@@ -102,7 +102,7 @@ 
         // Compute list of blame nodes.
         if ($nodes instanceof \Traversable) {
             $nodes = iterator_to_array($nodes);
-        } elseif ($nodes && ! is_array($nodes)) {
+        } elseif ($nodes && !is_array($nodes)) {
             $nodes = [$nodes];
         }
 
@@ -207,7 +207,7 @@ 
     public function getSource()
     {
         if ($this->source === null) {
-            if (! empty($this->nodes[0]) && ! empty($this->nodes[0]->loc)) {
+            if (!empty($this->nodes[0]) && !empty($this->nodes[0]->loc)) {
                 $this->source = $this->nodes[0]->loc->source;
             }
         }
@@ -220,9 +220,9 @@ 
      */
     public function getPositions()
     {
-        if ($this->positions === null && ! empty($this->nodes)) {
+        if ($this->positions === null && !empty($this->nodes)) {
             $positions = array_map(
-                function ($node) {
+                function($node) {
                     return isset($node->loc) ? $node->loc->start : null;
                 },
                 $this->nodes
@@ -230,7 +230,7 @@ 
 
             $this->positions = array_filter(
                 $positions,
-                function ($p) {
+                function($p) {
                     return $p !== null;
                 }
             );
@@ -262,7 +262,7 @@ 
 
             if ($positions && $source) {
                 $this->locations = array_map(
-                    function ($pos) use ($source) {
+                    function($pos) use ($source) {
                         return $source->getLocation($pos);
                     },
                     $positions
@@ -270,7 +270,7 @@ 
             } elseif ($nodes) {
                 $this->locations = array_filter(
                     array_map(
-                        function ($node) {
+                        function($node) {
                             if ($node->loc) {
                                 return $node->loc->source->getLocation($node->loc->start);
                             }
@@ -328,18 +328,18 @@ 
 
         $locations = Utils::map(
             $this->getLocations(),
-            function (SourceLocation $loc) {
+            function(SourceLocation $loc) {
                 return $loc->toSerializableArray();
             }
         );
 
-        if (! empty($locations)) {
+        if (!empty($locations)) {
             $arr['locations'] = $locations;
         }
-        if (! empty($this->path)) {
+        if (!empty($this->path)) {
             $arr['path'] = $this->path;
         }
-        if (! empty($this->extensions)) {
+        if (!empty($this->extensions)) {
             $arr['extensions'] = $this->extensions;
         }
 

src/Validator/ValidationContext.php

Spacing +11 added lines, -11 removed lines

@@ -97,7 +97,7 @@ 
     {
         $usages = $this->recursiveVariableUsages[$operation] ?? null;
 
-        if (! $usages) {
+        if (!$usages) {
             $usages    = $this->getVariableUsages($operation);
             $fragments = $this->getRecursivelyReferencedFragments($operation);
 
@@ -119,7 +119,7 @@ 
     {
         $usages = $this->variableUsages[$node] ?? null;
 
-        if (! $usages) {
+        if (!$usages) {
             $newUsages = [];
             $typeInfo  = new TypeInfo($this->schema);
             Visitor::visit(
@@ -127,10 +127,10 @@ 
                 Visitor::visitWithTypeInfo(
                     $typeInfo,
                     [
-                        NodeKind::VARIABLE_DEFINITION => function () {
+                        NodeKind::VARIABLE_DEFINITION => function() {
                             return false;
                         },
-                        NodeKind::VARIABLE            => function (VariableNode $variable) use (
+                        NodeKind::VARIABLE            => function(VariableNode $variable) use (
                             &$newUsages,
                             $typeInfo
                         ) {
@@ -153,23 +153,23 @@ 
     {
         $fragments = $this->recursivelyReferencedFragments[$operation] ?? null;
 
-        if (! $fragments) {
+        if (!$fragments) {
             $fragments      = [];
             $collectedNames = [];
             $nodesToVisit   = [$operation];
-            while (! empty($nodesToVisit)) {
+            while (!empty($nodesToVisit)) {
                 $node    = array_pop($nodesToVisit);
                 $spreads = $this->getFragmentSpreads($node);
                 for ($i = 0; $i < count($spreads); $i++) {
                     $fragName = $spreads[$i]->name->value;
 
-                    if (! empty($collectedNames[$fragName])) {
+                    if (!empty($collectedNames[$fragName])) {
                         continue;
                     }
 
                     $collectedNames[$fragName] = true;
                     $fragment                  = $this->getFragment($fragName);
-                    if (! $fragment) {
+                    if (!$fragment) {
                         continue;
                     }
 
@@ -189,10 +189,10 @@ 
     public function getFragmentSpreads(HasSelectionSet $node)
     {
         $spreads = $this->fragmentSpreads[$node] ?? null;
-        if (! $spreads) {
+        if (!$spreads) {
             $spreads     = [];
             $setsToVisit = [$node->selectionSet];
-            while (! empty($setsToVisit)) {
+            while (!empty($setsToVisit)) {
                 $set = array_pop($setsToVisit);
 
                 for ($i = 0; $i < count($set->selections); $i++) {
@@ -217,7 +217,7 @@ 
     public function getFragment($name)
     {
         $fragments = $this->fragments;
-        if (! $fragments) {
+        if (!$fragments) {
             $fragments = [];
             foreach ($this->getDocument()->definitions as $statement) {
                 if ($statement->kind !== NodeKind::FRAGMENT_DEFINITION) {

src/Validator/Rules/NoUndefinedVariables.php

Spacing +4 added lines, -4 removed lines

@@ -25,17 +25,17 @@ 
 
         return [
             NodeKind::OPERATION_DEFINITION => [
-                'enter' => function () use (&$variableNameDefined) {
+                'enter' => function() use (&$variableNameDefined) {
                     $variableNameDefined = [];
                 },
-                'leave' => function (OperationDefinitionNode $operation) use (&$variableNameDefined, $context) {
+                'leave' => function(OperationDefinitionNode $operation) use (&$variableNameDefined, $context) {
                     $usages = $context->getRecursiveVariableUsages($operation);
 
                     foreach ($usages as $usage) {
                         $node    = $usage['node'];
                         $varName = $node->name->value;
 
-                        if (! empty($variableNameDefined[$varName])) {
+                        if (!empty($variableNameDefined[$varName])) {
                             continue;
                         }
 
@@ -49,7 +49,7 @@ 
                     }
                 },
             ],
-            NodeKind::VARIABLE_DEFINITION  => function (VariableDefinitionNode $def) use (&$variableNameDefined) {
+            NodeKind::VARIABLE_DEFINITION  => function(VariableDefinitionNode $def) use (&$variableNameDefined) {
                 $variableNameDefined[$def->variable->name->value] = true;
             },
         ];

src/Validator/Rules/VariablesAreInputTypes.php

Spacing +2 added lines, -2 removed lines

@@ -18,11 +18,11 @@
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::VARIABLE_DEFINITION => function (VariableDefinitionNode $node) use ($context) {
+            NodeKind::VARIABLE_DEFINITION => function(VariableDefinitionNode $node) use ($context) {
                 $type = TypeInfo::typeFromAST($context->getSchema(), $node->type);
 
                 // If the variable type is not an input type, return an error.
-                if (! $type || Type::isInputType($type)) {
+                if (!$type || Type::isInputType($type)) {
                     return;
                 }
 

src/Validator/Rules/NoUnusedVariables.php

Spacing +4 added lines, -4 removed lines

@@ -22,10 +22,10 @@ 
 
         return [
             NodeKind::OPERATION_DEFINITION => [
-                'enter' => function () {
+                'enter' => function() {
                     $this->variableDefs = [];
                 },
-                'leave' => function (OperationDefinitionNode $operation) use ($context) {
+                'leave' => function(OperationDefinitionNode $operation) use ($context) {
                     $variableNameUsed = [];
                     $usages           = $context->getRecursiveVariableUsages($operation);
                     $opName           = $operation->name ? $operation->name->value : null;
@@ -38,7 +38,7 @@ 
                     foreach ($this->variableDefs as $variableDef) {
                         $variableName = $variableDef->variable->name->value;
 
-                        if (! empty($variableNameUsed[$variableName])) {
+                        if (!empty($variableNameUsed[$variableName])) {
                             continue;
                         }
 
@@ -49,7 +49,7 @@ 
                     }
                 },
             ],
-            NodeKind::VARIABLE_DEFINITION  => function ($def) {
+            NodeKind::VARIABLE_DEFINITION  => function($def) {
                 $this->variableDefs[] = $def;
             },
         ];