Discussion :

[digger]

Bonjour à tous

J'ai developpe ce peiti script en php qui permet d'uploader un fichier avec une restriction de taille et de format le script fonctionne mal je n'arrive pas trouver l'erreur
quand on upload un fichier de plus de 300 Ko le message d'erreur est toujours le premier

Veuillez sélectionner un fichier de type png, gif ou jpg

au lieu du second message j'ai l'impression qu'il ne rentre pas dans la seconde condition

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<? 
unset($erreur);
$rep_destination=$_SERVER['DOCUMENT_ROOT']."/Foto";
$extensions_ok = array('png', 'gif', 'jpg', 'jpeg');
$MAX_FILE_SIZE = 300000;
 
if (isset($_FILES["photo"])) {
 if( !in_array( substr(strrchr($_FILES['photo']['name'], '.'), 1), $extensions_ok ) )
  {
    $erreur = '<font color="red"><b>Veuillez sélectionner un fichier de type png, gif ou jpg !</b></font>';  
  }
  elseif( file_exists($_FILES['photo']['tmp_name'] > $MAX_FILE_SIZE))
  {
    $erreur = '<font color="red"><b>Votre fichier doit faire moins de 300Ko !</font></b>';
  }
 if(!isset($erreur)){
if (is_uploaded_file($_FILES["photo"]["tmp_name"]))
$destination=$rep_destination."/"
.$_FILES["photo"]["name"];
move_uploaded_file($_FILES["photo"]["tmp_name"],$destination);
 
}
}
?>

merci de votre aide

[crashyear]

Et si tu test d'abort la taille du fichier ca donne quoi.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<? 
unset($erreur);
$rep_destination=$_SERVER['DOCUMENT_ROOT']."/Foto";
$extensions_ok = array('png', 'gif', 'jpg', 'jpeg');
$MAX_FILE_SIZE = 300000;
 
if (isset($_FILES["photo"])) {
  if( file_exists($_FILES['photo']['tmp_name'] > $MAX_FILE_SIZE))
  {
    $erreur = '<font color="red"><b>Votre fichier doit faire moins de 300Ko !</font></b>';
  }
 elseif( !in_array( substr(strrchr($_FILES['photo']['name'], '.'), 1), $extensions_ok ) )
  {
    $erreur = '<font color="red"><b>Veuillez sélectionner un fichier de type png, gif ou jpg !</b></font>';  
  }
 if(!isset($erreur)){
if (is_uploaded_file($_FILES["photo"]["tmp_name"]))
$destination=$rep_destination."/"
.$_FILES["photo"]["name"];
move_uploaded_file($_FILES["photo"]["tmp_name"],$destination);
 
}
}
?>

[digger]

on peut tout envoyer il n'y a aucun message d'erreur

[Xunil]

Ta condition avec le file_exists n'est pas bonne.

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?php
 
$rep_destination=$_SERVER['DOCUMENT_ROOT']."/Foto";
$extensions_ok = array('png', 'gif', 'jpg', 'jpeg');
$MAX_FILE_SIZE = 300000;
 
if (isset($_FILES["photo"])) 
{
  if( !in_array( substr(strrchr($_FILES['photo']['name'], '.'), 1), $extensions_ok ) )
  {
    $erreur = '<font color="red"><b>Veuillez sélectionner un fichier de type png, gif ou jpg !</b></font>';  
  }
  elseif( file_exists($_FILES['photo']['tmp_name']) && filesize($_FILES['photo']['tmp_name']) > $MAX_FILE_SIZE )
  {
    $erreur = '<font color="red"><b>Votre fichier doit faire moins de 300Ko !</font></b>';
  }
  if(!isset($erreur))
  {
    if (is_uploaded_file($_FILES["photo"]["tmp_name"]))
    $destination=$rep_destination."/".$_FILES["photo"]["name"];
    move_uploaded_file($_FILES["photo"]["tmp_name"],$destination);
  }
}
 
?>

[digger]

il met met toujours Veuillez sélectionner un fichier de type png, gif ou jpg !

quand on upload un fichier de plus de 300 Ko
au lieu Votre fichier doit faire moins de 300Ko !</

[digger]

voila j'ai resolu le pb cela venait de la condition elseif
supprimer par un if

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?php
 
$rep_destination=$_SERVER['DOCUMENT_ROOT']."/Foto";
$extensions_ok = array('png', 'gif', 'jpg', 'jpeg');
$MAX_FILE_SIZE = 300000;
 
if (isset($_FILES["photo"])) 
{
  if( !in_array( substr(strrchr($_FILES['photo']['name'], '.'), 1), $extensions_ok ) )
  {
    $erreur = '<font color="red"><b>Veuillez sélectionner un fichier de type png, gif ou jpg !</b></font>';  
  }}
  if( file_exists($_FILES['photo']['tmp_name']) && filesize($_FILES['photo']['tmp_name']) > $MAX_FILE_SIZE )
  {
    $erreur = '<font color="red"><b>Votre fichier doit faire moins de 300Ko !</font></b>';
  }
  if(!isset($erreur))
  {
    if (is_uploaded_file($_FILES["photo"]["tmp_name"]))
    $destination=$rep_destination."/".$_FILES["photo"]["name"];
    move_uploaded_file($_FILES["photo"]["tmp_name"],$destination);
 
}
 
?>