#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
@
10 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
@
10 years ago
- Keywords commit added; needs-docs removed
The changelog entry in 36009.2.diff looks good.
#7
@
10 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 37276:
#8
follow-up:
↓ 10
@
10 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
ifblock added are also fixed for spacing issues to adhere to the WordPress coding standards.