Now that we have hook_image_alter(), most of the dirty tricks going on in image_enchanced_scaling_nodeapi() should be replaced with a nice and clean implementation of said hook. Apart from getting rid of a "hack", this would also eliminate the problem with double scaling (see the README file) which would improve the performance a lot.

This patch may be useful as a reference.

I'll do this myself as soon as I've got the time, but until then, patches are more than welcome, as always.

Comments

zoo33’s picture

zoo33 commented
Status: Active » Fixed

It's done! However, we're still doing the image scaling twice, since Image.module resizes the derivatives before invoking the hook. Too bad. But anyway, it's a lot cleaner now with this hook so it's still a good change.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)