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;

Relay/Connection/Output/ConnectionBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -136,7 +136,7 @@ 
      * Return the cursor associated with an object in an array.
      *
      * @param array $data
-     * @param mixed $object
+     * @param string $object
      *
      * @return null|string
      */
@@ -196,9 +196,9 @@ 
     /**
      * Redefines the offset from the cursor string.
      *
-     * @param $cursor
+     * @param string $cursor
      *
-     * @return int
+     * @return string
      */
     public static function cursorToOffset($cursor)
     {

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
             $edges,
             new PageInfo(
                 $firstEdge instanceof Edge ? $firstEdge->cursor : null,
-                $lastEdge instanceof Edge  ? $lastEdge->cursor : null,
+                $lastEdge instanceof Edge ? $lastEdge->cursor : null,
                 $last !== null ? $startOffset > $lowerBound : false,
                 $first !== null ? $endOffset < $upperBound : false
             )
@@ -178,7 +178,7 @@ 
         }
         $offset = static::cursorToOffset($cursor);
 
-        return !is_numeric($offset) ?  $defaultOffset : (int) $offset;
+        return !is_numeric($offset) ? $defaultOffset : (int) $offset;
     }
 
     /**

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.'