[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
Sat Nov 27 21:46:21 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 boonebgorges):

 Great patch, Paul. Like butter.

 One problem. If you set Register as your front page, and you try to access
 the home page while logged in or while registration is turned off, you get
 a redirect loop (because of the checks at the beginning of
 bp_core_screen_signup() ).

 I'm not sure how to handle this. We could filter the 'Front page displays
 dropdown so that you can't set Register as the front page (effective but
 unappealing). Or maybe a check at the beginning of bp_core_screen_signup()
 that checks for this situation and redirects to some page that is sure to
 be populated (see my half assed 2743-4.patch). What do you think?

 Other than that, nice job with this one.

-- 
Ticket URL: <https://trac.buddypress.org/ticket/2743#comment:5>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list