When I updating the term description, it saves to database, but on node pages it shows without changes. This is because function cache_clear_all() in file cacherouter.inc flush only 'cache_block' and 'cache_page' but must flush 'cache_taxonomy' too.

Comments

murz’s picture

Title: cache_clear_all must clear cache_taxonomy cache too » cache_clear_all must clear all cache items, not only cache_block and cache_block

After using cacherouter with module Advanced cache on working high-load site, I see that function cache_clear_all() must clear not only cache_block and cache_page, but all exists caches. But I don't know how to get list of all exists cache names for cleaning. Maybe anybody knows this?

murz’s picture

Title: cache_clear_all must clear all cache items, not only cache_block and cache_block » cache_clear_all must clear all cache items, not only cache_block and cache_page
Status: Needs review » Postponed (maintainer needs more info)
murz’s picture

Title: cache_clear_all must clear all cache items, not only cache_block and cache_page » APC cache type don't delete keys with non-empty $key variable
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new595 bytes

Sorry, I think I found the bug. Bug is when we using APC cache and in apc.php function delete($key) don't work with $key isn't empty!
We need to change
<? if (!empty($key) && !empty($value)) { ?>
to
<? if (!empty($key)) { ?>
Patch is attached.

slantview’s picture

Status: Needs review » Fixed

Fixed in latest commits.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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