WordPress.org
Get WordPress

Make WordPress Core

Ticket #47422: 47422.diff

File 47422.diff, 5.6 KB (added by davidbaumwald, 5 years ago)

Patch

  • src/wp-admin/includes/ms.php 

    diff --git a/src/wp-admin/includes/ms.php b/src/wp-admin/includes/ms.php
index 54d638f6d8..2983952e8b 100644
    a b function check_upload_size( $file ) { 
    2020                return $file;
    2121        }
    2222
    23         if ( $file['error'] != '0' ) { // there's already an error
     23        if ( '0' != $file['error'] ) { // there's already an error
    2424                return $file;
    2525        }
    2626
    … … function check_upload_size( $file ) { 
    4545                $file['error'] = __( 'You have used your space quota. Please delete files before uploading.' );
    4646        }
    4747
    48         if ( $file['error'] != '0' && ! isset( $_POST['html-upload'] ) && ! wp_doing_ajax() ) {
     48        if ( ! isset( $_POST['html-upload'] ) && ! wp_doing_ajax() ) {
    4949                wp_die( $file['error'] . ' <a href="javascript:history.go(-1)">' . __( 'Back' ) . '</a>' );
    5050        }
    5151
    … … function update_user_status( $id, $pref, $value, $deprecated = null ) { 
    339339        $user = new WP_User( $id );
    340340        clean_user_cache( $user );
    341341
    342         if ( $pref == 'spam' ) {
    343                 if ( $value == 1 ) {
     342        if ( 'spam' === $pref ) {
     343                if ( 1 === $value ) {
    344344                        /**
    345345                         * Fires after the user is marked as a SPAM user.
    346346                         *
    … … function update_user_status( $id, $pref, $value, $deprecated = null ) { 
    375375function refresh_user_details( $id ) {
    376376        $id = (int) $id;
    377377
    378         if ( ! $user = get_userdata( $id ) ) {
     378        $user = get_userdata( $id );
     379
     380        if ( ! $user ) {
    379381                return false;
    380382        }
    381383
    … … function format_code_lang( $code = '' ) { 
    608610 *                       if `$taxonomy` is 'category' or 'post_tag'.
    609611 */
    610612function sync_category_tag_slugs( $term, $taxonomy ) {
    611         if ( global_terms_enabled() && ( $taxonomy == 'category' || $taxonomy == 'post_tag' ) ) {
     613        if ( global_terms_enabled() && ( 'category' === $taxonomy || 'post_tag' === $taxonomy ) ) {
    612614                if ( is_object( $term ) ) {
    613615                        $term->slug = sanitize_title( $term->name );
    614616                } else {
    … … function mu_dropdown_languages( $lang_files = array(), $current = '' ) { 
    707709                }
    708710        }
    709711
    710         if ( $flag === false ) { // WordPress english
     712        if ( false === $flag ) { // WordPress english
    711713                $output[] = '<option value=""' . selected( $current, '', false ) . '>' . __( 'English' ) . '</option>';
    712714        }
    713715
    … … function site_admin_notice() { 
    745747                return false;
    746748        }
    747749
    748         if ( 'upgrade.php' == $pagenow ) {
     750        if ( 'upgrade.php' === $pagenow ) {
    749751                return;
    750752        }
    751753
    … … function avoid_blog_page_permalink_collision( $data, $postarr ) { 
    770772        if ( is_subdomain_install() ) {
    771773                return $data;
    772774        }
    773         if ( $data['post_type'] != 'page' ) {
     775        if ( 'page' !== $data['post_type'] ) {
    774776                return $data;
    775777        }
    776         if ( ! isset( $data['post_name'] ) || $data['post_name'] == '' ) {
     778        if ( ! isset( $data['post_name'] ) || '' === $data['post_name'] ) {
    777779                return $data;
    778780        }
    779781        if ( ! is_main_site() ) {
    … … function avoid_blog_page_permalink_collision( $data, $postarr ) { 
    786788                $post_name .= mt_rand( 1, 10 );
    787789                $c ++;
    788790        }
    789         if ( $post_name != $data['post_name'] ) {
     791        if ( $data['post_name'] != $post_name ) {
    790792                $data['post_name'] = $post_name;
    791793        }
    792794        return $data;
    … … function choose_primary_blog() { 
    816818                        <select name="primary_blog" id="primary_blog">
    817819                                <?php
    818820                                foreach ( (array) $all_blogs as $blog ) {
    819                                         if ( $primary_blog == $blog->userblog_id ) {
     821                                        if ( $blog->userblog_id == $primary_blog ) {
    820822                                                $found = true;
    821823                                        }
    822824                                        ?>
    … … function choose_primary_blog() { 
    830832                                $blog = reset( $all_blogs );
    831833                                update_user_meta( get_current_user_id(), 'primary_blog', $blog->userblog_id );
    832834                        }
    833                 } elseif ( count( $all_blogs ) == 1 ) {
     835                } elseif ( 1 === count( $all_blogs ) ) {
    834836                        $blog = reset( $all_blogs );
    835837                        echo esc_url( get_home_url( $blog->userblog_id ) );
    836838                        if ( $primary_blog != $blog->userblog_id ) { // Set the primary blog again if it's out of sync with blog list.
    … … function choose_primary_blog() { 
    858860 * @return bool True if network can be edited, otherwise false.
    859861 */
    860862function can_edit_network( $network_id ) {
    861         if ( $network_id == get_current_network_id() ) {
     863        if ( get_current_network_id() === $network_id ) {
    862864                $result = true;
    863865        } else {
    864866                $result = false;
    … … function confirm_delete_users( $users ) { 
    901903        ?>
    902904        <h1><?php esc_html_e( 'Users' ); ?></h1>
    903905
    904         <?php if ( 1 == count( $users ) ) : ?>
     906        <?php if ( 1 === count( $users ) ) : ?>
    905907                <p><?php _e( 'You have chosen to delete the user from all networks and sites.' ); ?></p>
    906908        <?php else : ?>
    907909                <p><?php _e( 'You have chosen to delete the following users from all networks and sites.' ); ?></p>
    … … function confirm_delete_users( $users ) { 
    916918        ?>
    917919        <table class="form-table" role="presentation">
    918920        <?php
    919         foreach ( ( $allusers = (array) $_POST['allusers'] ) as $user_id ) {
    920                 if ( $user_id != '' && $user_id != '0' ) {
     921        $allusers = (array) $_POST['allusers'];
     922        foreach ( $allusers as $user_id ) {
     923                if ( '' !== $user_id && '0' != $user_id ) {
    921924                        $delete_user = get_userdata( $user_id );
    922925
    923926                        if ( ! current_user_can( 'delete_user', $delete_user->ID ) ) {
    … … function confirm_delete_users( $users ) { 
    964967                                                                $user_list .= "<option value='{$user->ID}'>{$user->user_login}</option>";
    965968                                                        }
    966969                                                }
    967                                                 if ( '' == $user_list ) {
     970                                                if ( '' === $user_list ) {
    968971                                                        $user_list = $admin_out;
    969972                                                }
    970973                                                $user_dropdown .= $user_list;
    … … function confirm_delete_users( $users ) { 
    9971000        /** This action is documented in wp-admin/users.php */
    9981001        do_action( 'delete_user_form', $current_user, $allusers );
    9991002
    1000         if ( 1 == count( $users ) ) :
     1003        if ( 1 === count( $users ) ) :
    10011004                ?>
    10021005                <p><?php _e( 'Once you hit &#8220;Confirm Deletion&#8221;, the user will be permanently removed.' ); ?></p>
    10031006        <?php else : ?>