[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