simPod / graphql-php

src/Language/Visitor.php

Spacing +14 added lines, -14 removed lines

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

src/Validator/Rules/KnownArgumentNames.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
     public function getVisitor(ValidationContext $context)
     {
         return [
-            NodeKind::ARGUMENT => function (ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
+            NodeKind::ARGUMENT => function(ArgumentNode $node, $key, $parent, $path, $ancestors) use ($context) {
                 /** @var NodeList|Node[] $ancestors */
                 $argDef = $context->getArgument();
                 if ($argDef !== null) {
@@ -46,7 +46,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        function ($arg) {
+                                        function($arg) {
                                             return $arg->name;
                                         },
                                         $fieldDef->args
@@ -66,7 +66,7 @@ 
                                 Utils::suggestionList(
                                     $node->name->value,
                                     array_map(
-                                        function ($arg) {
+                                        function($arg) {
                                             return $arg->name;
                                         },
                                         $directive->args
@@ -87,7 +87,7 @@ 
     public static function unknownArgMessage($argName, $fieldName, $typeName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on field "%s" of type "%s".', $argName, $fieldName, $typeName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 
@@ -100,7 +100,7 @@ 
     public static function unknownDirectiveArgMessage($argName, $directiveName, array $suggestedArgs)
     {
         $message = sprintf('Unknown argument "%s" on directive "@%s".', $argName, $directiveName);
-        if (! empty($suggestedArgs)) {
+        if (!empty($suggestedArgs)) {
             $message .= sprintf(' Did you mean %s?', Utils::quotedOrList($suggestedArgs));
         }
 

src/Validator/Rules/LoneAnonymousOperation.php

Spacing +3 added lines, -3 removed lines

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

src/Utils/AST.php

Spacing +17 added lines, -17 removed lines

@@ -84,7 +84,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));
         }
 
@@ -169,7 +169,7 @@ 
                 $valuesNodes = [];
                 foreach ($value as $item) {
                     $itemNode = self::astFromValue($item, $itemType);
-                    if (! $itemNode) {
+                    if (!$itemNode) {
                         continue;
                     }
 
@@ -187,7 +187,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();
@@ -212,13 +212,13 @@ 
                     $fieldExists = property_exists($value, $fieldName);
                 }
 
-                if (! $fieldExists) {
+                if (!$fieldExists) {
                     continue;
                 }
 
                 $fieldNode = self::astFromValue($fieldValue, $field->getType());
 
-                if (! $fieldNode) {
+                if (!$fieldNode) {
                     continue;
                 }
 
@@ -341,7 +341,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;
             }
@@ -388,7 +388,7 @@ 
         }
 
         if ($type instanceof InputObjectType) {
-            if (! $valueNode instanceof ObjectValueNode) {
+            if (!$valueNode instanceof ObjectValueNode) {
                 // Invalid: intentionally return no value.
                 return $undefined;
             }
@@ -397,7 +397,7 @@ 
             $fields     = $type->getFields();
             $fieldNodes = Utils::keyMap(
                 $valueNode->fields,
-                function ($field) {
+                function($field) {
                     return $field->name->value;
                 }
             );
@@ -406,7 +406,7 @@ 
                 $fieldName = $field->name;
                 $fieldNode = $fieldNodes[$fieldName] ?? null;
 
-                if (! $fieldNode || self::isMissingVariable($fieldNode->value, $variables)) {
+                if (!$fieldNode || self::isMissingVariable($fieldNode->value, $variables)) {
                     if ($field->defaultValueExists()) {
                         $coercedObj[$fieldName] = $field->defaultValue;
                     } elseif ($field->getType() instanceof NonNull) {
@@ -429,11 +429,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;
             }
 
@@ -466,7 +466,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));
     }
 
     /**
@@ -506,7 +506,7 @@ 
                 return $valueNode->value;
             case $valueNode instanceof ListValueNode:
                 return array_map(
-                    function ($node) use ($variables) {
+                    function($node) use ($variables) {
                         return self::valueFromASTUntyped($node, $variables);
                     },
                     iterator_to_array($valueNode->values)
@@ -514,13 +514,13 @@ 
             case $valueNode instanceof ObjectValueNode:
                 return array_combine(
                     array_map(
-                        function ($field) {
+                        function($field) {
                             return $field->name->value;
                         },
                         iterator_to_array($valueNode->fields)
                     ),
                     array_map(
-                        function ($field) use ($variables) {
+                        function($field) use ($variables) {
                             return self::valueFromASTUntyped($field->value, $variables);
                         },
                         iterator_to_array($valueNode->fields)
@@ -575,11 +575,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;
                 }
             }

src/Utils/ASTDefinitionBuilder.php

Spacing +19 added lines, -19 removed lines

@@ -82,7 +82,7 @@ 
             'description' => $this->getDescription($directiveNode),
             'locations'   => Utils::map(
                 $directiveNode->locations,
-                function ($node) {
+                function($node) {
                     return $node->value;
                 }
             ),
@@ -112,7 +112,7 @@ 
     private function getLeadingCommentBlock($node)
     {
         $loc = $node->loc;
-        if (! $loc || ! $loc->startToken) {
+        if (!$loc || !$loc->startToken) {
             return null;
         }
         $comments = [];
@@ -135,10 +135,10 @@ 
     {
         return Utils::keyValMap(
             $values,
-            function ($value) {
+            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.
@@ -191,7 +191,7 @@ 
      */
     private function internalBuildType($typeName, $typeNode = null)
     {
-        if (! isset($this->cache[$typeName])) {
+        if (!isset($this->cache[$typeName])) {
             if (isset($this->typeDefintionsMap[$typeName])) {
                 $type = $this->makeSchemaDef($this->typeDefintionsMap[$typeName]);
                 if ($this->typeConfigDecorator) {
@@ -209,7 +209,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',
@@ -237,7 +237,7 @@ 
      */
     private function makeSchemaDef($def)
     {
-        if (! $def) {
+        if (!$def) {
             throw new Error('def must be defined.');
         }
         switch ($def->kind) {
@@ -265,10 +265,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,
@@ -280,10 +280,10 @@ 
         return $def->fields
             ? Utils::keyValMap(
                 $def->fields,
-                function ($field) {
+                function($field) {
                     return $field->name->value;
                 },
-                function ($field) {
+                function($field) {
                     return $this->buildField($field);
                 }
             )
@@ -326,7 +326,7 @@ 
             // validation with validateSchema() will produce more actionable results.
             return Utils::map(
                 $def->interfaces,
-                function ($iface) {
+                function($iface) {
                     return $this->buildType($iface);
                 }
             );
@@ -342,7 +342,7 @@ 
         return new InterfaceType([
             'name'        => $typeName,
             'description' => $this->getDescription($def),
-            'fields'      => function () use ($def) {
+            'fields'      => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
             'astNode'     => $def,
@@ -357,10 +357,10 @@ 
             'values'      => $def->values
                 ? Utils::keyValMap(
                     $def->values,
-                    function ($enumValue) {
+                    function($enumValue) {
                         return $enumValue->name->value;
                     },
-                    function ($enumValue) {
+                    function($enumValue) {
                         return [
                             'description'       => $this->getDescription($enumValue),
                             'deprecationReason' => $this->getDeprecationReason($enumValue),
@@ -384,7 +384,7 @@ 
             'types'       => $def->types
                 ? Utils::map(
                     $def->types,
-                    function ($typeNode) {
+                    function($typeNode) {
                         return $this->buildType($typeNode);
                     }
                 ) :
@@ -399,7 +399,7 @@ 
             'name'        => $def->name->value,
             'description' => $this->getDescription($def),
             'astNode'     => $def,
-            'serialize'   => function ($value) {
+            'serialize'   => function($value) {
                 return $value;
             },
         ]);
@@ -410,7 +410,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)
                     : [];
@@ -427,7 +427,7 @@ 
      */
     private function makeSchemaDefFromConfig($def, array $config)
     {
-        if (! $def) {
+        if (!$def) {
             throw new Error('def must be defined.');
         }
         switch ($def->kind) {

src/Utils/BreakingChangesFinder.php

Spacing +32 added lines, -32 removed lines

@@ -119,7 +119,7 @@ 
 
         $breakingChanges = [];
         foreach ($oldTypeMap as $typeName => $oldType) {
-            if (! isset($newTypeMap[$typeName])) {
+            if (!isset($newTypeMap[$typeName])) {
                 continue;
             }
             $newType = $newTypeMap[$typeName];
@@ -185,9 +185,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;
             }
@@ -196,7 +196,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.",
@@ -208,7 +208,7 @@ 
                         $oldFieldType,
                         $newFieldType
                     );
-                    if (! $isSafe) {
+                    if (!$isSafe) {
                         $oldFieldTypeString = $oldFieldType instanceof NamedType
                             ? $oldFieldType->name
                             : $oldFieldType;
@@ -278,14 +278,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.",
@@ -298,7 +298,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,
-                        function ($arg) use ($oldArgDef) {
+                        function($arg) use ($oldArgDef) {
                             return $arg->name === $oldArgDef->name;
                         }
                     );
                     if ($newArgDef) {
-                        $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,
-                            function ($arg) use ($newArgDef) {
+                            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) {
                 if (Utils::find(
                     $newInterfaces,
-                    function (InterfaceType $interface) use ($oldInterface) {
+                    function(InterfaceType $interface) use ($oldInterface) {
                         return $interface->name === $oldInterface->name;
                     }
                 )) {
@@ -629,7 +629,7 @@ 
     {
         return Utils::keyMap(
             $schema->getDirectives(),
-            function ($dir) {
+            function($dir) {
                 return $dir->name;
             }
         );
@@ -641,7 +641,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -678,7 +678,7 @@ 
     {
         return Utils::keyMap(
             $directive->args ?: [],
-            function ($arg) {
+            function($arg) {
                 return $arg->name;
             }
         );
@@ -690,7 +690,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -698,7 +698,7 @@ 
                 $oldSchemaDirectiveMap[$newDirective->name],
                 $newDirective
             ) as $arg) {
-                if (! $arg->getType() instanceof NonNull) {
+                if (!$arg->getType() instanceof NonNull) {
                     continue;
                 }
                 $addedNonNullableArgs[] = [
@@ -742,7 +742,7 @@ 
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
         foreach ($newSchema->getDirectives() as $newDirective) {
-            if (! isset($oldSchemaDirectiveMap[$newDirective->name])) {
+            if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
@@ -808,7 +808,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) {
                 if (Utils::find(
                     $oldInterfaces,
-                    function (InterfaceType $interface) use ($newInterface) {
+                    function(InterfaceType $interface) use ($newInterface) {
                         return $interface->name === $newInterface->name;
                     }
                 )) {
@@ -890,7 +890,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/Error/Error.php

Spacing +11 added lines, -11 removed lines

@@ -102,7 +102,7 @@ 
         // Compute list of blame nodes.
         if ($nodes instanceof \Traversable) {
             $nodes = iterator_to_array($nodes);
-        } elseif ($nodes && ! is_array($nodes)) {
+        } elseif ($nodes && !is_array($nodes)) {
             $nodes = [$nodes];
         }
 
@@ -207,7 +207,7 @@ 
     public function getSource()
     {
         if ($this->source === null) {
-            if (! empty($this->nodes[0]) && ! empty($this->nodes[0]->loc)) {
+            if (!empty($this->nodes[0]) && !empty($this->nodes[0]->loc)) {
                 $this->source = $this->nodes[0]->loc->source;
             }
         }
@@ -220,9 +220,9 @@ 
      */
     public function getPositions()
     {
-        if ($this->positions === null && ! empty($this->nodes)) {
+        if ($this->positions === null && !empty($this->nodes)) {
             $positions = array_map(
-                function ($node) {
+                function($node) {
                     return isset($node->loc) ? $node->loc->start : null;
                 },
                 $this->nodes
@@ -230,7 +230,7 @@ 
 
             $this->positions = array_filter(
                 $positions,
-                function ($p) {
+                function($p) {
                     return $p !== null;
                 }
             );
@@ -262,7 +262,7 @@ 
 
             if ($positions && $source) {
                 $this->locations = array_map(
-                    function ($pos) use ($source) {
+                    function($pos) use ($source) {
                         return $source->getLocation($pos);
                     },
                     $positions
@@ -270,7 +270,7 @@ 
             } elseif ($nodes) {
                 $this->locations = array_filter(
                     array_map(
-                        function ($node) {
+                        function($node) {
                             if ($node->loc && $node->loc->source) {
                                 return $node->loc->source->getLocation($node->loc->start);
                             }
@@ -328,18 +328,18 @@ 
 
         $locations = Utils::map(
             $this->getLocations(),
-            function (SourceLocation $loc) {
+            function(SourceLocation $loc) {
                 return $loc->toSerializableArray();
             }
         );
 
-        if (! empty($locations)) {
+        if (!empty($locations)) {
             $arr['locations'] = $locations;
         }
-        if (! empty($this->path)) {
+        if (!empty($this->path)) {
             $arr['path'] = $this->path;
         }
-        if (! empty($this->extensions)) {
+        if (!empty($this->extensions)) {
             $arr['extensions'] = $this->extensions;
         }
 

src/Utils/Value.php

Spacing +3 added lines, -3 removed lines

@@ -95,7 +95,7 @@ 
             $suggestions = Utils::suggestionList(
                 Utils::printSafe($value),
                 array_map(
-                    function ($enumValue) {
+                    function($enumValue) {
                         return $enumValue->name;
                     },
                     $type->getValues()
@@ -144,7 +144,7 @@ 
         }
 
         if ($type instanceof InputObjectType) {
-            if (! is_object($value) && ! is_array($value) && ! $value instanceof \Traversable) {
+            if (!is_object($value) && !is_array($value) && !$value instanceof \Traversable) {
                 return self::ofErrors([
                     self::coercionError(
                         sprintf('Expected type %s to be an object', $type->name),
@@ -265,7 +265,7 @@ 
         $pathStr     = '';
         $currentPath = $path;
         while ($currentPath) {
-            $pathStr     =
+            $pathStr =
                 (is_string($currentPath['key'])
                     ? '.' . $currentPath['key']
                     : '[' . $currentPath['key'] . ']') . $pathStr;

src/Utils/Utils.php

Spacing +14 added lines, -14 removed lines

@@ -79,13 +79,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),
@@ -133,10 +133,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;
             }
 
@@ -202,7 +202,7 @@ 
         $map = [];
         foreach ($traversable as $key => $value) {
             $newKey = $keyFn($value, $key);
-            if (! is_scalar($newKey)) {
+            if (!is_scalar($newKey)) {
                 continue;
             }
 
@@ -278,7 +278,7 @@ 
     public static function every($traversable, callable $predicate)
     {
         foreach ($traversable as $key => $value) {
-            if (! $predicate($value, $key)) {
+            if (!$predicate($value, $key)) {
                 return false;
             }
         }
@@ -292,7 +292,7 @@ 
      */
     public static function invariant($test, $message = '')
     {
-        if (! $test) {
+        if (!$test) {
             if (func_num_args() > 2) {
                 $args = func_get_args();
                 array_shift($args);
@@ -424,10 +424,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') {
@@ -501,7 +501,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
@@ -520,9 +520,9 @@ 
      */
     public static function withErrorHandling(callable $fn, array &$errors)
     {
-        return function () use ($fn, &$errors) {
+        return function() use ($fn, &$errors) {
             // Catch custom errors (to report them in query results)
-            set_error_handler(function ($severity, $message, $file, $line) use (&$errors) {
+            set_error_handler(function($severity, $message, $file, $line) use (&$errors) {
                 $errors[] = new \ErrorException($message, 0, $severity, $file, $line);
             });
 
@@ -541,7 +541,7 @@ 
     public static function quotedOrList(array $items)
     {
         $items = array_map(
-            function ($item) {
+            function($item) {
                 return sprintf('"%s"', $item);
             },
             $items
@@ -569,7 +569,7 @@ 
 
         return array_reduce(
             range(1, $selectedLength - 1),
-            function ($list, $index) use ($selected, $selectedLength) {
+            function($list, $index) use ($selected, $selectedLength) {
                 return $list .
                     ($selectedLength > 2 ? ', ' : ' ') .
                     ($index === $selectedLength - 1 ? 'or ' : '') .