Changes between Version 3 and Version 4 of Ticket #7020, comment 1
- Timestamp:
- 04/15/2016 10:17:20 PM (9 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #7020, comment 1
v3 v4 3 3 The `'bp_notifications_get_notifications_for_user'` filter is quite weird indeed. You either have to return a string of the content or `array( 'text' => 'CONTENT', 'link' => 'LINK' )` depending on the `$format`. I think this needs better documentation. 4 4 5 If you do that, then the `$component_action_name` is wiped out and no other plugin can do checks. comment:8:ticket:6669 goes into the problem as well.5 If a plugin does change the value of `$component_action_name`, then no other plugin can do checks against `$component_action_name`. comment:8:ticket:6669 goes into the problem as well. 6 6 7 7 We probably need to add some additional parameters so the action is passed again for better checks: … … 20 20 }}} 21 21 22 Unfortunately, this does not fix the issue. We might need to think about deprecating this filter for something else. 23 22 24 ---- 23 25 24 Personally, I would not recommend using this filter. Instead, read the latter half of comment:4:ticket:6669 for an alternative approach to formatting your custom notifications.26 For an alternative way to formatting your custom notifications, read the latter half of comment:4:ticket:6669.