WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (2 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/load.php

    r42217 r42343  
    2828 */
    2929function wp_unregister_GLOBALS() {
    30     if ( !ini_get( 'register_globals' ) )
     30    if ( ! ini_get( 'register_globals' ) ) {
    3131        return;
    32 
    33     if ( isset( $_REQUEST['GLOBALS'] ) )
     32    }
     33
     34    if ( isset( $_REQUEST['GLOBALS'] ) ) {
    3435        die( 'GLOBALS overwrite attempt detected' );
     36    }
    3537
    3638    // Variables that shouldn't be unset
     
    3840
    3941    $input = array_merge( $_GET, $_POST, $_COOKIE, $_SERVER, $_ENV, $_FILES, isset( $_SESSION ) && is_array( $_SESSION ) ? $_SESSION : array() );
    40     foreach ( $input as $k => $v )
    41         if ( !in_array( $k, $no_unset ) && isset( $GLOBALS[$k] ) ) {
    42             unset( $GLOBALS[$k] );
    43         }
     42    foreach ( $input as $k => $v ) {
     43        if ( ! in_array( $k, $no_unset ) && isset( $GLOBALS[ $k ] ) ) {
     44            unset( $GLOBALS[ $k ] );
     45        }
     46    }
    4447}
    4548
     
    5861    $default_server_values = array(
    5962        'SERVER_SOFTWARE' => '',
    60         'REQUEST_URI' => '',
     63        'REQUEST_URI'     => '',
    6164    );
    6265
     
    6972        if ( isset( $_SERVER['HTTP_X_ORIGINAL_URL'] ) ) {
    7073            $_SERVER['REQUEST_URI'] = $_SERVER['HTTP_X_ORIGINAL_URL'];
    71         }
    72         // IIS Isapi_Rewrite
     74        } // IIS Isapi_Rewrite
    7375        elseif ( isset( $_SERVER['HTTP_X_REWRITE_URL'] ) ) {
    7476            $_SERVER['REQUEST_URI'] = $_SERVER['HTTP_X_REWRITE_URL'];
    7577        } else {
    7678            // Use ORIG_PATH_INFO if there is no PATH_INFO
    77             if ( !isset( $_SERVER['PATH_INFO'] ) && isset( $_SERVER['ORIG_PATH_INFO'] ) )
     79            if ( ! isset( $_SERVER['PATH_INFO'] ) && isset( $_SERVER['ORIG_PATH_INFO'] ) ) {
    7880                $_SERVER['PATH_INFO'] = $_SERVER['ORIG_PATH_INFO'];
     81            }
    7982
    8083            // Some IIS + PHP configurations puts the script-name in the path-info (No need to append it twice)
    8184            if ( isset( $_SERVER['PATH_INFO'] ) ) {
    82                 if ( $_SERVER['PATH_INFO'] == $_SERVER['SCRIPT_NAME'] )
     85                if ( $_SERVER['PATH_INFO'] == $_SERVER['SCRIPT_NAME'] ) {
    8386                    $_SERVER['REQUEST_URI'] = $_SERVER['PATH_INFO'];
    84                 else
     87                } else {
    8588                    $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . $_SERVER['PATH_INFO'];
     89                }
    8690            }
    8791
     
    9498
    9599    // Fix for PHP as CGI hosts that set SCRIPT_FILENAME to something ending in php.cgi for all requests
    96     if ( isset( $_SERVER['SCRIPT_FILENAME'] ) && ( strpos( $_SERVER['SCRIPT_FILENAME'], 'php.cgi' ) == strlen( $_SERVER['SCRIPT_FILENAME'] ) - 7 ) )
     100    if ( isset( $_SERVER['SCRIPT_FILENAME'] ) && ( strpos( $_SERVER['SCRIPT_FILENAME'], 'php.cgi' ) == strlen( $_SERVER['SCRIPT_FILENAME'] ) - 7 ) ) {
    97101        $_SERVER['SCRIPT_FILENAME'] = $_SERVER['PATH_TRANSLATED'];
     102    }
    98103
    99104    // Fix for Dreamhost and other PHP as CGI hosts
    100     if ( strpos( $_SERVER['SCRIPT_NAME'], 'php.cgi' ) !== false )
     105    if ( strpos( $_SERVER['SCRIPT_NAME'], 'php.cgi' ) !== false ) {
    101106        unset( $_SERVER['PATH_INFO'] );
     107    }
    102108
    103109    // Fix empty PHP_SELF
    104110    $PHP_SELF = $_SERVER['PHP_SELF'];
    105     if ( empty( $PHP_SELF ) )
    106         $_SERVER['PHP_SELF'] = $PHP_SELF = preg_replace( '/(\?.*)?$/', '', $_SERVER["REQUEST_URI"] );
     111    if ( empty( $PHP_SELF ) ) {
     112        $_SERVER['PHP_SELF'] = $PHP_SELF = preg_replace( '/(\?.*)?$/', '', $_SERVER['REQUEST_URI'] );
     113    }
    107114}
    108115
     
    152159function wp_favicon_request() {
    153160    if ( '/favicon.ico' == $_SERVER['REQUEST_URI'] ) {
    154         header('Content-Type: image/vnd.microsoft.icon');
     161        header( 'Content-Type: image/vnd.microsoft.icon' );
    155162        exit;
    156163    }
     
    174181 */
    175182function wp_maintenance() {
    176     if ( ! file_exists( ABSPATH . '.maintenance' ) || wp_installing() )
     183    if ( ! file_exists( ABSPATH . '.maintenance' ) || wp_installing() ) {
    177184        return;
     185    }
    178186
    179187    global $upgrading;
     
    181189    include( ABSPATH . '.maintenance' );
    182190    // If the $upgrading timestamp is older than 10 minutes, don't die.
    183     if ( ( time() - $upgrading ) >= 600 )
     191    if ( ( time() - $upgrading ) >= 600 ) {
    184192        return;
     193    }
    185194
    186195    /**
     
    267276function timer_stop( $display = 0, $precision = 3 ) {
    268277    global $timestart, $timeend;
    269     $timeend = microtime( true );
     278    $timeend   = microtime( true );
    270279    $timetotal = $timeend - $timestart;
    271     $r = ( function_exists( 'number_format_i18n' ) ) ? number_format_i18n( $timetotal, $precision ) : number_format( $timetotal, $precision );
    272     if ( $display )
     280    $r         = ( function_exists( 'number_format_i18n' ) ) ? number_format_i18n( $timetotal, $precision ) : number_format( $timetotal, $precision );
     281    if ( $display ) {
    273282        echo $r;
     283    }
    274284    return $r;
    275285}
     
    319329     * @param bool $enable_debug_mode Whether to enable debug mode checks to occur. Default true.
    320330     */
    321     if ( ! apply_filters( 'enable_wp_debug_mode_checks', true ) ){
     331    if ( ! apply_filters( 'enable_wp_debug_mode_checks', true ) ) {
    322332        return;
    323333    }
     
    326336        error_reporting( E_ALL );
    327337
    328         if ( WP_DEBUG_DISPLAY )
     338        if ( WP_DEBUG_DISPLAY ) {
    329339            ini_set( 'display_errors', 1 );
    330         elseif ( null !== WP_DEBUG_DISPLAY )
     340        } elseif ( null !== WP_DEBUG_DISPLAY ) {
    331341            ini_set( 'display_errors', 0 );
     342        }
    332343
    333344        if ( WP_DEBUG_LOG ) {
     
    358369 */
    359370function wp_set_lang_dir() {
    360     if ( !defined( 'WP_LANG_DIR' ) ) {
    361         if ( file_exists( WP_CONTENT_DIR . '/languages' ) && @is_dir( WP_CONTENT_DIR . '/languages' ) || !@is_dir(ABSPATH . WPINC . '/languages') ) {
     371    if ( ! defined( 'WP_LANG_DIR' ) ) {
     372        if ( file_exists( WP_CONTENT_DIR . '/languages' ) && @is_dir( WP_CONTENT_DIR . '/languages' ) || ! @is_dir( ABSPATH . WPINC . '/languages' ) ) {
    362373            /**
    363374             * Server path of the language directory.
     
    368379             */
    369380            define( 'WP_LANG_DIR', WP_CONTENT_DIR . '/languages' );
    370             if ( !defined( 'LANGDIR' ) ) {
     381            if ( ! defined( 'LANGDIR' ) ) {
    371382                // Old static relative path maintained for limited backward compatibility - won't work in some cases.
    372383                define( 'LANGDIR', 'wp-content/languages' );
     
    381392             */
    382393            define( 'WP_LANG_DIR', ABSPATH . WPINC . '/languages' );
    383             if ( !defined( 'LANGDIR' ) ) {
     394            if ( ! defined( 'LANGDIR' ) ) {
    384395                // Old relative path maintained for backward compatibility.
    385396                define( 'LANGDIR', WPINC . '/languages' );
     
    400411
    401412    require_once( ABSPATH . WPINC . '/wp-db.php' );
    402     if ( file_exists( WP_CONTENT_DIR . '/db.php' ) )
     413    if ( file_exists( WP_CONTENT_DIR . '/db.php' ) ) {
    403414        require_once( WP_CONTENT_DIR . '/db.php' );
     415    }
    404416
    405417    if ( isset( $wpdb ) ) {
     
    424436function wp_set_wpdb_vars() {
    425437    global $wpdb, $table_prefix;
    426     if ( !empty( $wpdb->error ) )
     438    if ( ! empty( $wpdb->error ) ) {
    427439        dead_db();
    428 
    429     $wpdb->field_types = array( 'post_author' => '%d', 'post_parent' => '%d', 'menu_order' => '%d', 'term_id' => '%d', 'term_group' => '%d', 'term_taxonomy_id' => '%d',
    430         'parent' => '%d', 'count' => '%d','object_id' => '%d', 'term_order' => '%d', 'ID' => '%d', 'comment_ID' => '%d', 'comment_post_ID' => '%d', 'comment_parent' => '%d',
    431         'user_id' => '%d', 'link_id' => '%d', 'link_owner' => '%d', 'link_rating' => '%d', 'option_id' => '%d', 'blog_id' => '%d', 'meta_id' => '%d', 'post_id' => '%d',
    432         'user_status' => '%d', 'umeta_id' => '%d', 'comment_karma' => '%d', 'comment_count' => '%d',
     440    }
     441
     442    $wpdb->field_types = array(
     443        'post_author'      => '%d',
     444        'post_parent'      => '%d',
     445        'menu_order'       => '%d',
     446        'term_id'          => '%d',
     447        'term_group'       => '%d',
     448        'term_taxonomy_id' => '%d',
     449        'parent'           => '%d',
     450        'count'            => '%d',
     451        'object_id'        => '%d',
     452        'term_order'       => '%d',
     453        'ID'               => '%d',
     454        'comment_ID'       => '%d',
     455        'comment_post_ID'  => '%d',
     456        'comment_parent'   => '%d',
     457        'user_id'          => '%d',
     458        'link_id'          => '%d',
     459        'link_owner'       => '%d',
     460        'link_rating'      => '%d',
     461        'option_id'        => '%d',
     462        'blog_id'          => '%d',
     463        'meta_id'          => '%d',
     464        'post_id'          => '%d',
     465        'user_status'      => '%d',
     466        'umeta_id'         => '%d',
     467        'comment_karma'    => '%d',
     468        'comment_count'    => '%d',
    433469        // multisite:
    434         'active' => '%d', 'cat_id' => '%d', 'deleted' => '%d', 'lang_id' => '%d', 'mature' => '%d', 'public' => '%d', 'site_id' => '%d', 'spam' => '%d',
     470        'active'           => '%d',
     471        'cat_id'           => '%d',
     472        'deleted'          => '%d',
     473        'lang_id'          => '%d',
     474        'mature'           => '%d',
     475        'public'           => '%d',
     476        'site_id'          => '%d',
     477        'spam'             => '%d',
    435478    );
    436479
     
    441484        wp_die(
    442485            /* translators: 1: $table_prefix 2: wp-config.php */
    443             sprintf( __( '<strong>ERROR</strong>: %1$s in %2$s can only contain numbers, letters, and underscores.' ),
     486            sprintf(
     487                __( '<strong>ERROR</strong>: %1$s in %2$s can only contain numbers, letters, and underscores.' ),
    444488                '<code>$table_prefix</code>',
    445489                '<code>wp-config.php</code>'
     
    463507    global $_wp_using_ext_object_cache;
    464508    $current_using = $_wp_using_ext_object_cache;
    465     if ( null !== $using )
     509    if ( null !== $using ) {
    466510        $_wp_using_ext_object_cache = $using;
     511    }
    467512    return $current_using;
    468513}
     
    483528
    484529    $first_init = false;
    485     if ( ! function_exists( 'wp_cache_init' ) ) {
     530    if ( ! function_exists( 'wp_cache_init' ) ) {
    486531        if ( file_exists( WP_CONTENT_DIR . '/object-cache.php' ) ) {
    487             require_once ( WP_CONTENT_DIR . '/object-cache.php' );
     532            require_once( WP_CONTENT_DIR . '/object-cache.php' );
    488533            if ( function_exists( 'wp_cache_init' ) ) {
    489534                wp_using_ext_object_cache( true );
     
    508553
    509554    if ( ! wp_using_ext_object_cache() ) {
    510         require_once ( ABSPATH . WPINC . '/cache.php' );
     555        require_once( ABSPATH . WPINC . '/cache.php' );
    511556    }
    512557
     
    571616function wp_get_mu_plugins() {
    572617    $mu_plugins = array();
    573     if ( !is_dir( WPMU_PLUGIN_DIR ) )
     618    if ( ! is_dir( WPMU_PLUGIN_DIR ) ) {
    574619        return $mu_plugins;
    575     if ( ! $dh = opendir( WPMU_PLUGIN_DIR ) )
     620    }
     621    if ( ! $dh = opendir( WPMU_PLUGIN_DIR ) ) {
    576622        return $mu_plugins;
     623    }
    577624    while ( ( $plugin = readdir( $dh ) ) !== false ) {
    578         if ( substr( $plugin, -4 ) == '.php' )
     625        if ( substr( $plugin, -4 ) == '.php' ) {
    579626            $mu_plugins[] = WPMU_PLUGIN_DIR . '/' . $plugin;
     627        }
    580628    }
    581629    closedir( $dh );
     
    600648 */
    601649function wp_get_active_and_valid_plugins() {
    602     $plugins = array();
     650    $plugins        = array();
    603651    $active_plugins = (array) get_option( 'active_plugins', array() );
    604652
     
    609657    }
    610658
    611     if ( empty( $active_plugins ) || wp_installing() )
     659    if ( empty( $active_plugins ) || wp_installing() ) {
    612660        return $plugins;
     661    }
    613662
    614663    $network_plugins = is_multisite() ? wp_get_active_network_plugins() : false;
     
    620669            // not already included as a network plugin
    621670            && ( ! $network_plugins || ! in_array( WP_PLUGIN_DIR . '/' . $plugin, $network_plugins ) )
    622             )
    623         $plugins[] = WP_PLUGIN_DIR . '/' . $plugin;
     671            ) {
     672            $plugins[] = WP_PLUGIN_DIR . '/' . $plugin;
     673        }
    624674    }
    625675    return $plugins;
     
    638688    if ( function_exists( 'mb_internal_encoding' ) ) {
    639689        $charset = get_option( 'blog_charset' );
    640         if ( ! $charset || ! @mb_internal_encoding( $charset ) )
     690        if ( ! $charset || ! @mb_internal_encoding( $charset ) ) {
    641691            mb_internal_encoding( 'UTF-8' );
     692        }
    642693    }
    643694}
     
    655706    // If already slashed, strip.
    656707    if ( get_magic_quotes_gpc() ) {
    657         $_GET    = stripslashes_deep( $_GET    );
    658         $_POST   = stripslashes_deep( $_POST   );
     708        $_GET    = stripslashes_deep( $_GET );
     709        $_POST   = stripslashes_deep( $_POST );
    659710        $_COOKIE = stripslashes_deep( $_COOKIE );
    660711    }
    661712
    662713    // Escape with wpdb.
    663     $_GET    = add_magic_quotes( $_GET    );
    664     $_POST   = add_magic_quotes( $_POST   );
     714    $_GET    = add_magic_quotes( $_GET );
     715    $_POST   = add_magic_quotes( $_POST );
    665716    $_COOKIE = add_magic_quotes( $_COOKIE );
    666717    $_SERVER = add_magic_quotes( $_SERVER );
     
    714765 */
    715766function is_admin() {
    716     if ( isset( $GLOBALS['current_screen'] ) )
     767    if ( isset( $GLOBALS['current_screen'] ) ) {
    717768        return $GLOBALS['current_screen']->in_admin();
    718     elseif ( defined( 'WP_ADMIN' ) )
     769    } elseif ( defined( 'WP_ADMIN' ) ) {
    719770        return WP_ADMIN;
     771    }
    720772
    721773    return false;
     
    737789 */
    738790function is_blog_admin() {
    739     if ( isset( $GLOBALS['current_screen'] ) )
     791    if ( isset( $GLOBALS['current_screen'] ) ) {
    740792        return $GLOBALS['current_screen']->in_admin( 'site' );
    741     elseif ( defined( 'WP_BLOG_ADMIN' ) )
     793    } elseif ( defined( 'WP_BLOG_ADMIN' ) ) {
    742794        return WP_BLOG_ADMIN;
     795    }
    743796
    744797    return false;
     
    760813 */
    761814function is_network_admin() {
    762     if ( isset( $GLOBALS['current_screen'] ) )
     815    if ( isset( $GLOBALS['current_screen'] ) ) {
    763816        return $GLOBALS['current_screen']->in_admin( 'network' );
    764     elseif ( defined( 'WP_NETWORK_ADMIN' ) )
     817    } elseif ( defined( 'WP_NETWORK_ADMIN' ) ) {
    765818        return WP_NETWORK_ADMIN;
     819    }
    766820
    767821    return false;
     
    784838 */
    785839function is_user_admin() {
    786     if ( isset( $GLOBALS['current_screen'] ) )
     840    if ( isset( $GLOBALS['current_screen'] ) ) {
    787841        return $GLOBALS['current_screen']->in_admin( 'user' );
    788     elseif ( defined( 'WP_USER_ADMIN' ) )
     842    } elseif ( defined( 'WP_USER_ADMIN' ) ) {
    789843        return WP_USER_ADMIN;
     844    }
    790845
    791846    return false;
     
    800855 */
    801856function is_multisite() {
    802     if ( defined( 'MULTISITE' ) )
     857    if ( defined( 'MULTISITE' ) ) {
    803858        return MULTISITE;
    804 
    805     if ( defined( 'SUBDOMAIN_INSTALL' ) || defined( 'VHOST' ) || defined( 'SUNRISE' ) )
     859    }
     860
     861    if ( defined( 'SUBDOMAIN_INSTALL' ) || defined( 'VHOST' ) || defined( 'SUNRISE' ) ) {
    806862        return true;
     863    }
    807864
    808865    return false;
     
    820877function get_current_blog_id() {
    821878    global $blog_id;
    822     return absint($blog_id);
     879    return absint( $blog_id );
    823880}
    824881
     
    865922
    866923    static $loaded = false;
    867     if ( $loaded )
     924    if ( $loaded ) {
    868925        return;
     926    }
    869927    $loaded = true;
    870928
    871     if ( function_exists( 'did_action' ) && did_action( 'init' ) )
     929    if ( function_exists( 'did_action' ) && did_action( 'init' ) ) {
    872930        return;
     931    }
    873932
    874933    // We need $wp_local_package
     
    888947    while ( true ) {
    889948        if ( defined( 'WPLANG' ) ) {
    890             if ( '' == WPLANG )
     949            if ( '' == WPLANG ) {
    891950                break;
     951            }
    892952            $locales[] = WPLANG;
    893953        }
    894954
    895         if ( isset( $wp_local_package ) )
     955        if ( isset( $wp_local_package ) ) {
    896956            $locales[] = $wp_local_package;
    897 
    898         if ( ! $locales )
     957        }
     958
     959        if ( ! $locales ) {
    899960            break;
    900 
    901         if ( defined( 'WP_LANG_DIR' ) && @is_dir( WP_LANG_DIR ) )
     961        }
     962
     963        if ( defined( 'WP_LANG_DIR' ) && @is_dir( WP_LANG_DIR ) ) {
    902964            $locations[] = WP_LANG_DIR;
    903 
    904         if ( defined( 'WP_CONTENT_DIR' ) && @is_dir( WP_CONTENT_DIR . '/languages' ) )
     965        }
     966
     967        if ( defined( 'WP_CONTENT_DIR' ) && @is_dir( WP_CONTENT_DIR . '/languages' ) ) {
    905968            $locations[] = WP_CONTENT_DIR . '/languages';
    906 
    907         if ( @is_dir( ABSPATH . 'wp-content/languages' ) )
     969        }
     970
     971        if ( @is_dir( ABSPATH . 'wp-content/languages' ) ) {
    908972            $locations[] = ABSPATH . 'wp-content/languages';
    909 
    910         if ( @is_dir( ABSPATH . WPINC . '/languages' ) )
     973        }
     974
     975        if ( @is_dir( ABSPATH . WPINC . '/languages' ) ) {
    911976            $locations[] = ABSPATH . WPINC . '/languages';
    912 
    913         if ( ! $locations )
     977        }
     978
     979        if ( ! $locations ) {
    914980            break;
     981        }
    915982
    916983        $locations = array_unique( $locations );
     
    920987                if ( file_exists( $location . '/' . $locale . '.mo' ) ) {
    921988                    load_textdomain( 'default', $location . '/' . $locale . '.mo' );
    922                     if ( defined( 'WP_SETUP_CONFIG' ) && file_exists( $location . '/admin-' . $locale . '.mo' ) )
     989                    if ( defined( 'WP_SETUP_CONFIG' ) && file_exists( $location . '/admin-' . $locale . '.mo' ) ) {
    923990                        load_textdomain( 'default', $location . '/admin-' . $locale . '.mo' );
     991                    }
    924992                    break 2;
    925993                }
     
    9571025    if ( ! is_null( $is_installing ) ) {
    9581026        $old_installing = $installing;
    959         $installing = $is_installing;
     1027        $installing     = $is_installing;
    9601028        return (bool) $old_installing;
    9611029    }
     
    9811049            return true;
    9821050        }
    983     } elseif ( isset($_SERVER['SERVER_PORT'] ) && ( '443' == $_SERVER['SERVER_PORT'] ) ) {
     1051    } elseif ( isset( $_SERVER['SERVER_PORT'] ) && ( '443' == $_SERVER['SERVER_PORT'] ) ) {
    9841052        return true;
    9851053    }
     
    10361104            $ini_all = ini_get_all();
    10371105        }
    1038     }
     1106    }
    10391107
    10401108    // Bit operator to workaround https://bugs.php.net/bug.php?id=44936 which changes access level to 63 in PHP 5.2.6 - 5.2.17.
     
    11301198        return;
    11311199    }
    1132     $key = substr( sanitize_key( wp_unslash( $_REQUEST['wp_scrape_key'] ) ), 0, 32 );
     1200    $key   = substr( sanitize_key( wp_unslash( $_REQUEST['wp_scrape_key'] ) ), 0, 32 );
    11331201    $nonce = wp_unslash( $_REQUEST['wp_scrape_nonce'] );
    11341202
    11351203    if ( get_transient( 'scrape_key_' . $key ) !== $nonce ) {
    11361204        echo "###### wp_scraping_result_start:$key ######";
    1137         echo wp_json_encode( array(
    1138             'code' => 'scrape_nonce_failure',
    1139             'message' => __( 'Scrape nonce check failed. Please try again.' ),
    1140         ) );
     1205        echo wp_json_encode(
     1206            array(
     1207                'code'    => 'scrape_nonce_failure',
     1208                'message' => __( 'Scrape nonce check failed. Please try again.' ),
     1209            )
     1210        );
    11411211        echo "###### wp_scraping_result_end:$key ######";
    11421212        die();
Note: See TracChangeset for help on using the changeset viewer.