Allow data in mollom_form_get_values() to be altered

Patch is looking good. Going to wait what eshta is saying about it before RTBC'ing it.

Strange... the only commit made for the CMP removal was on the 8.x branch and the test failure is on the 7.x-2.x branch. I'll look into that.

In the meantime, I see that the hook is added to mollom_form_get_values(). This is called before submitting textual analysis, captcha, after form submission, and upon sending feedback to Mollom. It seems to me that you are only really looking to alter data in the textual analysis case. In the other cases, I can't see a reason that we'd want to allow manipulation of the contents sent. Also - in the textual analysis case the module continues to manipulate the data after this hook is called and before sending to Mollom (which may have been your intent?). I'm wondering if it might be better to place it right before the call to checkContent() within mollom_validate_analysis() here: http://cgit.drupalcode.org/mollom/tree/mollom.module#n2185

We should actually call this something different since there is already a mollom_data_insert, mollom_data_update, and mollom_data_delete hooks that refer to the data stored from mollom responses. mollom_content_alter works for me since it is the content analyzed by checkContent call. If this names works, though, it's RTBC to me.

minor semantics in api file

Back-porting....