![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 9 years ago
Closed 8 years ago
#42795 closed defect (bug) (fixed)
docs: Missing @return for WP_Comment_Query::get_comment_ids()
| Reported by: | birgire | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Comments | Version: | 4.4 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description (last modified by )
The inline documentation for WP_Comment_Query::get_comment_ids() is missing the @return description.
There are two return cases:
- (int) with the count query variable as true
- (array) for the comment ids
See here:
https://core.trac.wordpress.org/browser/tags/4.9/src/wp-includes/class-wp-comment-query.php#L889
We can e.g. look into WP_Site_Query::get_site_ids() for a similar case.
Good-first-bug?
References:
https://core.trac.wordpress.org/browser/tags/4.9/src/wp-includes/class-wp-comment-query.php#L468
https://core.trac.wordpress.org/browser/tags/4.9/src/wp-includes/class-wp-site-query.php#L318
Attachments (1)
Change History (8)
Note:
See TracTickets
for help on using tickets.
Created patch to add @return for WP_Comment_Query::get_comment_ids()