[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