overblog / GraphQLBundle

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +8 added lines, -8 removed lines

@@ -20,37 +20,37 @@
     public function getFunctions()
     {
         return [
-            new ExpressionFunction('service', function () {}, function (array $variables, $value) {
+            new ExpressionFunction('service', function() {}, function(array $variables, $value) {
                 return $variables['container']->get($value);
             }),
 
-            new ExpressionFunction('parameter', function () {}, function (array $variables, $value) {
+            new ExpressionFunction('parameter', function() {}, function(array $variables, $value) {
                 return $variables['container']->getParameter($value);
             }),
 
-            new ExpressionFunction('isTypeOf', function () {}, function (array $variables, $className) {
+            new ExpressionFunction('isTypeOf', function() {}, function(array $variables, $className) {
                 return $variables['value'] instanceof $className;
             }),
 
-            new ExpressionFunction('resolver', function () {}, function (array $variables, $alias, array $args = []) {
+            new ExpressionFunction('resolver', function() {}, function(array $variables, $alias, array $args = []) {
                 return $variables['container']->get('overblog_graphql.resolver_resolver')->resolve([$alias, $args]);
             }),
 
-            new ExpressionFunction('mutation', function () {}, function (array $variables, $alias, array $args = []) {
+            new ExpressionFunction('mutation', function() {}, function(array $variables, $alias, array $args = []) {
                 return $variables['container']->get('overblog_graphql.mutation_resolver')->resolve([$alias, $args]);
             }),
 
-            new ExpressionFunction('globalId', function () {}, function (array $variables, $id, $typeName = null) {
+            new ExpressionFunction('globalId', function() {}, function(array $variables, $id, $typeName = null) {
                 $type = !empty($typeName) ? $typeName : $variables['info']->parentType->name;
 
                 return GlobalId::toGlobalId($type, $id);
             }),
 
-            new ExpressionFunction('fromGlobalId', function () {}, function (array $variables, $globalId) {
+            new ExpressionFunction('fromGlobalId', function() {}, function(array $variables, $globalId) {
                 return GlobalId::fromGlobalId($globalId);
             }),
 
-            new ExpressionFunction('newObject', function () {}, function (array $variables, $className, array $args = []) {
+            new ExpressionFunction('newObject', function() {}, function(array $variables, $className, array $args = []) {
                 return (new \ReflectionClass($className))->newInstanceArgs($args);
             }),
         ];

ExpressionLanguage/AuthorizationExpressionProvider.php

Spacing +8 added lines, -8 removed lines

@@ -19,11 +19,11 @@ 
     public function getFunctions()
     {
         return [
-            new ExpressionFunction('hasRole', function () {}, function (array $variables, $role) {
+            new ExpressionFunction('hasRole', function() {}, function(array $variables, $role) {
                 return $variables['container']->get('security.authorization_checker')->isGranted($role);
             }),
 
-            new ExpressionFunction('hasAnyRole', function () {}, function (array $variables, array $roles) {
+            new ExpressionFunction('hasAnyRole', function() {}, function(array $variables, array $roles) {
                 foreach ($roles as $role) {
                     if ($variables['container']->get('security.authorization_checker')->isGranted($role)) {
                         return true;
@@ -33,29 +33,29 @@ 
                 return false;
             }),
 
-            new ExpressionFunction('isAnonymous', function () {}, function (array $variables) {
+            new ExpressionFunction('isAnonymous', function() {}, function(array $variables) {
                 return $variables['container']->get('security.authorization_checker')->isGranted('IS_AUTHENTICATED_ANONYMOUSLY');
             }),
 
-            new ExpressionFunction('isRememberMe', function () {}, function (array $variables) {
+            new ExpressionFunction('isRememberMe', function() {}, function(array $variables) {
                 return $variables['container']->get('security.authorization_checker')->isGranted('IS_AUTHENTICATED_REMEMBERED');
             }),
 
-            new ExpressionFunction('isFullyAuthenticated', function () {}, function (array $variables) {
+            new ExpressionFunction('isFullyAuthenticated', function() {}, function(array $variables) {
                 return $variables['container']->get('security.authorization_checker')->isGranted('IS_AUTHENTICATED_FULLY');
             }),
 
-            new ExpressionFunction('isAuthenticated', function () {}, function (array $variables) {
+            new ExpressionFunction('isAuthenticated', function() {}, function(array $variables) {
                 return
                     $variables['container']->get('security.authorization_checker')->isGranted('IS_AUTHENTICATED_REMEMBERED')
                     || $variables['container']->get('security.authorization_checker')->isGranted('IS_AUTHENTICATED_FULLY');
             }),
 
-            new ExpressionFunction('hasPermission', function () {}, function (array $variables, $object, $permission) {
+            new ExpressionFunction('hasPermission', function() {}, function(array $variables, $object, $permission) {
                 return $variables['container']->get('security.authorization_checker')->isGranted($permission, $object);
             }),
 
-            new ExpressionFunction('hasAnyPermission', function () {}, function (array $variables, $object, array $permissions) {
+            new ExpressionFunction('hasAnyPermission', function() {}, function(array $variables, $object, array $permissions) {
                 foreach ($permissions as $permission) {
                     if ($variables['container']->get('security.authorization_checker')->isGranted($permission, $object)) {
                         return true;

Tests/Resolver/ResolverTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
     public function setUp()
     {
-        $this->closureProperty = function () {
+        $this->closureProperty = function() {
             return $this->privatePropertyWithoutGetter;
         };
     }

Resolver/Config/FieldsConfigSolution.php

Doc Comments +11 added lines

@@ -247,6 +247,9 @@ 
         return $type;
     }
 
+    /**
+     * @param callable $expression
+     */
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
         return function () use ($expression, $resolveCallback) {
@@ -277,6 +280,10 @@ 
                 case is_array($result) || $result instanceof \ArrayAccess:
                     $result = array_filter(
                         array_map(
+
+                            /**
+                             * @param callable $object
+                             */
                             function ($object) use ($checkAccess) {
                                 return $checkAccess($object) ? $object : null;
                             },
@@ -287,6 +294,10 @@ 
 
                 case $result instanceof Connection:
                     $result->edges = array_map(
+
+                        /**
+                         * @param callable $edge
+                         */
                         function (Edge $edge) use ($checkAccess) {
                             $edge->node = $checkAccess($edge->node) ? $edge->node : null;
 

Spacing +5 added lines, -5 removed lines

@@ -130,14 +130,14 @@ 
 
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
-        return function () use ($expression, $resolveCallback) {
+        return function() use ($expression, $resolveCallback) {
             $args = func_get_args();
 
-            $result = null !== $resolveCallback  ? call_user_func_array($resolveCallback, $args) : null;
+            $result = null !== $resolveCallback ? call_user_func_array($resolveCallback, $args) : null;
 
             $values = call_user_func_array([$this, 'solveResolveCallbackArgs'], $args);
 
-            $checkAccess = function ($object, $throwException = false) use ($expression, $values) {
+            $checkAccess = function($object, $throwException = false) use ($expression, $values) {
                 try {
                     $access = $this->solveUsingExpressionLanguageIfNeeded(
                         $expression,
@@ -158,7 +158,7 @@ 
                 case is_array($result) || $result instanceof \ArrayAccess:
                     $result = array_filter(
                         array_map(
-                            function ($object) use ($checkAccess) {
+                            function($object) use ($checkAccess) {
                                 return $checkAccess($object) ? $object : null;
                             },
                             $result
@@ -168,7 +168,7 @@ 
 
                 case $result instanceof Connection:
                     $result->edges = array_map(
-                        function (Edge $edge) use ($checkAccess) {
+                        function(Edge $edge) use ($checkAccess) {
                             $edge->node = $checkAccess($edge->node) ? $edge->node : null;
 
                             return $edge;

Resolver/Config/ResolveCallbackConfigSolution.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
             return $value;
         }
 
-        return function () use ($value) {
+        return function() use ($value) {
             $args = func_get_args();
             $result = $this->solveUsingExpressionLanguageIfNeeded(
                 $value,

Resolver/Config/AbstractConfigSolution.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 
     protected function solveUsingExpressionLanguageIfNeeded($expression, array $values = [])
     {
-        if (is_string($expression) &&  0 === strpos($expression, '@=')) {
+        if (is_string($expression) && 0 === strpos($expression, '@=')) {
             return $this->expressionLanguage->evaluate(substr($expression, 2), $values);
         }
 

Resolver/Config/TypeConfigSolution.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 
     public function solveTypeCallback($values)
     {
-        return function () use ($values) {
+        return function() use ($values) {
             return $this->solveType($values);
         };
     }

Tests/Resolver/Config/FieldsConfigSolutionTest.php

Spacing +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
     private function invokeResolveAccessAndWrapResolveCallback($hasAccess, callable $callback = null)
     {
         if (null === $callback) {
-            $callback = function ($value) {
+            $callback = function($value) {
                 return $value;
             };
         }

Tests/Definition/MergeFieldTraitTest.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
                 ['foo'],
             ],
             [
-                function () {
+                function() {
                     return ['test'];
                 },
                 ['bar'],
@@ -64,7 +64,7 @@ 
 
     public function testGetFieldsWithDefaultsForceArray()
     {
-        $fields = $this->getFieldsWithDefaults(function () { return ['bar']; }, ['toto'], false);
+        $fields = $this->getFieldsWithDefaults(function() { return ['bar']; }, ['toto'], false);
         $this->assertInstanceOf('Closure', $fields);
         $this->assertEquals(['bar', 'toto'], $fields());
     }