#36009 closed enhancement (fixed)
wp_new_user_notification "user" option for third parameter
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | 4.4.2 |
Component: | Users | Keywords: | has-patch needs-unit-tests |
Focuses: | administration | Cc: |
Description
I propose adding a third option for the third parameter $notify
in wp_new_user_notification()
called "user" (string) which would allow for sending the notification to the user only. Many user-driven sites have the need for sending this email to new users but would rather not flood an admin email with registration notifications.
I don't think this only flaw is enough to justify plugging the function with a custom one. It'd be much preferred to allow for this third option in the core function instead.
Attachments (2)
Change History (16)
#3
@
9 years ago
- Keywords needs-docs added
- Milestone changed from Awaiting Review to 4.6
Patch seems pretty good to me. Can we get a @since
annotation for the new accepted value?
#6
@
9 years ago
- Keywords commit added; needs-docs removed
The changelog entry in 36009.2.diff looks good.
#7
@
9 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 37276:
#8
follow-up:
↓ 10
@
9 years ago
- Keywords needs-unit-tests added; needs-testing commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
I'm shocked that this landed without a unit test. May I remind you about [35735]?
Also,wp_send_new_user_notifications()
is a wrapper for wp_new_user_notification()
and should get a note about the new parameter too.
#13
@
9 years ago
I submitted the following suggestion here https://core.trac.wordpress.org/ticket/36970 which turns out to be a duplicate of this one!
Adding a user email only option is great idea.
#14
@
7 years ago
FYI: If BuddyPress is active, this is not sufficient.
Please see also: https://buddypress.trac.wordpress.org/ticket/7597#comment:7
I agree this would be a useful addition. I've attached 36009.diff which adds this functionality. The few lines indented by the
if
block added are also fixed for spacing issues to adhere to the WordPress coding standards.