overblog / GraphQLBundle

Config/EnumTypeDefinition.php

Spacing +2 added lines, -2 removed lines

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

Tests/Resolver/Toto.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
     public function getPrivatePropertyWithoutGetterUsingCallBack()
     {
-        return function () {
+        return function() {
             return $this->privatePropertyWithoutGetter;
         };
     }

DependencyInjection/Compiler/TypesPass.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     private function processConfig(array $configs)
     {
         return array_map(
-            function ($v) {
+            function($v) {
                 if (is_array($v)) {
                     return call_user_func([$this, 'processConfig'], $v);
                 } elseif (is_string($v) && 0 === strpos($v, '@=')) {

Resolver/AccessResolver.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         switch (true) {
             case is_array($result):
                 $result = array_map(
-                    function ($object) use ($accessChecker, $resolveArgs) {
+                    function($object) use ($accessChecker, $resolveArgs) {
                         return $this->checkAccess($accessChecker, $object, $resolveArgs) ? $object : null;
                     },
                     $result
@@ -44,7 +44,7 @@ 
                 break;
             case $result instanceof Connection:
                 $result->edges = array_map(
-                    function (Edge $edge) use ($accessChecker, $resolveArgs) {
+                    function(Edge $edge) use ($accessChecker, $resolveArgs) {
                         $edge->node = $this->checkAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
 
                         return $edge;

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +24 added lines, -24 removed lines

@@ -22,91 +22,91 @@ 
         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(
@@ -115,26 +115,26 @@ 
                         $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,72 +21,72 @@
         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() {}
             ),
         ];
     }

Relay/Node/PluralIdentifyingRootFieldDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
     private function cleanResolveSingleInput($resolveSingleInput)
     {
-        if (is_string($resolveSingleInput) &&  0 === strpos($resolveSingleInput, '@=')) {
+        if (is_string($resolveSingleInput) && 0 === strpos($resolveSingleInput, '@=')) {
             $cleanResolveSingleInput = substr($resolveSingleInput, 2);
         } else {
             $cleanResolveSingleInput = json_encode($resolveSingleInput);

Relay/Mutation/MutationFieldDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 
     private function cleanMutateAndGetPayload($mutateAndGetPayload)
     {
-        if (is_string($mutateAndGetPayload) &&  0 === strpos($mutateAndGetPayload, '@=')) {
+        if (is_string($mutateAndGetPayload) && 0 === strpos($mutateAndGetPayload, '@=')) {
             $cleanMutateAndGetPayload = substr($mutateAndGetPayload, 2);
         } else {
             $cleanMutateAndGetPayload = json_encode($mutateAndGetPayload);

Tests/ExpressionLanguage/AuthorizationExpressionProviderTest.php

Doc Comments +3 added lines

@@ -113,6 +113,9 @@
         );
     }
 
+    /**
+     * @param string $expression
+     */
     private function assertExpressionCompile($expression, $with, array $expressionValues = [], $expects = null, $return = true, $assertMethod = 'assertTrue')
     {
         $authChecker = $this->getAuthorizationCheckerIsGrantedWithExpectation($with, $expects, $return);