Posted by pillarsdotnet on March 11, 2011 at 11:52pm
2 followers
Jump to:
| Project: | Drupal core |
| Version: | 8.x-dev |
| Component: | mail system |
| Category: | feature request |
| Priority: | normal |
| Assigned: | pillarsdotnet |
| Status: | needs work |
Issue Summary
The Mail System module provides an Administrative UI and Developers API for safely setting the mail_system variable.
My hope is that this functionality eventually gets included into core.
Comments
#1
Related links:
If other modules implement MailSystemInterface or set the mail_system variable, I haven't found them yet.
#2
#3
You need a patch to have an issue "needs review"
#4
I'm hoping for feedback on whether this is the correct approach before I offer a patch to core. The module works as-is and could be added to core simply by copying the files from sites/all/modules/ to modules/ or by cut-and-pasting the code into system.module, by creating a new mail.module.
I am requesting a review of the code and a suggestion as to what direction I should take it in order to be considered for inclusion into core.
#5
Per dww an issue *can* be marked "needs review" if it has a module implementation.
Finally found a core developer on IRC. (thanks, chx!)
Recommendation is to provide a generalized UI/API for setting *any* configuration variable, with a hook override for theming a particular variable or group of variables and/or breaking them out into a separate tab.
So... setting this to "Needs Work" until I get that done.
#6
Looks like I should be able to work with the Variable module.
#7
Also see #1135262: drupal_mail() should support using different MailSystemInterface classes for format() and mail()
#8
Currently blocked by: