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

[buddypress-trac]

#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


Comment:

 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
 notification
 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