Skip to content
Snippets Groups Projects
Verified Commit 91d9c8c5 authored by Sebastien's avatar Sebastien Committed by David Beniamine
Browse files

Fix unit tests

parent 3256299f
No related branches found
No related tags found
1 merge request!46Tuleap 88 db migration
...@@ -105,8 +105,9 @@ class UserBuilder ...@@ -105,8 +105,9 @@ class UserBuilder
return $this; return $this;
} }
public function withNewsLetterSubscription(bool $newsLetterSubscription) public function withNewsLetterSubscription(bool $newsLetterSubscription): UserBuilder
{ {
$this->user->setSubscribedToNewsLetter($newsLetterSubscription); $this->user->setSubscribedToNewsLetter($newsLetterSubscription);
return $this;
} }
} }
...@@ -54,12 +54,11 @@ class RegistrationController extends AbstractController ...@@ -54,12 +54,11 @@ class RegistrationController extends AbstractController
$form->handleRequest($request); $form->handleRequest($request);
if ($form->isSubmitted() && $form->isValid()) { if ($form->isSubmitted() && $form->isValid()) {
$user = $form->getData();
$userBuilder = new UserBuilder($userPasswordHasher, $user); $userBuilder = new UserBuilder($userPasswordHasher, $user);
// Ugly fix because I don't understand why those values aren't set correctly // Ugly fix because I don't understand why those values aren't set correctly
$userBuilder->withAcceptGeneralConditions($form->get('acceptGeneralConditions')->getData()); $userBuilder->withAcceptGeneralConditions($form->get('acceptGeneralConditions')->getData());
$userBuilder->withNewsLetterSubscription($form->get('subscribedToNewsLetter')->getData() ?? false); $userBuilder->withNewsLetterSubscription(false);
$userBuilder $userBuilder
->withPassword( ->withPassword(
......
...@@ -105,6 +105,7 @@ class User implements UserInterface, LegacyPasswordAuthenticatedUserInterface ...@@ -105,6 +105,7 @@ class User implements UserInterface, LegacyPasswordAuthenticatedUserInterface
$this->acceptGeneralConditions = false; $this->acceptGeneralConditions = false;
$this->isVerified = false; $this->isVerified = false;
$this->credentialExpired = false; $this->credentialExpired = false;
$this->is_subscribed_news_letter = false;
} }
public function getId(): int public function getId(): int
......
...@@ -88,7 +88,6 @@ class RegistrationFormType extends AbstractType ...@@ -88,7 +88,6 @@ class RegistrationFormType extends AbstractType
'label' => 'registration.agreeTerms' 'label' => 'registration.agreeTerms'
] ]
) )
->add('subscribedToNewsLetter', HiddenType::class)
->add( ->add(
'submit', 'submit',
SubmitType::class, SubmitType::class,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment