Closed (fixed)
Project:
Site map
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jul 2010 at 11:37 UTC
Updated:
17 Dec 2020 at 12:56 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
isaakordonez commentedSuscribe I have the same problem...
Comment #2
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 commentedAfter some tests, translation for all menus titles do not work.
Comment #4
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 commentedHi,
I made the exact same patch than #4 (and I attach it). I confirm it works for me too.
Regards,
Comment #6
cpo commented#4 work fine for me
Comment #7
simon georges commentedChanging the status as several people confirmed it works.
Comment #8
isaakordonez commented#4 Working for me Thanks!!! :)
Comment #9
JamesBecht commented#4 works fine ! Muchas gracias !
Comment #10
doomed commentedWorks nicely after you add t() .
On version 6.x-2.2 this is on line 277.
Comment #11
diego.banchero commentedYes it works. Here the patch for 6.x-2.2
Comment #12
tiziano.sartori commentedSame problem on 7.x-1.0 and 7.x-1.x-dev, solved with t() function
Comment #13
mariomc commentedSolved with t() function here as well!
Comment #14
darrell_ulm commentedLooks possible. Seems like many people tested this.
Comment #15
darrell_ulm commentedOK, just looked above and it appears to be 'patch to be ported'
Comment #16
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 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 commentedIt looks like this functionality is already in 7.x-dev so moved this issue to version 6.x
Comment #19
justingeeslin commentedHi folks,
The patch here works for me, too.
Is this still not in the 6.x?
Comment #20
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