I'd like to be able to see the address of the user who's registering and that I got an email from my drupal site for. A little bit of added code in the patch.

It seems like this basic functionality might want to just go in the drupal core, but I'm not sure where to submit that part to.

CommentFileSizeAuthor
#3 add_admin_email.patch1.52 KBdrpchris
#1 add_admin_email.patch1.52 KBdrpchris
add_admin_email.patch1.52 KBdrpchris

Comments

drpchris’s picture

drpchris commented
StatusFileSize
new add_admin_email.patch1.52 KB

Whoops, didn't clean up the description text

kars-t’s picture

kars-t commented
Status: Needs review » Needs work 
+++ mail_edit_user.inc	2009-03-06 23:30:09.000000000 -0500
@@ -2,30 +2,32 @@
+    case 'register_pending_approval_admin';

Typo! There is an ";" where it should be an ":". Otherwise this patch seems fine.

Enhanced by Dreditor

drpchris’s picture

drpchris commented
StatusFileSize
new add_admin_email.patch1.52 KB

Ok, fixed the patch file. THanks for the review

drpchris’s picture

drpchris commented
Status: Needs work » Needs review
kars-t’s picture

kars-t commented 
+++ mail_edit_user.inc	2009-03-06 23:30:09.000000000 -0500
@@ -2,30 +2,32 @@
+    'register_pending_approval_admin' => t(' Message to admin, user is pending'),//has password
...
+    case 'register_pending_approval_admin':

Just adds the mailkey to the array and the switch. 'register_pending_approval_admin' is from user_module line 2157. I think this patch is fine and ready to commit.

Enhanced patch reviews

kars-t’s picture

kars-t commented
Status: Needs review » Reviewed & tested by the community
litwol’s picture

litwol commented
Status: Reviewed & tested by the community » Fixed

Committed to -dev. If you have more patches, dont be shy to add them to issue queue.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.