Closed (outdated)
Project:
Drupal core
Version:
9.5.x-dev
Component:
mail system
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
11 Mar 2011 at 23:52 UTC
Updated:
5 Aug 2022 at 16:53 UTC
Jump to comment: Most recent
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.