[buddypress-trac] [BuddyPress Trac] #8558: Adapt Group extensions registration to BP Rewrites

buddypress-trac noreply at wordpress.org
Fri Aug 27 13:50:54 UTC 2021


#8558: Adapt Group extensions registration to BP Rewrites
--------------------------+---------------------
 Reporter:  imath         |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  10.0.0
Component:  Groups        |     Version:  1.1
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+---------------------

Comment (by boonebgorges):

 Thanks for working on this, @imath ! The change described here seems fine
 to me at a glance.

 You mentioned in the corresponding GitHub ticket
 https://github.com/buddypress/bp-rewrites/issues/9#issuecomment-907216220
 that you were considering moving the setup of `BP_Group_Extension` classes
 to `bp_parse_query`. I would be very cautious when considering this
 change. I've written a number of plugins and customizations that are
 sensitive to the specific load-order of `BP_Group_Extension` at `bp_init`,
 and moving it all the way to `parse_query` would almost certainly result
 in widespread breakage.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8558#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list