I created a mollom implementation for project guestbook (see: http://drupal.org/node/717040#comment-3335266).
I have a question about the 'mail ids' in hook_mollom_form_info() in mollom.api.php (dev version). I presume the id is the parameter $key in drupal_mail() of the e-mail send by guestbook. But it doesn't insert a message like 'Report as inappropriate'.
I couldn't find the string 'mail ids' in the mollom module. Is it implemented? Or how do I have to use this parameter?

CommentFileSizeAuthor
#2 mollom-DRUPAL-6--1.api_.2.patch19.46 KBsun

Comments

dries’s picture

dries commented

Hello promes, did you look at mollom.api.php?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new mollom-DRUPAL-6--1.api_.2.patch19.46 KB

Yes, 'mail ids' is documented already, but only contained in the latest 6.x-1.x-dev. I've posted an updated patch over in #717040: Mollom support, leveraging cutting edge Mollom APIs. :)

However, something went really wrong in one of the last commits.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, mollom-DRUPAL-6--1.api_.2.patch, failed testing.

dries’s picture

dries commented

It looks like you are just deleting a bunch of documentation. Not sure why that can cause tests to fail.

Is that documentation no longer relevant for Drupal 6?

promes’s picture

promes
Primary language Dutch
commented
Status: Needs work » Active

Thanks for the reply. I will wait with a further implementation of guestbook-mollom module till the next official mollom version.

To Dries, comment #1: yes, as I stated in my original request (second line), I downloaded the dev version last week (version 16 august), only for the mollom.api.php. In some recent issue I did read about this being the up to date documentation. That version of the documentation helped me to accomplish the implementation.
For a production site I use versie 6.13, because it is a stable mollom version. Attention: the start section of this documentation is repeated three times.

I like mollom very much. I use it now in about 20 sites.

sun’s picture

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

#2: mollom-DRUPAL-6--1.api_.2.patch queued for re-testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

The @defgroup got duplicated 2 times somehow.

Status: Needs review » Needs work

The last submitted patch, mollom-DRUPAL-6--1.api_.2.patch, failed testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Version: 6.x-1.13 » 6.x-1.x-dev

Really, I have no idea what testing against tags is useful for.

sun’s picture

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

#2: mollom-DRUPAL-6--1.api_.2.patch queued for re-testing.

dries’s picture

dries commented
Version: 6.x-1.x-dev » 6.x-1.13
Status: Needs review » Fixed

Committed the patch that removes duplicate documentation. It can't fail so marking this as 'fixed'.

Status: Fixed » Closed (fixed)

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