Download & Extend
Keyboard shortcut utility » Issues

Remove t() from menu's description

Posted by pebosi on June 24, 2009 at 9:50am
Project:Keyboard shortcut utility
Version:6.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:fixed

Issue Summary

Small patch to remove use of t() in menu.

regards

AttachmentSize
shortcut_menu.patch793 bytes
Login or register to post comments

Comments

#1

Posted by nestor.mata on June 15, 2010 at 10:03pm
Status:needs review» closed (won't fix)

Removing the use of t() function is actually against the recommendations of Drupal, this would eliminate support for internationalization.
The use is also promoted in the guides http://api.drupal.org/api/drupal/developer--topics--forms_api_reference....

Will not fix, it is the way has to be.

~Nestor

#2

Posted by David_Rothstein on February 28, 2011 at 2:48am
Status:closed (won't fix)» reviewed & tested by the community

This patch is actually correct.

In Drupal 6 and higher, titles and descriptions should not be translated in hook_menu(), since they are translated later on display. See http://api.drupal.org/api/drupal/developer--hooks--core.php/function/hoo...

#3

Posted by nestor.mata on February 10, 2012 at 3:32pm
Status:reviewed & tested by the community» fixed

Implemented. Will be available in the next version