Discussion :

[Sergio_zero]

Bonjour à tous et à toutes,
J'ai un formulaire d'upload d'images qui les redimensionnes et les enregistres sur mon serveur.
Pour les jpg et les png aucun problème mais quand c'est une image webp il y a un fond noir.
Quelqu'un sais pourquoi ?
Voici mon script

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
 
<?php
if($ExtensionPresumee == "png")
$ImageChoisie = imagecreatefrompng($_FILES['photoNews']['tmp_name']);
else if($ExtensionPresumee == "webp")
$ImageChoisie = imagecreatefromwebp($_FILES['photoNews']['tmp_name']);
else if($ExtensionPresumee == ("jpg" || "jpeg"))
$ImageChoisie = imagecreatefromjpeg($_FILES['photoNews']['tmp_name']);
 
$TailleImageChoisie = getimagesize($_FILES['photoNews']['tmp_name']);
$NouvelleLargeur = 800; //Largeur choisie à 100px mais modifiable
$Reduction = ( ($NouvelleLargeur * 100)/$TailleImageChoisie[0] );
$NouvelleHauteur = ( ($TailleImageChoisie[1] * $Reduction)/100 );
$NouvelleImage = imagecreatetruecolor($NouvelleLargeur , $NouvelleHauteur) or die ("Erreur");
if($ExtensionPresumee == "png")
	{
imagealphablending($NouvelleImage,FALSE);
imagesavealpha($NouvelleImage,TRUE);
	}
if($ExtensionPresumee == "webp")
	{
imagepalettetotruecolor($NouvelleImage);
imagealphablending($NouvelleImage, true);
imagesavealpha($NouvelleImage, true);
						}
imagecopyresampled($NouvelleImage, $ImageChoisie, 0,0,0,0, $NouvelleLargeur, $NouvelleHauteur, 
$TailleImageChoisie[0],$TailleImageChoisie[1]);
$photoNews = caractereValideImg($_FILES['photoNews']['name']).'';
							$NomImageChoisie = strtolower($photoNews);
							$NomImageExploitable = ''.$temps.'-'.$NomImageChoisie.'';
 
							if($ExtensionPresumee == "png")
								{
									imagepng($NouvelleImage , ''.$adresse_gr.''.$NomImageExploitable, 0);// on enregistre
									imagepng($NouvelleImage1 , ''.$adresse_moy.''.$NomImageExploitable, 0);// on enregistre
									imagepng($NouvelleImage2 , ''.$adresse_min.''.$NomImageExploitable, 0);// on enregistre
								}
							else if($ExtensionPresumee == ("jpg" || "jpeg"))
								{
									imagejpeg($NouvelleImage , ''.$adresse_gr.''.$NomImageExploitable, 100);
									imagejpeg($NouvelleImage1 , ''.$adresse_moy.''.$NomImageExploitable, 100);
									imagejpeg($NouvelleImage2 , ''.$adresse_min.''.$NomImageExploitable, 100);
								}
							else if($ExtensionPresumee == ("webp"))
								{
									imagewebp($NouvelleImage , ''.$adresse_gr.''.$NomImageExploitable, 100);
									imagewebp($NouvelleImage1 , ''.$adresse_moy.''.$NomImageExploitable, 100);
									imagewebp($NouvelleImage2 , ''.$adresse_min.''.$NomImageExploitable, 100);
								}

Je pense que c'est une connerie de filtre mais je trouve pas.
Merci à tout ceux qui pourrons m'aider.

[laurentSc]

Bonjour,

il vient d'où ton code d'upload ?

[Sergio_zero]

Pourquoi ?
Je n'ai pas mis le code complet mais juste la partie qui traite l'image.

[laurentSc]

J'ai pensé que tu avais récupéré ce code quelque part. Si c'est pas le cas, oublie.

[Sergio_zero]

Je m'inspire bien souvent d'un tuto mais se script est vieux, je l'ai fait évoluer et bien modifié depuis.
Pour mon image wepb tu n'as pas une idée ?

[houd.ini]

Bonjour,

"jpg" || "jpeg" équivaut à true du coup tes imagewebp ne seront pas atteints.
Il faudrait modifier les 2 lignes concernées comme ceci (par exemple) :

Code :

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

1
2
3
if ($ExtensionPresumee === "jpg" || $ExtensionPresumee === "jpeg") {
  // ...
}

[laurentSc]

"jpg" || "jpeg" équivaut à true

C'est exact. Même si pas suivi la logique du raisonnement, c'est sûr qu'il faut au moins faire la correction proposée...

[houd.ini]

Désolé pour cette explication sommaire.
Un peu de code sera peut-être plus parlant.

Code :

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

var_dump("webp" == ("jpg" || "jpeg")); // true

