I wish to set primary menu in site map but the translation for text "primary link" does'nt work. In UK or others languages, I always have "primary links"

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

isaakordonez’s picture

Suscribe I have the same problem...

kaldimar’s picture

I am subscribing too... when installing the Spanish translation of the core, the menu titles are translated (Enlaces Primarios, etc) and their English counterparts are lost from everywhere. This issue became apparent in the Menus section under Site Building, and of course on the Site Map but I'm afraid it is not a site_map issue, rather it comes from the core translation itself... I'm going to research a little more and if I got news I'll be posting here.

cpo’s picture

After some tests, translation for all menus titles do not work.

LUTi’s picture

I've added t() to $menu['title'] at line 275 (site_map.module file):
$title = t($menu['title']);

and it seems to work fine for me.

Simon Georges’s picture

Status: Active » Needs review
FileSize
444 bytes

Hi,

I made the exact same patch than #4 (and I attach it). I confirm it works for me too.

Regards,

cpo’s picture

#4 work fine for me

Simon Georges’s picture

Status: Needs review » Reviewed & tested by the community

Changing the status as several people confirmed it works.

isaakordonez’s picture

#4 Working for me Thanks!!! :)

JamesBecht’s picture

#4 works fine ! Muchas gracias !

doomed’s picture

Works nicely after you add t() .

On version 6.x-2.2 this is on line 277.

diego.banchero’s picture

Version: 6.x-2.1 » 6.x-2.2
FileSize
512 bytes

Yes it works. Here the patch for 6.x-2.2

tiziano.sartori’s picture

Version: 6.x-2.2 » 7.x-1.0
Component: User interface » Code
Assigned: Unassigned » tiziano.sartori
Status: Reviewed & tested by the community » Needs work

Same problem on 7.x-1.0 and 7.x-1.x-dev, solved with t() function

mariomc’s picture

Solved with t() function here as well!

darrell_ulm’s picture

Title: Label "primary links" no translated » Label "primary links" not translated
Status: Needs work » Needs review

Looks possible. Seems like many people tested this.

darrell_ulm’s picture

Status: Needs review » Patch (to be ported)

OK, just looked above and it appears to be 'patch to be ported'

h3rj4n’s picture

The dev version currently has this fix in it's code:

$title = $menu['title'];
if (module_exists('i18n_string')) {
  $title = i18n_string_plain(array('menu', 'menu', $menu['menu_name'], 'title'), $title);
}

If so, it should be mentioned on the home page of this module that sitemap requires the module i18n if you want to translate it.

For now I'm going to use te patch supplied in post #4 / #5.

darrell_ulm’s picture

I'm looking at porting the patch and getting a new release out this week. The last two weeks were pretty busy. Thanks.

darrell_ulm’s picture

Version: 7.x-1.0 » 6.x-1.x-dev

It looks like this functionality is already in 7.x-dev so moved this issue to version 6.x

justingeeslin’s picture

Hi folks,

The patch here works for me, too.

Is this still not in the 6.x?

darrell_ulm’s picture

I'm more interested in the updating the D7 version of Site_Map, who has been handling the D6 version?

paulocs’s picture

Issue summary: View changes

As site map 6.x is no longer supported and this issue was already fixed in 7.x, I'll close this issue.
Thanks.

paulocs’s picture

Status: Patch (to be ported) » Fixed
paulocs’s picture

Assigned: tiziano.sartori » Unassigned
Status: Fixed » Closed (fixed)