Although this won't totally fix private files, at least it will make it possible to use devel and imagefield at the same time. Until http://drupal.org/node/140412 gets in, this won't benefit core modules (like upload.module, which is also broken by devel's query display), it will make a workaround possible for imagefield.

A better solution than this: wouldn't it be better to assume that all files which are not text/html shouldn't append the devel queries?

CommentFileSizeAuthor
shutdown_images.patch722 bytesquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Active » Needs review
moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Fixed

i went with the suggestion to bail if i don't see 'html' in headers

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented

reverted back to the strstr('images') solution. the html one did not work at all.