Needs review
Project:
Action email role
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Feb 2008 at 15:11 UTC
Updated:
6 Mar 2009 at 02:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
eliza411 commentedThat would be a useful option, indeed. Toward the same end, I think, would be a variable to allow you to include the names of all the people who received the e-mail in the body or from: field of the e-mail.
Comment #2
dgtlmoon commentedeliza, it sounds like your comment and this post are different issues?
Comment #3
eliza411 commentedYes, I was making a related note (making variables available to work with) but it's not the same issue at all.
Comment #4
jeff.ferrari commentedI thought the 2 ideas above were pretty cool, and I actually had a use for them. Attached are the patches for the respective features. They were pretty rapidly kludged together, so feel free to input on a better way of doing them.
condense_recipients.patch adds a configuration checkbox. Instead of multiple calls to drupal_mail() , it just gathers all the emails up then calls it once with all emails in recipient field.
change_sender.patch also adds a configuration checkbox. When checked, it grabs the global $user object and uses the email address found there. I tested with workflow.module, but prob will work with any trigger that has a $user global set (not cron).
Both have only undergone rudimentary testing, ymmv ;)
Comment #5
jeff.ferrari commentedHmm... can't get other patch to upload. Doesn't belong here anyway ;)
http://drupal.org/node/267921
Comment #6
davidneedham+1 this is also an awesome idea and would be very helpful. Can this get into the next version of the module please?
Comment #7
marcp commentedHere's a patch for the D6 version. Any chance of this getting in, or should we go about this in a different manner (ie. hook_mail_alter)?
Comment #8
marcp commentedJust checking on this to see if it's on the radar. We've been using this patch for a while now without any problems.
Comment #9
v1nce commentedDo we have any others that have reviewed this patch or have it implemented? Having this as a configuration option seems fine. However, in the event the email being used as the sender is from a different domain, it will probably end up in a junk mail folder. Should we implement another option that will use the author of the transition if their email is from the same domain as the site, otherwise use the default site email?
Thoughts?