WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 2 years ago

#12459 closed defect (bug) (fixed)

dashboard / my sites doesn't handle empty database prefixes properly

Reported by: Denis-de-Bernardy Owned by:
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Multisite Keywords: has-patch featured
Focuses: Cc:

Description (last modified by Denis-de-Bernardy)

Original description:


super admin on newly created MS site can't browse dashboard / my sites

It dies with no link: "You must be a member of at least one site to use this page."

Attachments (3)

12459.diff (791 bytes) - added by Denis-de-Bernardy 4 years ago.
12459.2.diff (865 bytes) - added by Denis-de-Bernardy 4 years ago.
12459.3.diff (867 bytes) - added by Denis-de-Bernardy 4 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Denis-de-Bernardy4 years ago

  • Keywords has-patch added

Denis-de-Bernardy4 years ago

Denis-de-Bernardy4 years ago

comment:2 Denis-de-Bernardy4 years ago

the second patch fixes an additional php warning when the blog prefix is an empty string.

Denis-de-Bernardy4 years ago

comment:3 Denis-de-Bernardy4 years ago

the third patch additionally fixes a potential security issue in the event a user manages to create arbitrary meta details.

comment:4 Denis-de-Bernardy4 years ago

  • Description modified (diff)
  • Summary changed from super admin on newly created MS site can't browse dashboard / my sites to dashboard / my sites doesn't handle empty database prefixes properly

comment:5 Denis-de-Bernardy4 years ago

  • Keywords featured added

comment:6 automattor4 years ago

(In [13562]) allow empty table prefix on my-sites.php, see #12459

comment:7 Denis-de-Bernardy4 years ago

r13562 does NOT work. the first check eliminates the root site, because the capabilities meta is called capabilities.

also, you apparently missed the correction to the regex in the 3rd diff.

comment:8 ryan4 years ago

(In [13563]) allow empty table prefix on my-sites.php, see #12459

comment:9 wpmuguru4 years ago

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

I've been running my test install on no prefix since I did that patch and have not encountered any issues.

comment:10 jk3us2 years ago

  • Cc jk3us added

Support for empty database prefixes is broken by WP 3.3, See this bug for more information.

Version 1, edited 2 years ago by jk3us (previous) (next) (diff)
Note: See TracTickets for help on using tickets.