Quote:
Originally Posted by MicroHellas
Max size in Kb or in dimensions?. I'm checking dimensions. Please try to understand something very important on coding. By adding lot's of restrictions, means lots of checks, something that is not so good to performance. Sometimes by using just simple logic is better. By allowing just gif,jpg,png on a max width eg 600, the photo only in a very few situations should be more than 200kb. So why to add one more restriction? I can accept this filesize restriction on audio and video files, but not on photos.
Mary
|
I can have a 4000x4000 image that is less than 20k.
If I set the limit to 800x600 in the options, this image, although very small in bytes, would not be allowed if I set the limits based on width/height.
The check can be for size in pixels, or the check can be for size in bytes. You chose dimensions - I respect your decision, but I don't necessarily agree with it.
>
Please try to understand something very important on coding
Thanks for the coding tip - I pretty much learned that long before I finished my masters degree in computer science.
btw: I had already added the multiple file upload before you released your 1.10 yesterday... I also added the ability to upload in .zip format and.... added additional "pay" options, added a maximum number of active ads per usergroupid option, and... changed the ad listings to show the title, the user, the price, and the expire date in tabular format in the "classifieds_categorieslistingadline" instead of just the title of the ad. Your original code however saved me a ton of work and for that, I definitely thank you.