Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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"
Comment | File | Size | Author |
---|---|---|---|
#11 | site_map-translation_label_menu-853818-11.patch | 512 bytes | diego.banchero |
#5 | 853818-t.patch | 444 bytes | Simon Georges |
Comments
Comment #1
isaakordonez CreditAttribution: isaakordonez commentedSuscribe I have the same problem...
Comment #2
kaldimar CreditAttribution: kaldimar commentedI 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.
Comment #3
cpo CreditAttribution: cpo commentedAfter some tests, translation for all menus titles do not work.
Comment #4
LUTi CreditAttribution: LUTi commentedI'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.
Comment #5
Simon Georges CreditAttribution: Simon Georges commentedHi,
I made the exact same patch than #4 (and I attach it). I confirm it works for me too.
Regards,
Comment #6
cpo CreditAttribution: cpo commented#4 work fine for me
Comment #7
Simon Georges CreditAttribution: Simon Georges commentedChanging the status as several people confirmed it works.
Comment #8
isaakordonez CreditAttribution: isaakordonez commented#4 Working for me Thanks!!! :)
Comment #9
JamesBecht CreditAttribution: JamesBecht commented#4 works fine ! Muchas gracias !
Comment #10
doomed CreditAttribution: doomed commentedWorks nicely after you add t() .
On version 6.x-2.2 this is on line 277.
Comment #11
diego.banchero CreditAttribution: diego.banchero commentedYes it works. Here the patch for 6.x-2.2
Comment #12
tiziano.sartori CreditAttribution: tiziano.sartori commentedSame problem on 7.x-1.0 and 7.x-1.x-dev, solved with t() function
Comment #13
mariomc CreditAttribution: mariomc commentedSolved with t() function here as well!
Comment #14
darrell_ulm CreditAttribution: darrell_ulm commentedLooks possible. Seems like many people tested this.
Comment #15
darrell_ulm CreditAttribution: darrell_ulm commentedOK, just looked above and it appears to be 'patch to be ported'
Comment #16
h3rj4n CreditAttribution: h3rj4n commentedThe dev version currently has this fix in it's code:
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.
Comment #17
darrell_ulm CreditAttribution: darrell_ulm commentedI'm looking at porting the patch and getting a new release out this week. The last two weeks were pretty busy. Thanks.
Comment #18
darrell_ulm CreditAttribution: darrell_ulm commentedIt looks like this functionality is already in 7.x-dev so moved this issue to version 6.x
Comment #19
justingeeslin CreditAttribution: justingeeslin commentedHi folks,
The patch here works for me, too.
Is this still not in the 6.x?
Comment #20
darrell_ulm CreditAttribution: darrell_ulm commentedI'm more interested in the updating the D7 version of Site_Map, who has been handling the D6 version?
Comment #21
paulocsAs site map 6.x is no longer supported and this issue was already fixed in 7.x, I'll close this issue.
Thanks.
Comment #22
paulocsComment #23
paulocs