Hi,

When I use LJXP in Drupal 5.x, I get 3 error messages from includes/common.inc. They relate to the t() function and the l() function. I tracked it down (I think) to line 425 where you give a default for "Read more of this post". the line currently reads

$message['event'] .= l(t('Read the rest of this post', $dest, NULL, NULL, NULL, TRUE));

The brackets are incorrectly positioned. The line should instead be:

$message['event'] .= l(t('Read the rest of this post'), $dest, NULL, NULL, NULL, TRUE);

Alex

Comments

domesticat’s picture

domesticat commented

I can confirm this. I'd been consistently seeing these errors when posting, and I'd narrowed it down to the ljxp module. Full paths snipped out below since they're irrelevant:

    * warning: Invalid argument supplied for foreach() in common.inc on line 739.
    * warning: strtr(): The second argument is not an array. in common.inc on line 754.
    * warning: Missing argument 2 for l() in common.inc on line 1269.

Entries were being posted, but they were not including the 'read the rest of this post' link. Making the code change included above takes care of the errors, and restores the link.

wrunt’s picture

wrunt commented
Status: Active » Fixed

Thanks for tracking down the bug. I've fixed this in cvs.

agileware’s picture

agileware commented

This fix is included in the 5.x-1.2 release.

Anonymous’s picture

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

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