#23058 closed defect (bug) (fixed)
Corrected inline docs for cache.php
| Reported by: | ocean90 | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Inline Docs | Version: | 2.5 |
| Severity: | normal | Keywords: | has-patch commit needs-codex |
| Cc: | Focuses: |
Description
See [6543].
There are some copy&paste issues.
Attachments (2)
Change History (10)
#3
follow-up:
↓ 4
@
14 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
@
14 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
@
14 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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