WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
05/04/2020 09:03:11 AM (5 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Fix WPCS issues in wp-includes/canonical.php.

Reformat long conditions and dense code blocks for better readability.

See #49542.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/canonical.php

    r47727 r47759  
    4949    // If we're not in wp-admin and the post has been published and preview nonce
    5050    // is non-existent or invalid then no need for preview in query.
    51     if ( is_preview() && get_query_var( 'p' ) && 'publish' == get_post_status( get_query_var( 'p' ) ) ) {
     51    if ( is_preview() && get_query_var( 'p' ) && 'publish' === get_post_status( get_query_var( 'p' ) ) ) {
    5252        if ( ! isset( $_GET['preview_id'] )
    5353            || ! isset( $_GET['preview_nonce'] )
    54             || ! wp_verify_nonce( $_GET['preview_nonce'], 'post_preview_' . (int) $_GET['preview_id'] ) ) {
     54            || ! wp_verify_nonce( $_GET['preview_nonce'], 'post_preview_' . (int) $_GET['preview_id'] )
     55        ) {
    5556            $wp_query->is_preview = false;
    5657        }
    5758    }
    5859
    59     if ( is_trackback() || is_search() || is_admin() || is_preview() || is_robots() || is_favicon() || ( $is_IIS && ! iis7_supports_permalinks() ) ) {
     60    if ( is_admin() || is_search() || is_preview() || is_trackback()
     61        || is_robots() || is_favicon()
     62        || ( $is_IIS && ! iis7_supports_permalinks() )
     63    ) {
    6064        return;
    6165    }
    … …  
    96100    }
    97101
    98     $id = get_query_var( 'p' );
    99 
    100     if ( is_feed() && $id ) {
    101         $redirect_url = get_post_comments_feed_link( $id, get_query_var( 'feed' ) );
     102    $post_id = get_query_var( 'p' );
     103
     104    if ( is_feed() && $post_id ) {
     105        $redirect_url = get_post_comments_feed_link( $post_id, get_query_var( 'feed' ) );
     106
    102107        if ( $redirect_url ) {
    103             $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type', 'feed' ), $redirect_url );
    104             $redirect['path']  = parse_url( $redirect_url, PHP_URL_PATH );
    105         }
    106     }
    107 
    108     if ( is_singular() && 1 > $wp_query->post_count && $id ) {
    109 
    110         $vars = $wpdb->get_results( $wpdb->prepare( "SELECT post_type, post_parent FROM $wpdb->posts WHERE ID = %d", $id ) );
     108            $redirect['query'] = _remove_qs_args_if_not_in_url(
     109                $redirect['query'],
     110                array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type', 'feed' ),
     111                $redirect_url
     112            );
     113
     114            $redirect['path'] = parse_url( $redirect_url, PHP_URL_PATH );
     115        }
     116    }
     117
     118    if ( is_singular() && $wp_query->post_count < 1 && $post_id ) {
     119
     120        $vars = $wpdb->get_results( $wpdb->prepare( "SELECT post_type, post_parent FROM $wpdb->posts WHERE ID = %d", $post_id ) );
    111121
    112122        if ( ! empty( $vars[0] ) ) {
    113123            $vars = $vars[0];
    114             if ( 'revision' == $vars->post_type && $vars->post_parent > 0 ) {
    115                 $id = $vars->post_parent;
    116             }
    117 
    118             $redirect_url = get_permalink( $id );
     124
     125            if ( 'revision' === $vars->post_type && $vars->post_parent > 0 ) {
     126                $post_id = $vars->post_parent;
     127            }
     128
     129            $redirect_url = get_permalink( $post_id );
     130
    119131            if ( $redirect_url ) {
    120                 $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ), $redirect_url );
     132                $redirect['query'] = _remove_qs_args_if_not_in_url(
     133                    $redirect['query'],
     134                    array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ),
     135                    $redirect_url
     136                );
    121137            }
    122138        }
    … …  
    127143
    128144        // Redirect ?page_id, ?p=, ?attachment_id= to their respective URLs.
    129         $id            = max( get_query_var( 'p' ), get_query_var( 'page_id' ), get_query_var( 'attachment_id' ) );
    130         $redirect_post = $id ? get_post( $id ) : false;
     145        $post_id = max( get_query_var( 'p' ), get_query_var( 'page_id' ), get_query_var( 'attachment_id' ) );
     146
     147        $redirect_post = $post_id ? get_post( $post_id ) : false;
     148
    131149        if ( $redirect_post ) {
    132150            $post_type_obj = get_post_type_object( $redirect_post->post_type );
    133             if ( $post_type_obj->public && 'auto-draft' != $redirect_post->post_status ) {
    134                 $redirect_url      = get_permalink( $redirect_post );
    135                 $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ), $redirect_url );
    136             }
    137         }
    138 
    139         if ( get_query_var( 'day' ) && get_query_var( 'monthnum' ) && get_query_var( 'year' ) ) {
    140             $year  = get_query_var( 'year' );
    141             $month = get_query_var( 'monthnum' );
    142             $day   = get_query_var( 'day' );
    143             $date  = sprintf( '%04d-%02d-%02d', $year, $month, $day );
     151
     152            if ( $post_type_obj->public && 'auto-draft' !== $redirect_post->post_status ) {
     153                $redirect_url = get_permalink( $redirect_post );
     154
     155                $redirect['query'] = _remove_qs_args_if_not_in_url(
     156                    $redirect['query'],
     157                    array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ),
     158                    $redirect_url
     159                );
     160            }
     161        }
     162
     163        $year  = get_query_var( 'year' );
     164        $month = get_query_var( 'monthnum' );
     165        $day   = get_query_var( 'day' );
     166
     167        if ( $year && $month && $day ) {
     168            $date = sprintf( '%04d-%02d-%02d', $year, $month, $day );
     169
    144170            if ( ! wp_checkdate( $month, $day, $year, $date ) ) {
    145                 $redirect_url      = get_month_link( $year, $month );
    146                 $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'year', 'monthnum', 'day' ), $redirect_url );
    147             }
    148         } elseif ( get_query_var( 'monthnum' ) && get_query_var( 'year' ) && 12 < get_query_var( 'monthnum' ) ) {
    149             $redirect_url      = get_year_link( get_query_var( 'year' ) );
    150             $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'year', 'monthnum' ), $redirect_url );
     171                $redirect_url = get_month_link( $year, $month );
     172
     173                $redirect['query'] = _remove_qs_args_if_not_in_url(
     174                    $redirect['query'],
     175                    array( 'year', 'monthnum', 'day' ),
     176                    $redirect_url
     177                );
     178            }
     179        } elseif ( $year && $month && $month > 12 ) {
     180            $redirect_url = get_year_link( $year );
     181
     182            $redirect['query'] = _remove_qs_args_if_not_in_url(
     183                $redirect['query'],
     184                array( 'year', 'monthnum' ),
     185                $redirect_url
     186            );
    151187        }
    152188
    153189        if ( ! $redirect_url ) {
    154190            $redirect_url = redirect_guess_404_permalink();
     191
    155192            if ( $redirect_url ) {
    156                 $redirect['query'] = _remove_qs_args_if_not_in_url( $redirect['query'], array( 'page', 'feed', 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ), $redirect_url );
    157             }
    158         }
    159 
     193                $redirect['query'] = _remove_qs_args_if_not_in_url(
     194                    $redirect['query'],
     195                    array( 'page', 'feed', 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ),
     196                    $redirect_url
     197                );
     198            }
     199        }
     200
     201        // Strip off non-existing page links from single posts or pages.
    160202        if ( get_query_var( 'page' ) && $wp_query->post ) {
    161203            $redirect['path']  = rtrim( $redirect['path'], (int) get_query_var( 'page' ) . '/' );
    162204            $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
    163             $redirect_url      = get_permalink( $wp_query->post->ID );
     205
     206            $redirect_url = get_permalink( $wp_query->post->ID );
    164207        }
    165208    } elseif ( is_object( $wp_rewrite ) && $wp_rewrite->using_permalinks() ) {
     209
    166210        // Rewriting of old ?p=X, ?m=2004, ?m=200401, ?m=20040101.
    167         if ( is_attachment() &&
    168             ! array_diff( array_keys( $wp->query_vars ), array( 'attachment', 'attachment_id' ) ) &&
    169             ! $redirect_url ) {
     211        if ( is_attachment()
     212            && ! array_diff( array_keys( $wp->query_vars ), array( 'attachment', 'attachment_id' ) )
     213            && ! $redirect_url
     214        ) {
    170215            if ( ! empty( $_GET['attachment_id'] ) ) {
    171216                $redirect_url = get_attachment_link( get_query_var( 'attachment_id' ) );
     217
    172218                if ( $redirect_url ) {
    173219                    $redirect['query'] = remove_query_arg( 'attachment_id', $redirect['query'] );
    … …  
    178224        } elseif ( is_single() && ! empty( $_GET['p'] ) && ! $redirect_url ) {
    179225            $redirect_url = get_permalink( get_query_var( 'p' ) );
     226
    180227            if ( $redirect_url ) {
    181228                $redirect['query'] = remove_query_arg( array( 'p', 'post_type' ), $redirect['query'] );
    … …  
    183230        } elseif ( is_single() && ! empty( $_GET['name'] ) && ! $redirect_url ) {
    184231            $redirect_url = get_permalink( $wp_query->get_queried_object_id() );
     232
    185233            if ( $redirect_url ) {
    186234                $redirect['query'] = remove_query_arg( 'name', $redirect['query'] );
    … …  
    188236        } elseif ( is_page() && ! empty( $_GET['page_id'] ) && ! $redirect_url ) {
    189237            $redirect_url = get_permalink( get_query_var( 'page_id' ) );
     238
    190239            if ( $redirect_url ) {
    191240                $redirect['query'] = remove_query_arg( 'page_id', $redirect['query'] );
    192241            }
    193         } elseif ( is_page() && ! is_feed() && 'page' == get_option( 'show_on_front' ) && get_queried_object_id() == get_option( 'page_on_front' ) && ! $redirect_url ) {
     242        } elseif ( is_page() && ! is_feed() && ! $redirect_url
     243            && 'page' === get_option( 'show_on_front' ) && get_queried_object_id() === (int) get_option( 'page_on_front' )
     244        ) {
    194245            $redirect_url = home_url( '/' );
    195         } elseif ( is_home() && ! empty( $_GET['page_id'] ) && 'page' == get_option( 'show_on_front' ) && get_query_var( 'page_id' ) == get_option( 'page_for_posts' ) && ! $redirect_url ) {
     246        } elseif ( is_home() && ! empty( $_GET['page_id'] ) && ! $redirect_url
     247            && 'page' === get_option( 'show_on_front' ) && get_query_var( 'page_id' ) === (int) get_option( 'page_for_posts' )
     248        ) {
    196249            $redirect_url = get_permalink( get_option( 'page_for_posts' ) );
     250
    197251            if ( $redirect_url ) {
    198252                $redirect['query'] = remove_query_arg( 'page_id', $redirect['query'] );
    … …  
    200254        } elseif ( ! empty( $_GET['m'] ) && ( is_year() || is_month() || is_day() ) ) {
    201255            $m = get_query_var( 'm' );
     256
    202257            switch ( strlen( $m ) ) {
    203258                case 4: // Yearly.
    … …  
    211266                    break;
    212267            }
     268
    213269            if ( $redirect_url ) {
    214270                $redirect['query'] = remove_query_arg( 'm', $redirect['query'] );
    215271            }
    216272            // Now moving on to non ?m=X year/month/day links.
    217         } elseif ( is_day() && get_query_var( 'year' ) && get_query_var( 'monthnum' ) && ! empty( $_GET['day'] ) ) {
    218             $redirect_url = get_day_link( get_query_var( 'year' ), get_query_var( 'monthnum' ), get_query_var( 'day' ) );
    219             if ( $redirect_url ) {
    220                 $redirect['query'] = remove_query_arg( array( 'year', 'monthnum', 'day' ), $redirect['query'] );
    221             }
    222         } elseif ( is_month() && get_query_var( 'year' ) && ! empty( $_GET['monthnum'] ) ) {
    223             $redirect_url = get_month_link( get_query_var( 'year' ), get_query_var( 'monthnum' ) );
    224             if ( $redirect_url ) {
    225                 $redirect['query'] = remove_query_arg( array( 'year', 'monthnum' ), $redirect['query'] );
    226             }
    227         } elseif ( is_year() && ! empty( $_GET['year'] ) ) {
    228             $redirect_url = get_year_link( get_query_var( 'year' ) );
    229             if ( $redirect_url ) {
    230                 $redirect['query'] = remove_query_arg( 'year', $redirect['query'] );
     273        } elseif ( is_date() ) {
     274            $year  = get_query_var( 'year' );
     275            $month = get_query_var( 'monthnum' );
     276            $day   = get_query_var( 'day' );
     277
     278            if ( is_day() && $year && $month && ! empty( $_GET['day'] ) ) {
     279                $redirect_url = get_day_link( $year, $month, $day );
     280
     281                if ( $redirect_url ) {
     282                    $redirect['query'] = remove_query_arg( array( 'year', 'monthnum', 'day' ), $redirect['query'] );
     283                }
     284            } elseif ( is_month() && $year && ! empty( $_GET['monthnum'] ) ) {
     285                $redirect_url = get_month_link( $year, $month );
     286
     287                if ( $redirect_url ) {
     288                    $redirect['query'] = remove_query_arg( array( 'year', 'monthnum' ), $redirect['query'] );
     289                }
     290            } elseif ( is_year() && ! empty( $_GET['year'] ) ) {
     291                $redirect_url = get_year_link( $year );
     292
     293                if ( $redirect_url ) {
     294                    $redirect['query'] = remove_query_arg( 'year', $redirect['query'] );
     295                }
    231296            }
    232297        } elseif ( is_author() && ! empty( $_GET['author'] ) && preg_match( '|^[0-9]+$|', $_GET['author'] ) ) {
    233298            $author = get_userdata( get_query_var( 'author' ) );
    234             if ( ( false !== $author ) && $wpdb->get_var( $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE $wpdb->posts.post_author = %d AND $wpdb->posts.post_status = 'publish' LIMIT 1", $author->ID ) ) ) {
     299
     300            if ( false !== $author
     301                && $wpdb->get_var( $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE $wpdb->posts.post_author = %d AND $wpdb->posts.post_status = 'publish' LIMIT 1", $author->ID ) )
     302            ) {
    235303                $redirect_url = get_author_posts_url( $author->ID, $author->user_nicename );
     304
    236305                if ( $redirect_url ) {
    237306                    $redirect['query'] = remove_query_arg( 'author', $redirect['query'] );
    … …  
    239308            }
    240309        } elseif ( is_category() || is_tag() || is_tax() ) { // Terms (tags/categories).
    241 
    242310            $term_count = 0;
     311
    243312            foreach ( $wp_query->tax_query->queried_terms as $tax_query ) {
    244313                $term_count += count( $tax_query['terms'] );
    … …  
    246315
    247316            $obj = $wp_query->get_queried_object();
     317
    248318            if ( $term_count <= 1 && ! empty( $obj->term_id ) ) {
    249319                $tax_url = get_term_link( (int) $obj->term_id, $obj->taxonomy );
     320
    250321                if ( $tax_url && ! is_wp_error( $tax_url ) ) {
    251322                    if ( ! empty( $redirect['query'] ) ) {
    252323                        // Strip taxonomy query vars off the URL.
    253324                        $qv_remove = array( 'term', 'taxonomy' );
     325
    254326                        if ( is_category() ) {
    255327                            $qv_remove[] = 'category_name';
    … …  
    275347                            // Create the destination URL for this taxonomy.
    276348                            $tax_url = parse_url( $tax_url );
     349
    277350                            if ( ! empty( $tax_url['query'] ) ) {
    278351                                // Taxonomy accessible via ?taxonomy=...&term=... or any custom query var.
    … …  
    295368            }
    296369        } elseif ( is_single() && strpos( $wp_rewrite->permalink_structure, '%category%' ) !== false ) {
    297             $cat = get_query_var( 'category_name' );
    298             if ( $cat ) {
    299                 $category = get_category_by_path( $cat );
    300                 if ( ( ! $category || is_wp_error( $category ) ) || ! has_term( $category->term_id, 'category', $wp_query->get_queried_object_id() ) ) {
     370            $category_name = get_query_var( 'category_name' );
     371
     372            if ( $category_name ) {
     373                $category = get_category_by_path( $category_name );
     374
     375                if ( ! $category || is_wp_error( $category )
     376                    || ! has_term( $category->term_id, 'category', $wp_query->get_queried_object_id() )
     377                ) {
    301378                    $redirect_url = get_permalink( $wp_query->get_queried_object_id() );
    302379                }
    … …  
    304381        }
    305382
    306             // Post paging.
     383        // Post paging.
    307384        if ( is_singular() && get_query_var( 'page' ) ) {
     385            $page = get_query_var( 'page' );
     386
    308387            if ( ! $redirect_url ) {
    309388                $redirect_url = get_permalink( get_queried_object_id() );
    310389            }
    311390
    312             $page = get_query_var( 'page' );
    313391            if ( $page > 1 ) {
     392                $redirect_url = trailingslashit( $redirect_url );
     393
    314394                if ( is_front_page() ) {
    315                     $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' );
     395                    $redirect_url .= user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' );
    316396                } else {
    317                     $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' );
    318                 }
    319             }
    320                 $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
    321         }
    322 
    323             // Paging and feeds.
     397                    $redirect_url .= user_trailingslashit( $page, 'single_paged' );
     398                }
     399            }
     400
     401            $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
     402        }
     403
     404        // Paging and feeds.
    324405        if ( get_query_var( 'paged' ) || is_feed() || get_query_var( 'cpage' ) ) {
    325             while ( preg_match( "#/$wp_rewrite->pagination_base/?[0-9]+?(/+)?$#", $redirect['path'] ) || preg_match( '#/(comments/?)?(feed|rss|rdf|atom|rss2)(/+)?$#', $redirect['path'] ) || preg_match( "#/{$wp_rewrite->comments_pagination_base}-[0-9]+(/+)?$#", $redirect['path'] ) ) {
    326                 // Strip off paging and feed.
    327                 $redirect['path'] = preg_replace( "#/$wp_rewrite->pagination_base/?[0-9]+?(/+)?$#", '/', $redirect['path'] ); // Strip off any existing paging.
    328                 $redirect['path'] = preg_replace( '#/(comments/?)?(feed|rss2?|rdf|atom)(/+|$)#', '/', $redirect['path'] ); // Strip off feed endings.
    329                 $redirect['path'] = preg_replace( "#/{$wp_rewrite->comments_pagination_base}-[0-9]+?(/+)?$#", '/', $redirect['path'] ); // Strip off any existing comment paging.
    330             }
    331 
    332             $addl_path = '';
    333             if ( is_feed() && in_array( get_query_var( 'feed' ), $wp_rewrite->feeds, true ) ) {
     406            $paged = get_query_var( 'paged' );
     407            $feed  = get_query_var( 'feed' );
     408            $cpage = get_query_var( 'cpage' );
     409
     410            while ( preg_match( "#/$wp_rewrite->pagination_base/?[0-9]+?(/+)?$#", $redirect['path'] )
     411                || preg_match( '#/(comments/?)?(feed|rss2?|rdf|atom)(/+)?$#', $redirect['path'] )
     412                || preg_match( "#/{$wp_rewrite->comments_pagination_base}-[0-9]+(/+)?$#", $redirect['path'] )
     413            ) {
     414                // Strip off any existing paging.
     415                $redirect['path'] = preg_replace( "#/$wp_rewrite->pagination_base/?[0-9]+?(/+)?$#", '/', $redirect['path'] );
     416                // Strip off feed endings.
     417                $redirect['path'] = preg_replace( '#/(comments/?)?(feed|rss2?|rdf|atom)(/+|$)#', '/', $redirect['path'] );
     418                // Strip off any existing comment paging.
     419                $redirect['path'] = preg_replace( "#/{$wp_rewrite->comments_pagination_base}-[0-9]+?(/+)?$#", '/', $redirect['path'] );
     420            }
     421
     422            $addl_path    = '';
     423            $default_feed = get_default_feed();
     424
     425            if ( is_feed() && in_array( $feed, $wp_rewrite->feeds, true ) ) {
    334426                $addl_path = ! empty( $addl_path ) ? trailingslashit( $addl_path ) : '';
     427
    335428                if ( ! is_singular() && get_query_var( 'withcomments' ) ) {
    336429                    $addl_path .= 'comments/';
    337430                }
    338                 if ( ( 'rss' == get_default_feed() && 'feed' == get_query_var( 'feed' ) ) || 'rss' == get_query_var( 'feed' ) ) {
    339                     $addl_path .= user_trailingslashit( 'feed/' . ( ( get_default_feed() == 'rss2' ) ? '' : 'rss2' ), 'feed' );
     431
     432                if ( ( 'rss' === $default_feed && 'feed' === $feed ) || 'rss' === $feed ) {
     433                    $format = ( 'rss2' === $default_feed ) ? '' : 'rss2';
    340434                } else {
    341                     $addl_path .= user_trailingslashit( 'feed/' . ( ( get_default_feed() == get_query_var( 'feed' ) || 'feed' == get_query_var( 'feed' ) ) ? '' : get_query_var( 'feed' ) ), 'feed' );
    342                 }
     435                    $format = ( $default_feed === $feed || 'feed' === $feed ) ? '' : $feed;
     436                }
     437
     438                $addl_path .= user_trailingslashit( 'feed/' . $format, 'feed' );
     439
    343440                $redirect['query'] = remove_query_arg( 'feed', $redirect['query'] );
    344             } elseif ( is_feed() && 'old' == get_query_var( 'feed' ) ) {
     441            } elseif ( is_feed() && 'old' === $feed ) {
    345442                $old_feed_files = array(
    346443                    'wp-atom.php'         => 'atom',
    347444                    'wp-commentsrss2.php' => 'comments_rss2',
    348                     'wp-feed.php'         => get_default_feed(),
     445                    'wp-feed.php'         => $default_feed,
    349446                    'wp-rdf.php'          => 'rdf',
    350447                    'wp-rss.php'          => 'rss2',
    351448                    'wp-rss2.php'         => 'rss2',
    352449                );
     450
    353451                if ( isset( $old_feed_files[ basename( $redirect['path'] ) ] ) ) {
    354452                    $redirect_url = get_feed_link( $old_feed_files[ basename( $redirect['path'] ) ] );
     453
    355454                    wp_redirect( $redirect_url, 301 );
    356455                    die();
    … …  
    358457            }
    359458
    360             if ( get_query_var( 'paged' ) > 0 ) {
    361                 $paged             = get_query_var( 'paged' );
     459            if ( $paged > 0 ) {
    362460                $redirect['query'] = remove_query_arg( 'paged', $redirect['query'] );
     461
    363462                if ( ! is_feed() ) {
    364                     if ( $paged > 1 && ! is_single() ) {
    365                         $addl_path = ( ! empty( $addl_path ) ? trailingslashit( $addl_path ) : '' ) . user_trailingslashit( "$wp_rewrite->pagination_base/$paged", 'paged' );
    366                     } elseif ( ! is_single() ) {
     463                    if ( ! is_single() ) {
    367464                        $addl_path = ! empty( $addl_path ) ? trailingslashit( $addl_path ) : '';
     465
     466                        if ( $paged > 1 ) {
     467                            $addl_path .= user_trailingslashit( "$wp_rewrite->pagination_base/$paged", 'paged' );
     468                        }
    368469                    }
    369470                } elseif ( $paged > 1 ) {
    … …  
    372473            }
    373474
    374             if ( get_option( 'page_comments' ) && (
    375             ( 'newest' == get_option( 'default_comments_page' ) && get_query_var( 'cpage' ) > 0 ) ||
    376             ( 'newest' != get_option( 'default_comments_page' ) && get_query_var( 'cpage' ) > 1 )
    377             ) ) {
    378                 $addl_path         = ( ! empty( $addl_path ) ? trailingslashit( $addl_path ) : '' ) . user_trailingslashit( $wp_rewrite->comments_pagination_base . '-' . get_query_var( 'cpage' ), 'commentpaged' );
     475            $default_comments_page = get_option( 'default_comments_page' );
     476
     477            if ( get_option( 'page_comments' )
     478                && ( 'newest' === $default_comments_page && $cpage > 0
     479                    || 'newest' !== $default_comments_page && $cpage > 1 )
     480            ) {
     481                $addl_path  = ( ! empty( $addl_path ) ? trailingslashit( $addl_path ) : '' );
     482                $addl_path .= user_trailingslashit( $wp_rewrite->comments_pagination_base . '-' . $cpage, 'commentpaged' );
     483
    379484                $redirect['query'] = remove_query_arg( 'cpage', $redirect['query'] );
    380485            }
    381486
    382             $redirect['path'] = user_trailingslashit( preg_replace( '|/' . preg_quote( $wp_rewrite->index, '|' ) . '/?$|', '/', $redirect['path'] ) ); // Strip off trailing /index.php/.
    383             if ( ! empty( $addl_path ) && $wp_rewrite->using_index_permalinks() && strpos( $redirect['path'], '/' . $wp_rewrite->index . '/' ) === false ) {
     487            // Strip off trailing /index.php/.
     488            $redirect['path'] = preg_replace( '|/' . preg_quote( $wp_rewrite->index, '|' ) . '/?$|', '/', $redirect['path'] );
     489            $redirect['path'] = user_trailingslashit( $redirect['path'] );
     490
     491            if ( ! empty( $addl_path )
     492                && $wp_rewrite->using_index_permalinks()
     493                && strpos( $redirect['path'], '/' . $wp_rewrite->index . '/' ) === false
     494            ) {
    384495                $redirect['path'] = trailingslashit( $redirect['path'] ) . $wp_rewrite->index . '/';
    385496            }
     497
    386498            if ( ! empty( $addl_path ) ) {
    387499                $redirect['path'] = trailingslashit( $redirect['path'] ) . $addl_path;
    388500            }
     501
    389502            $redirect_url = $redirect['scheme'] . '://' . $redirect['host'] . $redirect['path'];
    390503        }
    391504
    392         if ( 'wp-register.php' == basename( $redirect['path'] ) ) {
     505        if ( 'wp-register.php' === basename( $redirect['path'] ) ) {
    393506            if ( is_multisite() ) {
    394507                /** This filter is documented in wp-login.php */
    … …  
    403516    }
    404517
     518    $redirect['query'] = preg_replace( '#^\??&*?#', '', $redirect['query'] );
     519
    405520    // Tack on any additional query vars.
    406     $redirect['query'] = preg_replace( '#^\??&*?#', '', $redirect['query'] );
    407521    if ( $redirect_url && ! empty( $redirect['query'] ) ) {
    408522        parse_str( $redirect['query'], $_parsed_query );
    … …  
    421535            rawurlencode_deep( array_values( $_parsed_query ) )
    422536        );
    423         $redirect_url  = add_query_arg( $_parsed_query, $redirect_url );
     537
     538        $redirect_url = add_query_arg( $_parsed_query, $redirect_url );
    424539    }
    425540
    … …  
    430545    // www.example.com vs. example.com
    431546    $user_home = parse_url( home_url() );
     547
    432548    if ( ! empty( $user_home['host'] ) ) {
    433549        $redirect['host'] = $user_home['host'];
    434550    }
     551
    435552    if ( empty( $user_home['path'] ) ) {
    436553        $user_home['path'] = '/';
    … …  
    503620
    504621    // Trailing slashes.
    505     if ( is_object( $wp_rewrite ) && $wp_rewrite->using_permalinks() && ! is_404() && ( ! is_front_page() || ( is_front_page() && ( get_query_var( 'paged' ) > 1 ) ) ) ) {
     622    if ( is_object( $wp_rewrite ) && $wp_rewrite->using_permalinks()
     623        && ! is_404() && ( ! is_front_page() || is_front_page() && get_query_var( 'paged' ) > 1 )
     624    ) {
    506625        $user_ts_type = '';
     626
    507627        if ( get_query_var( 'paged' ) > 0 ) {
    508628            $user_ts_type = 'paged';
    … …  
    516636            }
    517637        }
     638
    518639        $redirect['path'] = user_trailingslashit( $redirect['path'], $user_ts_type );
    519640    } elseif ( is_front_page() ) {
    … …  
    527648
    528649    // Always trailing slash the Front Page URL.
    529     if ( trailingslashit( $redirect['path'] ) == trailingslashit( $user_home['path'] ) ) {
     650    if ( trailingslashit( $redirect['path'] ) === trailingslashit( $user_home['path'] ) ) {
    530651        $redirect['path'] = trailingslashit( $redirect['path'] );
    531652    }
     653
     654    $original_host_low = strtolower( $original['host'] );
     655    $redirect_host_low = strtolower( $redirect['host'] );
    532656
    533657    // Ignore differences in host capitalization, as this can lead to infinite redirects.
    534658    // Only redirect no-www <=> yes-www.
    535     if ( strtolower( $original['host'] ) == strtolower( $redirect['host'] ) ||
    536         ( strtolower( $original['host'] ) != 'www.' . strtolower( $redirect['host'] ) && 'www.' . strtolower( $original['host'] ) != strtolower( $redirect['host'] ) ) ) {
     659    if ( $original_host_low === $redirect_host_low
     660        || ( 'www.' . $original_host_low !== $redirect_host_low
     661            && 'www.' . $redirect_host_low !== $original_host_low )
     662    ) {
    537663        $redirect['host'] = $original['host'];
    538664    }
    … …  
    560686    if ( $compare_original !== $compare_redirect ) {
    561687        $redirect_url = $redirect['scheme'] . '://' . $redirect['host'];
     688
    562689        if ( ! empty( $redirect['port'] ) ) {
    563690            $redirect_url .= ':' . $redirect['port'];
    564691        }
     692
    565693        $redirect_url .= $redirect['path'];
     694
    566695        if ( ! empty( $redirect['query'] ) ) {
    567696            $redirect_url .= '?' . $redirect['query'];
    … …  
    569698    }
    570699
    571     if ( ! $redirect_url || $redirect_url == $requested_url ) {
     700    if ( ! $redirect_url || $redirect_url === $requested_url ) {
    572701        return;
    573702    }
    … …  
    589718            }
    590719        }
     720
    591721        $requested_url = preg_replace_callback( '|%[a-fA-F0-9][a-fA-F0-9]|', 'lowercase_octets', $requested_url );
    592722    }
    … …  
    605735
    606736    // Yes, again -- in case the filter aborted the request.
    607     if ( ! $redirect_url || strip_fragment_from_url( $redirect_url ) == strip_fragment_from_url( $requested_url ) ) {
     737    if ( ! $redirect_url || strip_fragment_from_url( $redirect_url ) === strip_fragment_from_url( $requested_url ) ) {
    608738        return;
    609739    }
    … …  
    638768function _remove_qs_args_if_not_in_url( $query_string, array $args_to_check, $url ) {
    639769    $parsed_url = parse_url( $url );
     770
    640771    if ( ! empty( $parsed_url['query'] ) ) {
    641772        parse_str( $parsed_url['query'], $parsed_query );
     773
    642774        foreach ( $args_to_check as $qv ) {
    643775            if ( ! isset( $parsed_query[ $qv ] ) ) {
    … …  
    648780        $query_string = remove_query_arg( $args_to_check, $query_string );
    649781    }
     782
    650783    return $query_string;
    651784}
    … …  
    661794function strip_fragment_from_url( $url ) {
    662795    $parsed_url = parse_url( $url );
     796
    663797    if ( ! empty( $parsed_url['host'] ) ) {
    664798        // This mirrors code in redirect_canonical(). It does not handle every case.
    … …  
    712846        }
    713847
     848        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    714849        $post_id = $wpdb->get_var( "SELECT ID FROM $wpdb->posts WHERE $where AND post_status = 'publish'" );
     850
    715851        if ( ! $post_id ) {
    716852            return false;
    717853        }
     854
    718855        if ( get_query_var( 'feed' ) ) {
    719856            return get_post_comments_feed_link( $post_id, get_query_var( 'feed' ) );
    720         } elseif ( get_query_var( 'page' ) && 1 < get_query_var( 'page' ) ) {
     857        } elseif ( get_query_var( 'page' ) > 1 ) {
    721858            return trailingslashit( get_permalink( $post_id ) ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
    722859        } else {
    … …  
    740877function wp_redirect_admin_locations() {
    741878    global $wp_rewrite;
     879
    742880    if ( ! ( is_404() && $wp_rewrite->using_permalinks() ) ) {
    743881        return;
    … …  
    751889        site_url( 'admin', 'relative' ),
    752890    );
     891
    753892    if ( in_array( untrailingslashit( $_SERVER['REQUEST_URI'] ), $admins, true ) ) {
    754893        wp_redirect( admin_url() );
    … …  
    761900        site_url( 'login', 'relative' ),
    762901    );
     902
    763903    if ( in_array( untrailingslashit( $_SERVER['REQUEST_URI'] ), $logins, true ) ) {
    764904        wp_redirect( wp_login_url() );
Note: See TracChangeset for help on using the changeset viewer.