Valider 80d940a1 rédigé par marcimat's avatar marcimat
Parcourir les fichiers

Code mort : suppression du dernier paramètre de la fonction...

Code mort : suppression du dernier paramètre de la fonction _image_creer_vignette() ($test_cache_only) et de 2 tests
dans la fonction, qui ne servent plus a priori depuis r8673 (https://github.com/spip/SPIP/commit/9fe35bd70d028)

$vignette valait tout le temps ''. 
Réindentation en conséquence.

Je ne suis pas certain que le paramètre $force non plus soit très utile.
parent bc37ff75
Chargement en cours
Chargement en cours
Chargement en cours
Chargement en cours
+144 −153
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -831,11 +831,10 @@ function _image_ecrire_tag($valeurs, $surcharge = array()){
 *     Librairie graphique à utiliser (gd1, gd2, netpbm, convert, imagick).
 *     AUTO utilise la librairie sélectionnée dans la configuration.
 * @param bool $force
 * @param bool $test_cache_only
 * @return array|null
 *     Description de l'image, sinon null.
**/
function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO', $force = false, $test_cache_only = false) {
function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO', $force = false) {
	// ordre de preference des formats graphiques pour creer les vignettes
	// le premier format disponible, selon la methode demandee, est utilise
	$image = $valeurs['fichier'];
@@ -858,20 +857,12 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
	if (!$force AND !$img) return;
	$destination = "$destdir/$destfile";

	// chercher un cache
	$vignette = '';
	if ($test_cache_only AND !$vignette) return;

	// utiliser le cache ?
	if (!$test_cache_only)
	if ($force OR !$vignette OR (@filemtime($vignette) < @filemtime($image))) {

		$creation = true;
	// calculer la taille
	if (($srcWidth=$valeurs['largeur']) && ($srcHeight=$valeurs['hauteur'])) {
			if (!($destWidth=$valeurs['largeur_dest']) || !($destHeight=$valeurs['hauteur_dest']))
		if (!($destWidth=$valeurs['largeur_dest']) || !($destHeight=$valeurs['hauteur_dest'])) {
			list ($destWidth,$destHeight) = _image_ratio($valeurs['largeur'], $valeurs['hauteur'], $maxWidth, $maxHeight);
		}
	}
	elseif ($process == 'convert' OR $process == 'imagick') {
		$destWidth = $maxWidth;
		$destHeight = $maxHeight;
@@ -880,13 +871,12 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
		return;
	}

		// Si l'image est de la taille demandee (ou plus petite), simplement
		// la retourner
		if ($srcWidth
		AND $srcWidth <= $maxWidth AND $srcHeight <= $maxHeight) {
	// Si l'image est de la taille demandee (ou plus petite), simplement la retourner
	if ($srcWidth AND $srcWidth <= $maxWidth AND $srcHeight <= $maxHeight) {
		$vignette = $destination.'.'.$format;
		@copy($image, $vignette);
	}

	// imagemagick en ligne de commande
	elseif ($process == 'convert') {
		if (!defined('_CONVERT_COMMAND')) define('_CONVERT_COMMAND', 'convert'); // Securite : mes_options.php peut preciser le chemin absolu
@@ -908,9 +898,9 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
			return;	// echec commande
		}
	}
		else

	// php5 imagemagick
		if ($process == 'imagick') {
	elseif ($process == 'imagick') {
		$vignette = "$destination.".$format_sortie;

		if (!class_exists('Imagick')) {
@@ -927,9 +917,9 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
			return;
		}
	}
		else

	// netpbm
		if ($process == "netpbm") {
	elseif ($process == "netpbm") {
		if (!defined('_PNMSCALE_COMMAND')) define('_PNMSCALE_COMMAND', 'pnmscale'); // Securite : mes_options.php peut preciser le chemin absolu
		if (_PNMSCALE_COMMAND == '') return;
		$vignette = $destination.".".$format_sortie;
@@ -964,6 +954,7 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
			}
		}
	}

	// gd ou gd2
	elseif ($process == 'gd1' OR $process == 'gd2') {
		if (!function_exists('gd_info')) {
@@ -1027,7 +1018,7 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
			ImageDestroy($srcImage);
		ImageDestroy($destImage);
	}
	}

	$size = @getimagesize($vignette);
	// Gaffe: en safe mode, pas d'acces a la vignette,
	// donc risque de balancer "width='0'", ce qui masque l'image sous MSIE