[wp-trac] [WordPress Trac] #31148: Extend wp_get_sites() to support queries on domain and path

WordPress Trac noreply at wordpress.org
Fri Jan 30 02:58:43 UTC 2015


#31148: Extend wp_get_sites() to support queries on domain and path
-------------------------------------------------+-------------------------
 Reporter:  earnjam                              |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Networks and Sites                   |  Review
 Severity:  normal                               |     Version:
 Keywords:  dev-feedback needs-testing has-      |  Resolution:
  patch                                          |     Focuses:  multisite
-------------------------------------------------+-------------------------
Changes (by earnjam):

 * keywords:   => dev-feedback needs-testing has-patch


Comment:

 31148.diff is a first stab at adding parameters for domain, path and a
 boolean to allow the function to run against large networks.

 If it's going to be used for some of these direct queries on the
 `wp_blogs` table, or replace `domain_exists()`, then it needs to have the
 ability to search large networks when required. This patch keeps the
 original functionality of returning an empty array on a large network by
 default unless you explicitly pass it a true value.

 One thing I did notice is that I think `wp_is_large_network()` is only
 checking the size of the current network, so If you run this function from
 within a network with > 10,000 sites, but are trying to search against a
 different small network, it would return an empty array. I guess the
 boolean param I added would allow you to bypass that, but something to
 think about.

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


More information about the wp-trac mailing list