File Upload Help

SP8600

Maybe some one can see what i am trying to achive here, but i shall try and explain it as best i can.

Ok i am trying to create an upload script that takes file types that will be allowed to be uploaded from an array. Once this is done it needs to check the extension of the file to see if it matches any in that array, and if it is not to big and is an allowed file type then upload it to the "upload" directory.

Well hope that kind of explains what i need to do. Hope some one knows how i can sort this code out to do that. Thanks.

$fileType =  array("$fileTypes");
if (!isset($_FILES['attachemnt'])) {}
if (is_uploaded_file($_FILES['attachemnt']['tmp_name'])) {
if ($_FILES['attachemnt']['size']>$attachmentMax) { echo "The file is too big<br>\n"; exit; }
$str = explode(".",$_FILES['attachemnt']['name']); 
$ext = end($str);
if ($_FILES['attachemnt']['type']=="$fileType ") {
$res = copy($_FILES['attachemnt']['tmp_name'], $path . $fname);
} 
else {echo "Wrong file type<br>\n"; exit; }
}

The_Chancer

There is a great tutorial on this site - look at this for an idea....

barand

I think this might do it for jpg. png or gif

$fileType =  array('jpg', 'gif', 'png');
if (!isset($_FILES['attachemnt'])) {}
if  (is_uploaded_file($_FILES['attachemnt']['tmp_name'])) {
if ($_FILES['attachemnt']['size']>$attachmentMax) { echo "The file is too big<br>\n"; exit; }
$fname = $_FILES['attachemnt']['name']; 
list ($name,$ext)= explode(".", $fname);
if (array_search($ext,$fileType) !== false) {
$res = copy($_FILES['attachemnt']['tmp_name'], $path . $fname);
} 
else {echo "Wrong file type<br>\n"; exit; }
}

hth

Bijan

Originally posted by barand
I think this might do it for jpg. png or gif

$fileType =  array('jpg', 'gif', 'png');
if (!isset($_FILES['attachemnt'])) {}
if  (is_uploaded_file($_FILES['attachemnt']['tmp_name'])) {
if ($_FILES['attachemnt']['size']>$attachmentMax) { echo "The file is too big<br>\n"; exit; }
$fname = $_FILES['attachemnt']['name']; 
list ($name,$ext)= explode(".", $fname);
if (array_search($ext,$fileType) !== false) {
$res = copy($_FILES['attachemnt']['tmp_name'], $path . $fname);
} 
else {echo "Wrong file type<br>\n"; exit; }
}

hth [/B]

Using this code isn't what I really suggest! You are only checking the extension, what if someone just changed the extension, huh? Well, you can use getimagesize() and find out whether it's an image or not, what type it is (swf, jpg, gif,bmp,png,...), whether it's CMYK or not and lots of other things!

About working with the $_FILES array, why don't you put your element in another variable that you only work with a one dim array?