panels_views.module error on line 699

Proper patches, please: http://drupal.org/patch/create.

I was receiving the same foreach() warning on the patched line and on two other lines. An updated patch is attached.

A simpler patch would be to make sure $pv->contexts is always an array() when the object is created/loaded -- and this is also more in line with how the rest of Panels operates.

Okay. Thank you. Like this?

I solved this using a simpler patch - prevents records with contexts = "N;" from entering the {panels_views} table.
Not sure this is the cleanest way to do this, though. At least you can pick :-)

status

Thank you. Unfortunately your patch (in comment #5) doesn't work for me. I receive the same foreach() warning as without the patch, just moved down a line from 699 to 700. Any thoughts why?

That's because the patch in #5 doesn't automatically fix your existing panels_views. You need to resave them (no need to change anything)

Perfect. Thank you! I'm using your #5 patch.

Works for me as well. :)

Patch in #5 is exactly how I do it elsewhere. Thanks! I'll commit this fairly soon.

Committed! THanks for the work on this!

I just tried the latest CVS today, unfortunately, I am getting the error above. But when I apply patch #4, I don't get the warnings.

@jmai : see #8 above - "That's because the patch in #5 (the one that got committed) doesn't automatically fix your existing panels_views. You need to resave them (no need to change anything)"

Sorry I wasn't detailed explaining my problem. I did resave and I don't get the warnings anymore, except only when I configure the Mini panels on the content page. That is strange.

How do you apply the patch? thanks

alanbold : the patch has already been committed (see comment #11), so you don't need to apply it manually, simply grab latest 5.x-2.x-dev release (http://drupal.org/node/128199)
for general instructions on how to apply patches, see http://drupal.org/patch

This has fixed my 699 error I now have a line 701 and 739 error.

Any ideas?

jmai, timmilwood: Those might be different problems. Please open new issues with specifics. Thanks.