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

CommentFileSizeAuthor
nodequeue-load-multiple.patch946 bytesjustafish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JKingsnorth’s picture

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

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.