wp_list_pages() doesn't apply the "current_page_item/parent/ancestor" classes when browsing image/attachment pages

[minusfive]

When you're browsing an "attachment" page which has been attached to a "Page", that Page and its ancestors don't get the "current_page_item/parent/ancestor" classes they should get when called using wp_list_pages() for navigation.

I simply added the "is_attachment()" conditional tag to the function and it works now. I'm attaching the wp-includes/post-template.php file with the modification applied.

Hope this was the right solution ;)

[minusfive]

Change applied to line 624 of wp-includes/post-template.php

[minusfive]

Finally figured out how to save a diff file (I think) :) Here's the patch.

[aaroncampbell]

I tested the patch and it works great. I'm attaching a new diff that should be easier to apply, but it's the same code.

[aaroncampbell]

Patch with file location in it

[ryan]

(In [10947]) Add page classes to page attachments. Props minusfive. fixes #9472