overblog / GraphQLBundle

DependencyInjection/TypesConfiguration.php

Doc Comments +3 added lines

@@ -121,6 +121,9 @@
         return $node;
     }
 
+    /**
+     * @param string $name
+     */
     private function addFieldsSelection($name, $enabledBuilder = true)
     {
         $builder = new TreeBuilder();

Spacing +5 added lines, -5 removed lines

@@ -86,9 +86,9 @@ 
             ->variableNode('resolveNode')->end()
         ->end()
         ->validate()
-            ->always(function ($v) {
+            ->always(function($v) {
                 // remove all empty value
-                $array_filter_recursive = function ($input) use (&$array_filter_recursive) {
+                $array_filter_recursive = function($input) use (&$array_filter_recursive) {
                     foreach ($input as $key => &$value) {
                         if ('defaultValue' === $key) {
                             continue;
@@ -130,7 +130,7 @@ 
             ->prototype('array')
                 ->beforeNormalization()
                     ->ifString()
-                    ->then(function ($v) { return array('builder' => $v); })
+                    ->then(function($v) { return array('builder' => $v); })
                 ->end()
                 ->children()
                     ->append($this->addTypeSelection())
@@ -138,7 +138,7 @@ 
                         ->info('Use to build dynamic args. Can be combine with args.')
                         ->beforeNormalization()
                             ->ifString()
-                            ->then(function ($v) { return array('name' => $v); })
+                            ->then(function($v) { return array('name' => $v); })
                         ->end()
                         ->children()
                             ->scalarNode('name')
@@ -183,7 +183,7 @@ 
             ->end();
 
         $node->validate()
-            ->ifTrue(function ($fields) use ($enabledBuilder) {
+            ->ifTrue(function($fields) use ($enabledBuilder) {
                 foreach ($fields as $v) {
                     if (empty($v['type']) && $enabledBuilder && empty($v['builder'])) {
                         return true;

Resolver/ConfigResolver.php

Doc Comments +11 added lines

@@ -246,6 +246,9 @@ 
         return $type;
     }
 
+    /**
+     * @param callable $expression
+     */
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
         return function () use ($expression, $resolveCallback) {
@@ -271,6 +274,10 @@ 
             if (is_array($result) || $result instanceof \ArrayAccess) {
                 $result = array_filter(
                     array_map(
+
+                        /**
+                         * @param callable $object
+                         */
                         function ($object) use ($checkAccess) {
                             return $checkAccess($object) ? $object : null;
                         },
@@ -279,6 +286,10 @@ 
                 );
             } elseif ($result instanceof Connection) {
                 $result->edges = array_map(
+
+                    /**
+                     * @param callable $edge
+                     */
                     function (Edge $edge) use ($checkAccess) {
                         $edge->node = $checkAccess($edge->node) ? $edge->node : null;
 

Spacing +8 added lines, -8 removed lines

@@ -212,7 +212,7 @@ 
 
     private function resolveTypeCallback($expr)
     {
-        return function () use ($expr) {
+        return function() use ($expr) {
             return $this->resolveType($expr);
         };
     }
@@ -248,14 +248,14 @@ 
 
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
-        return function () use ($expression, $resolveCallback) {
+        return function() use ($expression, $resolveCallback) {
             $args = func_get_args();
 
-            $result = null !== $resolveCallback  ? call_user_func_array($resolveCallback, $args) : null;
+            $result = null !== $resolveCallback ? call_user_func_array($resolveCallback, $args) : null;
 
             $values = call_user_func_array([$this, 'resolveResolveCallbackArgs'], $args);
 
-            $checkAccess = function ($object) use ($expression, $values) {
+            $checkAccess = function($object) use ($expression, $values) {
                 try {
                     $access = $this->resolveUsingExpressionLanguageIfNeeded(
                         $expression,
@@ -271,7 +271,7 @@ 
             if (is_array($result) || $result instanceof \ArrayAccess) {
                 $result = array_filter(
                     array_map(
-                        function ($object) use ($checkAccess) {
+                        function($object) use ($checkAccess) {
                             return $checkAccess($object) ? $object : null;
                         },
                         $result
@@ -279,7 +279,7 @@ 
                 );
             } elseif ($result instanceof Connection) {
                 $result->edges = array_map(
-                    function (Edge $edge) use ($checkAccess) {
+                    function(Edge $edge) use ($checkAccess) {
                         $edge->node = $checkAccess($edge->node) ? $edge->node : null;
 
                         return $edge;
@@ -300,7 +300,7 @@ 
             return $value;
         }
 
-        return function () use ($value) {
+        return function() use ($value) {
             $args = func_get_args();
             $result = $this->resolveUsingExpressionLanguageIfNeeded(
                 $value,
@@ -349,7 +349,7 @@ 
 
     private function resolveUsingExpressionLanguageIfNeeded($expression, array $values = [])
     {
-        if (is_string($expression) &&  0 === strpos($expression, '@=')) {
+        if (is_string($expression) && 0 === strpos($expression, '@=')) {
             return $this->expressionLanguage->evaluate(substr($expression, 2), $values);
         }
 

Definition/MergeFieldTrait.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 {
     protected function getFieldsWithDefaults($fields, array $defaultFields, $forceArray = true)
     {
-        $callback = function () use ($fields, $defaultFields) {
+        $callback = function() use ($fields, $defaultFields) {
             if (empty($fields)) {
                 return $defaultFields;
             }

Tests/Error/ErrorHandlerTest.php

Spacing +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
             ]
         );
 
-        $this->errorHandler->setErrorHandler(function () {
+        $this->errorHandler->setErrorHandler(function() {
             return [new Error('Override Error')];
         });
 

Tests/Functional/Relay/Connection/ConnectionTest.php

Indentation +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         $expectedData = [
             'user' => [
                 'friends' => [
-                   'totalCount' => 4,
+                    'totalCount' => 4,
                     'edges'     => [
                         [
                             'friendshipTime' => 'Yesterday',

Relay/Node/RawIdField.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             'name'        => $name,
             'description' => 'The raw ID of an object',
             'type'        => Type::nonNull(Type::int()),
-            'resolve'     => function ($obj, $args, ResolveInfo $info) use ($idFetcher) {
+            'resolve'     => function($obj, $args, ResolveInfo $info) use ($idFetcher) {
                 return is_callable($idFetcher) ? $idFetcher($obj, $info) : $obj->id;
             },
         ];

Relay/Node/GlobalIdField.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             'name'        => $name,
             'description' => 'The ID of an object',
             'type'        => Type::nonNull(Type::id()),
-            'resolve'     => function ($obj, $args, ResolveInfo $info) use ($idFetcher, $typeName) {
+            'resolve'     => function($obj, $args, ResolveInfo $info) use ($idFetcher, $typeName) {
                 return GlobalId::toGlobalId(
                     !empty($typeName) ? $typeName : $info->parentType->name,
                     is_callable($idFetcher) ? call_user_func_array($idFetcher, [$obj, $info]) : (is_object($obj) ? $obj->id : $obj['id'])

Relay/Node/NodeField.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
             'args'        => [
                 'id' => ['type' => Type::nonNull(Type::id()), 'description' => 'The ID of an object'],
             ],
-            'resolve' => function ($obj, $args, $info) use ($idFetcher) {
+            'resolve' => function($obj, $args, $info) use ($idFetcher) {
                 if (empty($args['id'])) {
                     throw new \InvalidArgumentException(
                         'Argument "id" is required but not provided.'

Relay/Node/PluralIdentifyingRootField.php

Spacing +2 added lines, -3 removed lines

@@ -41,15 +41,14 @@
             'description' => isset($config['description']) ? $config['description'] : null,
             'type'        => Type::listOf($config['outputType']),
             'args'        => $inputArgs,
-            'resolve'     => function ($obj, $args, $info) use ($config) {
+            'resolve'     => function($obj, $args, $info) use ($config) {
                 $inputs = $args[$config['argName']];
 
                 $data = [];
 
                 foreach ($inputs as $input) {
                     $data[$input] = is_callable($config['resolveSingleInput']) ?
-                        call_user_func_array($config['resolveSingleInput'], [$input, $info]) :
-                        null;
+                        call_user_func_array($config['resolveSingleInput'], [$input, $info]) : null;
                 }
 
                 return $data;