Changeset 7319

Timestamp:

06/27/2025 09:55:09 PM (5 months ago)

Author:

johnjamesjacoby

Message:

API - Actions/Filters: pass filtered author values into actions when adding/editing forums/topics/replies.

This change uses the pre-filtered _data versions of author IDs rather than the original/unfiltered ones, fixing an unintended behavior where the newly filtered author ID was not being used by the subsequent hooked-in functions.

See: ​https://meta.trac.wordpress.org/ticket/6839

Props dd32.

In branches/2.6, for 2.6.14.

Fixes #3578.

Location:

branches/2.6/src/includes

Files:

• forums/functions.php (modified) (5 diffs)
• replies/functions.php (modified) (2 diffs)
• topics/functions.php (modified) (2 diffs)

branches/2.6/src/includes/forums/functions.php

@@ -316 +316 @@
         do_action( 'bbp_new_forum', array(
             'forum_id'           => $forum_id,
-            'post_parent'        => $forum_parent_id,
-            'forum_author'       => $forum_author,
+            'post_parent'        => $forum_data['post_parent'],
+            'forum_author'       => $forum_data['post_author'],
             'last_topic_id'      => 0,
             'last_reply_id'      => 0,
@@ -384 +384 @@
     // Define local variable(s)
     $anonymous_data = array();
-    $forum = $forum_id = $forum_parent_id = 0;
+    $forum = $forum_id = $forum_author = $forum_parent_id = 0;
     $forum_title = $forum_content = $forum_edit_reason = '';
 
@@ -422 +422 @@
         remove_filter( 'bbp_edit_forum_pre_content', 'bbp_filter_kses', 30 );
     }
+
+    // Get forum author
+    $forum_author = bbp_get_forum_author_id( $forum_id );
 
     /** Forum Parent ***********************************************************/
@@ -519 +522 @@
         'post_content' => $forum_content,
         'post_status'  => $forum_status,
-        'post_parent'  => $forum_parent_id
+        'post_parent'  => $forum_parent_id,
+        'post_author'  => $forum_author
     ) );
 
@@ -532 +536 @@
         do_action( 'bbp_edit_forum', array(
             'forum_id'           => $forum_id,
-            'post_parent'        => $forum_parent_id,
-            'forum_author'       => $forum->post_author,
+            'post_parent'        => $forum_data['post_parent'],
+            'forum_author'       => $forum_data['post_author'],
             'last_topic_id'      => 0,
             'last_reply_id'      => 0,

branches/2.6/src/includes/replies/functions.php

@@ -463 +463 @@
         /** Update counts, etc... *********************************************/
 
-        do_action( 'bbp_new_reply', $reply_id, $topic_id, $forum_id, $anonymous_data, $reply_author, false, $reply_to );
+        do_action( 'bbp_new_reply', $reply_id, $topic_id, $forum_id, $anonymous_data, $reply_data['post_author'], false, $reply_to );
 
         /** Additional Actions (After Save) ***********************************/
@@ -744 +744 @@
 
         // Update counts, etc...
-        do_action( 'bbp_edit_reply', $reply_id, $topic_id, $forum_id, $anonymous_data, $reply_author , true, $reply_to );
+        do_action( 'bbp_edit_reply', $reply_id, $topic_id, $forum_id, $anonymous_data, $reply_data['post_author'], true, $reply_to );
 
         /** Revisions *********************************************************/

branches/2.6/src/includes/topics/functions.php

@@ -374 +374 @@
         /** Update counts, etc... *********************************************/
 
-        do_action( 'bbp_new_topic', $topic_id, $forum_id, $anonymous_data, $topic_author );
+        do_action( 'bbp_new_topic', $topic_id, $forum_id, $anonymous_data, $topic_data['post_author'] );
 
         /** Additional Actions (After Save) ***********************************/
@@ -665 +665 @@
 
         // Update counts, etc...
-        do_action( 'bbp_edit_topic', $topic_id, $forum_id, $anonymous_data, $topic_author , true /* Is edit */ );
+        do_action( 'bbp_edit_topic', $topic_id, $forum_id, $anonymous_data, $topic_data['post_author'], true /* Is edit */ );
 
         /** Revisions *********************************************************/