WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 18 months ago

#12459 closed defect (bug) (fixed)

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

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

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 3 years ago.
12459.2.diff (865 bytes) - added by Denis-de-Bernardy 3 years ago.
12459.3.diff (867 bytes) - added by Denis-de-Bernardy 3 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Denis-de-Bernardy3 years ago

  • Keywords has-patch added

Denis-de-Bernardy3 years ago

Denis-de-Bernardy3 years ago

comment:2 Denis-de-Bernardy3 years ago

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

Denis-de-Bernardy3 years ago

comment:3 Denis-de-Bernardy3 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-Bernardy3 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-Bernardy3 years ago

  • Keywords featured added

comment:6 automattor3 years ago

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

comment:7 Denis-de-Bernardy3 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 ryan3 years ago

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

comment:9 wpmuguru3 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 jk3us18 months ago

  • Cc jk3us added

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

Last edited 18 months ago by jk3us (previous) (diff)
Note: See TracTickets for help on using tickets.