Skip to content

Add imagick extension support

Import Userrequested to merge
github/fork/danloveg/dev/2163-switch-to-imagick-library into qa/2.x

Closes #2163

  • Installs the imagick extension in the application image.
  • Adds a new sfImagickAdapter that can be used with the sfThumbnail class.
  • Adds deprecation warnings to methods using pdfinfo, convert, or identify with sfImageMagickAdapter. The old adapter still works, but the new adapter is used if imagick is installed.
  • Refactored exploding multi-page assets to use imagick if it's installed.

Things that are not complete:

  • I have not created a policy.xml file, I wanted some feedback on this PR before looking into that.
  • This PR does not update any of the deployment documentation or any Ansible scripts.
  • Unit tests. (Tests added)
  • Copyright block at the top of the new adapter? I added a new file to a plugin that was not developed at Artefactual, not sure what sort of copyright case that falls under. I'd guess the regular AtoM copyright block but I'm not sure whether the licenses are compatible. (Added the default AtoM copyright block based on CODE_OF_CONDUCT.md)

Also of note, I did not implement the fancy thumbnail method that sfImageMagickAdapter does since I couldn't see any place where those custom methods were being used.

Merge request reports