correct menu title translation [#870360]

Hello,
- $title = check_plain($result);
line 376 replace to this?
+ $title = (function_exists('tt')) ? tt('menu:menu:'. $menu_name .':title', check_plain($result)) : check_plain($result);

Is it bad solution?

Comment	File	Size	Author
#4	nice_menus-title-translation-870360-4.patch	1.17 KB	reszli
#3	nice_menus-title-translation-870360.patch	388 bytes	mrf

Comments

Comment #1

bsztreha commented 15 November 2010 at 11:36

Version:	6.x-2.1-beta2	» 6.x-2.1
Status:	Needs review	» Active

Anyone can comment it?

Comment #2

add1sun commented 9 December 2010 at 12:54

Seems sane. I don't have any multilingual sites set up and haven't had a chance to test this. If someone can review and give a +1, I'll roll it in.

Comment #3

mrf commented 17 December 2012 at 20:49

Version:	6.x-2.1	» 7.x-2.x-dev
Category:	feature	» bug

Status	File	Size
new	nice_menus-title-translation-870360.patch	388 bytes

$title already gets run through a check_plain before it hits this point so this simpler solution works for me.

Comment #4

reszli

he/him

Hungarian

Târgu Mureș

commented 2 August 2013 at 08:20

Status	File	Size
new	nice_menus-title-translation-870360-4.patch	1.17 KB

additionally to the title of the menu, I also needed to add a couple of t() functions for the menu items as well
to make a nicemenu block render the menu items in the current language