When saving my Main menu on "admin/structure/menu/manage/main-menu", I got the following Notice:

Notice: Undefined index: attributes in i18n_menu_menu_link_update() (regel 273 van /home/diederik/Openbaar/dev/mobiel21.statiklabs.be/sites/all/modules/i18n/i18n_menu/i18n_menu.module).

I reordered some of the links (moved a first-level item with a subtree under another first-level menu-item) and disabled the "Home" link.

The settings seem to be saved though.

CommentFileSizeAuthor
#2 1125816.patch3.12 KBfietserwin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fietserwin’s picture

patch attached (patch does contain some white space removal). The only line that really changed:

from:

  if (!empty($link['customized']) && i18n_menu_mode($link['menu_name'], I18N_MODE_LOCALIZE) && !i18n_object_langcode($link)) {

to:

  if (isset($link['options']['attributes']['title']) && !empty($link['customized']) && i18n_menu_mode($link['menu_name'], I18N_MODE_LOCALIZE) && !i18n_object_langcode($link)) {

This situation (i.e. !isset($link['options']['attributes']['title']) is to be expected as not all menu items will define a title attribute as this is the on-hover tooltip, not the menu title itself.

fietserwin’s picture

Version: 7.x-1.0-beta4 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
3.12 KB

and now the patch...

joostvdl’s picture

Patch from #2 works

Jose Reyero’s picture

Version: 7.x-1.x-dev » 7.x-1.0-beta5
Status: Needs review » Fixed

Fixed in latest -dev.

Thanks for the patch.

joostvdl’s picture

Version: 7.x-1.0-beta5 » 7.x-1.x-dev

This patch is not in -beta5, only in -dev

Status: Fixed » Closed (fixed)

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