[buddypress-trac] [BuddyPress Trac] #5938: BP Activity Hooks Documentation

buddypress-trac noreply at wordpress.org
Sat Oct 18 11:36:40 UTC 2014


#5938: BP Activity Hooks Documentation
--------------------+-----------------------------
 Reporter:  tw2113  |       Owner:
     Type:  task    |      Status:  new
 Priority:  normal  |   Milestone:  Future Release
Component:  Core    |     Version:
 Severity:  normal  |  Resolution:
 Keywords:          |
--------------------+-----------------------------

Comment (by boonebgorges):

 > With bp-activity-classes.php, how do we want to handle the save method?
 I'm counting 14 filters right at the start

 Let's not bother documenting the filters that we're encouraging should not
 be used. Just do the one with `do_action_ref_array()`.

 > Also curious how we want to handle the bp_activity_get_user_join_filter
 as it's used in two places in bp-activity-classes.php but conditionally
 has some extra parameters passed through.

 This is going to pose problems for the parser. Let's do the following:
 create full documentation for the hook with the most params; mark the
 other one with the `/* This filter is documented in... */` notation; then
 open a ticket for that specific hook so we can add the additional params
 to the hook that doesn't have them. (All hooks should have the same
 arguments passed to them.)

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


More information about the buddypress-trac mailing list