Beta1 to Beta2 breaks Lightbox2 support

I have encountered the same problem. Looking forward to a fix too. (Though I appreciate the progress in the gallery views a lot !)

Same problem for me. Subscribing.

Same problem, also subscribing.

Thanks for the progress and I'm sure a fix is right around the corner. The shift to using Views is a bit jarring for me (I'm a Views-newbie) but I'll get the hang of it.

This might be a separate issue, but how do we get the gallery descriptions to display again? When viewing the /image page in Drupal 6, it used to show the paragraph descriptions that we put for each gallery. Now it only says how many images are in the gallery and when it was last updated. Is this is a Views thing that I need to figure out? Just curious, because we really liked it when it showed the information about the gallery. Thanks!

This might be a separate issue, but how do we get the gallery descriptions to display again?

This IS a separate issue, but I can help with that:

• Go to: /admin/build/views/edit/image_gallery_terms
• click on the plus sign next to "Fields"
• select "taxonomy" from "groups"
• check "taxonomy: description" and hit the "Add"-button
• clear the input field "description" and hit the "Update" button
• finally hit the "Save" button and you're done

I am not quite sure, whether i named the fields / buttons correctly. On my page it is all translated to German. Hope this helps.

There is the same problem with Thickbox

+1 Me too. Subscribing.

Hi folks, sorry for the sidebar conversation but I wanted to thank fischerj (post #4, above) -- I greatly appreciate your help and the steps you describe worked like a charm. Thanks so much!

Now, about that Lightbox2 ... ;-)

Hi all,
This is probably due to the markup changing as the gallery is now generated by views.
I'll have a look when I get a chance.

subscribing

same problem here. subscribing...

Links with rel="lightbox" added manually to a node are still working, though.
The automatic handling settings stopped working. There's no rel="lightbox" added to the random image block and to the galleries for the slideshow.

Duplicate of #565774: Thickbox printing curious <img>-tag instead of title of image
as far as I can tell.

alpha5, alpha6 and beta1 worked for me using lightbox2,
so i'm not sure if this really is a duplicate,
but i will try your patch from #565774 as soon as possible

http://drupal.org/node/565774#comment-2015540 doesn't work for Lightbox2 so this is probably not a duplicate.

It was the patch at
http://drupal.org/node/565774#comment-2008866
that should have fixed it.

Have you customised your views?

The javascript in Lightbox2 expects certain markup, so if you have varied from that, it won't work.

Any public version where I could take a look?

Not sure we're all talking about the same issue here.

My problem is not related to any gallery views really, the whole Lightbox functionality is broken.

