Add ability to edit email to admin at account creation

druchris1 - March 7, 2009 - 04:26
Project:Mail Editor
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

AttachmentSize
add_admin_email.patch1.52 KB

#1

druchris1 - March 7, 2009 - 04:32

Whoops, didn't clean up the description text

AttachmentSize
add_admin_email.patch 1.52 KB

#2

Kars-T - September 8, 2009 - 12:54
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

#3

druchris1 - September 13, 2009 - 00:29

Ok, fixed the patch file. THanks for the review

AttachmentSize
add_admin_email.patch 1.52 KB

#4

druchris1 - September 13, 2009 - 00:29
Status:needs work» needs review

#5

Kars-T - September 13, 2009 - 18:55

+++ 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

#6

Kars-T - September 13, 2009 - 18:57
Status:needs review» reviewed & tested by the community

#7

litwol - September 14, 2009 - 00:30
Status:reviewed & tested by the community» fixed

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

#8

System Message - September 28, 2009 - 00:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.