Ticket #846 (closed defect: fixed)

Opened 14 months ago

Last modified 14 months ago

not able to select member theme (WPMU 2.8.1)

Reported by: DJPaul Owned by:
Priority: major Milestone:
Component: Keywords:
Cc:

Description

In the BP admin page, we're not able to select the member theme when using WPMU 2.8.1. This is because the drop-down box returns a list of the /wp-content/themes/.

This is because get_themes() checks to see a cached version of the list of themes has already been generated, and uses it if it has. There is a cached version, because when the admin pages are loaded, get_themes() is called by add_action( 'admin_init', '_maybe_update_themes' ); in update.php.

_maybe_update_themes() has changed since WPMU 2.7.1 as it uses get_transient() rather than get_option(). get_option() itself has been changed in WPMU 2.8.1, so swapping the function call back doesn't work.

get_themes() has not changed in WPMU 2.8.1.

Suggestion: let member themes live in /wp-content/themes/. Add a criteria to the member themes' style.css (e.g. "BuddyPress Theme: Yes"). get_themes() returns all themes in the themes directory, not just those themes which have been enabled by the site admin - so BP Member themes would not need enabling, so they won't be selectable by Joe Noob on a hosted blog.

Change History

Changed 14 months ago by DJPaul

Clarification: add_action( 'admin_init', '_maybe_update_themes' ); is also present in WPMU 2.7.1. The only thing that jjj and I could find different was the get_option/get_transient thing.

Changed 14 months ago by ev3rywh3re

This is a dup of #818. The patch there can fix it temporarily, but this issue may need a closer look.

Changed 14 months ago by apeatling

  • status changed from new to closed
  • resolution set to fixed

Thanks guys, this is fixed in r1601

Note: See TracTickets for help on using tickets.