Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#19977 closed defect (bug) (fixed)

Don't prevent the Edit Comment page from loading for comments marked as spam

Reported by: josephscott's profile josephscott Owned by: nacin's profile nacin
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.3.1
Component: Comments Keywords: has-patch
Focuses: Cc:

Description (last modified by josephscott)

r19296 made it so that loading the Edit Comment page returned an error message for comments marked as spam, specifically - http://core.trac.wordpress.org/changeset/19296/trunk/wp-admin/comment.php - which came in from #18340

What I'm suggesting is a revert of just http://core.trac.wordpress.org/changeset/19296/trunk/wp-admin/comment.php so that the edit action row link still isn't there, but if the Edit Comment page will still function for comments marked as spam.

The specific issue that came up for us is that the 'Akismet History' feature became effectively disabled for comments marked as spam because of r19296. The history details were still there, but users had no way of accessing the information for comments marked as spam.

Chatted with Nacin about this issue and he originated the suggestion to revert http://core.trac.wordpress.org/changeset/19296/trunk/wp-admin/comment.php as middle ground solution.

Attachments (1)

19977.diff (615 bytes) - added by josephscott 13 years ago.

Download all attachments as: .zip

Change History (4)

@josephscott
13 years ago

#1 @josephscott
13 years ago

  • Description modified (diff)

#2 @nacin
13 years ago

  • Component changed from Administration to Comments
  • Milestone changed from Awaiting Review to 3.4

Sounded like both ryan and I were fine with this in IRC.

#3 @nacin
13 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [19856]:

Allow comment.php?action=editcomment to be accessible for a comment that is spam. props josephscott, fixes #19977, see #18340.

Note: See TracTickets for help on using tickets.