simPod / graphql-php

src/Executor/Executor.php

Spacing +41 added lines, -43 removed lines

@@ -64,7 +64,7 @@ 
 
     private function __construct(ExecutionContext $context)
     {
-        if (! self::$UNDEFINED) {
+        if (!self::$UNDEFINED) {
             self::$UNDEFINED = Utils::undefined();
         }
 
@@ -212,10 +212,10 @@ 
         foreach ($documentNode->definitions as $definition) {
             switch ($definition->kind) {
                 case NodeKind::OPERATION_DEFINITION:
-                    if (! $operationName && $operation) {
+                    if (!$operationName && $operation) {
                         $hasMultipleAssumedOperations = true;
                     }
-                    if (! $operationName ||
+                    if (!$operationName ||
                         (isset($definition->name) && $definition->name->value === $operationName)) {
                         $operation = $definition;
                     }
@@ -226,7 +226,7 @@ 
             }
         }
 
-        if (! $operation) {
+        if (!$operation) {
             if ($operationName) {
                 $errors[] = new Error(sprintf('Unknown operation named "%s".', $operationName));
             } else {
@@ -285,14 +285,14 @@ 
         // 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) {
+                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.
@@ -301,7 +301,7 @@ 
                     return null;
                 }
             )
-            ->then(function ($data) {
+            ->then(function($data) {
                 if ($data !== null) {
                     $data = (array) $data;
                 }
@@ -330,14 +330,13 @@ 
         // 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) {
                 return $promise->then(
                     null,
-                    function ($error) {
+                    function($error) {
                         $this->exeContext->addError($error);
 
                         return null;
@@ -364,7 +363,7 @@ 
         switch ($operation->operation) {
             case 'query':
                 $queryType = $schema->getQueryType();
-                if (! $queryType) {
+                if (!$queryType) {
                     throw new Error(
                         'Schema does not define the required query root type.',
                         [$operation]
@@ -374,7 +373,7 @@ 
                 return $queryType;
             case 'mutation':
                 $mutationType = $schema->getMutationType();
-                if (! $mutationType) {
+                if (!$mutationType) {
                     throw new Error(
                         'Schema is not configured for mutations.',
                         [$operation]
@@ -384,7 +383,7 @@ 
                 return $mutationType;
             case 'subscription':
                 $subscriptionType = $schema->getSubscriptionType();
-                if (! $subscriptionType) {
+                if (!$subscriptionType) {
                     throw new Error(
                         'Schema is not configured for subscriptions.',
                         [$operation]
@@ -423,18 +422,18 @@ 
         foreach ($selectionSet->selections as $selection) {
             switch ($selection->kind) {
                 case NodeKind::FIELD:
-                    if (! $this->shouldIncludeNode($selection)) {
+                    if (!$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $name = self::getFieldEntryKey($selection);
-                    if (! isset($fields[$name])) {
+                    if (!isset($fields[$name])) {
                         $fields[$name] = new \ArrayObject();
                     }
                     $fields[$name][] = $selection;
                     break;
                 case NodeKind::INLINE_FRAGMENT:
-                    if (! $this->shouldIncludeNode($selection) ||
-                        ! $this->doesFragmentConditionMatch($selection, $runtimeType)
+                    if (!$this->shouldIncludeNode($selection) ||
+                        !$this->doesFragmentConditionMatch($selection, $runtimeType)
                     ) {
                         break;
                     }
@@ -447,14 +446,14 @@ 
                     break;
                 case NodeKind::FRAGMENT_SPREAD:
                     $fragName = $selection->name->value;
-                    if (! empty($visitedFragmentNames[$fragName]) || ! $this->shouldIncludeNode($selection)) {
+                    if (!empty($visitedFragmentNames[$fragName]) || !$this->shouldIncludeNode($selection)) {
                         break;
                     }
                     $visitedFragmentNames[$fragName] = true;
 
                     /** @var FragmentDefinitionNode|null $fragment */
                     $fragment = $exeContext->fragments[$fragName] ?? null;
-                    if (! $fragment || ! $this->doesFragmentConditionMatch($fragment, $runtimeType)) {
+                    if (!$fragment || !$this->doesFragmentConditionMatch($fragment, $runtimeType)) {
                         break;
                     }
                     $this->collectFields(
@@ -557,7 +556,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);
@@ -566,7 +565,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;
@@ -578,7 +577,7 @@ 
         };
 
         foreach ($fields as $responseName => $fieldNodes) {
-            $prevPromise = $prevPromise->then(function ($resolvedResults) use (
+            $prevPromise = $prevPromise->then(function($resolvedResults) use (
                 $process,
                 $responseName,
                 $path,
@@ -590,7 +589,7 @@ 
             });
         }
 
-        return $prevPromise->then(function ($resolvedResults) {
+        return $prevPromise->then(function($resolvedResults) {
             return self::fixResultsIfEmptyArray($resolvedResults);
         });
     }
@@ -615,7 +614,7 @@ 
         $fieldName = $fieldNode->name->value;
         $fieldDef  = $this->getFieldDef($exeContext->schema, $parentType, $fieldName);
 
-        if (! $fieldDef) {
+        if (!$fieldDef) {
             return self::$UNDEFINED;
         }
 
@@ -781,7 +780,7 @@ 
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($exeContext) {
+                    function($error) use ($exeContext) {
                         $exeContext->addError($error);
 
                         return $this->exeContext->promises->createFulfilled(null);
@@ -824,11 +823,11 @@ 
                 $path,
                 $result
             );
-            $promise   = $this->getPromise($completed);
+            $promise = $this->getPromise($completed);
             if ($promise) {
                 return $promise->then(
                     null,
-                    function ($error) use ($fieldNodes, $path) {
+                    function($error) use ($fieldNodes, $path) {
                         return $this->exeContext->promises->createRejected(Error::createLocatedError(
                             $error,
                             $fieldNodes,
@@ -885,7 +884,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);
             });
         }
@@ -976,7 +975,7 @@ 
         }
         if ($this->exeContext->promises->isThenable($value)) {
             $promise = $this->exeContext->promises->convertThenable($value);
-            if (! $promise instanceof Promise) {
+            if (!$promise instanceof Promise) {
                 throw new InvariantViolation(sprintf(
                     '%s::convertThenable is expected to return instance of GraphQL\Executor\Promise\Promise, got: %s',
                     get_class($this->exeContext->promises),
@@ -1015,7 +1014,7 @@ 
             $fieldPath     = $path;
             $fieldPath[]   = $i++;
             $completedItem = $this->completeValueCatchingError($itemType, $fieldNodes, $info, $fieldPath, $item);
-            if (! $containsPromise && $this->getPromise($completedItem)) {
+            if (!$containsPromise && $this->getPromise($completedItem)) {
                 $containsPromise = true;
             }
             $completedItems[] = $completedItem;
@@ -1071,7 +1070,7 @@ 
 
         $promise = $this->getPromise($runtimeType);
         if ($promise) {
-            return $promise->then(function ($resolvedRuntimeType) use (
+            return $promise->then(function($resolvedRuntimeType) use (
                 $returnType,
                 $fieldNodes,
                 $info,
@@ -1165,9 +1164,9 @@ 
             }
         }
 
-        if (! empty($promisedIsTypeOfResults)) {
+        if (!empty($promisedIsTypeOfResults)) {
             return $this->exeContext->promises->all($promisedIsTypeOfResults)
-                ->then(function ($isTypeOfResults) use ($possibleTypes) {
+                ->then(function($isTypeOfResults) use ($possibleTypes) {
                     foreach ($isTypeOfResults as $index => $result) {
                         if ($result) {
                             return $possibleTypes[$index];
@@ -1200,14 +1199,14 @@ 
         if ($isTypeOf !== null) {
             $promise = $this->getPromise($isTypeOf);
             if ($promise) {
-                return $promise->then(function ($isTypeOfResult) use (
+                return $promise->then(function($isTypeOfResult) use (
                     $returnType,
                     $fieldNodes,
                     $info,
                     $path,
                     &$result
                 ) {
-                    if (! $isTypeOfResult) {
+                    if (!$isTypeOfResult) {
                         throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
                     }
 
@@ -1220,7 +1219,7 @@ 
                     );
                 });
             }
-            if (! $isTypeOf) {
+            if (!$isTypeOf) {
                 throw $this->invalidReturnTypeError($returnType, $result, $fieldNodes);
             }
         }
@@ -1321,14 +1320,14 @@ 
             if ($result === self::$UNDEFINED) {
                 continue;
             }
-            if (! $containsPromise && $this->getPromise($result)) {
+            if (!$containsPromise && $this->getPromise($result)) {
                 $containsPromise = true;
             }
             $finalResults[$responseName] = $result;
         }
 
         // If there are no promises, we can just return the object
-        if (! $containsPromise) {
+        if (!$containsPromise) {
             return self::fixResultsIfEmptyArray($finalResults);
         }
 
@@ -1371,7 +1370,7 @@ 
 
         $promise = $this->exeContext->promises->all($valuesAndPromises);
 
-        return $promise->then(function ($values) use ($keys) {
+        return $promise->then(function($values) use ($keys) {
             $resolvedResults = [];
             foreach ($values as $i => $value) {
                 $resolvedResults[$keys[$i]] = $value;
@@ -1394,10 +1393,9 @@ 
         &$result
     ) {
         $runtimeType = is_string($runtimeTypeOrName) ?
-            $this->exeContext->schema->getType($runtimeTypeOrName) :
-            $runtimeTypeOrName;
+            $this->exeContext->schema->getType($runtimeTypeOrName) : $runtimeTypeOrName;
 
-        if (! $runtimeType instanceof ObjectType) {
+        if (!$runtimeType instanceof ObjectType) {
             throw new InvariantViolation(
                 sprintf(
                     'Abstract type %s must resolve to an Object type at ' .
@@ -1414,7 +1412,7 @@ 
             );
         }
 
-        if (! $this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
+        if (!$this->exeContext->schema->isPossibleType($returnType, $runtimeType)) {
             throw new InvariantViolation(
                 sprintf('Runtime Object type "%s" is not a possible type for "%s".', $runtimeType, $returnType)
             );

src/Language/Lexer.php

Spacing +1 added lines, -1 removed lines

@@ -489,7 +489,7 @@
                     case 117:
                         $position   = $this->position;
                         list ($hex) = $this->readChars(4, true);
-                        if (! preg_match('/[0-9a-fA-F]{4}/', $hex)) {
+                        if (!preg_match('/[0-9a-fA-F]{4}/', $hex)) {
                             throw new SyntaxError(
                                 $this->source,
                                 $position - 1,

src/Language/AST/Location.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         $this->endToken   = $endToken;
         $this->source     = $source;
 
-        if (! $startToken || ! $endToken) {
+        if (!$startToken || !$endToken) {
             return;
         }
 

src/Language/Printer.php

Spacing +46 added lines, -46 removed lines

@@ -94,19 +94,19 @@ 
             $ast,
             [
                 'leave' => [
-                    NodeKind::NAME => function (Node $node) {
+                    NodeKind::NAME => function(Node $node) {
                         return '' . $node->value;
                     },
 
-                    NodeKind::VARIABLE => function ($node) {
+                    NodeKind::VARIABLE => function($node) {
                         return '$' . $node->name;
                     },
 
-                    NodeKind::DOCUMENT => function (DocumentNode $node) {
+                    NodeKind::DOCUMENT => function(DocumentNode $node) {
                         return $this->join($node->definitions, "\n\n") . "\n";
                     },
 
-                    NodeKind::OPERATION_DEFINITION => function (OperationDefinitionNode $node) {
+                    NodeKind::OPERATION_DEFINITION => function(OperationDefinitionNode $node) {
                         $op           = $node->operation;
                         $name         = $node->name;
                         $varDefs      = $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')');
@@ -114,20 +114,20 @@ 
                         $selectionSet = $node->selectionSet;
                         // Anonymous queries with no directives or variable definitions can use
                         // the query short form.
-                        return ! $name && ! $directives && ! $varDefs && $op === 'query'
+                        return !$name && !$directives && !$varDefs && $op === 'query'
                             ? $selectionSet
                             : $this->join([$op, $this->join([$name, $varDefs]), $directives, $selectionSet], ' ');
                     },
 
-                    NodeKind::VARIABLE_DEFINITION => function (VariableDefinitionNode $node) {
+                    NodeKind::VARIABLE_DEFINITION => function(VariableDefinitionNode $node) {
                         return $node->variable . ': ' . $node->type . $this->wrap(' = ', $node->defaultValue);
                     },
 
-                    NodeKind::SELECTION_SET => function (SelectionSetNode $node) {
+                    NodeKind::SELECTION_SET => function(SelectionSetNode $node) {
                         return $this->block($node->selections);
                     },
 
-                    NodeKind::FIELD => function (FieldNode $node) {
+                    NodeKind::FIELD => function(FieldNode $node) {
                         return $this->join(
                             [
                                 $this->wrap('', $node->alias, ': ') . $node->name . $this->wrap(
@@ -142,15 +142,15 @@ 
                         );
                     },
 
-                    NodeKind::ARGUMENT => function (ArgumentNode $node) {
+                    NodeKind::ARGUMENT => function(ArgumentNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::FRAGMENT_SPREAD => function (FragmentSpreadNode $node) {
+                    NodeKind::FRAGMENT_SPREAD => function(FragmentSpreadNode $node) {
                         return '...' . $node->name . $this->wrap(' ', $this->join($node->directives, ' '));
                     },
 
-                    NodeKind::INLINE_FRAGMENT => function (InlineFragmentNode $node) {
+                    NodeKind::INLINE_FRAGMENT => function(InlineFragmentNode $node) {
                         return $this->join(
                             [
                                 '...',
@@ -162,7 +162,7 @@ 
                         );
                     },
 
-                    NodeKind::FRAGMENT_DEFINITION => function (FragmentDefinitionNode $node) {
+                    NodeKind::FRAGMENT_DEFINITION => function(FragmentDefinitionNode $node) {
                         // Note: fragment variable definitions are experimental and may be changed or removed in the future.
                         return sprintf('fragment %s', $node->name)
                             . $this->wrap('(', $this->join($node->variableDefinitions, ', '), ')')
@@ -171,15 +171,15 @@ 
                             . $node->selectionSet;
                     },
 
-                    NodeKind::INT => function (IntValueNode $node) {
+                    NodeKind::INT => function(IntValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::FLOAT => function (FloatValueNode $node) {
+                    NodeKind::FLOAT => function(FloatValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::STRING => function (StringValueNode $node, $key) {
+                    NodeKind::STRING => function(StringValueNode $node, $key) {
                         if ($node->block) {
                             return $this->printBlockString($node->value, $key === 'description');
                         }
@@ -187,47 +187,47 @@ 
                         return json_encode($node->value);
                     },
 
-                    NodeKind::BOOLEAN => function (BooleanValueNode $node) {
+                    NodeKind::BOOLEAN => function(BooleanValueNode $node) {
                         return $node->value ? 'true' : 'false';
                     },
 
-                    NodeKind::NULL => function (NullValueNode $node) {
+                    NodeKind::NULL => function(NullValueNode $node) {
                         return 'null';
                     },
 
-                    NodeKind::ENUM => function (EnumValueNode $node) {
+                    NodeKind::ENUM => function(EnumValueNode $node) {
                         return $node->value;
                     },
 
-                    NodeKind::LST => function (ListValueNode $node) {
+                    NodeKind::LST => function(ListValueNode $node) {
                         return '[' . $this->join($node->values, ', ') . ']';
                     },
 
-                    NodeKind::OBJECT => function (ObjectValueNode $node) {
+                    NodeKind::OBJECT => function(ObjectValueNode $node) {
                         return '{' . $this->join($node->fields, ', ') . '}';
                     },
 
-                    NodeKind::OBJECT_FIELD => function (ObjectFieldNode $node) {
+                    NodeKind::OBJECT_FIELD => function(ObjectFieldNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
-                    NodeKind::DIRECTIVE => function (DirectiveNode $node) {
+                    NodeKind::DIRECTIVE => function(DirectiveNode $node) {
                         return '@' . $node->name . $this->wrap('(', $this->join($node->arguments, ', '), ')');
                     },
 
-                    NodeKind::NAMED_TYPE => function (NamedTypeNode $node) {
+                    NodeKind::NAMED_TYPE => function(NamedTypeNode $node) {
                         return $node->name;
                     },
 
-                    NodeKind::LIST_TYPE => function (ListTypeNode $node) {
+                    NodeKind::LIST_TYPE => function(ListTypeNode $node) {
                         return '[' . $node->type . ']';
                     },
 
-                    NodeKind::NON_NULL_TYPE => function (NonNullTypeNode $node) {
+                    NodeKind::NON_NULL_TYPE => function(NonNullTypeNode $node) {
                         return $node->type . '!';
                     },
 
-                    NodeKind::SCHEMA_DEFINITION => function (SchemaDefinitionNode $def) {
+                    NodeKind::SCHEMA_DEFINITION => function(SchemaDefinitionNode $def) {
                         return $this->join(
                             [
                                 'schema',
@@ -238,15 +238,15 @@ 
                         );
                     },
 
-                    NodeKind::OPERATION_TYPE_DEFINITION => function (OperationTypeDefinitionNode $def) {
+                    NodeKind::OPERATION_TYPE_DEFINITION => function(OperationTypeDefinitionNode $def) {
                         return $def->operation . ': ' . $def->type;
                     },
 
-                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function (ScalarTypeDefinitionNode $def) {
+                    NodeKind::SCALAR_TYPE_DEFINITION => $this->addDescription(function(ScalarTypeDefinitionNode $def) {
                         return $this->join(['scalar', $def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function (ObjectTypeDefinitionNode $def) {
+                    NodeKind::OBJECT_TYPE_DEFINITION => $this->addDescription(function(ObjectTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'type',
@@ -259,14 +259,14 @@ 
                         );
                     }),
 
-                    NodeKind::FIELD_DEFINITION => $this->addDescription(function (FieldDefinitionNode $def) {
+                    NodeKind::FIELD_DEFINITION => $this->addDescription(function(FieldDefinitionNode $def) {
                         return $def->name
                             . $this->wrap('(', $this->join($def->arguments, ', '), ')')
                             . ': ' . $def->type
                             . $this->wrap(' ', $this->join($def->directives, ' '));
                     }),
 
-                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function (InputValueDefinitionNode $def) {
+                    NodeKind::INPUT_VALUE_DEFINITION => $this->addDescription(function(InputValueDefinitionNode $def) {
                         return $this->join(
                             [
                                 $def->name . ': ' . $def->type,
@@ -278,7 +278,7 @@ 
                     }),
 
                     NodeKind::INTERFACE_TYPE_DEFINITION => $this->addDescription(
-                        function (InterfaceTypeDefinitionNode $def) {
+                        function(InterfaceTypeDefinitionNode $def) {
                             return $this->join(
                                 [
                                     'interface',
@@ -291,7 +291,7 @@ 
                         }
                     ),
 
-                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function (UnionTypeDefinitionNode $def) {
+                    NodeKind::UNION_TYPE_DEFINITION => $this->addDescription(function(UnionTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'union',
@@ -305,7 +305,7 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function (EnumTypeDefinitionNode $def) {
+                    NodeKind::ENUM_TYPE_DEFINITION => $this->addDescription(function(EnumTypeDefinitionNode $def) {
                         return $this->join(
                             [
                                 'enum',
@@ -317,11 +317,11 @@ 
                         );
                     }),
 
-                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function (EnumValueDefinitionNode $def) {
+                    NodeKind::ENUM_VALUE_DEFINITION => $this->addDescription(function(EnumValueDefinitionNode $def) {
                         return $this->join([$def->name, $this->join($def->directives, ' ')], ' ');
                     }),
 
-                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function (
+                    NodeKind::INPUT_OBJECT_TYPE_DEFINITION => $this->addDescription(function(
                         InputObjectTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -335,7 +335,7 @@ 
                         );
                     }),
 
-                    NodeKind::SCALAR_TYPE_EXTENSION => function (ScalarTypeExtensionNode $def) {
+                    NodeKind::SCALAR_TYPE_EXTENSION => function(ScalarTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend scalar',
@@ -346,7 +346,7 @@ 
                         );
                     },
 
-                    NodeKind::OBJECT_TYPE_EXTENSION => function (ObjectTypeExtensionNode $def) {
+                    NodeKind::OBJECT_TYPE_EXTENSION => function(ObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend type',
@@ -359,7 +359,7 @@ 
                         );
                     },
 
-                    NodeKind::INTERFACE_TYPE_EXTENSION => function (InterfaceTypeExtensionNode $def) {
+                    NodeKind::INTERFACE_TYPE_EXTENSION => function(InterfaceTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend interface',
@@ -371,7 +371,7 @@ 
                         );
                     },
 
-                    NodeKind::UNION_TYPE_EXTENSION => function (UnionTypeExtensionNode $def) {
+                    NodeKind::UNION_TYPE_EXTENSION => function(UnionTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend union',
@@ -385,7 +385,7 @@ 
                         );
                     },
 
-                    NodeKind::ENUM_TYPE_EXTENSION => function (EnumTypeExtensionNode $def) {
+                    NodeKind::ENUM_TYPE_EXTENSION => function(EnumTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend enum',
@@ -397,7 +397,7 @@ 
                         );
                     },
 
-                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function (InputObjectTypeExtensionNode $def) {
+                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION => function(InputObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                                 'extend input',
@@ -409,7 +409,7 @@ 
                         );
                     },
 
-                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function (DirectiveDefinitionNode $def) {
+                    NodeKind::DIRECTIVE_DEFINITION => $this->addDescription(function(DirectiveDefinitionNode $def) {
                         return 'directive @'
                             . $def->name
                             . $this->wrap('(', $this->join($def->arguments, ', '), ')')
@@ -422,7 +422,7 @@ 
 
     public function addDescription(\Closure $cb)
     {
-        return function ($node) use ($cb) {
+        return function($node) use ($cb) {
             return $this->join([$node->description, $cb($node)], "\n");
         };
     }
@@ -469,8 +469,8 @@ 
                 $separator,
                 Utils::filter(
                     $maybeArray,
-                    function ($x) {
-                        return ! ! $x;
+                    function($x) {
+                        return !!$x;
                     }
                 )
             )

src/Language/Parser.php

Spacing +22 added lines, -24 removed lines

@@ -299,7 +299,7 @@ 
         $this->expect($openKind);
 
         $nodes = [];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -323,7 +323,7 @@ 
         $this->expect($openKind);
 
         $nodes = [$parseFn($this)];
-        while (! $this->skip($closeKind)) {
+        while (!$this->skip($closeKind)) {
             $nodes[] = $parseFn($this);
         }
 
@@ -360,7 +360,7 @@ 
         $definitions = [];
         do {
             $definitions[] = $this->parseDefinition();
-        } while (! $this->skip(Token::EOF));
+        } while (!$this->skip(Token::EOF));
 
         return new DocumentNode([
             'definitions' => new NodeList($definitions),
@@ -492,7 +492,7 @@ 
         return $this->peek(Token::PAREN_L) ?
             $this->many(
                 Token::PAREN_L,
-                function () {
+                function() {
                     return $this->parseVariableDefinition();
                 },
                 Token::PAREN_R
@@ -547,7 +547,7 @@ 
             [
                 'selections' => $this->many(
                     Token::BRACE_L,
-                    function () {
+                    function() {
                         return $this->parseSelection();
                     },
                     Token::BRACE_R
@@ -568,8 +568,7 @@ 
     private function parseSelection()
     {
         return $this->peek(Token::SPREAD) ?
-            $this->parseFragment() :
-            $this->parseField();
+            $this->parseFragment() : $this->parseField();
     }
 
     /**
@@ -607,16 +606,15 @@ 
     private function parseArguments($isConst)
     {
         $parseFn = $isConst ?
-            function () {
+            function() {
                 return $this->parseConstArgument();
             } :
-            function () {
+            function() {
                 return $this->parseArgument();
             };
 
         return $this->peek(Token::PAREN_L) ?
-            $this->many(Token::PAREN_L, $parseFn, Token::PAREN_R) :
-            new NodeList([]);
+            $this->many(Token::PAREN_L, $parseFn, Token::PAREN_R) : new NodeList([]);
     }
 
     /**
@@ -808,7 +806,7 @@ 
                 break;
 
             case Token::DOLLAR:
-                if (! $isConst) {
+                if (!$isConst) {
                     return $this->parseVariable();
                 }
                 break;
@@ -855,9 +853,9 @@ 
     private function parseArray($isConst)
     {
         $start   = $this->lexer->token;
-        $parseFn = $isConst ? function () {
+        $parseFn = $isConst ? function() {
             return $this->parseConstValue();
-        } : function () {
+        } : function() {
             return $this->parseVariableValue();
         };
 
@@ -878,7 +876,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);
         }
 
@@ -1065,7 +1063,7 @@ 
 
         $operationTypes = $this->many(
             Token::BRACE_L,
-            function () {
+            function() {
                 return $this->parseOperationTypeDefinition();
             },
             Token::BRACE_R
@@ -1158,7 +1156,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))
             );
         }
 
@@ -1172,7 +1170,7 @@ 
     private function parseFieldsDefinition()
     {
         // 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
         ) {
@@ -1185,7 +1183,7 @@ 
         return $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseFieldDefinition();
                 },
                 Token::BRACE_R
@@ -1223,13 +1221,13 @@ 
      */
     private function parseArgumentDefs()
     {
-        if (! $this->peek(Token::PAREN_L)) {
+        if (!$this->peek(Token::PAREN_L)) {
             return new NodeList([]);
         }
 
         return $this->many(
             Token::PAREN_L,
-            function () {
+            function() {
                 return $this->parseInputValueDef();
             },
             Token::PAREN_R
@@ -1362,7 +1360,7 @@ 
         return $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseEnumValueDefinition();
                 },
                 Token::BRACE_R
@@ -1420,7 +1418,7 @@ 
         return $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
-                function () {
+                function() {
                     return $this->parseInputValueDef();
                 },
                 Token::BRACE_R
@@ -1559,7 +1557,7 @@ 
         $directives = $this->parseDirectives(true);
         $types      = $this->parseUnionMemberTypes();
         if (count($directives) === 0 &&
-            ! $types
+            !$types
         ) {
             throw $this->unexpected();
         }

src/Language/Visitor.php

Spacing +14 added lines, -14 removed lines

@@ -201,7 +201,7 @@ 
             $isEdited  = $isLeaving && count($edits) !== 0;
 
             if ($isLeaving) {
-                $key    = ! $ancestors ? $UNDEFINED : $path[count($path) - 1];
+                $key    = !$ancestors ? $UNDEFINED : $path[count($path) - 1];
                 $node   = $parent;
                 $parent = array_pop($ancestors);
 
@@ -255,8 +255,8 @@ 
             }
 
             $result = null;
-            if (! $node instanceof NodeList && ! is_array($node)) {
-                if (! ($node instanceof Node)) {
+            if (!$node instanceof NodeList && !is_array($node)) {
+                if (!($node instanceof Node)) {
                     throw new \Exception('Invalid AST Node: ' . json_encode($node));
                 }
 
@@ -270,7 +270,7 @@ 
                             if ($result->doBreak) {
                                 break;
                             }
-                            if (! $isLeaving && $result->doContinue) {
+                            if (!$isLeaving && $result->doContinue) {
                                 array_pop($path);
                                 continue;
                             }
@@ -282,8 +282,8 @@ 
                         }
 
                         $edits[] = [$key, $editValue];
-                        if (! $isLeaving) {
-                            if (! ($editValue instanceof Node)) {
+                        if (!$isLeaving) {
+                            if (!($editValue instanceof Node)) {
                                 array_pop($path);
                                 continue;
                             }
@@ -301,7 +301,7 @@ 
             if ($isLeaving) {
                 array_pop($path);
             } else {
-                $stack   = [
+                $stack = [
                     'inArray' => $inArray,
                     'index'   => $index,
                     'keys'    => $keys,
@@ -379,9 +379,9 @@ 
         $skipping      = new \SplFixedArray($visitorsCount);
 
         return [
-            'enter' => function (Node $node) use ($visitors, $skipping, $visitorsCount) {
+            'enter' => function(Node $node) use ($visitors, $skipping, $visitorsCount) {
                 for ($i = 0; $i < $visitorsCount; $i++) {
-                    if (! empty($skipping[$i])) {
+                    if (!empty($skipping[$i])) {
                         continue;
                     }
 
@@ -391,7 +391,7 @@ 
                         false
                     );
 
-                    if (! $fn) {
+                    if (!$fn) {
                         continue;
                     }
 
@@ -410,7 +410,7 @@ 
                     }
                 }
             },
-            'leave' => function (Node $node) use ($visitors, $skipping, $visitorsCount) {
+            'leave' => function(Node $node) use ($visitors, $skipping, $visitorsCount) {
                 for ($i = 0; $i < $visitorsCount; $i++) {
                     if (empty($skipping[$i])) {
                         $fn = self::getVisitFn(
@@ -446,7 +446,7 @@ 
     public static function visitWithTypeInfo(TypeInfo $typeInfo, $visitor)
     {
         return [
-            'enter' => function (Node $node) use ($typeInfo, $visitor) {
+            'enter' => function(Node $node) use ($typeInfo, $visitor) {
                 $typeInfo->enter($node);
                 $fn = self::getVisitFn($visitor, $node->kind, false);
 
@@ -464,7 +464,7 @@ 
 
                 return null;
             },
-            'leave' => function (Node $node) use ($typeInfo, $visitor) {
+            'leave' => function(Node $node) use ($typeInfo, $visitor) {
                 $fn     = self::getVisitFn($visitor, $node->kind, true);
                 $result = $fn ? call_user_func_array($fn, func_get_args()) : null;
                 $typeInfo->leave($node);
@@ -488,7 +488,7 @@ 
 
         $kindVisitor = $visitor[$kind] ?? null;
 
-        if (! $isLeaving && is_callable($kindVisitor)) {
+        if (!$isLeaving && is_callable($kindVisitor)) {
             // { Kind() {} }
             return $kindVisitor;
         }

src/Validator/ValidationContext.php

Spacing +11 added lines, -11 removed lines

@@ -98,7 +98,7 @@ 
     {
         $usages = $this->recursiveVariableUsages[$operation] ?? null;
 
-        if (! $usages) {
+        if (!$usages) {
             $usages    = $this->getVariableUsages($operation);
             $fragments = $this->getRecursivelyReferencedFragments($operation);
 
@@ -120,7 +120,7 @@ 
     {
         $usages = $this->variableUsages[$node] ?? null;
 
-        if (! $usages) {
+        if (!$usages) {
             $newUsages = [];
             $typeInfo  = new TypeInfo($this->schema);
             Visitor::visit(
@@ -128,10 +128,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
                         ) {
@@ -154,23 +154,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;
                     }
 
@@ -190,11 +190,11 @@ 
     public function getFragmentSpreads(HasSelectionSet $node)
     {
         $spreads = $this->fragmentSpreads[$node] ?? null;
-        if (! $spreads) {
+        if (!$spreads) {
             $spreads     = [];
             /** @var SelectionSetNode[] $setsToVisit */
             $setsToVisit = [$node->selectionSet];
-            while (! empty($setsToVisit)) {
+            while (!empty($setsToVisit)) {
                 $set = array_pop($setsToVisit);
 
                 for ($i = 0; $i < count($set->selections); $i++) {
@@ -219,7 +219,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/KnownArgumentNames.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::ARGUMENT => function (ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
+            NodeKind::ARGUMENT => function(ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
                 /** @var NodeList|Node[] $ancestors */
                 $argDef = $context->getArgument();
                 if ($argDef !== null) {
@@ -46,7 +46,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        function ($arg) {
+                                        function($arg) {
                                             return $arg->name;
                                         },
                                         $fieldDef->args
@@ -66,7 +66,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        function ($arg) {
+                                        function($arg) {
                                             return $arg->name;
                                         },
                                         $directive->args
@@ -87,7 +87,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));
         }
 
@@ -100,7 +100,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/LoneAnonymousOperation.php

Spacing +3 added lines, -3 removed lines

@@ -26,17 +26,17 @@
         $operationCount = 0;
 
         return [
-            NodeKind::DOCUMENT             => function (DocumentNode $node) use (&$operationCount) {
+            NodeKind::DOCUMENT             => function(DocumentNode $node) use (&$operationCount) {
                 $tmp = Utils::filter(
                     $node->definitions,
-                    function (Node $definition) {
+                    function(Node $definition) {
                         return $definition->kind === NodeKind::OPERATION_DEFINITION;
                     }
                 );
 
                 $operationCount = count($tmp);
             },
-            NodeKind::OPERATION_DEFINITION => function (OperationDefinitionNode $node) use (
+            NodeKind::OPERATION_DEFINITION => function(OperationDefinitionNode $node) use (
                 &$operationCount,
                 $context
             ) {