minor #1269 Update SecurityController.php (seb-jean)

javiereguiluz · javiereguiluz · commit 996255195806 · 2021-11-26T09:32:15.000+01:00
This PR was merged into the main branch. Discussion ---------- Update SecurityController.php I propose this PR to be in agreement with maker-bundle: https://github.com/symfony/maker-bundle/blob/main/src/Security/SecurityControllerBuilder.php#L38 Commits ------- 38e9adb Update SecurityController.php
diff --git a/src/Controller/SecurityController.php b/src/Controller/SecurityController.php
@@ -36,7 +36,7 @@ class SecurityController extends AbstractController
     public function login(Request $request, Security $security, AuthenticationUtils $helper): Response
     {
         // if user is already logged in, don't display the login page again
-        if ($security->isGranted('ROLE_USER')) {
+        if ($this->getUser()) {
             return $this->redirectToRoute('blog_index');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class SecurityController extends AbstractController`
`36`	`36`	`public function login(Request $request, Security $security, AuthenticationUtils $helper): Response`
`37`	`37`	`{`
`38`	`38`	`// if user is already logged in, don't display the login page again`
`39`		`- if ($security->isGranted('ROLE_USER')) {`
	`39`	`+ if ($this->getUser()) {`
`40`	`40`	`return $this->redirectToRoute('blog_index');`
`41`	`41`	`}`
`42`	`42`