[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