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

  1. Set up textual analysis on a form
  2. Submit unsure content to Mollom via that form and note a CAPTCHA is presented
  3. 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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eshta created an issue. See original summary.

eshta’s picture

This is a WIP patch that still needs corresponding tests, etc.

Status: Needs review » Needs work

The last submitted patch, 3: 2744237.captcha_validation_3.patch, failed testing.

The last submitted patch, 3: 2744237.captcha_validation_3.patch, failed testing.

The last submitted patch, 3: 2744237.captcha_validation_3.patch, failed testing.

The last submitted patch, 3: 2744237.captcha_validation_3.patch, failed testing.

eshta’s picture

  • eshta committed b555528 on 7.x-2.x
    Issue #2744237 by eshta, Nick_vh: Always show another CAPTCHA upon...
eshta’s picture

eshta’s picture

Status: Needs review » Patch (to be ported)