[wp-trac] [WordPress Trac] #11773: weird looking logic in ms-edit.php vs ms-functions.php

WordPress Trac wp-trac at lists.automattic.com
Wed Jan 6 18:49:05 UTC 2010


#11773: weird looking logic in ms-edit.php vs ms-functions.php
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:              
     Type:  defect (bug)       |      Status:  new         
 Priority:  normal             |   Milestone:  3.0         
Component:  Multisite          |     Version:  3.0         
 Severity:  normal             |    Keywords:  dev-feedback
-------------------------------+--------------------------------------------
 In ms-edit.php, you can only update the site admin's email if it's a valid
 email:

 {{{
                 if( is_email( $_POST['admin_email'] ) )
                         update_site_option( "admin_email",
 $_POST['admin_email'] );
 }}}

 But in ms-functions.php, there is a check in newblog_notify_siteadmin(),
 that goes:

 {{{
         $email = get_site_option( 'admin_email' );
         if( is_email($email) == false )
                 return false;
 }}}

 which seems to indicate that a MU install can live without a site admin
 email. so, question... is it possible, or not?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11773>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list