I am not sure I am on totally secure ground here but I think I have come across a problem with this module vis a vis node_load and caching.

From looking at the node_load function you will not get a cache node (if it exists) if the parameter sent to node_load is not numeric (a nid).

This module calls hook_form_alter and when node_load is invoked it uses the array('nid' => $nid) syntax rather just passing a $nid.

I noticed via devel.module that it appeared that multiple node_load calls where going to the database rather than using the cache. When I changed this module to call node_load with the $nid rather than the array, all those extra queries (for node_load calls that could not utilize cache) went away.

Hope my explanation makes sense.

Comments

ahoeben’s picture

Makes sense, will look in to it. Thanks

ahoeben’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

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

ahoeben’s picture

Title: Problem with noad_load » Problem with node_load