The email sent out to verify the email address does not allow to use common tokens like [site:name] or [user:name], as it is possible with the other user emails.

The attached patch fixes the missing token replacements.

CommentFileSizeAuthor
#3 token_fix-2650136-3.patch2.09 KBmerilainen
user_email_verification_token_replacement.patch986 bytesmh86

Comments

mh86 created an issue. See original summary.

merilainen’s picture

merilainen commented
Status: Needs review » Needs work

This doesn't work, because the user_email_verification_url() function is never called, so there is no way that the link will ever appear in the mail which is sent to the user.

merilainen’s picture

merilainen commented
Status: Needs work » Needs review
StatusFileSize
new token_fix-2650136-3.patch2.09 KB

It was actually missing the proper generation of the token, let's see if this works.

tijsdeboeck’s picture

tijsdeboeck
Location Antwerp 🇧🇪 🇪🇺 🌎
commented
Status: Needs review » Reviewed & tested by the community

Tested the patch and it works!

  • skek committed 766c835 on 7.x-1.x authored by mErilainen 
    Issue #2650136 by mh86, mErilainen, tijsdeboeck: Fix missing token...
skek’s picture

skek commented
Status: Reviewed & tested by the community » Fixed

Thank you for testing this.
The issue has been fixed.

Status: Fixed » Closed (fixed)

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