Discussion :

[julien.63]

salut,
je suis en train d'écrire un peu script pour redimensionner des images placées dans un dossier. J'ai un message d'erreur :

Code :

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

1
2
Warning: imagejpeg() [function.imagejpeg]: Unable to open 'photo' 
for writing in /var/www/test/resizeImg/resize_function.php on line 52

voici ma 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
function resize($file){
	$path = $this->f.'/'.$file;
	$imgSrc = imagecreatefromjpeg($path);
	$newImg = imagecreatetruecolor($this->w,$this->h);
	imagecopyresampled($newImg,$imgSrc,0,0,0,0,$this->w,$this->h,$width,$height);	
	if (imagejpeg($imgSrc,$this->newFolderName)){ //**LIGNE 52****
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		return true;
	}
	else {
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		$this->result = $file.' problem <br>';
		return false;
	}
}

pourtant mon dossier est bien accessible en écriture. J'ai même fait un chmod 777 pour être sûr !!

[Yoteco]

essaye de faire ça:

Code :

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

1
2
3
4
5
6
 
// Remplace ceci
imagecopyresampled($newImg,$imgSrc,0,0,0,0,$this->w,$this->h,$width,$height);
 
// Par ceci
imagecopyresized($newImg, $imgSrc, 0, 0, 0, 0, $this->w, $this->h, $width, $height);

Mais je ne comprend pas d'où viennent les varaible $width et $height moi je ferais ceci ==> imagesx($imgSrc) et imagesy($imgSrc)

[julien.63]

salut,
merci pour ta réponse. Malheureusement ça ne fonctionne pas.

En fait, si j'affiche l'image dans le navigateur au lieu de l'enregistrer, c'est ok.

Le souci semble donc provenir de la fonction imagejpeg(); quand je mets le second argument pour sauver le fichier.

je viens de re-re-re-re-re-vérifier que mon dossier est bien autorisé en écriture !

Sinon concernant les valeurs $width et $height, j'avais viré quelques lignes de la fonction pour n'afficher que le principal. Je mets tout ci-dessous :

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
function resize($file){
	$path = $this->f.'/'.$file;
	$imgSrc = imagecreatefromjpeg($path);
	list($width,$height) = getimagesize($path);
	$this->setRatio($width,$height);
	$newImg = imagecreatetruecolor($this->w,$this->h);
	//imagecopyresampled($newImg,$imgSrc,0,0,0,0,$this->w,$this->h,$width,$height);	
	imagecopyresized($newImg, $imgSrc, 0, 0, 0, 0, $this->w, $this->h, $width, $height);
	if (imagejpeg($newImg,$this->newFolderName)){  //** ne fonctionne pas ****
	//if (imagejpeg($newImg)){ 	//** fonctionne bien ****
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		return true;
	}
	else {
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		return false;
	}
}

[Yoteco]

ton $this->newFolderName il te retourne quoi ?

[julien.63]

je viens de trouver, enfin en regardant le code mis sur php.net à la fonction imagejpeg.
Yoteco était sur la bonne piste également. C'est tout bête mais j'indiquais le chemin du dossier sans indiquer le nom du fichier derrière.
ce qui donne :

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
function resize($file){
	$srcPath = $this->f.'/'.$file;
	$destPath = $this->newFolderName.'/'.$file;
	$imgSrc = imagecreatefromjpeg($srcPath);
	list($width,$height) = getimagesize($srcPath);
	$this->setRatio($width,$height);
	$newImg = imagecreatetruecolor($this->w,$this->h);
	imagecopyresized($newImg, $imgSrc, 0, 0, 0, 0, $this->w, $this->h, $width, $height);
	if (imagejpeg($newImg,$destPath)){
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		return true;
	}
	else {
		imagedestroy($imgSrc);
		imagedestroy($newImg);
		return false;
	}
}

merci pour votre aide

[Yoteco]