WordPress.org

Make WordPress Core

Ticket #41155: 41155.patch

File 41155.patch, 27.0 KB (added by yahil, 4 years ago)

@adamsilverstein if increased #adminmenuwrap selectors z-index 9990 to 10001 we can solve this issue.

  • src/wp-admin/css/admin-menu.css

     
    1010        position: fixed;
    1111        top: 0;
    1212        bottom: -120px;
    13         z-index: 1; /* positive z-index to avoid elastic scrolling woes in Safari */
     13        z-index: 1;
     14
     15        /* positive z-index to avoid elastic scrolling woes in Safari */
    1416}
    1517
    1618#adminmenu {
     
    3840/* New Menu icons */
    3941
    4042.icon16:before {
    41         color: #82878c; /* same as new icons */
     43        color: #82878c;
     44
     45        /* same as new icons */
    4246        font: normal 20px/1 dashicons;
    4347        speak: none;
    4448        padding: 6px 0;
     
    141145#adminmenuwrap {
    142146        position: relative;
    143147        float: left;
    144         z-index: 9990;
     148        z-index: 10001;
    145149}
    146150
    147151/* side admin menu */
     
    167171
    168172#adminmenu .wp-submenu a {
    169173        color: #b4b9be;
    170         color: rgba(240,245,250,0.7);
     174        color: rgba(240, 245, 250, 0.7);
    171175}
    172176
    173177#adminmenu .wp-submenu a:hover,
     
    203207        padding: 7px 0 8px;
    204208        z-index: 9999;
    205209        background-color: #32373c;
    206         -webkit-box-shadow: 0 3px 5px rgba(0,0,0,0.2);
    207         box-shadow: 0 3px 5px rgba(0,0,0,0.2);
     210        -webkit-box-shadow: 0 3px 5px rgba(0, 0, 0, 0.2);
     211        box-shadow: 0 3px 5px rgba(0, 0, 0, 0.2);
    208212}
    209213
    210214.js #adminmenu .sub-open,
     
    340344#adminmenu .wp-menu-image img {
    341345        padding: 9px 0 0 0;
    342346        opacity: 0.6;
    343         filter: alpha(opacity=60);
     347        filter: alpha(opacity = 60);
    344348}
    345349
    346350#adminmenu div.wp-menu-name {
     
    364368
    365369div.wp-menu-image:before {
    366370        color: #a0a5aa;
    367         color: rgba(240,245,250,0.6);
     371        color: rgba(240, 245, 250, 0.6);
    368372        padding: 7px 0;
    369373        -webkit-transition: all .1s ease-in-out;
    370374        transition: all .1s ease-in-out;
     
    372376
    373377#adminmenu div.wp-menu-image:before {
    374378        color: #a0a5aa;
    375         color: rgba(240,245,250,0.6);
     379        color: rgba(240, 245, 250, 0.6);
    376380}
    377381
    378382#adminmenu li.wp-has-current-submenu:hover div.wp-menu-image:before,
     
    419423.no-font-face.auto-fold #adminmenu .wp-menu-name {
    420424        margin-left: 0;
    421425}
     426
    422427/* End no @font-face support */
    423428
    424429/* Sticky admin menu */
     
    487492#adminmenu li.menu-top:hover .wp-menu-image img,
    488493#adminmenu li.wp-has-current-submenu .wp-menu-image img {
    489494        opacity: 1;
    490         filter: alpha(opacity=100);
     495        filter: alpha(opacity = 100);
    491496}
    492497
    493498#adminmenu li.wp-menu-separator {
     
    538543}
    539544
    540545#adminmenu li.current a .awaiting-mod,
    541 #adminmenu      li a.wp-has-current-submenu .update-plugins {
     546#adminmenu li a.wp-has-current-submenu .update-plugins {
    542547        background-color: #00b9eb;
    543548        color: #fff;
    544549}
     
    679684                padding-left: 14px;
    680685        }
    681686
    682 
    683687        .auto-fold #adminmenu li.menu-top .wp-submenu > li > a {
    684688                padding-left: 12px;
    685689        }
     
    750754                -ms-transform: none;
    751755                transform: none;
    752756        }
    753 
    754757}
    755758
    756 @media screen and ( max-width: 782px ) {
     759@media screen and (max-width: 782px) {
    757760        .auto-fold #wpcontent {
    758761                position: relative;
    759762                margin-left: 0;
     
    940943        .auto-fold #adminmenu {
    941944                top: 46px;
    942945        }
    943 }
     946}
     947 No newline at end of file
  • src/wp-includes/user.php

     
    3131 * @return WP_User|WP_Error WP_User on success, WP_Error on failure.
    3232 */
    3333function wp_signon( $credentials = array(), $secure_cookie = '' ) {
    34         if ( empty($credentials) ) {
     34        if ( empty( $credentials ) ) {
    3535                $credentials = array(); // Back-compat for plugins passing an empty string.
    3636
    37                 if ( ! empty($_POST['log']) )
     37                if ( ! empty( $_POST['log'] ) )
    3838                        $credentials['user_login'] = $_POST['log'];
    39                 if ( ! empty($_POST['pwd']) )
     39                if ( ! empty( $_POST['pwd'] ) )
    4040                        $credentials['user_password'] = $_POST['pwd'];
    41                 if ( ! empty($_POST['rememberme']) )
     41                if ( ! empty( $_POST['rememberme'] ) )
    4242                        $credentials['remember'] = $_POST['rememberme'];
    4343        }
    4444
    45         if ( !empty($credentials['remember']) )
     45        if ( ! empty( $credentials['remember'] ) )
    4646                $credentials['remember'] = true;
    4747        else
    4848                $credentials['remember'] = false;
     
    8282         */
    8383        $secure_cookie = apply_filters( 'secure_signon_cookie', $secure_cookie, $credentials );
    8484
    85         global $auth_secure_cookie; // XXX ugly hack to pass this to wp_authenticate_cookie
     85        global $auth_secure_cookie; // XXX ugly hack to pass this to wp_authenticate_cookie.
    8686        $auth_secure_cookie = $secure_cookie;
    8787
    88         add_filter('authenticate', 'wp_authenticate_cookie', 30, 3);
     88        add_filter( 'authenticate', 'wp_authenticate_cookie', 30, 3 );
    8989
    90         $user = wp_authenticate($credentials['user_login'], $credentials['user_password']);
     90        $user = wp_authenticate( $credentials['user_login'], $credentials['user_password'] );
    9191
    9292        if ( is_wp_error($user) ) {
    93                 if ( $user->get_error_codes() == array('empty_username', 'empty_password') ) {
    94                         $user = new WP_Error('', '');
     93                if ( $user->get_error_codes() == array( 'empty_username', 'empty_password' ) ) {
     94                        $user = new WP_Error( '', '' );
    9595                }
    9696
    9797                return $user;
    9898        }
    9999
    100         wp_set_auth_cookie($user->ID, $credentials['remember'], $secure_cookie);
     100        wp_set_auth_cookie( $user->ID, $credentials['remember'], $secure_cookie );
    101101        /**
    102102         * Fires after the user has successfully logged in.
    103103         *
     
    120120 * @param string                $password Password for authentication.
    121121 * @return WP_User|WP_Error WP_User on success, WP_Error on failure.
    122122 */
    123 function wp_authenticate_username_password($user, $username, $password) {
     123function wp_authenticate_username_password( $user, $username, $password ) {
    124124        if ( $user instanceof WP_User ) {
    125125                return $user;
    126126        }
    127127
    128         if ( empty($username) || empty($password) ) {
     128        if ( empty( $username ) || empty( $password ) ) {
    129129                if ( is_wp_error( $user ) )
    130130                        return $user;
    131131
    132132                $error = new WP_Error();
    133133
    134                 if ( empty($username) )
    135                         $error->add('empty_username', __('<strong>ERROR</strong>: The username field is empty.'));
     134                if ( empty( $username ) )
     135                        $error->add( 'empty_username', __( '<strong>ERROR</strong>: The username field is empty.' ) );
    136136
    137                 if ( empty($password) )
    138                         $error->add('empty_password', __('<strong>ERROR</strong>: The password field is empty.'));
     137                if ( empty( $password ) )
     138                        $error->add( 'empty_password', __( '<strong>ERROR</strong>: The password field is empty.' ) );
    139139
    140140                return $error;
    141141        }
    142142
    143         $user = get_user_by('login', $username);
     143        $user = get_user_by( 'login', $username );
    144144
    145         if ( !$user ) {
     145        if ( ! $user ) {
    146146                return new WP_Error( 'invalid_username',
    147147                        __( '<strong>ERROR</strong>: Invalid username.' ) .
    148148                        ' <a href="' . wp_lostpassword_url() . '">' .
     
    161161         * @param string           $password Password to check against the user.
    162162         */
    163163        $user = apply_filters( 'wp_authenticate_user', $user, $password );
    164         if ( is_wp_error($user) )
     164        if ( is_wp_error( $user ) )
    165165                return $user;
    166166
    167167        if ( ! wp_check_password( $password, $user->user_pass, $user->ID ) ) {
     
    204204                $error = new WP_Error();
    205205
    206206                if ( empty( $email ) ) {
    207                         $error->add( 'empty_username', __( '<strong>ERROR</strong>: The email field is empty.' ) ); // Uses 'empty_username' for back-compat with wp_signon()
     207                        $error->add( 'empty_username', __( '<strong>ERROR</strong>: The email field is empty.' ) ); // Uses 'empty_username' for back-compat with wp_signon().
    208208                }
    209209
    210210                if ( empty( $password ) ) {
     
    264264 * @param string                $password Password. If not empty, cancels the cookie authentication.
    265265 * @return WP_User|WP_Error WP_User on success, WP_Error on failure.
    266266 */
    267 function wp_authenticate_cookie($user, $username, $password) {
     267function wp_authenticate_cookie( $user, $username, $password ) {
    268268        if ( $user instanceof WP_User ) {
    269269                return $user;
    270270        }
    271271
    272         if ( empty($username) && empty($password) ) {
     272        if ( empty( $username ) && empty( $password ) ) {
    273273                $user_id = wp_validate_auth_cookie();
    274274                if ( $user_id )
    275275                        return new WP_User($user_id);
     
    281281                else
    282282                        $auth_cookie = AUTH_COOKIE;
    283283
    284                 if ( !empty($_COOKIE[$auth_cookie]) )
    285                         return new WP_Error('expired_session', __('Please log in again.'));
     284                if ( ! empty( $_COOKIE[$auth_cookie] ) )
     285                        return new WP_Error( 'expired_session', __( 'Please log in again.' ) );
    286286
    287287                // If the cookie is not set, be silent.
    288288        }
     
    337337                return $user_id;
    338338        }
    339339
    340         if ( is_blog_admin() || is_network_admin() || empty( $_COOKIE[LOGGED_IN_COOKIE] ) ) {
     340        if ( is_blog_admin() || is_network_admin() || empty( $_COOKIE[ LOGGED_IN_COOKIE ] ) ) {
    341341                return false;
    342342        }
    343343
    344         return wp_validate_auth_cookie( $_COOKIE[LOGGED_IN_COOKIE], 'logged_in' );
     344        return wp_validate_auth_cookie( $_COOKIE[ LOGGED_IN_COOKIE ], 'logged_in' );
    345345}
    346346
    347347/**
     
    416416        return $count;
    417417}
    418418
    419 //
    420 // User option functions
    421 //
     419// User option functions.
    422420
    423421/**
    424422 * Get the current user's ID
     
    456454function get_user_option( $option, $user = 0, $deprecated = '' ) {
    457455        global $wpdb;
    458456
    459         if ( !empty( $deprecated ) )
     457        if ( ! empty( $deprecated ) )
    460458                _deprecated_argument( __FUNCTION__, '3.0.0' );
    461459
    462460        if ( empty( $user ) )
     
    511509function update_user_option( $user_id, $option_name, $newvalue, $global = false ) {
    512510        global $wpdb;
    513511
    514         if ( !$global )
     512        if ( ! $global )
    515513                $option_name = $wpdb->get_blog_prefix() . $option_name;
    516514
    517515        return update_user_meta( $user_id, $option_name, $newvalue );
     
    528526 *
    529527 * @global wpdb $wpdb WordPress database abstraction object.
    530528 *
    531  * @param int    $user_id     User ID
     529 * @param int    $user_id     User ID.
    532530 * @param string $option_name User option name.
    533531 * @param bool   $global      Optional. Whether option name is global or blog specific.
    534532 *                            Default false (blog specific).
     
    537535function delete_user_option( $user_id, $option_name, $global = false ) {
    538536        global $wpdb;
    539537
    540         if ( !$global )
     538        if ( ! $global )
    541539                $option_name = $wpdb->get_blog_prefix() . $option_name;
    542540        return delete_user_meta( $user_id, $option_name );
    543541}
     
    558556        $args = wp_parse_args( $args );
    559557        $args['count_total'] = false;
    560558
    561         $user_search = new WP_User_Query($args);
     559        $user_search = new WP_User_Query( $args );
    562560
    563561        return (array) $user_search->get_results();
    564562}
     
    571569 *
    572570 * @global wpdb $wpdb WordPress database abstraction object.
    573571 *
    574  * @param int  $user_id User ID
     572 * @param int  $user_id User ID.
    575573 * @param bool $all     Whether to retrieve all sites, or only sites that are not
    576574 *                      marked as deleted, archived, or spam.
    577575 * @return array A list of the user's sites. An empty array if the user doesn't exist
     
    582580
    583581        $user_id = (int) $user_id;
    584582
    585         // Logged out users can't have sites
     583        // Logged out users can't have sites.
    586584        if ( empty( $user_id ) )
    587585                return array();
    588586
     
    613611                $site_id = get_current_blog_id();
    614612                $sites = array( $site_id => new stdClass );
    615613                $sites[ $site_id ]->userblog_id = $site_id;
    616                 $sites[ $site_id ]->blogname = get_option('blogname');
     614                $sites[ $site_id ]->blogname = get_option( 'blogname' );
    617615                $sites[ $site_id ]->domain = '';
    618616                $sites[ $site_id ]->path = '';
    619617                $sites[ $site_id ]->site_id = 1;
    620                 $sites[ $site_id ]->siteurl = get_option('siteurl');
     618                $sites[ $site_id ]->siteurl = get_option( 'siteurl' );
    621619                $sites[ $site_id ]->archived = 0;
    622620                $sites[ $site_id ]->spam = 0;
    623621                $sites[ $site_id ]->deleted = 0;
     
    709707        }
    710708
    711709        // Technically not needed, but does save calls to get_site and get_user_meta
    712         // in the event that the function is called when a user isn't logged in
     710        // in the event that the function is called when a user isn't logged in.
    713711        if ( empty( $user_id ) ) {
    714712                return false;
    715713        } else {
     
    738736                return false;
    739737        }
    740738
    741         // no underscore before capabilities in $base_capabilities_key
     739        // no underscore before capabilities in $base_capabilities_key.
    742740        $base_capabilities_key = $wpdb->base_prefix . 'capabilities';
    743741        $site_capabilities_key = $wpdb->base_prefix . $blog_id . '_capabilities';
    744742
    745         if ( isset( $keys[ $base_capabilities_key ] ) && $blog_id == 1 ) {
     743        if ( isset( $keys[ $base_capabilities_key ] ) && 1 == $blog_id ) {
    746744                return true;
    747745        }
    748746
     
    767765 * @param bool   $unique     Optional, default is false. Whether the same key should not be added.
    768766 * @return int|false Meta ID on success, false on failure.
    769767 */
    770 function add_user_meta($user_id, $meta_key, $meta_value, $unique = false) {
    771         return add_metadata('user', $user_id, $meta_key, $meta_value, $unique);
     768function add_user_meta( $user_id, $meta_key, $meta_value, $unique = false ) {
     769        return add_metadata( 'user', $user_id, $meta_key, $meta_value, $unique );
    772770}
    773771
    774772/**
     
    781779 * @since 3.0.0
    782780 * @link https://codex.wordpress.org/Function_Reference/delete_user_meta
    783781 *
    784  * @param int    $user_id    User ID
     782 * @param int    $user_id    User ID.
    785783 * @param string $meta_key   Metadata name.
    786784 * @param mixed  $meta_value Optional. Metadata value.
    787785 * @return bool True on success, false on failure.
    788786 */
    789 function delete_user_meta($user_id, $meta_key, $meta_value = '') {
    790         return delete_metadata('user', $user_id, $meta_key, $meta_value);
     787function delete_user_meta( $user_id, $meta_key, $meta_value = '' ) {
     788        return delete_metadata( 'user', $user_id, $meta_key, $meta_value );
    791789}
    792790
    793791/**
     
    801799 * @param bool   $single  Whether to return a single value.
    802800 * @return mixed Will be an array if $single is false. Will be value of meta data field if $single is true.
    803801 */
    804 function get_user_meta($user_id, $key = '', $single = false) {
    805         return get_metadata('user', $user_id, $key, $single);
     802function get_user_meta( $user_id, $key = '', $single = false ) {
     803        return get_metadata( 'user', $user_id, $key, $single );
    806804}
    807805
    808806/**
     
    822820 * @param mixed  $prev_value Optional. Previous value to check before removing.
    823821 * @return int|bool Meta ID if the key didn't exist, true on successful update, false on failure.
    824822 */
    825 function update_user_meta($user_id, $meta_key, $meta_value, $prev_value = '') {
    826         return update_metadata('user', $user_id, $meta_key, $meta_value, $prev_value);
     823function update_user_meta( $user_id, $meta_key, $meta_value, $prev_value = '' ) {
     824        return update_metadata( 'user', $user_id, $meta_key, $meta_value, $prev_value );
    827825}
    828826
    829827/**
     
    839837 *
    840838 * @global wpdb $wpdb WordPress database abstraction object.
    841839 *
    842  * @param string $strategy 'time' or 'memory'
     840 * @param string $strategy 'time' or 'memory'.
    843841 * @return array Includes a grand total and an array of counts indexed by role strings.
    844842 */
    845 function count_users($strategy = 'time') {
     843function count_users( $strategy = 'time' ) {
    846844        global $wpdb;
    847845
    848         // Initialize
     846        // Initialize.
    849847        $id = get_current_blog_id();
    850         $blog_prefix = $wpdb->get_blog_prefix($id);
     848        $blog_prefix = $wpdb->get_blog_prefix( $id );
    851849        $result = array();
    852850
    853851        if ( 'time' == $strategy ) {
     
    859857                        $select_count[] = $wpdb->prepare( "COUNT(NULLIF(`meta_value` LIKE %s, false))", '%' . $wpdb->esc_like( '"' . $this_role . '"' ) . '%');
    860858                }
    861859                $select_count[] = "COUNT(NULLIF(`meta_value` = 'a:0:{}', false))";
    862                 $select_count = implode(', ', $select_count);
     860                $select_count = implode( ', ', $select_count );
    863861
    864862                // Add the meta_value index to the selection list, then run the query.
    865863                $row = $wpdb->get_row( "
     
    899897                " );
    900898
    901899                foreach ( $users_of_blog as $caps_meta ) {
    902                         $b_roles = maybe_unserialize($caps_meta);
     900                        $b_roles = maybe_unserialize( $caps_meta );
    903901                        if ( ! is_array( $b_roles ) )
    904902                                continue;
    905903                        if ( empty( $b_roles ) ) {
     
    906904                                $avail_roles['none']++;
    907905                        }
    908906                        foreach ( $b_roles as $b_role => $val ) {
    909                                 if ( isset($avail_roles[$b_role]) ) {
     907                                if ( isset( $avail_roles[$b_role] ) ) {
    910908                                        $avail_roles[$b_role]++;
    911909                                } else {
    912910                                        $avail_roles[$b_role] = 1;
     
    916914
    917915                $result['total_users'] = count( $users_of_blog );
    918916                $result['avail_roles'] =& $avail_roles;
    919         }
     917        } // End if().
    920918
    921919        if ( is_multisite() ) {
    922920                $result['avail_roles']['none'] = 0;
     
    954952        $user = get_userdata( $for_user_id );
    955953
    956954        if ( ! $user ) {
    957                 $user_ID = 0;
     955                $user_ID    = 0;
    958956                $user_level = 0;
    959                 $userdata = null;
     957                $userdata   = null;
    960958                $user_login = $user_email = $user_url = $user_identity = '';
    961959                return;
    962960        }
     
    11291127                }
    11301128
    11311129                $output .= "</select>";
    1132         }
     1130        } // End if().
    11331131
    11341132        /**
    11351133         * Filters the wp_dropdown_users() HTML output.
     
    11621160 *                        'attribute' and 'js'.
    11631161 * @return mixed Sanitized value.
    11641162 */
    1165 function sanitize_user_field($field, $value, $user_id, $context) {
     1163function sanitize_user_field( $field, $value, $user_id, $context ) {
    11661164        $int_fields = array('ID');
    1167         if ( in_array($field, $int_fields) )
     1165        if ( in_array( $field, $int_fields ) )
    11681166                $value = (int) $value;
    11691167
    11701168        if ( 'raw' == $context )
    11711169                return $value;
    11721170
    1173         if ( !is_string($value) && !is_numeric($value) )
     1171        if ( ! is_string( $value ) && ! is_numeric( $value ) )
    11741172                return $value;
    11751173
    11761174        $prefixed = false !== strpos( $field, 'user_' );
     
    11991197                if ( 'description' == $field )
    12001198                        $value = esc_html( $value ); // textarea_escaped?
    12011199                else
    1202                         $value = esc_attr($value);
     1200                        $value = esc_attr( $value );
    12031201        } elseif ( 'db' == $context ) {
    12041202                if ( $prefixed ) {
    12051203                        /** This filter is documented in wp-includes/post.php */
     
    12401238                         */
    12411239                        $value = apply_filters( "user_{$field}", $value, $user_id, $context );
    12421240                }
    1243         }
     1241        } // End if().
    12441242
    12451243        if ( 'user_url' == $field )
    12461244                $value = esc_url($value);
     
    12581256 *
    12591257 * @since 3.0.0
    12601258 *
    1261  * @param object|WP_User $user User object to be cached
     1259 * @param object|WP_User $user User object to be cached.
    12621260 * @return bool|null Returns false on failure.
    12631261 */
    12641262function update_user_caches( $user ) {
     
    12701268                $user = $user->data;
    12711269        }
    12721270
    1273         wp_cache_add($user->ID, $user, 'users');
    1274         wp_cache_add($user->user_login, $user->ID, 'userlogins');
    1275         wp_cache_add($user->user_email, $user->ID, 'useremail');
    1276         wp_cache_add($user->user_nicename, $user->ID, 'userslugs');
     1271        wp_cache_add( $user->ID, $user, 'users' );
     1272        wp_cache_add( $user->user_login, $user->ID, 'userlogins' );
     1273        wp_cache_add( $user->user_email, $user->ID, 'useremail' );
     1274        wp_cache_add( $user->user_nicename, $user->ID, 'userslugs' );
    12771275}
    12781276
    12791277/**
     
    12821280 * @since 3.0.0
    12831281 * @since 4.4.0 'clean_user_cache' action was added.
    12841282 *
    1285  * @param WP_User|int $user User object or ID to be cleaned from the cache
     1283 * @param WP_User|int $user User object or ID to be cleaned from the cache.
    12861284 */
    12871285function clean_user_cache( $user ) {
    12881286        if ( is_numeric( $user ) )
     
    14241422
    14251423        // Are we updating or creating?
    14261424        if ( ! empty( $userdata['ID'] ) ) {
    1427                 $ID = (int) $userdata['ID'];
     1425                $ID = ( int ) $userdata['ID'];
    14281426                $update = true;
    14291427                $old_user_data = get_userdata( $ID );
    14301428
     
    14321430                        return new WP_Error( 'invalid_user_id', __( 'Invalid user ID.' ) );
    14331431                }
    14341432
    1435                 // hashed in wp_update_user(), plaintext if called directly
     1433                // hashed in wp_update_user(), plaintext if called directly.
    14361434                $user_pass = ! empty( $userdata['user_pass'] ) ? $userdata['user_pass'] : $old_user_data->user_pass;
    14371435        } else {
    14381436                $update = false;
    1439                 // Hash the password
     1437                // Hash the password.
    14401438                $user_pass = wp_hash_password( $userdata['user_pass'] );
    14411439        }
    14421440
     
    14531451         */
    14541452        $pre_user_login = apply_filters( 'pre_user_login', $sanitized_user_login );
    14551453
    1456         //Remove any non-printable chars from the login string to see if we have ended up with an empty username
     1454        //Remove any non-printable chars from the login string to see if we have ended up with an empty username.
    14571455        $user_login = trim( $pre_user_login );
    14581456
    14591457        // user_login must be between 0 and 60 characters.
    14601458        if ( empty( $user_login ) ) {
    1461                 return new WP_Error('empty_user_login', __('Cannot create a user with an empty login name.') );
     1459                return new WP_Error( 'empty_user_login', __( 'Cannot create a user with an empty login name.' ) );
    14621460        } elseif ( mb_strlen( $user_login ) > 60 ) {
    14631461                return new WP_Error( 'user_login_too_long', __( 'Username may not be longer than 60 characters.' ) );
    14641462        }
     
    16291627
    16301628        if ( $user_nicename_check ) {
    16311629                $suffix = 2;
    1632                 while ($user_nicename_check) {
     1630                while ( $user_nicename_check ) {
    16331631                        // user_nicename allows 50 chars. Subtract one for a hyphen, plus the length of the suffix.
    16341632                        $base_length = 49 - mb_strlen( $suffix );
    16351633                        $alt_user_nicename = mb_substr( $user_nicename, 0, $base_length ) . "-$suffix";
     
    16961694        if ( isset( $userdata['role'] ) ) {
    16971695                $user->set_role( $userdata['role'] );
    16981696        } elseif ( ! $update ) {
    1699                 $user->set_role(get_option('default_role'));
     1697                $user->set_role( get_option( 'default_role' ) );
    17001698        }
    17011699        wp_cache_delete( $user_id, 'users' );
    17021700        wp_cache_delete( $user_login, 'userlogins' );
     
    17411739 * @param mixed $userdata An array of user data or a user object of type stdClass or WP_User.
    17421740 * @return int|WP_Error The updated user's ID or a WP_Error object if the user could not be updated.
    17431741 */
    1744 function wp_update_user($userdata) {
     1742function wp_update_user( $userdata ) {
    17451743        if ( $userdata instanceof stdClass ) {
    17461744                $userdata = get_object_vars( $userdata );
    17471745        } elseif ( $userdata instanceof WP_User ) {
     
    17531751                return new WP_Error( 'invalid_user_id', __( 'Invalid user ID.' ) );
    17541752        }
    17551753
    1756         // First, get all of the original fields
     1754        // First, get all of the original fields.
    17571755        $user_obj = get_userdata( $ID );
    17581756        if ( ! $user_obj ) {
    17591757                return new WP_Error( 'invalid_user_id', __( 'Invalid user ID.' ) );
     
    17611759
    17621760        $user = $user_obj->to_array();
    17631761
    1764         // Add additional custom fields
     1762        // Add additional custom fields.
    17651763        foreach ( _get_additional_user_keys( $user_obj ) as $key ) {
    17661764                $user[ $key ] = get_user_meta( $ID, $key, true );
    17671765        }
     
    17701768        $user = add_magic_quotes( $user );
    17711769
    17721770        if ( ! empty( $userdata['user_pass'] ) && $userdata['user_pass'] !== $user_obj->user_pass ) {
    1773                 // If password is changing, hash it now
     1771                // If password is changing, hash it now.
    17741772                $plaintext_pass = $userdata['user_pass'];
    17751773                $userdata['user_pass'] = wp_hash_password( $userdata['user_pass'] );
    17761774
     
    18751873                        $pass_change_email['message'] = str_replace( '###SITEURL###', home_url(), $pass_change_email['message'] );
    18761874
    18771875                        wp_mail( $pass_change_email['to'], sprintf( $pass_change_email['subject'], $blog_name ), $pass_change_email['message'], $pass_change_email['headers'] );
    1878                 }
     1876                } // End if().
    18791877
    18801878                if ( ! empty( $send_email_change_email ) ) {
    18811879                        /* translators: Do not translate USERNAME, ADMIN_EMAIL, EMAIL, SITENAME, SITEURL: those are placeholders. */
     
    19301928                        $email_change_email['message'] = str_replace( '###SITEURL###', home_url(), $email_change_email['message'] );
    19311929
    19321930                        wp_mail( $email_change_email['to'], sprintf( $email_change_email['subject'], $blog_name ), $email_change_email['message'], $email_change_email['headers'] );
    1933                 }
     1931                } // End if().
    19341932
    19351933                if ( $switched_locale ) {
    19361934                        restore_previous_locale();
    19371935                }
    1938         }
     1936        } // End if().
    19391937
    19401938        // Update the cookies if the password changed.
    19411939        $current_user = wp_get_current_user();
    19421940        if ( $current_user->ID == $ID ) {
    1943                 if ( isset($plaintext_pass) ) {
     1941                if ( isset( $plaintext_pass ) ) {
    19441942                        wp_clear_auth_cookie();
    19451943
    19461944                        // Here we calculate the expiration length of the current auth cookie and compare it to the default expiration.
     
    19721970 * @return int|WP_Error The newly created user's ID or a WP_Error object if the user could not
    19731971 *                      be created.
    19741972 */
    1975 function wp_create_user($username, $password, $email = '') {
     1973function wp_create_user( $username, $password, $email = '' ) {
    19761974        $user_login = wp_slash( $username );
    1977         $user_email = wp_slash( $email    );
     1975        $user_email = wp_slash( $email );
    19781976        $user_pass = $password;
    19791977
    1980         $userdata = compact('user_login', 'user_email', 'user_pass');
    1981         return wp_insert_user($userdata);
     1978        $userdata = compact( 'user_login', 'user_email', 'user_pass' );
     1979        return wp_insert_user( $userdata );
    19821980}
    19831981
    19841982/**
     
    20142012                $methods = array(
    20152013                        'aim'    => __( 'AIM' ),
    20162014                        'yim'    => __( 'Yahoo IM' ),
    2017                         'jabber' => __( 'Jabber / Google Talk' )
     2015                        'jabber' => __( 'Jabber / Google Talk' ),
    20182016                );
    20192017        }
    20202018
     
    20242022         * @since 2.9.0
    20252023         *
    20262024         * @param array   $methods Array of contact methods and their labels.
     2025         *
    20272026         * @param WP_User $user    WP_User object.
    20282027         */
    20292028        return apply_filters( 'user_contactmethods', $methods, $user );
     
    21652164 * @param string $login     The user login.
    21662165 * @return WP_User|WP_Error WP_User object on success, WP_Error object for invalid or expired keys.
    21672166 */
    2168 function check_password_reset_key($key, $login) {
     2167function check_password_reset_key( $key, $login ) {
    21692168        global $wpdb, $wp_hasher;
    21702169
    21712170        $key = preg_replace('/[^a-z0-9]/i', '', $key);
    21722171
    2173         if ( empty( $key ) || !is_string( $key ) )
     2172        if ( empty( $key ) || ! is_string( $key ) )
    21742173                return new WP_Error('invalid_key', __('Invalid key'));
    21752174
    2176         if ( empty($login) || !is_string($login) )
     2175        if ( empty($login) || ! is_string($login) )
    21772176                return new WP_Error('invalid_key', __('Invalid key'));
    21782177
    21792178        $row = $wpdb->get_row( $wpdb->prepare( "SELECT ID, user_activation_key FROM $wpdb->users WHERE user_login = %s", $login ) );
     
    22112210        if ( $hash_is_correct && $expiration_time && time() < $expiration_time ) {
    22122211                return get_userdata( $row->ID );
    22132212        } elseif ( $hash_is_correct && $expiration_time ) {
    2214                 // Key has an expiration time that's passed
     2213                // Key has an expiration time that's passed.
    22152214                return new WP_Error( 'expired_key', __( 'Invalid key' ) );
    22162215        }
    22172216
     
    22412240 *
    22422241 * @since 2.5.0
    22432242 *
    2244  * @param object $user     The user
    2245  * @param string $new_pass New password for the user in plaintext
     2243 * @param object $user     The user.
     2244 * @param string $new_pass New password for the user in plaintext.
    22462245 */
    22472246function reset_password( $user, $new_pass ) {
    22482247        /**
     
    22742273 *
    22752274 * @since 2.5.0
    22762275 *
    2277  * @param string $user_login User's username for logging in
    2278  * @param string $user_email User's email address to send password and add
     2276 * @param string $user_login User's username for logging in.
     2277 * @param string $user_email User's email address to send password and add.
    22792278 * @return int|WP_Error Either user's ID or error on failure.
    22802279 */
    22812280function register_new_user( $user_login, $user_email ) {
     
    22912290         */
    22922291        $user_email = apply_filters( 'user_registration_email', $user_email );
    22932292
    2294         // Check the username
    2295         if ( $sanitized_user_login == '' ) {
     2293        // Check the username.
     2294        if ( '' == $sanitized_user_login ) {
    22962295                $errors->add( 'empty_username', __( '<strong>ERROR</strong>: Please enter a username.' ) );
    22972296        } elseif ( ! validate_username( $user_login ) ) {
    22982297                $errors->add( 'invalid_username', __( '<strong>ERROR</strong>: This username is invalid because it uses illegal characters. Please enter a valid username.' ) );
     
    23082307                }
    23092308        }
    23102309
    2311         // Check the email address
    2312         if ( $user_email == '' ) {
     2310        // Check the email address.
     2311        if ( '' == $user_email ) {
    23132312                $errors->add( 'empty_email', __( '<strong>ERROR</strong>: Please type your email address.' ) );
    23142313        } elseif ( ! is_email( $user_email ) ) {
    23152314                $errors->add( 'invalid_email', __( '<strong>ERROR</strong>: The email address isn&#8217;t correct.' ) );
     
    25062505                        return $current_user;
    25072506                }
    25082507
    2509                 // Upgrade stdClass to WP_User
     2508                // Upgrade stdClass to WP_User.
    25102509                if ( is_object( $current_user ) && isset( $current_user->ID ) ) {
    25112510                        $cur_id = $current_user->ID;
    25122511                        $current_user = null;