Hi - this is really a great module!!

Only small problem I have found is that a full node is shown even when teaser is set to true eg

print dashboard_get_node(1, TRUE, FALSE, TRUE);

will print the whole node with nid = 1 eventhough TRUE is stated for teaser

Best wishes,
Lennart Kiil

Comments

lennart’s picture

lennart CreditAttribution: lennart commented

Also I am getting this error:
Invalid argument supplied for foreach() i /home/www/modules/dashboard/dashboard.module on line 172

But this seems to be related to the nodelist function

lennart’s picture

lennart CreditAttribution: lennart commented

the problem is more general it seems --

TRUE and FALSE statements get ignored, regardless if they are concerned with teaser, page or links.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Assigned: Unassigned » merlinofchaos

Looking into this. Fix is probably simple.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Active » Fixed

Yea, stupid simple fix. Took me freakin' hours to spot, too, because it just wasn't really visible. I knew what it was supposed to be doing, and that's all I could see.

Modify line 167 from:

  $node = node_view(node_load(array('nid'=>$nid), $teaser, $page, $links));

to

  $node = node_view(node_load(array('nid'=>$nid)), $teaser, $page, $links);

(Fix committed to HEAD and tagged for 4.6 -- will be available on next release run)

lennart’s picture

lennart CreditAttribution: lennart commented
Status: Fixed » Closed (fixed)

Thanks merlinofchaos - it works like a charm now! closing the issue