Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
"The entered e-mail address "[current-user:mail]" does not appear valid."
(See image)
This appears to have worked in 4.6 or 4.7 (not sure when it changed exactly, but it was working a couple months ago and we keep our modules pretty up-to-date).
Comment | File | Size | Author |
---|---|---|---|
#2 | webform-email_to_from_tokens-2494103-2.patch | 5.25 KB | DanChadwick |
webform_email_token_not_working.PNG | 39.35 KB | mausolos |
Comments
Comment #1
DanChadwick CreditAttribution: DanChadwick commentedConfirmed. This is a regression from #1873498: Email To does not permit recipient name.
Actually, this didn't really work properly before either. During validation, the tokens are replaced by the node's tokens. But it should be possible to use submission tokens too, such as
[submission:values:email1], [submission:values:email2]
. It isn't possible to know at e-mail definition time whether the tokens will be valid later.Comment #2
DanChadwick CreditAttribution: DanChadwick commentedThis patch introduced a function signature change on two new functions introduced in 4.8. Since this version is recent, and since I don't think these two functions are likely to be used by other modules, I decided to make the signatures make more sense rather than be backward compatible. The argument $allow_tokens was added to:
Committed to 7.x-4.x and 8.x.
Comment #4
fenstratThis didn't factor in the token API changes in D8 as per #2493805: Convert token API calls to use token service. Fixing now.
Comment #6
fenstratCommitted and pushed to 8.x-4.x.
Appears this didn't actually get applied to 7.x-4.x, will commit #2 now.
Comment #7
fenstratHmm my bad, the commit to 7.x-4.x. doesn't show here in this issue nor in the d.o commit log https://www.drupal.org/node/7404/commits - bad d.o - however it was committed http://cgit.drupalcode.org/webform/commit/?id=835e990 - back to fixed.
Comment #8
mausolos CreditAttribution: mausolos commentedThanks for the quick response, guys! I'll check it out this week.
Comment #9
DanChadwick CreditAttribution: DanChadwick commented@fenstrat -- d.o has started to have issues when more than one branch is pushed at the same time. I've reported it, but there isn't much progress yet.
Comment #10
fenstrat@DanChadwick right, any chance you could give me a link to that issue so I can chime in / follow?
Comment #11
DanChadwick CreditAttribution: DanChadwick commentedRe d.o commit message problems:
#2468631: Commit comment messages suddenly stopped for 1 of 2 branches
Also: #2286097: Commit messages comment cron OOM, comment 9805783 at the bottom.