WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#22787 closed defect (bug) (fixed)

New (3.5) subdirectory multisite installs create 'path' without the required leading slash

Reported by: markjaquith Owned by: nacin
Milestone: 3.5 Priority: highest omg bbq
Severity: blocker Version: 3.5
Component: Multisite Keywords:
Focuses: Cc:

Description

New (that is, not converted to multisite in 3.4.x or earlier) multisite installs in subdirectory mode will create new signups and blogs with a path that looks like this: "sitename/" instead of "/sitename/", and thus they don't work.

Issue has to do with the dropping of $base from wp-config.php, but it still being used in one location.

See #19796 for the work that dropped $base.

Attachments (2)

22787.diff (448 bytes) - added by nacin 2 years ago.
22787.2.diff (580 bytes) - added by markjaquith 2 years ago.
The other half

Download all attachments as: .zip

Change History (5)

@nacin2 years ago

comment:1 @nacin2 years ago

I am also debating 22787.diff, for plugins. But no one has complained, so inclined to ignore it.

@markjaquith2 years ago

The other half

comment:2 @markjaquith2 years ago

I tried my patch, and then a compat $base fix (by itself, in a similar location). Both fixed the issue.

comment:3 @nacin2 years ago

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

In 23107:

Use $current_site->path instead of $base in wpmu_validate_blog_signup(), as $base is now dead. props MarkJaquith. fixes #22787. see #19796.

Note: See TracTickets for help on using tickets.