#37239 closed defect (bug) (fixed)
Correction for the docs of wp_list_authors()
Reported by: | birgire | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | Users | Keywords: | has-patch |
Focuses: | docs | Cc: |
Description
Regarding the wp_list_author()
docs:
- The description of
exclude
is duplicated but there's none ofinclude
.
- The input type for
include
andexclude
can 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
@
8 years ago
- Keywords needs-refresh added
- Milestone changed from Awaiting Review to Future Release
- Version changed from trunk to 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
@
8 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)