Download & Extend
Acquia Marina » Issues

Incompatibitily with i18nmenu

Posted by sinasalek on September 3, 2009 at 11:18am
Project:Acquia Marina
Version:6.x-1.9
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

Login or register to post comments

Comments

#1

Posted by sinasalek on September 3, 2009 at 11:20am

Please remove "and 1==0"

#2

Posted by sinasalek on September 18, 2009 at 8:47am

here is the patch

AttachmentSize
i18nmenu_compatibility.patch 791 bytes

#3

Posted by dddave on September 20, 2009 at 11:58am
Status:active» needs review

#4

Posted by sinasalek on September 21, 2009 at 5:58am

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

#5

Posted by sinasalek on December 20, 2009 at 6:20am

You can find the patched version here

#6

Posted by jwolf on December 22, 2009 at 7:17am
Status:needs review» fixed

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

#7

Posted by System Message on January 5, 2010 at 7:20am
Status:fixed» closed (fixed)

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