[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