[wp-trac] [WordPress Trac] #10329: sort_by and count_limit options for wp_list_authors

WordPress Trac wp-trac at lists.automattic.com
Tue Mar 2 21:51:27 UTC 2010


#10329: sort_by and count_limit options for wp_list_authors
-----------------------------+----------------------------------------------
 Reporter:  takaitra         |       Owner:  westi      
     Type:  feature request  |      Status:  reviewing  
 Priority:  normal           |   Milestone:  3.1        
Component:  Template         |     Version:             
 Severity:  normal           |    Keywords:  needs-patch
-----------------------------+----------------------------------------------

Comment(by Denis-de-Bernardy):

 Replying to [comment:23 takaitra]:
 > Removing the NULL check on the post_type causes authors with zero posts
 to be incorrectly excluded. I realize there is no NULL post_type but the
 check is necessary due to the joins.

 I think that's covered by the $min_count = min(1, $min_count) statement
 that I added

 > How do you suggest the GROUP BY be removed while retaining the
 "optioncount" and "number" argument functionality?

 mysql is quite loose on the SQL spec and doesn't actually require a group
 by. (it groups by automatically.) when there *is* an explicit a group by,
 it also sorts, which introduces overhead.

 > I read through miqrogroove's patch in #11914. I see how User objects
 could be queried for first and last name. If displaying full name, we
 would then need to sort in PHP as opposed to in SQL.

 right. so let's get #11914 in first, and revisit this once it's checked
 in.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10329#comment:24>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list