Attached is a patch (against DRUPAL-5) that follows the Drupal 6 variable naming conventions in cache_clear_all(). The problem is, some variables within this function in Drupal 5 are named $key, while others are named $cid. As a result, wildcard caches are not being run.

CommentFileSizeAuthor
#1 cacherouter_cache_clear_all.patch787 bytesTBarregren
cacherouter.inc_.txt1.04 KBmundanity

Comments

TBarregren’s picture

TBarregren commented
Priority: Normal » Critical
StatusFileSize
new cacherouter_cache_clear_all.patch787 bytes

I think this is a critical bug, since Drupal can't clear the cache. However, mundanity patch introduce new bugs, because it renames $key to $cid. I therefore suggest the opposite, rename $cid to $key as in the attached patch.

psynaptic’s picture

psynaptic commented

Does this have any effect on the 6.x version? I'm using filecache and I can't clear cache unless I manually delete the files.

slantview’s picture

slantview commented
Status: Active » Fixed

Thank you so much. I renamed cid in all cacherouter.inc files to key. This should now work as expected. This was definitely a CRITICAL bug. I'm so sorry it has taken me so long to get this working correctly!

Status: Fixed » Closed (fixed)

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