[buddypress-trac] [BuddyPress] #2743: Update handling of "front page displays" option for trunk
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Wed Nov 17 22:16:48 UTC 2010
#2743: Update handling of "front page displays" option for trunk
--------------------+-------------------------------------------------------
Reporter: DJPaul | Owner: DJPaul
Type: defect | Status: new
Priority: normal | Milestone: 1.3
Component: Core | Version:
Keywords: |
--------------------+-------------------------------------------------------
Comment(by DJPaul):
Issue with patch so far is with these bits:
{{{
if ( bp_is_blogs_component() && !bp_is_blog_page() ||
bp_is_component_on_front_page( $bp->blogs->slug ) )
}}}
We need to check if $bp->blogs is set. This applies to all components, not
just blogs. I've spoken about this with Boone on IRC and one of the
options we have is to set current_component to the appropriate component
if on the front page, and use bp_is_blogs_component().
Another is to add extra checks to the bp_is_x_component() set of functions
to check for the front page without messing with current_component.
--
Ticket URL: <http://trac.buddypress.org/ticket/2743#comment:1>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list