#51690 closed defect (bug) (fixed)
Change "stati" to "statuses" in WP_Comment_Query::__construct() DocBlock
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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
@
5 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
@
5 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