diff --git a/src/Builder/UserBuilder.php b/src/Builder/UserBuilder.php index e314af4f99eabb27860507ef3c851d31e18e4013..e457f777d8ee4d6ad98d50b84564f60eb56c7aae 100644 --- a/src/Builder/UserBuilder.php +++ b/src/Builder/UserBuilder.php @@ -105,8 +105,9 @@ class UserBuilder return $this; } - public function withNewsLetterSubscription(bool $newsLetterSubscription) + public function withNewsLetterSubscription(bool $newsLetterSubscription): UserBuilder { $this->user->setSubscribedToNewsLetter($newsLetterSubscription); + return $this; } } diff --git a/src/Controller/RegistrationController.php b/src/Controller/RegistrationController.php index 7c98102df2474d523b2dc21013ac3584ae705a18..ae8a72414bc2136933401e6cdd785acfbd720dc6 100644 --- a/src/Controller/RegistrationController.php +++ b/src/Controller/RegistrationController.php @@ -54,12 +54,11 @@ class RegistrationController extends AbstractController $form->handleRequest($request); if ($form->isSubmitted() && $form->isValid()) { - $user = $form->getData(); $userBuilder = new UserBuilder($userPasswordHasher, $user); // Ugly fix because I don't understand why those values aren't set correctly $userBuilder->withAcceptGeneralConditions($form->get('acceptGeneralConditions')->getData()); - $userBuilder->withNewsLetterSubscription($form->get('subscribedToNewsLetter')->getData() ?? false); + $userBuilder->withNewsLetterSubscription(false); $userBuilder ->withPassword( diff --git a/src/Entity/User.php b/src/Entity/User.php index e707f67d7cbe7cc65d6a3a5b862f1c3fa218ff24..1157a35cc77f036cea638825a64a330d620907b9 100644 --- a/src/Entity/User.php +++ b/src/Entity/User.php @@ -105,6 +105,7 @@ class User implements UserInterface, LegacyPasswordAuthenticatedUserInterface $this->acceptGeneralConditions = false; $this->isVerified = false; $this->credentialExpired = false; + $this->is_subscribed_news_letter = false; } public function getId(): int diff --git a/src/Form/RegistrationFormType.php b/src/Form/RegistrationFormType.php index 912919521defbf0d21e17154757051e2372897ed..403bfa89de8e5cffb84b3bde9fc7836c8cf00cf2 100644 --- a/src/Form/RegistrationFormType.php +++ b/src/Form/RegistrationFormType.php @@ -88,7 +88,6 @@ class RegistrationFormType extends AbstractType 'label' => 'registration.agreeTerms' ] ) - ->add('subscribedToNewsLetter', HiddenType::class) ->add( 'submit', SubmitType::class,