Closed (fixed)
Project:
User Registration Notification
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
21 Feb 2007 at 22:35 UTC
Updated:
17 Jun 2008 at 03:12 UTC
Jump to comment: Most recent file
Allow the administrator to customize the email recipient, subject, and body of the user registration notification email.
Patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | user_register_notify-email.patch | 2.08 KB | davemybes |
| user_notify_customization.patch | 4.91 KB | samo |
Comments
Comment #1
walterhoct commentedWhen I applied this patch, I got the following.
(Stripping trailing CRs from patch.)
patching file user_register_notify.module
patch unexpectedly ends in middle of line
Hunk #2 succeeded at 66 with fuzz 1.
Any idea?
Comment #2
salvisThe patch applied just fine on 5.x-1.x-dev.
#1: Please do not change the title!
OP: I really like your patch, but it has a few minor problems:
$variables =!uriand!uri_briefvariables don't work!mailtovariable isn't mentionedand append
$headersto thedrupal_mail()argument list (and update theuser_mail()call, too).We never want to reply to the site_mail address, but we might want to send e-mail to the new user.
Comment #3
davemybes commentedI was going to create a new issue, but seeing as its almost the same request, I thought I'd post here instead.
I like the idea of customizing the actual email as well, but usually its the site admin dealing with the notifications, so it doesn't have to be fancy. However, having the ability to change the recipient of the notification would be quite useful.
I have created my own patch to do this (attached). I have just looked at the patch supplied above, and I see that essentially the same technique is used, except that I could never get the nested variable_get statements (that get the email addresses) to work, so I used an if...else statement instead (Drupal 5.5, MySQL & PHP 5).
Comment #4
rmiddle commenteduser_notify_customization.patch - Wont Fix. Something for the 2.x branch
user_register_notify-email.patch - Was one of 2 features I was looking to add.
Comment #5
rmiddle commentedPatch #3 is commited to CVS.
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.