If mollom is enabled as a module but there's not yet a key present it should not display "Delete post" in node links.

Currently there's no way to remove the link than disabling the module.

CommentFileSizeAuthor
#3 453252-mollom-links.patch3.86 KBdave reid

Comments

dries’s picture

Why would you enable the module but not configure a key?

jeffschuler’s picture

It would be nice to be able to choose whether--and on which content types and to which roles (or on a by-user basis)--the "Delete post" link appears.

dave reid’s picture

Title: Remove "Delete post" from Links » Improvements to mollom_link()
Version: 6.x-1.7 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new3.86 KB

Attached patch make some improvements to the mollom_link() implementation:
1. Only displays the links if Mollom is configured (public and private keys are provided)
2. Change the link titles for both comments and nodes to 'Report to Mollom' so users are confused by possible duplicate 'Delete' links on nodes (see Admin links module).
3. Change the menu title for reporting e-mails from 'Report and delete' to just 'Report' since we can't 'delete' e-mails.
4. Prefixes the mollom link key strings with mollom_ to prevent name-space collision.

dave reid’s picture

Assigned: Unassigned » dave reid
dave reid’s picture

Status: Needs review » Fixed

Committed to CVS. Please file a new feature request about choosing which node types to allow the 'Report to Mollom' link on.

miro_dietiker’s picture

Thanks dave! Great

Status: Fixed » Closed (fixed)

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