téléchargerment : image abimée

**koKoTis** · 29/09/2008, 13h48

OK, merci encore

Donc si au lieu d'avoir mon deuxième fichier comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
if(!isset($_GET['jpg'])) 
    {
       header('location:erreur.php');
    }
else
    {
       $file = $_GET['jpg'];
    }
header("Content-Type: image/jpeg"); 
header("Content-Disposition: attachment; filename=".$file); 
readfile($file);
unlink($file);
exit();

Je créer ma fonction dans mon premier fichier comme cela:

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
<?php

function Arial()
{
   if(!isset($_GET['jpg'])) 
    {
       header('location:erreur.php');
    }
   else
    {
       $file = $_GET['jpg'];
    }
   header("Content-Type: image/jpeg"); 
   header("Content-Disposition: attachment; filename=".$file); 
   readfile($file);
   unlink($file);
   exit();
}


if(isset($_FILES['photo']))
{ 
     $dossier = 'imgs/';
     $fichier = basename($_FILES['photo']['name']);
     if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
         session_start();
         $_SESSION['upload_fichier'] = "imgs/".$fichier;
          echo 'Upload effectué avec succès !<br>';
           exec("mogrify -resize 640x640 imgs/".$fichier);
              echo ("<meta http-equiv='Refresh' content='2;URL=telecharg_suppr.php?jpg=imgs/$fichier'>");


     }
     else //Sinon (la fonction renvoie FALSE).
     {
          echo 'Echec de l\'upload !';
           echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>");
     }
}


?>

Comment dois-je appeler cette fonction ?

**leodi** · 29/09/2008, 14h08

Non, si tu fais tout dans la meme page, tu ne dois plus utiliser les $_GET, mais les variables correspondantes.
je t'invite a revoir tes sources, esseyer de voir comment marche chaque fonction, regarder les valeur de retour, et de comprendre parfaitement ton code, en commentant les lignes ou ce n'est pas clair pour toi, et en regroupant chaque grosse partie (upload / telechargement / suppression) en fonctions.

**koKoTis** · 29/09/2008, 14h20

je suis entrain de commenter et de modifier mon code, mais il y a une condition que je ne comprend pas, pourrai tu me l'expliquer ?

Code :

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

if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier))

**koKoTis** · 29/09/2008, 14h31

Donc voila ce que j'ai pu faire:

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
<?php
 
///////////////////////////////////////////////////////////////////////////////////////
// Fonction de téléchargement de l'image redimentionner:
////////////////////////////////////////////////////////
 
function telecarg_suppr()
{
   if(!isset($vrFichier)) 
    {
       header('location:erreur.php');
    }
   else
    {
       $file = $vrFichier;
    }
   header("Content-Type: image/jpeg"); 
   header("Content-Disposition: attachment; filename=".$file); 
   readfile($file);
   unlink($file); // Ici on supprime l'image une foi quelle à été télécharger.
   exit();
}
 
///////////////////////////////////////////////////////////////////////////////////////
 
///////////////////////////////////////////////////////////////////////////////////////
// Upload de l'image + apelle fonction télécharge:
//////////////////////////////////////////////////
 
if(isset($_FILES['photo']))
{ 
     $dossier = 'imgs/';
     $fichier = basename($_FILES['photo']['name']);
     $vrFichier = "imgs/".$fichier;
print($vrFichier);
     if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
 
          echo 'Upload effectué avec succès !<br>';
           exec("mogrify -resize 640x640 ".$vrFichier); // Redimention de l'image avec ImageMagick.
              telecarg_suppr(); 
              //echo ("<meta http-equiv='Refresh' content='0;URL=telecharg_suppr.php?jpg=$vrFichier'>");
 
 
     }
     else // Si l'image n'a pas été uploader.
     {
          echo 'Echec de l\'upload !';
           echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>");
     }
}
 
 
?>

Mais ca me renvoi ces erreurs:

Warning: Cannot modify header information - headers already sent by (output started at z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php:35) in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 11

