Make WordPress Core

Opened 8 months ago

Last modified 2 months ago

#53355 reviewing defect (bug)

wp-signup.php with `new` in the query string results in a "site does not exist" message even if the site does exist

Reported by: henry.wright Owned by: SergeyBiryukov
Milestone: 6.0 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: has-patch needs-testing needs-refresh
Focuses: multisite Cc:


Steps to reproduce:

  1. Visit where sport is the address of an existing site in your network. Note I'm using sport as an example. You should replace sport with the address of an existing blog in your network.

You will get the following message appear on the page:

The site you were looking for,, does not exist, but you can create it now!

Attachments (4)

53355.diff (1.6 KB) - added by henry.wright 8 months ago.
53355.2.diff (1.8 KB) - added by henry.wright 8 months ago.
53355.3.diff (1.8 KB) - added by henry.wright 8 months ago.
53355.4.diff (1.9 KB) - added by henry.wright 8 months ago.

Download all attachments as: .zip

Change History (13)

#1 @henry.wright
8 months ago

To add, the network install I used to test allows both user and blog registrations

8 months ago

#2 @henry.wright
8 months ago

  • Keywords has-patch added

53355.diff uses domain_exists() to check if a site name is already taken before outputting the message.

#3 @henry.wright
8 months ago

53355.2.diff fixes the value of $domain which is passed to domain_exists() as the first param.

#4 @henry.wright
8 months ago

53355.3.diff ensures the right parts of the extracted host name are used to build the domain value.

#5 @SergeyBiryukov
8 months ago

  • Milestone changed from Awaiting Review to 5.9
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

Related: #53348

#6 @henry.wright
8 months ago

53355.4.diff adds a separation character when building the domain value

This ticket was mentioned in Slack in #core by audrasjb. View the logs.

2 months ago

#9 @audrasjb
2 months ago

  • Keywords needs-testing needs-refresh added
  • Milestone changed from 5.9 to 6.0

As per today's bug scrub and since this ticket needs to be refreshed against trunk and properly tested, let's move it to next milestone.

Note: See TracTickets for help on using tickets.