I was testing out beta1 today and found that 'verify using audio' was 404ing trying to fetch the JSON response. The fix was to remove the last argument from the AJAX menu callback.

CommentFileSizeAuthor
#2 mollom.captcha-switch.2.patch2.8 KBsun
#1 mollom-captcha-audio-1404456-1.patch527 bytesdags

Comments

dags’s picture

dags commented
StatusFileSize
new mollom-captcha-audio-1404456-1.patch527 bytes

Here's the patch.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: 'verify using audio' link not working. » Verify using audio link not working on a CAPTCHA-only protected form.
StatusFileSize
new mollom.captcha-switch.2.patch2.8 KB

Excellent, thanks!

I've added a test to prevent this bug from appearing in the future.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to 7.x-2.x and 6.x-2.x.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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

  • Commit 775c35c on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1404456 by davidjdagino, sun: Fixed Verify using audio link not...

  • Commit 775c35c on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1404456 by davidjdagino, sun: Fixed Verify using audio link not...