Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (6 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/rewrite.php

    r41786 r42343  
    1212 * @since 2.1.0
    1313 */
    14 define('EP_NONE', 0);
     14define( 'EP_NONE', 0 );
    1515
    1616/**
     
    1919 * @since 2.1.0
    2020 */
    21 define('EP_PERMALINK', 1);
     21define( 'EP_PERMALINK', 1 );
    2222
    2323/**
     
    2626 * @since 2.1.0
    2727 */
    28 define('EP_ATTACHMENT', 2);
     28define( 'EP_ATTACHMENT', 2 );
    2929
    3030/**
     
    3333 * @since 2.1.0
    3434 */
    35 define('EP_DATE', 4);
     35define( 'EP_DATE', 4 );
    3636
    3737/**
     
    4040 * @since 2.1.0
    4141 */
    42 define('EP_YEAR', 8);
     42define( 'EP_YEAR', 8 );
    4343
    4444/**
     
    4747 * @since 2.1.0
    4848 */
    49 define('EP_MONTH', 16);
     49define( 'EP_MONTH', 16 );
    5050
    5151/**
     
    5454 * @since 2.1.0
    5555 */
    56 define('EP_DAY', 32);
     56define( 'EP_DAY', 32 );
    5757
    5858/**
     
    6161 * @since 2.1.0
    6262 */
    63 define('EP_ROOT', 64);
     63define( 'EP_ROOT', 64 );
    6464
    6565/**
     
    6868 * @since 2.1.0
    6969 */
    70 define('EP_COMMENTS', 128);
     70define( 'EP_COMMENTS', 128 );
    7171
    7272/**
     
    7575 * @since 2.1.0
    7676 */
    77 define('EP_SEARCH', 256);
     77define( 'EP_SEARCH', 256 );
    7878
    7979/**
     
    8282 * @since 2.1.0
    8383 */
    84 define('EP_CATEGORIES', 512);
     84define( 'EP_CATEGORIES', 512 );
    8585
    8686/**
     
    8989 * @since 2.3.0
    9090 */
    91 define('EP_TAGS', 1024);
     91define( 'EP_TAGS', 1024 );
    9292
    9393/**
     
    9696 * @since 2.1.0
    9797 */
    98 define('EP_AUTHORS', 2048);
     98define( 'EP_AUTHORS', 2048 );
    9999
    100100/**
     
    103103 * @since 2.1.0
    104104 */
    105 define('EP_PAGES', 4096);
     105define( 'EP_PAGES', 4096 );
    106106
    107107/**
     
    159159function add_rewrite_tag( $tag, $regex, $query = '' ) {
    160160    // validate the tag's name
    161     if ( strlen( $tag ) < 3 || $tag[0] != '%' || $tag[ strlen($tag) - 1 ] != '%' )
     161    if ( strlen( $tag ) < 3 || $tag[0] != '%' || $tag[ strlen( $tag ) - 1 ] != '%' ) {
    162162        return;
     163    }
    163164
    164165    global $wp_rewrite, $wp;
     
    204205
    205206    // Back-compat for the old parameters: $with_front and $ep_mask.
    206     if ( ! is_array( $args ) )
     207    if ( ! is_array( $args ) ) {
    207208        $args = array( 'with_front' => $args );
    208     if ( func_num_args() == 4 )
     209    }
     210    if ( func_num_args() == 4 ) {
    209211        $args['ep_mask'] = func_get_arg( 3 );
     212    }
    210213
    211214    $wp_rewrite->add_permastruct( $name, $struct, $args );
     
    267270 *
    268271 * @param bool $hard Whether to update .htaccess (hard flush) or just update
    269  *                   rewrite_rules transient (soft flush). Default is true (hard).
     272 *                   rewrite_rules transient (soft flush). Default is true (hard).
    270273 */
    271274function flush_rewrite_rules( $hard = true ) {
     
    323326 */
    324327function _wp_filter_taxonomy_base( $base ) {
    325     if ( !empty( $base ) ) {
     328    if ( ! empty( $base ) ) {
    326329        $base = preg_replace( '|^/index\.php/|', '', $base );
    327330        $base = trim( $base, '/' );
     
    481484
    482485    // First, check to see if there is a 'p=N' or 'page_id=N' to match against
    483     if ( preg_match('#[?&](p|page_id|attachment_id)=(\d+)#', $url, $values) )   {
    484         $id = absint($values[2]);
    485         if ( $id )
     486    if ( preg_match( '#[?&](p|page_id|attachment_id)=(\d+)#', $url, $values ) ) {
     487        $id = absint( $values[2] );
     488        if ( $id ) {
    486489            return $id;
     490        }
    487491    }
    488492
    489493    // Get rid of the #anchor
    490     $url_split = explode('#', $url);
    491     $url = $url_split[0];
     494    $url_split = explode( '#', $url );
     495    $url       = $url_split[0];
    492496
    493497    // Get rid of URL ?query=string
    494     $url_split = explode('?', $url);
    495     $url = $url_split[0];
     498    $url_split = explode( '?', $url );
     499    $url       = $url_split[0];
    496500
    497501    // Set the correct URL scheme.
    498502    $scheme = parse_url( home_url(), PHP_URL_SCHEME );
    499     $url = set_url_scheme( $url, $scheme );
     503    $url    = set_url_scheme( $url, $scheme );
    500504
    501505    // Add 'www.' if it is absent and should be there
    502     if ( false !== strpos(home_url(), '://www.') && false === strpos($url, '://www.') )
    503         $url = str_replace('://', '://www.', $url);
     506    if ( false !== strpos( home_url(), '://www.' ) && false === strpos( $url, '://www.' ) ) {
     507        $url = str_replace( '://', '://www.', $url );
     508    }
    504509
    505510    // Strip 'www.' if it is present and shouldn't be
    506     if ( false === strpos(home_url(), '://www.') )
    507         $url = str_replace('://www.', '://', $url);
     511    if ( false === strpos( home_url(), '://www.' ) ) {
     512        $url = str_replace( '://www.', '://', $url );
     513    }
    508514
    509515    if ( trim( $url, '/' ) === home_url() && 'page' == get_option( 'show_on_front' ) ) {
     
    519525
    520526    // Not using rewrite rules, and 'p=N' and 'page_id=N' methods failed, so we're out of options
    521     if ( empty($rewrite) )
     527    if ( empty( $rewrite ) ) {
    522528        return 0;
     529    }
    523530
    524531    // Strip 'index.php/' if we're not using path info permalinks
    525     if ( !$wp_rewrite->using_index_permalinks() )
     532    if ( ! $wp_rewrite->using_index_permalinks() ) {
    526533        $url = str_replace( $wp_rewrite->index . '/', '', $url );
     534    }
    527535
    528536    if ( false !== strpos( trailingslashit( $url ), home_url( '/' ) ) ) {
    529537        // Chop off http://domain.com/[path]
    530         $url = str_replace(home_url(), '', $url);
     538        $url = str_replace( home_url(), '', $url );
    531539    } else {
    532540        // Chop off /path/to/blog
    533541        $home_path = parse_url( home_url( '/' ) );
    534         $home_path = isset( $home_path['path'] ) ? $home_path['path'] : '' ;
    535         $url = preg_replace( sprintf( '#^%s#', preg_quote( $home_path ) ), '', trailingslashit( $url ) );
     542        $home_path = isset( $home_path['path'] ) ? $home_path['path'] : '';
     543        $url       = preg_replace( sprintf( '#^%s#', preg_quote( $home_path ) ), '', trailingslashit( $url ) );
    536544    }
    537545
    538546    // Trim leading and lagging slashes
    539     $url = trim($url, '/');
    540 
    541     $request = $url;
     547    $url = trim( $url, '/' );
     548
     549    $request              = $url;
    542550    $post_type_query_vars = array();
    543551
    544     foreach ( get_post_types( array() , 'objects' ) as $post_type => $t ) {
    545         if ( ! empty( $t->query_var ) )
     552    foreach ( get_post_types( array(), 'objects' ) as $post_type => $t ) {
     553        if ( ! empty( $t->query_var ) ) {
    546554            $post_type_query_vars[ $t->query_var ] = $post_type;
     555        }
    547556    }
    548557
    549558    // Look for matches.
    550559    $request_match = $request;
    551     foreach ( (array)$rewrite as $match => $query) {
     560    foreach ( (array) $rewrite as $match => $query ) {
    552561
    553562        // If the requesting file is the anchor of the match, prepend it
    554563        // to the path info.
    555         if ( !empty($url) && ($url != $request) && (strpos($match, $url) === 0) )
     564        if ( ! empty( $url ) && ( $url != $request ) && ( strpos( $match, $url ) === 0 ) ) {
    556565            $request_match = $url . '/' . $request;
    557 
    558         if ( preg_match("#^$match#", $request_match, $matches) ) {
     566        }
     567
     568        if ( preg_match( "#^$match#", $request_match, $matches ) ) {
    559569
    560570            if ( $wp_rewrite->use_verbose_page_rules && preg_match( '/pagename=\$matches\[([0-9]+)\]/', $query, $varmatch ) ) {
     
    574584            // Got a match.
    575585            // Trim the query of everything up to the '?'.
    576             $query = preg_replace("!^.+\?!", '', $query);
     586            $query = preg_replace( '!^.+\?!', '', $query );
    577587
    578588            // Substitute the substring matches into the query.
    579             $query = addslashes(WP_MatchesMapRegex::apply($query, $matches));
     589            $query = addslashes( WP_MatchesMapRegex::apply( $query, $matches ) );
    580590
    581591            // Filter out non-public query vars
     
    584594            $query = array();
    585595            foreach ( (array) $query_vars as $key => $value ) {
    586                 if ( in_array( $key, $wp->public_query_vars ) ){
    587                     $query[$key] = $value;
    588                     if ( isset( $post_type_query_vars[$key] ) ) {
    589                         $query['post_type'] = $post_type_query_vars[$key];
    590                         $query['name'] = $value;
     596                if ( in_array( $key, $wp->public_query_vars ) ) {
     597                    $query[ $key ] = $value;
     598                    if ( isset( $post_type_query_vars[ $key ] ) ) {
     599                        $query['post_type'] = $post_type_query_vars[ $key ];
     600                        $query['name']      = $value;
    591601                    }
    592602                }
     
    598608            // Do the query
    599609            $query = new WP_Query( $query );
    600             if ( ! empty( $query->posts ) && $query->is_singular )
     610            if ( ! empty( $query->posts ) && $query->is_singular ) {
    601611                return $query->post->ID;
    602             else
     612            } else {
    603613                return 0;
     614            }
    604615        }
    605616    }
Note: See TracChangeset for help on using the changeset viewer.