sergiotarxz/Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Canonical: Ensure a redirect for posts missing postname in a query with an ID.

Browse Source 
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
This commit is contained in:
Jake Spurlock 2020-06-09 19:24:23 +00:00
parent 137443482c
commit 78d1ab2ed4
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/wp-includes/canonical.php
View File

@ -409,6 +409,17 @@ function redirect_canonical( $requested_url = null, $do_redirect = true ) {
} }
$redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); $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. // Paging and feeds.