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

Pages: get_page_uri() should return the URI at which the resource being accessed is available at, this may include non-'publish' status posts.

Browse Source 
Reverts [34001] and fixes the original issue in #15963 - avoiding a PHP Notice for when the post doesn't exist.

Props tharsheblows.
See #15963.
Fixes #35084.


git-svn-id: https://develop.svn.wordpress.org/trunk@36094 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Dion Hulse 2015-12-26 03:45:28 +00:00
parent 1fdddd302e
commit 1d2904bf2a
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/wp-includes/post.php
View File

@ -4300,7 +4300,7 @@ function get_page_uri( $page ) {
foreach ( $page->ancestors as $parent ) {
$parent = get_post( $parent );
if ( 'publish' === $parent->post_status ) {
if ( $parent ) {
$uri = $parent->post_name . '/' . $uri;
}
}

4
tests/phpunit/tests/post.php
View File

@ -674,8 +674,8 @@ class Tests_Post extends WP_UnitTestCase {
// try the child normally
$this->assertEquals( 'parent/child', get_page_uri( $child_id ) );
// now delete the parent and check
wp_delete_post( $parent_id );
// now delete the parent from the database and check
wp_delete_post( $parent_id, true );
$this->assertEquals( 'child', get_page_uri( $child_id ) );
}