After upgrading from 8.x.5.0.0-beta15 to 8.x.5.0.0-beta16 our email templates broke down. The [webform_submission] placeholder was replaced by empty values, instead of the submission content.
This bug was introduced after commit 224e93c "Issue #2898158 by jrockowitz, amairesse:

tags are stripped from email since 8.x-5.0-beta13" on 29-07-2017.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

PtrTon created an issue. See original summary.

jrockowitz’s picture

Priority: Normal » Major
Status: Active » Needs review

Thanks for the patch.

Changing this ticket's status to 'Needs review' to trigger the automated tested.

I think the attached patch might have issues dealing with nested elements but I am not able to verify this right now. (I am on vacation)

jeroen.b’s picture

I think tests only work when patch is in a comment.

Status: Needs review » Needs work

The last submitted patch, 3: webform_submission_values.patch, failed testing. View results

jeroen.b’s picture

Status: Needs work » Needs review
FileSize
863 bytes

I think the format of the patch was wrong. Here's an updated version.

jrockowitz’s picture

I can't replicate the issue using the attached webform. Can someone please upload an example of a broken webform that uses the [webform_submission:values] token.

I suspect the issue is related to nested elements and/or multistep forms.

This is a major issue so we will definitely need to include some test coverage to prevent any additional regressions.

jrockowitz’s picture

Status: Needs review » Postponed (maintainer needs more info)
nachosalvador’s picture

I experienced a similar behaviour after upgrading from 8.x.5.0.0-beta15 to 8.x.5.0.0-beta16.

In my case I use a [webform_submission:values] token in the confirmation message. This value is calculated/setted in the submitForm() and before beta16 works correctly, right now this token value comes empty.

At the moment I solved it moving to version 8.x-5.x-dev :)

jrockowitz’s picture

I can't replicate the [webform_submission:values] issue in the confirmation. Please test the attached webform.

jrockowitz’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)