[wp-trac] Re: [WordPress Trac] #9211: Recent comments widget with "private" entries

WordPress Trac wp-trac at lists.automattic.com
Sun Feb 22 06:35:38 GMT 2009

#9211: Recent comments widget with "private" entries
 Reporter:  menelicte     |       Owner:                 
     Type:  defect (bug)  |      Status:  new            
 Priority:  lowest        |   Milestone:  Future Release 
Component:  Widgets       |     Version:  2.7            
 Severity:  trivial       |    Keywords:  widget comments

Comment(by DD32):

 I meant to link to it.. but forgot,
 http://codex.wordpress.org/Using_Subversion (However, it only mentions
 Command line..), Westi's post he linked to is good:

 Looking at the quoted code a bit closer however:
  * Joins are expensive, It could be more efficient to do a get_post() and
 a check that the post_status is set to 'publish' or 'inherit'(attachments)
  * Pretty sure WP has a function hidden somewhere to generate the WHERE
 case for a users private drafts & all that.

 I'd be in favour of the first method however, Due to the join benefiting a
 small amount of people, increasing query time for something that is used
 on very active blogs should be avoided IMO.. The posts table is queried
 during the loop for the post title regardless, so a get_post() in the loop
 wouldn't affect the speed.

 I might whip up a patch based on that at some point if that idea seems
 acceptable to others (Given the closure of the other mentioned ticket)

Ticket URL: <http://core.trac.wordpress.org/ticket/9211#comment:3>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software

More information about the wp-trac mailing list