[wp-trac] [WordPress Trac] #18088: Network Admin Email setting in wp-admin/network/settings.php fails silently

[WordPress Trac]

#18088: Network Admin Email setting in wp-admin/network/settings.php fails silently
--------------------------------+----------------------------------------
 Reporter:  kawauso             |       Owner:
     Type:  defect (bug)        |      Status:  reopened
 Priority:  normal              |   Milestone:  Future Release
Component:  Networks and Sites  |     Version:  3.2
 Severity:  normal              |  Resolution:
 Keywords:  needs-patch         |     Focuses:  administration, multisite
--------------------------------+----------------------------------------
Changes (by jeremyfelt):

 * keywords:  has-patch => needs-patch
 * status:  closed => reopened
 * resolution:  worksforme =>
 * milestone:   => Future Release


Comment:

 I am very surprised, but this seems to still be valid. :)

 In [21993], this chunk of code was moved so that it would be processed by
 `sanitize_option()`. While that changeset does not explicitly handle
 `admin_email`, code inside `sanitize_option()` does.

 If an invalid email address is entered in the network admin email input
 box and the settings are saved, the network admin email does not change
 and the message at the top of the screen is green and says "Options
 saved." Lies!

 `sanitize_option()` does store the error into `add_settings_error()`, but
 nothing on the network settings page does anything to present that to the
 user. I haven't dug any further than that.

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