[buddypress-trac] [BuddyPress Trac] #9254: Stop using a component for the Members Invitations feature navigation

buddypress-trac noreply at wordpress.org
Fri Nov 1 11:24:13 UTC 2024


#9254: Stop using a component for the Members Invitations feature navigation
--------------------------+--------------------
 Reporter:  imath         |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  15.0.0
Component:  Members       |    Version:  12.0.0
 Severity:  normal        |   Keywords:
--------------------------+--------------------
 As @emaralive noted here #9237, we currently have a `members_invitations`
 component active in BuddyPress although it's one of the features of the
 Members component.

 This component was born during the 12.0 development cycle (see #9013) to
 register its navigation and make corresponding slugs customizable. The
 downside of this move is that we are wrongly populating our components
 globals with it.

 I believe we should stop, and more globally, I believe we should have a
 specific API to organize a component's feature. I've been working and
 improving this API building these PRs :

 - #9098 : [https://github.com/buddypress/buddypress/pull/368 Revamped
 notices feature]
 - #9057 : [https://github.com/buddypress/buddypress/pull/386 Make Signups
 a deactivable (using a filter) Members component feature]

 If the 2 above are introducing huge changes and couldn't be tested enough
 to be introduced into 15.0.0 imho, I believe we should at the very minimum
 introduce this new `BP_Component_Feature` class and start using it to
 organize components features.

 The Members Invitations one seems to be a MVP for this introduction as it
 is currently bringing some mess (I'm the one to blame about it) into our
 globals.

 I've tested the PR I've attached to this ticket and did as few changes I
 could to take as less risks as possible. Ideally, from what I've seen we
 should carry on reorganizing some of the Members Invitations code to be
 only loaded when the feature is not deactivated.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/9254>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list