Notice on line 335

Any other information about what you were doing? What kind of comment?

Looking through the Mollom specification it looks like the response doesn't have to contain that value, but it would probably be good to kind of clean up the logic in those structures. For instance:

Instead of
isset($GLOBALS['mollom_response']) && isset($GLOBALS['mollom_response']['session_id'])
we can just do:
isset($GLOBALS['mollom_response']['session_id'])

I also tried add the the 'merge default values' code in mollom_validate_analysis() but it led to one failure in the Fallback SimpleTests. Moving the code to mollom_set_data() worked perfectly.

Commited to CVS.