Audio CAPTCHA not accepting correct response

Please test with the lastest module version and re-confirm this still happens. I haven't been able to reproduce this locally.

I'm having the same problem on 6.x-1.11, whatever audio plays, it's the image answer that works. Tested on the drupal's user registration form.

Sounds like a confirmation then.

Actually, I'm not sure how this was able to work in the past. Probably some magic involved...

Attached patch - untested - should hopefully do the trick.

Alright - tested that patch on a production site, fixed a conflict caused by my custom theme, and can confirm that this patch is working.

Committed to DRUPAL-6--1. Thanks a lot, sun.

Ported to D7. Required a couple of adjustments, because we no longer use {cache_mollom} but native form caching there, so we additionally need the $form_build_id to update the Mollom session id in the cached form state.

At some point in the future, this entire code should probably be updated to use the new AJAX framework instead (i.e. invoke a real form submission, limiting validation errors, etc.), but that's way too much for now.

Note that this patch "should work", but I don't have a production site running on D7 with Mollom yet ;)

I'd therefore recommend to commit this patch and defer in-production testing to when D7 can actually be used for production sites.

I don't understand this part. Looks like a hack?

+++ mollom.pages.inc	9 Mar 2010 16:58:26 -0000
@@ -47,14 +61,30 @@ function mollom_captcha_js($type, $sessi
+  // Update cached session id for the form.
+  if (!empty($response['session_id'])) {
+    if ($cache = cache_get('form_state_' . $form_build_id, 'cache_form')) {
+      $form_state = $cache->data;
+      $form_state['mollom']['session_id'] = $response['session_id'];
+      cache_set('form_state_' . $form_build_id, $form_state, 'cache_form', REQUEST_TIME + 21600);
+      // After successfully updating the cache, replace the original session id.
+      $mollom_session_id = $response['session_id'];
+    }
+  }

This makes sense but it would be great to have a better code comment because one might expect the form API to handle that more gracefully -- instead of having to fiddle with the form cache ourselves.

+++ mollom.js	9 Mar 2010 16:51:57 -0000
@@ -20,47 +20,58 @@ Drupal.behaviors.mollomPrivacy = {
+    // @todo Pass local behavior settings.

I don't understand this todo. Maybe you could expand the code comment a bit?

+++ mollom.js	9 Mar 2010 16:51:57 -0000
@@ -20,47 +20,58 @@ Drupal.behaviors.mollomPrivacy = {
+      if (!(data && data.content)) {
+        return;
+      }

I don't understand this part. Looks like a hack.

Added plenty of docs. :)

re 3): righto, that was only remotely possible in D6, removed that condition.

Those test fails are caused by something else, very odd.

Committed to CVS HEAD. Thanks.