Silent BCC option
deekayen - August 7, 2009 - 18:01
| Project: | Reroute Email |
| Version: | 6.x-1.0 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Description
Even in a production environment, there have been times where I have wanted to be BCCed on all outgoing emails, just to prove they were being sent. #543022: How do I know if it's working? on inactive user reminds me of that need and I found a few other similar requests to other projects when searching drupal.org for "bcc". This project seems like the closest one to being a good fit for adding such a feature. What do you think about adding it here vs its own, or is there an even better option to submit it to?
#1
Go ahead and submit a patch for it.
#2
I would also like to be able to BCC the mails
#3
Here you go; There's a new option in the settings for silent BCC.
Note that this patch is for the 1.0 release.
#4
Just a few minor things:
The standard capitalizing for "E-Mail" is "e-mail" in Drupal core.
The
} else {format should have a new line after the first bracket, before the else.The watchdog call assigns the t(), third parameter variables to
$variablesinstead of just stating the array. I don't think you need the variable assignment in the parameters. I'd also maybe set that watchdog to be at WATCHDOG_INFO level.#5
Really cool idea.