Changeset 6616 for trunk/src/includes/common/functions.php

Timestamp:

07/04/2017 05:43:05 PM (9 years ago)

Author:

johnjamesjacoby

Message:

Subscriptions: Include closed status in subscription triggers.

This change allows activity in closed topics to trigger subscription emails. It also fully implements bbp_get_subscribers() completing the deprecation of topic/forum specific handlers.

Fixes #2729.

File:

• trunk/src/includes/common/functions.php (modified) (4 diffs)

trunk/src/includes/common/functions.php

@@ -1040 +1040 @@
     /** Topic *****************************************************************/
 
-    // Bail if topic is not published
-    if ( ! bbp_is_topic_published( $topic_id ) ) {
+    // Bail if topic is not public (includes closed)
+    if ( ! bbp_is_topic_public( $topic_id ) ) {
         return false;
     }
@@ -1058 +1058 @@
 
     // Get topic subscribers and bail if empty
-    $user_ids = bbp_get_topic_subscribers( $topic_id, true );
+    $user_ids = bbp_get_subscribers( $topic_id );
 
     // Dedicated filter to manipulate user ID's to send emails to
@@ -1200 +1200 @@
     /** Topic *****************************************************************/
 
-    // Bail if topic is not published
-    if ( ! bbp_is_topic_published( $topic_id ) ) {
+    // Bail if topic is not public (includes closed)
+    if ( ! bbp_is_topic_public( $topic_id ) ) {
         return false;
     }
@@ -1211 +1211 @@
 
     // Get topic subscribers and bail if empty
-    $user_ids = bbp_get_forum_subscribers( $forum_id, true );
+    $user_ids = bbp_get_subscribers( $forum_id );
 
     // Dedicated filter to manipulate user ID's to send emails to