[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