Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#43355 closed defect (bug) (fixed)

Improve inline docs for wp_dropdown_pages() - mention support for get_pages() arguments

Reported by: birgire Owned by: SergeyBiryukov
Priority: normal Milestone: 5.1
Component: Posts, Post Types Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

It's not clear from the documentation that wp_dropdown_pages() also supports get_pages() input arguments.

Here are two ways to fix that:

1) For comparison we have the following docblock line in wp_list_pages() src:

@see get_pages() 

2) We also have this docblock line in wp_page_menu() src:

*     Optional. Arguments to generate a page menu. See wp_list_pages() for additional arguments.

where we look at the wp_list_pages() reference for comparison.

Attachments (1)

43355.diff​ (710 bytes ) - added by birgire 9 years ago.

Download all attachments as: .zip

Change History (5)

@birgire
9 years ago

#1 @birgire
9 years ago

  • Keywords has-patch added

43355.diff​ adds a get_pages() references in the docblock of wp_dropdown_pages().

#2 @SergeyBiryukov
9 years ago

  • Milestone Awaiting Review → 5.0

#3 @SergeyBiryukov
9 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 42721:

Docs: Add a reference to get_pages() arguments in wp_dropdown_pages() and wp_list_pages(), as they both use the function internally.

Props birgire.
Fixes #43355.

#4 @johnbillion
8 years ago

  • Milestone 5.0 → 5.1
Note: See TracTickets for help on using tickets.