![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#51690 closed defect (bug) (fixed)
Change "stati" to "statuses" in WP_Comment_Query::__construct() DocBlock
| Reported by: | Collizo4sky | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.6 |
| Component: | Comments | Version: | 5.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
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
@
6 years ago
- Focuses docs added
- Keywords 2nd-opinion added
- Version → 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
@
6 years ago
- Component General → Comments
- Keywords 2nd-opinion removed
- Milestone Awaiting Review → 5.6
- Owner set to
- Status new → reviewing
- Summary Typo fix in docbloc → 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