CS

Author: aschempp

Authorization/AccessDecisionManager.php

@@ -86,17 +86,13 @@ private function decideAffirmative(TokenInterface $token, array $attributes, $ob
         $deny = 0;
         foreach ($this->voters as $voter) {
             $result = $this->vote($voter, $token, $object, $attributes);
-            switch ($result) {
-                case VoterInterface::ACCESS_GRANTED:
-                    return true;
 
-                case VoterInterface::ACCESS_DENIED:
-                    ++$deny;
-
-                    break;
+            if (VoterInterface::ACCESS_GRANTED === $result) {
+                return true;
+            }
 
-                default:
-                    break;
+            if (VoterInterface::ACCESS_DENIED === $result) {
+                ++$deny;
             }
         }
 
@@ -128,16 +124,10 @@ private function decideConsensus(TokenInterface $token, array $attributes, $obje
         foreach ($this->voters as $voter) {
             $result = $this->vote($voter, $token, $object, $attributes);
 
-            switch ($result) {
-                case VoterInterface::ACCESS_GRANTED:
-                    ++$grant;
-
-                    break;
-
-                case VoterInterface::ACCESS_DENIED:
-                    ++$deny;
-
-                    break;
+            if (VoterInterface::ACCESS_GRANTED === $result) {
+                ++$grant;
+            } elseif (VoterInterface::ACCESS_DENIED === $result) {
+                ++$deny;
             }
         }
 
@@ -169,17 +159,12 @@ private function decideUnanimous(TokenInterface $token, array $attributes, $obje
             foreach ($attributes as $attribute) {
                 $result = $this->vote($voter, $token, $object, [$attribute]);
 
-                switch ($result) {
-                    case VoterInterface::ACCESS_GRANTED:
-                        ++$grant;
-
-                        break;
-
-                    case VoterInterface::ACCESS_DENIED:
-                        return false;
+                if (VoterInterface::ACCESS_DENIED === $result) {
+                    return false;
+                }
 
-                    default:
-                        break;
+                if (VoterInterface::ACCESS_GRANTED === $result) {
+                    ++$grant;
                 }
             }
         }