WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26703 closed enhancement (fixed)

Improve wp_new_user_notification() description

Reported by: antorome's profile antorome Owned by: drewapicture's profile DrewAPicture
Milestone: 3.9 Priority: normal
Severity: normal Version: 2.0
Component: Login and Registration Keywords: has-patch commit
Focuses: docs Cc:

Description

Hello,

in pluggable.php, line 1280 we have wp_new_user_notification.

In the function description it just says "notify the blog admin" but it is also notifying the new user. I have spent quite a lot of time looking for the function to modify the message to be sent to the user thinking this one was just for the admin.

It is a minor thing but could help others willing to modify these emails.

BR

Attachments (2)

pluggable.diff​ (440 bytes) - added by antorome 11 years ago.
Modification proposal
26703.diff​ (538 bytes) - added by DrewAPicture 11 years ago.

Download all attachments as: .zip

Change History (5)

@antorome
11 years ago

Modification proposal

@DrewAPicture
11 years ago

#1 @DrewAPicture
11 years ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 3.9
  • Summary changed from New user notification comments to Improve wp_new_user_notification() description
  • Version changed from trunk to 2.0

26703.diff​ takes a slightly different tack.

Notify the blog admin of a new user, normally via email.

becomes

Email login credentials to a newly-registered user.

A new user registration notification is also sent to admin email.

#2 @nacin
11 years ago

  • Component changed from Inline Docs to Login and Registration
  • Focuses docs added

#3 @DrewAPicture
11 years ago

  • Owner set to DrewAPicture
  • Resolution set to fixed
  • Status changed from new to closed

In 27149:

Improve inline documentation for wp_new_user_notification().

Props antorome for the initial patch.
Fixes #26703.

Note: See TracTickets for help on using tickets.