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.
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
Comment #1
pillarsdotnet commentedRelated links:
If other modules implement MailSystemInterface or set the mail_system variable, I haven't found them yet.
Comment #2
pillarsdotnet commentedComment #3
scor commentedYou need a patch to have an issue "needs review"
Comment #4
pillarsdotnet commentedI'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.
Comment #5
pillarsdotnet commentedPer 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.
Comment #6
pillarsdotnet commentedLooks like I should be able to work with the Variable module.
Comment #7
pillarsdotnet commentedAlso see #1135262: drupal_mail() should support using different MailSystemInterface classes for format() and mail()
Comment #8
pillarsdotnet commentedCurrently blocked by:
Comment #9
oadaeh commentedMoving to 9.x.
Comment #10
catchComment #24
smustgrave commentedSince there hasn't been an update since being moved into PNMI 8 years ago
Going to move to outdated
If you feel it's still an issue please reopen with an updated issue summary.