[wp-trac] [WordPress Trac] #31985: WP_Network class

WordPress Trac noreply at wordpress.org
Wed Apr 22 04:09:19 UTC 2015


#31985: WP_Network class
--------------------------------+-----------------------------
 Reporter:  johnjamesjacoby     |       Owner:
     Type:  enhancement         |      Status:  new
 Priority:  normal              |   Milestone:  Future Release
Component:  Networks and Sites  |     Version:  3.0
 Severity:  normal              |  Resolution:
 Keywords:  dev-feedback        |     Focuses:  multisite
--------------------------------+-----------------------------
Changes (by jeremyfelt):

 * milestone:  Awaiting Review => Future Release


Comment:

 `WP_Network` will be easy to nail down some basics for as `$current_site`
 isn't too complex and we don't do much with networks inside core. It will
 be hard to nail down because the pieces it touches are all seemingly tough
 to change.

 I think this is a great start.

 A few thoughts (some in reaction, some in general):

 * I think we'll benefit from planning on a `WP_Network_Query` immediately
 and moving some of the (currently static) methods there. (JJJ planned
 ahead)
 * I don't doubt we can do this in concert with `WP_Site`. This will make
 things like "how do we populate `$current_site->blog_id` somewhat easier
 to grasp.
 * `add()` is empty right now, we could probably use what's done in
 `populate_network()` as a base?
 * Is there a reason to separate `ms_set_switched_stacks()` from
 `ms_set_current_globals()`?

 It would also be worth really staring at what test coverage is missing in
 advance so that adding `WP_Network` is as painless as possible.

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


More information about the wp-trac mailing list