Make WordPress Core

Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#15961 closed defect (bug) (fixed)

WP_Comments_List_Table causes get_pending_comments_num to run a lame query

Reported by: westi Owned by: westi
Priority: normal Milestone: 3.2
Component: Comments Version: 3.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description (last modified by westi)

In WP_Comments_List_Table::prepare_items we call get_comments to get the comments for the current view.

We then loop through the comments to build an array of post_ids.

We then call get_pending_comments_num with this array of ids.

At no point does prepare_items or get_pending_comments_num do any thing to the array to ensure uniqueness.

Therefore we run queries like this:

SELECT comment_post_ID, COUNT(comment_ID) as num_comments FROM wp__comments WHERE comment_post_ID IN ( '627', '627', '1', '5', '1', '20', '40', '30', '20', '10', '10', '627', '627', '627', '627', '627', '627', '627', '627', '627' ) AND comment_approved = '0' GROUP BY comment_post_ID

Attachments (1)

15961.patch (520 bytes ) - added by SergeyBiryukov 16 years ago.

Download all attachments as: .zip

Change History (7)

#1 @westi
16 years ago

  • Description modified (diff)

#2 @SergeyBiryukov
16 years ago

  • Keywords has-patch added

#3 @ramiy
15 years ago

Related #17275 (?)

#4 @SergeyBiryukov
15 years ago

  • Keywords 3.2-early removed
  • Milestone Future Release3.3

#5 @westi
15 years ago

  • Resolutionfixed
  • Status newclosed

Was fixed in [18041]

#6 @SergeyBiryukov
15 years ago

  • Milestone 3.33.2
Note: See TracTickets for help on using tickets.