diff --git a/src/Controller/RegistrationController.php b/src/Controller/RegistrationController.php index ca23364..5b7364c 100644 --- a/src/Controller/RegistrationController.php +++ b/src/Controller/RegistrationController.php @@ -4,8 +4,8 @@ namespace App\Controller; use App\Entity\User; use App\Form\RegistrationFormType; -use App\Security\EmailVerifier; use App\Security\AppAuthenticator; +use App\Security\EmailVerifier; use App\Repository\UserRepository; use Symfony\Bridge\Twig\Mime\TemplatedEmail; use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; @@ -14,7 +14,6 @@ use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Mime\Address; use Symfony\Component\Routing\Annotation\Route; use Symfony\Component\PasswordHasher\Hasher\UserPasswordHasherInterface; -use Symfony\Component\Security\Guard\GuardAuthenticatorHandler; use SymfonyCasts\Bundle\VerifyEmail\Exception\VerifyEmailExceptionInterface; class RegistrationController extends AbstractController @@ -27,7 +26,7 @@ class RegistrationController extends AbstractController } #[Route('/register', name: 'app_register')] - public function register(Request $request, UserPasswordHasherInterface $passwordHasher, GuardAuthenticatorHandler $guardHandler, AppAuthenticator $authenticator): Response + public function register(Request $request, UserPasswordHasherInterface $passwordEncoder): Response { $user = new User(); $form = $this->createForm(RegistrationFormType::class, $user); @@ -36,7 +35,7 @@ class RegistrationController extends AbstractController if ($form->isSubmitted() && $form->isValid()) { // encode the plain password $user->setPassword( - $passwordHasher->encodePassword( + $passwordEncoder->encodePassword( $user, $form->get('plainPassword')->getData() ) @@ -54,14 +53,8 @@ class RegistrationController extends AbstractController ->subject('Please Confirm your Email') ->htmlTemplate('registration/confirmation_email.html.twig') ); - // do anything else you need here, like send an email - return $guardHandler->authenticateUserAndHandleSuccess( - $user, - $request, - $authenticator, - 'main' // firewall name in security.yaml - ); + return $this->redirectToRoute('security'); } return $this->render('registration/register.html.twig', [ @@ -70,23 +63,13 @@ class RegistrationController extends AbstractController } #[Route('/verify/email', name: 'app_verify_email')] - public function verifyUserEmail(Request $request, UserRepository $userRepository): Response + public function verifyUserEmail(Request $request): Response { - $id = $request->get('id'); - - if (null === $id) { - return $this->redirectToRoute('app_register'); - } - - $user = $userRepository->find($id); - - if (null === $user) { - return $this->redirectToRoute('app_register'); - } + $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY'); // validate email confirmation link, sets User::isVerified=true and persists try { - $this->emailVerifier->handleEmailConfirmation($request, $user); + $this->emailVerifier->handleEmailConfirmation($request, $this->getUser()); } catch (VerifyEmailExceptionInterface $exception) { $this->addFlash('verify_email_error', $exception->getReason()); @@ -96,6 +79,6 @@ class RegistrationController extends AbstractController // @TODO Change the redirect on success and handle or remove the flash message in your templates $this->addFlash('success', 'Your email address has been verified.'); - return $this->redirectToRoute('user_page'); + return $this->redirectToRoute('app_register'); } }