Since Drupal 7.33 there is a preferred_menu_links tag added to the database query that is used by menu_link_get_preferred() to find the preferred menu link for a given path.

This allows us to remove all the code in that is currently in i18n_menu_init and replace it with one line of code in hook_query_preferred_menu_links_alter().

Patch attached removes the hook_init and replaces it with a hook_query_preferred_menu_links_alter.

CommentFileSizeAuthor
i18n_menu_replace_hook_init_with_hook_query_preferred_menu_links_alter.patch4.27 KBJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeroenT created an issue. See original summary.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs review » Postponed (maintainer needs more info)

i18n_menu_init was removed

if this doesn't fix the issue above, please let me know either way.

The issue where i18n_menu_info was removed is described below:
#881322: Multilevel menu fails to create active menu trail with same link_path in different languages

it was replaced with:

i18n_menu_query_preferred_menu_links_alter

by @stefan.r

Thanks.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Postponed (maintainer needs more info) » Fixed
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

@JeroenT , thanks for the followup.
We should probably release 7.x-1.15 soon.

Status: Fixed » Closed (fixed)

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