Download & Extend

cache_clear_all() in cacherouter.inc

Project:Cache Router
Version:5.x-1.0-beta9
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

AttachmentSize
cacherouter.inc_.txt1.04 KB

Comments

#1

Priority:normal» critical

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.

AttachmentSize
cacherouter_cache_clear_all.patch 787 bytes

#2

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.

#3

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!

#4

Status:fixed» closed (fixed)

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

nobody click here