Whenever you change the value of the "Field key" value, there is a problem in the generation of the email: the field merely does not appear. This has to do with the implementation of the theme_webform_mail_fields function in webform.inc. The reason seems to be that the $node->webformcomponents is indexed by the CIDs (the 113124343-like numbers) instead of the form_key.

I have created a patch that looks for the cid instead of the form_key. It solves the issue.

CommentFileSizeAuthor
webform.inc_.form_key-mail.patch.txt579 bytestatien
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

philippejadin’s picture

philippejadin CreditAttribution: philippejadin commented

I confirm the patch works as advertised.

budda’s picture

budda
Primary language English
Location Cheshire
CreditAttribution: budda commented
Status: Needs review » Reviewed & tested by the community

Yeah this made my form elements appear in the emails now. Good stuff.

budda’s picture

budda
Primary language English
Location Cheshire
CreditAttribution: budda commented
Status: Reviewed & tested by the community » Needs work

Ah the selects are not working.

I have a list of checkboxes, and i select a couple of the options then submit the form. On the thankyou page i get:

warning: Illegal offset type in /var/www/html/br/marketing/sites/all/modules/webform/components/select.inc on line 258.

The email also fails to contain any of the checkbox values i selected.

quicksketch’s picture

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

This has been corrected in both the 1.9 and 2.x versions of the module.