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

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

Tests/Relay/Connection/PaginatorTest.php

Spacing +14 added lines, -14 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);
 

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)) {