Make WordPress Core

Opened 13 years ago

Last modified 7 months ago

#24411 new defect (bug)

New site email being sent despite registrationnotification set to no

Reported by: tomk-m's profile tomk-m Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5.1
Component: Networks and Sites Keywords: needs-refresh 2nd-opinion
Focuses: multisite Cc:

Description

When a new site is created in wp-admin/network/site-new.php it sends an e-mail to the admin email address to say the site has been created (line 92 in WP 3.5.1).

I had assumed that the registrationnotification setting in wp-admin/network/settings.php would control this but it sends the e-mail regardless of this setting.

I believe that sending this email should respect the registrationnotification setting or a new configuration option should be added to prevent sending out these e-mails.

Attachments (1)

site-new.php.diff (1.2 KB) - added by tomk-m 13 years ago.
Patch to respect registrationnotification setting

Download all attachments as: .zip

Change History (6)

@tomk-m
13 years ago

Patch to respect registrationnotification setting

#1 @tomk-m
13 years ago

  • Keywords needs-testing added

#2 @SergeyBiryukov
13 years ago

To be clear, the option currently only has an effect on newblog_notify_siteadmin(): tags/3.5.1/wp-includes/ms-functions.php#L1001.

The option label says: "Send the network admin an email notification every time someone registers a site or user account." (tags/3.5.1/wp-admin/network/settings.php#L127).

#3 @jeremyfelt
12 years ago

  • Component changed from Network Admin to Login and Registration

#4 @chriscct7
10 years ago

  • Component changed from Login and Registration to Networks and Sites

#5 @SirLouen
7 months ago

  • Keywords needs-refresh 2nd-opinion added; has-patch needs-testing removed

A very old ticket, still relevant. I can see things have changed significantly code-level, and obviously the patch is not applying.

Still, I wonder if this behaviour commented in this post is expected and desirable.

If not, at least, the copy is not ideal, and should be adapted to reflect the expectations

cc @spacedmonkey @realloc

Note: See TracTickets for help on using tickets.