[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