simPod / graphql-php

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

Spacing +25 added lines, -25 removed lines

@@ -50,11 +50,11 @@ 
     {
         return self::printFilteredSchema(
             $schema,
-            function ($type) {
-                return ! Directive::isSpecifiedDirective($type);
+            function($type) {
+                return !Directive::isSpecifiedDirective($type);
             },
-            function ($type) {
-                return ! Type::isBuiltInType($type);
+            function($type) {
+                return !Type::isBuiltInType($type);
             },
             $options
         );
@@ -67,11 +67,11 @@ 
     {
         $directives = array_filter(
             $schema->getDirectives(),
-            function ($directive) use ($directiveFilter) {
+            function($directive) use ($directiveFilter) {
                 return $directiveFilter($directive);
             }
         );
-        $types      = $schema->getTypeMap();
+        $types = $schema->getTypeMap();
         ksort($types);
         $types = array_filter($types, $typeFilter);
 
@@ -80,11 +80,11 @@ 
             array_filter(
                 array_merge(
                     [self::printSchemaDefinition($schema)],
-                    array_map(function ($directive) use ($options) {
+                    array_map(function($directive) use ($options) {
                             return self::printDirective($directive, $options);
                     },
                             $directives),
-                    array_map(function ($type) use ($options) {
+                    array_map(function($type) use ($options) {
                             return self::printType($type, $options);
                     },
                             $types)
@@ -160,7 +160,7 @@ 
 
     private static function printDescription($options, $def, $indentation = '', $firstInBlock = true)
     {
-        if (! $def->description) {
+        if (!$def->description) {
             return '';
         }
         $lines = self::descriptionLines($def->description, 120 - strlen($indentation));
@@ -168,7 +168,7 @@ 
             return self::printDescriptionWithComments($lines, $indentation, $firstInBlock);
         }
 
-        $description = ($indentation && ! $firstInBlock)
+        $description = ($indentation && !$firstInBlock)
             ? "\n" . $indentation . '"""'
             : $indentation . '"""';
 
@@ -186,13 +186,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";
@@ -235,7 +235,7 @@ 
 
     private static function printDescriptionWithComments($lines, $indentation, $firstInBlock)
     {
-        $description = $indentation && ! $firstInBlock ? "\n" : '';
+        $description = $indentation && !$firstInBlock ? "\n" : '';
         foreach ($lines as $line) {
             if ($line === '') {
                 $description .= $indentation . "#\n";
@@ -254,14 +254,14 @@ 
 
     private static function printArgs($options, $args, $indentation = '')
     {
-        if (! $args) {
+        if (!$args) {
             return '';
         }
 
         // If every arg does not have a description, print them on one line.
         if (Utils::every(
             $args,
-            function ($arg) {
+            function($arg) {
                 return empty($arg->description);
             }
         )) {
@@ -273,8 +273,8 @@ 
             implode(
                 "\n",
                 array_map(
-                    function ($arg, $i) use ($indentation, $options) {
-                        return self::printDescription($options, $arg, '  ' . $indentation, ! $i) . '  ' . $indentation .
+                    function($arg, $i) use ($indentation, $options) {
+                        return self::printDescription($options, $arg, '  ' . $indentation, !$i) . '  ' . $indentation .
                             self::printInputValue($arg);
                     },
                     $args,
@@ -341,10 +341,10 @@ 
     private static function printObject(ObjectType $type, array $options)
     {
         $interfaces            = $type->getInterfaces();
-        $implementedInterfaces = ! empty($interfaces) ?
+        $implementedInterfaces = !empty($interfaces) ?
             ' implements ' . implode(
                 ', ',
-                array_map(function ($i) {
+                array_map(function($i) {
                     return $i->name;
                 },
                     $interfaces)
@@ -364,8 +364,8 @@ 
         return implode(
             "\n",
             array_map(
-                function ($f, $i) use ($options) {
-                    return self::printDescription($options, $f, '  ', ! $i) . '  ' .
+                function($f, $i) use ($options) {
+                    return self::printDescription($options, $f, '  ', !$i) . '  ' .
                         $f->name . self::printArgs($options, $f->args, '  ') . ': ' .
                         (string) $f->getType() . self::printDeprecated($f);
                 },
@@ -424,8 +424,8 @@ 
         return implode(
             "\n",
             array_map(
-                function ($value, $i) use ($options) {
-                    return self::printDescription($options, $value, '  ', ! $i) . '  ' .
+                function($value, $i) use ($options) {
+                    return self::printDescription($options, $value, '  ', !$i) . '  ' .
                         $value->name . self::printDeprecated($value);
                 },
                 $values,
@@ -448,8 +448,8 @@ 
                 implode(
                     "\n",
                     array_map(
-                        function ($f, $i) use ($options) {
-                            return self::printDescription($options, $f, '  ', ! $i) . '  ' . self::printInputValue($f);
+                        function($f, $i) use ($options) {
+                            return self::printDescription($options, $f, '  ', !$i) . '  ' . self::printInputValue($f);
                         },
                         $fields,
                         array_keys($fields)

src/Utils/BuildSchema.php

Spacing +12 added lines, -12 removed lines

@@ -111,7 +111,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;
@@ -134,13 +134,13 @@ 
         $defintionBuilder = new ASTDefinitionBuilder(
             $this->nodeMap,
             $this->options,
-            function ($typeName) {
+            function($typeName) {
                 throw new Error('Type "' . $typeName . '" not found in document.');
             },
             $this->typeConfigDecorator
         );
 
-        $directives = array_map(function ($def) use ($defintionBuilder) {
+        $directives = array_map(function($def) use ($defintionBuilder) {
             return $defintionBuilder->buildDirective($def);
         },
             $directiveDefs);
@@ -148,31 +148,31 @@ 
         // If specified directives were not explicitly declared, add them.
         $skip = array_reduce(
             $directives,
-            function ($hasSkip, $directive) {
+            function($hasSkip, $directive) {
                 return $hasSkip || $directive->name === 'skip';
             }
         );
-        if (! $skip) {
+        if (!$skip) {
             $directives[] = Directive::skipDirective();
         }
 
         $include = array_reduce(
             $directives,
-            function ($hasInclude, $directive) {
+            function($hasInclude, $directive) {
                 return $hasInclude || $directive->name === 'include';
             }
         );
-        if (! $include) {
+        if (!$include) {
             $directives[] = Directive::includeDirective();
         }
 
         $deprecated = array_reduce(
             $directives,
-            function ($hasDeprecated, $directive) {
+            function($hasDeprecated, $directive) {
                 return $hasDeprecated || $directive->name === 'deprecated';
             }
         );
-        if (! $deprecated) {
+        if (!$deprecated) {
             $directives[] = Directive::deprecatedDirective();
         }
 
@@ -190,12 +190,12 @@ 
             'subscription' => isset($operationTypes['subscription'])
                 ? $defintionBuilder->buildType($operationTypes['subscription'])
                 : null,
-            'typeLoader'   => function ($name) use ($defintionBuilder) {
+            'typeLoader'   => function($name) use ($defintionBuilder) {
                 return $defintionBuilder->buildType($name);
             },
             'directives'   => $directives,
             'astNode'      => $schemaDef,
-            'types'        => function () use ($defintionBuilder) {
+            'types'        => function() use ($defintionBuilder) {
                 $types = [];
                 foreach ($this->nodeMap as $name => $def) {
                     $types[] = $defintionBuilder->buildType($def->name->value);
@@ -225,7 +225,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/Value.php

Spacing +4 added lines, -4 removed lines

@@ -94,12 +94,12 @@ 
 
             $suggestions = Utils::suggestionList(
                 Utils::printSafe($value),
-                array_map(function ($enumValue) {
+                array_map(function($enumValue) {
                     return $enumValue->name;
                 },
                     $type->getValues())
             );
-            $didYouMean  = $suggestions
+            $didYouMean = $suggestions
                 ? 'did you mean ' . Utils::orList($suggestions) . '?'
                 : null;
 
@@ -141,7 +141,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),
@@ -258,7 +258,7 @@ 
         $pathStr     = '';
         $currentPath = $path;
         while ($currentPath) {
-            $pathStr     =
+            $pathStr =
                 (is_string($currentPath['key'])
                     ? '.' . $currentPath['key']
                     : '[' . $currentPath['key'] . ']') . $pathStr;

src/Utils/Utils.php

Spacing +15 added lines, -15 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);
             });
 
@@ -540,7 +540,7 @@ 
      */
     public static function quotedOrList(array $items)
     {
-        $items = array_map(function ($item) {
+        $items = array_map(function($item) {
             return sprintf('"%s"', $item);
         },
             $items);
@@ -554,7 +554,7 @@ 
      */
     public static function orList(array $items)
     {
-        if (! $items) {
+        if (!$items) {
             throw new \LogicException('items must not need to be empty.');
         }
         $selected       = array_slice($items, 0, 5);
@@ -567,7 +567,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 ' : '') .

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