I got message "warning: Invalid argument supplied for foreach()..." [#999986]

This is really strange. Everything worked, and all of a sudden this error shows up:

warning: Invalid argument supplied for foreach() in /home/path/sites/all/modules/colorbox/colorbox.module on line 337.

I did some update on drupal core, emfield and colorbox, today and yesterday, but this error was visible before that (i'm not sure, i didn't check recently :/), because it's indexed on Google :@

I tried to downgrade modules but nothing is changed. And colorbox popup is working, only problem I have is this warning and it's ugly to leave it as it is.
This is the site: http://msp.telekomapp.com/media/gallery/gallery-by-events/rolling-stones...

Does anyone have an idea?

Thanks!

Comment	File	Size	Author
#1	colorbox_link_alter_999986.patch	907 bytes	frjo
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

frjo CreditAttribution: frjo commented 16 December 2010 at 15:11

Assigned:	vojvotkinja	» Unassigned
Category:	task	» support
Priority:	Major	» Normal

File	Size
colorbox_link_alter_999986.patch	907 bytes

You should not write errors to the screen on a production site, set it to only log instead. Looks better for your users and unnecessary information is not revealed.

Have you tested stuff like clearing the cache etc?

I guess you are using the image module. The line in the error comes from the function colorbox_link_alter() that tries to add a colorbox class to the image link sizes on the image node.

You can also test this patch. It just ads a check to see if $node->images is an array or not.

Comment #2

vojvotkinja CreditAttribution: vojvotkinja commented 20 December 2010 at 09:26

Hm, I'm kinda newbie so I didn't knew about this option to turn of showing of warning messages, sweet :) I'm a little afraid to apply this patch because client will update modules and then patch will be overwritten, or am I wrong?
Thanks for your help!

Cheers!