No webforms show up on the Molloms settings form. This is due to a small typo in the implementation of hook_mollom_form_list:

while ($node = db_fetch_object($node)) {

should be

while ($node = db_fetch_object($result)) {

CommentFileSizeAuthor
webform-mollom-typo.patch886 bytesdavyvdb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Reviewed & tested by the community

Looks good and fixes a small but important oopsie. :)

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Priority: Critical » Normal
Status: Reviewed & tested by the community » Fixed

Huh. I wonder how that happened. Thanks Davy Van Den Bremt! Committed.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Title: Small typo prevents Mollom integratio from working » Small typo prevents Mollom integration from working

Status: Fixed » Closed (fixed)

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