WordPress.org

Make WordPress Core

Opened 7 years ago

Last modified 4 months ago

#7051 new enhancement

Delete blacklisted comments rather than mark them as spam

Reported by: donncha Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.5.1
Component: Comments Keywords: has-patch dev-feedback
Focuses: Cc:

Description

Comments that are blacklisted because of the Blacklist keys on options-discussion.php should be deleted immediately instead of being marked as spam. Patch attached.

Attachments (3)

comment.diff (1.1 KB) - added by donncha 7 years ago.
Patch for wp-includes/comment.php
7051.diff (1.6 KB) - added by solarissmoke 4 years ago.
Place blacklisted comments in the trash
7051.2.diff (1.7 KB) - added by MikeHansenMe 4 months ago.
Refreshed

Download all attachments as: .zip

Change History (23)

@donncha7 years ago

Patch for wp-includes/comment.php

comment:1 @mdawaffe7 years ago

This seems like a pretty big change. Can you say why blacklist matching comments should be deleted?

I have "casino" in my blacklist, but if I ever write a post about going to Vegas, I don't want all the comments to get deleted.

comment:2 @Viper007Bond7 years ago

Yeah, I think spam is best. Allows for recovery.

comment:3 @donncha7 years ago

This is something Photomatt and Markr wanted. Markr brought it up in conversation with a blogger who is getting lots of stupid comments from one particular user. He added that person's email address or IP into the blacklist expecting the comments to be deleted but instead he had to read the comments in the Akismet spam page and delete them.

They weren't submitted to Akismet so the spam page is probably the wrong page for these comments to be left in.

This is a result of confusion over what the blacklist does. Is it a spam list or stupid list? :)

comment:4 @Viper007Bond7 years ago

Perhaps a new status is in order or something.

It just seems like not having them be instantly deleted would be a good idea incase there was a false positive as mdawaffe mentioned (Vegas).

comment:5 @ryan7 years ago

  • Milestone changed from 2.5.2 to 2.9

Milestone 2.5.2 deleted

comment:6 @Denis-de-Bernardy6 years ago

  • Component changed from General to Comments
  • Owner anonymous deleted

comment:7 @Denis-de-Bernardy6 years ago

  • Keywords needs-patch close added; has-patch removed
  • Milestone changed from 2.9 to Future Release
  • Type changed from defect (bug) to enhancement

suggesting wontfix, personally.

comment:8 @solarissmoke4 years ago

  • Keywords needs-patch removed

comment:9 follow-up: @nacin4 years ago

Perhaps now that we have Trash, that could be where they end up?

comment:10 in reply to: ↑ 9 @Viper007Bond4 years ago

Replying to nacin:

Perhaps now that we have Trash, that could be where they end up?

Strong +1

Allows recovery still and is a better fit. The comments from people on the blacklist aren't necessarily spam, they're just unwanted comments.

@solarissmoke4 years ago

Place blacklisted comments in the trash

comment:11 @solarissmoke4 years ago

  • Keywords has-patch added; close removed

comment:12 @chriscct79 months ago

  • Keywords dev-feedback added

Does someone from core want to quickly review the idea?
Essentially, blacklisted comments are moved to spam instead of trash. Its a simple 2 line patch file, so the only major thing really on this is whether or not blacklisted comments should go to trash or spam (where they currently go). I'm in favor of trash, as blacklisted comments can be non-spam, and they can be unintentionally flagged on the blacklist, but many users might think of all the comments in spam are spam and simply empty the spam folder without thinking.

@MikeHansenMe4 months ago

Refreshed

comment:13 @MikeHansenMe4 months ago

I think this makes good sense. Easy patch as well. I do see it causing any issues.

comment:14 @helen4 months ago

What happens when somebody disables trash?

comment:15 @MikeHansenMe4 months ago

If they are being set to trash and the trash is disabled they would be auto deleted. I am not sure if this is something we should account for or expected behavior.

comment:16 follow-up: @helen4 months ago

Seems like it would be bad if comments started getting auto-deleted now, just because of the destructive/unrecoverable change in behavior.

comment:17 in reply to: ↑ 16 ; follow-up: @Denis-de-Bernardy4 months ago

Replying to helen:

Seems like it would be bad if comments started getting auto-deleted now, just because of the destructive/unrecoverable change in behavior.

If someone bothers to disable the trash, wouldn't you agree that this person actually wants that destructive / unrecoverable behavior?

comment:18 in reply to: ↑ 17 ; follow-up: @helen4 months ago

Replying to Denis-de-Bernardy:

If someone bothers to disable the trash, wouldn't you agree that this person actually wants that destructive / unrecoverable behavior?

I would not expect an upgrade of WordPress to suddenly cause blacklisted comments to be outright deleted rather than in some moderated state where I could recover them if I wanted to.

comment:19 in reply to: ↑ 18 @Denis-de-Bernardy4 months ago

Replying to helen:

I would not expect an upgrade of WordPress to suddenly cause blacklisted comments to be outright deleted rather than in some moderated state where I could recover them if I wanted to.

I'll let you do what's right then:

https://core.trac.wordpress.org/ticket/7051#comment:7

comment:20 @helen4 months ago

Stepping back a little further to look at the wider view, would it perhaps make more sense to note in comment meta that the comment is in spam because of the blacklist? If it does, then I think we can close this (as documentation that we've had this discussion) and make a new ticket.

Note: See TracTickets for help on using tickets.