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

Comments

dsdeiz’s picture

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 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 commented
StatusFileSize
new make-under-construction-translatable-1846700-3.patch692 bytes

Hi! Attached patch.

donquixote’s picture

donquixote commented
Status: Active » Needs review

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

dsdeiz’s picture

dsdeiz commented

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

donquixote’s picture

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.