There is a little bug with the cache code. The cache name just used the vid's imploded with no sperator eg,
vids[] = 2;
vids[] = 3;

thus the id is ..._23_...

which would be the same cache name for vocab 23.

The patch fixes this my just using _ as a seperator for the vid's.

CommentFileSizeAuthor
tagadelic.module.patch735 bytesthebaron88

Comments

Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs review » Needs work

Good catch and nice solution.

Can one more person please test this patch and confirm that it:
* does not break a site with existing data
* that the old 231_-type cahced items dissapear after a while: we don;t want them to clutter the cache :)

Bèr

Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs work » Needs review

Sorry. Should be "code needs review"

Bèr Kessels’s picture

Bèr Kessels commented
Priority: Critical » Normal
neochief’s picture

neochief commented
Status: Needs review » Reviewed & tested by the community

Works for me

chrissearle’s picture

chrissearle commented

Works here too. Except for

* that the old 231_-type cahced items dissapear after a while: we don;t want them to clutter the cache :)

But - if http://drupal.org/node/305380 goes in then they'll go ;)

chrissearle’s picture

chrissearle commented

Applied patch to 6.x-1.2 - cache is cleared correctly (the old ones do disappear).

I believe this one ready to go in ;)

Bèr Kessels’s picture

Bèr Kessels commented

committed to D6 branch.

Bèr Kessels’s picture

Bèr Kessels commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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