overblog / GraphQLBundle

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;

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]);
             });
         }

ExpressionLanguage/ExpressionFunction.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     public function __construct($name, callable $compiler)
     {
-        parent::__construct($name, $compiler, function () {
+        parent::__construct($name, $compiler, function() {
             throw new \RuntimeException('No need evaluator.');
         });
     }

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +11 added lines, -11 removed lines

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

ExpressionLanguage/AuthorizationExpressionProvider.php

Spacing +8 added lines, -8 removed lines

@@ -20,14 +20,14 @@ 
         return [
             new ExpressionFunction(
                 'hasRole',
-                function ($role) {
+                function($role) {
                     return sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s)', $role);
                 }
             ),
 
             new ExpressionFunction(
                 'hasAnyRole',
-                function ($roles) {
+                function($roles) {
                     $code = sprintf('array_reduce(%s, function ($isGranted, $role) use ($container) { return $isGranted || $container->get(\'security.authorization_checker\')->isGranted($role); }, false)', $roles);
 
                     return $code;
@@ -36,35 +36,35 @@ 
 
             new ExpressionFunction(
                 'isAnonymous',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_ANONYMOUSLY\')';
                 }
             ),
 
             new ExpressionFunction(
                 'isRememberMe',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\')';
                 }
             ),
 
             new ExpressionFunction(
                 'isFullyAuthenticated',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 }
             ),
 
             new ExpressionFunction(
                 'isAuthenticated',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\') || $container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 }
             ),
 
             new ExpressionFunction(
                 'hasPermission',
-                function ($object, $permission) {
+                function($object, $permission) {
                     $code = sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s, %s)', $permission, $object);
 
                     return $code;
@@ -73,7 +73,7 @@ 
 
             new ExpressionFunction(
                 'hasAnyPermission',
-                function ($object, $permissions) {
+                function($object, $permissions) {
                     $code = sprintf('array_reduce(%s, function ($isGranted, $permission) use ($container, $object) { return $isGranted || $container->get(\'security.authorization_checker\')->isGranted($permission, %s); }, false)', $permissions, $object);
 
                     return $code;

Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -58,10 +58,10 @@
         /** @var \React\Promise\PromiseInterface $reactPromise */
         $reactPromise = $promise->adoptedPromise;
 
-        $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/Executor/Promise/Adapter/ReactPromiseAdapterTest.php

Spacing +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
 EOF
         );
 
-        $promise = $this->adapter->create(function (callable $resolve) use (&$process) {
-            $process->start(function () use ($resolve, &$process) {
+        $promise = $this->adapter->create(function(callable $resolve) use (&$process) {
+            $process->start(function() use ($resolve, &$process) {
                 $output = $process->getOutput();
                 $resolve($output);
             });
@@ -64,7 +64,7 @@ 
 
         $this->assertEquals(
             $output,
-            $this->adapter->wait($promise, function () use (&$process) {
+            $this->adapter->wait($promise, function() use (&$process) {
                 $process->wait();
             })
         );

Tests/DIContainerMockTrait.php

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
  */
 trait DIContainerMockTrait
 {
+    /**
+     * @return null|\Symfony\Component\DependencyInjection\ContainerInterface
+     */
     private function getDIContainerMock(array $services = [], array $parameters = [])
     {
         $container = $this->getMockBuilder('Symfony\\Component\\DependencyInjection\\Container')