Skip to:
Content

bbPress.org


Ignore:
Timestamp:
09/21/2021 01:11:47 AM (2 months 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 branches/2.6, for 2.6.7.

Fixes #3418.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/2.6/src/includes/topics/functions.php

    r7192 r7208  
    250250    /** Topic Duplicate *******************************************************/
    251251
    252     if ( ! bbp_check_for_duplicate( array( 'post_type' => bbp_get_topic_post_type(), 'post_author' => $topic_author, 'post_content' => $topic_content, 'anonymous_data' => $anonymous_data ) ) ) {
     252    $dupe_args = array(
     253        'post_type'      => bbp_get_topic_post_type(),
     254        'post_author'    => $topic_author,
     255        'post_content'   => $topic_content,
     256        'post_parent'    => $forum_id,
     257        'anonymous_data' => $anonymous_data
     258    );
     259
     260    if ( ! bbp_check_for_duplicate( $dupe_args ) ) {
    253261        bbp_add_error( 'bbp_topic_duplicate', __( '<strong>Error</strong>: Duplicate topic detected; it looks as though you&#8217;ve already said that.', 'bbpress' ) );
    254262    }
Note: See TracChangeset for help on using the changeset viewer.