Add an action to convert JPG to PNG.
This is needed for the imagecache_actions rounded corners at http://drupal.org/node/320186 to work.

Comments

druvision’s picture

druvision commented
Version: 5.x-2.3 » 6.x-1.x-dev
Category: feature » bug
Priority: Normal » Critical

This is implemented in 6.x, but when I try to use it, I get the following errors:

warning: imagecopyresampled(): supplied argument is not a valid Image resource in imageapi_gd.module on line 85.
warning: imagesavealpha(): supplied argument is not a valid Image resource in canvasactions.inc on line 637.
warning: imagealphablending(): supplied argument is not a valid Image resource in canvasactions.inc on line 638.
warning: imagecolortransparent(): supplied argument is not a valid Image resource in canvasactions.inc on line 641.
warning: imagesx(): supplied argument is not a valid Image resource in canvasactions.inc on line 643.
warning: imagesy(): supplied argument is not a valid Image resource in canvasactions.inc on line 644

It seems the change to the new API has caused lots of issues.

druvision’s picture

druvision commented

OK, the errors disappeared once I've upgraded my ImageAPI module. Rounded corners now work for the latest Drupal6 version once I and a 'change format' action to convert the image type to PNG.

druvision’s picture

druvision commented
Status: Active » Closed (fixed)
dman’s picture

dman commented

o_O

Well the big issue is that imagecache and imageAPI changed their internals at some point a month or so ago.
Without a version number update to indicate that things were different.
Now I can just say that this version of imagecache actions worked for me last month ... but I can't point at which version works with which reliably.
I did not expect to have to go back and path the DRUPAL5 branch to allow for a minor (variable renaming) change. And for me to do so will put my release code out of sync with the old DRUPAL-5 stable.
fecking frustrating.

I try to keep my latest dev and recommended up to date with latest imagecache recommended, but older versions may be out of sync with each other. (!)

dman’s picture

dman commented

@ngf, I think you are on the wrong website.
You may be interested to find that Drupal runs PHP, not VB.NET

- edit. Actually scratch that, those posts are just spam. Removed.