PHP Upload Form

Question

I want to build a working upload file form and I use codes in this tutorial - http://www.w3schools.com/PHP/php_file_upload.asp

Everything works well. The only problem is, the original code only includes file type of gif and jpg. I need png also.

What I did is modified this lines :

    > if ((($_FILES["file"]["type"] == "image/gif")
    || ($_FILES["file"]["type"] == "image/jpeg")
    || ($_FILES["file"]["type"] == "image/pjpeg"))

to this :

if ((($_FILES["file"]["type"] == "image/gif")
|| ($_FILES["file"]["type"] == "image/jpeg")
|| ($_FILES["file"]["type"] == "image/pjpeg"))
|| ($_FILES["file"]["type"] == "image/png"))

but it turns error. Any advice?

Using IDEs like netbeans, eclipse, etc. provide syntax highlighting which make these class of errors like extra parentheses easy to spot. — Kinjal Dixit
– Kinjal Dixit, Commented Dec 28, 2010 at 14:55

Kinjal Dixit · Accepted Answer · 2010-12-28 14:26:26Z

5

remove the extra ) after pjpeg to make it look like this:

if (($_FILES["file"]["type"] == "image/gif") || 
($_FILES["file"]["type"] == "image/jpeg") || 
($_FILES["file"]["type"] == "image/pjpeg") || 
($_FILES["file"]["type"] == "image/png"))

There is also an extra ( before the IF which I have removed, but that could be a copy paste problem from you.

answered Dec 28, 2010 at 14:26

Kinjal Dixit

7,9412 gold badges63 silver badges68 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Tomás Pérez · Accepted Answer · 2010-12-28 14:33:34Z

0

There is a syntax error in the "if" statement:

Change

if ((($_FILES["file"]["type"] == "image/gif") || ($_FILES["file"]["type"] == "image/jpeg") || ($_FILES["file"]["type"] == "image/pjpeg")) || ($_FILES["file"]["type"] == "image/png"))

to:

if (($_FILES["file"]["type"] == "image/gif") || ($_FILES["file"]["type"] == "image/jpeg") || ($_FILES["file"]["type"] == "image/pjpeg") || ($_FILES["file"]["type"] == "image/png"))

answered Dec 28, 2010 at 14:33

Tomás Pérez

11 bronze badge

Comments

Damiqib · Accepted Answer · 2010-12-28 14:39:52Z

0

You could modify it like this to make it look nicer and easier to modify later on:

if(in_array($_FILES['file']['type'], array('image/jpeg', 'image/pjpeg', 'image/gif', 'image/png'))

Now if you need to add more image types, you can just add a value to the array.

answered Dec 28, 2010 at 14:39

Damiqib

1,0772 gold badges13 silver badges23 bronze badges

Collectives™ on Stack Overflow

PHP Upload Form

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related