cache_clear_all() in cacherouter.inc
mundanity - March 30, 2009 - 22:02
| Project: | Cache Router |
| Version: | 5.x-1.0-beta9 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed |
Description
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.
| Attachment | Size |
|---|---|
| cacherouter.inc_.txt | 1.04 KB |
#1
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.
#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
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
Automatically closed -- issue fixed for 2 weeks with no activity.