[wpmu-trac] Re: [WordPress MU Trac] #429: Bug when updating site options

[WordPress MU Trac]

#429: Bug when updating site options
------------------------+---------------------------------------------------
 Reporter:  hovenko     |        Owner:  somebody
     Type:  defect      |       Status:  new     
 Priority:  highest     |    Milestone:  WPMU 1.0
Component:  component1  |      Version:  1.0     
 Severity:  critical    |   Resolution:          
 Keywords:  has-patch   |  
------------------------+---------------------------------------------------
Comment (by robcwright):

 downloaded and installed the above mentioned patch and received this error
 message when attempting to load any page of the site:

 Index: .
 =================================================================== --- .
 (revision 63) +++ . (working copy) @@ -279,7 +279,7 @@ $exists =
 $wpdb->get_row("SELECT meta_value FROM $wpdb->sitemeta WHERE meta_key =
 '$safe_key' AND site_id = '{$wpdb->siteid}'"); if ( null !== $exists ) {//
 If we already have it - update_site_option( $key, $value ); +
 //update_site_option( $key, $value ); return false; } @@ -902,6 +902,22 @@
 return false; } +function wpmu_get_illegal_names() { + $illegal_names =
 get_site_option("illegal_names"); + + if (!$illegal_names) { +
 $illegal_names = array( + "www", "web", + "root", "admin", + "main",
 "invite", + "administrator", + ); + update_site_option("illegal_names",
 $illegal_names); + } + + return $illegal_names; +} + function
 wpmu_validate_user_signup($user_name, $user_email) { global $wpdb,
 $current_site; @@ -919,11 +935,8 @@ $errors->add('user_name', __("Only
 letters and numbers allowed")); } - $illegal_names = get_site_option(
 "illegal_names" ); - if( is_array( $illegal_names ) == false ) { -
 $illegal_names = array( "www", "web", "root", "admin", "main", "invite",
 "administrator" ); - add_site_option( "illegal_names", $illegal_names ); -
 } + $illegal_names = wpmu_get_illegal_names(); + if( in_array( $user_name,
 $illegal_names ) == true ) { $errors->add('user_name', __("That username
 is not allowed")); } @@ -1006,11 +1019,7 @@ $blog_title = strip_tags(
 $blog_title ); $errors = new WP_Error(); - $illegal_names =
 get_site_option( "illegal_names" ); - if( $illegal_names == false ) { -
 $illegal_names = array( "www", "web", "root", "admin", "main", "invite",
 "administrator" ); - add_site_option( "illegal_names", $illegal_names ); -
 } + $illegal_names = wpmu_get_illegal_names(); if ( empty( $blog_id ) )
 $errors->add('blog_id', __("Please enter a blog name"));
 Fatal error: Call to undefined function get_site_option() in
 /homepages/46/d202194328/htdocs/mysite/wp-settings.php on line 210

-- 
Ticket URL: <http://trac.mu.wordpress.org/ticket/429#comment:1>
WordPress MU Trac <http://mu.wordpress.org/>
WordPress Multiuser