The variable $base_url should be $basepath.

Line 101 of bookmark_us.module is :

  $output .= '<script type="text/javascript"> displayLink("1", "'. $base_url .'", "'. $sitename .'", "'. $bmarkus_stitle .'");</script>';

It should be:

  $output .= '<script type="text/javascript"> displayLink("1", "'. $basepath .'", "'. $sitename .'", "'. $bmarkus_stitle .'");</script>';

-Corey

Comments

coreyp_1’s picture

coreyp_1 commented
Title: Wrong variable » global declaration missing for $base_url

Sorry, I should pay closer attention.

$base_url is correct, but it is never declared "global" inside the function, and therefore the javascript does not function correctly.

-Corey

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Active » Fixed

Thanks! Released 6.x-1.1 http://drupal.org/node/294717 !

Thanks again!

Anonymous’s picture

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

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