#51690 closed defect (bug) (fixed)
Change "stati" to "statuses" in WP_Comment_Query::__construct() DocBlock
Reported by: | Collizo4sky | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 5.1 |
Component: | Comments | Keywords: | has-patch |
Focuses: | docs | Cc: |
Description
In https://core.trac.wordpress.org/browser/trunk/src/wp-includes/class-wp-comment-query.php#L225, the stati
should be status
Attachments (2)
Change History (9)
#2
@
4 years ago
- Focuses docs added
- Keywords 2nd-opinion added
- Version set to 5.1
Is this an actual issue? stati
is plural for status
in WP Core, and the parameter accepts either a single status as a string or an array of multiple stati. Should this instead be changed to Comment status or stati...
?
Originally changed to stati
in [42165].
#4
@
4 years ago
- Component changed from General to Comments
- Keywords 2nd-opinion removed
- Milestone changed from Awaiting Review to 5.6
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
- Summary changed from Typo fix in docbloc to Rename "stati" to "statuses"
Thanks for the patch!
Looking at some online sources, it seems like "statuses" is indeed the preferred form here.
https://english.stackexchange.com/questions/877/what-is-the-plural-form-of-status
https://onlinewritingtraining.com.au/plural-of-status/
https://www.merriam-webster.com/dictionary/status
https://en.wiktionary.org/wiki/status
https://en.wikipedia.org/wiki/Talk%3AStatus
It also seems to be the only instance of "stati" in core, whereas "statuses" appears in 34 files.
the patch