Skip to content
Snippets Groups Projects
Commit cdec4b1b authored by Camille Simiand's avatar Camille Simiand
Browse files

Route email confirmation

parent 568dc761
No related branches found
No related tags found
1 merge request!69tuleap-142-when-editing-email-address-user-should-confirm-it-with-email-validation
......@@ -70,6 +70,13 @@ class UserController extends AbstractController
if ($form->isSubmitted() && $form->isValid()) {
if ($current_user->getEmail() !== $form->get('email')->getData()) {
$last_pending_email_address = $this->pending_email_address_repository->findOneBy(
['user' => $current_user->getId()]
);
if ($last_pending_email_address !== null) {
$this->entity_manager->remove($last_pending_email_address);
}
$pending_email_address = new PendingEmailAddress();
$pending_email_address->setEmail($form->get('email')->getData());
$pending_email_address->setUser($current_user);
......@@ -146,7 +153,7 @@ class UserController extends AbstractController
]);
}
#[Route('/edit/email/', name:'verify_new_email_address')]
#[Route('/edit/email', name:'verify_new_email_address')]
public function verifyNewEmailAddress(): Response
{
$current_user = $this->getUser();
......@@ -157,7 +164,7 @@ class UserController extends AbstractController
$pending_email_address = $this->pending_email_address_repository->findOneBy(['user' => $current_user->getId()]);
if (! $pending_email_address instanceof PendingEmailAddress) {
throw new \Exception('Not found');
throw new \Exception('Pending email address not found');
}
$current_user->setEmail($pending_email_address->getEmail());
......
......@@ -23,5 +23,4 @@
{{ form_row(editUserProfileForm.update, {'row_attr': {'class' : 'form-group d-flex flex-column m-auto mb-4 col-2'}}) }}
{{ form_end(editUserProfileForm) }}
</div>
{% endblock %}
\ No newline at end of file
......@@ -5,7 +5,7 @@
<p>
{{ 'user.edit.email.text'|trans }}:
<br><br>
<a href="{{ path('verify_new_email_address') }}">
<a href="{{ url('verify_new_email_address') }}">
{{ 'user.edit.email.confirm_email'|trans }}
</a>
{{ 'general.link_expire'|trans
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment