If an anonymous user uses the 2.4 form, in which the %username and %useremail are used, the token values are visible instead of spaces.

CommentFileSizeAuthor
#2 webform_user_replacements.patch976 bytesquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cerebral’s picture

cerebral CreditAttribution: cerebral commented
Version: 5.x-2.4 » 6.x-2.4

Facing the same problem with version 6.x-2.4

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Active » Needs review
FileSize
webform_user_replacements.patch976 bytes

Seems to be caused by #337578: 3 x undefined variable/index errors + coder style cleanup. You can correct it with the attached patch. I'll wait a week before releasing another version to shake out any other bugs that pop up in the 2.4 version.

crac’s picture

crac CreditAttribution: crac commented

btw. the version of drupal-5 seems also to be affected by this issue..

chriscohen’s picture

chriscohen CreditAttribution: chriscohen commented

I have applied the patch to webform-6.x-2.4 but this does not appear to fix the problem.

cerebral’s picture

cerebral CreditAttribution: cerebral commented

Yes it originally was targeted to version 5 but mistakenly changed it to 6 thinking it would add to 5.. any way to target several versions in here ?

joewang’s picture

joewang CreditAttribution: joewang commented

The patch seems logical--why doesn't it work?

LUTi’s picture

LUTi CreditAttribution: LUTi commented

For me it seems to work just fine.

Try to clear your browser cache and Drupal cache, and then reload the webform...

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Title: %username and %useremail not removed if anonymous user » %username and %useremail not removed for anonymous users
Status: Needs review » Fixed

I figured this patch worked fine. Thanks for the confirmations. I've committed it to both D5 and D6.

Status: Fixed » Closed (fixed)

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

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Closed (fixed) » Fixed

Keeping in the queue until 2.5 is out.

Status: Fixed » Closed (fixed)

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