[Security] removed useless else condition in SwitchUserListener class.

Hugo Hamon · Hugo Hamon · commit fc28cae8ad73 · 2015-07-22T15:05:05.000+02:00
diff --git a/Firewall/SwitchUserListener.php b/Firewall/SwitchUserListener.php
@@ -116,9 +116,9 @@ private function attemptSwitchUser(Request $request)
         if (false !== $originalToken) {
             if ($token->getUsername() === $request->get($this->usernameParameter)) {
                 return $token;
-            } else {
-                throw new \LogicException(sprintf('You are already switched to "%s" user.', $token->getUsername()));
             }
+
+            throw new \LogicException(sprintf('You are already switched to "%s" user.', $token->getUsername()));
         }
 
         if (false === $this->accessDecisionManager->decide($token, array($this->role))) {

Original file line number	Diff line number	Diff line change
`@@ -116,9 +116,9 @@ private function attemptSwitchUser(Request $request)`
`116`	`116`	`if (false !== $originalToken) {`
`117`	`117`	`if ($token->getUsername() === $request->get($this->usernameParameter)) {`
`118`	`118`	`return $token;`
`119`		`- } else {`
`120`		`- throw new \LogicException(sprintf('You are already switched to "%s" user.', $token->getUsername()));`
`121`	`119`	`}`
	`120`	`+`
	`121`	`+ throw new \LogicException(sprintf('You are already switched to "%s" user.', $token->getUsername()));`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`if (false === $this->accessDecisionManager->decide($token, array($this->role))) {`