I fixed this issue.

CommentFileSizeAuthor
mail_edit_logintoboggan.inc_.patch657 bytesDrakeRemory

Comments

salvis’s picture

Title: 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?

Bilmar’s picture

Status: Needs review » Reviewed & tested by the community

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!

Thanks for the great work DrakeRemory

salvis’s picture

Is the condition correct, i.e. can users still obtain their password reset url if requested?

Bilmar’s picture

yes, users are able to obtain the password reset url when requested.
please let me know if more information is needed. thanks!

salvis’s picture

Status: Reviewed & tested by the community » Fixed

Thanks.

Committed to the -dev version, give it up to 12h to be repackaged.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.