webonyx / graphql-php

tools/gendocs.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 
 use GraphQL\Utils\Utils;
 
-$outputFile = __DIR__  . '/../docs/reference.md';
+$outputFile = __DIR__ . '/../docs/reference.md';
 
 $entries = [
     \GraphQL\GraphQL::class,

src/Server/OperationParams.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 
         // Some parameters could be provided as serialized JSON.
         foreach (['extensions', 'variables'] as $param) {
-            if (! is_string($params[$param])) {
+            if (!is_string($params[$param])) {
                 continue;
             }
 

src/Server/ServerConfig.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,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);
@@ -166,7 +166,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/Server/StandardServer.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         if (is_array($config)) {
             $config = ServerConfig::create($config);
         }
-        if (! $config instanceof ServerConfig) {
+        if (!$config instanceof ServerConfig) {
             throw new InvariantViolation('Expecting valid server config, but got ' . Utils::printSafe($config));
         }
         $this->config = $config;

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/Type/Definition/ObjectType.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -189,7 +189,7 @@ 
                 ? call_user_func($interfaces)
                 : $interfaces;
 
-            if ($interfaces !== null && ! is_array($interfaces)) {
+            if ($interfaces !== null && !is_array($interfaces)) {
                 throw new InvariantViolation(
                     sprintf('%s interfaces must be an Array or a callable which returns an Array.', $this->name)
                 );

src/Type/Definition/UnionType.php

Spacing +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function __construct(array $config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -53,7 +53,7 @@ 
 
     public function isPossibleType(Type $type) : bool
     {
-        if (! $type instanceof ObjectType) {
+        if (!$type instanceof ObjectType) {
             return false;
         }
 
@@ -73,7 +73,7 @@ 
     public function getTypes()
     {
         if ($this->types === null) {
-            if (! isset($this->config['types'])) {
+            if (!isset($this->config['types'])) {
                 $types = null;
             } elseif (is_callable($this->config['types'])) {
                 $types = call_user_func($this->config['types']);
@@ -81,7 +81,7 @@ 
                 $types = $this->config['types'];
             }
 
-            if (! is_array($types)) {
+            if (!is_array($types)) {
                 throw new InvariantViolation(
                     sprintf(
                         'Must provide Array of types or a callable which returns such an array for Union %s',
@@ -122,7 +122,7 @@ 
     {
         parent::assertValid();
 
-        if (! isset($this->config['resolveType'])) {
+        if (!isset($this->config['resolveType'])) {
             return;
         }
 

src/Type/Definition/FieldDefinition.php

Spacing +6 added lines, -6 removed lines

@@ -89,7 +89,7 @@ 
         if (is_callable($fields)) {
             $fields = $fields();
         }
-        if (! is_array($fields)) {
+        if (!is_array($fields)) {
             throw new InvariantViolation(
                 sprintf('%s fields must be an array or a callable which returns such an array.', $type->name)
             );
@@ -97,8 +97,8 @@ 
         $map = [];
         foreach ($fields as $name => $field) {
             if (is_array($field)) {
-                if (! isset($field['name'])) {
-                    if (! is_string($name)) {
+                if (!isset($field['name'])) {
+                    if (!is_string($name)) {
                         throw new InvariantViolation(
                             sprintf(
                                 '%s fields must be an associative array with field names as keys or a function which returns such an array.',
@@ -109,7 +109,7 @@ 
 
                     $field['name'] = $name;
                 }
-                if (isset($field['args']) && ! is_array($field['args'])) {
+                if (isset($field['args']) && !is_array($field['args'])) {
                     throw new InvariantViolation(
                         sprintf('%s.%s args must be an array.', $type->name, $name)
                     );
@@ -118,7 +118,7 @@ 
             } elseif ($field instanceof self) {
                 $fieldDef = $field;
             } else {
-                if (! is_string($name) || ! $field) {
+                if (!is_string($name) || !$field) {
                     throw new InvariantViolation(
                         sprintf(
                             '%s.%s field config must be an array, but got: %s',
@@ -209,7 +209,7 @@ 
             throw new InvariantViolation(sprintf('%s.%s: %s', $parentType->name, $this->name, $e->getMessage()));
         }
         Utils::invariant(
-            ! isset($this->config['isDeprecated']),
+            !isset($this->config['isDeprecated']),
             sprintf(
                 '%s.%s should provide "deprecationReason" instead of "isDeprecated".',
                 $parentType->name,

src/Type/Definition/QueryPlan.php

Spacing +6 added lines, -6 removed lines

@@ -79,7 +79,7 @@ 
 
     public function hasType(string $type) : bool
     {
-        return count(array_filter($this->getReferencedTypes(), static function (string $referencedType) use ($type) {
+        return count(array_filter($this->getReferencedTypes(), static function(string $referencedType) use ($type) {
                 return $type === $referencedType;
         })) > 0;
     }
@@ -94,7 +94,7 @@ 
 
     public function hasField(string $field) : bool
     {
-        return count(array_filter($this->getReferencedFields(), static function (string $referencedField) use ($field) {
+        return count(array_filter($this->getReferencedFields(), static function(string $referencedField) use ($field) {
             return $field === $referencedField;
         })) > 0;
     }
@@ -104,7 +104,7 @@ 
      */
     public function subFields(string $typename) : array
     {
-        if (! array_key_exists($typename, $this->types)) {
+        if (!array_key_exists($typename, $this->types)) {
             return [];
         }
 
@@ -120,7 +120,7 @@ 
         $implementors = [];
         /** @var FieldNode $fieldNode */
         foreach ($fieldNodes as $fieldNode) {
-            if (! $fieldNode->selectionSet) {
+            if (!$fieldNode->selectionSet) {
                 continue;
             }
 
@@ -228,7 +228,7 @@ 
      */
     private function mergeFields(Type $parentType, Type $type, array $fields, array $subfields, array &$implementors) : array
     {
-        if ($this->groupImplementorFields && $parentType instanceof AbstractType && ! $type instanceof AbstractType) {
+        if ($this->groupImplementorFields && $parentType instanceof AbstractType && !$type instanceof AbstractType) {
             $implementors[$type->name] = [
                 'type'   => $type,
                 'fields' => $this->arrayMergeDeep(
@@ -268,7 +268,7 @@ 
 
         foreach ($array2 as $key => & $value) {
             if (is_numeric($key)) {
-                if (! in_array($value, $merged, true)) {
+                if (!in_array($value, $merged, true)) {
                     $merged[] = $value;
                 }
             } elseif (is_array($value) && isset($merged[$key]) && is_array($merged[$key])) {