[laurentSc]

Je viens de faire un test qui du coup semble réfuter la proposition :

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
<?php
 
function which_image($ExtensionPresumee){
if($ExtensionPresumee == "png")
    $ImageChoisie = "png";
else if($ExtensionPresumee == "webp")
    $ImageChoisie = "webp";
else if($ExtensionPresumee == ("jpg" || "jpeg"))
    $ImageChoisie = "jpg";
return($ExtensionPresumee);
}
 
$ExtensionPresumee="jpg";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="png";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="webp";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";

affiche :

ImageChoisie=jpg
ImageChoisie=png
ImageChoisie=webp

[Séb.]

Je viens de faire un test qui du coup semble réfuter la proposition

C'est parce que ton test sur le jpg/jpeg est le dernier, et qu'il est toujours TRUE (enfin tant que $ExtensionPresumee est évalué TRUE). C'est une sorte de default

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function which_image($ExtensionPresumee){
    if($ExtensionPresumee == "png")
        $ImageChoisie = "png";
    else if($ExtensionPresumee == "webp")
        $ImageChoisie = "webp";
    else if($ExtensionPresumee == ("jpg" || "jpeg"))
        $ImageChoisie = "jpg";
    return($ExtensionPresumee);
}
 
$ExtensionPresumee="je n'existe pas";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>"; // "je n'existe pas"

[laurentSc]

En effet, j'avais fait une erreur, que je viens de corriger :

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
<?php
 
function which_image($ExtensionPresumee){
if($ExtensionPresumee == "png")
    $ImageChoisie = "png";
else if($ExtensionPresumee == "webp")
    $ImageChoisie = "webp";
else if($ExtensionPresumee == ("jpg" || "jpeg"))
    $ImageChoisie = "jpg";
return($ImageChoisie);
}
 
$ExtensionPresumee="jpg";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="png";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="webp";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="je n'existe pas";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";

affiche

ImageChoisie=jpg
ImageChoisie=png
ImageChoisie=webp
ImageChoisie=jpg

[Séb.]

C'est toujours pas bon, on ne veut pas qu'un fichier "fichier.je n'existe pas" soit reconnu "jpg"

La syntaxe

Code :

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

else if($ExtensionPresumee == ("jpg" || "jpeg"))

n'a, pour ainsi dire, aucun sens en PHP.

Ce qui s'en rapprocherait le plus visuellement est

Code :

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

elseif (in_array($ExtensionPresumee, ['jpg', 'jpeg']))

Mais pour seulement 2 extensions à comparer mieux vaut utiliser la solution de houd.ini

[laurentSc]

J'ai été absent un long moment. Ca devrait aller :

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
 
function which_image($ExtensionPresumee){
    $ImageChoisie = "not found";
if($ExtensionPresumee == "png")
    $ImageChoisie = "png";
else if($ExtensionPresumee == "webp")
    $ImageChoisie = "webp";
else if ($ExtensionPresumee === "jpg" || $ExtensionPresumee === "jpeg")
    $ImageChoisie = "jpg";
return($ImageChoisie);
}
 
$ExtensionPresumee="jpg";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="png";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="webp";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";
 
$ExtensionPresumee="je n'existe pas";
echo "ImageChoisie=".which_image($ExtensionPresumee)."<br/>";

affiche :

ImageChoisie=jpg
ImageChoisie=png
ImageChoisie=webp
ImageChoisie=not found

[Sergio_zero]

Bonsoir tout le monde et merci de m'aider.
J'ai modifié mon code comme conseillé par houd.ini
C''est ici que ça déconnait.

Code :

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

imagealphablending($NouvelleImage, true);

il faut le mettre à false pour que le transparent soit géré

Code :

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

imagealphablending($NouvelleImage, FALSE);

[ABCIWEB]

Salut,

Au passage, faire attention à utiliser php 7.1 minimum car le support WebP pour "getimagesize" n'a été ajouté qu'à partir de cette version.

Cela dit tu pourrais aussi te passer de "getimagesize" car à partir du moment où tu as créé une ressource image avec $ImageChoisie = imagecreatefrom..., tu peux avoir les dimensions en faisant $im_width = imagesx($ImageChoisie); et $im_height = imagesy($ImageChoisie);. Ce serait aussi légèrement plus optimisé car si une ressource image est créée mieux vaut utiliser les fonctions spécifiques à cette ressource plutôt que "getimagesize" qui est plus généraliste et retourne plus d'informations.

Enfin tu mets une qualité maximale pour les redimensionnements, mais si ce sont des images pour afficher sur le web c'est beaucoup (trop). Généralement on utilise plutôt des valeurs autour de 80, 90 étant la valeur maximale après quoi le poids des fichiers explose sans que l'on observe de différence visuelle significative.