[wp-trac] [WordPress Trac] #53228: Unspamming a comment should respect `comment_previously_approved` setting

WordPress Trac noreply at wordpress.org
Tue May 18 21:28:27 UTC 2021


#53228: Unspamming a comment should respect `comment_previously_approved` setting
--------------------------+-----------------------------
 Reporter:  r-a-y         |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Comments      |    Version:
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 On the "Settings > Discussion" page, the "Comment author must have a
 previously approved comment" setting (`comment_previously_approved`)
 allows a user's comments to be automatically set to `approved` status
 (a.k.a. `comment_approved = 1`).

 However, if a comment from a previously-approved author is incorrectly-
 marked as spam (either internally via `check_comment()` or from an anti-
 spam plugin like Akismet) and later, a moderator marks the comment as "Not
 spam", this comment is not automatically approved. Instead, the comment is
 placed in the moderation queue (`comment_approved = 0`).

 WordPress makes this determination during unspamming here:
 https://github.com/WordPress/WordPress/blob/f4631a98f50f6275d0284527cf9fd7009e439e55
 /wp-includes/comment.php#L1717-L1722

 Specifically, if the comment has a previously-trashed status from the
 `_wp_trash_meta_status` meta value, WordPress will use it. Otherwise, an
 unspammed comment's status will be marked as `pending` or `0`.

 Our users believe this is a UX issue. Previously-approved authors should
 not have their comments placed in `pending` after the comments are
 unspammed.

 We're working around this issue at the moment, but thought we would ask if
 this is a bug or if this is by design.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53228>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list