[wp-trac] [WordPress Trac] #30594: New function to network enable a theme

WordPress Trac noreply at wordpress.org
Thu Jan 21 00:27:07 UTC 2016


#30594: New function to network enable a theme
-------------------------------------+-------------------------------------
 Reporter:  igmoweb                  |       Owner:  igmoweb
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  4.5
Component:  Themes                   |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch has-unit-      |     Focuses:  administration,
  tests                              |  multisite
-------------------------------------+-------------------------------------
Changes (by jeremyfelt):

 * owner:   => igmoweb
 * status:  new => assigned
 * milestone:  Future Release => 4.5


Comment:

 Thanks @igmoweb, this is looking good.

 Some minor adjustments:
 * I think we can follow how `WP_Theme` is setup in general and use
 `WP_Theme::network_enable_theme()` and `WP_Theme::network_disable_theme()`
 directly without providing the `wp_network_*` wrapper functions.
 * The additions to `class-wp-theme.php` need to be formatted per core code
 style. It looks like spacing is a bit off in the methods and the `if`
 statements need braces.
 * Let's split the tests up into 4 parts: network enable a single theme,
 network enable an array of themes, network disable a single theme, and
 network disable an array of themes. No need to

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


More information about the wp-trac mailing list