WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#18479 closed defect (bug) (fixed)

Remove 'BASE' == $base check in ms-settings.php?

Reported by: mdawaffe's profile mdawaffe Owned by: nacin's profile nacin
Milestone: 3.3 Priority: normal
Severity: trivial Version: 3.2.1
Component: Multisite Keywords:
Focuses: Cc:

Description

​http://core.trac.wordpress.org/browser/trunk/wp-includes/ms-settings.php?rev=16633#L14

Is this check still necessary? I think it's just a holdover from MU: ​http://mu.trac.wordpress.org/browser/trunk/wp-config-sample.php?rev=2022#L40 (old trac)

Change History (4)

#1 @nacin
13 years ago

This can indeed go. 'BASE' was the original base in the MU config-sample, but there's no way we need it this many versions out.

#2 @nacin
13 years ago

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

In [18568]:

Remove old $base/BASE MU check. props mdawaffe, fixes #18479.

#3 @jamescollins
13 years ago

Who feels like changing the milestone to 3.3 (and possibly the version to 3.0)?

#4 @dd32
13 years ago

  • Milestone changed from Awaiting Review to 3.3
Note: See TracTickets for help on using tickets.