Closed (fixed)
Project:
ImageAPI
Version:
5.x-1.0-beta
Component:
ImageAPI Imagick
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jan 2008 at 04:30 UTC
Updated:
6 Mar 2008 at 11:21 UTC
(the id tag in the file was wrong, so I can't give you the exact version number :-) ... but I guess it's 1.4 2007/12/28 )
I know, I hate having to think about PHP4, but:
Parse error: syntax error, unexpected '{' in /home/.../sites/default/modules/imageapi/imageapi_imagick.module on line 42
function imageapi_imagick_image_close($image, $destination) {
try {
$image->res->writeImage($destination);
}
catch(Exception $e) {
//print_r($e);
return FALSE;
}
}
.. I swore a bit, but then found a way to convince my host d'jour to upgrade, so I can't be bothered backporting a fix for this.
I don't mind if you also call this "won't fix" as I really want to pressure for PHP5
But you might wanna stick it in the requirements or readme, because when it bites it brings someones whole site down in an hard-to-fix way
Comments
Comment #1
dopry commentedYeppers add php 5.2 to imageapi_requirements... Since imagemagick etc depend on it I think it should be sufficient.
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.