[wp-trac] [WordPress Trac] #35791: WP_Site_Query class
WordPress Trac
noreply at wordpress.org
Fri Apr 15 15:39:04 UTC 2016
#35791: WP_Site_Query class
-------------------------------------+------------------------
Reporter: spacedmonkey | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 4.6
Component: Networks and Sites | Version: 4.4
Severity: normal | Resolution:
Keywords: has-patch needs-testing | Focuses: multisite
-------------------------------------+------------------------
Comment (by spacedmonkey):
Updated my patch with better starting params. Also public params wasn't
working :( and I fixed it.
I had a look into support getting site by blogname but couldn't think of a
way of I could do it in a scalable way. If you get requesting a list of
1000 sites, I don't want to do 1000 switch to site / joins to get the
blogname. It the blogname was cached in another table, such as wp_blogs
this would be different.
@flixos90 good idea about making the `wp_get_sites` backwards-compatible.
I think it should be an option to select what format the function returns.
As for count, I don't agree with your change. This class is based on
wp_comment_query, which does have count. I think this shouldn't change.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/35791#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list