Posted by jtbayly on October 15, 2010 at 3:10pm
6 followers
| Project: | Email Field |
| Version: | 6.x-1.2 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
It would make the contact form really useful if it supported Mollom.
Comments
#1
My friend says I'm a jerk.
What I should have said was:
I really like this module, and I'm excited about using it. Thanks for all your work! Is there any chance of including support for Mollom any time soon?
Thanks,
-Joseph
#2
I would also be interested in seeing Mollom integrated with this module.
#3
The attached patch adds mollom support to 6.x-1.2. Please note that I had to move the validation function from the submit button to the root of the form in email_mail_page_form() due to this issue: http://drupal.org/node/372818
#4
#5
Thanks for the patch! Very essential. I tested the patch and functionality and it works as expected.
#6
Yup, works for me.
#7
I haven't tested the patch yet, but there are still some issues. It contains a 'mymodule_comment_mail' and some whitespaces.
Are there plans for D7?
#8
Sorry for the late reply (over the moon with email notifications on d.o). I can update the current D6 patch, D7 is not likely something I'll address soon, although the concepts are pretty simply, so porting over de patch to D7 shouldn't be that hard.
#9
Sorry for the daft question, but as the patch from #3 hasn't been updated, I was going to apply the changes manually, but what does the 'mymodule_comment_mail' need to be changed to?
Thanks.
#10
It should be safe to completely remove that line, it was a "copy-paste" error from the Mollom examples. In fact, I don't think making use of this option is something that can be done "wholesale". For my use case, the mail form will be sending emails to arbitrary users. I don't want them to control what is spam and what isn't for my sites Mollom account. It might be a nice feature addition, but in that case it should be configurable (probably per field-instance) whether or not to show the spam-report link in the email, which would increase the complexity of this patch quite a lot.
I've attached an altered version of the patch that does not contain the 'mail ids' field in the hook_mollom_form_info and which removes the extraneous whitepase, as mentioned by mh86 in #7.
#11