Support for multi language menu

forgot to add the classed to the output
fixed code

function theme_nice_menu_build($menu) {
  global $language;

  $output = '';
  /*http://drupal.org/node/301247#comment-1497198*/
  $index = 0;
  $count = 0;
  //count only if item is not hidden and is shown only for the current language
  foreach ($menu as $menu_count) {
    if ( ($menu_count['link']['hidden'] == 0) && ($menu_count['link']['localized_options']['langcode'] == $language->language) ) {
      $count++;
    }
  }
  $output = '';

  foreach ($menu as $menu_item) {
    $mlid = $menu_item['link']['mlid'];
    // Check to see if it is a visible menu item.
    if ($menu_item['link']['hidden'] == 0) {
      //check page language and render the menu only item if same as page language
      if ( $menu_item['link']['localized_options']['langcode'] == $language->language ){
        //prepare marking of li tag as odd, even, first or last
        $index++;
        $first_class = $index == 1 ? ' first ' : '';
        $oddeven_class = $index % 2 == 0 ? ' even' : ' odd';
        $last_class = $index == $count ? ' last ' : '';

        // Build class name based on menu path
        // e.g. to give each menu item individual style.
        // Strip funny symbols.
        $clean_path = str_replace(array('http://', '<', '>', '&', '=', '?', ':'), '', $menu_item['link']['href']);
        // Convert slashes to dashes.
        $clean_path = str_replace('/', '-', $clean_path);
        $path_class = 'menu-path-'. $clean_path;
        // If it has children build a nice little tree under it.
        if ((!empty($menu_item['link']['has_children'])) && (!empty($menu_item['below']))) {
          // Keep passing children into the function 'til we get them all.
          $children = theme('nice_menu_build', $menu_item['below']);
          // Set the class to parent only of children are displayed.
          $parent_class = $children ? 'menuparent ' : '';
          $output .= '<li id="menu-'. $mlid .'" class="'. $parent_class . $path_class .'">'. theme('menu_item_link', $menu_item['link']);
          // Build the child UL only if children are displayed for the user.
          if ($children) {
            $output .= '<ul>';
            $output .= $children;
            $output .= "</ul>\n";
          }
          $output .= "</li>\n";
        }
        else {
          $output .= '<li id="menu-'. $mlid .'" class="'. $path_class .$trail_class . $path_class . $first_class . $last_class . $oddeven_class.'">'. theme('menu_item_link', $menu_item['link']) .'</li>'."\n";
        }
      }
    }
  }
  return $output;
}

Hi Greg,

maybe it is me who is missing something here.

I will try to explain my point:
The problem here is that unlike the core's menu module, the nice_menus will render all menu items with no regards to the language of the page.
thus, if we have a bi-lingual site, which has 5 menu items for each language, instead of showing the 5 right items for the relevant language, it will show all ten.

True, you can use css to make the unneeded items disappear, however, you may lose the odd even first last functionality. In any case i do not think this is best practice since you should not render what is not being used.

since drupal is a multi lang system, and the nice menu doesnt support this core feature, shouldnt this issue be makred as a bug for version 1.3? even though its fixed in later version