![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#37239 closed defect (bug) (fixed)
Correction for the docs of wp_list_authors()
| Reported by: | birgire | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.6 |
| Component: | Users | Version: | 4.0 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
Regarding the wp_list_author() docs:
- The description of
excludeis duplicated but there's none ofinclude.
- The input type for
includeandexcludecan be either a string or an array. (seewp_parse_list_id()). Currently it's documented only as a string.
Attachments (2)
Change History (8)
#2
@
10 years ago
- Keywords needs-refresh added
- Milestone Awaiting Review → Future Release
- Version trunk → 4.0
@birgire Thanks for the patch. Let's keep the patch limited to the changes for the @include and @exclude params. Can you refresh it without the alphabetical re-ordering?
#4
@
10 years ago
@rachelbaker sure, I updated the patch without the alphabetical ordering.
ps:
What's the preferred order?
I'm not sure pure code readability adjustments, like
- alphabetical ordering of array arguments
- single argument per line in an array
are accepted as tickets, so I added them here with this ticket ;-)
Mainly because I think it makes it easier (in general) to skim through the core code.
Note:
See TracTickets
for help on using tickets.
Regarding the patch: I additionally improved the readability (alphabetical order + one argument per line)