Changeset 7207 for trunk/src/includes/forums/functions.php

Timestamp:

09/21/2021 01:08:32 AM (3 years ago)

Author:

johnjamesjacoby

Message:

Content: add post_parent to forum & topic duplicate checks.

This change limits the scope of "duplicate content" to its parent context, meaning that replies must be to the same topic, topics must be in the same forum, and forums must be in the same sub-forum (or root) to be considered a duplicate.

This better implements the spirit of this feature which is to prevent accidental/unintended duplicates, and is not intended to simply prevent globally similar content by the same author.

Props r-a-y, johnjamesjacoby.

In trunk, for 2.7.0.

See #3418.

File:

• trunk/src/includes/forums/functions.php (modified) (1 diff)

trunk/src/includes/forums/functions.php

@@ -231 +231 @@
     /** Forum Duplicate *******************************************************/
 
-    if ( ! bbp_check_for_duplicate( array( 'post_type' => bbp_get_forum_post_type(), 'post_author' => $forum_author, 'post_content' => $forum_content, 'anonymous_data' => $anonymous_data ) ) ) {
+    $dupe_args = array(
+        'post_type'      => bbp_get_forum_post_type(),
+        'post_author'    => $forum_author,
+        'post_content'   => $forum_content,
+        'post_parent'    => $forum_parent_id,
+        'anonymous_data' => $anonymous_data
+    );
+
+    if ( ! bbp_check_for_duplicate( $dupe_args ) ) {
         bbp_add_error( 'bbp_forum_duplicate', __( '<strong>Error</strong>: This forum already exists.', 'bbpress' ) );
     }