Le téléchargement de fichier ne fonctionne pas pour tous les fichiers

Question

J'utilise le code ci-dessous pour télécharger des fichiers avec une extension autorisée de moins de 5 Mo. Mais en utilisant ce code, tout doc ou pdf etc ne télécharge pas! Par exemple: un fichier docx de 4,78 Mo ou un téléphone Windows de 1,64 Mo ne télécharge pas!

$allowedExts = array("gif", "jpeg", "jpg","png","pdf","doc","docx","txt","rtf","bmp","psd","zip","rar","ppt","pptx"); 

$extension = end(explode(".", $_FILES["file"]["name"])); 

if (in_array($extension, $allowedExts) && $_FILES["file"]["size"]<5242880 && $_FILES["file"]["error"]<=0) { 
    $rand = rand(000,999); 
    $tempfile = $_FILES["file"]["name"]; 
    $file = $time . "=" . $rand . "=" . $tempfile; 

    if(file_exists("upload/".$file)) { 
     header("location:home.php?error=error"); 
    } else { 
     move_uploaded_file($_FILES["file"]["tmp_name"],"upload/".$file); 
    } 
} else { 
    header("location:home.php?error=error"); //this gets executed for some doc or pdf files ! 
} 

Answer 1

Pour ce fichier "Holidays.docx '" il me semble que le fichier dépasse upload_max_filesize. Au moins c'est ce que l'var_dump dit au sujet de $_FILES['file']['error'] qui est égale à 1 (UPLOAD_ERR_INI_SIZE)

Vous devez Chech paramètres php.iniupload_max_filesize et post_max_size parce que c'est la limite réelle de la taille fichier peut être téléchargé sur le serveur. Par défaut upload_max_filesize = 2M, votre limite de 5 Mo ne signifie rien.

Vous pouvez également utiliser ini_get('upload_max_filesize'); pour obtenir le paramètre d'exécution.
Si vous ne savez pas où se trouve votre fichier de configuration php.ini, utilisez phpinfo().

Aussi à savoir est que PHP retournera $_FILES['file']['size'] égal à 0 si le fichier dépasse la limite de taille de fichier - ce qui est exactement ce qui s'est passé.

Donc tout fonctionne comme il se doit. Vous devriez poster plus d'informations sur var_dump() pour les autres fichiers qui ne sont pas téléchargés.

À noter également, vous ne vérifiez pas les extensions de fichiers en respectant la casse de sorte que les fichiers avec des extensions comme .DOCX ne seront pas acceptés par votre script.

Answer 2

Vous combinez ici trois conditions dans vos instructions if et si l'une d'elles échoue, le bloc else sera exécuté. Mais il ne vous dira pas quelle condition a échoué.

if (in_array($extension, $allowedExts) && $_FILES["file"]["size"]<5242880 && 
$_FILES["file"]["error"]<=0) { 

ici:

• in_array($extension, $allowedExts) - de votre sortie var_dump, cela semble être correct.

• $_FILES["file"]["error"]<=0) - c'est correct, mais vous n'avez pas besoin de vérifier si elle est moins à 0. Les erreurs vont de 1 à 8 et ne seront pas aller en dessous de 0, mais qui ne sera pas à l'origine des problèmes cependant.

• $_FILES["file"]["size"]<5242880 - cela pourrait échouer. Vérifiez vos configurations php.ini et découvrez ce que upload_max_size est. Si elle est inférieure à la taille du fichier, la condition échouera et le bloc else sera exécuté.

À des fins de dépannage, je vous recommande de scinder les instructions if en blocs séparés. De cette façon, vous comprendrez quelle partie s'exécute et vous aidera à trouver pourquoi.

Un exemple:

$check = True; 
if (!condition) { 
    $check = False; 
} 
if (!condition) { 
    $check = False; 
} 
if (!condition) { 
    $check = False; 
} 

if ($check == True) { 
    # code... 
} 

---

Voici quelques autres améliorations mineures pour votre code:

Vous utilisez actuellement:

$extension = end(explode(".", $_FILES["file"]["name"])); 

pour obtenir l'extension de l'image. Cela pourrait échouer si l'extension est dans un cas différent.

Modifier ceci:

$extension = strotolower(end(explode(".", $_FILES["file"]["name"]))); 

ou mieux encore:

$extension = strtolower(pathinfo($_FILES["file"]["name"], PATHINFO_EXTENSION)); 

Deuxièmement, vous utilisez $time variable dans la déclaration suivante:

$file = $time . "=" . $rand . "=" . $tempfile; 

Ce ISN variable » t défini nulle part et PHP va lancer une erreur Undefined variable lorsque vous exécutez le s cript. Changez-le pour:

$time = time(); 
$file = $time . "=" . $rand . "=" . $tempfile; 

Espérons que cela aide!