Sice I implemented v6x-1.14 I get this error on spam send by the contact module and a the guestbook module with the guestbook_mollom module:
implode() [function.implode]: Bad arguments. in /var/www/html/sites/all/modules/mollom/mollom.module on line 1439.
This is the only line in the watchdog.

CommentFileSizeAuthor
#3 mollom-DRUPAL-6--1.php4_.3.patch660 bytessun

Comments

chris.chambers’s picture

Title: implode() error on line 1439 » Parse error on line 1842

When trying to apply the latest mollom security update (15th September) this crashes the whole web site with the message.
Parse error: syntax error, unexpected T_ARRAY, expecting ')' in /freeola/users/x/x/xxxxxx/htdocs/elliots_messages/modules/mollom/mollom.module on line 1842

The only way to recover the site is to revert to version 6.x-1.13.
This happens on a number of web sites I have running Drupal in exactly the same way.

sun’s picture

Looks like we have to different issues here:

@promes: There is no official suport for Mollom in Guestbook module yet, so I'm not sure which code you are using. To fix your issue, we need to fix #717040: Mollom support (Guestbook), so let's move over there, please.

@chris: I guess you are on PHP4, right?

sun’s picture

Title: Parse error on line 1842 » PHP 4 incompatibility
Status: Active » Needs review
StatusFileSize
new660 bytes
dave reid’s picture

Version: 6.x-1.14 » 6.x-1.x-dev
Status: Needs review » Reviewed & tested by the community
sun’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for reporting! Committed to D6.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

promes’s picture

Title: PHP 4 incompatibility » implode() error on line 1439
Version: 6.x-1.x-dev » 6.x-1.14
Status: Fixed » Active

@sun about #2
I discovered it first in the contact page. Later I found the same problem in a guestbook entry. I know the guestbook_mollom module is not official (see #717040: Mollom support #19) but I reported it more as additional information.
I will install the latest development snapshot later today.

Thanks fot the fix. I have some very nasty spammers on some sites: one site with over 25 spam attacs a day.

promes’s picture

I installed the latest dev version this afternoon. I still have the same problem in the contact page. Maybe you fixed the other problem only?

sun’s picture

@promes: Did you run update.php? That implode() error should not appear after running it.

promes’s picture

I realized last night I forgot to run update.php and did run it on all sites this morning. Later on I found your post #8.
Thanks for reminding me.

sun’s picture

Status: Active » Fixed

So I guess this is resolved now.

Status: Fixed » Closed (fixed)

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