Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.7.0

Bug #1203 mega bug summary / with some wishes :)
Submitted: 2004-04-14 06:07 UTC
From: alan at akkbhome dot com Assigned: toby
Status: Closed Package: Image_Text
PHP Version: 4.3.3 OS: linux
Roadmaps: (Not assigned)    
Subscription  


 [2004-04-14 06:07 UTC] alan at akkbhome dot com
Description: ------------ Constructor: would be better to use: - $img = Image::construct($options); rather than $img = new ... $img->init(); Options: - var $options could do with additial comments (eg. example values.) - options['canvas'] = filename = autoload the file... - the options[colors] array needs explaining - default value of color causes e_Warning error. should be: array('#000000'), - optons['font_path'] should note it must explicty end with a directory seperator. eg. /xxx/yyy/ (or note that it auto adds it.. in set..) - no default for image_type is used. Display - error rasied after sending headers if no image type set.. in display. save - save does not appear to have an $imgout set anywhere

Comments

 [2004-04-14 06:08 UTC] alan_k
CVS file structure is also a bit non-standard :)
 [2004-04-14 06:13 UTC] alan at akkbhome dot com
defines for middle/center etc. are redundant, expensive (albiet not an issue here - as it will be slow anyway) and rather pointless.
 [2004-04-14 06:22 UTC] alan at akkbhome dot com
bugger - my email address is incorrect. on the report - can you email me directly when you fix it.. alan at akbkhome dot com.
 [2004-04-14 20:19 UTC] toby
Since I didn't catch you on IRC, some comments here: :) Some things have been updated in CVS. - Changes for ::construct() are not so easily possible, since I recently announced the API to be fixed. I would do, if necessary, but I guess it's just a cosmetic. - A huge doc is provided for the $options array by the set() method. Did you see? I added some more stuff regarding the path/file options. - The default image type is set in the $options array: 'image_type' => IMAGETYPE_PNG, - What exactly did you mean for the save() method? You can either set the file to save to through the $options array or directly when calling the method. Or did I get you wrong? - The CVS structure is my personal favor, because I can just include the packages from their CVS checkout directory. That's much more comfortable than handling endless links/copies/packagings. :) - Defines: I love it clear. ;) Therefor constants for the horizontal and vertical alignment exist. -
 [2004-04-19 05:07 UTC] toby
Thank you for your bug report. This issue has already been fixed in the latest released version of the package, which you can download at http://pear.php.net/packages.php