I tried to use YUI menu in combination with the localization module but it is not working right. When I use the language switching with the default Drupal menu or lucid menu for example, the links of the menu change from the default language node to the translated node. This is not working with YUI menu. Furthermore the menu labels are not switched on switching the language, they remain on the default language. Please give me some hints how to solve the problem or provide a patch. I will work on it myself. If I got it working I will post it here.

Comments

maxx_’s picture

maxx_ commented
Status: Active » Needs work

A little modification of the function compose_markap_body in yuimenu.module does the trick. Please implement it in the right coding style to the function and add it in the new version, thank you.


function compose_markap_body ($pid) {
  $menu = menu_get_menu();
  if (isset($menu['visible'][$pid]) && $menu['visible'][$pid]['children']) {
    foreach ($menu['visible'][$pid]['children'] as $mid) {
	
	//modification here:
	$my_menu_temp=menu_get_item($mid);
    $my_menu_link=$my_menu_temp['path'];
    $my_menu_title=$my_menu_temp['title'];
	
      if (count($menu['visible'][$mid]['children']) > 0) {
        if (is_menu_child_of_root($mid)) {
          //$output .= " <li class=\"yuimenubaritem\">".l($menu['visible'][$mid]['title'],$menu['visible'][$mid]['path'],array('class'=>'yuimenubaritemlabel'));
		  //modification here:
		  $output .= " <li class=\"yuimenubaritem\">".l($my_menu_title,$my_menu_link,array('class'=>'yuimenubaritemlabel'));
        }
        else {
		  //$output .= " <li class=\"yuimenuitem\">".l($menu['visible'][$mid]['title'],$menu['visible'][$mid]['path'],array('class'=>'yuimenuitemlabel'));
          //modification here:
		  $output .= " <li class=\"yuimenuitem\">".l($my_menu_title,$my_menu_link,array('class'=>'yuimenuitemlabel'));
        }
        /*$output .= '<div id="'. rep_char($menu['visible'][$mid]['title']).'" class="yuimenu">
                     <div class="bd">
				     <ul class="first-of-type">';    */
        //modification here:
		$output .= '<div id="'. rep_char($my_menu_title).'" class="yuimenu">
                     <div class="bd">
                     <ul class="first-of-type">';
        $output .= compose_markap_body($mid);
        $output .= '</ul>
                </div>
            </div></li>'; 
      }
      else {
        if (is_menu_child_of_root($mid)) {
		  //$output .= " <li class=\"yuimenubaritem\">".l($menu['visible'][$mid]['title'],$menu['visible'][$mid]['path'],array('class'=>'yuimenubaritemlabel'))."</li>";
          //modification here:
		  $output .= " <li class=\"yuimenubaritem\">".l($my_menu_title,$my_menu_link,array('class'=>'yuimenubaritemlabel'))."</li>";
        }
        else {
		  //$output .= " <li class=\"yuimenuitem\">".l($menu['visible'][$mid]['title'],$menu['visible'][$mid]['path'],array('class'=>'yuimenuitemlabel'))."</li>";
          //modification here:
		  $output .= " <li class=\"yuimenuitem\">".l($my_menu_title,$my_menu_link,array('class'=>'yuimenuitemlabel'))."</li>";

        }
      }
    }
  }
  return ($output);
}

addition1: this code is only working with the option submenues built from markup selected, maybe in the future all three options should support it.

bakyildiz’s picture

bakyildiz commented
Version: 5.x-2.2 » 5.x-2.3
Status: Needs work » Fixed

your patch is applied. Thank you.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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