This way, the tag clouds caches will be wiped each time a node or a taxonomy term is added/edited/deleted. At line 232, just replace
cache_set($cache_name, $tags, 'cache', CACHE_TEMPORARY);
by
cache_set($cache_name, $tags, 'cache_page', CACHE_TEMPORARY);

As cache_page is wiped each time a node or a taxonomy term is added/edited/deleted, there will be no inconsistency on the tag cloud.

Does that sound good?

CommentFileSizeAuthor
#1 tagadelic_cache_page.patch934 bytesDavid Stosik

Comments

David Stosik’s picture

David Stosik commented
StatusFileSize
new tagadelic_cache_page.patch934 bytes

By the way, cache_get shoul also be changed! :)
Here is a patch that can be tested on 6.x-1.x-dev and 6.x-1.2.

David Stosik’s picture

David Stosik commented
Status: Active » Needs review
David Stosik’s picture

David Stosik commented

Hello, anybody's opinion on this?
I changed the title to reflect my problem.
When I create or remove a tag, tagadelic's block displays an old cached version not taking into account the changes I just made.
If tagadelic blocs are cached in cache_page, then the problem is solved, as this cache table is flushed whenever an action on nodes or tags is done.

pyxio’s picture

pyxio commented
Title: Tagadelic cache should be set in cache_page table » Tagadelic block cache displays wrong tags

I'm having the same problem. Where is cache_page located???? Thanks! Kevin

David Stosik’s picture

David Stosik commented

Hello,
cache_page is a Drupal core cache table in the database.
Applying my patch only changes, in Tagadelic module, the cache table used, from 'cache' table, to 'cache_page' table.

pyxio’s picture

pyxio commented

Hi David,

Thanks for the heads up. I'm new to Drupal and have never installed a patch before. It looks like I need to copy and paste your code somewhere? Into one of the Tagadelic files? Can you please give me step-by-step instructions? As an aside, this seems to be a major issue. Is it being addressed in the next version? Cheers, Kevin

David Stosik’s picture

David Stosik commented

Hello iteego.
Either you follow the instructions to apply a patch, or, as this patch is simple, you open tagadelic.module with a text editor, then, replace the lines in the patch starting with a '-' by those starting with a '+'.

Regards,
David

pyxio’s picture

pyxio commented

Hi David,

Many thanks for the information. I appreciate. Cheers, Kevin

Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs review » Fixed

Looks good to me.
Applied and committed to DRUPAL-6--1 (available in next -dev nightly).

Will be included in next tagadelic release.

Status: Fixed » Closed (fixed)

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