Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#17303 closed defect (bug) (fixed)

wpmu_validate_blog_signup does not check the current site, but the one with id == 1

Reported by: larysa's profile larysa Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: major Version: 3.1
Component: Multisite Keywords: has-patch commit
Focuses: Cc:

Description

default WP blog validation procedure wpmu_validate_blog_signup () only checks the first site ( id == 1 and in the case of multisite install skips the check for the current main site)

file: wp-includes/ms-functions.php
function: wpmu_validate_blog_signup

calls: domain_exists($mydomain, $path)

should check: domain_exists($mydomain, $path, $current_site->id)

Attachments (1)

17303.patch (501 bytes) - added by SergeyBiryukov 14 years ago.

Download all attachments as: .zip

Change History (5)

#1 @SergeyBiryukov
14 years ago

  • Keywords has-patch added; needs-patch removed

#2 @wonderboymusic
13 years ago

  • Milestone changed from Awaiting Review to 3.5

Keywords has-sergey added.

#3 @nacin
13 years ago

  • Keywords commit added

Looks good.

#4 @nacin
13 years ago

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

In [21853]:

Pass current network ID to domain_exists() in wpmu_validate_blog_signup(). props larysa, fixes #17303.

Note: See TracTickets for help on using tickets.