Andrew,

the CCK's theme function theme_imagecache_formatter() loads the node for each image displayed. And nothing but the $node->nid is used (and its already given parameter). Displaying images on Views may force 80+ extra queries on each page. Generally doubling queries per page.

I suggest to avoid this load by changing the code to this no imagecache.module line 630 (a small change so i did not create a formal patch file):

function theme_imagecache_formatter($element) {
  if (isset($element['#item']['nid'])) {
    return imagecache_field_formatter($element['#field_name'], $element['#item'], $element['#formatter'], (object) $element['#item']);
  }
}

best regards,

massa

Comments

swentel’s picture

swentel commented
Status: Needs review » Closed (fixed)

I was bitten by this too and was looking for an issue that addressed this. But after some digging in the dev version to create a proper patch, this seems to be fixed however, so we can close this one, jeej!