[wpmu-trac] [WordPress MU Trac] #867: user can not change blog admin email address

WordPress MU Trac wpmu-trac at lists.automattic.com
Mon Jan 26 16:13:39 GMT 2009


#867: user can not change blog admin email address
----------------------------+-----------------------------------------------
 Reporter:  recoilfx        |       Owner:  somebody
     Type:  defect          |      Status:  new     
 Priority:  normal          |   Milestone:          
Component:  Administration  |     Version:          
 Severity:  normal          |    Keywords:  2.6.5   
----------------------------+-----------------------------------------------
 user can not change blog admin email address.  When inputting a new email
 address, confirmation email is not sent.  update_option_new_admin_email
 hook is not being fired.  The problem stems from option.php 's
 update_option loop beginning line 62.

 Here is how the function is broken.
 1.) options.php has filter with a whitelist index($options) of all the
 $_POST varaibles
 2.) mu.php also has a specific whitelist filter, but it remvoves
 admin_email and adding new_admin_email in $options.
 3.) options.php iterates the $_POST variable using the $options index.
 Because of mu.php, it ignores admin_email, and tries to new_admin_email.
 4. Because $_POST['new_admin_email'] is null, update_option terminates,
 and the hook for update_new_admin_email is not fired, so email is never
 changed.

 Here is my fix, replace line 67 at options.php with this :
 if($option=='new_admin_email')
 {
  update_option($option, $_POST['admin_email']);
 }else{
  update_option($option, $value);
 }

 This is for wp-mu. 2.6.5

-- 
Ticket URL: <http://trac.mu.wordpress.org/ticket/867>
WordPress MU Trac <http://mu.wordpress.org/>
WordPress Multiuser


More information about the wpmu-trac mailing list