diff --git a/src/Oro/Bundle/CustomerBundle/Controller/Frontend/ResetController.php b/src/Oro/Bundle/CustomerBundle/Controller/Frontend/ResetController.php index 45adbf1a1..afc5da1d4 100644 --- a/src/Oro/Bundle/CustomerBundle/Controller/Frontend/ResetController.php +++ b/src/Oro/Bundle/CustomerBundle/Controller/Frontend/ResetController.php @@ -28,13 +28,11 @@ class ResetController extends Controller * @Layout() * @Route("/reset-request", name="oro_customer_frontend_customer_user_reset_request") * @Method({"GET", "POST"}) - * @param Request $request - * @return array|RedirectResponse */ - public function requestAction(Request $request) + public function requestAction() { if ($this->getUser()) { - return $this->redirectToRoute('oro_customer_frontend_customer_user_profile'); + return $this->redirect($this->generateUrl('oro_customer_frontend_customer_user_profile')); } /** @var CustomerUserPasswordRequestHandler $handler */ @@ -42,10 +40,11 @@ public function requestAction(Request $request) $form = $this->get('oro_customer.provider.frontend_customer_user_form') ->getForgotPasswordForm(); + $request = $this->get('request_stack')->getCurrentRequest(); $email = $handler->process($form, $request); if ($email) { $this->get('session')->set(static::SESSION_EMAIL, $this->getObfuscatedEmail($email)); - return $this->redirectToRoute('oro_customer_frontend_customer_user_reset_check_email'); + return $this->redirect($this->generateUrl('oro_customer_frontend_customer_user_reset_check_email')); } return []; @@ -57,10 +56,8 @@ public function requestAction(Request $request) * @Layout() * @Route("/check-email", name="oro_customer_frontend_customer_user_reset_check_email") * @Method({"GET"}) - * @param Request $request - * @return array|RedirectResponse */ - public function checkEmailAction(Request $request) + public function checkEmailAction() { $session = $this->get('session'); $email = $session->get(static::SESSION_EMAIL); @@ -68,7 +65,7 @@ public function checkEmailAction(Request $request) if (empty($email)) { // the user does not come from the sendEmail action - return $this->redirectToRoute('oro_customer_frontend_customer_user_reset_request'); + return $this->redirect($this->generateUrl('oro_customer_frontend_customer_user_reset_request')); } return [ @@ -103,14 +100,14 @@ public function resetAction(Request $request) } $session = $this->get('session'); - $ttl = $this->getParameter('oro_user.reset.ttl'); + $ttl = $this->container->getParameter('oro_user.reset.ttl'); if (!$user->isPasswordRequestNonExpired($ttl)) { - $this->addFlash( + $session->getFlashBag()->add( 'warn', 'oro.customer.customeruser.profile.password.reset.ttl_expired.message' ); - return $this->redirectToRoute('oro_customer_frontend_customer_user_reset_request'); + return $this->redirect($this->generateUrl('oro_customer_frontend_customer_user_reset_request')); } /** @var CustomerUserPasswordResetHandler $handler */ @@ -124,7 +121,7 @@ public function resetAction(Request $request) $session->invalidate(); $this->get('security.token_storage')->setToken(null); - $this->addFlash( + $session->getFlashBag()->add( 'success', 'oro.customer.customeruser.profile.password_reset.message' ); @@ -132,7 +129,7 @@ public function resetAction(Request $request) if ($actionParameter) { $response = $this->get('oro_ui.router')->redirect($user); } else { - $response = $this->redirectToRoute('oro_customer_customer_user_security_login'); + $response = $this->redirect($this->generateUrl('oro_customer_customer_user_security_login')); } return $response;