Ticket #6750: 6750.patch

src/bp-messages/bp-messages-notifications.php

@@ -163 +163 @@
  */
 function messages_format_notifications( $action, $item_id, $secondary_item_id, $total_items, $format = 'string' ) {
         $total_items = (int) $total_items;
+        $text        = '';
         $link        = trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() . '/inbox' );
         $title       = __( 'Inbox', 'buddypress' );
         $amount      = 'single';
@@ -187 +188 @@
                                 $text = sprintf( _n( 'You have %s new private message', 'You have %s new private messages', $total_items, 'buddypress' ), bp_core_number_format( $total_items ) );
                         }
                 }
-        }
 
-        if ( 'string' === $format ) {
-                if ( ! empty( $link ) ) {
-                        $retval = '<a href="' . esc_url( $link ) . '" title="' . esc_attr( $title ) . '">' . esc_html( $text ) . '</a>';
+                if ( 'string' === $format ) {
+                        if ( ! empty( $link ) ) {
+                                $retval = '<a href="' . esc_url( $link ) . '" title="' . esc_attr( $title ) . '">' . esc_html( $text ) . '</a>';
+                        } else {
+                                $retval = esc_html( $text );
+                        }
+
+                        /**
+                         * Filters the new message notification text before the notification is created.
+                         *
+                         * This is a dynamic filter. Possible filter names are:
+                         *   - 'bp_messages_multiple_new_message_notification'.
+                         *   - 'bp_messages_single_new_message_notification'.
+                         *
+                         * @param string $retval            Notification text.
+                         * @param int    $total_items       Number of messages referred to by the notification.
+                         * @param string $text              The raw notification test (ie, not wrapped in a link).
+                         * @param int    $item_id           ID of the associated item.
+                         * @param int    $secondary_item_id ID of the secondary associated item.
+                         */
+                        $return = apply_filters( 'bp_messages_' . $amount . '_new_message_notification', $retval, (int) $total_items, $text, $link, $item_id, $secondary_item_id );
                 } else {
-                        $retval = esc_html( $text );
+                        /** This filter is documented in bp-messages/bp-messages-notifications.php */
+                        $return = apply_filters( 'bp_messages_' . $amount . '_new_message_notification', array(
+                                'text' => $text,
+                                'link' => $link
+                        ), $link, (int) $total_items, $text, $link, $item_id, $secondary_item_id );
+                }
+
+        // Custom notification action for the Messages component
+        } else {
+                $return_array = array(
+                        'text' => $text,
+                        'link' => $link
+                );
+
+                /**
+                 * Backcompat for plugins that used to filter bp_messages_single_new_message_notification
+                 * for their custom actions. These plugins should now use 'bp_messages_' . $action . '_notification'
+                 */
+                if ( 'new_message' !== $action && has_filter( 'bp_messages_single_new_message_notification' ) ) {
+                        $return_array = apply_filters( 'bp_messages_single_new_message_notification', $return_array, $link, (int) $total_items, $text, $link, $item_id, $secondary_item_id );
                 }
 
                 /**
-                 * Filters the new message notification text before the notification is created.
+                 * Filters the custom action notification text before the notification is created.
                  *
-                 * This is a dynamic filter. Possible filter names are:
-                 *   - 'bp_messages_multiple_new_message_notification'.
-                 *   - 'bp_messages_single_new_message_notification'.
+                 * @since 2.5
                  *
-                 * @param string $retval            Notification text.
-                 * @param int    $total_items       Number of messages referred to by the notification.
-                 * @param string $text              The raw notification test (ie, not wrapped in a link).
+                 * @param array  $value             An associative array containing the text and the link of the notification
                  * @param int    $item_id           ID of the associated item.
                  * @param int    $secondary_item_id ID of the secondary associated item.
+                 * @param int    $total_items       Number of messages referred to by the notification.
+                 * @param string $format            Return value format. 'string' for BuddyBar-compatible
+                 *                                  notifications; 'array' for WP Toolbar. Default: 'string'.
                  */
-                $return = apply_filters( 'bp_messages_' . $amount . '_new_message_notification', $retval, (int) $total_items, $text, $link, $item_id, $secondary_item_id );
-        } else {
-                /** This filter is documented in bp-messages/bp-messages-notifications.php */
-                $return = apply_filters( 'bp_messages_' . $amount . '_new_message_notification', array(
-                        'text' => $text,
-                        'link' => $link
-                ), $link, (int) $total_items, $text, $link, $item_id, $secondary_item_id );
+                $return = apply_filters( 'bp_messages_' . $action . '_notification', $return_array, $item_id, $secondary_item_id, $total_items, $format );
         }
 
         /**