Problem/Motivation

The webform module sends email with only the handler id as the ID. This cause a sent email's unique id being just webform_email_HANDLER_ID. This is not a unique enough id and should be webform_WEBFORM_ID_HANDLE_ID.

Proposed resolution

Change sent mail id from webform_email_HANDLER_ID to webform_WEBFORM_ID_HANDLE_ID

Remaining tasks

  • Update mail
  • Update tests
  • Write change record
CommentFileSizeAuthor
#7 2940422-7.patch2.15 KBjrockowitz
#5 2940422-5.patch1.43 KBjrockowitz
#3 2940422-3.patch3.27 KBjrockowitz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Issue summary: View changes
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Active » Needs review
FileSize
2940422-3.patch3.27 KB

Status: Needs review » Needs work

The last submitted patch, 3: 2940422-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs work » Needs review
FileSize
2940422-5.patch1.43 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2940422-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs work » Needs review
FileSize
2940422-7.patch2.15 KB

  • jrockowitz committed afa8f4e on 8.x-5.x 
    Issue #2940422 by jrockowitz: Webform email ids are not specific enough
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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