Ticket #25750: 25750.2.diff

src/wp-includes/post.php

@@ -3706 +3706 @@
         if ( !in_array( $post_type, $hierarchical_post_types ) )
                 return $pages;
 
-        if ( $parent && ! $child_of )
+        if ( $parent > 0 && ! $child_of )
                 $hierarchical = false;
 
         // Make sure we have a valid post status

tests/phpunit/tests/post.php

@@ -879 +879 @@
                 $pages = get_pages( array( 'parent' => array( $page_id1, $page_id2 ) ) );
                 $this->assertEqualSets( array( $page_id2, $page_id3, $page_id4 ), wp_list_pluck( $pages, 'ID' ) );
         }
+
+        /**
+         * @ticket 25750
+         */
+        function test_get_pages_hierarchical_and_no_parent() {
+                global $wpdb;
+                $page_1 = $this->factory->post->create( array( 'post_type' => 'page' ) );
+                $page_2 = $this->factory->post->create( array( 'post_type' => 'page', 'post_parent' => $page_1 ) );
+                $page_3 = $this->factory->post->create( array( 'post_type' => 'page', 'post_parent' => $page_1 ) );
+                $page_4 = $this->factory->post->create( array( 'post_type' => 'page', 'post_parent' => $page_2 ) );
+
+                $pages = get_pages(); // Defaults: hierarchical = true, parent = -1
+                $pages_default_args = get_pages( array( 'hierarchical' => true, 'parent' => -1 ) );
+                // Confirm the defaults.
+                $this->assertEquals( $pages, $pages_default_args );
+
+                /*
+                 * Here's the tree we are testing:
+                 *
+                 * page 1
+                 * - page 2
+                 * -- page 4
+                 * - page 3
+                 *
+                 * If hierarchical => true works, the order will be 1,2,4,3.
+                 * If it doesn't, they will be in the creation order, 1,2,3,4.
+                 */
+
+                $this->assertEqualSets( array( $page_1, $page_2, $page_4, $page_3 ), wp_list_pluck( $pages, 'ID' ) );
+        }
 }