Warning: Cannot modify header information - headers already sent by (output started at z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php:35) in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 17

Notice: Undefined variable: file in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 18

Warning: Cannot modify header information - headers already sent by (output started at z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php:35) in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 18

Notice: Undefined variable: file in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 19

Notice: Undefined variable: file in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 20

Warning: unlink(): Permission denied in z:\kookatix_interface\interface\kookatix_www\scripting\redimentionne image\upld_redim_640.php on line 20

Qu'est-ce qui ne va pas dans mon code ?

**leodi** · 29/09/2008, 14h45

Google est ton ami

http://www.google.fr/search?q=Warnin...ient=firefox-a

**koKoTis** · 29/09/2008, 14h53

oki,

Code :

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

PHP Warning: Impossible de modifier les informations d'en-tête

donc il disent:

Code :

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

Car la fonction header doit être appelée avant la génération de la moindre mise en page

Mis il n'y a pas de mise en page dans mon code, si ?

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
 
<?php
 
///////////////////////////////////////////////////////////////////////////////////////
// Fonction de téléchargement de l'image redimentionner:
////////////////////////////////////////////////////////
 
function telecarg_suppr($vrFichier)
 {
 
   header("Content-Type: image/jpeg"); 
   header("Content-Disposition: attachment; filename=".$vrFichier); 
   readfile($vrFichier);
   unlink($vrFichier); // Ici on supprime l'image une foi quelle à été télécharger.
   exit();
 }
 
///////////////////////////////////////////////////////////////////////////////////////
 
///////////////////////////////////////////////////////////////////////////////////////
// Upload de l'image + apelle fonction télécharge:
//////////////////////////////////////////////////
 
if(isset($_FILES['photo']))
{ 
     $dossier = 'imgs/';
     $fichier = basename($_FILES['photo']['name']);
     $vrFichier = "imgs/".$fichier;
     //print($vrFichier); // Vérification de contenu de la variable avec chemin et nom de l'image.
     if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
 
           exec("mogrify -resize 640x640 ".$vrFichier); // Redimention de l'image avec ImageMagick.
            telecarg_suppr(); // Appel de la fonction telecarg_suppr.
 
     } else { // Si l'image n'a pas été uploader.
 
          echo 'Echec de l\'upload !';
           echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>"); // Redirection ver le formulaire.
     }
}
 
 
?>

**leodi** · 29/09/2008, 14h58

En général il t'indique la ligne. Regarde bien les messages d'erreur.

**koKoTis** · 29/09/2008, 15h14

et ben enfaite c'été même pas la peine de passer par la créeation d'un fonction:

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
<?php
 
 
if(isset($_FILES['photo']))
{ 
     $dossier = 'imgs/';
     $fichier = basename($_FILES['photo']['name']);
     $vrFichier = "imgs/".$fichier;
 
     if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
         header("Content-Type: image/jpeg"); // Indispenssable pour la fonction de téléchargement de l'image redimentionner.
         header("Content-Disposition: attachment; filename=".$vrFichier); // Indispenssable pour la fonction de téléchargement de l'image redimentionner.
 
           exec("mogrify -resize 640x640 ".$vrFichier); // Redimention de l'image avec ImageMagick.
 
            readfile($vrFichier); // Ici on lance le téléchargent de l'image uploader.
            unlink($vrFichier); // Ici on supprime l'image une foi quelle à été télécharger.
            exit();
 
     } else { // Si l'image n'a pas été uploader.
 
          echo 'Echec de l\'upload !';
           echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>"); // Redirection ver le formulaire.
     }
}
 
 
?>

Maitenant ca marche nikel.

Cepandant il reste deux point a éclaircire:

1) cette ligne déclare les image jpg, il y a t-il moyen de déclarer aussi les image gif, png, tif, et autre ?
header("Content-Type: image/jpeg");

2) Comment avant l'upload tester si l'extesnion et jpg ou gif si non ne pas effectuer l'upload ?

