Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
There is no integration with Mollom 1.11 anymore, with Mollom 1.10 the CAPTCHA was working fine!
Comment | File | Size | Author |
---|---|---|---|
#32 | guestbook-mollom-717040-32.patch | 1.77 KB | Eric_A |
#21 | guestbook.mollom.21.patch | 1.73 KB | sun |
#20 | guestbook.mollom.20.patch | 1.68 KB | sun |
#19 | guestbook_mollom.zip | 1.37 KB | PROMES |
Comments
Comment #1
sun#302941: Integrate with Guestbook was never officially supported by Mollom module. The changes that have been committed to Guestbook module did not really integrate the module with Mollom.
We can work on a proper integration though.
Comment #2
else CreditAttribution: else commentedThat should be great!!
Comment #3
Jochen Wendebaum CreditAttribution: Jochen Wendebaum commentedoh yes please! Mollom is great, and I don't like to use another spam module just because of my guestbook.
Subscribing.
Comment #4
Jochen Wendebaum CreditAttribution: Jochen Wendebaum commentedCan I help out, I am a module contributor...
Comment #5
midmood CreditAttribution: midmood commentedSubscribe
Comment #6
risp CreditAttribution: risp commentedI'm not sure how to submit a patch and for which version of the guestbook I should make a patch but the following code is working great for me:
I've added this code between the following functions:
Comment #7
sahuni CreditAttribution: sahuni commentedHello risp,
I have thousands of spams on guestbook.
Did the function you mentioned above is to be copied like that or there is a part of variable?
We don't have to add a call to this function?
I ask because I added this function in guestbook.module, but I can see still spams arriving in my page guestbook.
I have mollom-6.x-1.12 and guestbook-6.x-1.1
Comment #8
risp CreditAttribution: risp commentedSorry I forgot to mention that after you added the above code that you need to edit the Mollom settings and add the guestbook form. Mollom -> add form and then select the guestbook.
Comment #9
sunSorry, it's unfortunate, but the hook_mollom_form_info() API will most likely need to be completely changed in Mollom 6.x-1.13. We're trying to resolve some ugly unforeseen scaling issues.
Postponing until it's safe to implement this hook (i.e. when Mollom 1.13 has been released).
Comment #10
sahuni CreditAttribution: sahuni commentedStupid of me!
You are right, it works
thanks for all
Comment #11
sahuni CreditAttribution: sahuni commentedNow there is a new version of mollom : mollom-6.x-1.13
Is there now a better integration of guestbook module?
Do we have to upgrade mollom or it will not change anything about attacks every 5mn more or less?
Mollom stop attacks but logs are too numerous.
Comment #12
sunYes, we can now work on a patch to integrate with Mollom's latest stable release 1.13.
Comment #13
sahuni CreditAttribution: sahuni commentedSorry, I am not one who understand easily.
What do you mean? A new version of guestbook? a patch of guestbook but if yes where to get it?
Or simply, do I have to try and write myself my hook_mollom_form_list in addition to hook_mollom_form_info?
Comment #14
PROMESSince the update frequency are different for guestbook and mollom I think this should be an add-on to the guestbook and/or mollom module. There can be different module owners who can independent of each other update the module.
Comment #15
JordanMagnuson CreditAttribution: JordanMagnuson commentedAny news on this? Does the solution in #6 work for current versions of Guestbook and Mollom modules?
Comment #16
czoper CreditAttribution: czoper commentedI've got Guestbook 6.x-1.1 and Mollom 6.x-1.13 but solution in #6 doesn't work for me :-/
The only and fastest option I see is to install Captcha "standalone" module:
http://drupal.org/project/captcha
if heaving any problems adding captcha to Guestbook, pls refer to this instruction:
http://drupal.org/node/272050#comment-1409088
good luck ;)
Comment #17
sahuni CreditAttribution: sahuni commentedLook at mollom.api.php in mollom library module. Code is a little different in 6.x-1.13 as in 6.x-1.11.
Comment #18
sunMore details currently in the workings over in #775444: API implementor documentation
I'd be interested whether someone is able to write a proper patch for Guestbook based on that information.
Comment #19
PROMESI created a version 1.0-dev of guestbook_mollom module.
I did test it for site-guestbooks only, but since the code for user-guestbooks is the same I expect this to work as well.
I am new to the guestbook module, I don't know who can post a comment on an entry. I did implement a mollom-check but didn't test it.
I couldn't find in the mollom documentation, nor in the mollom module how to get a 'Report as inappropriate' line in the e-mail, send by guestbook. I will ask this in the mollom queue.
I present the module this early, so everyone can benefit.
Please help me to test this module.
Comment #20
sunAttached patch is against 6.x-2.x-dev.
Comment #21
sunPlus 'mail ids'.
Comment #22
PROMESHello Sun,
It turned out we both were searching for the same information to implement mollom. Your patch for 6.x-2.x-dev confirmed my implementation seems right.
I forgot to mention in comment #19 that my guestbook_molom module is for guestbook 6.x-1.1 and mollom 6.x-1.13 and the current 6.x-1.x-dev version. No code change is needed in the guestbook module.
Comment #23
sndo CreditAttribution: sndo commentedHi
I've installed 6.x-2.x-dev incl. guestbook.mollom.21.patch.
It works. Thanks.
Only the input formula is always above or under the entries while setting separate side is used.
==> http://drupal.org/node/301779
An after deleting an entry, the redirection path is wrong.
http://fw-kleinluetzel.ch/fwkleinl//fwkleinl/guestbook
instead
http://fw-kleinluetzel.ch/fwkleinl/guestbook
==> http://fw-kleinluetzel.ch/fwkleinl/guestbook/delete/6?destination=/fwkle...
==> http://fw-kleinluetzel.ch/fwkleinl/guestbook/delete/6?destination=guestbook
this works
==> http://drupal.org/node/411486
Comment #24
lostcarpark CreditAttribution: lostcarpark commentedThe guestbook_mollom module is working perfectly for me. I like this approach since it won't be broken by the next official guestbook relaease. Thanks!
James
Comment #25
chris.chambers CreditAttribution: chris.chambers commented"Sun" suggested that I move this mollom issue over to guestbook so here its is. I am using PHP4
When trying to apply the latest mollom security update (15th September) this crashes the whole web site with the message.
Parse error: syntax error, unexpected T_ARRAY, expecting ')' in /freeola/users/x/x/xxxxxx/htdocs/elliots_messages/modules/mollom/mollom.module on line 1842
The only way to recover the site is to revert to version 6.x-1.13.
This happens on a number of web sites I have running Drupal in exactly the same way. However that are all on the same hosting service.
Chris
Comment #26
joelstein CreditAttribution: joelstein commentedThe patch in #21 works great for me.
Comment #27
stella CreditAttribution: stella commentedYep patch in #21 works for me! :)
Comment #28
sunThanks for reporting, reviewing, and testing! Committed to HEAD (2.x).
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
Comment #30
chrisi1002 CreditAttribution: chrisi1002 commenteddelete
Comment #31
JamesMarie CreditAttribution: JamesMarie commentedOK - I am guessing I'm doing something wrong... I have tried to use the patch in #21 several times, and every time my site turns into a white screen until I replace the code with the original again. All I am doing is taking the code from #21 and using it to replace the code in guestbook.module - is that the correct way? That is what I have done before with patches, but I thought I might be missing something. :)
Any help is greatly appreciated! Thanks!
Comment #32
Eric_A CreditAttribution: Eric_A commentedThis code lived for a short time in CVS HEAD, but we've lost this in 6.x-2.x after the Great Git Migration, unintentionally.
Here's a re-roll of the actual committed code, including the changes to CHANGELOG.txt.
Comment #33
sunGood catch!