Opened 2 years ago

Closed 8 months 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 Owned by: nacin
Priority: normal Milestone: 3.5
Component: Multisite Version: 3.1
Severity: major Keywords: has-patch commit
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 22 months ago.

Download all attachments as: .zip

Change History (5)

SergeyBiryukov — 22 months ago

comment:1   SergeyBiryukov — 22 months ago

  • Keywords has-patch added; needs-patch removed

comment:2   wonderboymusic — 8 months ago

  • Milestone changed from Awaiting Review to 3.5

Keywords has-sergey added.

comment:3   nacin — 8 months ago

  • Keywords commit added

Looks good.

comment:4   nacin — 8 months 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.