It should be at the very least be hook_menu_link_translated_alter().

It should even better not have the word "translated" in the name since that implies entirely different things than what it's actually about.

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Issue tags: +API clean-up

Tagging for D8 API clean-up phase.

Agreeing on the proposed hook name, but disagreeing with changing/removing "translated", as long as the menu system keeps on using this term everywhere else in its code and documentation.

damien tournoud’s picture

damien tournoud commented
Title: hook_translated_menu_link_alter() is named stupidly » hook_translated_menu_link_alter() name is unclear

There is nothing "stupid" here. The "translation" operation has a very clear and precise meaning in the menu system. If we want to change the name of the hook, we also need to change all the documentation and the name of several functions in API.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: hook_translated_menu_link_alter() name is unclear » Change hook_translated_menu_link_alter() into hook_menu_link_translated_alter()

Yup, fully agreed, so let's use an even more precise title.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

This hook has been removed entirely.