overblog / GraphQLBundle

DependencyInjection/Configuration.php

Spacing +5 added lines, -5 removed lines

@@ -48,14 +48,14 @@ 
                         ->booleanNode('config_validation')->defaultValue($this->debug)->end()
                         ->arrayNode('schema')
                             ->beforeNormalization()
-                                ->ifTrue(function ($v) {
+                                ->ifTrue(function($v) {
                                     $needNormalization = isset($v['query']) && is_string($v['query']) ||
                                         isset($v['mutation']) && is_string($v['mutation']) ||
                                         isset($v['subscription']) && is_string($v['subscription']);
 
                                     return $needNormalization;
                                 })
-                                ->then(function ($v) {
+                                ->then(function($v) {
                                     return ['default' => $v];
                                 })
                             ->end()
@@ -175,16 +175,16 @@ 
         $node
             ->info('Disabled if equal to false.')
             ->beforeNormalization()
-                ->ifTrue(function ($v) {
+                ->ifTrue(function($v) {
                     return false === $v;
                 })
-                ->then(function () use ($disabledValue) {
+                ->then(function() use ($disabledValue) {
                     return $disabledValue;
                 })
             ->end()
             ->defaultFalse()
             ->validate()
-                ->ifTrue(function ($v) {
+                ->ifTrue(function($v) {
                     return $v < 0;
                 })
                 ->thenInvalid('"overblog_graphql.security.'.$name.'" must be greater or equal to 0.')

Tests/Request/ExecutorTest.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
             'fields' => [
                 'myField' => [
                     'type' => Type::boolean(),
-                    'resolve' => function () {
+                    'resolve' => function() {
                         return false;
                     },
                 ],

Config/EnumTypeDefinition.php

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
                     ->useAttributeAsKey('name')
                     // if value not define we use name as value
                     ->beforeNormalization()
-                        ->ifTrue(function ($v) {
+                        ->ifTrue(function($v) {
                             return is_array($v);
                         })
-                        ->then(function ($v) {
+                        ->then(function($v) {
                             foreach ($v as $name => &$options) {
                                 // short syntax NAME: VALUE
                                 if (!is_null($options) && !is_array($options)) {

Relay/Connection/Paginator.php

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
 
         $entities = call_user_func($this->fetcher, $offset, $limit);
 
-        return $this->handleEntities($entities, function ($entities) use ($args, $offset, $total) {
+        return $this->handleEntities($entities, function($entities) use ($args, $offset, $total) {
             return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
                 'sliceStart' => $offset,
                 'arrayLength' => $total,
@@ -81,13 +81,13 @@ 
         if (!is_numeric(ConnectionBuilder::cursorToOffset($args['after'])) || !$args['after']) {
             $entities = call_user_func($this->fetcher, $offset, $limit + 1);
 
-            return $this->handleEntities($entities, function ($entities) use ($args) {
+            return $this->handleEntities($entities, function($entities) use ($args) {
                 return ConnectionBuilder::connectionFromArray($entities, $args);
             });
         } else {
             $entities = call_user_func($this->fetcher, $offset, $limit + 2);
 
-            return $this->handleEntities($entities, function ($entities) use ($args, $offset) {
+            return $this->handleEntities($entities, function($entities) use ($args, $offset) {
                 return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
                     'sliceStart' => $offset,
                     'arrayLength' => $offset + count($entities),

Tests/Relay/Connection/PaginatorTest.php

Spacing +15 added lines, -15 removed lines

@@ -55,7 +55,7 @@ 
 
     public function testForward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -71,7 +71,7 @@ 
 
     public function testForwardAfterInMiddle()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(3, $limit); // Includes the extra element to check if next page is available
 
@@ -87,7 +87,7 @@ 
 
     public function testForwardAfterAtTheEnd()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(4, $limit); // Includes the extra element to check if next page is available
 
@@ -103,7 +103,7 @@ 
 
     public function testForwardAfterLast()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(4, $offset);
             $this->assertSame(7, $limit); // Includes the extra element to check if next page is available
 
@@ -119,7 +119,7 @@ 
 
     public function testForwardAfterWithUnvalidCursorAndSlice()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -136,7 +136,7 @@ 
 
     public function testBackward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(3, $limit);
 
@@ -153,7 +153,7 @@ 
 
     public function testBackwardWithLimitEqualsToTotal()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit);
 
@@ -170,7 +170,7 @@ 
 
     public function testBackwardBeforeLast()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(4, $limit);
 
             return $this->getData($offset);
@@ -185,7 +185,7 @@ 
 
     public function testBackwardPartialBeforeInMiddle()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(2, $limit);
 
@@ -201,7 +201,7 @@ 
 
     public function testAutoBackward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -218,7 +218,7 @@ 
 
     public function testAutoForward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -234,7 +234,7 @@ 
 
     public function testAutoBackwardWithCallable()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -242,7 +242,7 @@ 
         });
 
         $countCalled = false;
-        $result = $paginator->auto(new Argument(['last' => 4]), function () use (&$countCalled) {
+        $result = $paginator->auto(new Argument(['last' => 4]), function() use (&$countCalled) {
             $countCalled = true;
 
             return 5;
@@ -265,7 +265,7 @@ 
 
         $this->assertSame(5, call_user_func_array($callable, ['array' => $this->data]));
 
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -292,7 +292,7 @@ 
 
         $promise->expects($this->once())->method('then');
 
-        $paginator = new Paginator(function ($offset, $limit) use ($promise) {
+        $paginator = new Paginator(function($offset, $limit) use ($promise) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit);
 

Request/Executor.php

Doc Comments +7 added lines

@@ -81,6 +81,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function addSchema($name, Schema $schema)
     {
         $this->schemas[$name] = $schema;
@@ -176,6 +179,10 @@ 
         return $this->prepareResult($result, $startTime, $startMemoryUsage);
     }
 
+    /**
+     * @param double $startTime
+     * @param integer $startMemoryUsage
+     */
     private function prepareResult($result, $startTime, $startMemoryUsage)
     {
         if ($this->hasDebugInfo()) {

Resolver/AccessResolver.php

Spacing +3 added lines, -3 removed lines

@@ -53,7 +53,7 @@ 
                 $result = $this->promiseAdapter->convertThenable($result);
             }
 
-            return $this->promiseAdapter->then($result, function ($result) use ($accessChecker, $resolveArgs) {
+            return $this->promiseAdapter->then($result, function($result) use ($accessChecker, $resolveArgs) {
                 return $this->processFilter($result, $accessChecker, $resolveArgs);
             });
         }
@@ -65,14 +65,14 @@ 
     {
         if (is_array($result)) {
             $result = array_map(
-                function ($object) use ($accessChecker, $resolveArgs) {
+                function($object) use ($accessChecker, $resolveArgs) {
                     return $this->hasAccess($accessChecker, $object, $resolveArgs) ? $object : null;
                 },
                 $result
             );
         } elseif ($result instanceof Connection) {
             $result->edges = array_map(
-                function (Edge $edge) use ($accessChecker, $resolveArgs) {
+                function(Edge $edge) use ($accessChecker, $resolveArgs) {
                     $edge->node = $this->hasAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
 
                     return $edge;

Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -59,10 +59,10 @@
         $reactPromise = $promise->adoptedPromise;
 
         while ($wait) {
-            $reactPromise->then(function ($values) use (&$resolvedValue, &$wait) {
+            $reactPromise->then(function($values) use (&$resolvedValue, &$wait) {
                 $resolvedValue = $values;
                 $wait = false;
-            }, function ($reason) use (&$exception, &$wait) {
+            }, function($reason) use (&$exception, &$wait) {
                 $exception = $reason;
                 $wait = false;
             });

Tests/Functional/app/Resolver/ConnectionResolver.php

Spacing +5 added lines, -5 removed lines

@@ -56,21 +56,21 @@ 
 
     public function friendsResolver($user, $args)
     {
-        return $this->promiseAdapter->create(function (callable $resolve) use ($user, $args) {
+        return $this->promiseAdapter->create(function(callable $resolve) use ($user, $args) {
             return $resolve(ConnectionBuilder::connectionFromArray($user['friends'], $args));
         });
     }
 
     public function resolveNode(Edge $edge)
     {
-        return $this->promiseAdapter->create(function (callable $resolve) use ($edge) {
+        return $this->promiseAdapter->create(function(callable $resolve) use ($edge) {
             return $resolve(isset($this->allUsers[$edge->node]) ? $this->allUsers[$edge->node] : null);
         });
     }
 
     public function resolveConnection()
     {
-        return $this->promiseAdapter->create(function (callable $resolve) {
+        return $this->promiseAdapter->create(function(callable $resolve) {
             return $resolve(count($this->allUsers) - 1);
         });
     }
@@ -78,11 +78,11 @@ 
     public function resolveQuery()
     {
         if ($this->promiseAdapter instanceof GraphQLPromiseAdapter) {
-            return new Deferred(function () {
+            return new Deferred(function() {
                 return $this->allUsers[0];
             });
         } elseif ($this->promiseAdapter instanceof ReactPromiseAdapter) {
-            return new Promise(function (callable $resolve) {
+            return new Promise(function(callable $resolve) {
                 return $resolve($this->allUsers[0]);
             });
         }