simPod / graphql-php

src/GraphQL.php

Spacing +3 added lines, -3 removed lines

@@ -138,7 +138,7 @@ 
                 $queryComplexity->setRawVariableValues($variableValues);
             } else {
                 foreach ($validationRules as $rule) {
-                    if (! ($rule instanceof QueryComplexity)) {
+                    if (!($rule instanceof QueryComplexity)) {
                         continue;
                     }
 
@@ -148,7 +148,7 @@ 
 
             $validationErrors = DocumentValidator::validate($schema, $documentNode, $validationRules);
 
-            if (! empty($validationErrors)) {
+            if (!empty($validationErrors)) {
                 return $promiseAdapter->createFulfilled(
                     new ExecutionResult(null, $validationErrors)
                 );
@@ -208,7 +208,7 @@ 
         if ($promiseAdapter instanceof SyncPromiseAdapter) {
             $result = $promiseAdapter->wait($result)->toArray();
         } else {
-            $result = $result->then(static function (ExecutionResult $r) {
+            $result = $result->then(static function(ExecutionResult $r) {
                 return $r->toArray();
             });
         }

src/Experimental/Executor/Collector.php

Spacing +10 added lines, -10 removed lines

@@ -120,7 +120,7 @@ 
             $fieldName = $fieldNode->name->value;
 
             $argumentValueMap = null;
-            if (! empty($fieldNode->arguments)) {
+            if (!empty($fieldNode->arguments)) {
                 foreach ($fieldNode->arguments as $argumentNode) {
                     $argumentValueMap                             = $argumentValueMap ?? [];
                     $argumentValueMap[$argumentNode->name->value] = $argumentNode->value;
@@ -128,8 +128,8 @@ 
             }
 
             if ($fieldName !== Introspection::TYPE_NAME_FIELD_NAME &&
-                ! ($runtimeType === $this->schema->getQueryType() && ($fieldName === Introspection::SCHEMA_FIELD_NAME || $fieldName === Introspection::TYPE_FIELD_NAME)) &&
-                ! $runtimeType->hasField($fieldName)
+                !($runtimeType === $this->schema->getQueryType() && ($fieldName === Introspection::SCHEMA_FIELD_NAME || $fieldName === Introspection::TYPE_FIELD_NAME)) &&
+                !$runtimeType->hasField($fieldName)
             ) {
                 // do not emit error
                 continue;
@@ -148,7 +148,7 @@ 
         foreach ($selectionSet->selections as $selection) {
             /** @var FieldNode|FragmentSpreadNode|InlineFragmentNode $selection */
 
-            if (! empty($selection->directives)) {
+            if (!empty($selection->directives)) {
                 foreach ($selection->directives as $directiveNode) {
                     if ($directiveNode->name->value === Directive::SKIP_NAME) {
                         /** @var ValueNode|null $condition */
@@ -199,7 +199,7 @@ 
 
                 $resultName = $selection->alias ? $selection->alias->value : $selection->name->value;
 
-                if (! isset($this->fields[$resultName])) {
+                if (!isset($this->fields[$resultName])) {
                     $this->fields[$resultName] = [];
                 }
 
@@ -211,7 +211,7 @@ 
 
                 if (isset($this->visitedFragments[$fragmentName])) {
                     continue;
-                } elseif (! isset($this->fragments[$fragmentName])) {
+                } elseif (!isset($this->fragments[$fragmentName])) {
                     $this->runtime->addError(new Error(
                         sprintf('Fragment "%s" does not exist.', $fragmentName),
                         $selection
@@ -224,7 +224,7 @@ 
                 $fragmentDefinition = $this->fragments[$fragmentName];
                 $conditionTypeName  = $fragmentDefinition->typeCondition->name->value;
 
-                if (! $this->schema->hasType($conditionTypeName)) {
+                if (!$this->schema->hasType($conditionTypeName)) {
                     $this->runtime->addError(new Error(
                         sprintf('Cannot spread fragment "%s", type "%s" does not exist.', $fragmentName, $conditionTypeName),
                         $selection
@@ -239,7 +239,7 @@ 
                         continue;
                     }
                 } elseif ($conditionType instanceof AbstractType) {
-                    if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                    if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                         continue;
                     }
                 }
@@ -251,7 +251,7 @@ 
                 if ($selection->typeCondition !== null) {
                     $conditionTypeName = $selection->typeCondition->name->value;
 
-                    if (! $this->schema->hasType($conditionTypeName)) {
+                    if (!$this->schema->hasType($conditionTypeName)) {
                         $this->runtime->addError(new Error(
                             sprintf('Cannot spread inline fragment, type "%s" does not exist.', $conditionTypeName),
                             $selection
@@ -266,7 +266,7 @@ 
                             continue;
                         }
                     } elseif ($conditionType instanceof AbstractType) {
-                        if (! $this->schema->isPossibleType($conditionType, $runtimeType)) {
+                        if (!$this->schema->isPossibleType($conditionType, $runtimeType)) {
                             continue;
                         }
                     }

src/Experimental/Executor/CoroutineExecutor.php

Spacing +17 added lines, -17 removed lines

@@ -172,7 +172,7 @@ 
         $this->collector = new Collector($this->schema, $this);
         $this->collector->initialize($this->documentNode, $this->operationName);
 
-        if (! empty($this->errors)) {
+        if (!empty($this->errors)) {
             return $this->promiseAdapter->createFulfilled($this->finishExecute(null, $this->errors));
         }
 
@@ -182,7 +182,7 @@ 
             $this->rawVariableValues ?: []
         );
 
-        if (! empty($errors)) {
+        if (!empty($errors)) {
             return $this->promiseAdapter->createFulfilled($this->finishExecute(null, $errors));
         }
 
@@ -203,11 +203,11 @@ 
             );
 
             $fieldDefinition = $this->findFieldDefinition($ctx);
-            if (! $fieldDefinition->getType() instanceof NonNull) {
+            if (!$fieldDefinition->getType() instanceof NonNull) {
                 $ctx->nullFence = [$shared->resultName];
             }
 
-            if ($this->collector->operation->operation === 'mutation' && ! $this->queue->isEmpty()) {
+            if ($this->collector->operation->operation === 'mutation' && !$this->queue->isEmpty()) {
                 $this->schedule->enqueue($ctx);
             } else {
                 $this->queue->enqueue(new Strand($this->spawn($ctx)));
@@ -217,7 +217,7 @@ 
         $this->run();
 
         if ($this->pending > 0) {
-            return $this->promiseAdapter->create(function (callable $resolve) {
+            return $this->promiseAdapter->create(function(callable $resolve) {
                 $this->doResolve = $resolve;
             });
         }
@@ -265,7 +265,7 @@ 
     private function run()
     {
         RUN:
-        while (! $this->queue->isEmpty()) {
+        while (!$this->queue->isEmpty()) {
             /** @var Strand $strand */
             $strand = $this->queue->dequeue();
 
@@ -298,13 +298,13 @@ 
                         $this->promiseAdapter
                             ->convertThenable($value)
                             ->then(
-                                function ($value) use ($strand) {
+                                function($value) use ($strand) {
                                     $strand->success = true;
                                     $strand->value   = $value;
                                     $this->queue->enqueue($strand);
                                     $this->done();
                                 },
-                                function (Throwable $throwable) use ($strand) {
+                                function(Throwable $throwable) use ($strand) {
                                     $strand->success = false;
                                     $strand->value   = $throwable;
                                     $this->queue->enqueue($strand);
@@ -416,7 +416,7 @@ 
 
             $value = $resolve($ctx->value, $arguments, $this->contextValue, $ctx->resolveInfo);
 
-            if (! $this->completeValueFast($ctx, $returnType, $value, $ctx->path, $returnValue)) {
+            if (!$this->completeValueFast($ctx, $returnType, $value, $ctx->path, $returnValue)) {
                 $returnValue = yield $this->completeValue(
                     $ctx,
                     $returnType,
@@ -438,12 +438,12 @@ 
         if ($returnValue !== self::$undefined) {
             $ctx->result->{$ctx->shared->resultName} = $returnValue;
         } elseif ($ctx->resolveInfo !== null && $ctx->resolveInfo->returnType instanceof NonNull) { // !!! $ctx->resolveInfo might not have been initialized yet
-            $result =& $this->rootResult;
+            $result = & $this->rootResult;
             foreach ($ctx->nullFence ?? [] as $key) {
                 if (is_string($key)) {
-                    $result =& $result->{$key};
+                    $result = & $result->{$key};
                 } else {
-                    $result =& $result[$key];
+                    $result = & $result[$key];
                 }
             }
             $result = null;
@@ -485,7 +485,7 @@ 
             $type    = $type->getWrappedType();
         }
 
-        if (! $type instanceof LeafType) {
+        if (!$type instanceof LeafType) {
             return false;
         }
 
@@ -617,7 +617,7 @@ 
                 $itemPath[] = $index; // !!! use arrays COW semantics
 
                 try {
-                    if (! $this->completeValueFast($ctx, $itemType, $itemValue, $itemPath, $itemReturnValue)) {
+                    if (!$this->completeValueFast($ctx, $itemType, $itemValue, $itemPath, $itemReturnValue)) {
                         $itemReturnValue = yield $this->completeValue($ctx, $itemType, $itemValue, $itemPath, $nullFence);
                     }
                 } catch (Throwable $reason) {
@@ -708,7 +708,7 @@ 
 
                         $returnValue = self::$undefined;
                         goto CHECKED_RETURN;
-                    } elseif (! $objectType instanceof ObjectType) {
+                    } elseif (!$objectType instanceof ObjectType) {
                         $this->addError(Error::createLocatedError(
                             new InvariantViolation(sprintf(
                                 'Abstract type %s must resolve to an Object type at ' .
@@ -728,7 +728,7 @@ 
 
                         $returnValue = null;
                         goto CHECKED_RETURN;
-                    } elseif (! $this->schema->isPossibleType($type, $objectType)) {
+                    } elseif (!$this->schema->isPossibleType($type, $objectType)) {
                         $this->addError(Error::createLocatedError(
                             new InvariantViolation(sprintf(
                                 'Runtime Object type "%s" is not a possible type for "%s".',
@@ -780,7 +780,7 @@ 
                 if ($typeCheck !== null) {
                     // !!! $objectType->isTypeOf() might return promise, yield to resolve
                     $typeCheck = yield $typeCheck;
-                    if (! $typeCheck) {
+                    if (!$typeCheck) {
                         $this->addError(Error::createLocatedError(
                             sprintf('Expected value of type "%s" but got: %s.', $type->name, Utils::printSafe($value)),
                             $ctx->shared->fieldNodes,

src/Utils/BuildSchema.php

Spacing +12 added lines, -12 removed lines

@@ -115,7 +115,7 @@ 
                 case NodeKind::UNION_TYPE_DEFINITION:
                 case NodeKind::INPUT_OBJECT_TYPE_DEFINITION:
                     $typeName = $d->name->value;
-                    if (! empty($this->nodeMap[$typeName])) {
+                    if (!empty($this->nodeMap[$typeName])) {
                         throw new Error(sprintf('Type "%s" was defined more than once.', $typeName));
                     }
                     $typeDefs[]               = $d;
@@ -138,14 +138,14 @@ 
         $DefinitionBuilder = new ASTDefinitionBuilder(
             $this->nodeMap,
             $this->options,
-            static function ($typeName) {
+            static function($typeName) {
                 throw new Error('Type "' . $typeName . '" not found in document.');
             },
             $this->typeConfigDecorator
         );
 
         $directives = array_map(
-            static function ($def) use ($DefinitionBuilder) {
+            static function($def) use ($DefinitionBuilder) {
                 return $DefinitionBuilder->buildDirective($def);
             },
             $directiveDefs
@@ -154,31 +154,31 @@ 
         // If specified directives were not explicitly declared, add them.
         $skip = array_reduce(
             $directives,
-            static function ($hasSkip, $directive) {
+            static function($hasSkip, $directive) {
                 return $hasSkip || $directive->name === 'skip';
             }
         );
-        if (! $skip) {
+        if (!$skip) {
             $directives[] = Directive::skipDirective();
         }
 
         $include = array_reduce(
             $directives,
-            static function ($hasInclude, $directive) {
+            static function($hasInclude, $directive) {
                 return $hasInclude || $directive->name === 'include';
             }
         );
-        if (! $include) {
+        if (!$include) {
             $directives[] = Directive::includeDirective();
         }
 
         $deprecated = array_reduce(
             $directives,
-            static function ($hasDeprecated, $directive) {
+            static function($hasDeprecated, $directive) {
                 return $hasDeprecated || $directive->name === 'deprecated';
             }
         );
-        if (! $deprecated) {
+        if (!$deprecated) {
             $directives[] = Directive::deprecatedDirective();
         }
 
@@ -196,12 +196,12 @@ 
             'subscription' => isset($operationTypes['subscription'])
                 ? $DefinitionBuilder->buildType($operationTypes['subscription'])
                 : null,
-            'typeLoader'   => static function ($name) use ($DefinitionBuilder) {
+            'typeLoader'   => static function($name) use ($DefinitionBuilder) {
                 return $DefinitionBuilder->buildType($name);
             },
             'directives'   => $directives,
             'astNode'      => $schemaDef,
-            'types'        => function () use ($DefinitionBuilder) {
+            'types'        => function() use ($DefinitionBuilder) {
                 $types = [];
                 foreach ($this->nodeMap as $name => $def) {
                     $types[] = $DefinitionBuilder->buildType($def->name->value);
@@ -231,7 +231,7 @@ 
                 throw new Error(sprintf('Must provide only one %s type in schema.', $operation));
             }
 
-            if (! isset($this->nodeMap[$typeName])) {
+            if (!isset($this->nodeMap[$typeName])) {
                 throw new Error(sprintf('Specified %s type "%s" not found in document.', $operation, $typeName));
             }
 

src/Utils/Utils.php

Spacing +14 added lines, -14 removed lines

@@ -84,13 +84,13 @@ 
     public static function assign($obj, array $vars, array $requiredKeys = [])
     {
         foreach ($requiredKeys as $key) {
-            if (! isset($vars[$key])) {
+            if (!isset($vars[$key])) {
                 throw new InvalidArgumentException(sprintf('Key %s is expected to be set and not to be null', $key));
             }
         }
 
         foreach ($vars as $key => $value) {
-            if (! property_exists($obj, $key)) {
+            if (!property_exists($obj, $key)) {
                 $cls = get_class($obj);
                 Warning::warn(
                     sprintf("Trying to set non-existing property '%s' on class '%s'", $key, $cls),
@@ -141,10 +141,10 @@ 
         $result = [];
         $assoc  = false;
         foreach ($traversable as $key => $value) {
-            if (! $assoc && ! is_int($key)) {
+            if (!$assoc && !is_int($key)) {
                 $assoc = true;
             }
-            if (! $predicate($value, $key)) {
+            if (!$predicate($value, $key)) {
                 continue;
             }
 
@@ -216,7 +216,7 @@ 
         $map = [];
         foreach ($traversable as $key => $value) {
             $newKey = $keyFn($value, $key);
-            if (! is_scalar($newKey)) {
+            if (!is_scalar($newKey)) {
                 continue;
             }
 
@@ -295,7 +295,7 @@ 
     public static function every($traversable, callable $predicate)
     {
         foreach ($traversable as $key => $value) {
-            if (! $predicate($value, $key)) {
+            if (!$predicate($value, $key)) {
                 return false;
             }
         }
@@ -309,7 +309,7 @@ 
      */
     public static function invariant($test, $message = '')
     {
-        if (! $test) {
+        if (!$test) {
             if (func_num_args() > 2) {
                 $args = func_get_args();
                 array_shift($args);
@@ -446,10 +446,10 @@ 
      */
     public static function ord($char, $encoding = 'UTF-8')
     {
-        if (! $char && $char !== '0') {
+        if (!$char && $char !== '0') {
             return 0;
         }
-        if (! isset($char[1])) {
+        if (!isset($char[1])) {
             return ord($char);
         }
         if ($encoding !== 'UCS-4BE') {
@@ -527,7 +527,7 @@ 
             );
         }
 
-        if (! preg_match('/^[_a-zA-Z][_a-zA-Z0-9]*$/', $name)) {
+        if (!preg_match('/^[_a-zA-Z][_a-zA-Z0-9]*$/', $name)) {
             return new Error(
                 sprintf('Names must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but "%s" does not.', $name),
                 $node
@@ -547,9 +547,9 @@ 
      */
     public static function withErrorHandling(callable $fn, array &$errors)
     {
-        return static function () use ($fn, &$errors) {
+        return static function() use ($fn, &$errors) {
             // Catch custom errors (to report them in query results)
-            set_error_handler(static function ($severity, $message, $file, $line) use (&$errors) {
+            set_error_handler(static function($severity, $message, $file, $line) use (&$errors) {
                 $errors[] = new ErrorException($message, 0, $severity, $file, $line);
             });
 
@@ -569,7 +569,7 @@ 
     public static function quotedOrList(array $items)
     {
         $items = array_map(
-            static function ($item) {
+            static function($item) {
                 return sprintf('"%s"', $item);
             },
             $items
@@ -598,7 +598,7 @@ 
 
         return array_reduce(
             range(1, $selectedLength - 1),
-            static function ($list, $index) use ($selected, $selectedLength) {
+            static function($list, $index) use ($selected, $selectedLength) {
                 return $list .
                     ($selectedLength > 2 ? ', ' : ' ') .
                     ($index === $selectedLength - 1 ? 'or ' : '') .

src/Utils/BreakingChangesFinder.php

Spacing +32 added lines, -32 removed lines

@@ -120,7 +120,7 @@ 
 
         $breakingChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
-            if (! isset($newTypeMap[$typeName])) {
+            if (!isset($newTypeMap[$typeName])) {
                 continue;
             }
             $newType = $newTypeMap[$typeName];
@@ -186,9 +186,9 @@ 
         $breakingChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
-                ! ($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
-                ! ($newType instanceof $oldType)
+            if (!($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
+                !($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
+                !($newType instanceof $oldType)
             ) {
                 continue;
             }
@@ -197,7 +197,7 @@ 
             $newTypeFieldsDef = $newType->getFields();
             foreach ($oldTypeFieldsDef as $fieldName => $fieldDefinition) {
                 // Check if the field is missing on the type in the new schema.
-                if (! isset($newTypeFieldsDef[$fieldName])) {
+                if (!isset($newTypeFieldsDef[$fieldName])) {
                     $breakingChanges[] = [
                         'type'        => self::BREAKING_CHANGE_FIELD_REMOVED,
                         'description' => "${typeName}.${fieldName} was removed.",
@@ -209,7 +209,7 @@ 
                         $oldFieldType,
                         $newFieldType
                     );
-                    if (! $isSafe) {
+                    if (!$isSafe) {
                         $oldFieldTypeString = $oldFieldType instanceof NamedType
                             ? $oldFieldType->name
                             : $oldFieldType;
@@ -279,14 +279,14 @@ 
         $dangerousChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof InputObjectType) || ! ($newType instanceof InputObjectType)) {
+            if (!($oldType instanceof InputObjectType) || !($newType instanceof InputObjectType)) {
                 continue;
             }
 
             $oldTypeFieldsDef = $oldType->getFields();
             $newTypeFieldsDef = $newType->getFields();
             foreach (array_keys($oldTypeFieldsDef) as $fieldName) {
-                if (! isset($newTypeFieldsDef[$fieldName])) {
+                if (!isset($newTypeFieldsDef[$fieldName])) {
                     $breakingChanges[] = [
                         'type'        => self::BREAKING_CHANGE_FIELD_REMOVED,
                         'description' => "${typeName}.${fieldName} was removed.",
@@ -299,7 +299,7 @@ 
                         $oldFieldType,
                         $newFieldType
                     );
-                    if (! $isSafe) {
+                    if (!$isSafe) {
                         $oldFieldTypeString = $oldFieldType instanceof NamedType
                             ? $oldFieldType->name
                             : $oldFieldType;
@@ -370,7 +370,7 @@ 
                         $newType->getWrappedType()
                     )) ||
                 // moving from non-null to nullable of the same underlying type is safe
-                ! ($newType instanceof NonNull) &&
+                !($newType instanceof NonNull) &&
                 self::isChangeSafeForInputObjectFieldOrFieldArg($oldType->getWrappedType(), $newType);
         }
 
@@ -393,7 +393,7 @@ 
         $typesRemovedFromUnion = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof UnionType) || ! ($newType instanceof UnionType)) {
+            if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) {
                 continue;
             }
             $typeNamesInNewUnion = [];
@@ -431,7 +431,7 @@ 
         $valuesRemovedFromEnums = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof EnumType) || ! ($newType instanceof EnumType)) {
+            if (!($oldType instanceof EnumType) || !($newType instanceof EnumType)) {
                 continue;
             }
             $valuesInNewEnum = [];
@@ -473,9 +473,9 @@ 
 
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
-                ! ($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
-                ! ($newType instanceof $oldType)
+            if (!($oldType instanceof ObjectType || $oldType instanceof InterfaceType) ||
+                !($newType instanceof ObjectType || $newType instanceof InterfaceType) ||
+                !($newType instanceof $oldType)
             ) {
                 continue;
             }
@@ -484,7 +484,7 @@ 
             $newTypeFields = $newType->getFields();
 
             foreach ($oldTypeFields as $fieldName => $oldField) {
-                if (! isset($newTypeFields[$fieldName])) {
+                if (!isset($newTypeFields[$fieldName])) {
                     continue;
                 }
 
@@ -492,18 +492,18 @@ 
                     $newArgs   = $newTypeFields[$fieldName]->args;
                     $newArgDef = Utils::find(
                         $newArgs,
-                        static function ($arg) use ($oldArgDef) {
+                        static function($arg) use ($oldArgDef) {
                             return $arg->name === $oldArgDef->name;
                         }
                     );
                     if ($newArgDef !== null) {
-                        $isSafe     = self::isChangeSafeForInputObjectFieldOrFieldArg(
+                        $isSafe = self::isChangeSafeForInputObjectFieldOrFieldArg(
                             $oldArgDef->getType(),
                             $newArgDef->getType()
                         );
                         $oldArgType = $oldArgDef->getType();
                         $oldArgName = $oldArgDef->name;
-                        if (! $isSafe) {
+                        if (!$isSafe) {
                             $newArgType        = $newArgDef->getType();
                             $breakingChanges[] = [
                                 'type'        => self::BREAKING_CHANGE_ARG_CHANGED_KIND,
@@ -531,7 +531,7 @@ 
                         $oldArgs   = $oldTypeFields[$fieldName]->args;
                         $oldArgDef = Utils::find(
                             $oldArgs,
-                            static function ($arg) use ($newArgDef) {
+                            static function($arg) use ($newArgDef) {
                                 return $arg->name === $newArgDef->name;
                             }
                         );
@@ -577,7 +577,7 @@ 
 
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType) || ! ($newType instanceof ObjectType)) {
+            if (!($oldType instanceof ObjectType) || !($newType instanceof ObjectType)) {
                 continue;
             }
 
@@ -586,7 +586,7 @@ 
             foreach ($oldInterfaces as $oldInterface) {
                 $interface = Utils::find(
                     $newInterfaces,
-                    static function (InterfaceType $interface) use ($oldInterface) : bool {
+                    static function(InterfaceType $interface) use ($oldInterface) : bool {
                         return $interface->name === $oldInterface->name;
                     }
                 );
@@ -630,7 +630,7 @@ 
     {
         return Utils::keyMap(
             $schema->getDirectives(),
-            static function ($dir) {
+            static function($dir) {
                 return $dir->name;
             }
         );
@@ -642,7 +642,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -679,7 +679,7 @@ 
     {
         return Utils::keyMap(
             $directive->args ?: [],
-            static function ($arg) {
+            static function($arg) {
                 return $arg->name;
             }
         );
@@ -691,7 +691,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -699,7 +699,7 @@ 
                 $oldSchemaDirectiveMap[$newDirective->name],
                 $newDirective
             ) as $arg) {
-                if (! $arg->getType() instanceof NonNull) {
+                if (!$arg->getType() instanceof NonNull) {
                     continue;
                 }
                 $addedNonNullableArgs[] = [
@@ -743,7 +743,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -809,7 +809,7 @@ 
         $valuesAddedToEnums = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
             $newType = $newTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof EnumType) || ! ($newType instanceof EnumType)) {
+            if (!($oldType instanceof EnumType) || !($newType instanceof EnumType)) {
                 continue;
             }
             $valuesInOldEnum = [];
@@ -844,7 +844,7 @@ 
 
         foreach ($newTypeMap as $typeName => $newType) {
             $oldType = $oldTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof ObjectType) || ! ($newType instanceof ObjectType)) {
+            if (!($oldType instanceof ObjectType) || !($newType instanceof ObjectType)) {
                 continue;
             }
 
@@ -853,7 +853,7 @@ 
             foreach ($newInterfaces as $newInterface) {
                 $interface = Utils::find(
                     $oldInterfaces,
-                    static function (InterfaceType $interface) use ($newInterface) : bool {
+                    static function(InterfaceType $interface) use ($newInterface) : bool {
                         return $interface->name === $newInterface->name;
                     }
                 );
@@ -892,7 +892,7 @@ 
         $typesAddedToUnion = [];
         foreach ($newTypeMap as $typeName => $newType) {
             $oldType = $oldTypeMap[$typeName] ?? null;
-            if (! ($oldType instanceof UnionType) || ! ($newType instanceof UnionType)) {
+            if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) {
                 continue;
             }
 

src/Utils/SchemaPrinter.php

Spacing +24 added lines, -24 removed lines

@@ -51,11 +51,11 @@ 
     {
         return self::printFilteredSchema(
             $schema,
-            static function ($type) {
-                return ! Directive::isSpecifiedDirective($type);
+            static function($type) {
+                return !Directive::isSpecifiedDirective($type);
             },
-            static function ($type) {
-                return ! Type::isBuiltInType($type);
+            static function($type) {
+                return !Type::isBuiltInType($type);
             },
             $options
         );
@@ -68,7 +68,7 @@ 
     {
         $directives = array_filter(
             $schema->getDirectives(),
-            static function ($directive) use ($directiveFilter) {
+            static function($directive) use ($directiveFilter) {
                 return $directiveFilter($directive);
             }
         );
@@ -85,13 +85,13 @@ 
                     array_merge(
                         [self::printSchemaDefinition($schema)],
                         array_map(
-                            static function ($directive) use ($options) {
+                            static function($directive) use ($options) {
                                 return self::printDirective($directive, $options);
                             },
                             $directives
                         ),
                         array_map(
-                            static function ($type) use ($options) {
+                            static function($type) use ($options) {
                                 return self::printType($type, $options);
                             },
                             $types
@@ -169,7 +169,7 @@ 
 
     private static function printDescription($options, $def, $indentation = '', $firstInBlock = true) : string
     {
-        if (! $def->description) {
+        if (!$def->description) {
             return '';
         }
         $lines = self::descriptionLines($def->description, 120 - strlen($indentation));
@@ -177,7 +177,7 @@ 
             return self::printDescriptionWithComments($lines, $indentation, $firstInBlock);
         }
 
-        $description = $indentation && ! $firstInBlock
+        $description = $indentation && !$firstInBlock
             ? "\n" . $indentation . '"""'
             : $indentation . '"""';
 
@@ -195,13 +195,13 @@ 
                 substr($lines[0], 0, 1) === ' ' ||
                 substr($lines[0], 0, 1) === '\t'
             );
-        if (! $hasLeadingSpace) {
+        if (!$hasLeadingSpace) {
             $description .= "\n";
         }
 
         $lineLength = count($lines);
         for ($i = 0; $i < $lineLength; $i++) {
-            if ($i !== 0 || ! $hasLeadingSpace) {
+            if ($i !== 0 || !$hasLeadingSpace) {
                 $description .= $indentation;
             }
             $description .= self::escapeQuote($lines[$i]) . "\n";
@@ -250,7 +250,7 @@ 
 
     private static function printDescriptionWithComments($lines, $indentation, $firstInBlock) : string
     {
-        $description = $indentation && ! $firstInBlock ? "\n" : '';
+        $description = $indentation && !$firstInBlock ? "\n" : '';
         foreach ($lines as $line) {
             if ($line === '') {
                 $description .= $indentation . "#\n";
@@ -269,14 +269,14 @@ 
 
     private static function printArgs($options, $args, $indentation = '') : string
     {
-        if (! $args) {
+        if (!$args) {
             return '';
         }
 
         // If every arg does not have a description, print them on one line.
         if (Utils::every(
             $args,
-            static function ($arg) {
+            static function($arg) {
                 return empty($arg->description);
             }
         )) {
@@ -288,8 +288,8 @@ 
             implode(
                 "\n",
                 array_map(
-                    static function ($arg, $i) use ($indentation, $options) {
-                        return self::printDescription($options, $arg, '  ' . $indentation, ! $i) . '  ' . $indentation .
+                    static function($arg, $i) use ($indentation, $options) {
+                        return self::printDescription($options, $arg, '  ' . $indentation, !$i) . '  ' . $indentation .
                             self::printInputValue($arg);
                     },
                     $args,
@@ -356,11 +356,11 @@ 
     private static function printObject(ObjectType $type, array $options) : string
     {
         $interfaces            = $type->getInterfaces();
-        $implementedInterfaces = ! empty($interfaces) ?
+        $implementedInterfaces = !empty($interfaces) ?
             ' implements ' . implode(
                 ' & ',
                 array_map(
-                    static function ($i) {
+                    static function($i) {
                         return $i->name;
                     },
                     $interfaces
@@ -381,8 +381,8 @@ 
         return implode(
             "\n",
             array_map(
-                static function ($f, $i) use ($options) {
-                    return self::printDescription($options, $f, '  ', ! $i) . '  ' .
+                static function($f, $i) use ($options) {
+                    return self::printDescription($options, $f, '  ', !$i) . '  ' .
                         $f->name . self::printArgs($options, $f->args, '  ') . ': ' .
                         (string) $f->getType() . self::printDeprecated($f);
                 },
@@ -441,8 +441,8 @@ 
         return implode(
             "\n",
             array_map(
-                static function ($value, $i) use ($options) {
-                    return self::printDescription($options, $value, '  ', ! $i) . '  ' .
+                static function($value, $i) use ($options) {
+                    return self::printDescription($options, $value, '  ', !$i) . '  ' .
                         $value->name . self::printDeprecated($value);
                 },
                 $values,
@@ -465,8 +465,8 @@ 
                 implode(
                     "\n",
                     array_map(
-                        static function ($f, $i) use ($options) {
-                            return self::printDescription($options, $f, '  ', ! $i) . '  ' . self::printInputValue($f);
+                        static function($f, $i) use ($options) {
+                            return self::printDescription($options, $f, '  ', !$i) . '  ' . self::printInputValue($f);
                         },
                         $fields,
                         array_keys($fields)

src/Utils/ASTDefinitionBuilder.php

Spacing +19 added lines, -19 removed lines

@@ -84,7 +84,7 @@ 
             'description' => $this->getDescription($directiveNode),
             'locations'   => Utils::map(
                 $directiveNode->locations,
-                static function ($node) {
+                static function($node) {
                     return $node->value;
                 }
             ),
@@ -114,7 +114,7 @@ 
     private function getLeadingCommentBlock($node)
     {
         $loc = $node->loc;
-        if (! $loc || ! $loc->startToken) {
+        if (!$loc || !$loc->startToken) {
             return null;
         }
         $comments = [];
@@ -137,10 +137,10 @@ 
     {
         return Utils::keyValMap(
             $values,
-            static function ($value) {
+            static function($value) {
                 return $value->name->value;
             },
-            function ($value) {
+            function($value) {
                 // Note: While this could make assertions to get the correctly typed
                 // value, that would throw immediately while type system validation
                 // with validateSchema() will produce more actionable results.
@@ -198,7 +198,7 @@ 
      */
     private function internalBuildType($typeName, $typeNode = null)
     {
-        if (! isset($this->cache[$typeName])) {
+        if (!isset($this->cache[$typeName])) {
             if (isset($this->typeDefinitionsMap[$typeName])) {
                 $type = $this->makeSchemaDef($this->typeDefinitionsMap[$typeName]);
                 if ($this->typeConfigDecorator) {
@@ -216,7 +216,7 @@ 
                             $e
                         );
                     }
-                    if (! is_array($config) || isset($config[0])) {
+                    if (!is_array($config) || isset($config[0])) {
                         throw new Error(
                             sprintf(
                                 'Type config decorator passed to %s is expected to return an array, but got %s',
@@ -246,7 +246,7 @@ 
      */
     private function makeSchemaDef($def)
     {
-        if (! $def) {
+        if (!$def) {
             throw new Error('def must be defined.');
         }
         switch ($def->kind) {
@@ -274,10 +274,10 @@ 
         return new ObjectType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
-            'interfaces'  => function () use ($def) {
+            'interfaces'  => function() use ($def) {
                 return $this->makeImplementedInterfaces($def);
             },
             'astNode'     => $def,
@@ -289,10 +289,10 @@ 
         return $def->fields
             ? Utils::keyValMap(
                 $def->fields,
-                static function ($field) {
+                static function($field) {
                     return $field->name->value;
                 },
-                function ($field) {
+                function($field) {
                     return $this->buildField($field);
                 }
             )
@@ -336,7 +336,7 @@ 
             // validation with validateSchema() will produce more actionable results.
             return Utils::map(
                 $def->interfaces,
-                function ($iface) {
+                function($iface) {
                     return $this->buildType($iface);
                 }
             );
@@ -352,7 +352,7 @@ 
         return new InterfaceType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
             'astNode'     => $def,
@@ -367,10 +367,10 @@ 
             'values'      => $def->values
                 ? Utils::keyValMap(
                     $def->values,
-                    static function ($enumValue) {
+                    static function($enumValue) {
                         return $enumValue->name->value;
                     },
-                    function ($enumValue) {
+                    function($enumValue) {
                         return [
                             'description'       => $this->getDescription($enumValue),
                             'deprecationReason' => $this->getDeprecationReason($enumValue),
@@ -394,7 +394,7 @@ 
             'types'       => $def->types
                 ? Utils::map(
                     $def->types,
-                    function ($typeNode) {
+                    function($typeNode) {
                         return $this->buildType($typeNode);
                     }
                 ) :
@@ -409,7 +409,7 @@ 
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
             'astNode'     => $def,
-            'serialize'   => static function ($value) {
+            'serialize'   => static function($value) {
                 return $value;
             },
         ]);
@@ -420,7 +420,7 @@ 
         return new InputObjectType([
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $def->fields
                     ? $this->makeInputValues($def->fields)
                     : [];
@@ -439,7 +439,7 @@ 
      */
     private function makeSchemaDefFromConfig($def, array $config)
     {
-        if (! $def) {
+        if (!$def) {
             throw new Error('def must be defined.');
         }
         switch ($def->kind) {

src/Utils/AST.php

Spacing +16 added lines, -16 removed lines

@@ -88,7 +88,7 @@ 
      */
     public static function fromArray(array $node) : Node
     {
-        if (! isset($node['kind']) || ! isset(NodeKind::$classMap[$node['kind']])) {
+        if (!isset($node['kind']) || !isset(NodeKind::$classMap[$node['kind']])) {
             throw new InvariantViolation('Unexpected node structure: ' . Utils::printSafeJson($node));
         }
 
@@ -176,7 +176,7 @@ 
                 $valuesNodes = [];
                 foreach ($value as $item) {
                     $itemNode = self::astFromValue($item, $itemType);
-                    if (! $itemNode) {
+                    if (!$itemNode) {
                         continue;
                     }
 
@@ -194,7 +194,7 @@ 
         if ($type instanceof InputObjectType) {
             $isArray     = is_array($value);
             $isArrayLike = $isArray || $value instanceof ArrayAccess;
-            if ($value === null || (! $isArrayLike && ! is_object($value))) {
+            if ($value === null || (!$isArrayLike && !is_object($value))) {
                 return null;
             }
             $fields     = $type->getFields();
@@ -219,13 +219,13 @@ 
                     $fieldExists = property_exists($value, $fieldName);
                 }
 
-                if (! $fieldExists) {
+                if (!$fieldExists) {
                     continue;
                 }
 
                 $fieldNode = self::astFromValue($fieldValue, $field->getType());
 
-                if (! $fieldNode) {
+                if (!$fieldNode) {
                     continue;
                 }
 
@@ -349,7 +349,7 @@ 
         if ($valueNode instanceof VariableNode) {
             $variableName = $valueNode->name->value;
 
-            if (! $variables || ! array_key_exists($variableName, $variables)) {
+            if (!$variables || !array_key_exists($variableName, $variables)) {
                 // No valid return value.
                 return $undefined;
             }
@@ -396,7 +396,7 @@ 
         }
 
         if ($type instanceof InputObjectType) {
-            if (! $valueNode instanceof ObjectValueNode) {
+            if (!$valueNode instanceof ObjectValueNode) {
                 // Invalid: intentionally return no value.
                 return $undefined;
             }
@@ -405,7 +405,7 @@ 
             $fields     = $type->getFields();
             $fieldNodes = Utils::keyMap(
                 $valueNode->fields,
-                static function ($field) {
+                static function($field) {
                     return $field->name->value;
                 }
             );
@@ -441,11 +441,11 @@ 
         }
 
         if ($type instanceof EnumType) {
-            if (! $valueNode instanceof EnumValueNode) {
+            if (!$valueNode instanceof EnumValueNode) {
                 return $undefined;
             }
             $enumValue = $type->getValue($valueNode->value);
-            if (! $enumValue) {
+            if (!$enumValue) {
                 return $undefined;
             }
 
@@ -480,7 +480,7 @@ 
     private static function isMissingVariable($valueNode, $variables)
     {
         return $valueNode instanceof VariableNode &&
-            (count($variables) === 0 || ! array_key_exists($valueNode->name->value, $variables));
+            (count($variables) === 0 || !array_key_exists($valueNode->name->value, $variables));
     }
 
     /**
@@ -523,7 +523,7 @@ 
                 return $valueNode->value;
             case $valueNode instanceof ListValueNode:
                 return array_map(
-                    static function ($node) use ($variables) {
+                    static function($node) use ($variables) {
                         return self::valueFromASTUntyped($node, $variables);
                     },
                     iterator_to_array($valueNode->values)
@@ -531,13 +531,13 @@ 
             case $valueNode instanceof ObjectValueNode:
                 return array_combine(
                     array_map(
-                        static function ($field) {
+                        static function($field) {
                             return $field->name->value;
                         },
                         iterator_to_array($valueNode->fields)
                     ),
                     array_map(
-                        static function ($field) use ($variables) {
+                        static function($field) use ($variables) {
                             return self::valueFromASTUntyped($field->value, $variables);
                         },
                         iterator_to_array($valueNode->fields)
@@ -597,11 +597,11 @@ 
     {
         if ($document->definitions) {
             foreach ($document->definitions as $def) {
-                if (! ($def instanceof OperationDefinitionNode)) {
+                if (!($def instanceof OperationDefinitionNode)) {
                     continue;
                 }
 
-                if (! $operationName || (isset($def->name->value) && $def->name->value === $operationName)) {
+                if (!$operationName || (isset($def->name->value) && $def->name->value === $operationName)) {
                     return $def->operation;
                 }
             }