Discussion :

[bambou]

Bonjour,

J'essaye de faire un formulaire qui comporte un upload de fichier. Lorsque je test d'uplader un fichier, ca ne marche pas, et je n'arrive pas a avoir un descriptif de l'erreur qui se produit. Voici le code que j'utilise:

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
26
    // Upload picture
    if(!empty($_FILES['picture']['tmp_name']) AND is_uploaded_file($_FILES['picture']['tmp_name'])){
      //Verifying size
      if(filesize($_FILES['picture']['tmp_name'])<51200){
        //Verifying Type
        list($largeur, $hauteur, $type, $attr)=getimagesize($_FILES['picture']['tmp_name']);
        //If Type is GIF/JPEG/PNG (Respectivly 1/2/3) then the picture is copying
        if($type==1){
          $ext = ".gif";
        }
        if($type==2){
          $ext = ".jpg";
        }
        if($type==3){
          $ext = ".png";
        }
        if($type==3 || $type==2 || $type==1){
          $picture = "../uploadedIMG/".$barcode."-".$shelf."-".$concentration."-".$row."-".$col."-".$time.$ext;
          //Copie le fichier dans le répertoire de destination
          if(!move_uploaded_file($_FILES['picture']['tmp_name'], $rep)){
            $error = 'Error during picture copy on the server, try again later or contact an administrator';
          }
        }else{$error = "Picture must be a jpeg, gif or png file";}
      }else{$error = "Picture is too big, please compress it!";}
    }else{$error = "Error during upload : ".$_FILES['picture']['error'];}
// END Upload Picture

et voici la configuration du php.ini :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
;;;;;;;;;;;;;;;;
; File Uploads ;
;;;;;;;;;;;;;;;;
 
; Whether to allow HTTP file uploads.
file_uploads = On
 
; Temporary directory for HTTP uploaded files (will use system default if not
; specified).
;upload_tmp_dir =
 
; Maximum allowed size for uploaded files.
;set a 200M pour test
upload_max_filesize = 200M

Le seul retour que j'ai, est l'affichage du message d'erreur :

Error during upload :

et enfin, voici l'HTML...on sait jamais

Code :

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

<input class="input" type="file" name="picture" />

merci pour votre aide

[Anduriel]

$rep est bien défini?

[Yogui]

Salut

Je te recommande d'utiliser switch plutôt que de mettre des if() à la suite comme cela.

Rassure-moi, tu fais bien echo $error; quelque part ?

[Anduriel]

Ah désolé j'ai pas vu de quel if sortait l'erreur... Sinon, au cas où, as-tu bien mis un champ MAX_FILE_SIZE?

[Yogui]

Oops pardon, je n'avais pas vu non plus... Pour me faire pardonner, voici comment j'aurais écrit ton code :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
<?php
 
$error = NULL;
 
// Upload picture
if(empty($_FILES['picture']['tmp_name']) or !is_uploaded_file($_FILES['picture']['tmp_name']))
{
    $error = "Error during upload : ".$_FILES['picture']['error'];
}
else
{
    //Verifying size
    if(filesize($_FILES['picture']['tmp_name']) >= 51200)
    {
        $error = "Picture is too big, please compress it!";
    }
    else
    {
        //Verifying Type
        list($largeur, $hauteur, $type, $attr)=getimagesize($_FILES['picture']['tmp_name']);
 
        //If Type is GIF/JPEG/PNG (Respectivly 1/2/3) then the picture is copying
        switch($type){
            case IMAGETYPE_GIF:
                $ext = '.gif';
            break;
 
            case IMAGETYPE_JPEG:
                $ext = '.jpg';
            break;
 
            case IMAGETYPE_PNG:
                $ext = '.png';
            break;
 
            default:
                $error = 'Picture must be a jpeg, gif or png file';
        }
 
        if(empty($error))
        {
            $picture = "../uploadedIMG/".$barcode."-".$shelf."-".$concentration."-".$row."-".$col."-".$time.$ext;
            //Copie le fichier dans le répertoire de destination
            if(!move_uploaded_file($_FILES['picture']['tmp_name'], $rep)){
                $error = 'Error during picture copy on the server, try again later or contact an administrator';
            }
        }
    }
 
    if(!empty($error)){
        echo $error;
    }
}
// END Upload Picture
 
?>

Tu peux trouver la gestion d'erreurs ici : http://g-rossolini.developpez.com/tu...p/formulaires/

[bambou]

Bonjour,

desole pour le retard ( long week-end ) et merci beaucoup pour toutes ses reponses

$rep est bien défini?

erf..effectivement $rep n'est pas defini

Code :

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

if(!move_uploaded_file($_FILES['picture']['tmp_name'], $rep)){

a ete remplace par (cf code du 1er post) :

Code :

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

if(!move_uploaded_file($_FILES['picture']['tmp_name'], $picture)){

Rassure-moi, tu fais bien echo $error; quelque part ?

le echo est fait plus bas dans le code, mais il est la

Sinon, au cas où, as-tu bien mis un champ MAX_FILE_SIZE

erf...non, je suis alle voir le lien de yogui ou il y a un champs MAX_FILE_SIZE, de type hidden qui est utilise, mais je ne comprend pas a quoi il sert...

Tu peux trouver la gestion d'erreurs ici : http://g-rossolini.developpez.com/tu...p/formulaires/

J'ai remplace mes if() par le switch, merci pour le code
Pour le lien, le serveur ou va tourner la page est sous php 4, donc certaine parties du code ne vont pas fonctionner...

...

Je pense avoir trouve d'ou venait l'erreur...pour d'obscures raisons, mon boss m'a demande de faire le formulaire avec GET comme methode d'envoi et apparemment (selon la console javascript de firefox), il faut faire du POST pour faire de l'upload d'image...je vais changer la methode et tester voir si c'est mieu...

[bambou]

ok c'etait bien due au fait que j'envoyais les info par GET et non par POST

Donc je vais "tag resolu" le post, mais si vous pouvez m'expliquer ( en PM eventuelement) tout de meme le coup du MAX_FILE_SIZE et pour passer le code du lien de yogui pour qu'il soit compatible php 4, ce serait vraiment sympa

En tout cas, merci beaucoup a tous les deux pour votre coup de pouce

a bientot

[Yogui]

Il n'y a guère que quelques constantes qu ne sont pas disponibles en PHP4. Essaie, si ça marche c'est bon.
Le MAX_FILE_SIZE n'est pas géré par tous les navigateurs.

[bambou]

d'accord, encore merci pour tout!