I`ve noticed this bug long time ago and even told about it to Ber and others.
Functionality of bookmarks.module with weblink.module is great, BUT we
have a little bug wich wasn`t corrected after 4.4.x versions.

Here it is on string # 64:

// Node links for a weblink
  if ($type == "node" && $node->type == "weblink") {
    
    if (module_exist('bookmarks')) {
      $links[] = l(t('add to bookmarks'), "user/bookmarks/add/weblink/$node->nid", array('title' => t('Adds the destination to your bookmarks.')));
    }

This must be corrected due to changes in users system, cause with this bug our users can`t add a weblink to their bookmarks cause of wrong url. Nowadays we must call user id which is adding to url of a link-bookmark.

Corrected strings:

 // Node links for a weblink
    global $user;
  if ($type == "node" && $node->type == "weblink") {
    
    if (module_exist('bookmarks')&&  user_access('access bookmarks')) {
      $links[] = l(t('add to bookmarks'), "bookmarks/$user->uid/add/weblink/$node->nid", array('title' => t('Adds the destination to your bookmarks.')));
    }

Thats all:)

Comments

Bèr Kessels’s picture

Fixed in HEAD.

Anonymous’s picture