From 4a5dea0d62ef98386f7364acfec3caf3956f8ec6 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Wed, 23 Mar 2016 21:54:26 +0000 Subject: [PATCH] Canonical / Query: Restore the `is_404()` check in `wp_old_slug_redirect()` which was removed in [34659]. This reverts part of [34659] due to excessive canonical problems it's caused in 4.4.x. Remove the unit tests which are no longer supported. This also removes the `is_feed()` code to avoid confusion - only pages & embeds will be redirected. Merges [36280] and [36281] to trunk. Props dd32. See #21602, #35344. Built from https://develop.svn.wordpress.org/trunk@37075 git-svn-id: http://core.svn.wordpress.org/trunk@37042 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/query.php | 20 +++----------------- wp-includes/version.php | 2 +- 2 files changed, 4 insertions(+), 18 deletions(-) diff --git a/wp-includes/query.php b/wp-includes/query.php index 7bebfb4932..988787a96e 100644 --- a/wp-includes/query.php +++ b/wp-includes/query.php @@ -4885,16 +4885,11 @@ class WP_Query { * * @global WP_Query $wp_query Global WP_Query instance. * @global wpdb $wpdb WordPress database abstraction object. - * @global WP_Rewrite $wp_rewrite WordPress rewrite component. */ function wp_old_slug_redirect() { - global $wp_query, $wp_rewrite; + global $wp_query; - if ( get_queried_object() ) { - return; - } - - if ( '' !== $wp_query->query_vars['name'] ) : + if ( is_404() && '' !== $wp_query->query_vars['name'] ) : global $wpdb; // Guess the current post_type based on the query vars. @@ -4936,19 +4931,10 @@ function wp_old_slug_redirect() { $link = get_permalink( $id ); - if ( is_feed() ) { - $link = user_trailingslashit( trailingslashit( $link ) . 'feed' ); - } elseif ( isset( $GLOBALS['wp_query']->query_vars['paged'] ) && $GLOBALS['wp_query']->query_vars['paged'] > 1 ) { + if ( isset( $GLOBALS['wp_query']->query_vars['paged'] ) && $GLOBALS['wp_query']->query_vars['paged'] > 1 ) { $link = user_trailingslashit( trailingslashit( $link ) . 'page/' . $GLOBALS['wp_query']->query_vars['paged'] ); } elseif( is_embed() ) { $link = user_trailingslashit( trailingslashit( $link ) . 'embed' ); - } elseif ( is_404() ) { - // Add rewrite endpoints if necessary. - foreach ( $wp_rewrite->endpoints as $endpoint ) { - if ( $endpoint[2] && false !== get_query_var( $endpoint[2], false ) ) { - $link = user_trailingslashit( trailingslashit( $link ) . $endpoint[1] ); - } - } } /** diff --git a/wp-includes/version.php b/wp-includes/version.php index e218b7bcde..fdde97c12c 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -4,7 +4,7 @@ * * @global string $wp_version */ -$wp_version = '4.5-beta4-37074'; +$wp_version = '4.5-beta4-37075'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.