Hi,

CivicActions is reviewing and upgrading modules for use on client sites. Part of that process is an internationalization code review. There was just one problem identified in mollom and that was a missing t() call on the string passed to drupal_set_message(). The attached patch fixes this, as well as a handful of styling (whitespace) issues.

Cheers,
Stella

CommentFileSizeAuthor
mollom_i18n.patch2.16 KBstella
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Reviewed & tested by the community

Patch looks good and applies cleanly. RTBC.

Freso’s picture

Freso CreditAttribution: Freso commented

Not tested, but is a simple patch and looks good to me. So RTBC++. :)

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to DRUPAL-6. Thanks!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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