[wp-trac] [WordPress Trac] #15961: WP_Comments_List_Table causes get_pending_comments_num to run a lame query
WordPress Trac
wp-trac at lists.automattic.com
Thu Dec 23 10:15:10 UTC 2010
#15961: WP_Comments_List_Table causes get_pending_comments_num to run a lame query
--------------------------+----------------------------
Reporter: westi | Owner: westi
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future Release
Component: Comments | Version: 3.1
Severity: normal | Keywords: 3.2-early
--------------------------+----------------------------
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
}}}
--
Ticket URL: <http://core.trac.wordpress.org/ticket/15961>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list