I ran into a problem, working on a recent project: I wanted direct access to the node objects for a specific nodequeue, to make custom theming easier. The way it stands now, you must either use a view or accept the rendered output provided by nodequeue_nodes().
This patch is a small rewrite of nodequeue_nodes(). It adds a helper function, nodequeue_fetch_nodes(), which returns an ordered array of the node objects for a given queue.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | fetch_nodequeue_patch2.diff | 4.62 KB | davideads |
| fetch_nodequeue_patch.diff | 2.46 KB | davideads |
Comments
Comment #1
davideads commentedHere's a better version of this patch, which properly enables random fetching, and drops display-related arguments from the fetch function signature.
Comment #2
ezra-g commentedThis sounds reasonable to me . Thanks for the revised patch - I hope to review this soon,
Comment #3
tirdadc commentedThis is definitely useful, it would be great seeing this integrated into the next release.
Comment #4
ezra-g commentedhope to review and commit this this coming weekend. It does need some rerolling by someone, possibly me, to remove the lowercase 'true' changes that are unrelated to this patch.
Comment #5
ezra-g commentedI finally got a chance to give this some attention.
The code here is solid but it seems like the function naming has the potential to be confusing. nodequeue_fetch_nodes returns loaded (but not rendered) node objects, while nodequeue_fetch_front, nodequeue_fetch_back and nodequeue_fetch_random return rendered node objects. Perhaps the fetch functions should be renamed nodequeue_load_front (and so on), nodequeue_nodes would then be renamed nodequeue_view_nodes, and nodequeue_fetch_nodes would become nodequeue_load_nodes. Then, when using the load (formerly fetch) functions, module developers could simple call node_view from within their modules.
What do you think?
Thanks for your interest and patience. I'll review any feedback (or rerolls) quicker this time ;).
Comment #6
ezra-g commentedI made those name changes and did the related refactoring. Committed. Thanks!