Does Author Contact support Mollom?

CommentFileSizeAuthor
#3 authorcontact-mollom-553424-3.patch1.98 KBmirzu

Comments

milos.kroulik’s picture

subscribe

JmsCrk’s picture

I haven't done anything special to make it compatible with Mollom. Have you tried using it? I haven't set Mollom up so not sure how it works.

Author Contact does work by default with Captcha, so Mollom may work the same way.

mirzu’s picture

Component: Miscellaneous » Code
Status: Active » Needs review
StatusFileSize
new1.98 KB

Submitting a patch that adds the necessary code to make author contact work with Mollom. There are two functions that are required by Mollom. Additionally there is a a new implementation of hook_mail_alter that is only triggered when Mollom is being used that adds a "report as inappropriate" link to the email being sent.

Everything was pulled from the Mollom module's implementation of the site wide contact form.

Tested and seams to work fine with both text analysis and Captacha.

pieterdc’s picture

I'd like to test this, if only I had a little more time to spend :(

pieterdc’s picture

I made some time to test this.

  1. I wouldn't define a bypass access right as the Mollom module already defines a 'bypass mollom protection' permission which you can assign to the user roles you like.
    But, in a way, it makes sense as you code it. If someone may administer AuthorContact he/she won't probably abuse the AuthorContact forms...
  2. Rename itrader_mail_alter() to authorcontact_mail_alter()
    There should be an easier way to do this, follow #791358: Provide 'mail ids' in hook_mollom_form_info()

It seems to work. I'm going to deploy it on a live site for further testing.