in reference to the foreach

Greg K — Wed, 10 Dec 2008 18:39:00 +0000

in reference to the foreach above, even simpler would be:

$fileExt = substr($file['name'],strrpos($file['name'],"."));

return in_array($file['type'],$this->imageTypes) &&
!in_array($fileExt,$this->badextentions);

Note the use of strrpos(), this gets the LAST match of the period, not the first.

-Greg

This block of code will

pr0gr4mm3r — Wed, 10 Dec 2008 16:42:39 +0000

This block of code will always return false:

                foreach($this->imageTypes as $type){
                    if($file['type'] != $type){
                        return FALSE;
                    }
                }

The only way this could be true is if the image type of the uploaded image matched every type in your array.

Something like this would work:

                $match = false;
                foreach($this->imageTypes as $type){
                    if($file['type'] == $type){
                        $match = true;
                    }
                }
                return $match;

You use $_FILE['image'];,

greg — Wed, 10 Dec 2008 16:35:00 +0000

You use $_FILE['image'];, firstly, is the input field in your form also called "image"?
And second, shouldn't that be $_FILES['image']; (S)

As I can't see the forum, i have to ask, does the form have enctype="multipart/form-data", which is required.

Here's some handy reading if you hadn't already found it
http://uk2.php.net/features.file-upload