I think, it is easier to handle patches, if there is a own issue for D7. Therefore i create a new issue for mollom support.
For D6 see issue #942640: Mollom Support

CommentFileSizeAuthor
#1 D7-add_mollom_support-1354454-1.patch1.57 KBmoloc

Comments

moloc’s picture

moloc commented
Status: Active » Needs review
StatusFileSize
new D7-add_mollom_support-1354454-1.patch1.57 KB

Here is the patch.

Note: I moved the submit-form into the actions array. So they get a default #weight of 100 by drupal and mollom is able to place a message between the form-elements and the submit-button.

prossel’s picture

prossel commented

I applied (manually) the changes proposed by Moloc in #1 and found no problem. I could then add the email contact form to the list of Mollom controlled forms.

Thank you Moloc for the patch.

The patch does not seem to be included in current dev version.

I propose the patch to be properly reviewed and included in current dev version, ready for next release.

Anonymous’s picture

Anonymous (not verified) commented

I also tested the patch (applied cleanly) and found that it worked, allowing me to use Mollom to protect the email contact form.

However, I think we need to test whether it protects all email contact forms, or allows you to protect forms on an email field-by-field basis (I would imagine it'd be better to allow you to select which email contact forms you want protected...).

Poieo’s picture

Poieo commented

@mh86 - I assume there is a reason this hasn't been added. Can you provide some insight as to why?

It's kinda ironic that the email form is primarily used to prevent publicly displaying an email address for the purposes of avoiding spam. But, we can't protect the form itself from spam.

I'm sure there is a good reason why...any input would be helpful!

danon1981’s picture

danon1981 commented
Status: Needs review » Reviewed & tested by the community

Applied patch #1 to 7.x-1.2 and have mollom check enabled now.

danon1981’s picture

danon1981 commented

The mollom support for the email field(form) is great, but I ran into the issue of the 'report as inappropriate' message that gets added to the email message. When the receiver of the e-mail message replies to the message this notification and the report link is sent on to someone not ment to file a report.

mh86’s picture

mh86 commented
Issue summary: View changes

Patch from #1 still looks good. I haven't used Mollom so far but I agree that it makes sense to add support.
I'm interested in what danon1981 mentioned. Is there a way to remove this link in the sent mail, and if so, should we remove it or does it even make sense to have a 'report as inappropriate' link there?

mh86’s picture

mh86 commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)
dandaman’s picture

dandaman commented

Tried putting this patch on a website. It does include a "Report as inappropriate" link so that whomever receives the e-mail can tell them whether it was spam or not. The client whose website it was, since this is only used to send to staff, just sent out an e-mail to all the staff detailing how it worked. So this patch works for me. Looks like if you removed the following line it should not include "Report" link in the e-mail:

'mail ids' => array('email_contact'),
nsjski’s picture

nsjski commented

The patch and #9 work nicely for me. The patch was posted 3 years ago now... could it be moved into the main module code please?

eshta’s picture

eshta commented

Hi there - I'm one of the Mollom maintainers and I happened to see this recently due to a request in our own issue queue regarding this module. I'm happy to help if there are any lingering questions. @danaman is correct that the "mail ids" are what you want to adjust to control the email behavior.

The following is from the Mollom API docs:

mail ids: (optional) An array of mail IDs that will be sent as a result
* of this form being submitted. When these mails are sent, a 'report to
* Mollom' link will be included at the bottom of the mail body. Be sure to
* include only user-submitted mails and not any mails sent by Drupal since
* they should never be reported as spam.

http://cgit.drupalcode.org/mollom/tree/mollom.api.php#n329

Poieo’s picture

Poieo commented
Status: Postponed (maintainer needs more info) » Needs review

Regarding the report as inappropriate link, the same is true for the Contact module. If someone replies to the email they receive from the contact form, the report as inappropriate link is in that email as well.

For consistency, it makes sense to leave it there so spam can get reported and let the reply sender delete it before replying if necessary.

Would like to see this get committed.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Mollom is no longer supported.