mollom_validate_captcha includes logic to allow textual analysis validation to run in the case of a textual analysis form submission that includes a failing captcha in order to allow the textual analysis to make the determination of whether a CAPTCHA should be shown again. While there is some valid logic here, the user experience created is difficult for end users to comprehend.
Steps to Reproduce
- Set up textual analysis on a form
- Submit unsure content to Mollom via that form and note a CAPTCHA is presented
- Update the content to ham, incorrectly fill in the CAPTCHA, and submit the form
Expected Result
The user would expect to be presented with another CAPTCHA to solve because the value was incorrect.
Actual Result
The textual analysis response is used, which is ham, so the content is accepted.
This is probably relevant to the D8 and D6. Starting here for investigation purposes.
Comment | File | Size | Author |
---|---|---|---|
#8 | 2744237.captcha_validation_8.patch | 2.46 KB | eshta |
#3 | 2744237.captcha_validation_3.patch | 2.46 KB | eshta |
#2 | 2744237.captcha_validation_wip.patch | 1.14 KB | eshta |
Comments
Comment #2
eshta CreditAttribution: eshta at Acquia commentedThis is a WIP patch that still needs corresponding tests, etc.
Comment #3
eshta CreditAttribution: eshta at Acquia commentedAdding a test for the bots.
Comment #8
eshta CreditAttribution: eshta at Acquia commentedFixing the test.
Comment #10
eshta CreditAttribution: eshta at Acquia commentedComment #11
eshta CreditAttribution: eshta at Acquia commented