Closed (fixed)
Project:
Webform
Version:
7.x-4.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Dec 2012 at 15:14 UTC
Updated:
30 Apr 2015 at 20:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
stevengraff commentedI need this, but... not much demand for this function among the greater drupal community I suppose...
Comment #2
quicksketchHi there, sorry sometimes it takes a *really* long time for me to get through all the issues in the Webform queue. Some slip in the cracks like this one. I've moved this to a feature request, as it should be possible for Webform to support this functionality. I think it's just not supported because not many people are aware of the longer format (at least site administrators). Considering this is the first request I've ever seen for this functionality, it doesn't seem like it's a highly demanded feature. Nonetheless I think we should keep this open as a potential feature to add. Seems like it wouldn't be too difficult to support, we'd just need to build a more elaborate validation function most likely.
Comment #3
stevengraff commentedThanks for considering this. I was hoping it might be simple, since there is already the provision to this on the "Email from" side of the house, i.e. fields for "E-mail from address" and "E-mail from name" in the E-mail header details section under Edit E-mail Settings.
Comment #4
danchadwick commentedSeems reasonable. Anyone want to make a patch?
Comment #5
danchadwick commentedI spent a few days straightening out the e-mail code, which had gotten rather messy over the years. This is a big patch. My strategy was to create some powerful, flexible utilities and then just call them from the various places where e-mail parsing, validating, and formatting is needed.
webform.module
"Some Name" <some_name@example.com". It returns FALSE if an address is invalid, or the number of valid addresses.webform.email.inc
webform.pages.inc
webform.submissions.inc
As a consequence of this, multiple TO addresses may be used in the e-mail configuration, and they may be long format. Note that this issue does not address the contents of an e-mail component. There is a distinct issue for that.
Committed to 7.x-4.x and (amazingly) 8.x.
Comment #7
danchadwick commentedAn additional change, probably a regression. When the To address is empty, don't attempt to send the message.
Committed to 7.x-4.x and conflict-resolved to 8.x.