Closed (fixed)
Project:
Mollom
Version:
6.x-1.8
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
14 Jun 2009 at 07:43 UTC
Updated:
10 Sep 2009 at 21:50 UTC
Jump to comment: Most recent file
It seems that with the latest version of Mollom (6.x-1.8), if a I try to log in using the login block while viewing a comment page, I get the following message:
* Your submission has triggered the spam filter and will not be accepted.
* Sorry, unrecognized username or password. Have you forgotten your password?
If I navigate to a page other than a comment page, the login (from the login block) is successful. I just updated Mollom yesterday, but I don't know if this behavior was present before. It is 100% reproducible (for me, anyway).
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | 491186-test.patch | 2.45 KB | dave reid |
| #5 | 491186-submittability-check.patch | 857 bytes | damien tournoud |
| #4 | 491186-submittability-check.patch | 839 bytes | damien tournoud |
Comments
Comment #1
Bob4613 commentedI upgraded yesterday also and I had the same issue this morning when attempting to log in from a comment page . In addition, it appears that Mollom allowed about 20 comments to be posted that were all spam. I noticed I had comments before I logged on that all seemed like "SPAM". When I went to the comments page and tried to log in I received the same message you mentioned above.
When I went back to the home page I was able to log in and then went to admin / mollom and was denied. It said I did not have access to mollom. So I went to the user permissions and looked at the mollom settings. The "admin mollom" for administrators was "unchecked". I checked it and went to mollom to check if the settings for "comments" had changed. Everything was still checked with "Text analysis and CAPTCHA Backup"
I would be interested to hear what is going on with this update.
Comment #2
nemsis commentedI am getting something similar from the search block. Only seems to occur if I am in a node, not from the home page.
Any search entered while viewing a content node seems to be affected.
Message is the same. "Your submission has triggered the spam filter and will not be accepted."
I don't know if this is related but it seemed like it might be.
Comment #3
dries commentedInvestigating, but it would be helpful if someone could help debug this problem. If it is reproducible, you could alter _mollom_debug() to do some logging to your Apache log files or to watchdog() if you want. Hopefully that will provide more insight in why Mollom is triggered.
Comment #4
damien tournoud commentedMollom incorrectly detects the "submittability" of the form. We only check if $form['#post'] is not empty, without checking the form_id, as does drupal_process_form().
Here is a patch.
Comment #5
damien tournoud commentedAnd a less stupid one.
Comment #6
dries commentedI've been testing this one and it seems to address the problem. I'll commit this to the Drupal 6 development branch after some more testing. Thanks Damien.
Comment #7
dave reidI was just about to report I was experiencing this on my blog recently as well. With Damien's patch it seems to be working properly. I'll try and actually get a SimpleTest written for this so we don't regress this bug.
Comment #8
Bob4613 commentedDamien,
How do I apply this patch? I'm not really a coder, so I have not idea where to apply this patch. If you could explain what I need to do, it would be greatly appreciated.
Thanks,
Comment #9
dries commentedI committed to DRUPAL-6 HEAD but we still need tests.
Comment #10
dave reidWriting regression test and will post shortly. Assigning myself.
Comment #11
dave reidHere's a regression test. It was a PITA since DWTC->drupalPost() doesn't allow us to submit fields from multiple forms at once. This might be a little cleaner with the 'additional_post' option in #335035: drupalPost() incorrectly submits input for disabled elements but we'll still need to wait for it to be backported.
I confirm that the test has 2 fails as expected without the patch in #5 and passes successfully with the current fixed/HEAD code.
Comment #12
ha5bro commentedSubscribing.
Upgraded to the new version, spam filter is still triggered when searching content from node pages. Fine from the homepage and from views pages. I'm going to disable molllum for now.
Is there any way to tell Mollom to let the spam filter alone? Seems like that should be an option.
Comment #13
dave reid@robotoverload: Can you confirm you are using the current 6.x-1.9 version of the module? It's been working fine for the rest of us.
Comment #14
dries commentedThis has been fixed in the 6.x-1.9 version so marking this 'fixed' for now. Feel free to re-open if it is not fixed.
Comment #15
ha5bro commentedYes, it's v1.9 that I'm having problems with. I will try to reproduce the error on a local copy of the site and get a better idea of what's happening.
Strange that everyone else is fixo.