overblog / GraphQLBundle

DependencyInjection/OverblogGraphQLTypesExtension.php

Spacing +2 added lines, -2 removed lines

@@ -104,8 +104,8 @@
         // from config
         if (!empty($config['definitions']['mappings']['types'])) {
             $typesMappings = array_filter(array_map(
-                function (array $typeMapping) use ($container) {
-                    $params = $this->detectConfigFiles($container, $typeMapping['dir'],  $typeMapping['type']);
+                function(array $typeMapping) use ($container) {
+                    $params = $this->detectConfigFiles($container, $typeMapping['dir'], $typeMapping['type']);
 
                     return $params;
                 },

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

Config/EnumTypeDefinition.php

Spacing +2 added lines, -2 removed lines

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

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +24 added lines, -24 removed lines

@@ -22,100 +22,100 @@ 
         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(
@@ -124,28 +124,28 @@ 
                         $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() {
                 }
             ),
         ];

ExpressionLanguage/AuthorizationExpressionProvider.php

Spacing +16 added lines, -16 removed lines

@@ -21,79 +21,79 @@
         return [
             new ExpressionFunction(
                 'hasRole',
-                function ($role) {
+                function($role) {
                     return sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s)', $role);
                 },
-                function () {
+                function() {
                 }
             ),
 
             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;
                 },
-                function () {
+                function() {
                 }
             ),
 
             new ExpressionFunction(
                 'isAnonymous',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_ANONYMOUSLY\')';
                 },
-                function () {
+                function() {
                 }
             ),
 
             new ExpressionFunction(
                 'isRememberMe',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\')';
                 },
-                function () {
+                function() {
                 }
             ),
 
             new ExpressionFunction(
                 'isFullyAuthenticated',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 },
-                function () {
+                function() {
                 }
             ),
 
             new ExpressionFunction(
                 'isAuthenticated',
-                function () {
+                function() {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\') || $container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 },
-                function () {
+                function() {
                 }
             ),
 
             new ExpressionFunction(
                 'hasPermission',
-                function ($object, $permission) {
+                function($object, $permission) {
                     $code = sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s, %s)', $permission, $object);
 
                     return $code;
                 },
-                function () {
+                function() {
                 }
             ),
 
             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;
                 },
-                function () {
+                function() {
                 }
             ),
         ];

Tests/Request/ExecutorTest.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
             'fields' => [
                 'myField' => [
                     'type' => Type::boolean(),
-                    'resolve' => function () {
+                    'resolve' => function() {
                         return false;
                     },
                 ],
@@ -46,7 +46,7 @@ 
      */
     public function testInvalidExecutorReturnNotObject()
     {
-        $this->executor->setExecutor(function () {
+        $this->executor->setExecutor(function() {
             return false;
         });
         $this->executor->execute($this->request);
@@ -58,7 +58,7 @@ 
      */
     public function testInvalidExecutorReturnInvalidObject()
     {
-        $this->executor->setExecutor(function () {
+        $this->executor->setExecutor(function() {
             return new \stdClass();
         });
         $this->executor->execute($this->request);

Relay/Connection/Paginator.php

Doc Comments +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
     }
 
     /**
-     * @param Argument|array $args
+     * @param Argument $args
      * @param int|callable   $total
      *
      * @return Connection
@@ -42,7 +42,7 @@ 
     }
 
     /**
-     * @param Argument|array $args
+     * @param Argument $args
      *
      * @return Connection
      */
@@ -62,7 +62,7 @@ 
     }
 
     /**
-     * @param Argument|array $args
+     * @param Argument $args
      * @param int|callable   $total
      *
      * @return Connection

Tests/Relay/Connection/PaginatorTest.php

Spacing +8 added lines, -8 removed lines

@@ -18,7 +18,7 @@ 
 {
     public function testForward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(6, $limit); // Includes the extra element to check if next page is available
 
@@ -30,7 +30,7 @@ 
 
     public function testForwardAfter()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(5, $offset);
             $this->assertSame(6, $limit); // Includes the extra element to check if next page is available
 
@@ -42,7 +42,7 @@ 
 
     public function testBackward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(5, $offset);
             $this->assertSame(5, $limit);
 
@@ -54,7 +54,7 @@ 
 
     public function testBackwardBefore()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit);
 
@@ -67,7 +67,7 @@ 
     public function testAuto()
     {
         // Backward
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(5, $offset);
             $this->assertSame(5, $limit);
 
@@ -77,7 +77,7 @@ 
         $this->assertCount(5, $paginator->auto(new Argument(['last' => 5]), 10)->edges);
 
         // Forward
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(6, $limit); // Includes the extra element to check if next page is available
 
@@ -87,7 +87,7 @@ 
         $this->assertCount(5, $paginator->auto(new Argument(['first' => 5]), 10)->edges);
 
         // Backward + callable
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(5, $offset);
             $this->assertSame(5, $limit);
 
@@ -95,7 +95,7 @@ 
         });
 
         $countCalled = false;
-        $result = $paginator->auto(new Argument(['last' => 5]), function () use (&$countCalled) {
+        $result = $paginator->auto(new Argument(['last' => 5]), function() use (&$countCalled) {
             $countCalled = true;
 
             return 10;