#23058 closed defect (bug) (fixed)
Corrected inline docs for cache.php
Reported by: | ocean90 | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 2.5 |
Component: | Inline Docs | Keywords: | has-patch commit needs-codex |
Focuses: | Cc: |
Description
See [6543].
There are some copy&paste issues.
Attachments (2)
Change History (10)
#3
follow-up:
↓ 4
@
12 years ago
For the build in WP_Object_Cache, yes. But other implementations should be able to return false too. For example memcache::set.
#4
in reply to:
↑ 3
;
follow-up:
↓ 5
@
12 years ago
Replying to ocean90:
But other implementations should be able to return false too. For example memcache::set.
Then wp_cache_flush()
comment should probably also say "False on failure, true on success".
#5
in reply to:
↑ 4
@
12 years ago
Replying to SergeyBiryukov:
Then
wp_cache_flush()
comment should probably also say "False on failure, true on success".
Done in 23058.2.patch.
Note: See
TracTickets for help on using
tickets.
I guess the comment for
wp_cache_set()
should be "Always returns true", like the one forWP_Object_Cache::set()
:http://core.trac.wordpress.org/browser/tags/3.5/wp-includes/cache.php#L560
Same as for
wp_cache_flush()
andWP_Object_Cache::flush()
:http://core.trac.wordpress.org/browser/tags/3.5/wp-includes/cache.php#L88