Discussion :

[Agité]

J'ai fais un script pour upload des avatars sur mon disque et mettre ma base de donnée a jour, j'arrive bien a retourner une erreur quand le format n'est pas bon mais pas quand la taille depasse.

De plus il ne me copie plus les fichiers dans le repertoire de destination, je ne comprends pas d'ou viens l'erreur et je cherche depuis hier a comprendre.

J'ai rajouter des exit() mais le script ne s'arrete pas et fonctionne pourtant bien puisqu'il me retourne l'erreur.

Si quelqu'un voit une faute quelque pars :

<?php
require ("connect.php");

if(isset($_FILES['photo']))

{
//parametres
unset($erreur);
$extensions_ok = array('png', 'gif', 'jpg', 'jpeg');
$taille_max = 100000;
$dest_dossier = 'c:/www/Avatars/';

// On verrifie les extensions
if( !in_array( substr(strrchr($_FILES['photo']['name'], '.'), 1), $extensions_ok ) )
{
header("location: chavatar.php?erreur1&PHPSESSID=".$_POST['session']);
$error = true;
exit();
}
// On verrifie la taille du fichier
elseif( file_exists($_FILES['photo']['name']) and filesize($_FILES['photo']['name']) > $taille_max)
{
$error = true;
header("location: chavatar.php?erreur2&PHPSESSID=".$_POST['session']);
exit();
}

// Si ficher = bonne taille & bonne extension
if(!isset($erreur))
{

$dest_fichier = basename($_FILES['photo']['name']);
//formatage nom fichier
//enlever les accents
$dest_fichier = strtr($dest_fichier,
'ÀÁÂÃÄÅÇÈÉÊËÌÍÎÏÒÓÔÕÖÙÚÛÜÝàáâãäåçèéêëìíîïðòóôõöùúûüýÿ',
'AAAAAACEEEEIIIIOOOOOUUUUYaaaaaaceeeeiiiioooooouuuuyy');
// remplacer les caracteres autres que lettres, chiffres et point par _
$dest_fichier = preg_replace('/([^.a-z0-9]+)/i', '_', $dest_fichier);
// On upload le fichier
move_uploaded_file($_FILES['photo']['name'], $dest_dossier . $dest_fichier);

// On remet a jour la base de donnée
$valide = "UPDATE `art` SET avatar = '".$_FILES['photo']['name']."' WHERE nom = '".$_POST['pseudo']."' ";
$test = mysql_query($valide) or die("erreur ! ".mysql_error());

//Redirection sur page precedente
header("location: chavatar.php?1&PHPSESSID=".$_POST['session']);
}
}else{
echo "erreur";
header("location: chavatar.php?erreur1&PHPSESSID=".$_POST['session']);
}




?>

[Agité]

Je viens de faire quelque test supplementaire et j'avais oublier de mettre le nom temporaire pour le fichier ...

move_uploaded_file($_FILES['photo']['tmp_name'], $dest_dossier . $dest_fichier);

Par contre il ne me verrifie toujours pas la taille

[bkill]

salut,

tu essayes de vérifier la taille du fichier en te basant sur ['name'], mais ca c'est le nom original du fichier... Au pire, il faudrait que tu utilises ['tmp_name'], au mieux, tu peux encore comparer avec ['size'] qui contient déjà la taille du fichier en question!

++

[Agité]

Je test avec :

elseif( file_exists($_FILES['photo']['name']) and $_FILES['size'] < $taille_max)

mais ne marche pas non plus

[bkill]

sorry de pas avoir été assez clair

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$_FILES['photo']['size']

mieux comme ca

[Agité]

elseif( file_exists($_FILES['photo']['name']) and $_FILES['photo']['size'] > $taille_max)

ne me retourne rien non plus il ne copie pas le fichier sur le disque mais ne me retourne pas d'erreur :/