WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#10921 closed defect (bug) (fixed)

Add 'walker' to defaults in wp_list_pages

Reported by: josh23french Owned by:
Milestone: 2.9 Priority: normal
Severity: minor Version: 2.9
Component: Template Keywords: has-patch commit
Focuses: Cc:

Description

(r11999 2.9-rare)

There is a way that a theme developer can use his own walker in wp_list_pages, or there seems to be one. The only thing blocking this is that wp_parse_args only compacts the default keys and walker is not one of them. And I really don't feel like hacking core for this theme. It's only 1 line!

Attachments (1)

post-template.php.diff (520 bytes) - added by scribu 5 years ago.

Download all attachments as: .zip

Change History (3)

scribu5 years ago

comment:1 scribu5 years ago

  • Keywords has-patch commit added
  • Milestone changed from Unassigned to 2.9
  • Priority changed from highest omg bbq to normal
  • Type changed from feature request to defect (bug)

You're right: walk_page_tree() has a 'walker' argument which isn't propagated from wp_list_pages().

comment:2 ryan4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [12149]) Propagate walker arg down the stack. Props scribu. fixes #10921

Note: See TracTickets for help on using tickets.