For whatever reason, my host account has been timing out looking for the trash icon in this module. I just grabbed the latest version and have the same behavior. The problem is that the bookmark delete icon is hard coded. This means that "alternative" locations, such as a modules/Adins/bookmarks will fail to display the icon. the sample code is below:

 @return string the delete icon to be emitted
**/
function theme_bookmarks_delete($url) {
  global $user;

  $query = 'url=' . urlencode($url). '&block=1';
  return l(theme('image', 'modules/bookmarks/trash.gif', t('delete')), "bookmarks/$user->uid/delete", array("title" => t("Delete this bookmark from your list.")), $query, NULL, FALSE, TRUE);
}
/**  @} End of addtogroup themeable **/

The "fix" I implemented was hardcoding the path as in

 @return string the delete icon to be emitted
**/
function theme_bookmarks_delete($url) {
  global $user;

  $query = 'url=' . urlencode($url). '&block=1';
  return l(theme('image', 'modules/Adins/bookmarks/trash.gif', t('delete')), "bookmarks/$user->uid/delete", array("title" => t("Delete this bookmark from your list.")), $query, NULL, FALSE, TRUE);
}
/**  @} End of addtogroup themeable **/

But the real fix should be to use some kind of system variable to identify where the file is supposed to be. A confirmation dialog would be in my wish list, too.

Comments

Anonymous’s picture

Status: Active » Closed (fixed)

This issue should no longer happen as we are using a new codebase.