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
Comment #1
Anonymous (not verified) commentedThis issue should no longer happen as we are using a new codebase.