Doesn't works in multilingual sites

I would like to propose this fix. It uses the url() so no need to check if the pathauto module exists also, because url() does this for you. And ofcourse this function does link correct to the language prefix for the selected language.

I will commit if anyone can confirm it works!

Tested on 7.x-2.5 (==HEAD below)

--- HEAD
+++ Modified In Working Tree
@@ -307,17 +307,12 @@
     $path_module = $node->path;
   }
   else {
-    $path_module = "/node/" . $node->nid;
+    $path_module = "node/" . $node->nid;
   }
 
-  // Pathauto integration !
-  if (module_exists('pathauto')) {
-    $path_module = '/' . drupal_lookup_path('alias',"node/" . $node->nid);
-  }
-
   global $base_url;
   // Get the full path to insert into the Share Buttons.
-  $mPath = $base_url . $path_module;
+  $mPath = url($path_module, array('absolute'=>TRUE));
   $mTitle = $node->title;
   
   // Only display the ShareThis buttons if this node fits all the requirements

little mistake...

Hello,
I just tested it and its working.
Is there any concern in committing it?
Thanks, Yael

Not the same problem but in same category mutli-lang $node->path becomes array, so we have to check if path is array.

Notice : Array to string conversion in sharethis_node_view() (ligne 320 in .../drupal/sites/all/modules/custom/sharethis/sharethis.module).

I fixed issue by Modifiying this.

Line 307 :

- $path_module = $node->path;
+ $path_module = (is_array($node->path))?$node->path["source"]:$node->path;

Note that module pathauto is not active.

This issue probably has been fixed now on 7.x-2.x-dev branch.

Marking this issue as fixed. Thanks a lot for all your inputs. We really appreciate your efforts towards this issue. Feel free to re-open if issue persists.

Thanks