The inclusion of utility.inc in some of the modules seems to be problematic, as they are just included via

require_once('utility.inc');

but should be included like

require_once drupal_get_path('module', 'imagecache_actions') .'utility.inc';

otherwise there might occur an error.

Patches attached

CommentFileSizeAuthor
imagecache_textactions-includeFix.patch512 bytessuit4
imagecache_coloractions-includeFix.patch402 bytessuit4
imagecache_canvasactions-includeFix.patch416 bytessuit4

Comments

dman’s picture

dman commented
Status: Needs review » Fixed

Thanks. True enough, we are supposed to be more careful with our includes, although I've not found problems with this one so far.
I get the idea with the patches - although
- there is no actual 'imagecache_actions' module to lookup, so that would have failed.
- if successful, that require would have needed an extra '/' in the pathname.

Anyway, I'm pathing the affected files with the simple:

require_once(dirname(__FILE__) .'/utility.inc');

and all is cool.

Rolling soon when I add another feature (imagefield)
Thanks!

Status: Fixed » Closed (fixed)

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