[buddypress-trac] [BuddyPress Trac] #7020: bp_notifications_get_notifications_for_user() bug

buddypress-trac noreply at wordpress.org
Wed Apr 27 19:18:02 UTC 2016

#7020: bp_notifications_get_notifications_for_user() bug
 Reporter:  rogercoathup                   |       Owner:  r-a-y
     Type:  defect (bug)                   |      Status:  closed
 Priority:  normal                         |   Milestone:  2.6
Component:  Component - Notifications      |     Version:  1.9
 Severity:  normal                         |  Resolution:  fixed
 Keywords:  dev-feedback has-patch commit  |
Changes (by r-a-y):

 * owner:   => r-a-y
 * status:  new => closed
 * resolution:   => fixed


 In [changeset:"10725"]:
 #!CommitTicketReference repository="" revision="10725"
 Notifications: Introduce two new parameters to the
 `'bp_notifications_get_notifications_for_user'` filter.

 This commit:
 * Adds `$component_action_name` and `$component_name` as the 6th and 7th
 parameters to the filter.
 * Updates the DocBlock for the filter.

 In the DocBlock, we are deprecating the first parameter, which is the
 component action name.  It is recommended to do notification action name
 checks against the 6th parameter from now on.

 The reason why we are deprecating the first parameter is due to potential
 plugin conflicts. If a plugin is doing a specific check on the
 action and changes the return value for the filter, then no other plugin
 can do checks for that same notification action.  Also see #6669.

 Fixes #7020.

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

More information about the buddypress-trac mailing list