i18n support

Thanks. First comment - you're changing two things here, and that's generally not a good practice for submitting patches. You're doing the i18n thing (an obvious bug), and you're deciding that there shouldn't be a popup window. The second part needs to be taken out of the patch so that we just focus on fixing the bug.

So my next complaint is that we can probably cut down on the extra path* variables.

 function theme_helpinject_topic($nid) {
   if (user_access('view advanced help popup')) {
+    global $language;
     $node = node_load($nid);

// get rid of this... see below
+    $path = 'node/' . $node->nid;
+    // Check if current language is the help language, if not switch the node
+    if ($node->language != $language->language) {

// don't need this line
+      $path2 = array();

// here's how we get rid of $path
+      $path2 = translation_path_get_translations( 'node/' . $node->nid);
+      $path3 = 'helpinject/' . $path2[$language->language];
+    }
+    else {
+    $path3 = 'helpinject/node/' . $node->nid;
+    }
     drupal_add_css(drupal_get_path('module', 'advanced_help') . '/help-icon.css');
-    return l('<span>' . t('Help') . '</span>', "helpinject/node/$nid", array(
+    return l('<span>' . t('Help') . '</span>', $path3, array(

// maybe try something like
if ($translations =  translation_path_get_translations( 'node/' . $nid)) {
 $path =  'helpinject/' . $translations[$language->language];
}
else {
 $path =  'helpinject/node/' . $nid;
}

@asak - do you want to pursure this further? Your help is appreciated.