[buddypress-trac] [BuddyPress Trac] #6597: Targeting the 'groups_leave_group' action can be difficult
buddypress-trac
noreply at wordpress.org
Mon Aug 24 19:10:08 UTC 2015
#6597: Targeting the 'groups_leave_group' action can be difficult
------------------------------------+------------------------------
Reporter: thebrandonallen | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Component - Groups | Version: 2.3.0
Severity: normal | Resolution:
Keywords: dev-feedback has-patch |
------------------------------------+------------------------------
Comment (by thebrandonallen):
This was, more or less, the idea I had as well. My brain took me one step
further, and ended with `groups_leave_group()` being deprecated. Your
"trimmed down" approach probably makes more sense ;)
After spending some time thinking about this, how about the below patch as
an iteration? The dynamic action probably isn't needed, since it really
only has one purpose. The result is functionally the same, but with less
code, and we have the added benefit that the `groups_leave_group` action
stays in it's original location.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6597#comment:3>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list