Nuking a couple of instances of node_load being used inside loops.

CommentFileSizeAuthor
nodequeue-load-multiple.patch946 bytesjustafish

Comments

jkingsnorth’s picture

jkingsnorth commented

Seems reasonable to me. I haven't tested the actual benefits but the code looks cleaner. Patch applies and no errors appeared on a quick test.

+1

One more for RTBC?

  • fizk committed cf5cec9 on authored by justafish 
    Issue #2334847 by justafish: Load nodes more efficiently
fizk’s picture

fizk commented
Status: Needs review » Fixed
Related issues: +#2314177: Fix D6 style nodequeue_view_nodes() arguments

Committed, thanks!

Status: Fixed » Closed (fixed)

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