Closed (fixed)
Project:
Nodequeue
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2014 at 22:31 UTC
Updated:
12 Jan 2015 at 19:54 UTC
Jump to comment: Most recent
nodequeue_view_nodes() still calls node_view() with D6 style arguments: where D7 node_view() expects a string for $view_mode, nodequeue_view_nodes() passes it a boolean (the boolean argument used to be called $teaser). Also it passes FALSE for $language, which is bogus.
Other modules may behave strangely if we illegally pass TRUE to node_view() instead of a display mode. (In my case it was nodehierarchy, which sets the breadcrumb to the node you're rendering in teaser mode. This is nodehierarchy's fault / not your problem; just FYI.)
--
This patch changes the expected argument to string (but correctly converts old-style TRUE/FALSE to 'teaser'/'full') and fixes the $language parameter.
It also
| Comment | File | Size | Author |
|---|---|---|---|
| nodequeue-view-nodes.patch | 3.25 KB | roderik |
Comments
Comment #2
fizk commentedCommitted. Thanks!