Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
Mail edit is not usinf logintoboggans validation link
» Mail edit is not using logintoboggan's validation link
It would help if you could explain the issue for those who don't use logintoboggan, and if some others who do use logintoboggan could review this patch.
litwol needs to decide whether he wants to accept logintoboggan-specific special-casing into mail_edit. If it works, I'm in favor of doing it.
EDIT: Can anyone confirm that this works as intended?
This is needed if the user selects "Set Password" option in LoginToboggan. This option creates a non-authenticated role for the newly registered user and the user must click on the validation link in their welcome email to remove this non-authenticated role. LoginToboggan uses !login_url in the Welcome Email template to create this validation link.
If you use Mail Editor's template for your welcome email, it changes !login_url back to the password reset url.
We patched this successfully to the most recent dev version and it works perfectly!
Comments
Comment #1
salvisIt would help if you could explain the issue for those who don't use logintoboggan, and if some others who do use logintoboggan could review this patch.
litwol needs to decide whether he wants to accept logintoboggan-specific special-casing into mail_edit. If it works, I'm in favor of doing it.EDIT: Can anyone confirm that this works as intended?
Comment #2
Bilmar commentedThis is needed if the user selects "Set Password" option in LoginToboggan. This option creates a non-authenticated role for the newly registered user and the user must click on the validation link in their welcome email to remove this non-authenticated role. LoginToboggan uses !login_url in the Welcome Email template to create this validation link.
If you use Mail Editor's template for your welcome email, it changes !login_url back to the password reset url.
We patched this successfully to the most recent dev version and it works perfectly!
Thanks for the great work DrakeRemory
Comment #3
salvisIs the condition correct, i.e. can users still obtain their password reset url if requested?
Comment #4
Bilmar commentedyes, users are able to obtain the password reset url when requested.
please let me know if more information is needed. thanks!
Comment #5
salvisThanks.
Committed to the -dev version, give it up to 12h to be repackaged.