Whenever the UI for selecting access controls is shown for a node edit form, and CCK is enabled, the entire site cache is dumped. This is caused by an incorrect call to content_extra_field_weight() where the first parameter is NULL. CCK assumes that the cache of content types might need to be rebuilt, and rebuilds all cached data. This is a pretty significant performance hit. On a local copy of a client site, I was getting around 300ms savings per page request.

Patch forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deviantintegral’s picture

Status: Active » Needs review
FileSize
971 bytes

Here's the patch. Note this bug doesn't appear to exist in 7.x-1.x as we don't need to use extra weights from CCK at all.

mrf’s picture

Status: Needs review » Fixed

Tested and committed to 6.x-1.x-dev.

Status: Fixed » Closed (fixed)

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