[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