Opened 12 years ago
Last modified 6 years ago
#24026 new defect (bug)
No /themes/ folder causes strange behaviour
Reported by: | slaFFik | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 3.1 |
Component: | Themes | Keywords: | needs-patch |
Focuses: | Cc: |
Description
I know, that it's a tricky behaviour, but the problem exists.
- Unzip WordPress files to appropriate place
- Go to /wp-content/ and delete /themes/ folder
- Install WordPress using it's wizard (ignore error in Dashboard that no theme activated)
- Go to Plugins page in admin area
- Install BuddyPress (this plugin register the theme BP Default that is situated in plugin folder - this is very important)
- Activate BuddyPress and complete its wizard to make BP functional
- Go to Themes page in wp-admin
- You will now see BP Default Theme. Try to activate it
- Ta-da! You've just unlocked a new badge for catching an error.
Change History (6)
Note: See
TracTickets for help on using
tickets.
Got "Cheatin' uh?" message from
wp-admin/themes.php
:http://core.trac.wordpress.org/browser/tags/3.5.1/wp-admin/themes.php#L20
BuddyPress Default theme is correctly registered with
/plugins/buddypress/bp-themes
as theme root, howeverwp_get_theme( 'bp-default' )
later looks forWP_CONTENT_DIR . '/themes'
instead.Caused by the early return in
get_theme_roots()
andget_raw_theme_root()
:http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/theme.php#L285
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/theme.php#L510
Commenting out these lines in both functions allowed me to activate BuddyPress Default theme.
Introduced in [15641], corrected in [16424]. It assumes that if there's only one theme directory, it's
wp-content/themes
. According to the ticket description, this assumption isn't always correct, although it's probably an edge case.Reproduced in 3.1 as well. There was no "Cheatin' uh?" message (it was introduced later in [21010]), however
validate_current_theme()
still failed, and the theme could not be activated:http://core.trac.wordpress.org/browser/tags/3.1/wp-admin/themes.php#L60