Make WordPress Core

Opened 3 years ago

Last modified 3 years ago

#16235 new enhancement

Site Welcome Email sent without regard to noconfirmation flag

Reported by: transom Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.1
Component: Users Keywords: has-patch
Focuses: multisite Cc:

Description (last modified by dd32)

In multisite and the super_admin adds a new user to a site,
if the user is new to the network, WP sends out the welcome email (but not the confirmation of being added to the site)

In my client's use case, they were looking for the SBD (silent but deadly) mode, as in no email at all should go out.

The language implies there is "no" notification but the welcome email belies that language.

if even the welcome email should be disabled, then
in wp-admin/user-new.php (around line 108)
add_filter('wpmu_welcome_user_notification', ''__return_false');
would kill the new user notification from wpmu_activate_signup() (a few lines later)

Attachments (2)

welcome.diff (819 bytes) - added by transom 3 years ago.
Add filter to kill Welcome Email if nonotification
16235.diff (795 bytes) - added by tmoorewp 3 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 transom3 years ago

  • Component changed from General to Users
  • Keywords new user multisite added
  • Type changed from defect (bug) to feature request
  • Version set to 3.1

arggg - the second term of the add_filter should be the underline-underline_return_false function.

comment:2 dd323 years ago

  • Description modified (diff)
  • Keywords needs-patch added; new user multisite removed

comment:3 dd323 years ago

  • Keywords multisite added

comment:4 nacin3 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Type changed from feature request to enhancement

transom3 years ago

Add filter to kill Welcome Email if nonotification

comment:5 transom3 years ago

patch added

comment:6 markjaquith3 years ago

This looks good to me.

tmoorewp3 years ago

comment:7 tmoorewp3 years ago

  • Cc tim@… added
  • Keywords has-patch added; needs-patch removed

Refreshed patch against current trunk.

comment:8 danielbachhuber3 years ago

  • Cc d@… added

What about a filter in wp_new_user_notification() too, for similar end result?

Note: See TracTickets for help on using tickets.