overblog / GraphQLBundle

Config/TypeWithOutputFieldsDefinition.php

Doc Comments +6 added lines

@@ -51,6 +51,9 @@ 
         self::$fieldBuilderClassMap[$name] = $fieldBuilderClass;
     }
 
+    /**
+     * @param string $type
+     */
     protected static function checkBuilderClass($builderClass, $type)
     {
         $interface = 'Overblog\\GraphQLBundle\\Definition\\Builder\\MappingInterface';
@@ -113,6 +116,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     protected function outputFieldsSelection($name, $withAccess = false)
     {
         $builder = new TreeBuilder();

Spacing +3 added lines, -3 removed lines

@@ -127,10 +127,10 @@ 
         $prototype
             // build args if argsBuilder exists
             ->beforeNormalization()
-                ->ifTrue(function ($field) {
+                ->ifTrue(function($field) {
                     return isset($field['argsBuilder']);
                 })
-                ->then(function ($field) {
+                ->then(function($field) {
                     $argsBuilderName = null;
 
                     if (is_string($field['argsBuilder'])) {
@@ -156,7 +156,7 @@ 
             ->end()
             // build field if builder exists
             ->beforeNormalization()
-                ->always(function ($field) {
+                ->always(function($field) {
                     $fieldBuilderName = null;
 
                     if (isset($field['builder']) && is_string($field['builder'])) {

DependencyInjection/TypesConfiguration.php

Doc Comments +4 added lines

@@ -152,6 +152,10 @@
             ->end();
     }
 
+    /**
+     * @param string $typeToTreat
+     * @param string $definitionBuilderClass
+     */
     private function relayNormalizer($typeToTreat, $definitionBuilderClass)
     {
         return function ($types) use ($typeToTreat, $definitionBuilderClass) {

Spacing +11 added lines, -11 removed lines

@@ -36,7 +36,7 @@ 
         $rootNode = $treeBuilder->root('overblog_graphql_types');
 
         $configTypeKeys = array_map(
-            function ($type) {
+            function($type) {
                 return $this->normalizedConfigTypeKey($type);
             },
             static::$types
@@ -49,7 +49,7 @@ 
             ->prototype('array')
                 // config is the unique config entry allowed
                 ->beforeNormalization()
-                    ->ifTrue(function ($v) use ($configTypeKeys) {
+                    ->ifTrue(function($v) use ($configTypeKeys) {
                         if (!empty($v) && is_array($v)) {
                             $keys = array_keys($v);
                             foreach ($configTypeKeys as $configTypeKey) {
@@ -64,16 +64,16 @@ 
                         ->thenInvalid(
                             sprintf(
                                 'Don\'t use internal config keys %s, replace it by "config" instead.',
-                                implode(', ',  $configTypeKeys)
+                                implode(', ', $configTypeKeys)
                             )
                         )
                 ->end()
                 // config is renamed _{TYPE}_config
                 ->beforeNormalization()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return isset($v['type']) && is_string($v['type']);
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         $key = $this->normalizedConfigTypeKey($v['type']);
 
                         if (empty($v[$key])) {
@@ -96,10 +96,10 @@ 
                 ->end()
                 // _{TYPE}_config is renamed config
                 ->validate()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return isset($v[$this->normalizedConfigTypeKey($v['type'])]);
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         $key = $this->normalizedConfigTypeKey($v['type']);
                         $v['config'] = $v[$key];
                         unset($v[$key]);
@@ -115,13 +115,13 @@ 
 
     private function addBeforeNormalization(ArrayNodeDefinition $node)
     {
-        $typeKeyExists = function ($types) { return !empty($types) && is_array($types); };
+        $typeKeyExists = function($types) { return !empty($types) && is_array($types); };
 
         $node
             // set type config.name
             ->beforeNormalization()
                 ->ifTrue($typeKeyExists)
-                ->then(function ($types) {
+                ->then(function($types) {
                     foreach ($types as $name => &$type) {
                         $type['config'] = isset($type['config']) && is_array($type['config']) ? $type['config'] : [];
                         $type['config']['name'] = $name;
@@ -147,14 +147,14 @@ 
             ->end()
             // normalized relay-mutation-payload
             ->beforeNormalization()
-                ->ifTrue(function ($types) { return !empty($types) && is_array($types); })
+                ->ifTrue(function($types) { return !empty($types) && is_array($types); })
                 ->then($this->relayNormalizer('relay-mutation-payload', 'Overblog\GraphQLBundle\Relay\Mutation\PayloadDefinition'))
             ->end();
     }
 
     private function relayNormalizer($typeToTreat, $definitionBuilderClass)
     {
-        return function ($types) use ($typeToTreat, $definitionBuilderClass) {
+        return function($types) use ($typeToTreat, $definitionBuilderClass) {
             foreach ($types as $name => $type) {
                 if (isset($type['type']) && is_string($type['type']) && $typeToTreat === $type['type']) {
                     $config = isset($type['config']) && is_array($type['config']) ? $type['config'] : [];

Config/TypeDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         $node = $builder->root('name', 'scalar');
         $node->isRequired();
         $node->validate()
-            ->ifTrue(function ($name) {
+            ->ifTrue(function($name) {
                 return !preg_match('/^[_a-z][_0-9a-z]*$/i', $name);
             })
                 ->thenInvalid('Invalid type name "%s". (see https://facebook.github.io/graphql/#Name)')

Config/ObjectTypeDefinition.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
             ->end();
 
         $node->validate()
-            ->ifTrue(function ($v) {
+            ->ifTrue(function($v) {
 
                 return array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess'];
             })
-            ->then(function ($v) {
+            ->then(function($v) {
                 foreach ($v['fields'] as &$field) {
                     if (array_key_exists('access', $field) && null !== $field['access']) {
                         continue;

Config/EnumTypeDefinition.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
                     ->useAttributeAsKey('name')
                     ->prototype('array')
                         ->beforeNormalization()
-                            ->ifTrue(function ($v) { return !is_null($v) && !is_array($v); })
-                            ->then(function ($v) { return ['value' => $v]; })
+                            ->ifTrue(function($v) { return !is_null($v) && !is_array($v); })
+                            ->then(function($v) { return ['value' => $v]; })
                         ->end()
                         ->isRequired()
                         ->children()

Tests/Resolver/Toto.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
     public function getPrivatePropertyWithoutGetterUsingCallBack()
     {
-        return function () {
+        return function() {
             return $this->privatePropertyWithoutGetter;
         };
     }

DependencyInjection/Compiler/TypesPass.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     private function processConfig(array $configs)
     {
         return array_map(
-            function ($v) {
+            function($v) {
                 if (is_array($v)) {
                     return call_user_func([$this, 'processConfig'], $v);
                 } elseif (is_string($v) && 0 === strpos($v, '@=')) {

Resolver/AccessResolver.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         switch (true) {
             case is_array($result):
                 $result = array_map(
-                    function ($object) use ($accessChecker, $resolveArgs) {
+                    function($object) use ($accessChecker, $resolveArgs) {
                         return $this->checkAccess($accessChecker, $object, $resolveArgs) ? $object : null;
                     },
                     $result
@@ -44,7 +44,7 @@ 
                 break;
             case $result instanceof Connection:
                 $result->edges = array_map(
-                    function (Edge $edge) use ($accessChecker, $resolveArgs) {
+                    function(Edge $edge) use ($accessChecker, $resolveArgs) {
                         $edge->node = $this->checkAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
 
                         return $edge;

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +24 added lines, -24 removed lines

@@ -22,91 +22,91 @@ 
         return [
             new ExpressionFunction(
                 'service',
-                function ($value) {
+                function($value) {
                     return sprintf('$container->get(%s)', $value);
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'parameter',
-                function ($value) {
+                function($value) {
                     return sprintf('$container->getParameter(%s)', $value);
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'isTypeOf',
-                function ($className) {
+                function($className) {
                     return sprintf('($className = %s) && $value instanceof $className', $className);
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'resolver',
-                function ($alias, $args = '[]') {
+                function($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.resolver_resolver\')->resolve([%s, %s])', $alias, $args);
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'mutateAndGetPayloadCallback',
-                function ($mutateAndGetPayload) {
+                function($mutateAndGetPayload) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
                     $code .= 'return '.$mutateAndGetPayload.'; }';
 
                     return $code;
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'mutateAndGetPayloadCallback',
-                function ($mutateAndGetPayload) {
+                function($mutateAndGetPayload) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
                     $code .= 'return '.$mutateAndGetPayload.'; }';
 
                     return $code;
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'idFetcherCallback',
-                function ($idFetcher) {
+                function($idFetcher) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
                     $code .= 'return '.$idFetcher.'; }';
 
                     return $code;
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'resolveSingleInputCallback',
-                function ($resolveSingleInput) {
+                function($resolveSingleInput) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
                     $code .= 'return '.$resolveSingleInput.'; }';
 
                     return $code;
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'mutation',
-                function ($alias, $args = '[]') {
+                function($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.mutation_resolver\')->resolve([%s, %s])', $alias, $args);
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'globalId',
-                function ($id, $typeName = null) {
+                function($id, $typeName = null) {
                     $typeNameEmpty = null === $typeName || '""' === $typeName || 'null' === $typeName || 'false' === $typeName;
 
                     return sprintf(
@@ -115,26 +115,26 @@ 
                         $id
                     );
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'fromGlobalId',
-                function ($globalId) {
+                function($globalId) {
                     return sprintf(
                         '\\Overblog\\GraphQLBundle\\Relay\\Node\\GlobalId::fromGlobalId(%s)',
                         $globalId
                     );
                 },
-                function () {}
+                function() {}
             ),
 
             new ExpressionFunction(
                 'newObject',
-                function ($className, $args = '[]') {
+                function($className, $args = '[]') {
                     return sprintf('(new \ReflectionClass(%s))->newInstanceArgs(%s)', $className, $args);
                 },
-                function () {}
+                function() {}
             ),
         ];
     }