[wp-trac] [WordPress Trac] #35791: WP_Site_Query class

WordPress Trac noreply at wordpress.org
Wed Apr 13 20:19:12 UTC 2016


#35791: WP_Site_Query class
-------------------------------------+------------------------
 Reporter:  spacedmonkey             |       Owner:
     Type:  enhancement              |      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):

 So I have gone through the code and implemented the class where I think it
 would be useful.
 At some point this patch should be broken up into two. One for
 wp_site_query and another for the implementation of it in core functions.

 Points of interest
 - orderby CHAR_LENGTH on domain and path. Need a solution for this
 - Bootstrap loader, am I loading it to late / soon.
 - Should update_site_cache do more, warm other caches? Should it have a
 hook?
 - Comments need a lot of work.
 - This query object should be used to create a sites endpoint in the api

 Not sure that this is secure / best way of doing this
 {{{#!php
 implode( "', '", $wpdb->_escape( $this->query_vars['path__not_in'] ) ) .
 "' )
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/35791#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list