When you have a content type with an image field and you add or remove the image you get an exception thrown:

Notice: Undefined property: stdClass::$storage in og_menu_form_node_form_alter() (line 331 of /var/www/html/sites/all/modules/og_menu/og_menu.module).

The issue appears to be that in this case the og_context is not set but the result from og_context() is an empty array. Will supply a patch shortly that I think resolves this issue.

CommentFileSizeAuthor
#1 og_menu-undefined_property_expection-1993718-1.patch819 bytespgillis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pgillis’s picture

pgillis CreditAttribution: pgillis commented
Status: Active » Needs review
FileSize
og_menu-undefined_property_expection-1993718-1.patch819 bytes
rv0’s picture

rv0 CreditAttribution: rv0 commented
Status: Needs review » Closed (cannot reproduce)

I dont see this happening in the 3.x branch, in fact, there's no $storage variable there.

This could have happened in 2.x branch, though I think there's a safety for it already (in dev perhaps)

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Issue summary: View changes
Status: Closed (cannot reproduce) » Fixed

This is very much applicable in 3.0. I don't see it as an issue in HEAD though, so I'm flipping this over to fixed instead of unable to reproduce.

rv0’s picture

rv0 CreditAttribution: rv0 commented

Thanks for this info.
I should find some time to roll a new release (and commit some other stuff)

Status: Fixed » Closed (fixed)

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