Needs work
Project:
ImageAPI
Version:
7.x-1.x-dev
Component:
ImageAPI Imagick
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Jun 2008 at 13:32 UTC
Updated:
28 Oct 2009 at 23:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
wrunt commentedHere's the patch.
Comment #2
drewish commentedthis would need to happen in D6 and then be backported
Comment #3
wrunt commentedComment #4
sunThis should use http://api.drupal.org/api/function/file_get_mimetype/6 instead.
Also, I'm not sure what to do with this issue and #269327: Support for more image types (PDF, TIFF, EPS, etc.). Both seem to depend on the core patch in #269337: Support for more image types (PDF, TIFF, EPS, etc.), which has not been committed yet and would only be available in D7.
Comment #5
egfrith commentedI believe that imageapi is heading for core:
http://lists.drupal.org/pipermail/development/2009-March/032407.html
, so I wonder if it would be worth seeing if this patch would have a good home at: #416254: Add equivalent of image_get_info() at the toolkit level
But I'm not 100% sure exactly what is going on with image stuff in D7.
Comment #6
egfrith commentedI'm moving this issue to what I think is a more suitable home in imageapi, and posting a patch to imageapi that should work with the patch to core at: #269337: Support for more image types (PDF, TIFF, EPS, etc.).
There is already a patch to imageapi containing similar code, but this is for the 6.x version of imageapi #416254: Add equivalent of image_get_info() at the toolkit level, and I'd rather keep this separate.
At present this patch seems to work, though I have been having problems with the ordering elements in the admin form, which I haven't yet resolved. However, it does allow you to test the functionality.
Comment #7
egfrith commentedComment #8
egfrith commentedI've discovered that the _imageapi_imagemagick_convert() trying to check that $type:$dest is a file, which then fails. See #375218: Changing file type with imagemagick. Will try to update the patch soonish, but not tonight...
Comment #9
egfrith commentedI'll revise this patch once the equivalent patch for D6, #416254: Add equivalent of image_get_info() at the toolkit level, has been reviewed.