[wp-trac] [WordPress Trac] #30989: Additional audit for recent documentation changes
WordPress Trac
noreply at wordpress.org
Mon Jan 12 11:35:32 UTC 2015
#30989: Additional audit for recent documentation changes
----------------------------+------------------------------
Reporter: SergeyBiryukov | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version: 4.1
Severity: normal | Resolution:
Keywords: dev-feedback | Focuses: docs
----------------------------+------------------------------
Description changed by SergeyBiryukov:
Old description:
> Some of the documentation changes in #30224 need an additional audit for
> consistency:
>
> * [30673] added `null` as a possible `@return` value for
> `set_user_setting()` and `delete_user_setting()`, but it's not explained
> in the comment when it can be returned.
> * [30674] removed `int` as a possible `@return` value for
> `get_the_time()`, `get_post_time()`, and `get_post_modified_time()`, but
> they all depend on `mysql2date()` and can return an integer if `'U'` or
> `'G'` is passed. [30682] added `int` back, but only for
> `get_post_time()`.
> * [30678] added `string` as a possible `@return` value for
> `WP_Filesystem_SSH2::chown()`, but it's not explained in the comment when
> it can be returned.
New description:
Some of the documentation changes in #30224 need an additional audit for
consistency:
* [30673] added `null` as a possible `@return` value for
`set_user_setting()` and `delete_user_setting()`, but it's not explained
in the comment when it can be returned.
* [30674] removed `int` as a possible `@return` value for
`get_the_time()`, `get_post_time()`, and `get_post_modified_time()`, but
they all depend on `mysql2date()` and can return an integer if `'U'` or
`'G'` is passed. [30682] added `int` back, but only for `get_post_time()`.
* [30678] added `string` as a possible `@return` value for
`WP_Filesystem_SSH2::chown()`, but it's not explained in the comment when
it can be returned.
* There are probably more, these are just the ones that stood out to me.
--
--
Ticket URL: <https://core.trac.wordpress.org/ticket/30989#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list