Notice: Undefined index: localized_options in menu_link_field_formatter_view() (line 375 of /sites/all/modules/menu_link/menu_link.field.inc).
Recoverable fatal error: Argument 3 passed to l() must be an array, null given, called in /includes/common.inc on line 5363 and defined in l() (line 2307 of /includes/common.inc).

error occurs on every page on site and cause lots of other problems.

Not obvious if I can disable the module without losing the fields that have been defined and their content.

Running current -dev and seeing other recent fixed errors that look they were causing other problems on the site, like incompatible with admin_menu so I upgraded. I agree that this belongs as a -dev and not an alpha to warn people that it is unstable.

CommentFileSizeAuthor
#1 menu_link-errors_when_hidden-1487830-1.patch1.47 KBjohntarling

Comments

johntarling’s picture

johntarling commented
Status: Active » Needs review
StatusFileSize
new menu_link-errors_when_hidden-1487830-1.patch1.47 KB

I was getting this error too and traced it to the fact that I had chosen to hide the menu link field on the display of a node and then had put a view on the page which was trying to make use of the menu link. The menu link formatter was attempting to access properties which weren't available to it.

I created a patch which sets up a menu link with the properties which are available if the default "Link" formatter has not been selected.

This seems to have sorted the issue I was having, my view now displays and there are no errors being generated.

casey’s picture

casey commented

I fixed it a little different. Let me know if it works.

http://drupalcode.org/project/menu_link.git/commit/83299c8

johntarling’s picture

johntarling commented
Priority: Normal » Critical
Status: Needs review » Needs work

Hi casey,

Thanks for the fast reply. I tried the latest version you have committed but unfortunately it has compile errors. There was a bracket missing on line 329 and then there was a "Only variables can be passed by reference" error on line 384. I don't have time to debug this at the moment so have reverted back to the previous version for now. I have changed this to critical now since the module will break a site in its current state.

casey’s picture

casey commented

Sorry about that. Didn't really test it. How about now?

s_leu’s picture

s_leu commented
Status: Needs work » Closed (fixed)

Seems to be fixed since there are no more complaints of johntarling and yktdan. Reopen if the problem appears again.