]> scripts.mit.edu Git - autoinstalls/wordpress.git/blobdiff - wp-includes/user.php
Wordpress 3.3.1-scripts
[autoinstalls/wordpress.git] / wp-includes / user.php
index 0a1a4354ac9271c9b61358302b0c57704ffd5e98..2b0508fad8dc17be0a2a15fe3ef298571bb0324f 100644 (file)
@@ -61,7 +61,7 @@ function wp_signon( $credentials = '', $secure_cookie = '' ) {
        }
 
        wp_set_auth_cookie($user->ID, $credentials['remember'], $secure_cookie);
        }
 
        wp_set_auth_cookie($user->ID, $credentials['remember'], $secure_cookie);
-       do_action('wp_login', $credentials['user_login']);
+       do_action('wp_login', $user->user_login, $user);
        return $user;
 }
 
        return $user;
 }
 
@@ -88,7 +88,7 @@ function wp_authenticate_username_password($user, $username, $password) {
        $userdata = get_user_by('login', $username);
 
        if ( !$userdata )
        $userdata = get_user_by('login', $username);
 
        if ( !$userdata )
-               return new WP_Error('invalid_username', sprintf(__('<strong>ERROR</strong>: Invalid username. <a href="%s" title="Password Lost and Found">Lost your password</a>?'), site_url('wp-login.php?action=lostpassword', 'login')));
+               return new WP_Error('invalid_username', sprintf(__('<strong>ERROR</strong>: Invalid username. <a href="%s" title="Password Lost and Found">Lost your password</a>?'), wp_lostpassword_url()));
 
        if ( is_multisite() ) {
                // Is user marked as spam?
 
        if ( is_multisite() ) {
                // Is user marked as spam?
@@ -109,7 +109,7 @@ function wp_authenticate_username_password($user, $username, $password) {
 
        if ( !wp_check_password($password, $userdata->user_pass, $userdata->ID) )
                return new WP_Error( 'incorrect_password', sprintf( __( '<strong>ERROR</strong>: The password you entered for the username <strong>%1$s</strong> is incorrect. <a href="%2$s" title="Password Lost and Found">Lost your password</a>?' ),
 
        if ( !wp_check_password($password, $userdata->user_pass, $userdata->ID) )
                return new WP_Error( 'incorrect_password', sprintf( __( '<strong>ERROR</strong>: The password you entered for the username <strong>%1$s</strong> is incorrect. <a href="%2$s" title="Password Lost and Found">Lost your password</a>?' ),
-               $username, site_url( 'wp-login.php?action=lostpassword', 'login' ) ) );
+               $username, wp_lostpassword_url() ) );
 
        $user =  new WP_User($userdata->ID);
        return $user;
 
        $user =  new WP_User($userdata->ID);
        return $user;
@@ -165,18 +165,19 @@ function count_user_posts($userid) {
  * Number of posts written by a list of users.
  *
  * @since 3.0.0
  * Number of posts written by a list of users.
  *
  * @since 3.0.0
- * @param array $users User ID number list.
+ * @param array $user_ids Array of user IDs.
+ * @param string|array $post_type Optional. Post type to check. Defaults to post.
  * @return array Amount of posts each user has written.
  */
  * @return array Amount of posts each user has written.
  */
-function count_many_users_posts($users) {
+function count_many_users_posts($users, $post_type = 'post' ) {
        global $wpdb;
 
        $count = array();
        global $wpdb;
 
        $count = array();
-       if ( ! is_array($users) || empty( $users ) )
+       if ( empty( $users ) || ! is_array( $users ) )
                return $count;
 
                return $count;
 
-       $userlist = implode( ',', $users );
-       $where = get_posts_by_author_sql( 'post' );
+       $userlist = implode( ',', array_map( 'absint', $users ) );
+       $where = get_posts_by_author_sql( $post_type );
 
        $result = $wpdb->get_results( "SELECT post_author, COUNT(*) FROM $wpdb->posts $where AND post_author IN ($userlist) GROUP BY post_author", ARRAY_N );
        foreach ( $result as $row ) {
 
        $result = $wpdb->get_results( "SELECT post_author, COUNT(*) FROM $wpdb->posts $where AND post_author IN ($userlist) GROUP BY post_author", ARRAY_N );
        foreach ( $result as $row ) {
@@ -253,20 +254,18 @@ function get_user_option( $option, $user = 0, $deprecated = '' ) {
        if ( !empty( $deprecated ) )
                _deprecated_argument( __FUNCTION__, '3.0' );
 
        if ( !empty( $deprecated ) )
                _deprecated_argument( __FUNCTION__, '3.0' );
 
-       if ( empty($user) ) {
+       if ( empty( $user ) )
                $user = wp_get_current_user();
                $user = wp_get_current_user();
-               $user = $user->ID;
-       }
-
-       $user = get_userdata($user);
+       else
+               $user = new WP_User( $user );
 
 
-       // Keys used as object vars cannot have dashes.
-       $key = str_replace('-', '', $option);
+       if ( ! isset( $user->ID ) )
+               return false;
 
 
-       if ( isset( $user->{$wpdb->prefix . $key} ) ) // Blog specific
-               $result = $user->{$wpdb->prefix . $key};
-       elseif ( isset( $user->{$key} ) ) // User specific and cross-blog
-               $result = $user->{$key};
+       if ( $user->has_prop( $wpdb->prefix . $option ) ) // Blog specific
+               $result = $user->get( $wpdb->prefix . $option );
+       elseif ( $user->has_prop( $option ) ) // User specific and cross-blog
+               $result = $user->get( $option );
        else
                $result = false;
 
        else
                $result = false;
 
@@ -360,12 +359,6 @@ class WP_User_Query {
        var $query_orderby;
        var $query_limit;
 
        var $query_orderby;
        var $query_limit;
 
-       /**
-        * PHP4 constructor
-        */
-       function WP_User_Query( $query = null ) {
-               $this->__construct( $query );
-       }
 
        /**
         * PHP5 constructor
 
        /**
         * PHP5 constructor
@@ -388,7 +381,8 @@ class WP_User_Query {
                                'search' => '',
                                'orderby' => 'login',
                                'order' => 'ASC',
                                'search' => '',
                                'orderby' => 'login',
                                'order' => 'ASC',
-                               'offset' => '', 'number' => '',
+                               'offset' => '',
+                               'number' => '',
                                'count_total' => true,
                                'fields' => 'all',
                                'who' => ''
                                'count_total' => true,
                                'fields' => 'all',
                                'who' => ''
@@ -423,6 +417,9 @@ class WP_User_Query {
                        $this->query_fields = "$wpdb->users.ID";
                }
 
                        $this->query_fields = "$wpdb->users.ID";
                }
 
+               if ( $this->query_vars['count_total'] )
+                       $this->query_fields = 'SQL_CALC_FOUND_ROWS ' . $this->query_fields;
+
                $this->query_from = "FROM $wpdb->users";
                $this->query_where = "WHERE 1=1";
 
                $this->query_from = "FROM $wpdb->users";
                $this->query_where = "WHERE 1=1";
 
@@ -496,13 +493,11 @@ class WP_User_Query {
 
                if ( 'authors' == $qv['who'] && $blog_id ) {
                        $qv['meta_key'] = $wpdb->get_blog_prefix( $blog_id ) . 'user_level';
 
                if ( 'authors' == $qv['who'] && $blog_id ) {
                        $qv['meta_key'] = $wpdb->get_blog_prefix( $blog_id ) . 'user_level';
-                       $qv['meta_value'] = '_wp_zero_value'; // Hack to pass '0'
+                       $qv['meta_value'] = 0;
                        $qv['meta_compare'] = '!=';
                        $qv['blog_id'] = $blog_id = 0; // Prevent extra meta query
                }
 
                        $qv['meta_compare'] = '!=';
                        $qv['blog_id'] = $blog_id = 0; // Prevent extra meta query
                }
 
-               _parse_meta_query( $qv );
-
                $role = trim( $qv['role'] );
 
                if ( $blog_id && ( $role || is_multisite() ) ) {
                $role = trim( $qv['role'] );
 
                if ( $blog_id && ( $role || is_multisite() ) ) {
@@ -517,10 +512,16 @@ class WP_User_Query {
                        $qv['meta_query'][] = $cap_meta_query;
                }
 
                        $qv['meta_query'][] = $cap_meta_query;
                }
 
-               if ( !empty( $qv['meta_query'] ) ) {
-                       $clauses = call_user_func_array( '_get_meta_sql', array( $qv['meta_query'], 'user', $wpdb->users, 'ID', &$this ) );
+               $meta_query = new WP_Meta_Query();
+               $meta_query->parse_query_vars( $qv );
+
+               if ( !empty( $meta_query->queries ) ) {
+                       $clauses = $meta_query->get_sql( 'user', $wpdb->users, 'ID', $this );
                        $this->query_from .= $clauses['join'];
                        $this->query_where .= $clauses['where'];
                        $this->query_from .= $clauses['join'];
                        $this->query_where .= $clauses['where'];
+
+                       if ( 'OR' == $meta_query->relation )
+                               $this->query_fields = 'DISTINCT ' . $this->query_fields;
                }
 
                if ( !empty( $qv['include'] ) ) {
                }
 
                if ( !empty( $qv['include'] ) ) {
@@ -550,7 +551,7 @@ class WP_User_Query {
                }
 
                if ( $this->query_vars['count_total'] )
                }
 
                if ( $this->query_vars['count_total'] )
-                       $this->total_users = $wpdb->get_var("SELECT COUNT(*) $this->query_from $this->query_where");
+                       $this->total_users = $wpdb->get_var( apply_filters( 'found_users_query', 'SELECT FOUND_ROWS()' ) );
 
                if ( !$this->results )
                        return;
 
                if ( !$this->results )
                        return;
@@ -644,16 +645,26 @@ function get_users( $args = array() ) {
  *
  * @since 3.0.0
  *
  *
  * @since 3.0.0
  *
- * @param int $id User Id
- * @param bool $all Whether to retrieve all blogs or only blogs that are not marked as deleted, archived, or spam.
+ * @param int $user_id User ID
+ * @param bool $all Whether to retrieve all blogs, or only blogs that are not marked as deleted, archived, or spam.
  * @return array A list of the user's blogs. False if the user was not found or an empty array if the user has no blogs.
  */
  * @return array A list of the user's blogs. False if the user was not found or an empty array if the user has no blogs.
  */
-function get_blogs_of_user( $id, $all = false ) {
+function get_blogs_of_user( $user_id, $all = false ) {
        global $wpdb;
 
        global $wpdb;
 
-       if ( !is_multisite() ) {
+       $user_id = (int) $user_id;
+
+       // Logged out users can't have blogs
+       if ( empty( $user_id ) )
+               return false;
+
+       $keys = get_user_meta( $user_id );
+       if ( empty( $keys ) )
+               return false;
+
+       if ( ! is_multisite() ) {
                $blog_id = get_current_blog_id();
                $blog_id = get_current_blog_id();
-               $blogs = array();
+               $blogs = array( $blog_id => new stdClass );
                $blogs[ $blog_id ]->userblog_id = $blog_id;
                $blogs[ $blog_id ]->blogname = get_option('blogname');
                $blogs[ $blog_id ]->domain = '';
                $blogs[ $blog_id ]->userblog_id = $blog_id;
                $blogs[ $blog_id ]->blogname = get_option('blogname');
                $blogs[ $blog_id ]->domain = '';
@@ -663,84 +674,82 @@ function get_blogs_of_user( $id, $all = false ) {
                return $blogs;
        }
 
                return $blogs;
        }
 
-       $blogs = wp_cache_get( 'blogs_of_user-' . $id, 'users' );
-
-       // Try priming the new cache from the old cache
-       if ( false === $blogs ) {
-               $cache_suffix = $all ? '_all' : '_short';
-               $blogs = wp_cache_get( 'blogs_of_user_' . $id . $cache_suffix, 'users' );
-               if ( is_array( $blogs ) ) {
-                       $blogs = array_keys( $blogs );
-                       if ( $all )
-                               wp_cache_set( 'blogs_of_user-' . $id, $blogs, 'users' );
+       $blogs = array();
+
+       if ( isset( $keys[ $wpdb->base_prefix . 'capabilities' ] ) && defined( 'MULTISITE' ) ) {
+               $blog = get_blog_details( 1 );
+               if ( $blog && isset( $blog->domain ) && ( $all || ( ! $blog->archived && ! $blog->spam && ! $blog->deleted ) ) ) {
+                       $blogs[ 1 ] = (object) array(
+                               'userblog_id' => 1,
+                               'blogname'    => $blog->blogname,
+                               'domain'      => $blog->domain,
+                               'path'        => $blog->path,
+                               'site_id'     => $blog->site_id,
+                               'siteurl'     => $blog->siteurl,
+                       );
                }
                }
+               unset( $keys[ $wpdb->base_prefix . 'capabilities' ] );
        }
 
        }
 
-       if ( false === $blogs ) {
-               $user = get_userdata( (int) $id );
-               if ( !$user )
-                       return false;
-
-               $blogs = $match = array();
-               $prefix_length = strlen($wpdb->base_prefix);
-               foreach ( (array) $user as $key => $value ) {
-                       if ( $prefix_length && substr($key, 0, $prefix_length) != $wpdb->base_prefix )
-                               continue;
-                       if ( substr($key, -12, 12) != 'capabilities' )
-                               continue;
-                       if ( preg_match( '/^' . $wpdb->base_prefix . '((\d+)_)?capabilities$/', $key, $match ) ) {
-                               if ( count( $match ) > 2 )
-                                       $blogs[] = (int) $match[ 2 ];
-                               else
-                                       $blogs[] = 1;
-                       }
-               }
-               wp_cache_set( 'blogs_of_user-' . $id, $blogs, 'users' );
-       }
+       $keys = array_keys( $keys );
+
+       foreach ( $keys as $key ) {
+               if ( 'capabilities' !== substr( $key, -12 ) )
+                       continue;
+               if ( $wpdb->base_prefix && 0 !== strpos( $key, $wpdb->base_prefix ) )
+                       continue;
+               $blog_id = str_replace( array( $wpdb->base_prefix, '_capabilities' ), '', $key );
+               if ( ! is_numeric( $blog_id ) )
+                       continue;
 
 
-       $blog_deets = array();
-       foreach ( (array) $blogs as $blog_id ) {
+               $blog_id = (int) $blog_id;
                $blog = get_blog_details( $blog_id );
                $blog = get_blog_details( $blog_id );
-               if ( $blog && isset( $blog->domain ) && ( $all == true || $all == false && ( $blog->archived == 0 && $blog->spam == 0 && $blog->deleted == 0 ) ) ) {
-                       $blog_deets[ $blog_id ]->userblog_id    = $blog_id;
-                       $blog_deets[ $blog_id ]->blogname               = $blog->blogname;
-                       $blog_deets[ $blog_id ]->domain         = $blog->domain;
-                       $blog_deets[ $blog_id ]->path                   = $blog->path;
-                       $blog_deets[ $blog_id ]->site_id                = $blog->site_id;
-                       $blog_deets[ $blog_id ]->siteurl                = $blog->siteurl;
+               if ( $blog && isset( $blog->domain ) && ( $all || ( ! $blog->archived && ! $blog->spam && ! $blog->deleted ) ) ) {
+                       $blogs[ $blog_id ] = (object) array(
+                               'userblog_id' => $blog_id,
+                               'blogname'    => $blog->blogname,
+                               'domain'      => $blog->domain,
+                               'path'        => $blog->path,
+                               'site_id'     => $blog->site_id,
+                               'siteurl'     => $blog->siteurl,
+                       );
                }
        }
 
                }
        }
 
-       return apply_filters( 'get_blogs_of_user', $blog_deets, $id, $all );
+       return apply_filters( 'get_blogs_of_user', $blogs, $user_id, $all );
 }
 
 /**
 }
 
 /**
- * Checks if the current user belong to a given blog.
+ * Find out whether a user is a member of a given blog.
  *
  *
- * @since 3.0.0
+ * @since MU 1.1
+ * @uses get_blogs_of_user()
  *
  *
- * @param int $blog_id Blog ID
- * @return bool True if the current users belong to $blog_id, false if not.
+ * @param int $user_id The unique ID of the user
+ * @param int $blog Optional. If no blog_id is provided, current site is used
+ * @return bool
  */
  */
-function is_blog_user( $blog_id = 0 ) {
-       global $wpdb;
-
-       $current_user = wp_get_current_user();
-       if ( !$blog_id )
-               $blog_id = $wpdb->blogid;
+function is_user_member_of_blog( $user_id = 0, $blog_id = 0 ) {
+       $user_id = (int) $user_id;
+       $blog_id = (int) $blog_id;
 
 
-       $cap_key = $wpdb->base_prefix . $blog_id . '_capabilities';
+       if ( empty( $user_id ) )
+               $user_id = get_current_user_id();
 
 
-       if ( is_array($current_user->$cap_key) && in_array(1, $current_user->$cap_key) )
-               return true;
+       if ( empty( $blog_id ) )
+               $blog_id = get_current_blog_id();
 
 
-       return false;
+       $blogs = get_blogs_of_user( $user_id );
+       if ( is_array( $blogs ) )
+               return array_key_exists( $blog_id, $blogs );
+       else
+               return false;
 }
 
 /**
  * Add meta data field to a user.
  *
 }
 
 /**
  * Add meta data field to a user.
  *
- * Post meta data is called "Custom Fields" on the Administration Panels.
+ * Post meta data is called "Custom Fields" on the Administration Screens.
  *
  * @since 3.0.0
  * @uses add_metadata()
  *
  * @since 3.0.0
  * @uses add_metadata()
@@ -789,7 +798,7 @@ function delete_user_meta($user_id, $meta_key, $meta_value = '') {
  * @return mixed Will be an array if $single is false. Will be value of meta data field if $single
  *  is true.
  */
  * @return mixed Will be an array if $single is false. Will be value of meta data field if $single
  *  is true.
  */
-function get_user_meta($user_id, $key, $single = false) {
+function get_user_meta($user_id, $key = '', $single = false) {
        return get_metadata('user', $user_id, $key, $single);
 }
 
        return get_metadata('user', $user_id, $key, $single);
 }
 
@@ -922,7 +931,7 @@ function setup_userdata($for_user_id = '') {
        else
                $user = new WP_User($for_user_id);
 
        else
                $user = new WP_User($for_user_id);
 
-       $userdata   = $user->data;
+       $userdata   = $user;
        $user_ID    = (int) $user->ID;
        $user_level = (int) isset($user->user_level) ? $user->user_level : 0;
 
        $user_ID    = (int) $user->ID;
        $user_level = (int) isset($user->user_level) ? $user->user_level : 0;
 
@@ -1038,141 +1047,6 @@ function wp_dropdown_users( $args = '' ) {
        return $output;
 }
 
        return $output;
 }
 
-/**
- * Add user meta data as properties to given user object.
- *
- * The finished user data is cached, but the cache is not used to fill in the
- * user data for the given object. Once the function has been used, the cache
- * should be used to retrieve user data. The intention is if the current data
- * had been cached already, there would be no need to call this function.
- *
- * @access private
- * @since 2.5.0
- * @uses $wpdb WordPress database object for queries
- *
- * @param object $user The user data object.
- */
-function _fill_user( &$user ) {
-       $metavalues = get_user_metavalues(array($user->ID));
-       _fill_single_user($user, $metavalues[$user->ID]);
-}
-
-/**
- * Perform the query to get the $metavalues array(s) needed by _fill_user and _fill_many_users
- *
- * @since 3.0.0
- * @param array $ids User ID numbers list.
- * @return array of arrays. The array is indexed by user_id, containing $metavalues object arrays.
- */
-function get_user_metavalues($ids) {
-       $objects = array();
-
-       $ids = array_map('intval', $ids);
-       foreach ( $ids as $id )
-               $objects[$id] = array();
-
-       $metas = update_meta_cache('user', $ids);
-
-       foreach ( $metas as $id => $meta ) {
-               foreach ( $meta as $key => $metavalues ) {
-                       foreach ( $metavalues as $value ) {
-                               $objects[$id][] = (object)array( 'user_id' => $id, 'meta_key' => $key, 'meta_value' => $value);
-                       }
-               }
-       }
-
-       return $objects;
-}
-
-/**
- * Unserialize user metadata, fill $user object, then cache everything.
- *
- * @since 3.0.0
- * @param object $user The User object.
- * @param array $metavalues An array of objects provided by get_user_metavalues()
- */
-function _fill_single_user( &$user, &$metavalues ) {
-       global $wpdb;
-
-       foreach ( $metavalues as $meta ) {
-               $value = maybe_unserialize($meta->meta_value);
-               // Keys used as object vars cannot have dashes.
-               $key = str_replace('-', '', $meta->meta_key);
-               $user->{$key} = $value;
-       }
-
-       $level = $wpdb->prefix . 'user_level';
-       if ( isset( $user->{$level} ) )
-               $user->user_level = $user->{$level};
-
-       // For backwards compat.
-       if ( isset($user->first_name) )
-               $user->user_firstname = $user->first_name;
-       if ( isset($user->last_name) )
-               $user->user_lastname = $user->last_name;
-       if ( isset($user->description) )
-               $user->user_description = $user->description;
-
-       update_user_caches($user);
-}
-
-/**
- * Take an array of user objects, fill them with metas, and cache them.
- *
- * @since 3.0.0
- * @param array $users User objects
- */
-function _fill_many_users( &$users ) {
-       $ids = array();
-       foreach( $users as $user_object ) {
-               $ids[] = $user_object->ID;
-       }
-
-       $metas = get_user_metavalues($ids);
-
-       foreach ( $users as $user_object ) {
-               if ( isset($metas[$user_object->ID]) ) {
-                       _fill_single_user($user_object, $metas[$user_object->ID]);
-               }
-       }
-}
-
-/**
- * Sanitize every user field.
- *
- * If the context is 'raw', then the user object or array will get minimal santization of the int fields.
- *
- * @since 2.3.0
- * @uses sanitize_user_field() Used to sanitize the fields.
- *
- * @param object|array $user The User Object or Array
- * @param string $context Optional, default is 'display'. How to sanitize user fields.
- * @return object|array The now sanitized User Object or Array (will be the same type as $user)
- */
-function sanitize_user_object($user, $context = 'display') {
-       if ( is_object($user) ) {
-               if ( !isset($user->ID) )
-                       $user->ID = 0;
-               if ( isset($user->data) )
-                       $vars = get_object_vars( $user->data );
-               else
-                       $vars = get_object_vars($user);
-               foreach ( array_keys($vars) as $field ) {
-                       if ( is_string($user->$field) || is_numeric($user->$field) )
-                               $user->$field = sanitize_user_field($field, $user->$field, $user->ID, $context);
-               }
-               $user->filter = $context;
-       } else {
-               if ( !isset($user['ID']) )
-                       $user['ID'] = 0;
-               foreach ( array_keys($user) as $field )
-                       $user[$field] = sanitize_user_field($field, $user[$field], $user['ID'], $context);
-               $user['filter'] = $context;
-       }
-
-       return $user;
-}
-
 /**
  * Sanitize user field based on context.
  *
 /**
  * Sanitize user field based on context.
  *
@@ -1260,7 +1134,7 @@ function sanitize_user_field($field, $value, $user_id, $context) {
  *
  * @param object $user User object to be cached
  */
  *
  * @param object $user User object to be cached
  */
-function update_user_caches(&$user) {
+function update_user_caches($user) {
        wp_cache_add($user->ID, $user, 'users');
        wp_cache_add($user->user_login, $user->ID, 'userlogins');
        wp_cache_add($user->user_email, $user->ID, 'useremail');
        wp_cache_add($user->ID, $user, 'users');
        wp_cache_add($user->user_login, $user->ID, 'userlogins');
        wp_cache_add($user->user_email, $user->ID, 'useremail');
@@ -1275,13 +1149,12 @@ function update_user_caches(&$user) {
  * @param int $id User ID
  */
 function clean_user_cache($id) {
  * @param int $id User ID
  */
 function clean_user_cache($id) {
-       $user = new WP_User($id);
+       $user = WP_User::get_data_by( 'id', $id );
 
        wp_cache_delete($id, 'users');
        wp_cache_delete($user->user_login, 'userlogins');
        wp_cache_delete($user->user_email, 'useremail');
        wp_cache_delete($user->user_nicename, 'userslugs');
 
        wp_cache_delete($id, 'users');
        wp_cache_delete($user->user_login, 'userlogins');
        wp_cache_delete($user->user_email, 'useremail');
        wp_cache_delete($user->user_nicename, 'userslugs');
-       wp_cache_delete('blogs_of_user-' . $id, 'users');
 }
 
 /**
 }
 
 /**
@@ -1293,7 +1166,7 @@ function clean_user_cache($id) {
  * @return null|int The user's ID on success, and null on failure.
  */
 function username_exists( $username ) {
  * @return null|int The user's ID on success, and null on failure.
  */
 function username_exists( $username ) {
-       if ( $user = get_userdatabylogin( $username ) ) {
+       if ( $user = get_user_by('login', $username ) ) {
                return $user->ID;
        } else {
                return null;
                return $user->ID;
        } else {
                return null;
@@ -1310,7 +1183,7 @@ function username_exists( $username ) {
  * @return bool|int The user's ID on success, and false on failure.
  */
 function email_exists( $email ) {
  * @return bool|int The user's ID on success, and false on failure.
  */
 function email_exists( $email ) {
-       if ( $user = get_user_by_email($email) )
+       if ( $user = get_user_by('email', $email) )
                return $user->ID;
 
        return false;
                return $user->ID;
 
        return false;
@@ -1355,9 +1228,7 @@ function validate_username( $username ) {
  * 'user_url' - A string containing the user's URL for the user's web site.
  * 'user_email' - A string containing the user's email address.
  * 'display_name' - A string that will be shown on the site. Defaults to user's
  * 'user_url' - A string containing the user's URL for the user's web site.
  * 'user_email' - A string containing the user's email address.
  * 'display_name' - A string that will be shown on the site. Defaults to user's
- *             username. It is likely that you will want to change this, for both
- *             appearance and security through obscurity (that is if you don't use and
- *             delete the default 'admin' user).
+ *             username. It is likely that you will want to change this, for appearance.
  * 'nickname' - The user's nickname, defaults to the user's username.
  * 'first_name' - The user's first name.
  * 'last_name' - The user's last name.
  * 'nickname' - The user's nickname, defaults to the user's username.
  * 'first_name' - The user's first name.
  * 'last_name' - The user's last name.
@@ -1388,7 +1259,7 @@ function wp_insert_user($userdata) {
        if ( !empty($ID) ) {
                $ID = (int) $ID;
                $update = true;
        if ( !empty($ID) ) {
                $ID = (int) $ID;
                $update = true;
-               $old_user_data = get_userdata($ID);
+               $old_user_data = WP_User::get_data_by( 'id', $ID );
        } else {
                $update = false;
                // Hash the password
        } else {
                $update = false;
                // Hash the password
@@ -1461,9 +1332,6 @@ function wp_insert_user($userdata) {
        if ( empty($show_admin_bar_front) )
                $show_admin_bar_front = 'true';
 
        if ( empty($show_admin_bar_front) )
                $show_admin_bar_front = 'true';
 
-       if ( empty($show_admin_bar_admin) )
-               $show_admin_bar_admin = is_multisite() ? 'true' : 'false';
-
        $user_nicename_check = $wpdb->get_var( $wpdb->prepare("SELECT ID FROM $wpdb->users WHERE user_nicename = %s AND user_login != %s LIMIT 1" , $user_nicename, $user_login));
 
        if ( $user_nicename_check ) {
        $user_nicename_check = $wpdb->get_var( $wpdb->prepare("SELECT ID FROM $wpdb->users WHERE user_nicename = %s AND user_login != %s LIMIT 1" , $user_nicename, $user_login));
 
        if ( $user_nicename_check ) {
@@ -1487,24 +1355,11 @@ function wp_insert_user($userdata) {
                $user_id = (int) $wpdb->insert_id;
        }
 
                $user_id = (int) $wpdb->insert_id;
        }
 
-       update_user_meta( $user_id, 'first_name', $first_name );
-       update_user_meta( $user_id, 'last_name', $last_name );
-       update_user_meta( $user_id, 'nickname', $nickname );
-       update_user_meta( $user_id, 'description', $description );
-       update_user_meta( $user_id, 'rich_editing', $rich_editing );
-       update_user_meta( $user_id, 'comment_shortcuts', $comment_shortcuts );
-       update_user_meta( $user_id, 'admin_color', $admin_color );
-       update_user_meta( $user_id, 'use_ssl', $use_ssl );
-       update_user_meta( $user_id, 'show_admin_bar_front', $show_admin_bar_front );
-       update_user_meta( $user_id, 'show_admin_bar_admin', $show_admin_bar_admin );
-
-       $user = new WP_User($user_id);
+       $user = new WP_User( $user_id );
 
 
-       foreach ( _wp_get_user_contactmethods( $user ) as $method => $name ) {
-               if ( empty($$method) )
-                       $$method = '';
-
-               update_user_meta( $user_id, $method, $$method );
+       foreach ( _get_additional_user_keys( $user ) as $key ) {
+               if ( isset( $$key ) )
+                       update_user_meta( $user_id, $key, $$key );
        }
 
        if ( isset($role) )
        }
 
        if ( isset($role) )
@@ -1546,10 +1401,17 @@ function wp_update_user($userdata) {
        $ID = (int) $userdata['ID'];
 
        // First, get all of the original fields
        $ID = (int) $userdata['ID'];
 
        // First, get all of the original fields
-       $user = get_userdata($ID);
+       $user_obj = get_userdata( $ID );
+
+       $user = get_object_vars( $user_obj->data );
+
+       // Add additional custom fields
+       foreach ( _get_additional_user_keys( $user_obj ) as $key ) {
+               $user[ $key ] = get_user_meta( $ID, $key, true );
+       }
 
        // Escape data pulled from DB.
 
        // Escape data pulled from DB.
-       $user = add_magic_quotes(get_object_vars($user));
+       $user = add_magic_quotes( $user );
 
        // If password is changing, hash it now.
        if ( ! empty($userdata['user_pass']) ) {
 
        // If password is changing, hash it now.
        if ( ! empty($userdata['user_pass']) ) {
@@ -1565,7 +1427,7 @@ function wp_update_user($userdata) {
 
        // Update the cookies if the password changed.
        $current_user = wp_get_current_user();
 
        // Update the cookies if the password changed.
        $current_user = wp_get_current_user();
-       if ( $current_user->id == $ID ) {
+       if ( $current_user->ID == $ID ) {
                if ( isset($plaintext_pass) ) {
                        wp_clear_auth_cookie();
                        wp_set_auth_cookie($ID);
                if ( isset($plaintext_pass) ) {
                        wp_clear_auth_cookie();
                        wp_set_auth_cookie($ID);
@@ -1578,8 +1440,8 @@ function wp_update_user($userdata) {
 /**
  * A simpler way of inserting an user into the database.
  *
 /**
  * A simpler way of inserting an user into the database.
  *
- * Creates a new user with just the username, password, and email. For more
- * detail creation of a user, use wp_insert_user() to specify more infomation.
+ * Creates a new user with just the username, password, and email. For more
+ * complex user creation use wp_insert_user() to specify more information.
  *
  * @since 2.0.0
  * @see wp_insert_user() More complete way to create a new user
  *
  * @since 2.0.0
  * @see wp_insert_user() More complete way to create a new user
@@ -1599,6 +1461,20 @@ function wp_create_user($username, $password, $email = '') {
 }
 
 
 }
 
 
+/**
+ * Return a list of meta keys that wp_insert_user() is supposed to set.
+ *
+ * @access private
+ * @since 3.3.0
+ *
+ * @param object $user WP_User instance
+ * @return array
+ */
+function _get_additional_user_keys( $user ) {
+       $keys = array( 'first_name', 'last_name', 'nickname', 'description', 'rich_editing', 'comment_shortcuts', 'admin_color', 'use_ssl', 'show_admin_bar_front' );
+       return array_merge( $keys, array_keys( _wp_get_user_contactmethods( $user ) ) );
+}
+
 /**
  * Set up the default contact methods
  *
 /**
  * Set up the default contact methods
  *