We had a similar issue in menu_editor 6.x #1037880: Translatable or configurable Strings
The string "Under Construction" can't be localized using Drupal's translation interface as it's not wrapped with a t() function.

Here's a patch to fix this, works for me.

CommentFileSizeAuthor
#3 make-under-construction-translatable-1846700-3.patch692 bytesdsdeiz
0001-Module-patch-made-Under-Construction-string-of-menu_.patch2 KBDaniel Wentsch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsdeiz’s picture

dsdeiz CreditAttribution: dsdeiz commented 
-    'title' => 'Under Construction',
+    'title' => t('Under Construction'),

I don't think this one is required. hook_menu docs say:

"title callback": Function to generate the title; defaults to t(). If you require only the raw string to be output, set this to FALSE.

And its gonna be overridden in menu_editor_placeholder_page anyway.

donquixote’s picture

donquixote CreditAttribution: donquixote commented

Yes, the title in hook_menu() will be translated automatically.

"its gonna be overridden in menu_editor_placeholder_page anyway"
It is still used for breadcrumbs and stuff.

Could you re-roll the patch w/o the hook_menu() stuff?

dsdeiz’s picture

dsdeiz CreditAttribution: dsdeiz commented
FileSize
make-under-construction-translatable-1846700-3.patch692 bytes

Hi! Attached patch.

donquixote’s picture

donquixote CreditAttribution: donquixote commented
Status: Active » Needs review

Looks good. If it doesn't WSOD, we shall commit it.

dsdeiz’s picture

dsdeiz CreditAttribution: dsdeiz commented

Perhaps this can be committed? Change is only minor anyway.

donquixote’s picture

donquixote CreditAttribution: donquixote commented
Status: Needs review » Fixed

yes.
http://drupalcode.org/project/menu_editor.git/commitdiff/f0db4d6be7b755b...

Status: Fixed » Closed (fixed)

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