.. as the title says: menus are not translated as "i18nmenu" is now "i18n_menu" hence module_exists('i18n_menu') returns false on line 291 in site_map.module

CommentFileSizeAuthor
#2 site_map_i18n_fixes_1199282.patch1.78 KBfrjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fjen’s picture

fjen CreditAttribution: fjen commented

This makes it work for 7.x-1.0-beta1 (around line 291):

if (module_exists('i18n_menu')) {
$tree = i18n_menu_localize_tree($tree, $language->language);
}

frjo’s picture

frjo CreditAttribution: frjo commented
FileSize
site_map_i18n_fixes_1199282.patch1.78 KB

Thanks for finding this bug, i18n has changed from Drupal 6 to 7 and Site map needs to be updated to fix this.

Please try out this patch and report back your results here.

I also attempted to fix the support for i18n_taxonomy.

ataimist’s picture

ataimist CreditAttribution: ataimist commented

Don't know about taxonomy (the site I tested this in doesn't use it), but otherwise this patch seems to work.

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Active » Fixed

Committed to 7-dev.

Status: Fixed » Closed (fixed)

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