[wpmu-trac] [WordPress MU Trac] #184: wpmu_create_blog Sql Bug

[WordPress MU Trac]

#184: wpmu_create_blog Sql Bug
------------------------+---------------------------------------------------
 Reporter:  abarker     |       Owner:  somebody
     Type:  defect      |      Status:  new     
 Priority:  normal      |   Milestone:  WPMU 1.0
Component:  component1  |     Version:  1.0     
 Severity:  major       |    Keywords:          
------------------------+---------------------------------------------------
 There appears to be an issue with wpmu_create_blog throwing some SQL
 Errors that I have listed below.  The problem appears to be with the
 insert_blog function that is called in wpmu_create_blog around line 1127
 of the wpmu-functions.php file.  On investigation the insert_blog function
 calls refresh_blog_details on line 1167 which calls get_blog_details that
 is tring to pull database values for the blog that dont exist yet as the
 blog has not been created!  This looks like a slight flaw in logic and
 apperently for some reason does not show up when accounts and blogs are
 normally created, however if blogs are created via 3rd party code like a
 plug-in the error is visiable.  I am not sure on a good fix however I am
 not sure that the insert_blog function needs to cal refresh_blog_details
 on line 1167.

 Below i have included an error as well as the code used to create the
 account and then blog.


 {{{
 WordPress database error: [Table 'weblogs.wp_3_options' doesn't exist]
 SELECT option_value FROM wp_3_options WHERE option_name = 'blogname' LIMIT
 1

 WordPress database error: [Table 'weblogs.wp_3_options' doesn't exist]
 SELECT option_value FROM wp_3_options WHERE option_name = 'siteurl' LIMIT
 1

 WordPress database error: [Table 'weblogs.wp_3_options' doesn't exist]
 SELECT option_value FROM wp_3_options WHERE option_name = 'post_count'
 LIMIT 1
 }}}


 {{{
 $sPassword = generate_random_password();
 $user_id = wpmu_create_user( $username, $sPassword,
 $user_data[LDAP_INDEX_EMAIL] );

 if (!$user_id) {
         $error = __('<strong>Error</strong>: Account Creation Failed.');
         return false;
 }

 //Update their first and last name from ldap
 update_usermeta( $user_id, 'first_name', $user_data[LDAP_INDEX_GIVEN_NAME]
 );
 update_usermeta( $user_id, 'last_name', $user_data[LDAP_INDEX_SURNAME] );

 //This is for plugin events
 do_action( 'wpmu_new_user', $user_id );
 do_action('wpmu_activate_user', $user_id, $password);

 $domain = strtolower( wp_specialchars( $username ) );
 if( constant( "VHOST" ) == 'yes' ) {
         $newdomain = $domain . "." . $current_site->domain;
         $path = $base;
 }
 else {
         $newdomain = $current_site->domain;
         $path = $base . $domain . '/';
 }

 //Create and update the users blog.
 $meta = apply_filters('signup_create_blog_meta', array ('lang_id' => 'en',
 'public' => 0));
 $blog_id = wpmu_create_blog($newdomain, $path, $username . "'s blog",
 $user_id, $meta);
 }}}

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