Incompatibitily with i18nmenu

sinasalek - September 3, 2009 - 11:18
Project:Acquia Marina
Version:6.x-1.9
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Since acquia does not uses default primary-links variable it should use i18nmenu's api localizing menu when it's enabled.
Replacing the following code in template.php fixes the problem :

<?php
    $vars
['primary_links_tree'] = menu_tree(variable_get('menu_primary_links_source', 'primary-links'));
?>

With :
<?php
 
if (module_exists('i18nmenu') and 1==0) {
   
$vars['primary_links_tree']=i18nmenu_translated_tree(variable_get('menu_primary_links_source', 'primary-links'));

  } else {
   
$vars['primary_links_tree'] = menu_tree(variable_get('menu_primary_links_source', 'primary-links'));
  }
?>

For default variable another issue submitted to i18nmenu module. #514108: Primary and secondary links' menus translation

#1

sinasalek - September 3, 2009 - 11:20

Please remove "and 1==0"

#2

sinasalek - September 18, 2009 - 08:47

here is the patch

AttachmentSize
i18nmenu_compatibility.patch 791 bytes

#3

dddave - September 20, 2009 - 11:58
Status:active» needs review

#4

sinasalek - September 21, 2009 - 05:58

Patch in action : http://sina.salek.ws
Change the language and checkout the menu

#5

sinasalek - December 20, 2009 - 06:20

#6

jwolf - December 22, 2009 - 07:17
Status:needs review» fixed

Patch applied to 2.x.
Thank you sinasalek for all your help!!!
http://drupal.org/cvs?commit=305228

#7

System Message - January 5, 2010 - 07:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.