overblog / GraphQLBundle

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;

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

Resolver/Config/FieldsConfigSolution.php

Doc Comments +3 added lines

@@ -149,6 +149,9 @@
         return $treatedOptions;
     }
 
+    /**
+     * @param callable $expression
+     */
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
         return function () use ($expression, $resolveCallback) {

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
 
         $value = $treatedOptions['complexity'];
 
-        $treatedOptions['complexity'] = function () use ($value) {
+        $treatedOptions['complexity'] = function() use ($value) {
             $args = func_get_args();
             $complexity = $this->solveUsingExpressionLanguageIfNeeded(
                 $value,
@@ -179,10 +179,10 @@ 
 
     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);
 
@@ -198,7 +198,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
@@ -208,7 +208,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;
@@ -227,7 +227,7 @@ 
 
     private function checkAccessCallback($expression, $values)
     {
-        return function ($object, $throwException = false) use ($expression, $values) {
+        return function($object, $throwException = false) use ($expression, $values) {
             try {
                 $access = $this->solveUsingExpressionLanguageIfNeeded(
                     $expression,

Tests/Functional/Controller/GraphControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
         $client->request('POST', '/?batch', [], [], ['CONTENT_TYPE' => 'application/json'], json_encode($data));
         $result = $client->getResponse()->getContent();
 
-        $expected  = [
+        $expected = [
             'friends' => ['data' => $this->expectedData],
             'friendsTotalCount' => ['data' => ['user' => ['friends' => ['totalCount' => 4]]]],
         ];

Request/Validator/Rule/AbstractQuerySecurity.php

Doc Comments +1 added lines

@@ -41,6 +41,7 @@
      * check if equal to 0 no check is done. Must be greater or equal to 0.
      *
      * @param $value
+     * @param string $name
      */
     protected static function checkIfGreaterOrEqualToZero($name, $value)
     {