simPod / graphql-php

src/Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -80,12 +80,12 @@
         // TODO: rework with generators when PHP minimum required version is changed to 5.5+
         $promisesOrValues = Utils::map(
             $promisesOrValues,
-            function ($item) {
+            function($item) {
                 return $item instanceof Promise ? $item->adoptedPromise : $item;
             }
         );
 
-        $promise = all($promisesOrValues)->then(function ($values) use ($promisesOrValues) {
+        $promise = all($promisesOrValues)->then(function($values) use ($promisesOrValues) {
             $orderedResults = [];
 
             foreach ($promisesOrValues as $key => $value) {

src/Executor/Promise/Adapter/SyncPromise.php

Spacing +7 added lines, -7 removed lines

@@ -40,7 +40,7 @@ 
     public static function runQueue()
     {
         $q = self::$queue;
-        while ($q && ! $q->isEmpty()) {
+        while ($q && !$q->isEmpty()) {
             $task = $q->dequeue();
             $task();
         }
@@ -55,10 +55,10 @@ 
                 }
                 if (is_object($value) && method_exists($value, 'then')) {
                     $value->then(
-                        function ($resolvedValue) {
+                        function($resolvedValue) {
                             $this->resolve($resolvedValue);
                         },
-                        function ($reason) {
+                        function($reason) {
                             $this->reject($reason);
                         }
                     );
@@ -84,7 +84,7 @@ 
 
     public function reject($reason)
     {
-        if (! $reason instanceof \Exception && ! $reason instanceof \Throwable) {
+        if (!$reason instanceof \Exception && !$reason instanceof \Throwable) {
             throw new \Exception('SyncPromise::reject() has to be called with an instance of \Throwable');
         }
 
@@ -114,7 +114,7 @@ 
         );
 
         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;
 
@@ -151,10 +151,10 @@ 
 
     public function then(?callable $onFulfilled = null, ?callable $onRejected = null)
     {
-        if ($this->state === self::REJECTED && ! $onRejected) {
+        if ($this->state === self::REJECTED && !$onRejected) {
             return $this;
         }
-        if ($this->state === self::FULFILLED && ! $onFulfilled) {
+        if ($this->state === self::FULFILLED && !$onFulfilled) {
             return $this;
         }
         $tmp             = new self();

src/Executor/Promise/Adapter/SyncPromiseAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function convertThenable($thenable)
     {
-        if (! $thenable instanceof Deferred) {
+        if (!$thenable instanceof Deferred) {
             throw new InvariantViolation('Expected instance of GraphQL\Deferred, got ' . Utils::printSafe($thenable));
         }
 
@@ -111,7 +111,7 @@ 
             if ($promiseOrValue instanceof Promise) {
                 $result[$index] = null;
                 $promiseOrValue->then(
-                    function ($value) use ($index, &$count, $total, &$result, $all) {
+                    function($value) use ($index, &$count, $total, &$result, $all) {
                         $result[$index] = $value;
                         $count++;
                         if ($count < $total) {
@@ -146,7 +146,7 @@ 
         $promiseQueue = SyncPromise::getQueue();
 
         while ($promise->adoptedPromise->state === SyncPromise::PENDING &&
-            ! ($dfdQueue->isEmpty() && $promiseQueue->isEmpty())
+            !($dfdQueue->isEmpty() && $promiseQueue->isEmpty())
         ) {
             Deferred::runQueue();
             SyncPromise::runQueue();

src/Executor/Promise/Promise.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function __construct($adoptedPromise, PromiseAdapter $adapter)
     {
-        Utils::invariant(! $adoptedPromise instanceof self, 'Expecting promise from adapted system, got ' . __CLASS__);
+        Utils::invariant(!$adoptedPromise instanceof self, 'Expecting promise from adapted system, got ' . __CLASS__);
 
         $this->adapter        = $adapter;
         $this->adoptedPromise = $adoptedPromise;

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/Printer.php

Spacing +46 added lines, -46 removed lines

@@ -94,16 +94,16 @@ 
             $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, ', '), ')');
@@ -111,17 +111,17 @@ 
                         $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(
@@ -135,15 +135,15 @@ 
                             ' '
                         );
                     },
-                    NodeKind::ARGUMENT                  => function (ArgumentNode $node) {
+                    NodeKind::ARGUMENT                  => function(ArgumentNode $node) {
                         return $node->name . ': ' . $node->value;
                     },
 
                     // Fragments
-                    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(
                             [
                             '...',
@@ -154,7 +154,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)
@@ -165,56 +165,56 @@ 
                     },
 
                     // Value
-                    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');
                         }
 
                         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;
                     },
 
                     // DirectiveNode
-                    NodeKind::DIRECTIVE                 => function (DirectiveNode $node) {
+                    NodeKind::DIRECTIVE                 => function(DirectiveNode $node) {
                         return '@' . $node->name . $this->wrap('(', $this->join($node->arguments, ', '), ')');
                     },
 
                     // Type
-                    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 . '!';
                     },
 
                     // Type System Definitions
-                    NodeKind::SCHEMA_DEFINITION         => function (SchemaDefinitionNode $def) {
+                    NodeKind::SCHEMA_DEFINITION         => function(SchemaDefinitionNode $def) {
                         return $this->join(
                             [
                             'schema',
@@ -224,16 +224,16 @@ 
                             ' '
                         );
                     },
-                    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 (
+                    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 (
+                    NodeKind::OBJECT_TYPE_DEFINITION       => $this->addDescription(function(
                         ObjectTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -247,14 +247,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 (
+                    NodeKind::INPUT_VALUE_DEFINITION       => $this->addDescription(function(
                         InputValueDefinitionNode $def
                     ) {
                         return $this->join(
@@ -266,7 +266,7 @@ 
                             ' '
                         );
                     }),
-                    NodeKind::INTERFACE_TYPE_DEFINITION    => $this->addDescription(function (
+                    NodeKind::INTERFACE_TYPE_DEFINITION    => $this->addDescription(function(
                         InterfaceTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -279,7 +279,7 @@ 
                             ' '
                         );
                     }),
-                    NodeKind::UNION_TYPE_DEFINITION        => $this->addDescription(function (
+                    NodeKind::UNION_TYPE_DEFINITION        => $this->addDescription(function(
                         UnionTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -294,7 +294,7 @@ 
                             ' '
                         );
                     }),
-                    NodeKind::ENUM_TYPE_DEFINITION         => $this->addDescription(function (
+                    NodeKind::ENUM_TYPE_DEFINITION         => $this->addDescription(function(
                         EnumTypeDefinitionNode $def
                     ) {
                         return $this->join(
@@ -307,12 +307,12 @@ 
                             ' '
                         );
                     }),
-                    NodeKind::ENUM_VALUE_DEFINITION        => $this->addDescription(function (
+                    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(
@@ -325,7 +325,7 @@ 
                             ' '
                         );
                     }),
-                    NodeKind::SCALAR_TYPE_EXTENSION        => function (ScalarTypeExtensionNode $def) {
+                    NodeKind::SCALAR_TYPE_EXTENSION        => function(ScalarTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend scalar',
@@ -335,7 +335,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::OBJECT_TYPE_EXTENSION        => function (ObjectTypeExtensionNode $def) {
+                    NodeKind::OBJECT_TYPE_EXTENSION        => function(ObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend type',
@@ -347,7 +347,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::INTERFACE_TYPE_EXTENSION     => function (InterfaceTypeExtensionNode $def) {
+                    NodeKind::INTERFACE_TYPE_EXTENSION     => function(InterfaceTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend interface',
@@ -358,7 +358,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::UNION_TYPE_EXTENSION         => function (UnionTypeExtensionNode $def) {
+                    NodeKind::UNION_TYPE_EXTENSION         => function(UnionTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend union',
@@ -371,7 +371,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::ENUM_TYPE_EXTENSION          => function (EnumTypeExtensionNode $def) {
+                    NodeKind::ENUM_TYPE_EXTENSION          => function(EnumTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend enum',
@@ -382,7 +382,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION  => function (InputObjectTypeExtensionNode $def) {
+                    NodeKind::INPUT_OBJECT_TYPE_EXTENSION  => function(InputObjectTypeExtensionNode $def) {
                         return $this->join(
                             [
                             'extend input',
@@ -393,7 +393,7 @@ 
                             ' '
                         );
                     },
-                    NodeKind::DIRECTIVE_DEFINITION         => $this->addDescription(function (
+                    NodeKind::DIRECTIVE_DEFINITION         => $this->addDescription(function(
                         DirectiveDefinitionNode $def
                     ) {
                         return 'directive @'
@@ -408,7 +408,7 @@ 
 
     public function addDescription(\Closure $cb)
     {
-        return function ($node) use ($cb) {
+        return function($node) use ($cb) {
             return $this->join([$node->description, $cb($node)], "\n");
         };
     }
@@ -455,8 +455,8 @@ 
                 $separator,
                 Utils::filter(
                     $maybeArray,
-                    function ($x) {
-                        return ! ! $x;
+                    function($x) {
+                        return !!$x;
                     }
                 )
             )

src/Language/Parser.php

Spacing +23 added lines, -25 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
@@ -1500,7 +1498,7 @@ 
         $directives = $this->parseDirectives(true);
         $fields     = $this->parseFieldsDefinition();
 
-        if (! $interfaces &&
+        if (!$interfaces &&
             count($directives) === 0 &&
             count($fields) === 0
         ) {
@@ -1559,7 +1557,7 @@ 
         $directives = $this->parseDirectives(true);
         $types      = $this->parseUnionMemberTypes();
         if (count($directives) === 0 &&
-            ! $types
+            !$types
         ) {
             throw $this->unexpected();
         }

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;
         }