<br><div class="gmail_quote">On Thu, Feb 26, 2009 at 11:21 AM, Nabil Alsharif wrote:<br><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
IMO having a hook_nodeapi_load and hook_nodeapi_post_load could get a<br>
little confusing and is redundant. I agree with Josh in that there<br>
should be a $node->cacheable field.<br>
</blockquote><div><br>This was in some of the earlier iterations of <a href="http://drupal.org/node/111127">http://drupal.org/node/111127</a> - the issue is that one rogue contrib module could completely disable node caching on a site by setting this flag. With _load() and _post_load() the worst that happens if you get them confused is you end up with some stale data or a single module not taking advantage of the cache as well as it could.<br>
<br>Either way, would be really great to have some more reviews of the issue itself, since it's been sitting at RTBC for a while, and discussion on this list doesn't get taken into account on the issue queue.<br><br>
Nat<br></div></div><br>