simPod / graphql-php

src/Utils/BlockString.php

Indentation +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     private static function leadingWhitespace($str) {
         $i = 0;
         while ($i < mb_strlen($str) && ($str[$i] === ' ' || $str[$i] === '\t')) {
-          $i++;
+            $i++;
         }
 
         return $i;

src/Utils/MixedStore.php

Spacing +3 added lines, -3 removed lines

@@ -157,7 +157,7 @@ 
             return $this->standardStore[$offset];
         }
         if (is_float($offset)) {
-            return $this->floatStore[(string)$offset];
+            return $this->floatStore[(string) $offset];
         }
         if (is_object($offset)) {
             return $this->objectStore->offsetGet($offset);
@@ -202,7 +202,7 @@ 
         } else if (is_int($offset) || is_string($offset)) {
             $this->standardStore[$offset] = $value;
         } else if (is_float($offset)) {
-            $this->floatStore[(string)$offset] = $value;
+            $this->floatStore[(string) $offset] = $value;
         } else if (is_object($offset)) {
             $this->objectStore[$offset] = $value;
         } else if (is_array($offset)) {
@@ -236,7 +236,7 @@ 
         } else if (is_int($offset) || is_string($offset)) {
             unset($this->standardStore[$offset]);
         } else if (is_float($offset)) {
-            unset($this->floatStore[(string)$offset]);
+            unset($this->floatStore[(string) $offset]);
         } else if (is_object($offset)) {
             $this->objectStore->offsetUnset($offset);
         } else if (is_array($offset)) {

src/Utils/SchemaPrinter.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
             function($type) {
                 return !Directive::isSpecifiedDirective($type);
             },
-            function ($type) {
+            function($type) {
                 return !Type::isBuiltInType($type);
             },
             $options
@@ -312,7 +312,7 @@ 
     private static function descriptionLines($description, $maxLen) {
         $lines = [];
         $rawLines = explode("\n", $description);
-        foreach($rawLines as $line) {
+        foreach ($rawLines as $line) {
             if ($line === '') {
                 $lines[] = $line;
             } else {

src/Utils/TypeInfo.php

Spacing +3 added lines, -3 removed lines

@@ -95,7 +95,7 @@ 
         }
         if (!$type instanceof Type) {
             Warning::warnOnce(
-                'One of the schema types is not a valid type definition instance. '.
+                'One of the schema types is not a valid type definition instance. ' .
                 'Try running $schema->assertValid() to find out the cause of this warning.',
                 Warning::WARNING_NOT_A_TYPE
             );
@@ -105,7 +105,7 @@ 
         if (!empty($typeMap[$type->name])) {
             Utils::invariant(
                 $typeMap[$type->name] === $type,
-                "Schema must contain unique named types but contains multiple types named \"$type\" ".
+                "Schema must contain unique named types but contains multiple types named \"$type\" " .
                 "(see http://webonyx.github.io/graphql-php/type-system/#type-registry)."
             );
             return $typeMap;
@@ -379,7 +379,7 @@ 
                 $fieldOrDirective = $this->getDirective() ?: $this->getFieldDef();
                 $argDef = $argType = null;
                 if ($fieldOrDirective) {
-                    $argDef = Utils::find($fieldOrDirective->args, function($arg) use ($node) {return $arg->name === $node->name->value;});
+                    $argDef = Utils::find($fieldOrDirective->args, function($arg) use ($node) {return $arg->name === $node->name->value; });
                     if ($argDef) {
                         $argType = $argDef->getType();
                     }

src/Utils/ASTDefinitionBuilder.php

Spacing +13 added lines, -13 removed lines

@@ -181,7 +181,7 @@ 
         return new Directive([
             'name' => $directiveNode->name->value,
             'description' => $this->getDescription($directiveNode),
-            'locations' => Utils::map($directiveNode->locations, function ($node) {
+            'locations' => Utils::map($directiveNode->locations, function($node) {
                 return $node->value;
             }),
             'args' => $directiveNode->arguments ? FieldArgument::createMap($this->makeInputValues($directiveNode->arguments)) : null,
@@ -255,10 +255,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
@@ -270,10 +270,10 @@ 
         return $def->fields
             ? Utils::keyValMap(
                 $def->fields,
-                function ($field) {
+                function($field) {
                     return $field->name->value;
                 },
-                function ($field) {
+                function($field) {
                     return $this->buildField($field);
                 }
             )
@@ -286,7 +286,7 @@ 
             // Note: While this could make early assertions to get the correctly
             // typed values, that would throw immediately while type system
             // validation with validateSchema() will produce more actionable results.
-            return Utils::map($def->interfaces, function ($iface) {
+            return Utils::map($def->interfaces, function($iface) {
                 return $this->buildType($iface);
             });
         }
@@ -297,10 +297,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.
@@ -325,7 +325,7 @@ 
         return new InterfaceType([
             'name' => $typeName,
             'description' => $this->getDescription($def),
-            'fields' => function () use ($def) {
+            'fields' => function() use ($def) {
                 return $this->makeFieldDefMap($def);
             },
             'astNode' => $def
@@ -340,10 +340,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),
@@ -365,7 +365,7 @@ 
             // values below, that would throw immediately while type system
             // validation with validateSchema() will produce more actionable results.
             'types' => $def->types
-                ? Utils::map($def->types, function ($typeNode) {
+                ? Utils::map($def->types, function($typeNode) {
                     return $this->buildType($typeNode);
                 }):
                 [],
@@ -390,7 +390,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)
                     : [];

src/Utils/AST.php

Indentation +1 added lines, -1 removed lines

@@ -470,7 +470,7 @@
                     function($field) use ($variables) { return self::valueFromASTUntyped($field->value, $variables); },
                     iterator_to_array($valueNode->fields)
                 )
-              );
+                );
             case $valueNode instanceof VariableNode:
                 $variableName = $valueNode->name->value;
                 return ($variables && isset($variables[$variableName]))

Spacing +3 added lines, -3 removed lines

@@ -81,7 +81,7 @@ 
 
         foreach ($node as $key => $value) {
             if ('loc' === $key || 'kind' === $key) {
-                continue ;
+                continue;
             }
             if (is_array($value)) {
                 if (isset($value[0]) || empty($value)) {
@@ -364,7 +364,7 @@ 
 
             $coercedObj = [];
             $fields = $type->getFields();
-            $fieldNodes = Utils::keyMap($valueNode->fields, function($field) {return $field->name->value;});
+            $fieldNodes = Utils::keyMap($valueNode->fields, function($field) {return $field->name->value; });
             foreach ($fields as $field) {
                 /** @var ValueNode $fieldNode */
                 $fieldName = $field->name;
@@ -377,7 +377,7 @@ 
                         // Invalid: intentionally return no value.
                         return $undefined;
                     }
-                    continue ;
+                    continue;
                 }
 
                 $fieldValue = self::valueFromAST($fieldNode ? $fieldNode->value : null, $field->getType(), $variables);

src/Utils/FindBreakingChanges.php

Spacing +16 added lines, -16 removed lines

@@ -181,7 +181,7 @@ 
                     $newArgs = $newTypeFields[$fieldName]->args;
                     $newArgDef = Utils::find(
                         $newArgs,
-                        function ($arg) use ($oldArgDef) {
+                        function($arg) use ($oldArgDef) {
                             return $arg->name === $oldArgDef->name;
                         }
                     );
@@ -215,7 +215,7 @@ 
                         $oldArgs = $oldTypeFields[$fieldName]->args;
                         $oldArgDef = Utils::find(
                             $oldArgs,
-                            function ($arg) use ($newArgDef) {
+                            function($arg) use ($newArgDef) {
                                 return $arg->name === $newArgDef->name;
                             }
                         );
@@ -622,7 +622,7 @@ 
             $oldInterfaces = $oldType->getInterfaces();
             $newInterfaces = $newType->getInterfaces();
             foreach ($oldInterfaces as $oldInterface) {
-                if (!Utils::find($newInterfaces, function (InterfaceType $interface) use ($oldInterface) {
+                if (!Utils::find($newInterfaces, function(InterfaceType $interface) use ($oldInterface) {
                     return $interface->name === $oldInterface->name;
                 })) {
                     $breakingChanges[] = [
@@ -658,7 +658,7 @@ 
             $oldInterfaces = $oldType->getInterfaces();
             $newInterfaces = $newType->getInterfaces();
             foreach ($newInterfaces as $newInterface) {
-                if (!Utils::find($oldInterfaces, function (InterfaceType $interface) use ($newInterface) {
+                if (!Utils::find($oldInterfaces, function(InterfaceType $interface) use ($newInterface) {
                     return $interface->name === $newInterface->name;
                 })) {
                     $interfacesAddedToObjectTypes[] = [
@@ -676,7 +676,7 @@ 
         $removedDirectives = [];
 
         $newSchemaDirectiveMap = self::getDirectiveMapForSchema($newSchema);
-        foreach($oldSchema->getDirectives() as $directive) {
+        foreach ($oldSchema->getDirectives() as $directive) {
             if (!isset($newSchemaDirectiveMap[$directive->name])) {
                 $removedDirectives[] = [
                     'type' => self::BREAKING_CHANGE_DIRECTIVE_REMOVED,
@@ -692,7 +692,7 @@ 
     {
         $removedArgs = [];
         $newArgMap = self::getArgumentMapForDirective($newDirective);
-        foreach((array) $oldDirective->args as $arg) {
+        foreach ((array) $oldDirective->args as $arg) {
             if (!isset($newArgMap[$arg->name])) {
                 $removedArgs[] = $arg;
             }
@@ -706,12 +706,12 @@ 
         $removedDirectiveArgs = [];
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
-        foreach($newSchema->getDirectives() as $newDirective) {
+        foreach ($newSchema->getDirectives() as $newDirective) {
             if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
-            foreach(self::findRemovedArgsForDirectives($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $arg) {
+            foreach (self::findRemovedArgsForDirectives($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $arg) {
                 $removedDirectiveArgs[] = [
                     'type' => self::BREAKING_CHANGE_DIRECTIVE_ARG_REMOVED,
                     'description' => "{$arg->name} was removed from {$newDirective->name}",
@@ -726,7 +726,7 @@ 
     {
         $addedArgs = [];
         $oldArgMap = self::getArgumentMapForDirective($oldDirective);
-        foreach((array) $newDirective->args as $arg) {
+        foreach ((array) $newDirective->args as $arg) {
             if (!isset($oldArgMap[$arg->name])) {
                 $addedArgs[] = $arg;
             }
@@ -740,12 +740,12 @@ 
         $addedNonNullableArgs = [];
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
-        foreach($newSchema->getDirectives() as $newDirective) {
+        foreach ($newSchema->getDirectives() as $newDirective) {
             if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
-            foreach(self::findAddedArgsForDirective($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $arg) {
+            foreach (self::findAddedArgsForDirective($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $arg) {
                 if (!$arg->getType() instanceof NonNull) {
                     continue;
                 }
@@ -763,7 +763,7 @@ 
     {
         $removedLocations = [];
         $newLocationSet = array_flip($newDirective->locations);
-        foreach($oldDirective->locations as $oldLocation) {
+        foreach ($oldDirective->locations as $oldLocation) {
             if (!array_key_exists($oldLocation, $newLocationSet)) {
                 $removedLocations[] = $oldLocation;
             }
@@ -777,12 +777,12 @@ 
         $removedLocations = [];
         $oldSchemaDirectiveMap = self::getDirectiveMapForSchema($oldSchema);
 
-        foreach($newSchema->getDirectives() as $newDirective) {
+        foreach ($newSchema->getDirectives() as $newDirective) {
             if (!isset($oldSchemaDirectiveMap[$newDirective->name])) {
                 continue;
             }
 
-            foreach(self::findRemovedLocationsForDirective($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $location) {
+            foreach (self::findRemovedLocationsForDirective($oldSchemaDirectiveMap[$newDirective->name], $newDirective) as $location) {
                 $removedLocations[] = [
                     'type' => self::BREAKING_CHANGE_DIRECTIVE_LOCATION_REMOVED,
                     'description' => "{$location} was removed from {$newDirective->name}",
@@ -795,11 +795,11 @@ 
 
     private static function getDirectiveMapForSchema(Schema $schema)
     {
-        return Utils::keyMap($schema->getDirectives(), function ($dir) { return $dir->name; });
+        return Utils::keyMap($schema->getDirectives(), function($dir) { return $dir->name; });
     }
 
     private static function getArgumentMapForDirective(Directive $directive)
     {
-        return Utils::keyMap($directive->args ?: [], function ($arg) { return $arg->name; });
+        return Utils::keyMap($directive->args ?: [], function($arg) { return $arg->name; });
     }
 }

src/Utils/BuildSchema.php

Spacing +6 added lines, -6 removed lines

@@ -102,7 +102,7 @@ 
         $defintionBuilder = new ASTDefinitionBuilder(
             $this->nodeMap,
             $this->options,
-            function($typeName) { throw new Error('Type "'. $typeName . '" not found in document.'); },
+            function($typeName) { throw new Error('Type "' . $typeName . '" not found in document.'); },
             $this->typeConfigDecorator
         );
 
@@ -111,21 +111,21 @@ 
         }, $directiveDefs);
 
         // If specified directives were not explicitly declared, add them.
-        $skip = array_reduce($directives, function ($hasSkip, $directive) {
+        $skip = array_reduce($directives, function($hasSkip, $directive) {
             return $hasSkip || $directive->name == 'skip';
         });
         if (!$skip) {
             $directives[] = Directive::skipDirective();
         }
 
-        $include = array_reduce($directives, function ($hasInclude, $directive) {
+        $include = array_reduce($directives, function($hasInclude, $directive) {
             return $hasInclude || $directive->name == 'include';
         });
         if (!$include) {
             $directives[] = Directive::includeDirective();
         }
 
-        $deprecated = array_reduce($directives, function ($hasDeprecated, $directive) {
+        $deprecated = array_reduce($directives, function($hasDeprecated, $directive) {
             return $hasDeprecated || $directive->name == 'deprecated';
         });
         if (!$deprecated) {
@@ -146,12 +146,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);

src/Utils/Utils.php

Spacing +3 added lines, -3 removed lines

@@ -460,7 +460,7 @@ 
     {
         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);
             });
 
@@ -498,8 +498,8 @@ 
 
         return array_reduce(
             range(1, $selectedLength - 1),
-            function ($list, $index) use ($selected, $selectedLength) {
-                return $list.
+            function($list, $index) use ($selected, $selectedLength) {
+                return $list .
                     ($selectedLength > 2 ? ', ' : ' ') .
                     ($index === $selectedLength - 1 ? 'or ' : '') .
                     $selected[$index];