Duplicate comment check is too strict

[scribu]

The duplicate comment detection mechanism doesn't allow duplicate comments on the same post, even from the post author. That's not good.

The duplicate comment check should only block the new comment if there is no other comment posted between the new comment and the old comment.

[Denis-de-Bernardy]

It would depend on the purpose.

As I understand things, its main purpose is to make sure that double clicking the publish comment button won't output two comments. Adding a unique ID for each comment, that gets passed into the comment form, and that expires upon the comment being saved, would easily defeat this (it would probably cut a lot of spam, too). The duplicate-comment check could then be rendered obsolete, in favor of redirecting to the newly published comment itself.

[Denis-de-Bernardy]

punting to future, pending patch

[scribu]

Replying to Denis-de-Bernardy:

As I understand things, its main purpose is to make sure that double clicking the publish comment button won't output two comments.

Yes, I think that's it's main purpose too.

Adding a unique ID for each comment, that gets passed into the comment form, and that expires upon the comment being saved, would easily defeat this (it would probably cut a lot of spam, too). The duplicate-comment check could then be rendered obsolete, in favor of redirecting to the newly published comment itself.

The unique ID could use a mechanism similar to post-new.php that assigns a neggative ID to a post, before it is saved.

Also, this plugin implements a similar technique: ​http://wordpresssupplies.com/wordpress-plugins/captcha-free/

[Denis-de-Bernardy]

it might interfere with caching plugins however.

[scribu]

Replying to Denis-de-Bernardy:

it might interfere with caching plugins however.

Not if you add it dynamically through JS. But then users with JavaScript disabled can't comment.

Therefore, I still think it's a more robust ideea to check for intermediate comments before blocking.

[caesarsgrunt]

Why would you ever want to post two identical comments, even if there were other comments between them?

[josephscott]

I think this should be closed as invalid. Providing a way to get around the duplicate comment check is asking for problems.

[skithund]

How about making the duplicate check pluggable, so people (like me) could get around it with a plugin if necessary?

[skithund]

This has been fixed by making duplicate comment check pluggable.

From #12188:
(In [13434]) Add comment_duplicate_trigger hook that is fired when a duplicate comment is detected. Fixes #12188

[scribu]

You can use 'comment_duplicate_trigger' only to customize the message that the user sees, after it has been decided that the comment is a dup.

Related: #21609

[SergeyBiryukov]

#27138 was marked as a duplicate.

[rachelbaker]

This needs a patch if we are going to add a filter here.

[boonebgorges]

I agree that the duplicate check is probably stricter than it really needs to be. It's perfectly legitimate for, say, a post author to reply to multiple comments on a thread by saying "Yes" or "Thanks" or whatever.

That said, I don't know how to allow that without also allowing the double-click problem that the duplicate check was originally intended to prevent. Let's put in a filter and let people adjust as they see fit.

[boonebgorges]

In 34536:

Introduce 'duplicate_comment_id' filter.

wp_allow_comment() disallows a comment if it matches a comment on the same
post with the same content, author email, and parent. This new filter allows
developers to circumvent or modify this logic, making the duplicate check
more or less lenient, as they see fit.

Fixes #9775.