Make WordPress Core

Opened 14 years ago

Last modified 5 years ago

#11297 new enhancement

Don't email the admin when they create a user from the backend

Reported by: caesarsgrunt's profile caesarsgrunt Owned by:
Milestone: Priority: low
Severity: minor Version: 2.9
Component: Users Keywords: dev-feedback needs-patch
Focuses: Cc:

Description

I just created the user myself. I already know I did it. I don't need to be told again.

Emails should only be sent to the admin when a user registers themselves (if that is enabled). They shouldn't be sent when the admin manually creates a new user.

Attachments (1)

11297_wp_new_user_notif-context.patch (1.5 KB) - added by nacin 14 years ago.

Download all attachments as: .zip

Change History (14)

#1 follow-up: @nacin
14 years ago

What about a site with multiple admins? More importantly, the admin_email isn't tied directly to a user account, so you'd have to compare the admin_email with the current user email, that way if the admin_email isn't the current user's, it can still be sent.

Perhaps wp_new_user_notification, as a pluggable function, could use an extra parameter specifying whether it came from wp-login.php or wp-admin?

See also #11210.

#2 @lloydbudd
14 years ago

  • Keywords reporter-feedback added
  • Type changed from defect (bug) to enhancement

Recommend closing as won't fix at this time.

Consistent, low friction... also as nacin describes "multiple admins" -- blog owner/admin is weak concept in WordPress. I don't think we should tweak it.

Functions are already pluggable.

PS. caesarsgrunt, please always include your specific build when setting the version to the current trunk version.

#3 follow-up: @nacin
14 years ago

Seems like wontfix, though the attached patch might not be a bad idea.

#4 in reply to: ↑ 3 ; follow-up: @lloydbudd
14 years ago

  • Priority changed from normal to low
  • Severity changed from normal to minor

Replying to nacin:

Seems like wontfix, though the attached patch might not be a bad idea.

nacin, maybe throw the patch in it's own ticket, describing that solves the problem of being able to filter based on $context.

#5 in reply to: ↑ 4 ; follow-up: @nacin
14 years ago

Replying to lloydbudd:

nacin, maybe throw the patch in it's own ticket, describing that solves the problem of being able to filter based on $context.

I could, but this patch would directly facilitate a solution for this via a plugin. A new ticket here just bloats Trac IMHO.

#6 in reply to: ↑ 5 @lloydbudd
14 years ago

Replying to nacin:

Replying to lloydbudd:

nacin, maybe throw the patch in it's own ticket, describing that solves the problem of being able to filter based on $context.

I could, but this patch would directly facilitate a solution for this via a plugin. A new ticket here just bloats Trac IMHO.

Which ever you prefer.

A concise problem + solution makes it easier for other to find, and encourages by example well written ticket submission.

#7 in reply to: ↑ 1 @caesarsgrunt
14 years ago

Replying to nacin:

What about a site with multiple admins? More importantly, the admin_email isn't tied directly to a user account, so you'd have to compare the admin_email with the current user email, that way if the admin_email isn't the current user's, it can still be sent.

Well, perhaps we'd also have to change that to fix this (and #11277).

I'm not sure if the email needs sending even in the event of multiple admins, but i it does then perhaps we could do it by either (a) checking the addresses and not sending to the address of the current user, or (b) removing the admin email field in Settings and emailing each admin instead using their profile email address. This is tidier anyway; having the same data in two places is redundant.

Replying to lloydbudd:

PS. caesarsgrunt, please always include your specific build when setting the version to the current trunk version.

I do when it's relevant, but in this case it isn't really. I can anyway if you like... (FWIW, this still applies in 12312.)

#8 @ocean90
13 years ago

  • Keywords reporter-feedback removed

#9 @chriscct7
9 years ago

  • Keywords dev-feedback needs-patch added

Core should review whether they want this. Last time, Nacin was against in comment 1

#10 @chriscct7
8 years ago

  • Keywords needs-nacin added

#11 @jeremyfelt
8 years ago

  • Keywords needs-nacin removed

#33587 added a context param to the pluggable wp_new_user_notification(), though we only handle a case for (1) admin emails only or (2) admin and new user.

We could also wrap the email to the admin and provide for 4 context possibilities - user only, admin only, both, or none. This would allow someone to unhook/rehook notifications up with a different wrapper than wp_send_new_user_notification() and thus context.

Pinging @boonebgorges for his thoughts as he was deep in the weeds of #33587.

Last edited 8 years ago by jeremyfelt (previous) (diff)

#12 @boonebgorges
8 years ago

@jeremyfelt I lean thumbs-down on this ticket. As @nacin notes, a site could have more than one admin, and it makes sense for all to be notified when a new user is created. The fact that the notification is now sent via callback means that it can be selectively unhooked if it's deemed redundant for a given site.

#13 @DrewAPicture
8 years ago

  • Summary changed from Don't email the admin when he creates a user from the backend to Don't email the admin when they create a user from the backend
Note: See TracTickets for help on using tickets.