**koKoTis** · 29/09/2008, 16h24

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
<?php
 
 
if(isset($_FILES['photo']))
{ 
   $dossier = 'imgs/';
   $fichier = basename($_FILES['photo']['name']);
   $vrFichier = "imgs/".$fichier;
 
   $extension = strrchr($vrFichier,'.'); // On récupére l'extension du fichier uploader
 
   if ($extension == ".jpg" or $extension == ".gif" or $extension == ".tif" or $extension == ".tiff" or $extension == ".png") 
   {
 
       if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
       {
           header("Content-Type: image/jpeg"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/gif"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/png"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/tiff"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Disposition: attachment; filename=".$vrFichier); // Indispenssable pour le téléchargement de l'image redimentionner.
 
             exec("mogrify -resize 640x640 ".$vrFichier); // Redimention de l'image avec ImageMagick.
 
              readfile($vrFichier); // Ici on lance le téléchargent de l'image redimentionnée.
              unlink($vrFichier); // Ici on supprime l'image une foi quelle à été télécharger.
              exit();
 
       } else { // Si l'image n'a pas été uploader.
 
            echo '!!! Impossible d\'envoyer le fichier selectionné';
             echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>"); // Redirection ver le formulaire si l'image n'a pas été uploader.
       }
 
   } else {
      echo "Ce n\'est pas une image valide que vous essayer de redimentioner, ou ce type d\'image ne peut être redimentionner ici ou bien ce n'est même pas un image";
   }
}
 
 
?>

**leodi** · 29/09/2008, 16h26

Envoyé par koKoTis

1) cette ligne déclare les image jpg, il y a t-il moyen de déclarer aussi les image gif, png, tif, et autre ?
header("Content-Type: image/jpeg");

Oui, il faut modifier le type mime correspondant a l'image (pour ca regarde l'extension, et mets le bon type mime :
http://fr.wikipedia.org/wiki/Type_MIME )

Envoyé par koKoTis

2) Comment avant l'upload tester si l'extesnion et jpg ou gif si non ne pas effectuer l'upload ?

Google : http://damienalexandre.fr/Upload-d-image-en-PHP.html

Bon courage,
Paul

**koKoTis** · 29/09/2008, 16h28

ben je vener juste de poster le script finale, ça y est je l'ai terminé, ta pas eu le temp de voir

**leodi** · 29/09/2008, 16h32

Il faudrait ne renvoyer qu'un seul type mime, et non pas tous, comme tu fais actuellement.
if if if

**koKoTis** · 29/09/2008, 16h34

comment faire ?

**leodi** · 29/09/2008, 16h43

Regarde ton code, esseye de bien comprendre comment il fonctionne.
Pour effectuer un upload d'image. c'est indispensable. Meme pour coder en général. en effet si tu ne sais pas ce que fais ton code, une personne mal intentionnée elle le saura

Ce n'est pas bien dure, de plus, tu as la solution dans ton code.

Une aide :

Code :

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

if ($extension == ".jpg" or $extension == ".gif" or $extension == ".tif" or $extension == ".tiff" or $extension == ".png")

Code :

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

