I was struck today by the fact that cache_get() and cache_set() don't @see each other. It seems like a natural reference, as I kept finding myself wanting to jump between them while looking on a.d.o. Note that this does not apply to D8 as both of these functions are gone, replaced by the new pluggable cache objects.

Patch to follow.

Files: 
CommentFileSizeAuthor
#8 cache_docs-1421330-d6-8.patch782 bytesAlbert Volkman
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch cache_docs-1421330-d6-8.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#5 cache_docs-1421330-d6-5.patch782 bytesAlbert Volkman
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch cache_docs-1421330-d6-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 1421330_cache_docs.patch786 bytesheyrocker
PASSED: [[SimpleTest]]: [MySQL] 37,003 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new786 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,003 pass(es).
[ View ]

Status:Needs review» Reviewed & tested by the community

Useful and logical, seeing as both variable_get and variable_set have '@see's to each other.

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 7.x. Thanks!

Version:7.x-dev» 6.x-dev
Status:Fixed» Patch (to be ported)

How about a D6 backport?

Status:Patch (to be ported)» Needs review
StatusFileSize
new782 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch cache_docs-1421330-d6-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

D6 backport.

Oops, double post. Ignore.

Status:Needs review» Needs work

You have @see cache_set on cache_set() :)

StatusFileSize
new782 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch cache_docs-1421330-d6-8.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

How to keep a developer busy :) Thanks for the catch!

Status:Needs work» Needs review

Status:Needs review» Reviewed & tested by the community

Looks good this time. :)

Status:Reviewed & tested by the community» Fixed

Thanks, committed, pushed.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.