Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.In the Drupal 6 branch of Nodequeue, you see a handy nodequeue_load_nodes function which will load the nodes in the queue, without rendering them. This function doesn't exist in the Drupal 5 version. We really should keep the API the same between both the Drupal 5 and 6 branches, so lets add that function........
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | nodequeue_load_node.patch | 2.6 KB | robloach |
| nodequeue_load_node.patch | 1.46 KB | robloach |
Comments
Comment #1
robloachUpdated status.
Comment #2
ezra-g commentedThis was intended as a replacement to nodequeue_nodes, since rendering nodes in this function was unnecessary. However, making this change between 5 and 6 caused some confusion. I've spent some time already indicating the difference between this part of the Nodequeue 5 and 6 APIs, so if we add this to Nodequeue for Drupal 5, I would appreciate if you also made the relevant changes to the documentation and the readme ;).
Comment #3
robloachGood call! The function signature of the nodequeue_load_nodes function now matches that of the Drupal 6 version. Also adds the documentation in README.txt.
Comment #4
ezra-g commentedHi Rob,
I'm sorry about any confusion, but what I meant in my original reply was that we could have both kinds of functions -- nodequeue_nodes and nodequeue_load nodes. This change between the D5 and D6 version caused a lot of confusion with some developers, so rather than make this API change in the Drupal 5 branch, I'd support adding an additional function, not replacing the existing one.
Thanks for taking the time to reroll the patch!
Comment #5
robloachThat patch doesn't replace the existing function, it simply adds a new function, and makes the original rendering function use it. If people get confused, then they can read the documentation that comes with it ;-) . Both the API and the functionality for
nodequeue_nodesstays the same, there's just an added newnodequeue_load_nodesfunction, like you suggested ;-).Comment #6
ezra-g commentedDrupal 5 is no longer a supported core version.