bug #1473 Fix return type of change password action (HeahDude)

javiereguiluz · javiereguiluz · commit 83dbfc94df24 · 2023-12-11T13:43:16.000+01:00
This PR was merged into the main branch. Discussion ---------- Fix return type of change password action Commits ------- d68b18c Fix return type of change password action
diff --git a/src/Controller/UserController.php b/src/Controller/UserController.php
@@ -72,7 +72,7 @@ public function changePassword(
 
             // The logout method applies an automatic protection against CSRF attacks;
             // it's explicitly disabled here because the form already has a CSRF token validated.
-            return $security->logout(validateCsrfToken: false);
+            return $security->logout(validateCsrfToken: false) ?? $this->redirectToRoute('homepage');
         }
 
         return $this->render('user/change_password.html.twig', [

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ public function changePassword(`
`72`	`72`
`73`	`73`	`// The logout method applies an automatic protection against CSRF attacks;`
`74`	`74`	`// it's explicitly disabled here because the form already has a CSRF token validated.`
`75`		`- return $security->logout(validateCsrfToken: false);`
	`75`	`+ return $security->logout(validateCsrfToken: false) ?? $this->redirectToRoute('homepage');`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`return $this->render('user/change_password.html.twig', [`