Goal

  • When text analysis is unsure, retain posts as "unpublished" for manual moderation instead of showing a CAPTCHA.

Details

  • The default behavior for an "unsure" response is to show a CAPTCHA.
  • The Mollom module already supports to retain posts for manual moderation when text analysis identifies spam.
  • What is missing and desired is to retain posts for manual moderation when text analysis is unsure, instead of showing a CAPTCHA.

Related issues

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Status: Active » Needs review
FileSize
4.59 KB

If all goes well, this should actually work already.

sun’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
8.85 KB

Added tests.

sun’s picture

FileSize
17.36 KB

Screenie of new setting:

mollom-unsure.2.png

sun’s picture

Title: Allow to retain + moderate unsure posts instead of showing CAPTCHA » Allow to retain and moderate unsure posts instead of showing CAPTCHA
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x-2.x.

OldAccount’s picture

Does this setting only show up for comments? I want to be able to retain/moderate spam webform submissions but am not seeing the option (7.x-2.3).

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

sun’s picture

Version: 7.x-2.x-dev » 6.x-2.x-dev
Status: Closed (fixed) » Needs review
FileSize
9.22 KB

@lrobeson: In general, the "retain as unpublished" facility is only available for posts/entities that support a notion of a "unpublished" status. Webform submissions do not have such a status.

Attached patch backports this code to 6.x-2.x.

sun’s picture

Missing $ret in db_add_field() of the module update.

Status: Needs review » Needs work

The last submitted patch, 1817446-moderate-unsure.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
FileSize
9.22 KB

Status: Needs review » Needs work

The last submitted patch, 1817446-moderate-unsure.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
FileSize
9.25 KB

Sorry, the form constructor used a facility that only exists in D7+. This one should pass.

Status: Needs review » Needs work

The last submitted patch, 1817446-moderate-unsure.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
FileSize
9.26 KB

Heh. It would definitely help if I'd locally run the tests on D6 instead of D7 ;)

sun’s picture

Status: Needs review » Fixed

Committed and pushed to 6.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.

  • Commit 3df1ace on 7.x-2.x, 8.x-2.x, fbajs, actions by sun:
    - #1817446 by sun: Allow to retain and moderate unsure posts instead of...

  • Commit 3df1ace on 7.x-2.x, 8.x-2.x, fbajs, actions by sun:
    - #1817446 by sun: Allow to retain and moderate unsure posts instead of...