$_FILES check if empty

[_matrix_]

When I use the following to check if files exist, the form still submits as successfull even when no files are added?

if(!$_FILES['photoname']) { 
     $error_msg = 'You must upload your portfolio photos';
} 

Field formated like so:

while($counter <= $number_of_fields){
     echo '<input name="photoname[]" type="file" id="upload" />';
$counter++;
} 

Thanks

[logic_earth]

if ( $_FILES['whatever']['error'] > UPLOAD_ERR_OK )
  // we have an error! 

The rest is an exercise for you.

[_matrix_]

Thanks I will look into it..

But shouldnt if(!$_FILES['photoname']) work.. as its just like checking on post or get for empty fields?

[ScallioXTX]

But shouldnt if(!$_FILES['photoname']) work.. as its just like checking on post or get for empty fields?

Nope. It's the same as a text field; when it is submitted but there was no text filled in $_POST['myfield'] is still set, but it's set to an empty string.
$_FILES['photoname'] is also set, but there is no file in there. Same thing.

[_matrix_]

if($_FILES['photoname']['error'] > 0) { 
      $error_msg = 'You must upload a photo';
} 

From what I read, the above should work?

Altho it appears to stop the script continuing even when atleast one file is uploaded..

[ScallioXTX]

Maybe the file you're uploading is too big? Default max size for PHP uploads is 2MB if I remember correctly (plus there is a limit on the web server as well, if that's lower than PHP's limit that's the limit you need to honour).

[_matrix_]

File sizes are small.. kb's.. I tried filling all 5 fields with files.. but $error_msg alway populates with error message

[logic_earth]

Maybe you should read the error messages that is why they are there.
http://www.php.net/manual/en/feature...oad.errors.php

[_matrix_]

There are no errors.. but I guess the problem is because its an array of files being sent.. not a single file..

Array ( [0] => 0 [1] => 0 [2] => 0 [3] => 0 [4] => 0 ) 0

[ScallioXTX]

In that case you could check

if (array_sum($_FILES['photoname']['error']) > 0) {
    // There was an error 
}