The last version 6.x-2.3 doesn't resolved issue #1287866: "Don't attach PDF file" setting doesn't work with email addresses taken from the webform

"Don't attach PDF file" option doesn't work with an email field from the webform or more options checked.

foreach ($no_mail as $eid) {
...
  $email_addresses = $submission->data[$eid]['value'];
...
}

$eid doesn't match ID of the data submission.

This patch resolved this issue, can you try it ?

CommentFileSizeAuthor
webform2pdf-issue-1287866.patch727 bytesBBaptiste

Comments

shafter’s picture

Why do you think $eid isn't valid?

BBaptiste’s picture

If "Don't attach PDF file" option was enabled on email field from the webform, $eid doesn't match to $submission->data[].

It must make the link between the $eid and id field.

joseamayo’s picture

I am experiencing this issue, as well and am hopeful that someone can help. Below is a summary:
I send automated emails from Webform 3.14 to a combination of hardcoded email addresses and user-entered email addresses.

  • Disabling all email addresses for PDF attachment works as expected.
  • Disabling all hardcoded email addresses for PDF attachment works as expected.
  • Disabling a user-generated email addresses for PDF attachment prevents the attachment of the PDF to all email addresses (hardcoded or user-entered). This does not work as expected. Attempted with base 6.x-2.3 and with patch installed with no change.

I have tried all of the above with multiple forms, using Webform 3.14-3.18, clearing caches each time, with all combinations of the standard default email and with customized email templates, to no avail. I get no error of any kind. Any insight would be greatly appreciated.

mr.york’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)

#3 works for me.

uldo’s picture

in 7.x-3.2 the issue ist still present; patch webform2pdf-issue-1287866.patch works as expected;

joe huggans’s picture

Patch doesn't work in 7.x-4.0

ahmed_samir’s picture