Fix error clientMutationId is removed when equal to 0 and access is…

… used...
overblog · Apr 7, 2016 · 2e2b860 · 2e2b860
1 parent 8659ab8
commit 2e2b860
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/Resolver/Config/FieldsConfigSolution.php b/Resolver/Config/FieldsConfigSolution.php
@@ -195,14 +195,12 @@ private function filterResultUsingAccess($result, $expression, $values)
         $checkAccess = $this->checkAccessCallback($expression, $values);
 
         switch (true) {
-            case is_array($result) || $result instanceof \ArrayAccess:
-                $result = array_filter(
-                    array_map(
-                        function ($object) use ($checkAccess) {
-                            return $checkAccess($object) ? $object : null;
-                        },
-                        $result
-                    )
+            case is_array($result):
+                $result = array_map(
+                    function ($object) use ($checkAccess) {
+                        return $checkAccess($object) ? $object : null;
+                    },
+                    $result
                 );
                 break;
 

diff --git a/Tests/Functional/Security/AccessTest.php b/Tests/Functional/Security/AccessTest.php
@@ -136,7 +136,7 @@ private function expectedFailedUserRoles()
         return [
             'data' => [
                 'user' => [
-                    'roles' => [],
+                    'roles' => [0 => null],
                 ],
             ],
         ];

diff --git a/Tests/Resolver/Config/FieldsConfigSolutionTest.php b/Tests/Resolver/Config/FieldsConfigSolutionTest.php
@@ -61,6 +61,8 @@ public function testResolveAccessAndWrapResolveCallbackWithArrayAndAccessDeniedT
                 'tata',
                 'titi',
                 'tata',
+                null,
+                null,
             ],
             $callback(
                 [