> > That's what I tried to do when > > I wrote version 2.0 of my CFC and I achieved a huge increase in quality. > > Massimo: Just to let you know... I was playing around with your CFC, > and noticed that on some images (usually horizontally oriented ones), > bufferedCrop() would throw an "outside of raster" error. I was able to > fix it by modifying the code to check for crop operations that exceed > the height of the image. > > With that said, thanks for putting the code out there.
Thanks Roger. I am aware of this but I am still not sure on what's the best way to handle it. Until now I tried to leave out any form of "logic" from the CFC; in my vision it only has to perform the basic operations, then it's up to the developer to wrap any logic around it (see the post about conditional cropping). The CFC could check if the crop area exceed image's dimensions and adjust the cropping area. I still have mixed feelings about this... It will surely make the CFC more "monkey proof", but I don't want a component that try to "outsmart" its user. Personally I would again, extend the CFC and add a "saveCrop" method that just wrap the logic around the base method. Feedback and opinions on this would be very much appreciated. ---------------------------- Massimo Foti Tools for ColdFusion and Dreamweaver developers: http://www.massimocorner.com ---------------------------- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Logware (www.logware.us): a new and convenient web-based time tracking application. Start tracking and documenting hours spent on a project or with a client with Logware today. Try it for free with a 15 day trial account. http://www.houseoffusion.com/banners/view.cfm?bannerid=67 Message: http://www.houseoffusion.com/lists.cfm/link=i:4:220518 Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4 Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4 Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4 Donations & Support: http://www.houseoffusion.com/tiny.cfm/54
