[buddypress-trac] [BuddyPress Trac] #8546: Messages notifications filters: it needs some clean-up!

[buddypress-trac]

#8546: Messages notifications filters: it needs some clean-up!
--------------------------+---------------------
 Reporter:  imath         |       Owner:  imath
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  10.0.0
Component:  Messages      |     Version:
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |
--------------------------+---------------------
Changes (by imath):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"13035" 13035]:
 {{{
 #!CommitTicketReference repository="" revision="13035"
 Make the `new_message` notification filter consistent across formats

 The 'bp_messages_' . $amount . '_new_message_notification' dynamic filter
 is including **6** parameters for the `string` format and **7** for the
 `array` format. This makes it difficult to use for Plugin developers.

 To make things more consistent across formats, we are deprecating this
 filter in favor of a new dynamic filter `'bp_messages_' . $amount .
 '_new_message_' . $format . '_notification'`. FYI, possible filter names
 are:

 - `'bp_messages_multiple_new_message_string_notification'`
 - `'bp_messages_single_new_message_string_notification'`
 - `'bp_messages_multiple_new_message_array_notification'`
 - `'bp_messages_single_new_message_array_notification'`

 Fixes #8546
 }}}

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