Hello,

I use thickbox, imagecache and CCK imagefield with multiple images. So when I post a story I can attach several images to that particular node. I display them with thickbox.

The problem is that when I post 2 stories like that, each with multiple images attached, and I click on one thumbnail, thickbox would cycle through all the images of both nodes.

To replicate:
1) fresh install of drupal
2) install cck, imagefield, imagecache and thickbox
3) create two preset namespaces in imagecache, one for thumbnail and one for display
4) add imagefield to story content type, allow multiple images
5) in display fields, select in the drop down menu "thickbox_thumbnail"
6) post two stories each with two images
7) click on the first thumbnail of the first story, the image will pop up and the pager will show image 1 of 4

Is this a bug or am I doing something wrong?

Chris

Comments

frjo’s picture

frjo commented

Take a look at the value of the rel tag in the image links. They are what Thickbox use to group images.

It's the function theme_imagefield_image_imagecache_thickbox() that build these links so it's there that changes would need to be done.

dagomar’s picture

dagomar commented

same issue, subscribing

StuartMackenzie’s picture

StuartMackenzie commented

don't see it as a security issue but am subscribing as I experience same bug

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review

I can confirm this issue.

The problem is the function theme_imagefield_image_imagecache_thickbox

When you call the function, your 2nd argument will be used to group your images (the 'rel' part of an image). The function however expects your 2nd argument to be an array, so you have to add your 'rel' to an array prior to calling the function. I did the following:

$rel['type_name'] = $node->nid;
theme_imagefield_image_imagecache_thickbox($arg1, $rel, $arg2);

The solution to this problem is either to remind the users of adding the 'rel' to the specific array prior to calling the function, or (as i would prefer), to let the second argument be an ordinary variable rather than an array.

vm’s picture

vm commented
Status: Needs review » Active

There is no patch to be reviewed. Changing status.

frjo’s picture

frjo commented
Version: 5.x-1.0 » 5.x-2.0
Status: Active » Fixed

In Thickbox 2.x there are a setting to have galleries per n0de or per page.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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