simPod / graphql-php

src/Validator/Rules/NoUnusedFragments.php

Spacing +4 added lines, -4 removed lines

@@ -26,18 +26,18 @@ 
         $this->fragmentDefs  = [];
 
         return [
-            NodeKind::OPERATION_DEFINITION => function ($node) {
+            NodeKind::OPERATION_DEFINITION => function($node) {
                 $this->operationDefs[] = $node;
 
                 return Visitor::skipNode();
             },
-            NodeKind::FRAGMENT_DEFINITION  => function (FragmentDefinitionNode $def) {
+            NodeKind::FRAGMENT_DEFINITION  => function(FragmentDefinitionNode $def) {
                 $this->fragmentDefs[] = $def;
 
                 return Visitor::skipNode();
             },
             NodeKind::DOCUMENT             => [
-                'leave' => function () use ($context) {
+                'leave' => function() use ($context) {
                     $fragmentNameUsed = [];
 
                     foreach ($this->operationDefs as $operation) {
@@ -48,7 +48,7 @@ 
 
                     foreach ($this->fragmentDefs as $fragmentDef) {
                         $fragName = $fragmentDef->name->value;
-                        if (! empty($fragmentNameUsed[$fragName])) {
+                        if (!empty($fragmentNameUsed[$fragName])) {
                             continue;
                         }
 

src/Validator/Rules/ValuesOfCorrectType.php

Spacing +21 added lines, -21 removed lines

@@ -45,9 +45,9 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::NULL         => function (NullValueNode $node) use ($context) {
+            NodeKind::NULL         => function(NullValueNode $node) use ($context) {
                 $type = $context->getInputType();
-                if (! ($type instanceof NonNull)) {
+                if (!($type instanceof NonNull)) {
                     return;
                 }
 
@@ -58,21 +58,21 @@ 
                     )
                 );
             },
-            NodeKind::LST          => function (ListValueNode $node) use ($context) {
+            NodeKind::LST          => function(ListValueNode $node) use ($context) {
                 // Note: TypeInfo will traverse into a list's item type, so look to the
                 // parent input type to check if it is a list.
                 $type = Type::getNullableType($context->getParentInputType());
-                if (! $type instanceof ListOfType) {
+                if (!$type instanceof ListOfType) {
                     $this->isValidScalar($context, $node);
 
                     return Visitor::skipNode();
                 }
             },
-            NodeKind::OBJECT       => function (ObjectValueNode $node) use ($context) {
+            NodeKind::OBJECT       => function(ObjectValueNode $node) use ($context) {
                 // Note: TypeInfo will traverse into a list's item type, so look to the
                 // parent input type to check if it is a list.
                 $type = Type::getNamedType($context->getInputType());
-                if (! $type instanceof InputObjectType) {
+                if (!$type instanceof InputObjectType) {
                     $this->isValidScalar($context, $node);
 
                     return Visitor::skipNode();
@@ -82,7 +82,7 @@ 
                 $nodeFields   = iterator_to_array($node->fields);
                 $fieldNodeMap = array_combine(
                     array_map(
-                        function ($field) {
+                        function($field) {
                             return $field->name->value;
                         },
                         $nodeFields
@@ -91,7 +91,7 @@ 
                 );
                 foreach ($inputFields as $fieldName => $fieldDef) {
                     $fieldType = $fieldDef->getType();
-                    if (isset($fieldNodeMap[$fieldName]) || ! ($fieldType instanceof NonNull)) {
+                    if (isset($fieldNodeMap[$fieldName]) || !($fieldType instanceof NonNull)) {
                         continue;
                     }
 
@@ -103,10 +103,10 @@ 
                     );
                 }
             },
-            NodeKind::OBJECT_FIELD => function (ObjectFieldNode $node) use ($context) {
+            NodeKind::OBJECT_FIELD => function(ObjectFieldNode $node) use ($context) {
                 $parentType = Type::getNamedType($context->getParentInputType());
                 $fieldType  = $context->getInputType();
-                if ($fieldType || ! ($parentType instanceof InputObjectType)) {
+                if ($fieldType || !($parentType instanceof InputObjectType)) {
                     return;
                 }
 
@@ -114,7 +114,7 @@ 
                     $node->name->value,
                     array_keys($parentType->getFields())
                 );
-                $didYouMean  = $suggestions
+                $didYouMean = $suggestions
                     ? 'Did you mean ' . Utils::orList($suggestions) . '?'
                     : null;
 
@@ -125,11 +125,11 @@ 
                     )
                 );
             },
-            NodeKind::ENUM         => function (EnumValueNode $node) use ($context) {
+            NodeKind::ENUM         => function(EnumValueNode $node) use ($context) {
                 $type = Type::getNamedType($context->getInputType());
-                if (! $type instanceof EnumType) {
+                if (!$type instanceof EnumType) {
                     $this->isValidScalar($context, $node);
-                } elseif (! $type->getValue($node->value)) {
+                } elseif (!$type->getValue($node->value)) {
                     $context->reportError(
                         new Error(
                             self::badValueMessage(
@@ -142,16 +142,16 @@ 
                     );
                 }
             },
-            NodeKind::INT          => function (IntValueNode $node) use ($context) {
+            NodeKind::INT          => function(IntValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::FLOAT        => function (FloatValueNode $node) use ($context) {
+            NodeKind::FLOAT        => function(FloatValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::STRING       => function (StringValueNode $node) use ($context) {
+            NodeKind::STRING       => function(StringValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
-            NodeKind::BOOLEAN      => function (BooleanValueNode $node) use ($context) {
+            NodeKind::BOOLEAN      => function(BooleanValueNode $node) use ($context) {
                 $this->isValidScalar($context, $node);
             },
         ];
@@ -168,13 +168,13 @@ 
         // Report any error at the full type expected by the location.
         $locationType = $context->getInputType();
 
-        if (! $locationType) {
+        if (!$locationType) {
             return;
         }
 
         $type = Type::getNamedType($locationType);
 
-        if (! $type instanceof ScalarType) {
+        if (!$type instanceof ScalarType) {
             $context->reportError(
                 new Error(
                     self::badValueMessage(
@@ -234,7 +234,7 @@ 
             $suggestions = Utils::suggestionList(
                 Printer::doPrint($node),
                 array_map(
-                    function (EnumValueDefinition $value) {
+                    function(EnumValueDefinition $value) {
                         return $value->name;
                     },
                     $type->getValues()

src/Validator/Rules/ProvidedNonNullArguments.php

Spacing +6 added lines, -6 removed lines

@@ -19,10 +19,10 @@ 
     {
         return [
             NodeKind::FIELD     => [
-                'leave' => function (FieldNode $fieldNode) use ($context) {
+                'leave' => function(FieldNode $fieldNode) use ($context) {
                     $fieldDef = $context->getFieldDef();
 
-                    if (! $fieldDef) {
+                    if (!$fieldDef) {
                         return Visitor::skipNode();
                     }
                     $argNodes = $fieldNode->arguments ?: [];
@@ -33,7 +33,7 @@ 
                     }
                     foreach ($fieldDef->args as $argDef) {
                         $argNode = $argNodeMap[$argDef->name] ?? null;
-                        if ($argNode || ! ($argDef->getType() instanceof NonNull)) {
+                        if ($argNode || !($argDef->getType() instanceof NonNull)) {
                             continue;
                         }
 
@@ -45,9 +45,9 @@ 
                 },
             ],
             NodeKind::DIRECTIVE => [
-                'leave' => function (DirectiveNode $directiveNode) use ($context) {
+                'leave' => function(DirectiveNode $directiveNode) use ($context) {
                     $directiveDef = $context->getDirective();
-                    if (! $directiveDef) {
+                    if (!$directiveDef) {
                         return Visitor::skipNode();
                     }
                     $argNodes   = $directiveNode->arguments ?: [];
@@ -58,7 +58,7 @@ 
 
                     foreach ($directiveDef->args as $argDef) {
                         $argNode = $argNodeMap[$argDef->name] ?? null;
-                        if ($argNode || ! ($argDef->getType() instanceof NonNull)) {
+                        if ($argNode || !($argDef->getType() instanceof NonNull)) {
                             continue;
                         }
 

src/Validator/DocumentValidator.php

Spacing +2 added lines, -2 removed lines

@@ -114,7 +114,7 @@ 
      */
     public static function allRules()
     {
-        if (! self::$initRules) {
+        if (!self::$initRules) {
             static::$rules     = array_merge(static::defaultRules(), self::securityRules(), self::$rules);
             static::$initRules = true;
         }
@@ -235,7 +235,7 @@ 
         return is_array($value)
             ? count(array_filter(
                 $value,
-                function ($item) {
+                function($item) {
                     return $item instanceof \Exception || $item instanceof \Throwable;
                 }
             )) === count($value)

src/Executor/ExecutionResult.php

Spacing +3 added lines, -3 removed lines

@@ -133,8 +133,8 @@ 
     {
         $result = [];
 
-        if (! empty($this->errors)) {
-            $errorsHandler = $this->errorsHandler ?: function (array $errors, callable $formatter) {
+        if (!empty($this->errors)) {
+            $errorsHandler = $this->errorsHandler ?: function(array $errors, callable $formatter) {
                 return array_map($formatter, $errors);
             };
 
@@ -148,7 +148,7 @@ 
             $result['data'] = $this->data;
         }
 
-        if (! empty($this->extensions)) {
+        if (!empty($this->extensions)) {
             $result['extensions'] = (array) $this->extensions;
         }
 

src/Executor/Values.php

Spacing +7 added lines, -7 removed lines

@@ -52,7 +52,7 @@ 
             /** @var InputType|Type $varType */
             $varType = TypeInfo::typeFromAST($schema, $varDefNode->type);
 
-            if (! Type::isInputType($varType)) {
+            if (!Type::isInputType($varType)) {
                 $errors[] = new Error(
                     sprintf(
                         'Variable "$%s" expected value of type "%s" which cannot be used as an input type.',
@@ -62,7 +62,7 @@ 
                     [$varDefNode->type]
                 );
             } else {
-                if (! array_key_exists($varName, $inputs)) {
+                if (!array_key_exists($varName, $inputs)) {
                     if ($varType instanceof NonNull) {
                         $errors[] = new Error(
                             sprintf(
@@ -80,7 +80,7 @@ 
                     $coerced = Value::coerceValue($value, $varType, $varDefNode);
                     /** @var Error[] $coercionErrors */
                     $coercionErrors = $coerced['errors'];
-                    if (! empty($coercionErrors)) {
+                    if (!empty($coercionErrors)) {
                         $messagePrelude = sprintf(
                             'Variable "$%s" got invalid value %s; ',
                             $varName,
@@ -125,7 +125,7 @@ 
         if (isset($node->directives) && $node->directives instanceof NodeList) {
             $directiveNode = Utils::find(
                 $node->directives,
-                function (DirectiveNode $directive) use ($directiveDef) {
+                function(DirectiveNode $directive) use ($directiveDef) {
                     return $directive->name->value === $directiveDef->name;
                 }
             );
@@ -162,7 +162,7 @@ 
         /** @var ArgumentNode[] $argNodeMap */
         $argNodeMap = $argNodes ? Utils::keyMap(
             $argNodes,
-            function (ArgumentNode $arg) {
+            function(ArgumentNode $arg) {
                 return $arg->name->value;
             }
         ) : [];
@@ -172,7 +172,7 @@ 
             $argType      = $argDef->getType();
             $argumentNode = $argNodeMap[$name] ?? null;
 
-            if (! $argumentNode) {
+            if (!$argumentNode) {
                 if ($argDef->defaultValueExists()) {
                     $coercedValues[$name] = $argDef->defaultValue;
                 } elseif ($argType instanceof NonNull) {
@@ -242,7 +242,7 @@ 
 
         return $errors
             ? array_map(
-                function (Throwable $error) {
+                function(Throwable $error) {
                     return $error->getMessage();
                 },
                 $errors

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();