I have beta2 of the Image module installed, 1.0 of the Upload Image module installed (http://drupal.org/project/upload_image) as I need the image nodes automatically created from attached images and then finally, the 1.9 of the Lightbox2 module (http://drupal.org/project/lightbox2). Straight out-of-the-box, no customization.

Before installing beta2, the image thumbnails (= image attachments) in my nodes opened up in a Lightbox. This seems broken now. Tried it with the Thickbox module too, to no avail.

edit: http://www.soinilkv.fi/kohteet/testi

> image thumbnails (= image attachments

Are they actually added to the original node as image_attach attachments? Reading that project page doesn't suggest this, rather a custom approach. This would make the problem to do with upload_image module, not this.

If I may, here's a real world, production example of what I think turbanov may be referring to...

http://www.johnnyguitars.com/content/brands/fender-squire

This site displays guitars and other music gear. Gear images are uploaded to the site using Image. They are available in galleries, but that isn't the primary purpose for adding them. Each product item has one of these images attached, via image_attach. (any additional images added to one product are added with image_assist, but most items only use one image)

When viewing a list of the products the thumbnails behave as links to the product item's node. When viewing the node, the image is shown in a preview size, and clicking on that will invoke Lightbox for a nice big view of a guitar or whatever the item is.

I tried beta2 on this site over the weekend, but it broke the ability to view these product images in Lightbox. Instead, clicking the images would simply take you away from the product item node and to the image node. We'd like to show that original sized view without leaving the product it is selling.

So I backed out of beta2 and restored the db from backups and went back to using alpha 5, because, for us, it works as desired.

Subscribing. Plz unbreak.

+1 - my experience of the upgrade is exactly the same as byrock.

no lightbox markup is being emitted in to the gallery html so an image gallery image links straight to the image node page.

Same problem here. Subscribing.

Alanburke, answering #15: I have slightly customized the view in order to have 20 images per page instead of 12.

You could have a look at sg-kaarst-volleyball.de if you speak some german words ;-) Otherwise: Look at the top-right menu, the german translation for "image galleries" is "Bildergalerien". If you need more info or admin login or even access to the webserver, then let me know.

Are they actually added to the original node as image_attach attachments? Reading that project page doesn't suggest this, rather a custom approach. This would make the problem to do with upload_image module, not this.

Given that I'm not exactly sure what you're referring to here and don't really have the time to get to the bottom of it, you maybe correct, but the fact of the matter is that it worked before and doesn't now and thus, this beta2 update is what caused it. That's all.

Anyways, thanks Alan (and everybody else) for your efforts, they are very much appreciated.

Hello again,

Our site is another that uses image and also Lightbox2, and this functionality worked great through the alphas and beta1, but broke with beta2.

We did not use image_attach but rather simply created image node content, associated the image nodes with a particular gallery, and let Lightbox2 work its magic. As stated in previous comments, when we click on a thumbnail, the system now displays the image node rather than kicking in the Lightbox2 effect.

Link to our list of galleries -- http://photos.berkleydems.org
Link to a particular gallery -- http://pubcrawl.berkleydems.org

The system works correctly on the list of galleries; we click on a thumbnail to display the corresponding gallery itself, etc. Lightbox2 didn't jump in for these pics (via a setting in the Lightbox2 configuration as I recall).

However, once viewing a particular gallery, a click on a thumbnail should invoke Lightbox2 rather than bring up the image node. Note that clicking on the caption text under a photo also displays the image node, but it did this before beta2 and it lets folks leave a comment about a particular photo, etc.

(Perhaps another unrelated issue, but the gallery view alignment is now off with the thumbnails in each gallery. It's centering them and the consequence is that items appear staggered because of the length of the caption beneath the photo. Is there a way to top align each row, instead of center align each row?)

Thanks again, and please let me know if there's anything we can do to help test anything, etc.

If you're not using image attach: please see #565774: Thickbox printing curious <img>-tag instead of title of image and try the patch there.

If you ARE using image attach: the only change from b1 to b2 that could affect you is: #567226: image attach views field produces multiple rows for each attached image.
In this case, the image attach Views field has completely changed due to the multi-attach feature.
It's very likely that *boxes aren't getting the markup they expect any more.
If there's anyone who can have a dig in the markup and tell me what's required to change for it to work, that would be a huge help.

I have image_attach enabled, but i guess this doesn't mean i'm using it, right?
i have an example gallery here: http://bdp-sh-hh.de/category/bildergalerien/bundeslager-09
the only customization is that there are 9 images instead of 12

in comment #15 alanburke linked to comment#1 in issue#565774,
but that patch is already included in image beta2,
or at least i find in line 16 of image-view-image-gallery.tpl.php
<<?php print $options['type']; ?> class="images">

lightbox2 worked with alpha5, alpha6 and beta1 of the image module,
but as i understand it issue #565774 is all about alpha5->alpha6

There is another patch at #565774: Thickbox printing curious <img>-tag instead of title of image, further down. Please test it if you are able.

It would greatly help matters if people explained exactly where they are looking at images and how they are being produced.
Where: gallery? gallery view? node?
How: actual image for the node? image_attach? some other module I've never heard of? view field?

Okay, I was about to say this was total crack as there's nothing in the changelog that could have done this, but I see from the diff for the only change to the image.module file itself that something did change, which doesn't match up to the commit message.
It looks like code sprint fatigue last Saturday... :(

Will post a patch for testing very shortly. If you feel brave, you can try hacking out those lines yourselves ;)

Hi
I have the problem with thickbox,

I don't use image galleries and this is not a views problem (as far as I know)

i use image module to put images inside of the body of a node

after upgrading image from alpha6 to beta2 thickbox inside the node body stopped working
I use image attach or image assist (frankly I am confused which does what but I think they work together) to attach image nodes inside of the body field

Basically I click on the little photo camera symbol in the wysiwyg editor, a pop up window opens, I upload a picture (as a node) and select "thumbnail" and "full node" for the "show as" resp "link to" settings

before the upgrade clicking on an image opened it in thickbox displaying the link to it as http://mysiteurl/sites/default/images/[imagename].jpg) after upgrading clicking on one of the images just opens the node page displaying the link differently as http://mysiteurl/node/[nodeid]

I since then reverted to before the module/db update and now thickbox works again
you can take a look at the working version http://www.cat-x.at/news/opening-relative-realities-mexico-city.html

Nice job! Congratulations on your first patch :D

I've also removed the exact change that commit made, so there's a heap of whitespace too.
It seems I was in the middle of testing a different patch on the same codebase ... :(

Committed to CVS:

#569526 by 2noame: Fixed mistaken commit to image.module from #569258.

A new dev version will be created by the system shortly. Please give it a try :)

ok, commenting out lines 641-644 with a // does the trick for me
thanks joachim and 2noame!

sorry, didn't mean to change the status

Outstanding news! I look for to testing it out on a production environment this weekend, during scheduled maintenance. Thanks.

Yeah I can confirm this does indeed fix it. Changed the status back though as those lines are there probably for a reason.

Thanks anyways to all participants.

Works for me on image galleries too, but inline node images attached with Image Assist still open the node for the image.

Those lines were there because I had applied a patch from another issue, then got distracted, then applied alanburke's patch for something completely different, and then committed, as it was a one-liner. DOH!

@#39: you should probably open an issue on image assist for that.

Ah! Shoot to early. I just had to clear the filter cache with calling http://<page_url>/img_assist/cache/clear. Maybe this helps others too. Thanks for the quick fix!

+1