[wp-trac] [WordPress Trac] #15691: Network admin should have its own settings API
WordPress Trac
wp-trac at lists.automattic.com
Mon Sep 17 23:20:27 UTC 2012
#15691: Network admin should have its own settings API
-----------------------------+-----------------------------
Reporter: joostdevalk | Owner:
Type: feature request | Status: new
Priority: normal | Milestone: Future Release
Component: Network Admin | Version:
Severity: normal | Resolution:
Keywords: has-patch |
-----------------------------+-----------------------------
Comment (by convissor):
boonebgorges:
The patch moves a large chunk of code from {{{wp-admin/options.php}}} to a
new function called {{{process_settings()}}} in {{{wp-
admin/includes/misc.php}}}. The patch also modifies the code that's been
moved. While both things are good ideas, doing them in one patch obscures
what changes are being made inside that block of moved code.
I have pretty much no say in what happens in core, but may I suggest
making two patches? One that moves the existing code into the function.
Then another that makes the rest of the changes.
Question: this new Network Settings API will use the same API (function
calls, HTML, etc) as the existing Settings API, right? So plugins will be
able to create one settings screen that'll work under both regular and
multisite installs. If so, a hearty +1 from me.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/15691#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list