I tested it on Drupal 6.10. The email was still sent properly but it gave the following error:

* warning: implode() [function.implode]: Invalid arguments passed in .../sites/all/modules/forward/forward.module on line 633.
* warning: Wrong parameter count for nl2br() in .../sites/all/modules/forward/forward.module on line 633.

CommentFileSizeAuthor
#5 foward.0.patch1.2 KBsuit4

Comments

seanr’s picture

seanr
he/him
commented

I noticed that yesterday too but am at a loss for why. Any ideas?

seanr’s picture

seanr
he/him
commented
Status: Active » Fixed

OK, found it and fixed it. Should have been using explode not implode, and a parenthesis was in the wrong place. Fixed in head and will be rolling out updated packages momentarily.

Status: Fixed » Closed (fixed)

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

suit4’s picture

suit4 commented
Status: Closed (fixed) » Active

Reopening - error still exists and the 'implode' is still in place.

What about the update? Thanks in advance!

suit4’s picture

suit4 commented
Status: Active » Patch (to be ported)
StatusFileSize
new foward.0.patch1.2 KB

ok, found the problem.

here's a patch.

seanr’s picture

seanr
he/him
commented
Status: Patch (to be ported) » Fixed

It was in CVS but not in a release. I've fixed that now: http://drupal.org/node/461876

Status: Fixed » Closed (fixed)

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