Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use imagick instead of command line for imagemagick #4498

Closed
rcubetrac opened this issue Mar 27, 2014 · 3 comments
Closed

Use imagick instead of command line for imagemagick #4498

rcubetrac opened this issue Mar 27, 2014 · 3 comments

Comments

@rcubetrac
Copy link

Reported by joydev on 27 Mar 2014 04:54 UTC as Trac ticket #1489734

Imagick is a native php extension to create and modify images using the ImageMagick API. This extension requires ImageMagick version 6.2.4+ and PHP 5.1.3+, which should not be a problem since Roundcube will drop PHP 5.2 support.

It should be used instead of executing the "convert" command since it is faster and safer, and we are not using advanced functions which is only available using command line.

You can refer to the example script in this page. There is no need to clone the image as the example though. To set color space (if required) use Imagick::setImageColorspace.

Keywords: image imagemagick imagick
Migrated-From: http://trac.roundcube.net/ticket/1489734

@rcubetrac
Copy link
Author

Comment by @alecpl on 27 Mar 2014 07:20 UTC

Could you create a patch/pull request? Note: this extension might be disabled, so a fallback to the old method will be still needed.

@rcubetrac
Copy link
Author

Comment by @alecpl on 6 May 2014 14:19 UTC

Done in 8968f95.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 6 May 2014 14:19 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant