[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