Correct way to write an IF statement based

kilbey1

I'm looking for the correct way to write this IF statement.

if($userfile_type != 'application/msword' || $userfile_type != 'text/plain' || $userfile_type != 'text/rtf') {

Where if it isn't a Word doc, Rich Text or Text, do this. I've tried this IF statement the above way, and a few other differert ways, each one indicating it's not a valid file, even if I upload a regular text file.

	if(($userfile_type != "application/msword") || ($userfile_type != "text/plain") || ($userfile_type != "text/rtf")) {

Thanks,
Eve

mtimdog

use && instead of || between them because you're trying to catch if it's not this and not that and not that ....then do this.

kilbey1

Ok, thanks. I wouldn't have known to do that. 🙂 Thanks. However, it's still coming back that it's an invalid file type, even if one of the files I upload is an allowed file type, i.e. Resume. txt.

Code:

	if(($userfile_type != "application/msword") && ($userfile_type != "text/plain") && ($userfile_type != "text/rtf")) {

Example:
http://www.wow-corp.com/temp/oppr.php?goto=apply&id=61
...and go ahead trying to upload a text file, rtf or Word format document.

mtimdog

If you echo out $userfile_type what does it say??

Note: type will only work if register_globals is on, otherwise you'd have to use $FILES (or $_HTTP_FILES... depending on your version of php)

kilbey1

OMG. :hangs head in shame: That clued me into something. Variable wasn't $userfile_type at all, it was $fileatt_type!

Thanks, works now.

mtimdog

Don't worry, that happens to everybody😃