[buddypress-trac] [BuddyPress Trac] #6750: Extending Messages notification should be improved

[buddypress-trac]

#6750: Extending Messages notification should be improved
-----------------------------------+-----------------------
 Reporter:  imath                  |       Owner:  imath
     Type:  enhancement            |      Status:  assigned
 Priority:  normal                 |   Milestone:  2.6
Component:  Component - Messaging  |     Version:  1.0
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch              |
-----------------------------------+-----------------------
Changes (by r-a-y):

 * keywords:  needs-patch => has-patch


Comment:

 `02.patch` handles the string format and also adds another edge case for
 the `'bp_messages_single_new_message_notification'` filter.  If `$format`
 is an array, there are seven parameters; if `$format` is a string, there
 are six...:
 https://buddypress.trac.wordpress.org/browser/tags/2.4.3/src/bp-messages
 /bp-messages-notifications.php#L212 :(

 Not sure how people are using this filter at the moment!

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