Nothing specific... it's just a revamp of a system I've used for a few years. The other system had a flaw which didn't become apparent until recently, so I thought having a "peer review" might help me avoid potential problems.
Yes, the tempImage folder is outside the webroot. As far as "older images" in the userImages folder, the clients control those. I do now have a line of code that deletes in the initially uploaded image from the tempImage folder after all processing on the image is complete. That gets rid of those 5MB photos clients upload just so I can get a 40KB photo from it. Thanks for the feedback! Rick -----Original Message----- From: Justin Scott [mailto:[email protected]] Sent: Saturday, August 20, 2011 3:34 PM To: cf-talk Subject: Re: Anyone care to review this image handling process? > This is part of a component method for handling uploaded images > that I've recently revamped. Thoughts on this? Is there anything specific you're concerned about with that code? It looks pretty straightforward, is uploading the image to a temp folder (presumably outside the webroot so that's good). The only thing I'd point out is that it appears to be keeping older versions indefinitely which could clutter up the image folder (unless there's some other process to clear out older images after a while). Overall it looks pretty simple and straightforward to me. -Justi ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Order the Adobe Coldfusion Anthology now! http://www.amazon.com/Adobe-Coldfusion-Anthology/dp/1430272155/?tag=houseoffusion Archive: http://www.houseoffusion.com/groups/cf-talk/message.cfm/messageid:346895 Subscription: http://www.houseoffusion.com/groups/cf-talk/subscribe.cfm Unsubscribe: http://www.houseoffusion.com/groups/cf-talk/unsubscribe.cfm