1
2
3
4
           header("Content-Type: image/jpeg"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/gif"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/png"); // Indispenssable pour le téléchargement de l'image redimentionner.
           header("Content-Type: image/tiff"

**koKoTis** · 29/09/2008, 16h44

je comprend pas

**leodi** · 29/09/2008, 16h52

Pour forcer le téléchargement d'une image, ou d'un fichier en général, tu dois spécifier son type mime au navigateur.

Là tu lui envois plusieurs type mime différents. ce qui fait que le navigateur qui ouvrira ta page ne saura pas forcément lequel est le bon, et pourra proposer un jpg alors que l'image est un gif.
C'est pour ca que tu dois faire des tests, et ne renvoyer qu'un seul type mime, en fonction des données que tu as (ici c'est l'extension, d'ailleur tu les a même testé dans la ligne que j'ai copié dans mon précédent post).

Code :

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

1
2
3
4
5
6
7
 
SI ext = .gif
- header de gif
SINON ext = .jpg
- header de jpg
SINON ext = .tiff
- header de tiff

**koKoTis** · 29/09/2008, 16h54

oui mai dans ce cas il faudra que je mette les code pour redimensionner télécharger et supprimer 3 fois ?

**leodi** · 29/09/2008, 16h56

Non, c'est juste pour l'affichage des bons headers. Pour l'upload / redim / delete ca reviens au même, quelque soit le type de l'image.

**koKoTis** · 29/09/2008, 17h08

donc voici la code fonctionel:

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
 
// /////////////////////////////////////////////////////////////////////////////////////
// Sujet ici: http://www.developpez.net/forums/d620549/php/langage/fichiers/telechargerment-image-abimee/
// Sujet ici: http://www.developpez.net/forums/d620374/php/scripts/redimentioner-image-imag-magik/
/////////////////////////////////////////////////////////////////////////////////////
 
 
 
if(isset($_FILES['photo']))
{ 
   $dossier = 'imgs/';
   $fichier = basename($_FILES['photo']['name']);
   $vrFichier = "imgs/".$fichier;
 
   $extension = strrchr($vrFichier,'.'); // On récupére l'extension du fichier uploader
 
///////////////////////////////////////////////////////////////////////////////////////
// Pour ajouter des extension c'est ici:
///////////////////////////////////////////////////////////////////////////////////////
 
   if ($extension == ".jpg") { header("Content-Type: image/jpeg"); } // Indispenssable pour le téléchargement de l'image redimentionner.
   if ($extension == ".gif") { header("Content-Type: image/gif"); } // Indispenssable pour le téléchargement de l'image redimentionner.
   if ($extension == ".tif" or $extension == ".tiff") { header("Content-Type: image/tiff"); } // Indispenssable pour le téléchargement de l'image redimentionner.
   if ($extension == ".png") { header("Content-Type: image/png"); } // Indispenssable pour le téléchargement de l'image redimentionner.
 
 
   if ($extension == ".jpg" or $extension == ".gif" or $extension == ".tif" or $extension == ".tiff" or $extension == ".png") 
///////////////////////////////////////////////////////////////////////////////////////
 
   {
 
       if(move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
       {
 
           header("Content-Disposition: attachment; filename=".$vrFichier); // Indispenssable pour le téléchargement de l'image redimentionner.
 
             exec("mogrify -resize 640x640 ".$vrFichier); // Redimention de l'image avec ImageMagick.
 
              readfile($vrFichier); // Ici on lance le téléchargent de l'image redimentionnée.
              unlink($vrFichier); // Ici on supprime l'image une foi quelle à été télécharger.
              exit();
 
       } else { // Si l'image n'a pas été uploader.
 
            echo '!!! Impossible d\'envoyer le fichier selectionné';
             echo ("<meta http-equiv='Refresh' content='2;URL=index.php'>"); // Redirection ver le formulaire si l'image n'a pas été uploader.
       }
 
   } else {
      echo "Ce n\'est pas une image valide que vous essayer de redimentioner, ou ce type d\'image ne peut être redimentionner ici ou bien ce n'est même pas un image";
   }
}
 
 
?>

C'est bon comme ça ?

**leodi** · 29/09/2008, 17h08

Tu pourrais vérifier que l'image en soit bien une, grâce a GD (avec la fonction getimagesize : google). On peut meme aller jusqu'à renommer l'image, pour des raisons de sécurité, mais comme tu la supprimes juste après ca devrait être bon.

Un peu de lecture : http://ha.ckers.org/blog/20070604/pa...h-getimagesize , et http://209.85.135.104/search?q=cache...ient=firefox-a (en cache car erreur 500 en ce moment : /)

Edit : j'ai vu

téléchargerment : image abimée

Langage PHP

Discussions similaires

Partager

Partager