[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:
http://blog.ftwr.co.uk/archives/2005/11/03/windows-wordpress-toolbox/
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