[buddypress-trac] [BuddyPress] #3741: BuddyPress should have theme compatibility

buddypress-trac noreply at wordpress.org
Mon Oct 22 10:58:35 UTC 2012


#3741: BuddyPress should have theme compatibility
-----------------------------+------------------------------
 Reporter:  johnjamesjacoby  |       Owner:  johnjamesjacoby
     Type:  enhancement      |      Status:  new
 Priority:  highest          |   Milestone:  1.7
Component:  Theme            |     Version:
 Severity:  critical         |  Resolution:
 Keywords:  dev-feedback     |
-----------------------------+------------------------------

Comment (by johnjamesjacoby):

 Replying to [comment:41 r-a-y]:
 > JJJ, after doing some more testing, you're absolutely right that
 bp_enable_theme_compat() is not necessary.
 >
 > In r6404, I have removed bp_enable_theme_compat() and introduced a new
 function to check if bp-default is running - bp_is_theme_bp_default().
 That should still do the same thing as what I originally intended.
 >
 > Let me know what you think.
 In various commits above, I've reworked some logic so that
 bp_is_theme_bp_default() isn't necessary either. bp_locate_template() can
 now cover all scenarios with relative ease: parent, child, theme-compat,
 filtered. It could before, but looking more into this issue allowed for
 some hardening of the logic involved.

 Give it a go, let me know if things are still working to your
 expectations.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3741#comment:52>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list