Setting the List display configuration option to Node body produces the word "Array" on the listing.
Module line 376 is at fault. Current code:

$cell = array('data' => array(l(.....

Should not have array in front of the link. Corrected code:

$cell = array('data' => l(....
CommentFileSizeAuthor
#2 classified_body_array-1397422-2.patch2.57 KBfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

Confirmed. Thanks for the report: can you provide a patch including a simpletest that fails on the current code and succeeds after your fix ?

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Priority: Major » Normal
Status: Active » Needs review
StatusFileSize
new classified_body_array-1397422-2.patch2.57 KB

Nevermind, here is a test that fails on RC2 and passes with the included patch.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Version: 6.x-3.0-rc2 » 6.x-3.x-dev
Status: Needs review » Fixed

Test OK.

Checked on D7: bug has no equivalent, because body is built differently (see _classified_list_nodes() for implementation).

Committed to D6, thanks.

Status: Fixed » Closed (fixed)

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

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Closed (fixed) » Needs work

Still a problem when node has expired: "Array' reappears.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs work » Fixed

Fixed. Note that expired ads that have not yet been unpublished are still listed, but with the node-unpublished class until they get unpublished by the next cron run after they expire.

Status: Fixed » Closed (fixed)

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