Ticket #2863: 2863.02.patch

src/includes/common/functions.php

@@ -1101 +1101 @@
         // Poster name
         $reply_author_name = bbp_get_reply_author_display_name( $reply_id );
 
+        /** Users *****************************************************************/
+
+        // Get topic subscribers and bail if empty
+        $user_ids = bbp_get_topic_subscribers( $topic_id, true );
+
+        // Dedicated filter to manipulate user ID's to send emails to
+        $user_ids = (array) apply_filters( 'bbp_topic_subscription_user_ids', $user_ids );
+        $user_ids = array_filter( array_map( 'intval', $user_ids ) );
+        if ( empty( $user_ids ) ) {
+                return false;
+        }
+
+        // Remove the reply author from the list.
+        $reply_author_key = array_search( (int) $reply_author, $user_ids, true );
+        if ( false !== $reply_author_key ) {
+                unset( $user_ids[ $reply_author_key ] );
+        }
+
+        // Bail of the reply author was the only one subscribed.
+        if ( empty( $user_ids ) ) {
+                return false;
+        }
+
         /** Mail ******************************************************************/
 
         // Remove filters from reply content and topic title to prevent content
@@ -1143 +1166 @@
                 return;
         }
 
-        /** Users *****************************************************************/
+        /** Headers ***************************************************************/
 
         // Get the noreply@ address
         $no_reply   = bbp_get_do_not_reply_address();
@@ -1154 +1177 @@
         // Setup the From header
         $headers = array( 'From: ' . get_bloginfo( 'name' ) . ' <' . $from_email . '>' );
 
-        // Get topic subscribers and bail if empty
-        $user_ids = bbp_get_topic_subscribers( $topic_id, true );
-
-        // Dedicated filter to manipulate user ID's to send emails to
-        $user_ids = apply_filters( 'bbp_topic_subscription_user_ids', $user_ids );
-        if ( empty( $user_ids ) ) {
-                return false;
-        }
-
         // Loop through users
         foreach ( (array) $user_ids as $user_id ) {
 
-                // Don't send notifications to the person who made the post
-                if ( ! empty( $reply_author ) && (int) $user_id === (int) $reply_author ) {
-                        continue;
-                }
-
                 // Get email address of subscribed user
                 $headers[] = 'Bcc: ' . get_userdata( $user_id )->user_email;
         }
@@ -1258 +1267 @@
         // Poster name
         $topic_author_name = bbp_get_topic_author_display_name( $topic_id );
 
+        /** Users *****************************************************************/
+
+        // Get topic subscribers and bail if empty
+        $user_ids = bbp_get_forum_subscribers( $forum_id, true );
+
+        // Dedicated filter to manipulate user ID's to send emails to
+        $user_ids = (array) apply_filters( 'bbp_forum_subscription_user_ids', $user_ids );
+        $user_ids = array_filter( array_map( 'intval', $user_ids ) );
+        if ( empty( $user_ids ) ) {
+                return false;
+        }
+
+        // Remove the topic author from the list.
+        $topic_author_key = array_search( (int) $topic_author, $user_ids, true );
+        if ( false !== $topic_author_key ) {
+                unset( $user_ids[ $topic_author_key ] );
+        }
+
+        // Bail of the topic author was the only one subscribed.
+        if ( empty( $user_ids ) ) {
+                return false;
+        }
+
         /** Mail ******************************************************************/
 
         // Remove filters from reply content and topic title to prevent content
@@ -1300 +1332 @@
                 return;
         }
 
-        /** User ******************************************************************/
+        /** Headers ***************************************************************/
 
         // Get the noreply@ address
         $no_reply   = bbp_get_do_not_reply_address();
@@ -1311 +1343 @@
         // Setup the From header
         $headers = array( 'From: ' . get_bloginfo( 'name' ) . ' <' . $from_email . '>' );
 
-        // Get topic subscribers and bail if empty
-        $user_ids = bbp_get_forum_subscribers( $forum_id, true );
-
-        // Dedicated filter to manipulate user ID's to send emails to
-        $user_ids = apply_filters( 'bbp_forum_subscription_user_ids', $user_ids );
-        if ( empty( $user_ids ) ) {
-                return false;
-        }
-
         // Loop through users
         foreach ( (array) $user_ids as $user_id ) {
 
-                // Don't send notifications to the person who made the post
-                if ( ! empty( $topic_author ) && (int) $user_id === (int) $topic_author ) {
-                        continue;
-                }
-
                 // Get email address of subscribed user
                 $headers[] = 'Bcc: ' . get_userdata( $user_id )->user_email;
         }