simPod / graphql-php

src/Executor/Executor.php

Spacing +23 added lines, -25 removed lines

@@ -299,19 +299,19 @@ 
         // field and its descendants will be omitted, and sibling fields will still
         // be executed. An execution which encounters errors will still result in a
         // resolved Promise.
-        $result = $this->exeContext->promises->create(function (callable $resolve) {
+        $result = $this->exeContext->promises->create(function(callable $resolve) {
             return $resolve($this->executeOperation($this->exeContext->operation, $this->exeContext->rootValue));
         });
         return $result
-            ->then(null, function ($error) {
+            ->then(null, function($error) {
                 // Errors from sub-fields of a NonNull type may propagate to the top level,
                 // at which point we still log the error and null the parent field, which
                 // in this case is the entire response.
                 $this->exeContext->addError($error);
                 return null;
             })
-            ->then(function ($data) {
-                if ($data !== null){
+            ->then(function($data) {
+                if ($data !== null) {
                     $data = (array) $data;
                 }
                 return new ExecutionResult($data, $this->exeContext->errors);
@@ -339,8 +339,7 @@ 
         // Similar to completeValueCatchingError.
         try {
             $result = $operation->operation === 'mutation' ?
-                $this->executeFieldsSerially($type, $rootValue, $path, $fields) :
-                $this->executeFields($type, $rootValue, $path, $fields);
+                $this->executeFieldsSerially($type, $rootValue, $path, $fields) : $this->executeFields($type, $rootValue, $path, $fields);
 
             $promise = $this->getPromise($result);
             if ($promise) {
@@ -391,7 +390,7 @@ 
                 if (!$subscriptionType) {
                     throw new Error(
                         'Schema is not configured for subscriptions.',
-                        [ $operation ]
+                        [$operation]
                     );
                 }
                 return $subscriptionType;
@@ -417,7 +416,7 @@ 
     {
         $prevPromise = $this->exeContext->promises->createFulfilled([]);
 
-        $process = function ($results, $responseName, $path, $parentType, $sourceValue, $fieldNodes) {
+        $process = function($results, $responseName, $path, $parentType, $sourceValue, $fieldNodes) {
             $fieldPath = $path;
             $fieldPath[] = $responseName;
             $result = $this->resolveField($parentType, $sourceValue, $fieldNodes, $fieldPath);
@@ -426,7 +425,7 @@ 
             }
             $promise = $this->getPromise($result);
             if ($promise) {
-                return $promise->then(function ($resolvedResult) use ($responseName, $results) {
+                return $promise->then(function($resolvedResult) use ($responseName, $results) {
                     $results[$responseName] = $resolvedResult;
                     return $results;
                 });
@@ -436,12 +435,12 @@ 
         };
 
         foreach ($fields as $responseName => $fieldNodes) {
-            $prevPromise = $prevPromise->then(function ($resolvedResults) use ($process, $responseName, $path, $parentType, $sourceValue, $fieldNodes) {
+            $prevPromise = $prevPromise->then(function($resolvedResults) use ($process, $responseName, $path, $parentType, $sourceValue, $fieldNodes) {
                 return $process($resolvedResults, $responseName, $path, $parentType, $sourceValue, $fieldNodes);
             });
         }
 
-        return $prevPromise->then(function ($resolvedResults) {
+        return $prevPromise->then(function($resolvedResults) {
             return self::fixResultsIfEmptyArray($resolvedResults);
         });
     }
@@ -826,7 +825,7 @@ 
 
             $promise = $this->getPromise($completed);
             if ($promise) {
-                return $promise->then(null, function ($error) use ($exeContext) {
+                return $promise->then(null, function($error) use ($exeContext) {
                     $exeContext->addError($error);
                     return $this->exeContext->promises->createFulfilled(null);
                 });
@@ -871,7 +870,7 @@ 
             );
             $promise = $this->getPromise($completed);
             if ($promise) {
-                return $promise->then(null, function ($error) use ($fieldNodes, $path) {
+                return $promise->then(null, function($error) use ($fieldNodes, $path) {
                     return $this->exeContext->promises->createRejected(Error::createLocatedError($error, $fieldNodes, $path));
                 });
             }
@@ -925,7 +924,7 @@ 
 
         // If result is a Promise, apply-lift over completeValue.
         if ($promise) {
-            return $promise->then(function (&$resolved) use ($returnType, $fieldNodes, $info, $path) {
+            return $promise->then(function(&$resolved) use ($returnType, $fieldNodes, $info, $path) {
                 return $this->completeValue($returnType, $fieldNodes, $info, $path, $resolved);
             });
         }
@@ -970,8 +969,8 @@ 
                 $hint = "Make sure that type loader returns the same instance as defined in {$info->parentType}.{$info->fieldName}";
             }
             throw new InvariantViolation(
-                "Schema must contain unique named types but contains multiple types named \"$returnType\". ".
-                "$hint ".
+                "Schema must contain unique named types but contains multiple types named \"$returnType\". " .
+                "$hint " .
                 "(see http://webonyx.github.io/graphql-php/type-system/#type-registry)."
             );
         }
@@ -1133,14 +1132,13 @@ 
     )
     {
         $runtimeType = is_string($runtimeTypeOrName) ?
-            $this->exeContext->schema->getType($runtimeTypeOrName) :
-            $runtimeTypeOrName;
+            $this->exeContext->schema->getType($runtimeTypeOrName) : $runtimeTypeOrName;
 
         if (!$runtimeType instanceof ObjectType) {
             throw new InvariantViolation(
                 "Abstract type {$returnType} must resolve to an Object type at " .
                 "runtime for field {$info->parentType}.{$info->fieldName} with " .
-                'value "' . Utils::printSafe($result) . '", received "'. Utils::printSafe($runtimeType) . '".' .
+                'value "' . Utils::printSafe($result) . '", received "' . Utils::printSafe($runtimeType) . '".' .
                 'Either the ' . $returnType . ' type should provide a "resolveType" ' .
                 'function or each possible types should provide an "isTypeOf" function.'
             );
@@ -1154,8 +1152,8 @@ 
 
         if ($runtimeType !== $this->exeContext->schema->getType($runtimeType->name)) {
             throw new InvariantViolation(
-                "Schema must contain unique named types but contains multiple types named \"$runtimeType\". ".
-                "Make sure that `resolveType` function of abstract type \"{$returnType}\" returns the same ".
+                "Schema must contain unique named types but contains multiple types named \"$runtimeType\". " .
+                "Make sure that `resolveType` function of abstract type \"{$returnType}\" returns the same " .
                 "type instance as referenced anywhere else within the schema " .
                 "(see http://webonyx.github.io/graphql-php/type-system/#type-registry)."
             );
@@ -1213,13 +1211,13 @@ 
             return $returnType->serialize($result);
         } catch (\Exception $error) {
             throw new InvariantViolation(
-                'Expected a value of type "'. Utils::printSafe($returnType) . '" but received: ' . Utils::printSafe($result),
+                'Expected a value of type "' . Utils::printSafe($returnType) . '" but received: ' . Utils::printSafe($result),
                 0,
                 $error
             );
         } catch (\Throwable $error) {
             throw new InvariantViolation(
-                'Expected a value of type "'. Utils::printSafe($returnType) . '" but received: ' . Utils::printSafe($result),
+                'Expected a value of type "' . Utils::printSafe($returnType) . '" but received: ' . Utils::printSafe($result),
                 0,
                 $error
             );
@@ -1359,9 +1357,9 @@ 
 
         if ($abstractType instanceof InterfaceType && $info->schema->getConfig()->typeLoader) {
             Warning::warnOnce(
-                "GraphQL Interface Type `{$abstractType->name}` returned `null` from it`s `resolveType` function ".
+                "GraphQL Interface Type `{$abstractType->name}` returned `null` from it`s `resolveType` function " .
                 'for value: ' . Utils::printSafe($value) . '. Switching to slow resolution method using `isTypeOf` ' .
-                'of all possible implementations. It requires full schema scan and degrades query performance significantly. '.
+                'of all possible implementations. It requires full schema scan and degrades query performance significantly. ' .
                 ' Make sure your `resolveType` always returns valid implementation or throws.',
                 Warning::WARNING_FULL_SCHEMA_SCAN
             );

src/Executor/Promise/Adapter/SyncPromise.php

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
         Utils::invariant($this->state !== self::PENDING, 'Cannot enqueue derived promises when parent is still pending');
 
         foreach ($this->waiting as $descriptor) {
-            self::getQueue()->enqueue(function () use ($descriptor) {
+            self::getQueue()->enqueue(function() use ($descriptor) {
                 /** @var $promise self */
                 list($promise, $onFulfilled, $onRejected) = $descriptor;
 

src/Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
     public function all(array $promisesOrValues)
     {
         // TODO: rework with generators when PHP minimum required version is changed to 5.5+
-        $promisesOrValues = Utils::map($promisesOrValues, function ($item) {
+        $promisesOrValues = Utils::map($promisesOrValues, function($item) {
             return $item instanceof Promise ? $item->adoptedPromise : $item;
         });
 

src/Executor/Values.php

Spacing +4 added lines, -4 removed lines

@@ -70,7 +70,7 @@ 
                     if ($coercionErrors) {
                         $messagePrelude = "Variable \"\$$varName\" got invalid value " . Utils::printSafeJson($value) . '; ';
 
-                        foreach($coercionErrors as $error) {
+                        foreach ($coercionErrors as $error) {
                             $errors[] = new Error(
                                 $messagePrelude . $error->getMessage(),
                                 $error->getNodes(),
@@ -112,7 +112,7 @@ 
         $coercedValues = [];
 
         /** @var ArgumentNode[] $argNodeMap */
-        $argNodeMap = $argNodes ? Utils::keyMap($argNodes, function (ArgumentNode $arg) {
+        $argNodeMap = $argNodes ? Utils::keyMap($argNodes, function(ArgumentNode $arg) {
             return $arg->name->value;
         }) : [];
 
@@ -146,7 +146,7 @@ 
                         'Argument "' . $name . '" of required type "' . Utils::printSafe($argType) . '" was ' .
                         'provided the variable "$' . $variableName . '" which was not provided ' .
                         'a runtime value.',
-                        [ $argumentNode->value ]
+                        [$argumentNode->value]
                     );
                 }
             } else {
@@ -158,7 +158,7 @@ 
                     // continue with an invalid argument value.
                     throw new Error(
                         'Argument "' . $name . '" has invalid value ' . Printer::doPrint($valueNode) . '.',
-                        [ $argumentNode->value ]
+                        [$argumentNode->value]
                     );
                 }
                 $coercedValues[$name] = $coercedValue;

src/Error/Error.php

Spacing +2 added lines, -2 removed lines

@@ -260,11 +260,11 @@
             $nodes = $this->nodes;
 
             if ($positions && $source) {
-                $this->locations = array_map(function ($pos) use ($source) {
+                $this->locations = array_map(function($pos) use ($source) {
                     return $source->getLocation($pos);
                 }, $positions);
             } else if ($nodes) {
-                $this->locations = array_filter(array_map(function ($node) {
+                $this->locations = array_filter(array_map(function($node) {
                     if ($node->loc) {
                         return $node->loc->source->getLocation($node->loc->start);
                     }

src/Error/FormattedError.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
         $printedLocations = [];
         if ($error->nodes) {
             /** @var Node $node */
-            foreach($error->nodes as $node) {
+            foreach ($error->nodes as $node) {
                 if ($node->loc) {
                     $printedLocations[] = self::highlightSourceAtLocation(
                         $node->loc->source,
@@ -51,7 +51,7 @@ 
             }
         } else if ($error->getSource() && $error->getLocations()) {
             $source = $error->getSource();
-            foreach($error->getLocations() as $location) {
+            foreach ($error->getLocations() as $location) {
                 $printedLocations[] = self::highlightSourceAtLocation($source, $location);
             }
         }
@@ -89,7 +89,7 @@ 
             $line >= 2 ? (self::lpad($padLen, $prevLineNum) . ': ' . $lines[$line - 2]) : null,
             self::lpad($padLen, $lineNum) . ': ' . $lines[$line - 1],
             self::whitespace(2 + $padLen + $contextColumn - 1) . '^',
-            $line < count($lines)? self::lpad($padLen, $nextLineNum) . ': ' . $lines[$line] : null
+            $line < count($lines) ? self::lpad($padLen, $nextLineNum) . ': ' . $lines[$line] : null
         ];
 
         return join("\n", array_filter($outputLines));
@@ -275,7 +275,7 @@ 
         // Remove invariant entries as they don't provide much value:
         if (
             isset($trace[0]['function']) && isset($trace[0]['class']) &&
-            ('GraphQL\Utils\Utils::invariant' === $trace[0]['class'].'::'.$trace[0]['function'])) {
+            ('GraphQL\Utils\Utils::invariant' === $trace[0]['class'] . '::' . $trace[0]['function'])) {
             array_shift($trace);
         }
 
@@ -354,7 +354,7 @@ 
         ];
 
         if (!empty($locations)) {
-            $formatted['locations'] = array_map(function($loc) { return $loc->toArray();}, $locations);
+            $formatted['locations'] = array_map(function($loc) { return $loc->toArray(); }, $locations);
         }
 
         return $formatted;

src/Language/Parser.php

Indentation +1 added lines, -1 removed lines

@@ -1251,7 +1251,7 @@
                 Token::BRACE_L,
                 [$this, 'parseEnumValueDefinition'],
                 Token::BRACE_R
-              )
+                )
             : new NodeList([]);
     }
 

Spacing +3 added lines, -6 removed lines

@@ -465,8 +465,7 @@ 
                 Token::PAREN_L,
                 [$this, 'parseVariableDefinition'],
                 Token::PAREN_R
-            ) :
-            new NodeList([]);
+            ) : new NodeList([]);
     }
 
     /**
@@ -528,8 +527,7 @@ 
     function parseSelection()
     {
         return $this->peek(Token::SPREAD) ?
-            $this->parseFragment() :
-            $this->parseField();
+            $this->parseFragment() : $this->parseField();
     }
 
     /**
@@ -568,8 +566,7 @@ 
     {
         $item = $isConst ? 'parseConstArgument' : 'parseArgument';
         return $this->peek(Token::PAREN_L) ?
-            $this->many(Token::PAREN_L, [$this, $item], Token::PAREN_R) :
-            new NodeList([]);
+            $this->many(Token::PAREN_L, [$this, $item], Token::PAREN_R) : new NodeList([]);
     }
 
     /**

src/Language/AST/Node.php

Braces +6 added lines, -4 removed lines

@@ -127,11 +127,13 @@
         }
 
         foreach (get_object_vars($node) as $prop => $propValue) {
-            if (isset($result[$prop]))
-                continue;
+            if (isset($result[$prop])) {
+                            continue;
+            }
 
-            if ($propValue === null)
-                continue;
+            if ($propValue === null) {
+                            continue;
+            }
 
             if (is_array($propValue) || $propValue instanceof NodeList) {
                 $tmp = [];

src/Language/Lexer.php

Spacing +1 added lines, -1 removed lines

@@ -211,7 +211,7 @@
         }
 
         $errMessage = $code === 39
-            ? "Unexpected single quote character ('), did you mean to use ". 'a double quote (")?'
+            ? "Unexpected single quote character ('), did you mean to use " . 'a double quote (")?'
             : 'Cannot parse the unexpected character ' . Utils::printCharCode($code) . '.';
 
         throw new SyntaxError(