[2.5] wp_list_pages "exclude" arg no longer works as advertised

[lyzadanger]

[c'n'p'ed from a post I put in the support forums]

I just upgraded to WP 2.5.

I am building navigation on the front page of my client's site using wp_list_pages() with some excluded pages using the "exclude" parameter. This worked as expected in 2.3.x.

After the upgrade, the excluded pages are not showing up (expected), but all of their sub-pages are (definitely unexpected). Additionally, they are rendering as
LIs at the same level as the top-level pages (i.e. they are not in a sub-list (ul) so are impossible to style invisible as a workaround). The docs specifically say:

"Excluding a Page will also exclude all of its sub-pages from the list to be generated."

Thanks!

[hailin]

I will take a look.

[hailin]

This issue is caused by a statement ($rhierarchical? = 0)
in post-template.php.

I believe Ryan put there because he wanted to improve the performance of page listing.

post-template.php

---

Query pages.
$rhierarchical? = 0; <== caused the issue !!
$pages = get_pages($r);

We probably need to rethink about how to take advantage of the fact that new walker class no longer depends on pages being in certain order. (perhaps we don't need to store pages in options anymore, thus making import much faster).

[Otto42]

Dupe of #6277

[Otto42]

Undo, my bad, this is not a dupe.

[jamescollins]

Replying to hailin:

This issue is caused by a statement ($rhierarchical? = 0)
in post-template.php.

I believe Ryan put there because he wanted to improve the performance of page listing.

post-template.php

---

Query pages.
$rhierarchical? = 0; <== caused the issue !!
$pages = get_pages($r);

I can confirm that commenting out this line fixes the problem. This change was made in ticket #5458 (revision [6399]).

[hailin]

Discussed with Ryan befoer, it appears this little change is more or less intended,
and it does not have any significant effect. No more complaints heard since 2.6, and 2.7. So close it.