From 78d1ab2ed40093a5bd2a75b01ceea37811739f55 Mon Sep 17 00:00:00 2001 From: Jake Spurlock Date: Tue, 9 Jun 2020 19:24:23 +0000 Subject: [PATCH] Canonical: Ensure a redirect for posts missing `postname` in a query with an ID. If a site is using `/%post_id%/%postname%/` as permalink structure, and is missing the `postname`, the site won't redirect to the appropriate URL. This change ensure that the redirect happens. Fixes: #12456. Props: Frank.Prendergast, dd32, Otto42, hlanggo, wonderboymusic, atimmer, seth17, calvin_ngan, Niresh12495, anbumz, SergeyBiryukov, donmhico. git-svn-id: https://develop.svn.wordpress.org/trunk@47937 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/canonical.php | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/wp-includes/canonical.php b/src/wp-includes/canonical.php index 42d6dcf44a..b73b9f72f2 100644 --- a/src/wp-includes/canonical.php +++ b/src/wp-includes/canonical.php @@ -409,6 +409,17 @@ function redirect_canonical( $requested_url = null, $do_redirect = true ) { } $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); + } elseif ( is_singular() && empty( get_query_var( 'name' ) ) && $post_id ) { + /* + * Redirect example.org/%post_id%/ to canonical url. + * @ticket 12456 + */ + $redirect_url = get_permalink( $post_id ); + $redirect['query'] = _remove_qs_args_if_not_in_url( + $redirect['query'], + array( 'p', 'page_id', 'attachment_id', 'pagename', 'name', 'post_type' ), + $redirect_url + ); } // Paging and feeds.