Hi,

I installed the latest version earlier today (7.x-1.x-dev, 2011-Aug-15) and I'm getting the following error message:

Notice: Undefined variable: caption in theme_colorbox_image_formatter() (line 99 of .../sites/all/modules/colorbox/colorbox.theme.inc).

when the caption is set to "Colorbox caption: Automatic".

Patch to follow.

Thanks for a great module.

CommentFileSizeAuthor
#2 colorbox_caption_bugfix_ 1249902_2.patch884 bytesfrjo
#1 auto_caption_setting-1249902.patch453 bytesLinL
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LinL’s picture

LinL CreditAttribution: LinL commented
FileSize
auto_caption_setting-1249902.patch453 bytes
frjo’s picture

frjo CreditAttribution: frjo commented
FileSize
colorbox_caption_bugfix_ 1249902_2.patch884 bytes

Thanks for finding this and for the patch!

The patch works well but could you test my version as well?

The result should be the same but I think my version makes it a bit more easy to see what is happening in the code.

spacereactor’s picture

spacereactor CreditAttribution: spacereactor commented

#2 work for me

LinL’s picture

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

Yes, that's great, patch in #2 works for me too.

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Reviewed & tested by the community » Fixed

Thanks for testing! Committed to 7-dev.

Status: Fixed » Closed (fixed)

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