[FreeTrade] Error (design flaw) in departments screen

[Jon Griffin]

In further research into the matter the problem is actually a corrupted .jpg. It is still a design flaw in FreeTrade though. The problem is the getimagesize() function in PHP is potentially hazardous to your health. If the image is corrupt the array is null. Therefore your code breaks. A simple if statement should fix this. I will work on it later tonight.


DISREGARD-----
In admin entering a description for (example) a department with quotes needs to be escaped. This is not the default behavior. One day I will look at the code and regsub " with \".
This is very confusing to the end users who are doing the admin. If people get errors only on certain products check this out.
The message will be something like this:

 
Warning: No such index in string in /home/bodylight/modules/screens/department on line 91
BORDER="0" ALIGN="left"> Vanilla Crunch

I don't have time to fix it now, but someone may be saved some grief if they see this. The description of  this department was: "The Juicy Candles." Incredibly fragrant travel candles...
and needed to be \"The Juicy Candles.\" ....

Thanks,

Jon Griffin

[EMAIL PROTECTED]

Check out:

<http://www.salsablanca.com>

<http://www.jongriffin.com>