Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#20123 closed defect (bug) (fixed)

"Approve" should not be a bulk action when viewing spam comments

Reported by: nacin Owned by: russellwwest
Priority: normal Milestone: 3.4
Component: Comments Version:
Severity: major Keywords: has-patch
Cc: Focuses:

Description

Approve, Quick Edit, and Edit action links are all hidden on edit-comments.php?comment_status=spam. The only action links are Not Spam and Delete Permanently. These are both bulk actions, but there is also an Approve bulk action.

No word on whether it works. Probably does. It should be removed for consistency, given that Restore simply returns it to the queue.

On the Trash view, there is no Approve bulk action, only Restore and Delete Permanently.

Attachments (1)

class-wp-comments-list-table.php.diff (749 bytes ) - added by russellwwest 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 in reply to: ↑ description @solarissmoke
14 years ago

Replying to nacin:

No word on whether it works. Probably does. It should be removed for consistency, given that Restore simply returns it to the queue.

Not always - the result of hitting "Not Spam" depends on what the comment status was before it was marked as spam (using _wp_trash_meta_status).

This means that doing a bulk action of "Not Spam" can cause those comments to end up in different places (approved or pending). Maybe that's why the "Approve" action is also there?

Version 0, edited 14 years ago by solarissmoke (next)

#2 @russellwwest
14 years ago

  • Owner set to russellwwest
  • Status newaccepted

#3 @russellwwest
14 years ago

  • Keywords has-patch added

#4 @imshashank
14 years ago

  • Severity normalmajor

the patch is working fine.... tested and verified..

#5 @ramiy
14 years ago

Related: #17275

#6 @duck_
14 years ago

  • Resolutionfixed
  • Status acceptedclosed

In [20533]:

Remove "Approve" bulk action when viewing spam comments as it isn't shown in the row actions. Props russellwwest. Fixes #20123.

Note: See TracTickets for help on using tickets.