Links built as instructed are opening blank windows filled with errors. Running cron and flushing caches turns up the following error --

Strict warning: Only variables should be passed by reference in colorbox_node_output() (line 52 of [...]/sites/all/modules/colorbox_node/colorbox_node.module).
Strict warning: Only variables should be passed by reference in colorbox_node_output() (line 67 of [...]/sites/all/modules/colorbox_node/colorbox_node.module).

Comments

iLLin’s picture

As soon as the next Dev release comes out, grab that and let me know if you still have any issues.

arisaves’s picture

Cool, man. Thanks. Nice to see somebody actively extending this great module.

iLLin’s picture

Status: Active » Postponed (maintainer needs more info)
arisaves’s picture

iLLin, everything is working out pretty nicely w/ the new dev version! Thanks again.

iLLin’s picture

Status: Postponed (maintainer needs more info) » Fixed

Ok Great, marking as fixed.

Status: Fixed » Closed (fixed)

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

bendev’s picture

Version: 7.x-2.x-dev » 7.x-2.6
Category: support » bug
Status: Closed (fixed) » Needs work

The problem still exists in latest version 7.X.2-6.

Here is the change to correct it

in .module file @ line 72

$node_view = $tmpNode['#node'];
$html .= drupal_render(node_view($node_view, 'colorbox'););

should be changed to

$node_view = $tmpNode['#node'];
$temp=node_view($node_view, 'colorbox');
$html .= drupal_render($temp);

rggoode’s picture

I'm not receiving an error message, but when I try to open a link with Colorbox Node, the Colorbox opens, but it's either empty or, in some cases, displays only the node title.

This is on an install where all the Colorboxes were working as expected using:
Colorbox 7.x-1.3
Colorbox Node 7.x-2.3

But when I updated to:
Colorbox 7.x-2.3+3-dev
Colorbox Node 7.x-2.6

The Colorbox Nodes function stopped working (as described above).

I tried the patch, as described in #7 Posted by bendev, but got a white screen (I may have done it wrong? I'm new to using patches).

iLLin’s picture

Check your manage display. This patch fixes a notice, I have not had time to fix it yet. This isn't your problem.

rggoode’s picture

Thank you. And I've discovered what I was doing wrong. I didn't realize that the new model (Colorbox or Colorbox-Node?) adds a Custom Display option in the Content Type specifically for Colorbox display. Actually a very nice improvement. Thank you very much

iLLin’s picture

Status: Needs work » Fixed

Fixed in the next DEV release.

Status: Fixed » Closed (fixed)

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