Closed (fixed)
Project:
Colorbox Node
Version:
7.x-2.6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 May 2012 at 19:22 UTC
Updated:
18 Apr 2013 at 17:50 UTC
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
Comment #1
iLLin commentedAs soon as the next Dev release comes out, grab that and let me know if you still have any issues.
Comment #2
arisaves commentedCool, man. Thanks. Nice to see somebody actively extending this great module.
Comment #3
iLLin commentedComment #4
arisaves commentediLLin, everything is working out pretty nicely w/ the new dev version! Thanks again.
Comment #5
iLLin commentedOk Great, marking as fixed.
Comment #7
bendev commentedThe 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);
Comment #8
rggoode commentedI'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).
Comment #9
iLLin commentedCheck your manage display. This patch fixes a notice, I have not had time to fix it yet. This isn't your problem.
Comment #10
rggoode commentedThank 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
Comment #11
iLLin commentedFixed in the next DEV release.