spip / SPIP

ecrire/inc/prepare_recherche.php

Indentation +139 added lines, -139 removed lines

@@ -16,12 +16,12 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/rechercher');
 if (!defined('_DELAI_CACHE_resultats')) {
-	define('_DELAI_CACHE_resultats', 600);
+    define('_DELAI_CACHE_resultats', 600);
 }
 
 /**
@@ -48,126 +48,126 @@ 
  * @return array
  */
 function inc_prepare_recherche_dist(
-	$recherche,
-	$table = 'articles',
-	$cond = false,
-	$serveur = '',
-	$modificateurs = [],
-	$primary = ''
+    $recherche,
+    $table = 'articles',
+    $cond = false,
+    $serveur = '',
+    $modificateurs = [],
+    $primary = ''
 ) {
-	$where = null;
-	$rows = null;
-	static $cache = [];
-	$delai_fraicheur = min(
-		\_DELAI_CACHE_resultats,
-		time() - ($GLOBALS['meta']['derniere_modif'] ?? 0)
-	);
-
-	// si recherche n'est pas dans le contexte, on va prendre en globals
-	// ca permet de faire des inclure simple.
-	if (!isset($recherche) && isset($GLOBALS['recherche'])) {
-		$recherche = $GLOBALS['recherche'];
-	}
-
-	// traiter le cas {recherche?}
-	if ($cond && !strlen($recherche)) {
-		return [
-			'0 as points' /* as points */, /* where */
-			''
-		];
-	}
-
-
-	$rechercher = false;
-
-	$where_resultat_recent = sql_date_proche('maj', (0 - ($delai_fraicheur + 100)), ' SECOND');
-	if (!isset($cache[$serveur][$table][$recherche])) {
-		$hash_serv = ($serveur ? substr(md5($serveur), 0, 16) : '');
-		$hash = substr(md5($recherche . $table), 0, 16);
-		$where = "(resultats.recherche='$hash' AND resultats.table_objet=" . sql_quote($table) . " AND resultats.serveur='$hash_serv')";
-		$row = sql_fetsel(
-			'recherche',
-			'spip_resultats AS resultats',
-			$where . " AND $where_resultat_recent",
-			'',
-			'',
-			'0,1'
-		);
-		if (!$row || (defined('_VAR_MODE') && _VAR_MODE == 'recalcul')) {
-			$rechercher = true;
-		}
-	}
-
-	// si on n'a pas encore traite les donnees dans une boucle precedente
-	if ($rechercher) {
-		//$tables = liste_des_champs();
-		$x = objet_type($table);
-		$points = recherche_en_base(
-			$recherche,
-			$x,
-			[
-				'score' => true,
-				'toutvoir' => true,
-				'jointures' => true
-			],
-			$serveur
-		);
-		// pas de résultat, pas de point
-		$points = $points[$x] ?? [];
-
-		// permettre aux plugins de modifier le resultat
-		$points = pipeline('prepare_recherche', [
-			'args' => [
-				'type' => $x,
-				'recherche' => $recherche,
-				'serveur' => $serveur,
-				'modificateurs' => $modificateurs
-			],
-			'data' => $points
-		]);
-
-		// supprimer les anciens resultats de cette recherche
-		// et les resultats trop vieux avec une marge
-		// pas de AS resultats dans un delete (mysql)
-		$whered = str_replace(
-			['resultats.recherche', 'resultats.table_objet', 'resultats.serveur'],
-			['recherche', 'table_objet', 'serveur'],
-			$where
-		);
-
-		sql_delete(
-			'spip_resultats',
-			"NOT($where_resultat_recent) OR ($whered)"
-		);
-
-		// inserer les resultats dans la table de cache des resultats
-		if (is_countable($points) ? count($points) : 0) {
-			$tab_couples = [];
-			foreach ($points as $id => $p) {
-				$tab_couples[] = [
-					'recherche' => $hash,
-					'id' => $id,
-					'points' => $p['score'],
-					'table_objet' => $table,
-					'serveur' => $hash_serv,
-				];
-			}
-			sql_insertq_multi('spip_resultats', $tab_couples, []);
-		}
-	}
-
-	if (!isset($cache[$serveur][$table][$recherche])) {
-		if (!$serveur) {
-			$cache[$serveur][$table][$recherche] = ['resultats.points AS points', $where];
-		} else {
-			if (sql_countsel('spip_resultats as resultats', $where)) {
-				$rows = sql_allfetsel('resultats.id,resultats.points', 'spip_resultats as resultats', $where);
-			}
-			$cache[$serveur][$table][$recherche] = generer_select_where_explicites($table, $primary, $rows, $serveur);
-		}
-	}
-
-	return $cache[$serveur][$table][$recherche];
+    $where = null;
+    $rows = null;
+    static $cache = [];
+    $delai_fraicheur = min(
+        \_DELAI_CACHE_resultats,
+        time() - ($GLOBALS['meta']['derniere_modif'] ?? 0)
+    );
+
+    // si recherche n'est pas dans le contexte, on va prendre en globals
+    // ca permet de faire des inclure simple.
+    if (!isset($recherche) && isset($GLOBALS['recherche'])) {
+        $recherche = $GLOBALS['recherche'];
+    }
+
+    // traiter le cas {recherche?}
+    if ($cond && !strlen($recherche)) {
+        return [
+            '0 as points' /* as points */, /* where */
+            ''
+        ];
+    }
+
+
+    $rechercher = false;
+
+    $where_resultat_recent = sql_date_proche('maj', (0 - ($delai_fraicheur + 100)), ' SECOND');
+    if (!isset($cache[$serveur][$table][$recherche])) {
+        $hash_serv = ($serveur ? substr(md5($serveur), 0, 16) : '');
+        $hash = substr(md5($recherche . $table), 0, 16);
+        $where = "(resultats.recherche='$hash' AND resultats.table_objet=" . sql_quote($table) . " AND resultats.serveur='$hash_serv')";
+        $row = sql_fetsel(
+            'recherche',
+            'spip_resultats AS resultats',
+            $where . " AND $where_resultat_recent",
+            '',
+            '',
+            '0,1'
+        );
+        if (!$row || (defined('_VAR_MODE') && _VAR_MODE == 'recalcul')) {
+            $rechercher = true;
+        }
+    }
+
+    // si on n'a pas encore traite les donnees dans une boucle precedente
+    if ($rechercher) {
+        //$tables = liste_des_champs();
+        $x = objet_type($table);
+        $points = recherche_en_base(
+            $recherche,
+            $x,
+            [
+                'score' => true,
+                'toutvoir' => true,
+                'jointures' => true
+            ],
+            $serveur
+        );
+        // pas de résultat, pas de point
+        $points = $points[$x] ?? [];
+
+        // permettre aux plugins de modifier le resultat
+        $points = pipeline('prepare_recherche', [
+            'args' => [
+                'type' => $x,
+                'recherche' => $recherche,
+                'serveur' => $serveur,
+                'modificateurs' => $modificateurs
+            ],
+            'data' => $points
+        ]);
+
+        // supprimer les anciens resultats de cette recherche
+        // et les resultats trop vieux avec une marge
+        // pas de AS resultats dans un delete (mysql)
+        $whered = str_replace(
+            ['resultats.recherche', 'resultats.table_objet', 'resultats.serveur'],
+            ['recherche', 'table_objet', 'serveur'],
+            $where
+        );
+
+        sql_delete(
+            'spip_resultats',
+            "NOT($where_resultat_recent) OR ($whered)"
+        );
+
+        // inserer les resultats dans la table de cache des resultats
+        if (is_countable($points) ? count($points) : 0) {
+            $tab_couples = [];
+            foreach ($points as $id => $p) {
+                $tab_couples[] = [
+                    'recherche' => $hash,
+                    'id' => $id,
+                    'points' => $p['score'],
+                    'table_objet' => $table,
+                    'serveur' => $hash_serv,
+                ];
+            }
+            sql_insertq_multi('spip_resultats', $tab_couples, []);
+        }
+    }
+
+    if (!isset($cache[$serveur][$table][$recherche])) {
+        if (!$serveur) {
+            $cache[$serveur][$table][$recherche] = ['resultats.points AS points', $where];
+        } else {
+            if (sql_countsel('spip_resultats as resultats', $where)) {
+                $rows = sql_allfetsel('resultats.id,resultats.points', 'spip_resultats as resultats', $where);
+            }
+            $cache[$serveur][$table][$recherche] = generer_select_where_explicites($table, $primary, $rows, $serveur);
+        }
+    }
+
+    return $cache[$serveur][$table][$recherche];
 }
 
 
@@ -183,22 +183,22 @@ 
  * @return array
  */
 function generer_select_where_explicites($table, $primary, $rows, $serveur) {
-	# calculer le {id_article IN()} et le {... as points}
-	if ($rows === []) {
-		return ["''", '0=1'];
-	} else {
-		$listes_ids = [];
-		$select = '0';
-		foreach ($rows as $r) {
-			$listes_ids[$r['points']][] = $r['id'];
-		}
-
-		foreach ($listes_ids as $p => $ids) {
-			$select .= "+$p*(" .
-				sql_in("$table.$primary", $ids, '', $serveur)
-				. ') ';
-		}
-
-		return ["$select AS points ", sql_in("$table.$primary", array_map('reset', $rows), '', $serveur)];
-	}
+    # calculer le {id_article IN()} et le {... as points}
+    if ($rows === []) {
+        return ["''", '0=1'];
+    } else {
+        $listes_ids = [];
+        $select = '0';
+        foreach ($rows as $r) {
+            $listes_ids[$r['points']][] = $r['id'];
+        }
+
+        foreach ($listes_ids as $p => $ids) {
+            $select .= "+$p*(" .
+                sql_in("$table.$primary", $ids, '', $serveur)
+                . ') ';
+        }
+
+        return ["$select AS points ", sql_in("$table.$primary", array_map('reset', $rows), '', $serveur)];
+    }
 }

ecrire/inc/preselectionner_parent_nouvel_objet.php

Indentation +34 added lines, -34 removed lines

@@ -10,10 +10,10 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 if (!defined('_AUTO_SELECTION_RUBRIQUE')) {
-	define('_AUTO_SELECTION_RUBRIQUE', false);
+    define('_AUTO_SELECTION_RUBRIQUE', false);
 }
 
 
@@ -27,39 +27,39 @@ 
  * @return string
  */
 function inc_preselectionner_parent_nouvel_objet_dist($objet, $row) {
-	if (!_AUTO_SELECTION_RUBRIQUE) {
-		return '';
-	}
+    if (!_AUTO_SELECTION_RUBRIQUE) {
+        return '';
+    }
 
-	if (!isset($row['id_rubrique'])) {
-		return '';
-	}
+    if (!isset($row['id_rubrique'])) {
+        return '';
+    }
 
-	$id_rubrique = '';
-	if ($GLOBALS['connect_id_rubrique']) {
-		// si admin restreint : sa rubrique
-		$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
-	} elseif (
-		is_int(_AUTO_SELECTION_RUBRIQUE)
-		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
-	) {
-		$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
-	} else {
-		// sinon la derniere rubrique cree
-		$row_rub = sql_fetsel('id_rubrique', 'spip_rubriques', '', '', 'id_rubrique DESC', '0,1');
-		$id_rubrique = $row_rub['id_rubrique'];
-	}
-	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
-		$id_rubrique = '';
-		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
-		$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
-		while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
-				$id_rubrique = $row_rub['id_rubrique'];
-			}
-		}
-	}
+    $id_rubrique = '';
+    if ($GLOBALS['connect_id_rubrique']) {
+        // si admin restreint : sa rubrique
+        $id_rubrique = $GLOBALS['connect_id_rubrique'][0];
+    } elseif (
+        is_int(_AUTO_SELECTION_RUBRIQUE)
+        && sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
+    ) {
+        $id_rubrique = _AUTO_SELECTION_RUBRIQUE;
+    } else {
+        // sinon la derniere rubrique cree
+        $row_rub = sql_fetsel('id_rubrique', 'spip_rubriques', '', '', 'id_rubrique DESC', '0,1');
+        $id_rubrique = $row_rub['id_rubrique'];
+    }
+    // si le choix ne convient pas, on cherche dans un secteur
+    if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+        $id_rubrique = '';
+        // manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
+        $res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
+        while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
+            if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+                $id_rubrique = $row_rub['id_rubrique'];
+            }
+        }
+    }
 
-	return $id_rubrique;
+    return $id_rubrique;
 }

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 		$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
 	} elseif (
 		is_int(_AUTO_SELECTION_RUBRIQUE)
-		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
+		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique='.(int) _AUTO_SELECTION_RUBRIQUE)
 	) {
 		$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
 	} else {
@@ -50,12 +50,12 @@ 
 		$id_rubrique = $row_rub['id_rubrique'];
 	}
 	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+	if (!autoriser('creer'.$objet.'dans', 'rubrique', $id_rubrique)) {
 		$id_rubrique = '';
 		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
 		$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
 		while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+			if (autoriser('creer'.$objet.'dans', 'rubrique', $row_rub['id_rubrique'])) {
 				$id_rubrique = $row_rub['id_rubrique'];
 			}
 		}

ecrire/inc/nfslock.php

Indentation +88 added lines, -88 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/acces');
@@ -101,93 +101,93 @@ 
  * @return int|bool Timestamp du verrou, false si erreur
  */
 function spip_nfslock($fichier, $max_age = 0) {
-	$tries = 0;
+    $tries = 0;
 
-	if (!$max_age) {
-		$max_age = _DEFAULT_LOCKTIME;
-	}
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+    if (!$max_age) {
+        $max_age = _DEFAULT_LOCKTIME;
+    }
+    $lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
 
 
-	/*
+    /*
 	 * 1. create a tmp file with a psuedo random file name. we also make
 	 *    tpath which is a buffer to store the full pathname of the tmp file.
 	 */
 
-	$id = creer_uniqid();
-	$tpath = _DIR_TMP . "slock.$id";
-	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
-	if (!$tmpfd) {  /* open failed */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
+    $id = creer_uniqid();
+    $tpath = _DIR_TMP . "slock.$id";
+    $tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
+    if (!$tmpfd) {  /* open failed */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
 
-		return false; //NFSL_SYSF
-	}
+        return false; //NFSL_SYSF
+    }
 
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file.
 	 *    then start looping trying to lock it
 	 */
 
-	while ($tries < 10) {
-		/*
+    while ($tries < 10) {
+        /*
 		 * 3. link tmp file to lock file.  if it goes, we win and we clean
 		 *    up and return the st_ctime of the lock file.
 		 */
 
-		if (link($tpath, $lock_file) == 1) {
-			spip_unlink($tpath); /* got it! */
-			@fclose($tmpfd);
-			if (($our_tmp = lstat($lock_file)) == false) {  /* stat failed... shouldn't happen */
-				spip_unlink($lock_file);
+        if (link($tpath, $lock_file) == 1) {
+            spip_unlink($tpath); /* got it! */
+            @fclose($tmpfd);
+            if (($our_tmp = lstat($lock_file)) == false) {  /* stat failed... shouldn't happen */
+                spip_unlink($lock_file);
 
-				return false; // (NFSL_SYSF);
-			}
+                return false; // (NFSL_SYSF);
+            }
 
-			return ($our_tmp['ctime']);
-		}
+            return ($our_tmp['ctime']);
+        }
 
-		/*
+        /*
 		 * 4. the lock failed.  check for a stale lock file, being mindful
 		 *    of NFS and the fact the time is set from the NFS server.  we
 		 *    do a write on the tmp file to update its time to the server's
 		 *    idea of "now."
 		 */
 
-		$old_stat = lstat($lock_file);
-		if (@fwrite($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
-			break;
-		} /* something bogus is going on */
+        $old_stat = lstat($lock_file);
+        if (@fwrite($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
+            break;
+        } /* something bogus is going on */
 
 
-		if ($old_stat != false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
-			spip_unlink($lock_file); /* break the stale lock */
-			$tries++;
-			/* It is CRITICAL that we sleep after breaking
+        if ($old_stat != false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
+            spip_unlink($lock_file); /* break the stale lock */
+            $tries++;
+            /* It is CRITICAL that we sleep after breaking
 			 * the lock. Otherwise, we could race with
 			 * another process and unlink it's newly-
 			 * created file.
 			 */
-			sleep(1 + random_int(0, 4));
-			continue;
-		}
+            sleep(1 + random_int(0, 4));
+            continue;
+        }
 
-		/*
+        /*
 		 * 5. try again
 		 */
 
-		$tries++;
-		sleep(1 + random_int(0, 4));
-	}
+        $tries++;
+        sleep(1 + random_int(0, 4));
+    }
 
-	/*
+    /*
 	 * 6. give up, failure.
 	 */
 
-	spip_unlink($tpath);
-	@fclose($tmpfd);
+    spip_unlink($tpath);
+    @fclose($tmpfd);
 
-	return false; //(NFSL_LOCKED);
+    return false; //(NFSL_LOCKED);
 }
 
 /**
@@ -231,71 +231,71 @@ 
  * return bool true si déverrouillé, false sinon
  */
 function spip_nfsunlock($fichier, $birth, $max_age = 0, $test = false) {
-	$id = creer_uniqid();
-	if (!$max_age) {
-		$max_age = _DEFAULT_LOCKTIME;
-	}
+    $id = creer_uniqid();
+    if (!$max_age) {
+        $max_age = _DEFAULT_LOCKTIME;
+    }
 
-	/*
+    /*
 	 * 1. Build a temp file and stat that to get an idea of what the server
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP . "stime.$id";
-	$tmpfd = @fopen($tpath, 'w');
-	if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
-		/* The open failed, or we can't write the file, or we can't stat it */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
+    $tpath = _DIR_TMP . "stime.$id";
+    $tmpfd = @fopen($tpath, 'w');
+    if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
+        /* The open failed, or we can't write the file, or we can't stat it */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
 
-		return false; //(NFSL_SYSF);
-	}
+        return false; //(NFSL_SYSF);
+    }
 
-	@fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
-	spip_unlink($tpath);
+    @fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
+    spip_unlink($tpath);
 
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+    $lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
 
-	/*
+    /*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
 	 *    lock has expired, sleep the usual random interval before returning.
 	 *    If we didn't sleep, there could be a race if the caller immediately
 	 *    tries to relock the file.
 	 */
 
-	if (
-		($old_stat = @lstat($lock_file))  /* stat succeeds so file is there */
-		&& ($old_stat['ctime'] == $birth)
-	) {  /* hasn't been modified since birth */
-		if (!$test) {
-			spip_unlink($lock_file);
-		}      /* so the lock is ours to remove */
-		if ($our_tmp['ctime'] >= $birth + $max_age) {  /* the lock has expired */
-			if (!$test) {
-				return false;
-			} //(NFSL_LOST);
-			sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
-		}
-
-		return true;//(NFSL_OK);			/* success */
-	}
-
-	/*
+    if (
+        ($old_stat = @lstat($lock_file))  /* stat succeeds so file is there */
+        && ($old_stat['ctime'] == $birth)
+    ) {  /* hasn't been modified since birth */
+        if (!$test) {
+            spip_unlink($lock_file);
+        }      /* so the lock is ours to remove */
+        if ($our_tmp['ctime'] >= $birth + $max_age) {  /* the lock has expired */
+            if (!$test) {
+                return false;
+            } //(NFSL_LOST);
+            sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
+        }
+
+        return true;//(NFSL_OK);			/* success */
+    }
+
+    /*
 	 * 4. Either ctime has been modified, or the entire lock file is missing.
 	 *    If the lock should still be ours, based on the ctime of the temp
 	 *    file, return with NFSL_STOLEN. If not, then our lock is expired and
 	 *    someone else has grabbed the file, so return NFSL_LOST.
 	 */
 
-	if ($our_tmp['ctime'] < $birth + $max_age) { /* lock was stolen */
-		return false;
-	} //(NFSL_STOLEN);
+    if ($our_tmp['ctime'] < $birth + $max_age) { /* lock was stolen */
+        return false;
+    } //(NFSL_STOLEN);
 
-	return false; //(NFSL_LOST);	/* The lock must have expired first. */
+    return false; //(NFSL_LOST);	/* The lock must have expired first. */
 }
 
 
@@ -319,5 +319,5 @@ 
  * return bool true si déverrouillé, false sinon
  */
 function spip_nfslock_test($fichier, $birth, $max_age = 0) {
-	return spip_nfsunlock($fichier, $birth, $max_age, true);
+    return spip_nfsunlock($fichier, $birth, $max_age, true);
 }

Spacing +7 added lines, -7 removed lines

@@ -106,7 +106,7 @@ 
 	if (!$max_age) {
 		$max_age = _DEFAULT_LOCKTIME;
 	}
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 
 	/*
@@ -115,7 +115,7 @@ 
 	 */
 
 	$id = creer_uniqid();
-	$tpath = _DIR_TMP . "slock.$id";
+	$tpath = _DIR_TMP."slock.$id";
 	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
 	if (!$tmpfd) {  /* open failed */
 		@fclose($tmpfd);
@@ -241,7 +241,7 @@ 
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP . "stime.$id";
+	$tpath = _DIR_TMP."stime.$id";
 	$tmpfd = @fopen($tpath, 'w');
 	if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
 		/* The open failed, or we can't write the file, or we can't stat it */
@@ -251,14 +251,14 @@ 
 		return false; //(NFSL_SYSF);
 	}
 
-	@fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
+	@fclose($tmpfd); /* We don't need this once we have our_tmp.st_ctime. */
 	spip_unlink($tpath);
 
 	/*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 	/*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
@@ -278,10 +278,10 @@ 
 			if (!$test) {
 				return false;
 			} //(NFSL_LOST);
-			sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
+			sleep(1 + (random_int(0, 4))); /* so sleep a bit */
 		}
 
-		return true;//(NFSL_OK);			/* success */
+		return true; //(NFSL_OK);			/* success */
 	}
 
 	/*

ecrire/inc/filtres_images_mini.php

Indentation +313 added lines, -313 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/filtres_images_lib_mini'); // par precaution
 
@@ -30,161 +30,161 @@ 
  *    Le code hexadécimal de la couleur (sans le #) ou le code couleur textuel si non trouvé
  */
 function couleur_html_to_hex($couleur) {
-	$couleurs_html = [
-		'aliceblue' => 'F0F8FF',
-		'antiquewhite' => 'FAEBD7',
-		'aqua' => '00FFFF',
-		'aquamarine' => '7FFFD4',
-		'azure' => 'F0FFFF',
-		'beige' => 'F5F5DC',
-		'bisque' => 'FFE4C4',
-		'black' => '000000',
-		'blanchedalmond' => 'FFEBCD',
-		'blue' => '0000FF',
-		'blueviolet' => '8A2BE2',
-		'brown' => 'A52A2A',
-		'burlywood' => 'DEB887',
-		'cadetblue' => '5F9EA0',
-		'chartreuse' => '7FFF00',
-		'chocolate' => 'D2691E',
-		'coral' => 'FF7F50',
-		'cornflowerblue' => '6495ED',
-		'cornsilk' => 'FFF8DC',
-		'crimson' => 'DC143C',
-		'cyan' => '00FFFF',
-		'darkblue' => '00008B',
-		'darkcyan' => '008B8B',
-		'darkgoldenrod' => 'B8860B',
-		'darkgray' => 'A9A9A9',
-		'darkgreen' => '006400',
-		'darkgrey' => 'A9A9A9',
-		'darkkhaki' => 'BDB76B',
-		'darkmagenta' => '8B008B',
-		'darkolivegreen' => '556B2F',
-		'darkorange' => 'FF8C00',
-		'darkorchid' => '9932CC',
-		'darkred' => '8B0000',
-		'darksalmon' => 'E9967A',
-		'darkseagreen' => '8FBC8F',
-		'darkslateblue' => '483D8B',
-		'darkslategray' => '2F4F4F',
-		'darkslategrey' => '2F4F4F',
-		'darkturquoise' => '00CED1',
-		'darkviolet' => '9400D3',
-		'deeppink' => 'FF1493',
-		'deepskyblue' => '00BFFF',
-		'dimgray' => '696969',
-		'dimgrey' => '696969',
-		'dodgerblue' => '1E90FF',
-		'firebrick' => 'B22222',
-		'floralwhite' => 'FFFAF0',
-		'forestgreen' => '228B22',
-		'fuchsia' => 'FF00FF',
-		'gainsboro' => 'DCDCDC',
-		'ghostwhite' => 'F8F8FF',
-		'gold' => 'FFD700',
-		'goldenrod' => 'DAA520',
-		'gray' => '808080',
-		'green' => '008000',
-		'greenyellow' => 'ADFF2F',
-		'grey' => '808080',
-		'honeydew' => 'F0FFF0',
-		'hotpink' => 'FF69B4',
-		'indianred' => 'CD5C5C',
-		'indigo' => '4B0082',
-		'ivory' => 'FFFFF0',
-		'khaki' => 'F0E68C',
-		'lavender' => 'E6E6FA',
-		'lavenderblush' => 'FFF0F5',
-		'lawngreen' => '7CFC00',
-		'lemonchiffon' => 'FFFACD',
-		'lightblue' => 'ADD8E6',
-		'lightcoral' => 'F08080',
-		'lightcyan' => 'E0FFFF',
-		'lightgoldenrodyellow' => 'FAFAD2',
-		'lightgray' => 'D3D3D3',
-		'lightgreen' => '90EE90',
-		'lightgrey' => 'D3D3D3',
-		'lightpink' => 'FFB6C1',
-		'lightsalmon' => 'FFA07A',
-		'lightseagreen' => '20B2AA',
-		'lightskyblue' => '87CEFA',
-		'lightslategray' => '778899',
-		'lightslategrey' => '778899',
-		'lightsteelblue' => 'B0C4DE',
-		'lightyellow' => 'FFFFE0',
-		'lime' => '00FF00',
-		'limegreen' => '32CD32',
-		'linen' => 'FAF0E6',
-		'magenta' => 'FF00FF',
-		'maroon' => '800000',
-		'mediumaquamarine' => '66CDAA',
-		'mediumblue' => '0000CD',
-		'mediumorchid' => 'BA55D3',
-		'mediumpurple' => '9370DB',
-		'mediumseagreen' => '3CB371',
-		'mediumslateblue' => '7B68EE',
-		'mediumspringgreen' => '00FA9A',
-		'mediumturquoise' => '48D1CC',
-		'mediumvioletred' => 'C71585',
-		'midnightblue' => '191970',
-		'mintcream' => 'F5FFFA',
-		'mistyrose' => 'FFE4E1',
-		'moccasin' => 'FFE4B5',
-		'navajowhite' => 'FFDEAD',
-		'navy' => '000080',
-		'oldlace' => 'FDF5E6',
-		'olive' => '808000',
-		'olivedrab' => '6B8E23',
-		'orange' => 'FFA500',
-		'orangered' => 'FF4500',
-		'orchid' => 'DA70D6',
-		'palegoldenrod' => 'EEE8AA',
-		'palegreen' => '98FB98',
-		'paleturquoise' => 'AFEEEE',
-		'palevioletred' => 'DB7093',
-		'papayawhip' => 'FFEFD5',
-		'peachpuff' => 'FFDAB9',
-		'peru' => 'CD853F',
-		'pink' => 'FFC0CB',
-		'plum' => 'DDA0DD',
-		'powderblue' => 'B0E0E6',
-		'purple' => '800080',
-		'rebeccapurple' => '663399',
-		'red' => 'FF0000',
-		'rosybrown' => 'BC8F8F',
-		'royalblue' => '4169E1',
-		'saddlebrown' => '8B4513',
-		'salmon' => 'FA8072',
-		'sandybrown' => 'F4A460',
-		'seagreen' => '2E8B57',
-		'seashell' => 'FFF5EE',
-		'sienna' => 'A0522D',
-		'silver' => 'C0C0C0',
-		'skyblue' => '87CEEB',
-		'slateblue' => '6A5ACD',
-		'slategray' => '708090',
-		'slategrey' => '708090',
-		'snow' => 'FFFAFA',
-		'springgreen' => '00FF7F',
-		'steelblue' => '4682B4',
-		'tan' => 'D2B48C',
-		'teal' => '008080',
-		'thistle' => 'D8BFD8',
-		'tomato' => 'FF6347',
-		'turquoise' => '40E0D0',
-		'violet' => 'EE82EE',
-		'wheat' => 'F5DEB3',
-		'white' => 'FFFFFF',
-		'whitesmoke' => 'F5F5F5',
-		'yellow' => 'FFFF00',
-		'yellowgreen' => '9ACD32',
-	];
-	if (isset($couleurs_html[$lc = strtolower($couleur ?? '')])) {
-		return $couleurs_html[$lc];
-	}
-
-	return $couleur;
+    $couleurs_html = [
+        'aliceblue' => 'F0F8FF',
+        'antiquewhite' => 'FAEBD7',
+        'aqua' => '00FFFF',
+        'aquamarine' => '7FFFD4',
+        'azure' => 'F0FFFF',
+        'beige' => 'F5F5DC',
+        'bisque' => 'FFE4C4',
+        'black' => '000000',
+        'blanchedalmond' => 'FFEBCD',
+        'blue' => '0000FF',
+        'blueviolet' => '8A2BE2',
+        'brown' => 'A52A2A',
+        'burlywood' => 'DEB887',
+        'cadetblue' => '5F9EA0',
+        'chartreuse' => '7FFF00',
+        'chocolate' => 'D2691E',
+        'coral' => 'FF7F50',
+        'cornflowerblue' => '6495ED',
+        'cornsilk' => 'FFF8DC',
+        'crimson' => 'DC143C',
+        'cyan' => '00FFFF',
+        'darkblue' => '00008B',
+        'darkcyan' => '008B8B',
+        'darkgoldenrod' => 'B8860B',
+        'darkgray' => 'A9A9A9',
+        'darkgreen' => '006400',
+        'darkgrey' => 'A9A9A9',
+        'darkkhaki' => 'BDB76B',
+        'darkmagenta' => '8B008B',
+        'darkolivegreen' => '556B2F',
+        'darkorange' => 'FF8C00',
+        'darkorchid' => '9932CC',
+        'darkred' => '8B0000',
+        'darksalmon' => 'E9967A',
+        'darkseagreen' => '8FBC8F',
+        'darkslateblue' => '483D8B',
+        'darkslategray' => '2F4F4F',
+        'darkslategrey' => '2F4F4F',
+        'darkturquoise' => '00CED1',
+        'darkviolet' => '9400D3',
+        'deeppink' => 'FF1493',
+        'deepskyblue' => '00BFFF',
+        'dimgray' => '696969',
+        'dimgrey' => '696969',
+        'dodgerblue' => '1E90FF',
+        'firebrick' => 'B22222',
+        'floralwhite' => 'FFFAF0',
+        'forestgreen' => '228B22',
+        'fuchsia' => 'FF00FF',
+        'gainsboro' => 'DCDCDC',
+        'ghostwhite' => 'F8F8FF',
+        'gold' => 'FFD700',
+        'goldenrod' => 'DAA520',
+        'gray' => '808080',
+        'green' => '008000',
+        'greenyellow' => 'ADFF2F',
+        'grey' => '808080',
+        'honeydew' => 'F0FFF0',
+        'hotpink' => 'FF69B4',
+        'indianred' => 'CD5C5C',
+        'indigo' => '4B0082',
+        'ivory' => 'FFFFF0',
+        'khaki' => 'F0E68C',
+        'lavender' => 'E6E6FA',
+        'lavenderblush' => 'FFF0F5',
+        'lawngreen' => '7CFC00',
+        'lemonchiffon' => 'FFFACD',
+        'lightblue' => 'ADD8E6',
+        'lightcoral' => 'F08080',
+        'lightcyan' => 'E0FFFF',
+        'lightgoldenrodyellow' => 'FAFAD2',
+        'lightgray' => 'D3D3D3',
+        'lightgreen' => '90EE90',
+        'lightgrey' => 'D3D3D3',
+        'lightpink' => 'FFB6C1',
+        'lightsalmon' => 'FFA07A',
+        'lightseagreen' => '20B2AA',
+        'lightskyblue' => '87CEFA',
+        'lightslategray' => '778899',
+        'lightslategrey' => '778899',
+        'lightsteelblue' => 'B0C4DE',
+        'lightyellow' => 'FFFFE0',
+        'lime' => '00FF00',
+        'limegreen' => '32CD32',
+        'linen' => 'FAF0E6',
+        'magenta' => 'FF00FF',
+        'maroon' => '800000',
+        'mediumaquamarine' => '66CDAA',
+        'mediumblue' => '0000CD',
+        'mediumorchid' => 'BA55D3',
+        'mediumpurple' => '9370DB',
+        'mediumseagreen' => '3CB371',
+        'mediumslateblue' => '7B68EE',
+        'mediumspringgreen' => '00FA9A',
+        'mediumturquoise' => '48D1CC',
+        'mediumvioletred' => 'C71585',
+        'midnightblue' => '191970',
+        'mintcream' => 'F5FFFA',
+        'mistyrose' => 'FFE4E1',
+        'moccasin' => 'FFE4B5',
+        'navajowhite' => 'FFDEAD',
+        'navy' => '000080',
+        'oldlace' => 'FDF5E6',
+        'olive' => '808000',
+        'olivedrab' => '6B8E23',
+        'orange' => 'FFA500',
+        'orangered' => 'FF4500',
+        'orchid' => 'DA70D6',
+        'palegoldenrod' => 'EEE8AA',
+        'palegreen' => '98FB98',
+        'paleturquoise' => 'AFEEEE',
+        'palevioletred' => 'DB7093',
+        'papayawhip' => 'FFEFD5',
+        'peachpuff' => 'FFDAB9',
+        'peru' => 'CD853F',
+        'pink' => 'FFC0CB',
+        'plum' => 'DDA0DD',
+        'powderblue' => 'B0E0E6',
+        'purple' => '800080',
+        'rebeccapurple' => '663399',
+        'red' => 'FF0000',
+        'rosybrown' => 'BC8F8F',
+        'royalblue' => '4169E1',
+        'saddlebrown' => '8B4513',
+        'salmon' => 'FA8072',
+        'sandybrown' => 'F4A460',
+        'seagreen' => '2E8B57',
+        'seashell' => 'FFF5EE',
+        'sienna' => 'A0522D',
+        'silver' => 'C0C0C0',
+        'skyblue' => '87CEEB',
+        'slateblue' => '6A5ACD',
+        'slategray' => '708090',
+        'slategrey' => '708090',
+        'snow' => 'FFFAFA',
+        'springgreen' => '00FF7F',
+        'steelblue' => '4682B4',
+        'tan' => 'D2B48C',
+        'teal' => '008080',
+        'thistle' => 'D8BFD8',
+        'tomato' => 'FF6347',
+        'turquoise' => '40E0D0',
+        'violet' => 'EE82EE',
+        'wheat' => 'F5DEB3',
+        'white' => 'FFFFFF',
+        'whitesmoke' => 'F5F5F5',
+        'yellow' => 'FFFF00',
+        'yellowgreen' => '9ACD32',
+    ];
+    if (isset($couleurs_html[$lc = strtolower($couleur ?? '')])) {
+        return $couleurs_html[$lc];
+    }
+
+    return $couleur;
 }
 
 /**
@@ -200,16 +200,16 @@ 
  * @return string Couleur tel que "hsl(200, 40%, 34%)" ou valeur formattée
  */
 function couleur_hex_to_hsl($couleur, $format = null) {
-	$hsl = _couleur_hex_to_hsl($couleur);
-	$hsl = [
-		'h' => round($hsl['h'] * 360),
-		's' => round($hsl['s'] * 100) . '%',
-		'l' => round($hsl['l'] * 100) . '%'
-	];
-	if ($format === null) {
-		return "hsl({$hsl['h']}, {$hsl['s']}, {$hsl['l']})";
-	}
-	return str_replace(array_keys($hsl), $hsl, $format);
+    $hsl = _couleur_hex_to_hsl($couleur);
+    $hsl = [
+        'h' => round($hsl['h'] * 360),
+        's' => round($hsl['s'] * 100) . '%',
+        'l' => round($hsl['l'] * 100) . '%'
+    ];
+    if ($format === null) {
+        return "hsl({$hsl['h']}, {$hsl['s']}, {$hsl['l']})";
+    }
+    return str_replace(array_keys($hsl), $hsl, $format);
 }
 
 
@@ -226,16 +226,16 @@ 
  * @return string Couleur tel que "rgb(200, 40, 84)" ou valeur formattée
  */
 function couleur_hex_to_rgb($couleur, $format = null) {
-	$rgb = _couleur_hex_to_dec($couleur);
-	$rgb = [
-		'r' => $rgb['red'],
-		'g' => $rgb['green'],
-		'b' => $rgb['blue'],
-	];
-	if ($format === null) {
-		return "rgb({$rgb['r']}, {$rgb['g']}, {$rgb['b']})";
-	}
-	return str_replace(array_keys($rgb), $rgb, $format);
+    $rgb = _couleur_hex_to_dec($couleur);
+    $rgb = [
+        'r' => $rgb['red'],
+        'g' => $rgb['green'],
+        'b' => $rgb['blue'],
+    ];
+    if ($format === null) {
+        return "rgb({$rgb['r']}, {$rgb['g']}, {$rgb['b']})";
+    }
+    return str_replace(array_keys($rgb), $rgb, $format);
 }
 
 /**
@@ -251,13 +251,13 @@ 
  *    Code hexadécimal de la couleur plus foncée
  */
 function couleur_foncer($couleur, $coeff = 0.5) {
-	$couleurs = _couleur_hex_to_dec($couleur);
+    $couleurs = _couleur_hex_to_dec($couleur);
 
-	$red = $couleurs['red'] - round(($couleurs['red']) * $coeff);
-	$green = $couleurs['green'] - round(($couleurs['green']) * $coeff);
-	$blue = $couleurs['blue'] - round(($couleurs['blue']) * $coeff);
+    $red = $couleurs['red'] - round(($couleurs['red']) * $coeff);
+    $green = $couleurs['green'] - round(($couleurs['green']) * $coeff);
+    $blue = $couleurs['blue'] - round(($couleurs['blue']) * $coeff);
 
-	return _couleur_dec_to_hex($red, $green, $blue);
+    return _couleur_dec_to_hex($red, $green, $blue);
 }
 
 /**
@@ -273,13 +273,13 @@ 
  *    Code hexadécimal de la couleur éclaircie
  */
 function couleur_eclaircir($couleur, $coeff = 0.5) {
-	$couleurs = _couleur_hex_to_dec($couleur);
+    $couleurs = _couleur_hex_to_dec($couleur);
 
-	$red = $couleurs['red'] + round((255 - $couleurs['red']) * $coeff);
-	$green = $couleurs['green'] + round((255 - $couleurs['green']) * $coeff);
-	$blue = $couleurs['blue'] + round((255 - $couleurs['blue']) * $coeff);
+    $red = $couleurs['red'] + round((255 - $couleurs['red']) * $coeff);
+    $green = $couleurs['green'] + round((255 - $couleurs['green']) * $coeff);
+    $blue = $couleurs['blue'] + round((255 - $couleurs['blue']) * $coeff);
 
-	return _couleur_dec_to_hex($red, $green, $blue);
+    return _couleur_dec_to_hex($red, $green, $blue);
 }
 
 /**
@@ -302,28 +302,28 @@ 
  *    Le tag html `<img src=... />` avec une class `filtre_inactif` ou pas
  */
 function image_select($img, $width_min = 0, $height_min = 0, $width_max = 10000, $height_max = 1000) {
-	if (!$img) {
-		return $img;
-	}
-	[$h, $l] = taille_image($img);
-	$select = true;
-	if ($l < $width_min || $l > $width_max || $h < $height_min || $h > $height_max) {
-		$select = false;
-	}
-
-	$class = extraire_attribut($img, 'class');
-	$p = strpos($class, 'filtre_inactif');
-	if ($select == false && $p === false) {
-		$class .= ' filtre_inactif';
-		$img = inserer_attribut($img, 'class', $class);
-	}
-	if ($select == true && $p !== false) {
-		// no_image_filtrer : historique, a virer
-		$class = preg_replace(',\s*(filtre_inactif|no_image_filtrer),', '', $class);
-		$img = inserer_attribut($img, 'class', $class);
-	}
-
-	return $img;
+    if (!$img) {
+        return $img;
+    }
+    [$h, $l] = taille_image($img);
+    $select = true;
+    if ($l < $width_min || $l > $width_max || $h < $height_min || $h > $height_max) {
+        $select = false;
+    }
+
+    $class = extraire_attribut($img, 'class');
+    $p = strpos($class, 'filtre_inactif');
+    if ($select == false && $p === false) {
+        $class .= ' filtre_inactif';
+        $img = inserer_attribut($img, 'class', $class);
+    }
+    if ($select == true && $p !== false) {
+        // no_image_filtrer : historique, a virer
+        $class = preg_replace(',\s*(filtre_inactif|no_image_filtrer),', '', $class);
+        $img = inserer_attribut($img, 'class', $class);
+    }
+
+    return $img;
 }
 
 /**
@@ -371,45 +371,45 @@ 
  *     Code HTML de l'image ou du texte.
  **/
 function image_passe_partout(
-	$img,
-	$taille_x = -1,
-	$taille_y = -1,
-	$force = false,
-	$cherche_image = false,
-	$process = 'AUTO'
+    $img,
+    $taille_x = -1,
+    $taille_y = -1,
+    $force = false,
+    $cherche_image = false,
+    $process = 'AUTO'
 ) {
-	// PHP 7+ type hint
-	$img = (string)$img;
-	$taille_x = (int)$taille_x;
-	$taille_y = (int)$taille_y;
-	$force = (bool)$force;
-	$process = (string)$process;
-
-	if (!$img) {
-		return '';
-	}
-	[$hauteur, $largeur] = taille_image($img);
-	if ($taille_x === -1) {
-		$taille_x = $GLOBALS['meta']['taille_preview'] ?? 150;
-	}
-	if ($taille_y === -1) {
-		$taille_y = $taille_x;
-	}
-
-	if ($taille_x === 0 && $taille_y > 0) {
-		$taille_x = 1;
-	} # {0,300} -> c'est 300 qui compte
-	elseif ($taille_x > 0 && $taille_y === 0) {
-		$taille_y = 1;
-	} # {300,0} -> c'est 300 qui compte
-	elseif ($taille_x == 0 && $taille_y === 0) {
-		return '';
-	}
-
-	[$destWidth, $destHeight, $ratio] = ratio_passe_partout($largeur ?? 0, $hauteur ?? 0, $taille_x, $taille_y);
-	$fonction = ['image_passe_partout', func_get_args()];
-
-	return process_image_reduire($fonction, $img, $destWidth, $destHeight, $force, $process);
+    // PHP 7+ type hint
+    $img = (string)$img;
+    $taille_x = (int)$taille_x;
+    $taille_y = (int)$taille_y;
+    $force = (bool)$force;
+    $process = (string)$process;
+
+    if (!$img) {
+        return '';
+    }
+    [$hauteur, $largeur] = taille_image($img);
+    if ($taille_x === -1) {
+        $taille_x = $GLOBALS['meta']['taille_preview'] ?? 150;
+    }
+    if ($taille_y === -1) {
+        $taille_y = $taille_x;
+    }
+
+    if ($taille_x === 0 && $taille_y > 0) {
+        $taille_x = 1;
+    } # {0,300} -> c'est 300 qui compte
+    elseif ($taille_x > 0 && $taille_y === 0) {
+        $taille_y = 1;
+    } # {300,0} -> c'est 300 qui compte
+    elseif ($taille_x == 0 && $taille_y === 0) {
+        return '';
+    }
+
+    [$destWidth, $destHeight, $ratio] = ratio_passe_partout($largeur ?? 0, $hauteur ?? 0, $taille_x, $taille_y);
+    $fonction = ['image_passe_partout', func_get_args()];
+
+    return process_image_reduire($fonction, $img, $destWidth, $destHeight, $force, $process);
 }
 
 /**
@@ -452,44 +452,44 @@ 
  *     Code HTML de l'image ou du texte.
  **/
 function image_reduire(
-	$img,
-	$taille = -1,
-	$taille_y = -1,
-	$force = false,
-	$cherche_image = false,
-	$process = 'AUTO'
+    $img,
+    $taille = -1,
+    $taille_y = -1,
+    $force = false,
+    $cherche_image = false,
+    $process = 'AUTO'
 ) {
-	// PHP 7+ type hint
-	$img = (string)$img;
-	$taille = (int)$taille;
-	$taille_y = (int)$taille_y;
-	$force = (bool)$force;
-	$process = (string)$process;
-
-	// Determiner la taille x,y maxi
-	// prendre le reglage de previsu par defaut
-	if ($taille === -1) {
-		$taille = (isset($GLOBALS['meta']['taille_preview']) && (int) $GLOBALS['meta']['taille_preview'])
-			? (int) $GLOBALS['meta']['taille_preview']
-			: 150;
-	}
-	if ($taille_y === -1) {
-		$taille_y = $taille;
-	}
-
-	if ($taille === 0 && $taille_y > 0) {
-		$taille = 10000;
-	} # {0,300} -> c'est 300 qui compte
-	elseif ($taille > 0 && $taille_y === 0) {
-		$taille_y = 10000;
-	} # {300,0} -> c'est 300 qui compte
-	elseif ($taille == 0 && $taille_y === 0) {
-		return '';
-	}
-
-	$fonction = ['image_reduire', func_get_args()];
-
-	return process_image_reduire($fonction, $img, $taille, $taille_y, $force, $process);
+    // PHP 7+ type hint
+    $img = (string)$img;
+    $taille = (int)$taille;
+    $taille_y = (int)$taille_y;
+    $force = (bool)$force;
+    $process = (string)$process;
+
+    // Determiner la taille x,y maxi
+    // prendre le reglage de previsu par defaut
+    if ($taille === -1) {
+        $taille = (isset($GLOBALS['meta']['taille_preview']) && (int) $GLOBALS['meta']['taille_preview'])
+            ? (int) $GLOBALS['meta']['taille_preview']
+            : 150;
+    }
+    if ($taille_y === -1) {
+        $taille_y = $taille;
+    }
+
+    if ($taille === 0 && $taille_y > 0) {
+        $taille = 10000;
+    } # {0,300} -> c'est 300 qui compte
+    elseif ($taille > 0 && $taille_y === 0) {
+        $taille_y = 10000;
+    } # {300,0} -> c'est 300 qui compte
+    elseif ($taille == 0 && $taille_y === 0) {
+        return '';
+    }
+
+    $fonction = ['image_reduire', func_get_args()];
+
+    return process_image_reduire($fonction, $img, $taille, $taille_y, $force, $process);
 }
 
 /**
@@ -500,16 +500,16 @@ 
  *
  **/
 function image_recadre_avec_fallback(
-	$im,
-	$width,
-	$height = '-',
-	$position = 'focus',
-	$background_color = 'white'
+    $im,
+    $width,
+    $height = '-',
+    $position = 'focus',
+    $background_color = 'white'
 ) {
-	if (function_exists('image_recadre') && ($GLOBALS['meta']['image_process'] ?? '') === 'gd2') {
-		return image_reduire(image_recadre($im, $width . ':' . $height, '-', $position, $background_color), $width, $height);
-	} else { return image_passe_partout($im, $width, $height);
-	}
+    if (function_exists('image_recadre') && ($GLOBALS['meta']['image_process'] ?? '') === 'gd2') {
+        return image_reduire(image_recadre($im, $width . ':' . $height, '-', $position, $background_color), $width, $height);
+    } else { return image_passe_partout($im, $width, $height);
+    }
 }
 
 /**
@@ -527,23 +527,23 @@ 
  *     Code HTML de l'image ou du texte.
  **/
 function image_reduire_par($img, $val = 1, $force = false) {
-	// PHP 7+ type hint
-	$img = (string)$img;
-	$val = (int)$val;
-	$force = (bool)$force;
+    // PHP 7+ type hint
+    $img = (string)$img;
+    $val = (int)$val;
+    $force = (bool)$force;
 
-	[$hauteur, $largeur] = taille_image($img);
+    [$hauteur, $largeur] = taille_image($img);
 
-	$l = round($largeur / $val);
-	$h = round($hauteur / $val);
+    $l = round($largeur / $val);
+    $h = round($hauteur / $val);
 
-	if ($l > $h) {
-		$h = 0;
-	} else {
-		$l = 0;
-	}
+    if ($l > $h) {
+        $h = 0;
+    } else {
+        $l = 0;
+    }
 
-	return image_reduire($img, $l, $h, $force);
+    return image_reduire($img, $l, $h, $force);
 }
 
 /**
@@ -566,10 +566,10 @@ 
  *      Couleur en écriture hexadécimale.
 **/
 function filtre_couleur_saturation_dist($couleur, $val, $strict = false) {
-	if (function_exists('couleur_saturation')) {
-		return couleur_saturation($couleur, $val, $strict);
-	}
-	return $couleur;
+    if (function_exists('couleur_saturation')) {
+        return couleur_saturation($couleur, $val, $strict);
+    }
+    return $couleur;
 }
 
 /**
@@ -590,8 +590,8 @@ 
  *      Couleur en écriture hexadécimale.
 **/
 function filtre_couleur_luminance_dist($couleur, $val) {
-	if (function_exists('couleur_luminance')) {
-		return couleur_luminance($couleur, $val);
-	}
-	return $couleur;
+    if (function_exists('couleur_luminance')) {
+        return couleur_luminance($couleur, $val);
+    }
+    return $couleur;
 }

ecrire/inc/puce_statut.php

Indentation +227 added lines, -227 removed lines

@@ -17,18 +17,18 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 if (!defined('_ACTIVER_PUCE_RAPIDE')) {
-	/**
-	 * Activer le changement rapide de statut sur les listes d'objets ?
-	 *
-	 * Peut ralentir un site sur des listes très longues.
-	 *
-	 * @var bool
-	 **/
-	define('_ACTIVER_PUCE_RAPIDE', true);
+    /**
+     * Activer le changement rapide de statut sur les listes d'objets ?
+     *
+     * Peut ralentir un site sur des listes très longues.
+     *
+     * @var bool
+     **/
+    define('_ACTIVER_PUCE_RAPIDE', true);
 }
 
 /**
@@ -56,31 +56,31 @@ 
  *     Code HTML de l'image de puce de statut à insérer (et du menu de changement si présent)
  */
 function inc_puce_statut_dist(
-	$id_objet,
-	$statut,
-	$id_parent,
-	$type,
-	$ajax = false,
-	$menu_rapide = _ACTIVER_PUCE_RAPIDE
+    $id_objet,
+    $statut,
+    $id_parent,
+    $type,
+    $ajax = false,
+    $menu_rapide = _ACTIVER_PUCE_RAPIDE
 ) {
-	static $f_puce_statut = [];
-	$type = objet_type($type);
-	// cas prioritaire : fonction perso, qui permet aussi de gerer les cas historiques
-	if (!isset($f_puce_statut[$type]) || is_null($f_puce_statut[$type])) {
-		$f_puce_statut[$type] = charger_fonction($type, 'puce_statut', true);
-	}
-	if ($f_puce_statut[$type]) {
-		return $f_puce_statut[$type]($id_objet, $statut, $id_parent, $type, $ajax, $menu_rapide);
-	}
-
-	// si statut_image trouve quelque chose (et '' est quelque chose)
-	// composer une puce, avec si possible changement rapide
-	elseif (!is_null($puce = puce_statut_changement_rapide($id_objet, $statut, $id_parent, $type, $ajax, $menu_rapide))) {
-		return $puce;
-	} // sinon fausse puce avec le type de l'image
-	else {
-		return http_img_pack("$type-16.png", '');
-	}
+    static $f_puce_statut = [];
+    $type = objet_type($type);
+    // cas prioritaire : fonction perso, qui permet aussi de gerer les cas historiques
+    if (!isset($f_puce_statut[$type]) || is_null($f_puce_statut[$type])) {
+        $f_puce_statut[$type] = charger_fonction($type, 'puce_statut', true);
+    }
+    if ($f_puce_statut[$type]) {
+        return $f_puce_statut[$type]($id_objet, $statut, $id_parent, $type, $ajax, $menu_rapide);
+    }
+
+    // si statut_image trouve quelque chose (et '' est quelque chose)
+    // composer une puce, avec si possible changement rapide
+    elseif (!is_null($puce = puce_statut_changement_rapide($id_objet, $statut, $id_parent, $type, $ajax, $menu_rapide))) {
+        return $puce;
+    } // sinon fausse puce avec le type de l'image
+    else {
+        return http_img_pack("$type-16.png", '');
+    }
 }
 
 /**
@@ -109,41 +109,41 @@ 
  *   null si pas capable de déterminer l'image
  */
 function statut_image($objet, $statut) {
-	$src = null;
-	$table = table_objet_sql($objet);
-	$desc = lister_tables_objets_sql($table);
-	if (isset($desc['statut_images'])) {
-		// si une declaration statut_images
-		// mais rien pour le statut demande, ne rien afficher
-		$src = '';
-		if (isset($desc['statut_images'][$statut])) {
-			$src = $desc['statut_images'][$statut];
-		} // sinon image par defaut ?
-		elseif (isset($desc['statut_images'][0])) {
-			$src = $desc['statut_images'][0];
-		}
-	} else {
-		switch ($statut) {
-			case 'prepa':
-				$src = 'puce-preparer-xx.svg?12px';
-				break;
-			case 'prop':
-				$src = 'puce-proposer-xx.svg?12px';
-				break;
-			case 'publie':
-				$src = 'puce-publier-xx.svg?12px';
-				break;
-			case 'refuse':
-				$src = 'puce-refuser-xx.svg?12px';
-				break;
-			case 'poubelle':
-			case 'poub':
-				$src = 'puce-supprimer-xx.svg?12px';
-				break;
-		}
-	}
-
-	return $src;
+    $src = null;
+    $table = table_objet_sql($objet);
+    $desc = lister_tables_objets_sql($table);
+    if (isset($desc['statut_images'])) {
+        // si une declaration statut_images
+        // mais rien pour le statut demande, ne rien afficher
+        $src = '';
+        if (isset($desc['statut_images'][$statut])) {
+            $src = $desc['statut_images'][$statut];
+        } // sinon image par defaut ?
+        elseif (isset($desc['statut_images'][0])) {
+            $src = $desc['statut_images'][0];
+        }
+    } else {
+        switch ($statut) {
+            case 'prepa':
+                $src = 'puce-preparer-xx.svg?12px';
+                break;
+            case 'prop':
+                $src = 'puce-proposer-xx.svg?12px';
+                break;
+            case 'publie':
+                $src = 'puce-publier-xx.svg?12px';
+                break;
+            case 'refuse':
+                $src = 'puce-refuser-xx.svg?12px';
+                break;
+            case 'poubelle':
+            case 'poub':
+                $src = 'puce-supprimer-xx.svg?12px';
+                break;
+        }
+    }
+
+    return $src;
 }
 
 /**
@@ -171,40 +171,40 @@ 
  * @return string
  */
 function statut_titre($objet, $statut) {
-	$titre = '';
-	$table = table_objet_sql($objet);
-	$desc = lister_tables_objets_sql($table);
-	if (isset($desc['statut_titres'])) {
-		// si une declaration statut_titres
-		// mais rien pour le statut demande, ne rien afficher
-		if (isset($desc['statut_titres'][$statut])) {
-			$titre = $desc['statut_titres'][$statut];
-		} // sinon image par defaut ?
-		elseif (isset($desc['statut_titres'][0])) {
-			$titre = $desc['statut_titres'][0];
-		}
-	} else {
-		switch ($statut) {
-			case 'prepa':
-				$titre = 'texte_statut_en_cours_redaction';
-				break;
-			case 'prop':
-				$titre = 'texte_statut_propose_evaluation';
-				break;
-			case 'publie':
-				$titre = 'texte_statut_publie';
-				break;
-			case 'refuse':
-				$titre = 'texte_statut_refuse';
-				break;
-			case 'poubelle':
-			case 'poub':
-				$titre = 'texte_statut_poubelle';
-				break;
-		}
-	}
-
-	return $titre ? _T($titre) : '';
+    $titre = '';
+    $table = table_objet_sql($objet);
+    $desc = lister_tables_objets_sql($table);
+    if (isset($desc['statut_titres'])) {
+        // si une declaration statut_titres
+        // mais rien pour le statut demande, ne rien afficher
+        if (isset($desc['statut_titres'][$statut])) {
+            $titre = $desc['statut_titres'][$statut];
+        } // sinon image par defaut ?
+        elseif (isset($desc['statut_titres'][0])) {
+            $titre = $desc['statut_titres'][0];
+        }
+    } else {
+        switch ($statut) {
+            case 'prepa':
+                $titre = 'texte_statut_en_cours_redaction';
+                break;
+            case 'prop':
+                $titre = 'texte_statut_propose_evaluation';
+                break;
+            case 'publie':
+                $titre = 'texte_statut_publie';
+                break;
+            case 'refuse':
+                $titre = 'texte_statut_refuse';
+                break;
+            case 'poubelle':
+            case 'poub':
+                $titre = 'texte_statut_poubelle';
+                break;
+        }
+    }
+
+    return $titre ? _T($titre) : '';
 }
 
 
@@ -224,37 +224,37 @@ 
  * @return string
  */
 function statut_texte_instituer($objet, $statut) {
-	$texte = '';
-	$table = table_objet_sql($objet);
-	$desc = lister_tables_objets_sql($table);
-	if (isset($desc['statut_textes_instituer'])) {
-		// si une declaration statut_titres
-		// mais rien pour le statut demande, ne rien afficher
-		if (isset($desc['statut_textes_instituer'][$statut])) {
-			$texte = $desc['statut_textes_instituer'][$statut];
-		}
-	} else {
-		switch ($statut) {
-			case 'prepa':
-				$texte = 'texte_statut_en_cours_redaction';
-				break;
-			case 'prop':
-				$texte = 'texte_statut_propose_evaluation';
-				break;
-			case 'publie':
-				$texte = 'texte_statut_publie';
-				break;
-			case 'refuse':
-				$texte = 'texte_statut_refuse';
-				break;
-			case 'poubelle':
-			case 'poub':
-				$texte = 'texte_statut_poubelle';
-				break;
-		}
-	}
-
-	return $texte ? _T($texte) : '';
+    $texte = '';
+    $table = table_objet_sql($objet);
+    $desc = lister_tables_objets_sql($table);
+    if (isset($desc['statut_textes_instituer'])) {
+        // si une declaration statut_titres
+        // mais rien pour le statut demande, ne rien afficher
+        if (isset($desc['statut_textes_instituer'][$statut])) {
+            $texte = $desc['statut_textes_instituer'][$statut];
+        }
+    } else {
+        switch ($statut) {
+            case 'prepa':
+                $texte = 'texte_statut_en_cours_redaction';
+                break;
+            case 'prop':
+                $texte = 'texte_statut_propose_evaluation';
+                break;
+            case 'publie':
+                $texte = 'texte_statut_publie';
+                break;
+            case 'refuse':
+                $texte = 'texte_statut_refuse';
+                break;
+            case 'poubelle':
+            case 'poub':
+                $texte = 'texte_statut_poubelle';
+                break;
+        }
+    }
+
+    return $texte ? _T($texte) : '';
 }
 
 
@@ -274,16 +274,16 @@ 
  * @return string
  */
 function puce_statut_auteur_dist($id, $statut, $id_parent, $type, $ajax = '', $menu_rapide = _ACTIVER_PUCE_RAPIDE) {
-	$img = statut_image('auteur', $statut);
-	if (!$img) {
-		return '';
-	}
-	$alt = statut_titre('auteur', $statut);
+    $img = statut_image('auteur', $statut);
+    if (!$img) {
+        return '';
+    }
+    $alt = statut_titre('auteur', $statut);
 
-	$fond = '';
-	$titre = '';
+    $fond = '';
+    $titre = '';
 
-	/*
+    /*
 	if ($type != 'auteur') {
 	  $img2 = chemin_image('del-16.png');
 	  $titre = _T('titre_image_redacteur');
@@ -293,12 +293,12 @@ 
 	}
 	*/
 
-	return http_img_pack($img, $alt, $fond, $alt);
+    return http_img_pack($img, $alt, $fond, $alt);
 }
 
 
 function puce_statut_rubrique_dist($id, $statut, $id_rubrique, $type, $ajax = '', $menu_rapide = _ACTIVER_PUCE_RAPIDE) {
-	return http_img_pack('rubrique-16.png', '');
+    return http_img_pack('rubrique-16.png', '');
 }
 
 /**
@@ -322,101 +322,101 @@ 
  *     Code HTML de l'image de puce de statut à insérer (et du menu de changement si présent)
  **/
 function puce_statut_changement_rapide(
-	$id,
-	$statut,
-	$id_rubrique,
-	$type = 'article',
-	$ajax = false,
-	$menu_rapide = _ACTIVER_PUCE_RAPIDE
+    $id,
+    $statut,
+    $id_rubrique,
+    $type = 'article',
+    $ajax = false,
+    $menu_rapide = _ACTIVER_PUCE_RAPIDE
 ) {
-	$src = statut_image($type, $statut);
-	if (!$src) {
-		return $src;
-	}
-
-	$ajax_node = (!$id || !_SPIP_AJAX || !$menu_rapide) ? '' : " class='imgstatut$type$id'";
-
-
-	$t = statut_titre($type, $statut);
-	$inser_puce = http_img_pack($src, $t, $ajax_node, $t);
-
-	if (!$ajax_node) {
-		return $inser_puce;
-	}
-
-	$table = table_objet_sql($type);
-	$desc = lister_tables_objets_sql($table);
-	if (!isset($desc['statut_textes_instituer'])) {
-		return $inser_puce;
-	}
-
-	if (!function_exists('autoriser')) {
-		include_spip('inc/autoriser');
-	}
-
-	// cas ou l'on a un parent connu (devrait disparaitre au profit du second cas plus generique)
-	if ($id_rubrique) {
-		if (!autoriser('publierdans', 'rubrique', $id_rubrique)) {
-			return $inser_puce;
-		}
-	} // si pas d'id_rubrique fourni, tester directement instituer type avec le statut publie
-	else {
-		if (!autoriser('instituer', $type, $id, null, ['statut' => 'publie'])) {
-			return $inser_puce;
-		}
-	}
-
-	$coord = array_flip(array_keys($desc['statut_textes_instituer']));
-	if (!isset($coord[$statut])) {
-		return $inser_puce;
-	}
-
-	$unit = 18/*widh de img*/ + 0/*padding*/
-	;
-	$margin = 0; /* marge a gauche + droite */
-	$zero = 0 /*border*/ + $margin / 2 + 0 /*padding*/
-	;
-	$clip = $zero + ($unit * $coord[$statut]);
-
-	if ($ajax) {
-		$width = $unit * (is_countable($desc['statut_textes_instituer']) ? count($desc['statut_textes_instituer']) : 0) + $margin;
-		$out = "<span class='puce_objet_fixe $type'>"
-			. $inser_puce
-			. '</span>'
-			. "<span class='puce_objet_popup $type statutdecal$type$id' style='width:{$width}px;margin-left:-{$clip}px;'>";
-		$i = 0;
-		foreach ($desc['statut_textes_instituer'] as $s => $t) {
-			$out .= afficher_script_statut($id, $type, -$zero - $i++ * $unit, statut_image($type, $s), $s, _T($t));
-		}
-
-		return $out . '</span>';
-	} else {
-		$nom = 'puce_statut_';
-		$action = generer_url_ecrire('puce_statut', '', true);
-		$lang_dir = lang_dir(lang_typo());
-
-		return "<span class='puce_objet $type' id='$nom$type$id' dir='$lang_dir' data-puce-nom='$nom' data-puce-type='$type' data-puce-id='$id' data-puce-action='$action'>"
-		. $inser_puce
-		. '</span>';
-	}
+    $src = statut_image($type, $statut);
+    if (!$src) {
+        return $src;
+    }
+
+    $ajax_node = (!$id || !_SPIP_AJAX || !$menu_rapide) ? '' : " class='imgstatut$type$id'";
+
+
+    $t = statut_titre($type, $statut);
+    $inser_puce = http_img_pack($src, $t, $ajax_node, $t);
+
+    if (!$ajax_node) {
+        return $inser_puce;
+    }
+
+    $table = table_objet_sql($type);
+    $desc = lister_tables_objets_sql($table);
+    if (!isset($desc['statut_textes_instituer'])) {
+        return $inser_puce;
+    }
+
+    if (!function_exists('autoriser')) {
+        include_spip('inc/autoriser');
+    }
+
+    // cas ou l'on a un parent connu (devrait disparaitre au profit du second cas plus generique)
+    if ($id_rubrique) {
+        if (!autoriser('publierdans', 'rubrique', $id_rubrique)) {
+            return $inser_puce;
+        }
+    } // si pas d'id_rubrique fourni, tester directement instituer type avec le statut publie
+    else {
+        if (!autoriser('instituer', $type, $id, null, ['statut' => 'publie'])) {
+            return $inser_puce;
+        }
+    }
+
+    $coord = array_flip(array_keys($desc['statut_textes_instituer']));
+    if (!isset($coord[$statut])) {
+        return $inser_puce;
+    }
+
+    $unit = 18/*widh de img*/ + 0/*padding*/
+    ;
+    $margin = 0; /* marge a gauche + droite */
+    $zero = 0 /*border*/ + $margin / 2 + 0 /*padding*/
+    ;
+    $clip = $zero + ($unit * $coord[$statut]);
+
+    if ($ajax) {
+        $width = $unit * (is_countable($desc['statut_textes_instituer']) ? count($desc['statut_textes_instituer']) : 0) + $margin;
+        $out = "<span class='puce_objet_fixe $type'>"
+            . $inser_puce
+            . '</span>'
+            . "<span class='puce_objet_popup $type statutdecal$type$id' style='width:{$width}px;margin-left:-{$clip}px;'>";
+        $i = 0;
+        foreach ($desc['statut_textes_instituer'] as $s => $t) {
+            $out .= afficher_script_statut($id, $type, -$zero - $i++ * $unit, statut_image($type, $s), $s, _T($t));
+        }
+
+        return $out . '</span>';
+    } else {
+        $nom = 'puce_statut_';
+        $action = generer_url_ecrire('puce_statut', '', true);
+        $lang_dir = lang_dir(lang_typo());
+
+        return "<span class='puce_objet $type' id='$nom$type$id' dir='$lang_dir' data-puce-nom='$nom' data-puce-type='$type' data-puce-id='$id' data-puce-action='$action'>"
+        . $inser_puce
+        . '</span>';
+    }
 }
 
 
 function afficher_script_statut($id, $type, $n, $img, $statut, $titre, $act = '') {
-	$h = generer_action_auteur('instituer_objet', "$type-$id-$statut");
-	$t = supprimer_tags($titre);
+    $h = generer_action_auteur('instituer_objet', "$type-$id-$statut");
+    $t = supprimer_tags($titre);
 
-	return "<a href=\"#\" data-puce-id=\"$id\" data-puce-type=\"$type\" data-puce-decal=\"$n\" data-puce-action=\"$h\" title=\"$t\"$act>" . http_img_pack($img, $t) . '</a>';
+    return "<a href=\"#\" data-puce-id=\"$id\" data-puce-type=\"$type\" data-puce-decal=\"$n\" data-puce-action=\"$h\" title=\"$t\"$act>" . http_img_pack($img, $t) . '</a>';
 }
 
 // compat
 // La couleur du statut
 
 function puce_statut($statut, $atts = '') {
-	$src = statut_image('article', $statut);
-	if (!$src) {
-		return '';
-	}
+    $src = statut_image('article', $statut);
+    if (!$src) {
+        return '';
+    }
 
-	return http_img_pack($src, statut_titre('article', $statut), $atts);
+    return http_img_pack($src, statut_titre('article', $statut), $atts);
 }

Spacing +2 added lines, -2 removed lines

@@ -389,7 +389,7 @@ 
 			$out .= afficher_script_statut($id, $type, -$zero - $i++ * $unit, statut_image($type, $s), $s, _T($t));
 		}
 
-		return $out . '</span>';
+		return $out.'</span>';
 	} else {
 		$nom = 'puce_statut_';
 		$action = generer_url_ecrire('puce_statut', '', true);
@@ -406,7 +406,7 @@ 
 	$h = generer_action_auteur('instituer_objet', "$type-$id-$statut");
 	$t = supprimer_tags($titre);
 
-	return "<a href=\"#\" data-puce-id=\"$id\" data-puce-type=\"$type\" data-puce-decal=\"$n\" data-puce-action=\"$h\" title=\"$t\"$act>" . http_img_pack($img, $t) . '</a>';
+	return "<a href=\"#\" data-puce-id=\"$id\" data-puce-type=\"$type\" data-puce-decal=\"$n\" data-puce-action=\"$h\" title=\"$t\"$act>".http_img_pack($img, $t).'</a>';
 }
 
 // compat

ecrire/inc/filtres_images_lib_mini.php

Indentation +1286 added lines, -1286 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/filtres'); // par precaution
 include_spip('inc/filtres_images_mini'); // par precaution
@@ -37,21 +37,21 @@ 
  *     Le code de la couleur en hexadécimal.
  */
 function _couleur_dec_to_hex($red, $green, $blue) {
-	$red = dechex($red);
-	$green = dechex($green);
-	$blue = dechex($blue);
-
-	if (strlen($red) == 1) {
-		$red = '0' . $red;
-	}
-	if (strlen($green) == 1) {
-		$green = '0' . $green;
-	}
-	if (strlen($blue) == 1) {
-		$blue = '0' . $blue;
-	}
-
-	return "$red$green$blue";
+    $red = dechex($red);
+    $green = dechex($green);
+    $blue = dechex($blue);
+
+    if (strlen($red) == 1) {
+        $red = '0' . $red;
+    }
+    if (strlen($green) == 1) {
+        $green = '0' . $green;
+    }
+    if (strlen($blue) == 1) {
+        $blue = '0' . $blue;
+    }
+
+    return "$red$green$blue";
 }
 
 /**
@@ -63,18 +63,18 @@ 
  *     Un tableau des 3 éléments : rouge, vert, bleu.
  */
 function _couleur_hex_to_dec($couleur) {
-	$couleur ??= '';
-	$couleur = couleur_html_to_hex($couleur);
-	$couleur = ltrim($couleur, '#');
-	if (strlen($couleur) === 3) {
-		$couleur = $couleur[0] . $couleur[0] . $couleur[1] . $couleur[1] . $couleur[2] . $couleur[2];
-	}
-	$retour = [];
-	$retour['red'] = hexdec(substr($couleur, 0, 2));
-	$retour['green'] = hexdec(substr($couleur, 2, 2));
-	$retour['blue'] = hexdec(substr($couleur, 4, 2));
-
-	return $retour;
+    $couleur ??= '';
+    $couleur = couleur_html_to_hex($couleur);
+    $couleur = ltrim($couleur, '#');
+    if (strlen($couleur) === 3) {
+        $couleur = $couleur[0] . $couleur[0] . $couleur[1] . $couleur[1] . $couleur[2] . $couleur[2];
+    }
+    $retour = [];
+    $retour['red'] = hexdec(substr($couleur, 0, 2));
+    $retour['green'] = hexdec(substr($couleur, 2, 2));
+    $retour['blue'] = hexdec(substr($couleur, 4, 2));
+
+    return $retour;
 }
 
 
@@ -91,8 +91,8 @@ 
  *     Le code de la couleur en hexadécimal.
  */
 function _couleur_hsl_to_hex($hue, $saturation, $lightness) {
-	$rgb = _couleur_hsl_to_rgb($hue, $saturation, $lightness);
-	return _couleur_dec_to_hex($rgb['r'], $rgb['g'], $rgb['b']);
+    $rgb = _couleur_hsl_to_rgb($hue, $saturation, $lightness);
+    return _couleur_dec_to_hex($rgb['r'], $rgb['g'], $rgb['b']);
 }
 
 /**
@@ -104,8 +104,8 @@ 
  *     Un tableau des 3 éléments : teinte, saturation, luminosité.
  */
 function _couleur_hex_to_hsl($couleur) {
-	$rgb = _couleur_hex_to_dec($couleur);
-	return _couleur_rgb_to_hsl($rgb['red'], $rgb['green'], $rgb['blue']);
+    $rgb = _couleur_hex_to_dec($couleur);
+    return _couleur_rgb_to_hsl($rgb['red'], $rgb['green'], $rgb['blue']);
 }
 
 /**
@@ -120,55 +120,55 @@ 
  * @return array
  */
 function _couleur_rgb_to_hsl($R, $G, $B) {
-	$H = null;
-	$var_R = ($R / 255); // Where RGB values = 0 ÷ 255
-	$var_G = ($G / 255);
-	$var_B = ($B / 255);
-
-	$var_Min = min($var_R, $var_G, $var_B);   //Min. value of RGB
-	$var_Max = max($var_R, $var_G, $var_B);   //Max. value of RGB
-	$del_Max = $var_Max - $var_Min;           //Delta RGB value
-
-	$L = ($var_Max + $var_Min) / 2;
-
-	if ($del_Max == 0) {
-		//This is a gray, no chroma...
-		$H = 0; //HSL results = 0 ÷ 1
-		$S = 0;
-	} else {
-		// Chromatic data...
-		$S = $L < 0.5 ? $del_Max / ($var_Max + $var_Min) : $del_Max / (2 - $var_Max - $var_Min);
-
-		$del_R = ((($var_Max - $var_R) / 6) + ($del_Max / 2)) / $del_Max;
-		$del_G = ((($var_Max - $var_G) / 6) + ($del_Max / 2)) / $del_Max;
-		$del_B = ((($var_Max - $var_B) / 6) + ($del_Max / 2)) / $del_Max;
-
-		if ($var_R === $var_Max) {
-			$H = $del_B - $del_G;
-		} else {
-			if ($var_G === $var_Max) {
-				$H = (1 / 3) + $del_R - $del_B;
-			} else {
-				if ($var_B === $var_Max) {
-					$H = (2 / 3) + $del_G - $del_R;
-				}
-			}
-		}
-
-		if ($H < 0) {
-			$H += 1;
-		}
-		if ($H > 1) {
-			$H -= 1;
-		}
-	}
-
-	$ret = [];
-	$ret['h'] = $H;
-	$ret['s'] = $S;
-	$ret['l'] = $L;
-
-	return $ret;
+    $H = null;
+    $var_R = ($R / 255); // Where RGB values = 0 ÷ 255
+    $var_G = ($G / 255);
+    $var_B = ($B / 255);
+
+    $var_Min = min($var_R, $var_G, $var_B);   //Min. value of RGB
+    $var_Max = max($var_R, $var_G, $var_B);   //Max. value of RGB
+    $del_Max = $var_Max - $var_Min;           //Delta RGB value
+
+    $L = ($var_Max + $var_Min) / 2;
+
+    if ($del_Max == 0) {
+        //This is a gray, no chroma...
+        $H = 0; //HSL results = 0 ÷ 1
+        $S = 0;
+    } else {
+        // Chromatic data...
+        $S = $L < 0.5 ? $del_Max / ($var_Max + $var_Min) : $del_Max / (2 - $var_Max - $var_Min);
+
+        $del_R = ((($var_Max - $var_R) / 6) + ($del_Max / 2)) / $del_Max;
+        $del_G = ((($var_Max - $var_G) / 6) + ($del_Max / 2)) / $del_Max;
+        $del_B = ((($var_Max - $var_B) / 6) + ($del_Max / 2)) / $del_Max;
+
+        if ($var_R === $var_Max) {
+            $H = $del_B - $del_G;
+        } else {
+            if ($var_G === $var_Max) {
+                $H = (1 / 3) + $del_R - $del_B;
+            } else {
+                if ($var_B === $var_Max) {
+                    $H = (2 / 3) + $del_G - $del_R;
+                }
+            }
+        }
+
+        if ($H < 0) {
+            $H += 1;
+        }
+        if ($H > 1) {
+            $H -= 1;
+        }
+    }
+
+    $ret = [];
+    $ret['h'] = $H;
+    $ret['s'] = $S;
+    $ret['l'] = $L;
+
+    return $ret;
 }
 
 
@@ -184,48 +184,48 @@ 
  * @return array
  */
 function _couleur_hsl_to_rgb($H, $S, $L) {
-	// helper
-	$hue_2_rgb = function ($v1, $v2, $vH) {
-		if ($vH < 0) {
-			$vH += 1;
-		}
-		if ($vH > 1) {
-			$vH -= 1;
-		}
-		if ((6 * $vH) < 1) {
-			return ($v1 + ($v2 - $v1) * 6 * $vH);
-		}
-		if ((2 * $vH) < 1) {
-			return ($v2);
-		}
-		if ((3 * $vH) < 2) {
-			return ($v1 + ($v2 - $v1) * ((2 / 3) - $vH) * 6);
-		}
-
-		return ($v1);
-	};
-
-	if ($S == 0) {
-		// HSV values = 0 -> 1
-		$R = $L * 255;
-		$G = $L * 255;
-		$B = $L * 255;
-	} else {
-		$var_2 = $L < 0.5 ? $L * (1 + $S) : ($L + $S) - ($S * $L);
-
-		$var_1 = 2 * $L - $var_2;
-
-		$R = 255 * $hue_2_rgb($var_1, $var_2, $H + (1 / 3));
-		$G = 255 * $hue_2_rgb($var_1, $var_2, $H);
-		$B = 255 * $hue_2_rgb($var_1, $var_2, $H - (1 / 3));
-	}
-
-	$ret = [];
-	$ret['r'] = floor($R);
-	$ret['g'] = floor($G);
-	$ret['b'] = floor($B);
-
-	return $ret;
+    // helper
+    $hue_2_rgb = function ($v1, $v2, $vH) {
+        if ($vH < 0) {
+            $vH += 1;
+        }
+        if ($vH > 1) {
+            $vH -= 1;
+        }
+        if ((6 * $vH) < 1) {
+            return ($v1 + ($v2 - $v1) * 6 * $vH);
+        }
+        if ((2 * $vH) < 1) {
+            return ($v2);
+        }
+        if ((3 * $vH) < 2) {
+            return ($v1 + ($v2 - $v1) * ((2 / 3) - $vH) * 6);
+        }
+
+        return ($v1);
+    };
+
+    if ($S == 0) {
+        // HSV values = 0 -> 1
+        $R = $L * 255;
+        $G = $L * 255;
+        $B = $L * 255;
+    } else {
+        $var_2 = $L < 0.5 ? $L * (1 + $S) : ($L + $S) - ($S * $L);
+
+        $var_1 = 2 * $L - $var_2;
+
+        $R = 255 * $hue_2_rgb($var_1, $var_2, $H + (1 / 3));
+        $G = 255 * $hue_2_rgb($var_1, $var_2, $H);
+        $B = 255 * $hue_2_rgb($var_1, $var_2, $H - (1 / 3));
+    }
+
+    $ret = [];
+    $ret['r'] = floor($R);
+    $ret['g'] = floor($G);
+    $ret['b'] = floor($B);
+
+    return $ret;
 }
 
 /**
@@ -243,11 +243,11 @@ 
  *     true si il faut supprimer le fichier temporaire ; false sinon.
  */
 function statut_effacer_images_temporaires($stat) {
-	static $statut = false; // par defaut on grave toute les images
-	if ($stat === 'get') {
-		return $statut;
-	}
-	$statut = (bool) $stat;
+    static $statut = false; // par defaut on grave toute les images
+    if ($stat === 'get') {
+        return $statut;
+    }
+    $statut = (bool) $stat;
 }
 
 
@@ -300,251 +300,251 @@ 
  *     - array : tableau décrivant de l'image
  */
 function _image_valeurs_trans($img, $effet, $forcer_format = false, $fonction_creation = null, $find_in_path = false, $support_svg = false) {
-	$valeurs = [];
-	$ret = [];
-	$f = null;
-	static $images_recalcul = [];
-	if (strlen($img) == 0) {
-		return false;
-	}
-
-	$source = trim(extraire_attribut($img, 'src') ?? '');
-	if (strlen($source) < 1) {
-		if (!str_starts_with($img, '<img ')) {
-			$source = $img;
-			$img = "<img src='$source' />";
-		} else {
-			// pas d'attribut src sur cette balise <img../>
-			return false;
-		}
-	} elseif (
-		preg_match('@^data:image/([^;]*);base64,(.*)$@isS', $source, $regs)
-		&& ($extension = _image_trouver_extension_depuis_mime('image/' . $regs[1]))
-		&& in_array($extension, _image_extensions_acceptees_en_entree())
-	) {
-		# gerer img src="data:....base64"
-		$local = sous_repertoire(_DIR_VAR, 'image-data') . md5($regs[2]) . '.' . _image_extension_normalisee($extension);
-		if (!file_exists($local)) {
-			ecrire_fichier($local, base64_decode($regs[2]));
-		}
-		if ($sanitizer = charger_fonction($extension, 'sanitizer', true)) {
-			$sanitizer($local);
-		}
-		$source = $local;
-		$img = inserer_attribut($img, 'src', $source);
-		# eviter les mauvaises surprises lors de conversions de format
-		$img = inserer_attribut($img, 'width', '');
-		$img = inserer_attribut($img, 'height', '');
-	}
-
-	// les protocoles web prennent au moins 3 lettres
-	if (tester_url_absolue($source)) {
-		include_spip('inc/distant');
-		$fichier = _DIR_RACINE . copie_locale($source);
-		if (!$fichier) {
-			return '';
-		}
-		if (
-			($extension = _image_trouver_extension($fichier))
-			&& ($sanitizer = charger_fonction($extension, 'sanitizer', true))
-		) {
-			$sanitizer($fichier);
-		}
-	} else {
-		// enlever le timestamp eventuel
-		if (str_contains($source, '?')) {
-			$source = preg_replace(',[?]\d+$,', '', $source);
-		}
-		if (
-			str_contains($source, '?')
-			&& str_starts_with($source, _DIR_IMG)
-			&& file_exists($f = preg_replace(',[?].*$,', '', $source))
-		) {
-			$source = $f;
-		}
-		$fichier = $source;
-	}
-
-	$terminaison_dest = '';
-	if ($terminaison = _image_trouver_extension($fichier)) {
-		$terminaison_dest = ($terminaison == 'gif') ? 'png' : $terminaison;
-	}
-
-	if (
-		$forcer_format !== false
-		&& ($terminaison_dest !== 'svg' || $support_svg || !in_array($forcer_format, _image_extensions_acceptees_en_sortie()))
-	) {
-		$terminaison_dest = $forcer_format;
-	}
-
-	if (!$terminaison_dest) {
-		return false;
-	}
-
-	$nom_fichier = substr($fichier, 0, strlen($fichier) - (strlen($terminaison) + 1));
-	$fichier_dest = $nom_fichier;
-	if (
-		$find_in_path && ($f = find_in_path($fichier)) && ($fichier = $f)
-		|| @file_exists($f = $fichier)
-	) {
-		// on ne passe jamais la balise img qui est peut-être en x2 et à laquelle on ne peut pas faire confiance
-		// on lit directement les infos du fichier
-		[$ret['hauteur'], $ret['largeur']] = taille_image($f);
-		$date_src = @filemtime($f);
-	} elseif (
-		@file_exists($f = "$fichier.src")
-		&& lire_fichier($f, $valeurs)
-		&& ($valeurs = unserialize($valeurs))
-		&& isset($valeurs['hauteur_dest'])
-		&& isset($valeurs['largeur_dest'])
-	) {
-		$ret['hauteur'] = $valeurs['hauteur_dest'];
-		$ret['largeur'] = $valeurs['largeur_dest'];
-		$date_src = $valeurs['date'];
-	} // pas de fichier source par la
-	else {
-		return false;
-	}
-
-	// pas de taille mesurable ?
-	if (
-		!$ret['hauteur']
-		|| !($ret['hauteur'] = (int) round($ret['hauteur']))
-		|| !$ret['largeur']
-		|| !($ret['largeur'] = (int) round($ret['largeur']))
-	) {
-		return false;
-	}
-
-	// les images calculees dependent du chemin du fichier source
-	// pour une meme image source et un meme filtre on aboutira a 2 fichiers selon si l'appel est dans le public ou dans le prive
-	// ce n'est pas totalement optimal en terme de stockage, mais chaque image est associee a un fichier .src
-	// qui contient la methode de reconstrucion (le filtre + les arguments d'appel) et les arguments different entre prive et public
-	// la mise en commun du fichier image cree donc un bug et des problemes qui necessiteraient beaucoup de complexite de code
-	// alors que ca concerne peu de site au final
-	// la release de r23632+r23633+r23634 a provoque peu de remontee de bug attestant du peu de sites impactes
-	$identifiant = $fichier;
-
-	// cas general :
-	// on a un dossier cache commun et un nom de fichier qui varie avec l'effet
-	// cas particulier de reduire :
-	// un cache par dimension, et le nom de fichier est conserve, suffixe par la dimension aussi
-	$cache = 'cache-gd2';
-	if (str_starts_with($effet, 'reduire')) {
-		[, $maxWidth, $maxHeight] = explode('-', $effet);
-		[$destWidth, $destHeight] = _image_ratio($ret['largeur'], $ret['hauteur'], $maxWidth, $maxHeight);
-		$ret['largeur_dest'] = $destWidth;
-		$ret['hauteur_dest'] = $destHeight;
-		$effet = "L{$destWidth}xH$destHeight";
-		$cache = 'cache-vignettes';
-		$fichier_dest = basename($fichier_dest);
-		if (($ret['largeur'] <= $maxWidth) && ($ret['hauteur'] <= $maxHeight)) {
-			// on garde la terminaison initiale car image simplement copiee
-			// et on postfixe son nom avec un md5 du path
-			$terminaison_dest = $terminaison;
-			$fichier_dest .= '-' . substr(md5("$identifiant"), 0, 5);
-		} else {
-			$fichier_dest .= '-' . substr(md5("$identifiant-$effet"), 0, 5);
-		}
-		$cache = sous_repertoire(_DIR_VAR, $cache);
-		$cache = sous_repertoire($cache, $effet);
-	} else {
-		$fichier_dest = md5("$identifiant-$effet");
-		$cache = sous_repertoire(_DIR_VAR, $cache);
-		$cache = sous_repertoire($cache, substr($fichier_dest, 0, 2));
-		$fichier_dest = substr($fichier_dest, 2);
-	}
-
-	$fichier_dest = $cache . $fichier_dest . '.' . $terminaison_dest;
-
-	$GLOBALS['images_calculees'][] = $fichier_dest;
-
-	$creer = true;
-	// si recalcul des images demande, recalculer chaque image une fois
-	if (defined('_VAR_IMAGES') && _VAR_IMAGES && !isset($images_recalcul[$fichier_dest])) {
-		$images_recalcul[$fichier_dest] = true;
-	} else {
-		if (@file_exists($f = $fichier_dest)) {
-			if (filemtime($f) >= $date_src) {
-				$creer = false;
-			}
-		} else {
-			if (
-				@file_exists($f = "$fichier_dest.src")
-				&& lire_fichier($f, $valeurs)
-				&& ($valeurs = unserialize($valeurs))
-				&& $valeurs['date'] >= $date_src
-			) {
-				$creer = false;
-			}
-		}
-	}
-	if ($creer && !@file_exists($fichier)) {
-		if (!@file_exists("$fichier.src")) {
-			spip_log("Image absente : $fichier", 'images' . _LOG_ERREUR);
-
-			return false;
-		}
-		# on reconstruit l'image source absente a partir de la chaine des .src
-		reconstruire_image_intermediaire($fichier);
-	}
-
-	if ($creer) {
-		spip_log(
-			'filtre image ' . ($fonction_creation ? reset($fonction_creation) : '') . "[$effet] sur $fichier",
-			'images' . _LOG_DEBUG
-		);
-	}
-
-	$term_fonction = _image_trouver_extension_pertinente($fichier);
-	$ret['fonction_imagecreatefrom'] = '_imagecreatefrom' . $term_fonction;
-	$ret['fichier'] = $fichier;
-	$ret['fonction_image'] = '_image_image' . $terminaison_dest;
-	$ret['fichier_dest'] = $fichier_dest;
-	$ret['format_source'] = _image_extension_normalisee($terminaison);
-	$ret['format_dest'] = $terminaison_dest;
-	$ret['date_src'] = $date_src;
-	$ret['creer'] = $creer;
-	$ret['class'] = extraire_attribut($img, 'class');
-	$ret['alt'] = extraire_attribut($img, 'alt');
-	$ret['style'] = extraire_attribut($img, 'style');
-	$ret['tag'] = $img;
-	if ($fonction_creation) {
-		$ret['reconstruction'] = $fonction_creation;
-		# ecrire ici comment creer le fichier, car il est pas sur qu'on l'ecrira reelement
-		# cas de image_reduire qui finalement ne reduit pas l'image source
-		# ca evite d'essayer de le creer au prochain hit si il n'est pas la
-		#ecrire_fichier($ret['fichier_dest'].'.src',serialize($ret),true);
-	}
-
-	$ret = pipeline('image_preparer_filtre', [
-			'args' => [
-				'img' => $img,
-				'effet' => $effet,
-				'forcer_format' => $forcer_format,
-				'fonction_creation' => $fonction_creation,
-				'find_in_path' => $find_in_path,
-			],
-			'data' => $ret
-		]);
-
-	// une globale pour le debug en cas de crash memoire
-	$GLOBALS['derniere_image_calculee'] = $ret;
-
-	// traiter le cas particulier des SVG : si le filtre n'a pas annonce explicitement qu'il savait faire, on delegue
-	if ($term_fonction === 'svg') {
-		if ($creer && !$support_svg) {
-			process_image_svg_identite($ret);
-			$ret['creer'] = false;
-		}
-	}
-	else {
-		if (!function_exists($ret['fonction_imagecreatefrom'])) {
-			return false;
-		}
-	}
-
-	return $ret;
+    $valeurs = [];
+    $ret = [];
+    $f = null;
+    static $images_recalcul = [];
+    if (strlen($img) == 0) {
+        return false;
+    }
+
+    $source = trim(extraire_attribut($img, 'src') ?? '');
+    if (strlen($source) < 1) {
+        if (!str_starts_with($img, '<img ')) {
+            $source = $img;
+            $img = "<img src='$source' />";
+        } else {
+            // pas d'attribut src sur cette balise <img../>
+            return false;
+        }
+    } elseif (
+        preg_match('@^data:image/([^;]*);base64,(.*)$@isS', $source, $regs)
+        && ($extension = _image_trouver_extension_depuis_mime('image/' . $regs[1]))
+        && in_array($extension, _image_extensions_acceptees_en_entree())
+    ) {
+        # gerer img src="data:....base64"
+        $local = sous_repertoire(_DIR_VAR, 'image-data') . md5($regs[2]) . '.' . _image_extension_normalisee($extension);
+        if (!file_exists($local)) {
+            ecrire_fichier($local, base64_decode($regs[2]));
+        }
+        if ($sanitizer = charger_fonction($extension, 'sanitizer', true)) {
+            $sanitizer($local);
+        }
+        $source = $local;
+        $img = inserer_attribut($img, 'src', $source);
+        # eviter les mauvaises surprises lors de conversions de format
+        $img = inserer_attribut($img, 'width', '');
+        $img = inserer_attribut($img, 'height', '');
+    }
+
+    // les protocoles web prennent au moins 3 lettres
+    if (tester_url_absolue($source)) {
+        include_spip('inc/distant');
+        $fichier = _DIR_RACINE . copie_locale($source);
+        if (!$fichier) {
+            return '';
+        }
+        if (
+            ($extension = _image_trouver_extension($fichier))
+            && ($sanitizer = charger_fonction($extension, 'sanitizer', true))
+        ) {
+            $sanitizer($fichier);
+        }
+    } else {
+        // enlever le timestamp eventuel
+        if (str_contains($source, '?')) {
+            $source = preg_replace(',[?]\d+$,', '', $source);
+        }
+        if (
+            str_contains($source, '?')
+            && str_starts_with($source, _DIR_IMG)
+            && file_exists($f = preg_replace(',[?].*$,', '', $source))
+        ) {
+            $source = $f;
+        }
+        $fichier = $source;
+    }
+
+    $terminaison_dest = '';
+    if ($terminaison = _image_trouver_extension($fichier)) {
+        $terminaison_dest = ($terminaison == 'gif') ? 'png' : $terminaison;
+    }
+
+    if (
+        $forcer_format !== false
+        && ($terminaison_dest !== 'svg' || $support_svg || !in_array($forcer_format, _image_extensions_acceptees_en_sortie()))
+    ) {
+        $terminaison_dest = $forcer_format;
+    }
+
+    if (!$terminaison_dest) {
+        return false;
+    }
+
+    $nom_fichier = substr($fichier, 0, strlen($fichier) - (strlen($terminaison) + 1));
+    $fichier_dest = $nom_fichier;
+    if (
+        $find_in_path && ($f = find_in_path($fichier)) && ($fichier = $f)
+        || @file_exists($f = $fichier)
+    ) {
+        // on ne passe jamais la balise img qui est peut-être en x2 et à laquelle on ne peut pas faire confiance
+        // on lit directement les infos du fichier
+        [$ret['hauteur'], $ret['largeur']] = taille_image($f);
+        $date_src = @filemtime($f);
+    } elseif (
+        @file_exists($f = "$fichier.src")
+        && lire_fichier($f, $valeurs)
+        && ($valeurs = unserialize($valeurs))
+        && isset($valeurs['hauteur_dest'])
+        && isset($valeurs['largeur_dest'])
+    ) {
+        $ret['hauteur'] = $valeurs['hauteur_dest'];
+        $ret['largeur'] = $valeurs['largeur_dest'];
+        $date_src = $valeurs['date'];
+    } // pas de fichier source par la
+    else {
+        return false;
+    }
+
+    // pas de taille mesurable ?
+    if (
+        !$ret['hauteur']
+        || !($ret['hauteur'] = (int) round($ret['hauteur']))
+        || !$ret['largeur']
+        || !($ret['largeur'] = (int) round($ret['largeur']))
+    ) {
+        return false;
+    }
+
+    // les images calculees dependent du chemin du fichier source
+    // pour une meme image source et un meme filtre on aboutira a 2 fichiers selon si l'appel est dans le public ou dans le prive
+    // ce n'est pas totalement optimal en terme de stockage, mais chaque image est associee a un fichier .src
+    // qui contient la methode de reconstrucion (le filtre + les arguments d'appel) et les arguments different entre prive et public
+    // la mise en commun du fichier image cree donc un bug et des problemes qui necessiteraient beaucoup de complexite de code
+    // alors que ca concerne peu de site au final
+    // la release de r23632+r23633+r23634 a provoque peu de remontee de bug attestant du peu de sites impactes
+    $identifiant = $fichier;
+
+    // cas general :
+    // on a un dossier cache commun et un nom de fichier qui varie avec l'effet
+    // cas particulier de reduire :
+    // un cache par dimension, et le nom de fichier est conserve, suffixe par la dimension aussi
+    $cache = 'cache-gd2';
+    if (str_starts_with($effet, 'reduire')) {
+        [, $maxWidth, $maxHeight] = explode('-', $effet);
+        [$destWidth, $destHeight] = _image_ratio($ret['largeur'], $ret['hauteur'], $maxWidth, $maxHeight);
+        $ret['largeur_dest'] = $destWidth;
+        $ret['hauteur_dest'] = $destHeight;
+        $effet = "L{$destWidth}xH$destHeight";
+        $cache = 'cache-vignettes';
+        $fichier_dest = basename($fichier_dest);
+        if (($ret['largeur'] <= $maxWidth) && ($ret['hauteur'] <= $maxHeight)) {
+            // on garde la terminaison initiale car image simplement copiee
+            // et on postfixe son nom avec un md5 du path
+            $terminaison_dest = $terminaison;
+            $fichier_dest .= '-' . substr(md5("$identifiant"), 0, 5);
+        } else {
+            $fichier_dest .= '-' . substr(md5("$identifiant-$effet"), 0, 5);
+        }
+        $cache = sous_repertoire(_DIR_VAR, $cache);
+        $cache = sous_repertoire($cache, $effet);
+    } else {
+        $fichier_dest = md5("$identifiant-$effet");
+        $cache = sous_repertoire(_DIR_VAR, $cache);
+        $cache = sous_repertoire($cache, substr($fichier_dest, 0, 2));
+        $fichier_dest = substr($fichier_dest, 2);
+    }
+
+    $fichier_dest = $cache . $fichier_dest . '.' . $terminaison_dest;
+
+    $GLOBALS['images_calculees'][] = $fichier_dest;
+
+    $creer = true;
+    // si recalcul des images demande, recalculer chaque image une fois
+    if (defined('_VAR_IMAGES') && _VAR_IMAGES && !isset($images_recalcul[$fichier_dest])) {
+        $images_recalcul[$fichier_dest] = true;
+    } else {
+        if (@file_exists($f = $fichier_dest)) {
+            if (filemtime($f) >= $date_src) {
+                $creer = false;
+            }
+        } else {
+            if (
+                @file_exists($f = "$fichier_dest.src")
+                && lire_fichier($f, $valeurs)
+                && ($valeurs = unserialize($valeurs))
+                && $valeurs['date'] >= $date_src
+            ) {
+                $creer = false;
+            }
+        }
+    }
+    if ($creer && !@file_exists($fichier)) {
+        if (!@file_exists("$fichier.src")) {
+            spip_log("Image absente : $fichier", 'images' . _LOG_ERREUR);
+
+            return false;
+        }
+        # on reconstruit l'image source absente a partir de la chaine des .src
+        reconstruire_image_intermediaire($fichier);
+    }
+
+    if ($creer) {
+        spip_log(
+            'filtre image ' . ($fonction_creation ? reset($fonction_creation) : '') . "[$effet] sur $fichier",
+            'images' . _LOG_DEBUG
+        );
+    }
+
+    $term_fonction = _image_trouver_extension_pertinente($fichier);
+    $ret['fonction_imagecreatefrom'] = '_imagecreatefrom' . $term_fonction;
+    $ret['fichier'] = $fichier;
+    $ret['fonction_image'] = '_image_image' . $terminaison_dest;
+    $ret['fichier_dest'] = $fichier_dest;
+    $ret['format_source'] = _image_extension_normalisee($terminaison);
+    $ret['format_dest'] = $terminaison_dest;
+    $ret['date_src'] = $date_src;
+    $ret['creer'] = $creer;
+    $ret['class'] = extraire_attribut($img, 'class');
+    $ret['alt'] = extraire_attribut($img, 'alt');
+    $ret['style'] = extraire_attribut($img, 'style');
+    $ret['tag'] = $img;
+    if ($fonction_creation) {
+        $ret['reconstruction'] = $fonction_creation;
+        # ecrire ici comment creer le fichier, car il est pas sur qu'on l'ecrira reelement
+        # cas de image_reduire qui finalement ne reduit pas l'image source
+        # ca evite d'essayer de le creer au prochain hit si il n'est pas la
+        #ecrire_fichier($ret['fichier_dest'].'.src',serialize($ret),true);
+    }
+
+    $ret = pipeline('image_preparer_filtre', [
+            'args' => [
+                'img' => $img,
+                'effet' => $effet,
+                'forcer_format' => $forcer_format,
+                'fonction_creation' => $fonction_creation,
+                'find_in_path' => $find_in_path,
+            ],
+            'data' => $ret
+        ]);
+
+    // une globale pour le debug en cas de crash memoire
+    $GLOBALS['derniere_image_calculee'] = $ret;
+
+    // traiter le cas particulier des SVG : si le filtre n'a pas annonce explicitement qu'il savait faire, on delegue
+    if ($term_fonction === 'svg') {
+        if ($creer && !$support_svg) {
+            process_image_svg_identite($ret);
+            $ret['creer'] = false;
+        }
+    }
+    else {
+        if (!function_exists($ret['fonction_imagecreatefrom'])) {
+            return false;
+        }
+    }
+
+    return $ret;
 }
 
 
@@ -553,54 +553,54 @@ 
  * @return array
  */
 function _image_extensions_acceptees_en_entree() {
-	static $extensions = null;
-	if (empty($extensions)) {
-		$extensions = ['png', 'gif', 'jpg', 'jpeg'];
-		if (!empty($GLOBALS['meta']['gd_formats'])) {
-			// action=tester renseigne gd_formats et detecte le support de webp
-			$extensions = array_merge(explode(',', $GLOBALS['meta']['gd_formats']));
-			$extensions = array_map('trim', $extensions);
-			$extensions = array_filter($extensions);
-			if (in_array('jpg', $extensions)) {
-				$extensions[] = 'jpeg';
-			}
-			$extensions = array_unique($extensions);
-		}
-		$extensions[] = 'svg'; // on le supporte toujours avec des fonctions specifiques
-	}
-
-	return $extensions;
+    static $extensions = null;
+    if (empty($extensions)) {
+        $extensions = ['png', 'gif', 'jpg', 'jpeg'];
+        if (!empty($GLOBALS['meta']['gd_formats'])) {
+            // action=tester renseigne gd_formats et detecte le support de webp
+            $extensions = array_merge(explode(',', $GLOBALS['meta']['gd_formats']));
+            $extensions = array_map('trim', $extensions);
+            $extensions = array_filter($extensions);
+            if (in_array('jpg', $extensions)) {
+                $extensions[] = 'jpeg';
+            }
+            $extensions = array_unique($extensions);
+        }
+        $extensions[] = 'svg'; // on le supporte toujours avec des fonctions specifiques
+    }
+
+    return $extensions;
 }
 
 /**
  * @return array|string[]|null
  */
 function _image_extensions_acceptees_en_sortie() {
-	static $extensions = null;
-	if (empty($extensions)) {
-		$extensions = _image_extensions_acceptees_en_entree();
-		$extensions = array_diff($extensions, ['jpeg']);
-		if (in_array('gif', $extensions) && !function_exists('imagegif')) {
-			$extensions = array_diff($extensions, ['gif']);
-		}
-		if (in_array('webp', $extensions) && !function_exists('imagewebp')) {
-			$extensions = array_diff($extensions, ['webp']);
-		}
-	}
-
-	return $extensions;
+    static $extensions = null;
+    if (empty($extensions)) {
+        $extensions = _image_extensions_acceptees_en_entree();
+        $extensions = array_diff($extensions, ['jpeg']);
+        if (in_array('gif', $extensions) && !function_exists('imagegif')) {
+            $extensions = array_diff($extensions, ['gif']);
+        }
+        if (in_array('webp', $extensions) && !function_exists('imagewebp')) {
+            $extensions = array_diff($extensions, ['webp']);
+        }
+    }
+
+    return $extensions;
 }
 
 function _image_extension_normalisee($extension) {
-	$extension = strtolower($extension);
-	if ($extension === 'jpeg') {
-		$extension = 'jpg';
-	}
-	return $extension;
+    $extension = strtolower($extension);
+    if ($extension === 'jpeg') {
+        $extension = 'jpg';
+    }
+    return $extension;
 }
 
 function _image_extensions_conservent_transparence() {
-	return ['png', 'webp'];
+    return ['png', 'webp'];
 }
 
 
@@ -610,11 +610,11 @@ 
  * @return string
  */
 function _image_trouver_extension($path) {
-	$preg_extensions = implode('|', _image_extensions_acceptees_en_entree());
-	if (preg_match(",\.($preg_extensions)($|[?]),i", $path, $regs)) {
-		return strtolower($regs[1]);
-	}
-	return '';
+    $preg_extensions = implode('|', _image_extensions_acceptees_en_entree());
+    if (preg_match(",\.($preg_extensions)($|[?]),i", $path, $regs)) {
+        return strtolower($regs[1]);
+    }
+    return '';
 }
 
 /**
@@ -625,28 +625,28 @@ 
  * @return string Extension, dans le format attendu par les fonctions 'gd' ('jpeg' pour les .jpg par exemple)
  */
 function _image_trouver_extension_pertinente($path) {
-	$path = supprimer_timestamp($path);
-	$terminaison = _image_trouver_extension($path);
-	if ($terminaison == 'jpg') {
-		$terminaison = 'jpeg';
-	}
-
-	if (!file_exists($path)) {
-		return $terminaison;
-	}
-
-	if (!$info = @spip_getimagesize($path)) {
-		return $terminaison;
-	}
-
-	$mime = $info['mime'] ?? image_type_to_mime_type($info[2]);
-
-	$_terminaison = _image_trouver_extension_depuis_mime($mime);
-	if ($_terminaison && $_terminaison !== $terminaison) {
-		spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.' . _LOG_INFO_IMPORTANTE);
-		$terminaison = $_terminaison;
-	}
-	return $terminaison;
+    $path = supprimer_timestamp($path);
+    $terminaison = _image_trouver_extension($path);
+    if ($terminaison == 'jpg') {
+        $terminaison = 'jpeg';
+    }
+
+    if (!file_exists($path)) {
+        return $terminaison;
+    }
+
+    if (!$info = @spip_getimagesize($path)) {
+        return $terminaison;
+    }
+
+    $mime = $info['mime'] ?? image_type_to_mime_type($info[2]);
+
+    $_terminaison = _image_trouver_extension_depuis_mime($mime);
+    if ($_terminaison && $_terminaison !== $terminaison) {
+        spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.' . _LOG_INFO_IMPORTANTE);
+        $terminaison = $_terminaison;
+    }
+    return $terminaison;
 }
 
 /**
@@ -654,14 +654,14 @@ 
  * @return string
  */
 function _image_trouver_extension_depuis_mime($mime) {
-	return match (strtolower($mime)) {
-		'image/png', 'image/x-png' => 'png',
-		'image/jpg', 'image/jpeg', 'image/pjpeg' => 'jpeg',
-		'image/gif' => 'gif',
-		'image/webp', 'image/x-webp' => 'webp',
-		'image/svg+xml' => 'svg',
-		default => '',
-	};
+    return match (strtolower($mime)) {
+        'image/png', 'image/x-png' => 'png',
+        'image/jpg', 'image/jpeg', 'image/pjpeg' => 'jpeg',
+        'image/gif' => 'gif',
+        'image/webp', 'image/x-webp' => 'webp',
+        'image/svg+xml' => 'svg',
+        default => '',
+    };
 }
 
 
@@ -681,18 +681,18 @@ 
  *     Une ressource de type Image GD.
  */
 function _imagecreatefrom_func(string $func, string $filename) {
-	if (!function_exists($func)) {
-		spip_log("GD indisponible : $func inexistante. Traitement $filename impossible.", _LOG_CRITIQUE);
-		erreur_squelette("GD indisponible : $func inexistante. Traitement $filename impossible.");
-		return null;
-	}
-	$img = @$func($filename);
-	if (!$img) {
-		spip_log("Erreur lecture imagecreatefromjpeg $filename", _LOG_CRITIQUE);
-		erreur_squelette("Erreur lecture imagecreatefromjpeg $filename");
-		$img = imagecreate(10, 10);
-	}
-	return $img;
+    if (!function_exists($func)) {
+        spip_log("GD indisponible : $func inexistante. Traitement $filename impossible.", _LOG_CRITIQUE);
+        erreur_squelette("GD indisponible : $func inexistante. Traitement $filename impossible.");
+        return null;
+    }
+    $img = @$func($filename);
+    if (!$img) {
+        spip_log("Erreur lecture imagecreatefromjpeg $filename", _LOG_CRITIQUE);
+        erreur_squelette("Erreur lecture imagecreatefromjpeg $filename");
+        $img = imagecreate(10, 10);
+    }
+    return $img;
 }
 
 /**
@@ -708,7 +708,7 @@ 
  *     Une ressource de type Image GD.
  */
 function _imagecreatefromjpeg($filename) {
-	return _imagecreatefrom_func('imagecreatefromjpeg', $filename);
+    return _imagecreatefrom_func('imagecreatefromjpeg', $filename);
 }
 
 /**
@@ -724,7 +724,7 @@ 
  *     Une ressource de type Image GD.
  */
 function _imagecreatefrompng($filename) {
-	return _imagecreatefrom_func('imagecreatefrompng', $filename);
+    return _imagecreatefrom_func('imagecreatefrompng', $filename);
 }
 
 /**
@@ -740,7 +740,7 @@ 
  *     Une ressource de type Image GD.
  */
 function _imagecreatefromgif($filename) {
-	return _imagecreatefrom_func('imagecreatefromgif', $filename);
+    return _imagecreatefrom_func('imagecreatefromgif', $filename);
 }
 
 
@@ -757,7 +757,7 @@ 
  *     Une ressource de type Image GD.
  */
 function _imagecreatefromwebp($filename) {
-	return _imagecreatefrom_func('imagecreatefromwebp', $filename);
+    return _imagecreatefrom_func('imagecreatefromwebp', $filename);
 }
 
 /**
@@ -775,24 +775,24 @@ 
  *     - true si une image est bien retournée.
  */
 function _image_imagepng($img, $fichier) {
-	if (!function_exists('imagepng')) {
-		return false;
-	}
-	$tmp = $fichier . '.tmp';
-	$ret = imagepng($img, $tmp);
-	if (file_exists($tmp)) {
-		$taille_test = @getimagesize($tmp);
-		if (empty($taille_test[0])) {
-			return false;
-		}
-
-		spip_unlink($fichier); // le fichier peut deja exister
-		@rename($tmp, $fichier);
-
-		return $ret;
-	}
-
-	return false;
+    if (!function_exists('imagepng')) {
+        return false;
+    }
+    $tmp = $fichier . '.tmp';
+    $ret = imagepng($img, $tmp);
+    if (file_exists($tmp)) {
+        $taille_test = @getimagesize($tmp);
+        if (empty($taille_test[0])) {
+            return false;
+        }
+
+        spip_unlink($fichier); // le fichier peut deja exister
+        @rename($tmp, $fichier);
+
+        return $ret;
+    }
+
+    return false;
 }
 
 /**
@@ -810,24 +810,24 @@ 
  *     - true si une image est bien retournée.
  */
 function _image_imagegif($img, $fichier) {
-	if (!function_exists('imagegif')) {
-		return false;
-	}
-	$tmp = $fichier . '.tmp';
-	$ret = imagegif($img, $tmp);
-	if (file_exists($tmp)) {
-		$taille_test = @getimagesize($tmp);
-		if (empty($taille_test[0])) {
-			return false;
-		}
-
-		spip_unlink($fichier); // le fichier peut deja exister
-		@rename($tmp, $fichier);
-
-		return $ret;
-	}
-
-	return false;
+    if (!function_exists('imagegif')) {
+        return false;
+    }
+    $tmp = $fichier . '.tmp';
+    $ret = imagegif($img, $tmp);
+    if (file_exists($tmp)) {
+        $taille_test = @getimagesize($tmp);
+        if (empty($taille_test[0])) {
+            return false;
+        }
+
+        spip_unlink($fichier); // le fichier peut deja exister
+        @rename($tmp, $fichier);
+
+        return $ret;
+    }
+
+    return false;
 }
 
 /**
@@ -850,29 +850,29 @@ 
  *     - true si une image est bien retournée.
  */
 function _image_imagejpg($img, $fichier, $qualite = _IMG_GD_QUALITE) {
-	if (!function_exists('imagejpeg')) {
-		return false;
-	}
-	$tmp = $fichier . '.tmp';
+    if (!function_exists('imagejpeg')) {
+        return false;
+    }
+    $tmp = $fichier . '.tmp';
 
-	// Enable interlancing
-	imageinterlace($img, true);
+    // Enable interlancing
+    imageinterlace($img, true);
 
-	$ret = imagejpeg($img, $tmp, $qualite);
+    $ret = imagejpeg($img, $tmp, $qualite);
 
-	if (file_exists($tmp)) {
-		$taille_test = @getimagesize($tmp);
-		if (empty($taille_test[0])) {
-			return false;
-		}
+    if (file_exists($tmp)) {
+        $taille_test = @getimagesize($tmp);
+        if (empty($taille_test[0])) {
+            return false;
+        }
 
-		spip_unlink($fichier); // le fichier peut deja exister
-		@rename($tmp, $fichier);
+        spip_unlink($fichier); // le fichier peut deja exister
+        @rename($tmp, $fichier);
 
-		return $ret;
-	}
+        return $ret;
+    }
 
-	return false;
+    return false;
 }
 
 /**
@@ -890,9 +890,9 @@ 
  *     true si le fichier a bien été créé ; false sinon.
  */
 function _image_imageico($img, $fichier) {
-	$gd_image_array = [$img];
+    $gd_image_array = [$img];
 
-	return ecrire_fichier($fichier, phpthumb_functions::GD2ICOstring($gd_image_array));
+    return ecrire_fichier($fichier, phpthumb_functions::GD2ICOstring($gd_image_array));
 }
 
 
@@ -911,24 +911,24 @@ 
  *     - true si une image est bien retournée.
  */
 function _image_imagewebp($img, $fichier, $qualite = _IMG_GD_QUALITE) {
-	if (!function_exists('imagewebp')) {
-		return false;
-	}
-	$tmp = $fichier . '.tmp';
-	$ret = imagewebp($img, $tmp, $qualite);
-	if (file_exists($tmp)) {
-		$taille_test = @getimagesize($tmp);
-		if (empty($taille_test[0])) {
-			return false;
-		}
-
-		spip_unlink($fichier); // le fichier peut deja exister
-		@rename($tmp, $fichier);
-
-		return $ret;
-	}
-
-	return false;
+    if (!function_exists('imagewebp')) {
+        return false;
+    }
+    $tmp = $fichier . '.tmp';
+    $ret = imagewebp($img, $tmp, $qualite);
+    if (file_exists($tmp)) {
+        $taille_test = @getimagesize($tmp);
+        if (empty($taille_test[0])) {
+            return false;
+        }
+
+        spip_unlink($fichier); // le fichier peut deja exister
+        @rename($tmp, $fichier);
+
+        return $ret;
+    }
+
+    return false;
 }
 
 /**
@@ -948,35 +948,35 @@ 
  */
 function _image_imagesvg($img, $fichier) {
 
-	$tmp = $fichier . '.tmp';
-	if (!str_contains($img, '<')) {
-		$img = supprimer_timestamp($img);
-		if (!file_exists($img)) {
-			return false;
-		}
-		@copy($img, $tmp);
-		if (filesize($tmp) === filesize($img)) {
-			spip_unlink($fichier); // le fichier peut deja exister
-			@rename($tmp, $fichier);
-			return true;
-		}
-		return false;
-	}
-
-	file_put_contents($tmp, $img);
-	if (file_exists($tmp)) {
-		$taille_test = spip_getimagesize($tmp);
-		if (empty($taille_test[0])) {
-			return false;
-		}
-
-		spip_unlink($fichier); // le fichier peut deja exister
-		@rename($tmp, $fichier);
-
-		return true;
-	}
-
-	return false;
+    $tmp = $fichier . '.tmp';
+    if (!str_contains($img, '<')) {
+        $img = supprimer_timestamp($img);
+        if (!file_exists($img)) {
+            return false;
+        }
+        @copy($img, $tmp);
+        if (filesize($tmp) === filesize($img)) {
+            spip_unlink($fichier); // le fichier peut deja exister
+            @rename($tmp, $fichier);
+            return true;
+        }
+        return false;
+    }
+
+    file_put_contents($tmp, $img);
+    if (file_exists($tmp)) {
+        $taille_test = spip_getimagesize($tmp);
+        if (empty($taille_test[0])) {
+            return false;
+        }
+
+        spip_unlink($fichier); // le fichier peut deja exister
+        @rename($tmp, $fichier);
+
+        return true;
+    }
+
+    return false;
 }
 
 
@@ -1004,30 +1004,30 @@ 
  *     - false sinon.
  */
 function _image_gd_output($img, $valeurs, $qualite = _IMG_GD_QUALITE, $fonction = null) {
-	if (is_null($fonction)) {
-		$fonction = '_image_image' . $valeurs['format_dest'];
-	}
-	$ret = false;
-	#un flag pour reperer les images gravees
-	$lock = (
-		!statut_effacer_images_temporaires('get')
-		|| @file_exists($valeurs['fichier_dest'])
-		&& !@file_exists($valeurs['fichier_dest'] . '.src')
-	);
-	if (
-		function_exists($fonction)
-		&& ($ret = $fonction($img, $valeurs['fichier_dest'], $qualite)) # on a reussi a creer l'image
-		&& isset($valeurs['reconstruction']) # et on sait comment la resonctruire le cas echeant
-		&& !$lock && @file_exists($valeurs['fichier_dest'])
-	) {
-		// dans tous les cas mettre a jour la taille de l'image finale
-		[$valeurs['hauteur_dest'], $valeurs['largeur_dest']] = taille_image($valeurs['fichier_dest']);
-		$valeurs['date'] = @filemtime($valeurs['fichier_dest']);
-		// pour la retrouver apres disparition
-		ecrire_fichier($valeurs['fichier_dest'] . '.src', serialize($valeurs), true);
-	}
-
-	return $ret;
+    if (is_null($fonction)) {
+        $fonction = '_image_image' . $valeurs['format_dest'];
+    }
+    $ret = false;
+    #un flag pour reperer les images gravees
+    $lock = (
+        !statut_effacer_images_temporaires('get')
+        || @file_exists($valeurs['fichier_dest'])
+        && !@file_exists($valeurs['fichier_dest'] . '.src')
+    );
+    if (
+        function_exists($fonction)
+        && ($ret = $fonction($img, $valeurs['fichier_dest'], $qualite)) # on a reussi a creer l'image
+        && isset($valeurs['reconstruction']) # et on sait comment la resonctruire le cas echeant
+        && !$lock && @file_exists($valeurs['fichier_dest'])
+    ) {
+        // dans tous les cas mettre a jour la taille de l'image finale
+        [$valeurs['hauteur_dest'], $valeurs['largeur_dest']] = taille_image($valeurs['fichier_dest']);
+        $valeurs['date'] = @filemtime($valeurs['fichier_dest']);
+        // pour la retrouver apres disparition
+        ecrire_fichier($valeurs['fichier_dest'] . '.src', serialize($valeurs), true);
+    }
+
+    return $ret;
 }
 
 /**
@@ -1040,28 +1040,28 @@ 
  *     Chemin vers le fichier manquant
  **/
 function reconstruire_image_intermediaire($fichier_manquant) {
-	$source = null;
-	$reconstruire = [];
-	$fichier = $fichier_manquant;
-	while (
-		!str_contains($fichier, '://')
-		&& !@file_exists($fichier)
-		&& lire_fichier($src = "$fichier.src", $source)
-		&& ($valeurs = unserialize($source))
-		&& ($fichier = $valeurs['fichier']) # l'origine est connue (on ne verifie pas son existence, qu'importe ...)
-	) {
-		spip_unlink($src); // si jamais on a un timeout pendant la reconstruction, elle se fera naturellement au hit suivant
-		$reconstruire[] = $valeurs['reconstruction'];
-	}
-	while (count($reconstruire)) {
-		$r = array_pop($reconstruire);
-		$fonction = $r[0];
-		$args = $r[1];
-		$fonction(...$args);
-	}
-	// cette image intermediaire est commune a plusieurs series de filtre, il faut la conserver
-	// mais l'on peut nettoyer les miettes de sa creation
-	ramasse_miettes($fichier_manquant);
+    $source = null;
+    $reconstruire = [];
+    $fichier = $fichier_manquant;
+    while (
+        !str_contains($fichier, '://')
+        && !@file_exists($fichier)
+        && lire_fichier($src = "$fichier.src", $source)
+        && ($valeurs = unserialize($source))
+        && ($fichier = $valeurs['fichier']) # l'origine est connue (on ne verifie pas son existence, qu'importe ...)
+    ) {
+        spip_unlink($src); // si jamais on a un timeout pendant la reconstruction, elle se fera naturellement au hit suivant
+        $reconstruire[] = $valeurs['reconstruction'];
+    }
+    while (count($reconstruire)) {
+        $r = array_pop($reconstruire);
+        $fonction = $r[0];
+        $args = $r[1];
+        $fonction(...$args);
+    }
+    // cette image intermediaire est commune a plusieurs series de filtre, il faut la conserver
+    // mais l'on peut nettoyer les miettes de sa creation
+    ramasse_miettes($fichier_manquant);
 }
 
 /**
@@ -1081,26 +1081,26 @@ 
  *     Chemin du fichier d'image calculé
  **/
 function ramasse_miettes($fichier) {
-	$source = null;
-	if (
-		str_contains($fichier, '://')
-		|| !lire_fichier($src = "$fichier.src", $source)
-		|| !$valeurs = unserialize($source)
-	) {
-		return;
-	}
-	spip_unlink($src); # on supprime la reference a sa source pour marquer cette image comme non intermediaire
-	while (
-		($fichier = $valeurs['fichier'])
-		&& str_starts_with($fichier, _DIR_VAR)
-		&& lire_fichier($src = "$fichier.src", $source)
-		&& ($valeurs = unserialize($source))  # et valide
-	) {
-		# on efface le fichier
-		spip_unlink($fichier);
-		# mais laisse le .src qui permet de savoir comment reconstruire l'image si besoin
-		#spip_unlink($src);
-	}
+    $source = null;
+    if (
+        str_contains($fichier, '://')
+        || !lire_fichier($src = "$fichier.src", $source)
+        || !$valeurs = unserialize($source)
+    ) {
+        return;
+    }
+    spip_unlink($src); # on supprime la reference a sa source pour marquer cette image comme non intermediaire
+    while (
+        ($fichier = $valeurs['fichier'])
+        && str_starts_with($fichier, _DIR_VAR)
+        && lire_fichier($src = "$fichier.src", $source)
+        && ($valeurs = unserialize($source))  # et valide
+    ) {
+        # on efface le fichier
+        spip_unlink($fichier);
+        # mais laisse le .src qui permet de savoir comment reconstruire l'image si besoin
+        #spip_unlink($src);
+    }
 }
 
 
@@ -1125,33 +1125,33 @@ 
  *     Code HTML de l'image
  **/
 function image_graver($img) {
-	// appeler le filtre post_image_filtrer qui permet de faire
-	// des traitements auto a la fin d'une serie de filtres
-	$img = pipeline('post_image_filtrer', $img);
-
-	$fichier_ori = $fichier = (extraire_attribut($img, 'src') ?? '');
-	if (($p = strpos($fichier, '?')) !== false) {
-		$fichier = substr($fichier, 0, $p);
-	}
-	if (strlen($fichier) < 1 && !str_starts_with($img, '<img ')) {
-		$fichier = $img;
-	}
-	if (strlen($fichier)) {
-		# si jamais le fichier final n'a pas ete calcule car suppose temporaire
-		# et qu'il ne s'agit pas d'une URL
-		if (!str_contains($fichier, '://') && !@file_exists($fichier)) {
-			reconstruire_image_intermediaire($fichier);
-		}
-		ramasse_miettes($fichier);
-
-		// ajouter le timestamp si besoin
-		if (!str_contains($fichier_ori, '?')) {
-			// on utilise str_replace pour attraper le onmouseover des logo si besoin
-			$img = str_replace($fichier_ori, timestamp($fichier_ori), $img);
-		}
-	}
-
-	return $img;
+    // appeler le filtre post_image_filtrer qui permet de faire
+    // des traitements auto a la fin d'une serie de filtres
+    $img = pipeline('post_image_filtrer', $img);
+
+    $fichier_ori = $fichier = (extraire_attribut($img, 'src') ?? '');
+    if (($p = strpos($fichier, '?')) !== false) {
+        $fichier = substr($fichier, 0, $p);
+    }
+    if (strlen($fichier) < 1 && !str_starts_with($img, '<img ')) {
+        $fichier = $img;
+    }
+    if (strlen($fichier)) {
+        # si jamais le fichier final n'a pas ete calcule car suppose temporaire
+        # et qu'il ne s'agit pas d'une URL
+        if (!str_contains($fichier, '://') && !@file_exists($fichier)) {
+            reconstruire_image_intermediaire($fichier);
+        }
+        ramasse_miettes($fichier);
+
+        // ajouter le timestamp si besoin
+        if (!str_contains($fichier_ori, '?')) {
+            // on utilise str_replace pour attraper le onmouseover des logo si besoin
+            $img = str_replace($fichier_ori, timestamp($fichier_ori), $img);
+        }
+    }
+
+    return $img;
 }
 
 /**
@@ -1178,32 +1178,32 @@ 
  *     Code html modifié de la balise.
  **/
 function _image_tag_changer_taille($tag, $width, $height, $style = false) {
-	if ($style === false) {
-		$style = extraire_attribut($tag, 'style');
-	}
-
-	// enlever le width et height du style
-	if ($style) {
-		$style = preg_replace(',(^|;)\s*(width|height)\s*:\s*[^;]+,ims', '', $style);
-	}
-	if ($style && $style[0] === ';') {
-		$style = substr($style, 1);
-	}
-
-	// mettre des attributs de width et height sur les images,
-	// ca accelere le rendu du navigateur
-	// ca permet aux navigateurs de reserver la bonne taille
-	// quand on a desactive l'affichage des images.
-	$tag = inserer_attribut($tag, 'width', round($width));
-	$tag = inserer_attribut($tag, 'height', round($height));
-
-	// attributs deprecies. Transformer en CSS
-	if ($espace = extraire_attribut($tag, 'hspace')) {
-		$style = "margin:{$espace}px;" . $style;
-		$tag = inserer_attribut($tag, 'hspace', '');
-	}
-
-	return inserer_attribut($tag, 'style', (string) $style, true, !(bool) $style);
+    if ($style === false) {
+        $style = extraire_attribut($tag, 'style');
+    }
+
+    // enlever le width et height du style
+    if ($style) {
+        $style = preg_replace(',(^|;)\s*(width|height)\s*:\s*[^;]+,ims', '', $style);
+    }
+    if ($style && $style[0] === ';') {
+        $style = substr($style, 1);
+    }
+
+    // mettre des attributs de width et height sur les images,
+    // ca accelere le rendu du navigateur
+    // ca permet aux navigateurs de reserver la bonne taille
+    // quand on a desactive l'affichage des images.
+    $tag = inserer_attribut($tag, 'width', round($width));
+    $tag = inserer_attribut($tag, 'height', round($height));
+
+    // attributs deprecies. Transformer en CSS
+    if ($espace = extraire_attribut($tag, 'hspace')) {
+        $style = "margin:{$espace}px;" . $style;
+        $tag = inserer_attribut($tag, 'hspace', '');
+    }
+
+    return inserer_attribut($tag, 'style', (string) $style, true, !(bool) $style);
 }
 
 
@@ -1229,70 +1229,70 @@ 
  *     Retourne le code HTML de l'image
  **/
 function _image_ecrire_tag($valeurs, $surcharge = []) {
-	$valeurs = pipeline('image_ecrire_tag_preparer', $valeurs);
-
-	// fermer les tags img pas bien fermes;
-	$tag = str_replace('>', '/>', str_replace('/>', '>', $valeurs['tag']));
-
-	// le style
-	$style = $valeurs['style'];
-	if (isset($surcharge['style'])) {
-		$style = $surcharge['style'];
-		unset($surcharge['style']);
-	}
-
-	// traiter specifiquement la largeur et la hauteur
-	$width = $valeurs['largeur'];
-	if (isset($surcharge['width'])) {
-		$width = $surcharge['width'];
-		unset($surcharge['width']);
-	}
-	$height = $valeurs['hauteur'];
-	if (isset($surcharge['height'])) {
-		$height = $surcharge['height'];
-		unset($surcharge['height']);
-	}
-
-	$tag = _image_tag_changer_taille($tag, $width, $height, $style);
-	// traiter specifiquement le src qui peut etre repris dans un onmouseout
-	// on remplace toute les ref a src dans le tag
-	$src = extraire_attribut($tag, 'src');
-	if (isset($surcharge['src'])) {
-		$tag = str_replace($src, $surcharge['src'], $tag);
-		// si il y a des & dans src, alors ils peuvent provenir d'un &amp
-		// pas garanti comme methode, mais mieux que rien
-		if (str_contains($src, '&')) {
-			$tag = str_replace(str_replace('&', '&amp;', $src), $surcharge['src'], $tag);
-		}
-		$src = $surcharge['src'];
-		unset($surcharge['src']);
-	}
-
-	$class = $valeurs['class'];
-	if (isset($surcharge['class'])) {
-		$class = $surcharge['class'];
-		unset($surcharge['class']);
-	}
-	if (is_scalar($class) && strlen($class)) {
-		$tag = inserer_attribut($tag, 'class', $class);
-	}
-
-	if ($surcharge !== []) {
-		foreach ($surcharge as $attribut => $valeur) {
-			$tag = inserer_attribut($tag, $attribut, $valeur);
-		}
-	}
-
-	return pipeline(
-		'image_ecrire_tag_finir',
-		[
-			'args' => [
-				'valeurs' => $valeurs,
-				'surcharge' => $surcharge,
-			],
-			'data' => $tag
-		]
-	);
+    $valeurs = pipeline('image_ecrire_tag_preparer', $valeurs);
+
+    // fermer les tags img pas bien fermes;
+    $tag = str_replace('>', '/>', str_replace('/>', '>', $valeurs['tag']));
+
+    // le style
+    $style = $valeurs['style'];
+    if (isset($surcharge['style'])) {
+        $style = $surcharge['style'];
+        unset($surcharge['style']);
+    }
+
+    // traiter specifiquement la largeur et la hauteur
+    $width = $valeurs['largeur'];
+    if (isset($surcharge['width'])) {
+        $width = $surcharge['width'];
+        unset($surcharge['width']);
+    }
+    $height = $valeurs['hauteur'];
+    if (isset($surcharge['height'])) {
+        $height = $surcharge['height'];
+        unset($surcharge['height']);
+    }
+
+    $tag = _image_tag_changer_taille($tag, $width, $height, $style);
+    // traiter specifiquement le src qui peut etre repris dans un onmouseout
+    // on remplace toute les ref a src dans le tag
+    $src = extraire_attribut($tag, 'src');
+    if (isset($surcharge['src'])) {
+        $tag = str_replace($src, $surcharge['src'], $tag);
+        // si il y a des & dans src, alors ils peuvent provenir d'un &amp
+        // pas garanti comme methode, mais mieux que rien
+        if (str_contains($src, '&')) {
+            $tag = str_replace(str_replace('&', '&amp;', $src), $surcharge['src'], $tag);
+        }
+        $src = $surcharge['src'];
+        unset($surcharge['src']);
+    }
+
+    $class = $valeurs['class'];
+    if (isset($surcharge['class'])) {
+        $class = $surcharge['class'];
+        unset($surcharge['class']);
+    }
+    if (is_scalar($class) && strlen($class)) {
+        $tag = inserer_attribut($tag, 'class', $class);
+    }
+
+    if ($surcharge !== []) {
+        foreach ($surcharge as $attribut => $valeur) {
+            $tag = inserer_attribut($tag, $attribut, $valeur);
+        }
+    }
+
+    return pipeline(
+        'image_ecrire_tag_finir',
+        [
+            'args' => [
+                'valeurs' => $valeurs,
+                'surcharge' => $surcharge,
+            ],
+            'data' => $tag
+        ]
+    );
 }
 
 /**
@@ -1315,268 +1315,268 @@ 
  *     Description de l'image, sinon null.
  **/
 function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO', $force = false) {
-	$srcHeight = null;
-	$retour = [];
-	// ordre de preference des formats graphiques pour creer les vignettes
-	// le premier format disponible, selon la methode demandee, est utilise
-	$image = $valeurs['fichier'];
-	$format = $valeurs['format_source'];
-	$destdir = dirname($valeurs['fichier_dest']);
-	$destfile = basename($valeurs['fichier_dest'], '.' . $valeurs['format_dest']);
-
-	$format_sortie = $valeurs['format_dest'];
-
-	if ($process == 'AUTO' && isset($GLOBALS['meta']['image_process'])) {
-		$process = $GLOBALS['meta']['image_process'];
-	}
-
-	// si le doc n'est pas une image dans un format accetpable, refuser
-	if (!$force && !in_array($format, formats_image_acceptables(in_array($process, ['gd1', 'gd2'])))) {
-		return;
-	}
-	$destination = "$destdir/$destfile";
-
-	// calculer la taille
-	if (($srcWidth = $valeurs['largeur']) && ($srcHeight = $valeurs['hauteur'])) {
-		if (!($destWidth = $valeurs['largeur_dest']) || !($destHeight = $valeurs['hauteur_dest'])) {
-			[$destWidth, $destHeight] = _image_ratio($srcWidth, $srcHeight, $maxWidth, $maxHeight);
-		}
-	} elseif ($process == 'convert' || $process == 'imagick') {
-		$destWidth = $maxWidth;
-		$destHeight = $maxHeight;
-	} else {
-		spip_log("echec $process sur $image");
-
-		return;
-	}
-
-	$vignette = '';
-
-	// Si l'image est de la taille demandee (ou plus petite), simplement la retourner
-	if ($srcWidth && $srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
-		$vignette = $destination . '.' . $format;
-		@copy($image, $vignette);
-	}
-
-	elseif ($valeurs['format_source'] === 'svg') {
-		include_spip('inc/svg');
-		if ($svg = svg_redimensionner($valeurs['fichier'], $destWidth, $destHeight)) {
-			$format_sortie = 'svg';
-			$vignette = $destination . '.' . $format_sortie;
-			$valeurs['fichier_dest'] = $vignette;
-			_image_gd_output($svg, $valeurs);
-		}
-	}
-
-	// 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
-		if (!defined('_RESIZE_COMMAND')) {
-			define('_RESIZE_COMMAND', _CONVERT_COMMAND . ' -quality ' . _IMG_CONVERT_QUALITE . ' -orient Undefined -resize %xx%y! %src %dest');
-		}
-		$vignette = $destination . '.' . $format_sortie;
-		$commande = str_replace(
-			['%x', '%y', '%src', '%dest'],
-			[
-				$destWidth,
-				$destHeight,
-				escapeshellcmd($image),
-				escapeshellcmd($vignette)
-			],
-			_RESIZE_COMMAND
-		);
-		spip_log($commande);
-		exec($commande);
-		if (!@file_exists($vignette)) {
-			spip_log("echec convert sur $vignette");
-
-			return;  // echec commande
-		}
-	}
-
-	// php5 imagemagick
-	elseif ($process == 'imagick') {
-		if (!class_exists(\Imagick::class)) {
-			spip_log('Classe Imagick absente !', _LOG_ERREUR);
-
-			return;
-		}
-
-		// chemin compatible Windows
-		$output = realpath(dirname($destination));
-		if (!$output) {
-			return;
-		}
-		$vignette = $output . DIRECTORY_SEPARATOR . basename($destination) . '.' . $format_sortie;
-
-		$imagick = new Imagick();
-		$imagick->readImage(realpath($image));
-		$imagick->resizeImage(
-			$destWidth,
-			$destHeight,
-			Imagick::FILTER_LANCZOS,
-			1
-		);//, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
-		$imagick->writeImage($vignette);
-
-		if (!@file_exists($vignette)) {
-			spip_log("echec imagick sur $vignette");
-
-			return;
-		}
-		// remettre le chemin relatif car c'est ce qu'attend SPIP pour la suite (en particlier action/tester)
-		$vignette = $destination . '.' . $format_sortie;
-	}
-
-	// 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;
-		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
-		if ($format == 'jpg') {
-			$jpegtopnm_command = str_replace('pnmscale', 'jpegtopnm', _PNMSCALE_COMMAND);
-			exec("$jpegtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
-			if (!($s = @filesize($vignette))) {
-				spip_unlink($vignette);
-			}
-			if (!@file_exists($vignette)) {
-				spip_log("echec netpbm-jpg sur $vignette");
-
-				return;
-			}
-		} else {
-			if ($format == 'gif') {
-				$giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
-				exec("$giftopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
-				if (!($s = @filesize($vignette))) {
-					spip_unlink($vignette);
-				}
-				if (!@file_exists($vignette)) {
-					spip_log("echec netpbm-gif sur $vignette");
-
-					return;
-				}
-			} else {
-				if ($format == 'png') {
-					$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
-					exec("$pngtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
-					if (!($s = @filesize($vignette))) {
-						spip_unlink($vignette);
-					}
-					if (!@file_exists($vignette)) {
-						spip_log("echec netpbm-png sur $vignette");
-
-						return;
-					}
-				}
-			}
-		}
-	}
-
-	// gd ou gd2
-	elseif ($process == 'gd1' || $process == 'gd2') {
-		if (!function_exists('gd_info')) {
-			spip_log('Librairie GD absente !', _LOG_ERREUR);
-
-			return;
-		}
-		if (_IMG_GD_MAX_PIXELS && $srcWidth * $srcHeight > _IMG_GD_MAX_PIXELS) {
-			spip_log('vignette gd1/gd2 impossible : ' . $srcWidth * $srcHeight . 'pixels');
-
-			return;
-		}
-		$destFormat = $format_sortie;
-		if (!$destFormat) {
-			spip_log("pas de format pour $image");
-
-			return;
-		}
-
-		$fonction_imagecreatefrom = $valeurs['fonction_imagecreatefrom'];
-		if (!function_exists($fonction_imagecreatefrom)) {
-			return;
-		}
-		$srcImage = @$fonction_imagecreatefrom($image);
-		if (!$srcImage) {
-			spip_log('echec gd1/gd2');
-
-			return;
-		}
-
-		// Initialisation de l'image destination
-		$destImage = null;
-		if ($process == 'gd2' && $destFormat != 'gif') {
-			$destImage = ImageCreateTrueColor($destWidth, $destHeight);
-		}
-		if (!$destImage) {
-			$destImage = ImageCreate($destWidth, $destHeight);
-		}
-
-		// Recopie de l'image d'origine avec adaptation de la taille
-		$ok = false;
-		if ($process == 'gd2' && function_exists('ImageCopyResampled')) {
-			if ($format == 'gif') {
-				// Si un GIF est transparent,
-				// fabriquer un PNG transparent
-				$transp = imagecolortransparent($srcImage);
-				if ($transp > 0) {
-					$destFormat = 'png';
-				}
-			}
-			if (in_array($destFormat, _image_extensions_conservent_transparence())) {
-				// Conserver la transparence
-				if (function_exists('imageAntiAlias')) {
-					imageAntiAlias($destImage, true);
-				}
-				@imagealphablending($destImage, false);
-				@imagesavealpha($destImage, true);
-			}
-			$ok = @ImageCopyResampled($destImage, $srcImage, 0, 0, 0, 0, $destWidth, $destHeight, $srcWidth, $srcHeight);
-		}
-		if (!$ok) {
-			$ok = ImageCopyResized($destImage, $srcImage, 0, 0, 0, 0, $destWidth, $destHeight, $srcWidth, $srcHeight);
-		}
-
-		// Sauvegarde de l'image destination
-		$valeurs['fichier_dest'] = $vignette = "$destination.$destFormat";
-		$valeurs['format_dest'] = $format = $destFormat;
-		_image_gd_output($destImage, $valeurs);
-
-		if ($srcImage) {
-			ImageDestroy($srcImage);
-		}
-		ImageDestroy($destImage);
-	}
-
-	if (!$vignette || !$size = @spip_getimagesize($vignette)) {
-		$size = [$destWidth, $destHeight];
-	}
-
-	// Gaffe: en safe mode, pas d'acces a la vignette,
-	// donc risque de balancer "width='0'", ce qui masque l'image sous MSIE
-	if ($size[0] < 1) {
-		$size[0] = $destWidth;
-	}
-	if ($size[1] < 1) {
-		$size[1] = $destHeight;
-	}
-
-	$retour['width'] = $largeur = $size[0];
-	$retour['height'] = $hauteur = $size[1];
-
-	$retour['fichier'] = $vignette;
-	$retour['format'] = $format;
-	$retour['date'] = @filemtime($vignette);
-
-	// renvoyer l'image
-	return $retour;
+    $srcHeight = null;
+    $retour = [];
+    // ordre de preference des formats graphiques pour creer les vignettes
+    // le premier format disponible, selon la methode demandee, est utilise
+    $image = $valeurs['fichier'];
+    $format = $valeurs['format_source'];
+    $destdir = dirname($valeurs['fichier_dest']);
+    $destfile = basename($valeurs['fichier_dest'], '.' . $valeurs['format_dest']);
+
+    $format_sortie = $valeurs['format_dest'];
+
+    if ($process == 'AUTO' && isset($GLOBALS['meta']['image_process'])) {
+        $process = $GLOBALS['meta']['image_process'];
+    }
+
+    // si le doc n'est pas une image dans un format accetpable, refuser
+    if (!$force && !in_array($format, formats_image_acceptables(in_array($process, ['gd1', 'gd2'])))) {
+        return;
+    }
+    $destination = "$destdir/$destfile";
+
+    // calculer la taille
+    if (($srcWidth = $valeurs['largeur']) && ($srcHeight = $valeurs['hauteur'])) {
+        if (!($destWidth = $valeurs['largeur_dest']) || !($destHeight = $valeurs['hauteur_dest'])) {
+            [$destWidth, $destHeight] = _image_ratio($srcWidth, $srcHeight, $maxWidth, $maxHeight);
+        }
+    } elseif ($process == 'convert' || $process == 'imagick') {
+        $destWidth = $maxWidth;
+        $destHeight = $maxHeight;
+    } else {
+        spip_log("echec $process sur $image");
+
+        return;
+    }
+
+    $vignette = '';
+
+    // Si l'image est de la taille demandee (ou plus petite), simplement la retourner
+    if ($srcWidth && $srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
+        $vignette = $destination . '.' . $format;
+        @copy($image, $vignette);
+    }
+
+    elseif ($valeurs['format_source'] === 'svg') {
+        include_spip('inc/svg');
+        if ($svg = svg_redimensionner($valeurs['fichier'], $destWidth, $destHeight)) {
+            $format_sortie = 'svg';
+            $vignette = $destination . '.' . $format_sortie;
+            $valeurs['fichier_dest'] = $vignette;
+            _image_gd_output($svg, $valeurs);
+        }
+    }
+
+    // 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
+        if (!defined('_RESIZE_COMMAND')) {
+            define('_RESIZE_COMMAND', _CONVERT_COMMAND . ' -quality ' . _IMG_CONVERT_QUALITE . ' -orient Undefined -resize %xx%y! %src %dest');
+        }
+        $vignette = $destination . '.' . $format_sortie;
+        $commande = str_replace(
+            ['%x', '%y', '%src', '%dest'],
+            [
+                $destWidth,
+                $destHeight,
+                escapeshellcmd($image),
+                escapeshellcmd($vignette)
+            ],
+            _RESIZE_COMMAND
+        );
+        spip_log($commande);
+        exec($commande);
+        if (!@file_exists($vignette)) {
+            spip_log("echec convert sur $vignette");
+
+            return;  // echec commande
+        }
+    }
+
+    // php5 imagemagick
+    elseif ($process == 'imagick') {
+        if (!class_exists(\Imagick::class)) {
+            spip_log('Classe Imagick absente !', _LOG_ERREUR);
+
+            return;
+        }
+
+        // chemin compatible Windows
+        $output = realpath(dirname($destination));
+        if (!$output) {
+            return;
+        }
+        $vignette = $output . DIRECTORY_SEPARATOR . basename($destination) . '.' . $format_sortie;
+
+        $imagick = new Imagick();
+        $imagick->readImage(realpath($image));
+        $imagick->resizeImage(
+            $destWidth,
+            $destHeight,
+            Imagick::FILTER_LANCZOS,
+            1
+        );//, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
+        $imagick->writeImage($vignette);
+
+        if (!@file_exists($vignette)) {
+            spip_log("echec imagick sur $vignette");
+
+            return;
+        }
+        // remettre le chemin relatif car c'est ce qu'attend SPIP pour la suite (en particlier action/tester)
+        $vignette = $destination . '.' . $format_sortie;
+    }
+
+    // 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;
+        $pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
+        if ($format == 'jpg') {
+            $jpegtopnm_command = str_replace('pnmscale', 'jpegtopnm', _PNMSCALE_COMMAND);
+            exec("$jpegtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+            if (!($s = @filesize($vignette))) {
+                spip_unlink($vignette);
+            }
+            if (!@file_exists($vignette)) {
+                spip_log("echec netpbm-jpg sur $vignette");
+
+                return;
+            }
+        } else {
+            if ($format == 'gif') {
+                $giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
+                exec("$giftopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+                if (!($s = @filesize($vignette))) {
+                    spip_unlink($vignette);
+                }
+                if (!@file_exists($vignette)) {
+                    spip_log("echec netpbm-gif sur $vignette");
+
+                    return;
+                }
+            } else {
+                if ($format == 'png') {
+                    $pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
+                    exec("$pngtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+                    if (!($s = @filesize($vignette))) {
+                        spip_unlink($vignette);
+                    }
+                    if (!@file_exists($vignette)) {
+                        spip_log("echec netpbm-png sur $vignette");
+
+                        return;
+                    }
+                }
+            }
+        }
+    }
+
+    // gd ou gd2
+    elseif ($process == 'gd1' || $process == 'gd2') {
+        if (!function_exists('gd_info')) {
+            spip_log('Librairie GD absente !', _LOG_ERREUR);
+
+            return;
+        }
+        if (_IMG_GD_MAX_PIXELS && $srcWidth * $srcHeight > _IMG_GD_MAX_PIXELS) {
+            spip_log('vignette gd1/gd2 impossible : ' . $srcWidth * $srcHeight . 'pixels');
+
+            return;
+        }
+        $destFormat = $format_sortie;
+        if (!$destFormat) {
+            spip_log("pas de format pour $image");
+
+            return;
+        }
+
+        $fonction_imagecreatefrom = $valeurs['fonction_imagecreatefrom'];
+        if (!function_exists($fonction_imagecreatefrom)) {
+            return;
+        }
+        $srcImage = @$fonction_imagecreatefrom($image);
+        if (!$srcImage) {
+            spip_log('echec gd1/gd2');
+
+            return;
+        }
+
+        // Initialisation de l'image destination
+        $destImage = null;
+        if ($process == 'gd2' && $destFormat != 'gif') {
+            $destImage = ImageCreateTrueColor($destWidth, $destHeight);
+        }
+        if (!$destImage) {
+            $destImage = ImageCreate($destWidth, $destHeight);
+        }
+
+        // Recopie de l'image d'origine avec adaptation de la taille
+        $ok = false;
+        if ($process == 'gd2' && function_exists('ImageCopyResampled')) {
+            if ($format == 'gif') {
+                // Si un GIF est transparent,
+                // fabriquer un PNG transparent
+                $transp = imagecolortransparent($srcImage);
+                if ($transp > 0) {
+                    $destFormat = 'png';
+                }
+            }
+            if (in_array($destFormat, _image_extensions_conservent_transparence())) {
+                // Conserver la transparence
+                if (function_exists('imageAntiAlias')) {
+                    imageAntiAlias($destImage, true);
+                }
+                @imagealphablending($destImage, false);
+                @imagesavealpha($destImage, true);
+            }
+            $ok = @ImageCopyResampled($destImage, $srcImage, 0, 0, 0, 0, $destWidth, $destHeight, $srcWidth, $srcHeight);
+        }
+        if (!$ok) {
+            $ok = ImageCopyResized($destImage, $srcImage, 0, 0, 0, 0, $destWidth, $destHeight, $srcWidth, $srcHeight);
+        }
+
+        // Sauvegarde de l'image destination
+        $valeurs['fichier_dest'] = $vignette = "$destination.$destFormat";
+        $valeurs['format_dest'] = $format = $destFormat;
+        _image_gd_output($destImage, $valeurs);
+
+        if ($srcImage) {
+            ImageDestroy($srcImage);
+        }
+        ImageDestroy($destImage);
+    }
+
+    if (!$vignette || !$size = @spip_getimagesize($vignette)) {
+        $size = [$destWidth, $destHeight];
+    }
+
+    // Gaffe: en safe mode, pas d'acces a la vignette,
+    // donc risque de balancer "width='0'", ce qui masque l'image sous MSIE
+    if ($size[0] < 1) {
+        $size[0] = $destWidth;
+    }
+    if ($size[1] < 1) {
+        $size[1] = $destHeight;
+    }
+
+    $retour['width'] = $largeur = $size[0];
+    $retour['height'] = $hauteur = $size[1];
+
+    $retour['fichier'] = $vignette;
+    $retour['format'] = $format;
+    $retour['date'] = @filemtime($vignette);
+
+    // renvoyer l'image
+    return $retour;
 }
 
 /**
@@ -1596,25 +1596,25 @@ 
  * @return array Liste [ largeur, hauteur, ratio de réduction ]
  **/
 function _image_ratio(int $srcWidth, int $srcHeight, int $maxWidth, int $maxHeight): array {
-	$ratioWidth = $srcWidth / $maxWidth;
-	$ratioHeight = $srcHeight / $maxHeight;
-
-	if ($srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
-		$destWidth = $srcWidth;
-		$destHeight = $srcHeight;
-	} elseif ($ratioWidth < $ratioHeight) {
-		$destWidth = $srcWidth / $ratioHeight;
-		$destHeight = $maxHeight;
-	} else {
-		$destWidth = $maxWidth;
-		$destHeight = $srcHeight / $ratioWidth;
-	}
-
-	return [
-		(int) round($destWidth),
-		(int) round($destHeight),
-		max($ratioWidth, $ratioHeight)
-	];
+    $ratioWidth = $srcWidth / $maxWidth;
+    $ratioHeight = $srcHeight / $maxHeight;
+
+    if ($srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
+        $destWidth = $srcWidth;
+        $destHeight = $srcHeight;
+    } elseif ($ratioWidth < $ratioHeight) {
+        $destWidth = $srcWidth / $ratioHeight;
+        $destHeight = $maxHeight;
+    } else {
+        $destWidth = $maxWidth;
+        $destHeight = $srcHeight / $ratioWidth;
+    }
+
+    return [
+        (int) round($destWidth),
+        (int) round($destHeight),
+        max($ratioWidth, $ratioHeight)
+    ];
 }
 
 /**
@@ -1634,25 +1634,25 @@ 
  * @return array Liste [ largeur, hauteur, ratio de réduction ]
  **/
 function ratio_passe_partout(int $srcWidth, int $srcHeight, int $maxWidth, int $maxHeight): array {
-	$ratioWidth = $srcWidth / $maxWidth;
-	$ratioHeight = $srcHeight / $maxHeight;
-
-	if ($srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
-		$destWidth = $srcWidth;
-		$destHeight = $srcHeight;
-	} elseif ($ratioWidth > $ratioHeight) {
-		$destWidth = $srcWidth / $ratioHeight;
-		$destHeight = $maxHeight;
-	} else {
-		$destWidth = $maxWidth;
-		$destHeight = $srcHeight / $ratioWidth;
-	}
-
-	return [
-		(int) round($destWidth),
-		(int) round($destHeight),
-		min($ratioWidth, $ratioHeight)
-	];
+    $ratioWidth = $srcWidth / $maxWidth;
+    $ratioHeight = $srcHeight / $maxHeight;
+
+    if ($srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
+        $destWidth = $srcWidth;
+        $destHeight = $srcHeight;
+    } elseif ($ratioWidth > $ratioHeight) {
+        $destWidth = $srcWidth / $ratioHeight;
+        $destHeight = $maxHeight;
+    } else {
+        $destWidth = $maxWidth;
+        $destHeight = $srcHeight / $ratioWidth;
+    }
+
+    return [
+        (int) round($destWidth),
+        (int) round($destHeight),
+        min($ratioWidth, $ratioHeight)
+    ];
 }
 
 
@@ -1665,12 +1665,12 @@ 
  * @return string
  */
 function process_image_svg_identite($image) {
-	if ($image['creer']) {
-		$source = $image['fichier'];
-		_image_gd_output($source, $image);
-	}
+    if ($image['creer']) {
+        $source = $image['fichier'];
+        _image_gd_output($source, $image);
+    }
 
-	return _image_ecrire_tag($image, ['src' => $image['fichier_dest']]);
+    return _image_ecrire_tag($image, ['src' => $image['fichier_dest']]);
 }
 
 
@@ -1703,101 +1703,101 @@ 
  *     Code HTML de la balise img produite
  **/
 function process_image_reduire($fonction, $img, $taille, $taille_y, $force, $process = 'AUTO') {
-	$image = false;
-	if ($process == 'AUTO' && isset($GLOBALS['meta']['image_process'])) {
-		$process = $GLOBALS['meta']['image_process'];
-	}
-	# determiner le format de sortie
-	$format_sortie = false; // le choix par defaut sera bon
-	if ($process == 'netpbm') {
-		$format_sortie = 'jpg';
-	} elseif ($process == 'gd1' || $process == 'gd2') {
-		$image = _image_valeurs_trans($img, "reduire-{$taille}-{$taille_y}", $format_sortie, $fonction, false, _SVG_SUPPORTED);
-		// on verifie que l'extension choisie est bonne (en principe oui)
-		$gd_formats = formats_image_acceptables(true);
-		if (
-			is_array($image) && (!in_array($image['format_dest'], $gd_formats) || !in_array($image['format_dest'], _image_extensions_acceptees_en_sortie()))
-		) {
-			$formats_sortie = $image['format_source'] == 'jpg' ? ['jpg', 'png', 'gif'] : ['png', 'jpg', 'gif'];
-			// Choisir le format destination
-			// - on sauve de preference en JPEG (meilleure compression)
-			// - pour le GIF : les GD recentes peuvent le lire mais pas l'ecrire
-			# bug : gd_formats contient la liste des fichiers qu'on sait *lire*,
-			# pas *ecrire*
-			$format_sortie = '';
-			foreach ($formats_sortie as $fmt) {
-				if (in_array($fmt, $gd_formats) && in_array($fmt, _image_extensions_acceptees_en_sortie())) {
-					$format_sortie = $fmt;
-					break;
-				}
-			}
-			$image = false;
-		}
-	}
-
-	if (!is_array($image)) {
-		$image = _image_valeurs_trans($img, "reduire-{$taille}-{$taille_y}", $format_sortie, $fonction, false, _SVG_SUPPORTED);
-	}
-
-	if (!is_array($image) || !$image['largeur'] || !$image['hauteur']) {
-		spip_log("image_reduire_src:pas de version locale de $img ou extension non prise en charge");
-		// on peut resizer en mode html si on dispose des elements
-		[$srcw, $srch] = taille_image($img);
-		if ($srcw && $srch) {
-			[$w, $h] = _image_ratio($srcw, $srch, $taille, $taille_y);
-
-			return _image_tag_changer_taille($img, $w, $h);
-		}
-		// la on n'a pas d'infos sur l'image source... on refile le truc a css
-		// sous la forme style='max-width: NNpx;'
-		return inserer_attribut(
-			$img,
-			'style',
-			"max-width: {$taille}px;max-width: min(100%,{$taille}px); max-height: {$taille_y}px"
-		);
-	}
-
-	// si l'image est plus petite que la cible retourner une copie cachee de l'image
-	if (($image['largeur'] <= $taille) && ($image['hauteur'] <= $taille_y)) {
-		if ($image['creer']) {
-			@copy($image['fichier'], $image['fichier_dest']);
-		}
-
-		return _image_ecrire_tag($image, ['src' => $image['fichier_dest']]);
-	}
-
-	if ($image['creer'] == false && !$force) {
-		return _image_ecrire_tag(
-			$image,
-			['src' => $image['fichier_dest'], 'width' => $image['largeur_dest'], 'height' => $image['hauteur_dest']]
-		);
-	}
-
-	if (in_array($image['format_source'], _image_extensions_acceptees_en_entree())) {
-		$destWidth = $image['largeur_dest'];
-		$destHeight = $image['hauteur_dest'];
-		$logo = $image['fichier'];
-		$date = $image['date_src'];
-		$preview = _image_creer_vignette($image, $taille, $taille_y, $process, $force);
-
-		if ($preview && $preview['fichier']) {
-			$logo = $preview['fichier'];
-			$destWidth = $preview['width'];
-			$destHeight = $preview['height'];
-			$date = $preview['date'];
-		}
-		// dans l'espace prive mettre un timestamp sur l'adresse
-		// de l'image, de facon a tromper le cache du navigateur
-		// quand on fait supprimer/reuploader un logo
-		// (pas de filemtime si SAFE MODE)
-		$date = test_espace_prive() ? ('?' . $date) : '';
-
-		return _image_ecrire_tag($image, ['src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight]);
-	}
-	else {
-		# BMP, tiff ... les redacteurs osent tout!
-		return $img;
-	}
+    $image = false;
+    if ($process == 'AUTO' && isset($GLOBALS['meta']['image_process'])) {
+        $process = $GLOBALS['meta']['image_process'];
+    }
+    # determiner le format de sortie
+    $format_sortie = false; // le choix par defaut sera bon
+    if ($process == 'netpbm') {
+        $format_sortie = 'jpg';
+    } elseif ($process == 'gd1' || $process == 'gd2') {
+        $image = _image_valeurs_trans($img, "reduire-{$taille}-{$taille_y}", $format_sortie, $fonction, false, _SVG_SUPPORTED);
+        // on verifie que l'extension choisie est bonne (en principe oui)
+        $gd_formats = formats_image_acceptables(true);
+        if (
+            is_array($image) && (!in_array($image['format_dest'], $gd_formats) || !in_array($image['format_dest'], _image_extensions_acceptees_en_sortie()))
+        ) {
+            $formats_sortie = $image['format_source'] == 'jpg' ? ['jpg', 'png', 'gif'] : ['png', 'jpg', 'gif'];
+            // Choisir le format destination
+            // - on sauve de preference en JPEG (meilleure compression)
+            // - pour le GIF : les GD recentes peuvent le lire mais pas l'ecrire
+            # bug : gd_formats contient la liste des fichiers qu'on sait *lire*,
+            # pas *ecrire*
+            $format_sortie = '';
+            foreach ($formats_sortie as $fmt) {
+                if (in_array($fmt, $gd_formats) && in_array($fmt, _image_extensions_acceptees_en_sortie())) {
+                    $format_sortie = $fmt;
+                    break;
+                }
+            }
+            $image = false;
+        }
+    }
+
+    if (!is_array($image)) {
+        $image = _image_valeurs_trans($img, "reduire-{$taille}-{$taille_y}", $format_sortie, $fonction, false, _SVG_SUPPORTED);
+    }
+
+    if (!is_array($image) || !$image['largeur'] || !$image['hauteur']) {
+        spip_log("image_reduire_src:pas de version locale de $img ou extension non prise en charge");
+        // on peut resizer en mode html si on dispose des elements
+        [$srcw, $srch] = taille_image($img);
+        if ($srcw && $srch) {
+            [$w, $h] = _image_ratio($srcw, $srch, $taille, $taille_y);
+
+            return _image_tag_changer_taille($img, $w, $h);
+        }
+        // la on n'a pas d'infos sur l'image source... on refile le truc a css
+        // sous la forme style='max-width: NNpx;'
+        return inserer_attribut(
+            $img,
+            'style',
+            "max-width: {$taille}px;max-width: min(100%,{$taille}px); max-height: {$taille_y}px"
+        );
+    }
+
+    // si l'image est plus petite que la cible retourner une copie cachee de l'image
+    if (($image['largeur'] <= $taille) && ($image['hauteur'] <= $taille_y)) {
+        if ($image['creer']) {
+            @copy($image['fichier'], $image['fichier_dest']);
+        }
+
+        return _image_ecrire_tag($image, ['src' => $image['fichier_dest']]);
+    }
+
+    if ($image['creer'] == false && !$force) {
+        return _image_ecrire_tag(
+            $image,
+            ['src' => $image['fichier_dest'], 'width' => $image['largeur_dest'], 'height' => $image['hauteur_dest']]
+        );
+    }
+
+    if (in_array($image['format_source'], _image_extensions_acceptees_en_entree())) {
+        $destWidth = $image['largeur_dest'];
+        $destHeight = $image['hauteur_dest'];
+        $logo = $image['fichier'];
+        $date = $image['date_src'];
+        $preview = _image_creer_vignette($image, $taille, $taille_y, $process, $force);
+
+        if ($preview && $preview['fichier']) {
+            $logo = $preview['fichier'];
+            $destWidth = $preview['width'];
+            $destHeight = $preview['height'];
+            $date = $preview['date'];
+        }
+        // dans l'espace prive mettre un timestamp sur l'adresse
+        // de l'image, de facon a tromper le cache du navigateur
+        // quand on fait supprimer/reuploader un logo
+        // (pas de filemtime si SAFE MODE)
+        $date = test_espace_prive() ? ('?' . $date) : '';
+
+        return _image_ecrire_tag($image, ['src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight]);
+    }
+    else {
+        # BMP, tiff ... les redacteurs osent tout!
+        return $img;
+    }
 }
 
 /**
@@ -1811,145 +1811,145 @@ 
  * Class phpthumb_functions
  */
 class phpthumb_functions {
-	/**
-	 * Retourne la couleur d'un pixel dans une image
-	 *
-	 * @param GdImage $img
-	 * @param int $x
-	 * @param int $y
-	 * @return array|bool
-	 */
-	public static function GetPixelColor(&$img, $x, $y) {
-		if ($img instanceof \GdImage) {
-			return @ImageColorsForIndex($img, @ImageColorAt($img, $x, $y));
-		}
-		return false;
-	}
-
-	/**
-	 * Retourne un nombre dans une représentation en Little Endian
-	 *
-	 * @param int $number
-	 * @param int $minbytes
-	 * @return string
-	 */
-	public static function LittleEndian2String($number, $minbytes = 1) {
-		$intstring = '';
-		while ($number > 0) {
-			$intstring .= chr($number & 255);
-			$number >>= 8;
-		}
-
-		return str_pad($intstring, $minbytes, "\x00", STR_PAD_RIGHT);
-	}
-
-	/**
-	 * Transforme une ressource GD en image au format ICO
-	 *
-	 * @param array $gd_image_array
-	 *     Tableau de ressources d'images GD
-	 * @return string
-	 *     Image au format ICO
-	 */
-	public static function GD2ICOstring(&$gd_image_array) {
-		foreach ($gd_image_array as $key => $gd_image) {
-			$ImageWidths[$key] = ImageSX($gd_image);
-			$ImageHeights[$key] = ImageSY($gd_image);
-			$bpp[$key] = ImageIsTrueColor($gd_image) ? 32 : 24;
-			$totalcolors[$key] = ImageColorsTotal($gd_image);
-
-			$icXOR[$key] = '';
-			for ($y = $ImageHeights[$key] - 1; $y >= 0; $y--) {
-				for ($x = 0; $x < $ImageWidths[$key]; $x++) {
-					$argb = phpthumb_functions::GetPixelColor($gd_image, $x, $y);
-					$a = round(255 * ((127 - $argb['alpha']) / 127));
-					$r = $argb['red'];
-					$g = $argb['green'];
-					$b = $argb['blue'];
-
-					if ($bpp[$key] == 32) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r) . chr($a);
-					} elseif ($bpp[$key] === 24) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r);
-					}
-
-					if ($a < 128) {
-						@$icANDmask[$key][$y] .= '1';
-					} else {
-						@$icANDmask[$key][$y] .= '0';
-					}
-				}
-				// mask bits are 32-bit aligned per scanline
-				while (strlen($icANDmask[$key][$y]) % 32) {
-					$icANDmask[$key][$y] .= '0';
-				}
-			}
-			$icAND[$key] = '';
-			foreach ($icANDmask[$key] as $y => $scanlinemaskbits) {
-				for ($i = 0; $i < strlen($scanlinemaskbits); $i += 8) {
-					$icAND[$key] .= chr(bindec(str_pad(substr($scanlinemaskbits, $i, 8), 8, '0', STR_PAD_LEFT)));
-				}
-			}
-		}
-
-		foreach (array_keys($gd_image_array) as $key) {
-			$biSizeImage = $ImageWidths[$key] * $ImageHeights[$key] * ($bpp[$key] / 8);
-
-			// BITMAPINFOHEADER - 40 bytes
-			$BitmapInfoHeader[$key] = '';
-			$BitmapInfoHeader[$key] .= "\x28\x00\x00\x00";                // DWORD  biSize;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);    // LONG   biWidth;
-			// The biHeight member specifies the combined
-			// height of the XOR and AND masks.
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;
-			$BitmapInfoHeader[$key] .= "\x01\x00";                    // WORD   biPlanes;
-			$BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";              // wBitCount;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biCompression;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);      // DWORD  biSizeImage;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biXPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biYPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrUsed;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrImportant;
-		}
-
-
-		$icondata = "\x00\x00";                    // idReserved;   // Reserved (must be 0)
-		$icondata .= "\x01\x00";                    // idType;	   // Resource Type (1 for icons)
-		$icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;	  // How many images?
-
-		$dwImageOffset = 6 + (count($gd_image_array) * 16);
-		foreach (array_keys($gd_image_array) as $key) {
-			// ICONDIRENTRY   idEntries[1]; // An entry for each image (idCount of 'em)
-
-			$icondata .= chr($ImageWidths[$key]);           // bWidth;		  // Width, in pixels, of the image
-			$icondata .= chr($ImageHeights[$key]);          // bHeight;		 // Height, in pixels, of the image
-			$icondata .= chr($totalcolors[$key]);           // bColorCount;	 // Number of colors in image (0 if >=8bpp)
-			$icondata .= "\x00";                    // bReserved;	   // Reserved ( must be 0)
-
-			$icondata .= "\x01\x00";                  // wPlanes;		 // Color Planes
-			$icondata .= chr($bpp[$key]) . "\x00";            // wBitCount;	   // Bits per pixel
-
-			$dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);
-			$icondata .= phpthumb_functions::LittleEndian2String(
-				$dwBytesInRes,
-				4
-			);     // dwBytesInRes;	// How many bytes in this resource?
-
-			$icondata .= phpthumb_functions::LittleEndian2String(
-				$dwImageOffset,
-				4
-			);    // dwImageOffset;   // Where in the file is this image?
-			$dwImageOffset += strlen($BitmapInfoHeader[$key]);
-			$dwImageOffset += strlen($icXOR[$key]);
-			$dwImageOffset += strlen($icAND[$key]);
-		}
-
-		foreach (array_keys($gd_image_array) as $key) {
-			$icondata .= $BitmapInfoHeader[$key];
-			$icondata .= $icXOR[$key];
-			$icondata .= $icAND[$key];
-		}
-
-		return $icondata;
-	}
+    /**
+     * Retourne la couleur d'un pixel dans une image
+     *
+     * @param GdImage $img
+     * @param int $x
+     * @param int $y
+     * @return array|bool
+     */
+    public static function GetPixelColor(&$img, $x, $y) {
+        if ($img instanceof \GdImage) {
+            return @ImageColorsForIndex($img, @ImageColorAt($img, $x, $y));
+        }
+        return false;
+    }
+
+    /**
+     * Retourne un nombre dans une représentation en Little Endian
+     *
+     * @param int $number
+     * @param int $minbytes
+     * @return string
+     */
+    public static function LittleEndian2String($number, $minbytes = 1) {
+        $intstring = '';
+        while ($number > 0) {
+            $intstring .= chr($number & 255);
+            $number >>= 8;
+        }
+
+        return str_pad($intstring, $minbytes, "\x00", STR_PAD_RIGHT);
+    }
+
+    /**
+     * Transforme une ressource GD en image au format ICO
+     *
+     * @param array $gd_image_array
+     *     Tableau de ressources d'images GD
+     * @return string
+     *     Image au format ICO
+     */
+    public static function GD2ICOstring(&$gd_image_array) {
+        foreach ($gd_image_array as $key => $gd_image) {
+            $ImageWidths[$key] = ImageSX($gd_image);
+            $ImageHeights[$key] = ImageSY($gd_image);
+            $bpp[$key] = ImageIsTrueColor($gd_image) ? 32 : 24;
+            $totalcolors[$key] = ImageColorsTotal($gd_image);
+
+            $icXOR[$key] = '';
+            for ($y = $ImageHeights[$key] - 1; $y >= 0; $y--) {
+                for ($x = 0; $x < $ImageWidths[$key]; $x++) {
+                    $argb = phpthumb_functions::GetPixelColor($gd_image, $x, $y);
+                    $a = round(255 * ((127 - $argb['alpha']) / 127));
+                    $r = $argb['red'];
+                    $g = $argb['green'];
+                    $b = $argb['blue'];
+
+                    if ($bpp[$key] == 32) {
+                        $icXOR[$key] .= chr($b) . chr($g) . chr($r) . chr($a);
+                    } elseif ($bpp[$key] === 24) {
+                        $icXOR[$key] .= chr($b) . chr($g) . chr($r);
+                    }
+
+                    if ($a < 128) {
+                        @$icANDmask[$key][$y] .= '1';
+                    } else {
+                        @$icANDmask[$key][$y] .= '0';
+                    }
+                }
+                // mask bits are 32-bit aligned per scanline
+                while (strlen($icANDmask[$key][$y]) % 32) {
+                    $icANDmask[$key][$y] .= '0';
+                }
+            }
+            $icAND[$key] = '';
+            foreach ($icANDmask[$key] as $y => $scanlinemaskbits) {
+                for ($i = 0; $i < strlen($scanlinemaskbits); $i += 8) {
+                    $icAND[$key] .= chr(bindec(str_pad(substr($scanlinemaskbits, $i, 8), 8, '0', STR_PAD_LEFT)));
+                }
+            }
+        }
+
+        foreach (array_keys($gd_image_array) as $key) {
+            $biSizeImage = $ImageWidths[$key] * $ImageHeights[$key] * ($bpp[$key] / 8);
+
+            // BITMAPINFOHEADER - 40 bytes
+            $BitmapInfoHeader[$key] = '';
+            $BitmapInfoHeader[$key] .= "\x28\x00\x00\x00";                // DWORD  biSize;
+            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);    // LONG   biWidth;
+            // The biHeight member specifies the combined
+            // height of the XOR and AND masks.
+            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;
+            $BitmapInfoHeader[$key] .= "\x01\x00";                    // WORD   biPlanes;
+            $BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";              // wBitCount;
+            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biCompression;
+            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);      // DWORD  biSizeImage;
+            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biXPelsPerMeter;
+            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biYPelsPerMeter;
+            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrUsed;
+            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrImportant;
+        }
+
+
+        $icondata = "\x00\x00";                    // idReserved;   // Reserved (must be 0)
+        $icondata .= "\x01\x00";                    // idType;	   // Resource Type (1 for icons)
+        $icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;	  // How many images?
+
+        $dwImageOffset = 6 + (count($gd_image_array) * 16);
+        foreach (array_keys($gd_image_array) as $key) {
+            // ICONDIRENTRY   idEntries[1]; // An entry for each image (idCount of 'em)
+
+            $icondata .= chr($ImageWidths[$key]);           // bWidth;		  // Width, in pixels, of the image
+            $icondata .= chr($ImageHeights[$key]);          // bHeight;		 // Height, in pixels, of the image
+            $icondata .= chr($totalcolors[$key]);           // bColorCount;	 // Number of colors in image (0 if >=8bpp)
+            $icondata .= "\x00";                    // bReserved;	   // Reserved ( must be 0)
+
+            $icondata .= "\x01\x00";                  // wPlanes;		 // Color Planes
+            $icondata .= chr($bpp[$key]) . "\x00";            // wBitCount;	   // Bits per pixel
+
+            $dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);
+            $icondata .= phpthumb_functions::LittleEndian2String(
+                $dwBytesInRes,
+                4
+            );     // dwBytesInRes;	// How many bytes in this resource?
+
+            $icondata .= phpthumb_functions::LittleEndian2String(
+                $dwImageOffset,
+                4
+            );    // dwImageOffset;   // Where in the file is this image?
+            $dwImageOffset += strlen($BitmapInfoHeader[$key]);
+            $dwImageOffset += strlen($icXOR[$key]);
+            $dwImageOffset += strlen($icAND[$key]);
+        }
+
+        foreach (array_keys($gd_image_array) as $key) {
+            $icondata .= $BitmapInfoHeader[$key];
+            $icondata .= $icXOR[$key];
+            $icondata .= $icAND[$key];
+        }
+
+        return $icondata;
+    }
 }

Spacing +67 added lines, -67 removed lines

@@ -42,13 +42,13 @@ 
 	$blue = dechex($blue);
 
 	if (strlen($red) == 1) {
-		$red = '0' . $red;
+		$red = '0'.$red;
 	}
 	if (strlen($green) == 1) {
-		$green = '0' . $green;
+		$green = '0'.$green;
 	}
 	if (strlen($blue) == 1) {
-		$blue = '0' . $blue;
+		$blue = '0'.$blue;
 	}
 
 	return "$red$green$blue";
@@ -67,7 +67,7 @@ 
 	$couleur = couleur_html_to_hex($couleur);
 	$couleur = ltrim($couleur, '#');
 	if (strlen($couleur) === 3) {
-		$couleur = $couleur[0] . $couleur[0] . $couleur[1] . $couleur[1] . $couleur[2] . $couleur[2];
+		$couleur = $couleur[0].$couleur[0].$couleur[1].$couleur[1].$couleur[2].$couleur[2];
 	}
 	$retour = [];
 	$retour['red'] = hexdec(substr($couleur, 0, 2));
@@ -125,9 +125,9 @@ 
 	$var_G = ($G / 255);
 	$var_B = ($B / 255);
 
-	$var_Min = min($var_R, $var_G, $var_B);   //Min. value of RGB
-	$var_Max = max($var_R, $var_G, $var_B);   //Max. value of RGB
-	$del_Max = $var_Max - $var_Min;           //Delta RGB value
+	$var_Min = min($var_R, $var_G, $var_B); //Min. value of RGB
+	$var_Max = max($var_R, $var_G, $var_B); //Max. value of RGB
+	$del_Max = $var_Max - $var_Min; //Delta RGB value
 
 	$L = ($var_Max + $var_Min) / 2;
 
@@ -185,7 +185,7 @@ 
  */
 function _couleur_hsl_to_rgb($H, $S, $L) {
 	// helper
-	$hue_2_rgb = function ($v1, $v2, $vH) {
+	$hue_2_rgb = function($v1, $v2, $vH) {
 		if ($vH < 0) {
 			$vH += 1;
 		}
@@ -319,11 +319,11 @@ 
 		}
 	} elseif (
 		preg_match('@^data:image/([^;]*);base64,(.*)$@isS', $source, $regs)
-		&& ($extension = _image_trouver_extension_depuis_mime('image/' . $regs[1]))
+		&& ($extension = _image_trouver_extension_depuis_mime('image/'.$regs[1]))
 		&& in_array($extension, _image_extensions_acceptees_en_entree())
 	) {
 		# gerer img src="data:....base64"
-		$local = sous_repertoire(_DIR_VAR, 'image-data') . md5($regs[2]) . '.' . _image_extension_normalisee($extension);
+		$local = sous_repertoire(_DIR_VAR, 'image-data').md5($regs[2]).'.'._image_extension_normalisee($extension);
 		if (!file_exists($local)) {
 			ecrire_fichier($local, base64_decode($regs[2]));
 		}
@@ -340,7 +340,7 @@ 
 	// les protocoles web prennent au moins 3 lettres
 	if (tester_url_absolue($source)) {
 		include_spip('inc/distant');
-		$fichier = _DIR_RACINE . copie_locale($source);
+		$fichier = _DIR_RACINE.copie_locale($source);
 		if (!$fichier) {
 			return '';
 		}
@@ -442,9 +442,9 @@ 
 			// on garde la terminaison initiale car image simplement copiee
 			// et on postfixe son nom avec un md5 du path
 			$terminaison_dest = $terminaison;
-			$fichier_dest .= '-' . substr(md5("$identifiant"), 0, 5);
+			$fichier_dest .= '-'.substr(md5("$identifiant"), 0, 5);
 		} else {
-			$fichier_dest .= '-' . substr(md5("$identifiant-$effet"), 0, 5);
+			$fichier_dest .= '-'.substr(md5("$identifiant-$effet"), 0, 5);
 		}
 		$cache = sous_repertoire(_DIR_VAR, $cache);
 		$cache = sous_repertoire($cache, $effet);
@@ -455,7 +455,7 @@ 
 		$fichier_dest = substr($fichier_dest, 2);
 	}
 
-	$fichier_dest = $cache . $fichier_dest . '.' . $terminaison_dest;
+	$fichier_dest = $cache.$fichier_dest.'.'.$terminaison_dest;
 
 	$GLOBALS['images_calculees'][] = $fichier_dest;
 
@@ -481,7 +481,7 @@ 
 	}
 	if ($creer && !@file_exists($fichier)) {
 		if (!@file_exists("$fichier.src")) {
-			spip_log("Image absente : $fichier", 'images' . _LOG_ERREUR);
+			spip_log("Image absente : $fichier", 'images'._LOG_ERREUR);
 
 			return false;
 		}
@@ -491,15 +491,15 @@ 
 
 	if ($creer) {
 		spip_log(
-			'filtre image ' . ($fonction_creation ? reset($fonction_creation) : '') . "[$effet] sur $fichier",
-			'images' . _LOG_DEBUG
+			'filtre image '.($fonction_creation ? reset($fonction_creation) : '')."[$effet] sur $fichier",
+			'images'._LOG_DEBUG
 		);
 	}
 
 	$term_fonction = _image_trouver_extension_pertinente($fichier);
-	$ret['fonction_imagecreatefrom'] = '_imagecreatefrom' . $term_fonction;
+	$ret['fonction_imagecreatefrom'] = '_imagecreatefrom'.$term_fonction;
 	$ret['fichier'] = $fichier;
-	$ret['fonction_image'] = '_image_image' . $terminaison_dest;
+	$ret['fonction_image'] = '_image_image'.$terminaison_dest;
 	$ret['fichier_dest'] = $fichier_dest;
 	$ret['format_source'] = _image_extension_normalisee($terminaison);
 	$ret['format_dest'] = $terminaison_dest;
@@ -643,7 +643,7 @@ 
 
 	$_terminaison = _image_trouver_extension_depuis_mime($mime);
 	if ($_terminaison && $_terminaison !== $terminaison) {
-		spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.'._LOG_INFO_IMPORTANTE);
 		$terminaison = $_terminaison;
 	}
 	return $terminaison;
@@ -778,7 +778,7 @@ 
 	if (!function_exists('imagepng')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	$ret = imagepng($img, $tmp);
 	if (file_exists($tmp)) {
 		$taille_test = @getimagesize($tmp);
@@ -813,7 +813,7 @@ 
 	if (!function_exists('imagegif')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	$ret = imagegif($img, $tmp);
 	if (file_exists($tmp)) {
 		$taille_test = @getimagesize($tmp);
@@ -853,7 +853,7 @@ 
 	if (!function_exists('imagejpeg')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 
 	// Enable interlancing
 	imageinterlace($img, true);
@@ -914,7 +914,7 @@ 
 	if (!function_exists('imagewebp')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	$ret = imagewebp($img, $tmp, $qualite);
 	if (file_exists($tmp)) {
 		$taille_test = @getimagesize($tmp);
@@ -948,7 +948,7 @@ 
  */
 function _image_imagesvg($img, $fichier) {
 
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	if (!str_contains($img, '<')) {
 		$img = supprimer_timestamp($img);
 		if (!file_exists($img)) {
@@ -1005,14 +1005,14 @@ 
  */
 function _image_gd_output($img, $valeurs, $qualite = _IMG_GD_QUALITE, $fonction = null) {
 	if (is_null($fonction)) {
-		$fonction = '_image_image' . $valeurs['format_dest'];
+		$fonction = '_image_image'.$valeurs['format_dest'];
 	}
 	$ret = false;
 	#un flag pour reperer les images gravees
 	$lock = (
 		!statut_effacer_images_temporaires('get')
 		|| @file_exists($valeurs['fichier_dest'])
-		&& !@file_exists($valeurs['fichier_dest'] . '.src')
+		&& !@file_exists($valeurs['fichier_dest'].'.src')
 	);
 	if (
 		function_exists($fonction)
@@ -1024,7 +1024,7 @@ 
 		[$valeurs['hauteur_dest'], $valeurs['largeur_dest']] = taille_image($valeurs['fichier_dest']);
 		$valeurs['date'] = @filemtime($valeurs['fichier_dest']);
 		// pour la retrouver apres disparition
-		ecrire_fichier($valeurs['fichier_dest'] . '.src', serialize($valeurs), true);
+		ecrire_fichier($valeurs['fichier_dest'].'.src', serialize($valeurs), true);
 	}
 
 	return $ret;
@@ -1199,7 +1199,7 @@ 
 
 	// attributs deprecies. Transformer en CSS
 	if ($espace = extraire_attribut($tag, 'hspace')) {
-		$style = "margin:{$espace}px;" . $style;
+		$style = "margin:{$espace}px;".$style;
 		$tag = inserer_attribut($tag, 'hspace', '');
 	}
 
@@ -1322,7 +1322,7 @@ 
 	$image = $valeurs['fichier'];
 	$format = $valeurs['format_source'];
 	$destdir = dirname($valeurs['fichier_dest']);
-	$destfile = basename($valeurs['fichier_dest'], '.' . $valeurs['format_dest']);
+	$destfile = basename($valeurs['fichier_dest'], '.'.$valeurs['format_dest']);
 
 	$format_sortie = $valeurs['format_dest'];
 
@@ -1354,7 +1354,7 @@ 
 
 	// Si l'image est de la taille demandee (ou plus petite), simplement la retourner
 	if ($srcWidth && $srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
-		$vignette = $destination . '.' . $format;
+		$vignette = $destination.'.'.$format;
 		@copy($image, $vignette);
 	}
 
@@ -1362,7 +1362,7 @@ 
 		include_spip('inc/svg');
 		if ($svg = svg_redimensionner($valeurs['fichier'], $destWidth, $destHeight)) {
 			$format_sortie = 'svg';
-			$vignette = $destination . '.' . $format_sortie;
+			$vignette = $destination.'.'.$format_sortie;
 			$valeurs['fichier_dest'] = $vignette;
 			_image_gd_output($svg, $valeurs);
 		}
@@ -1374,9 +1374,9 @@ 
 			define('_CONVERT_COMMAND', 'convert');
 		} // Securite : mes_options.php peut preciser le chemin absolu
 		if (!defined('_RESIZE_COMMAND')) {
-			define('_RESIZE_COMMAND', _CONVERT_COMMAND . ' -quality ' . _IMG_CONVERT_QUALITE . ' -orient Undefined -resize %xx%y! %src %dest');
+			define('_RESIZE_COMMAND', _CONVERT_COMMAND.' -quality '._IMG_CONVERT_QUALITE.' -orient Undefined -resize %xx%y! %src %dest');
 		}
-		$vignette = $destination . '.' . $format_sortie;
+		$vignette = $destination.'.'.$format_sortie;
 		$commande = str_replace(
 			['%x', '%y', '%src', '%dest'],
 			[
@@ -1392,7 +1392,7 @@ 
 		if (!@file_exists($vignette)) {
 			spip_log("echec convert sur $vignette");
 
-			return;  // echec commande
+			return; // echec commande
 		}
 	}
 
@@ -1409,7 +1409,7 @@ 
 		if (!$output) {
 			return;
 		}
-		$vignette = $output . DIRECTORY_SEPARATOR . basename($destination) . '.' . $format_sortie;
+		$vignette = $output.DIRECTORY_SEPARATOR.basename($destination).'.'.$format_sortie;
 
 		$imagick = new Imagick();
 		$imagick->readImage(realpath($image));
@@ -1418,7 +1418,7 @@ 
 			$destHeight,
 			Imagick::FILTER_LANCZOS,
 			1
-		);//, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
+		); //, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
 		$imagick->writeImage($vignette);
 
 		if (!@file_exists($vignette)) {
@@ -1427,7 +1427,7 @@ 
 			return;
 		}
 		// remettre le chemin relatif car c'est ce qu'attend SPIP pour la suite (en particlier action/tester)
-		$vignette = $destination . '.' . $format_sortie;
+		$vignette = $destination.'.'.$format_sortie;
 	}
 
 	// netpbm
@@ -1438,11 +1438,11 @@ 
 		if (_PNMSCALE_COMMAND == '') {
 			return;
 		}
-		$vignette = $destination . '.' . $format_sortie;
+		$vignette = $destination.'.'.$format_sortie;
 		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
 		if ($format == 'jpg') {
 			$jpegtopnm_command = str_replace('pnmscale', 'jpegtopnm', _PNMSCALE_COMMAND);
-			exec("$jpegtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+			exec("$jpegtopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 			if (!($s = @filesize($vignette))) {
 				spip_unlink($vignette);
 			}
@@ -1454,7 +1454,7 @@ 
 		} else {
 			if ($format == 'gif') {
 				$giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
-				exec("$giftopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+				exec("$giftopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 				if (!($s = @filesize($vignette))) {
 					spip_unlink($vignette);
 				}
@@ -1466,7 +1466,7 @@ 
 			} else {
 				if ($format == 'png') {
 					$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
-					exec("$pngtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+					exec("$pngtopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 					if (!($s = @filesize($vignette))) {
 						spip_unlink($vignette);
 					}
@@ -1488,7 +1488,7 @@ 
 			return;
 		}
 		if (_IMG_GD_MAX_PIXELS && $srcWidth * $srcHeight > _IMG_GD_MAX_PIXELS) {
-			spip_log('vignette gd1/gd2 impossible : ' . $srcWidth * $srcHeight . 'pixels');
+			spip_log('vignette gd1/gd2 impossible : '.$srcWidth * $srcHeight.'pixels');
 
 			return;
 		}
@@ -1790,7 +1790,7 @@ 
 		// de l'image, de facon a tromper le cache du navigateur
 		// quand on fait supprimer/reuploader un logo
 		// (pas de filemtime si SAFE MODE)
-		$date = test_espace_prive() ? ('?' . $date) : '';
+		$date = test_espace_prive() ? ('?'.$date) : '';
 
 		return _image_ecrire_tag($image, ['src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight]);
 	}
@@ -1868,9 +1868,9 @@ 
 					$b = $argb['blue'];
 
 					if ($bpp[$key] == 32) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r) . chr($a);
+						$icXOR[$key] .= chr($b).chr($g).chr($r).chr($a);
 					} elseif ($bpp[$key] === 24) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r);
+						$icXOR[$key] .= chr($b).chr($g).chr($r);
 					}
 
 					if ($a < 128) {
@@ -1897,48 +1897,48 @@ 
 
 			// BITMAPINFOHEADER - 40 bytes
 			$BitmapInfoHeader[$key] = '';
-			$BitmapInfoHeader[$key] .= "\x28\x00\x00\x00";                // DWORD  biSize;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);    // LONG   biWidth;
+			$BitmapInfoHeader[$key] .= "\x28\x00\x00\x00"; // DWORD  biSize;
+			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4); // LONG   biWidth;
 			// The biHeight member specifies the combined
 			// height of the XOR and AND masks.
 			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;
-			$BitmapInfoHeader[$key] .= "\x01\x00";                    // WORD   biPlanes;
-			$BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";              // wBitCount;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biCompression;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);      // DWORD  biSizeImage;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biXPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biYPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrUsed;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrImportant;
+			$BitmapInfoHeader[$key] .= "\x01\x00"; // WORD   biPlanes;
+			$BitmapInfoHeader[$key] .= chr($bpp[$key])."\x00"; // wBitCount;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biCompression;
+			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4); // DWORD  biSizeImage;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // LONG   biXPelsPerMeter;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // LONG   biYPelsPerMeter;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biClrUsed;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biClrImportant;
 		}
 
 
-		$icondata = "\x00\x00";                    // idReserved;   // Reserved (must be 0)
-		$icondata .= "\x01\x00";                    // idType;	   // Resource Type (1 for icons)
-		$icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;	  // How many images?
+		$icondata = "\x00\x00"; // idReserved;   // Reserved (must be 0)
+		$icondata .= "\x01\x00"; // idType;	   // Resource Type (1 for icons)
+		$icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2); // idCount;	  // How many images?
 
 		$dwImageOffset = 6 + (count($gd_image_array) * 16);
 		foreach (array_keys($gd_image_array) as $key) {
 			// ICONDIRENTRY   idEntries[1]; // An entry for each image (idCount of 'em)
 
-			$icondata .= chr($ImageWidths[$key]);           // bWidth;		  // Width, in pixels, of the image
-			$icondata .= chr($ImageHeights[$key]);          // bHeight;		 // Height, in pixels, of the image
-			$icondata .= chr($totalcolors[$key]);           // bColorCount;	 // Number of colors in image (0 if >=8bpp)
-			$icondata .= "\x00";                    // bReserved;	   // Reserved ( must be 0)
+			$icondata .= chr($ImageWidths[$key]); // bWidth;		  // Width, in pixels, of the image
+			$icondata .= chr($ImageHeights[$key]); // bHeight;		 // Height, in pixels, of the image
+			$icondata .= chr($totalcolors[$key]); // bColorCount;	 // Number of colors in image (0 if >=8bpp)
+			$icondata .= "\x00"; // bReserved;	   // Reserved ( must be 0)
 
-			$icondata .= "\x01\x00";                  // wPlanes;		 // Color Planes
-			$icondata .= chr($bpp[$key]) . "\x00";            // wBitCount;	   // Bits per pixel
+			$icondata .= "\x01\x00"; // wPlanes;		 // Color Planes
+			$icondata .= chr($bpp[$key])."\x00"; // wBitCount;	   // Bits per pixel
 
 			$dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);
 			$icondata .= phpthumb_functions::LittleEndian2String(
 				$dwBytesInRes,
 				4
-			);     // dwBytesInRes;	// How many bytes in this resource?
+			); // dwBytesInRes;	// How many bytes in this resource?
 
 			$icondata .= phpthumb_functions::LittleEndian2String(
 				$dwImageOffset,
 				4
-			);    // dwImageOffset;   // Where in the file is this image?
+			); // dwImageOffset;   // Where in the file is this image?
 			$dwImageOffset += strlen($BitmapInfoHeader[$key]);
 			$dwImageOffset += strlen($icXOR[$key]);
 			$dwImageOffset += strlen($icAND[$key]);

Braces +3 added lines, -7 removed lines

@@ -537,8 +537,7 @@ 
 			process_image_svg_identite($ret);
 			$ret['creer'] = false;
 		}
-	}
-	else {
+	} else {
 		if (!function_exists($ret['fonction_imagecreatefrom'])) {
 			return false;
 		}
@@ -1356,9 +1355,7 @@ 
 	if ($srcWidth && $srcWidth <= $maxWidth && $srcHeight <= $maxHeight) {
 		$vignette = $destination . '.' . $format;
 		@copy($image, $vignette);
-	}
-
-	elseif ($valeurs['format_source'] === 'svg') {
+	} elseif ($valeurs['format_source'] === 'svg') {
 		include_spip('inc/svg');
 		if ($svg = svg_redimensionner($valeurs['fichier'], $destWidth, $destHeight)) {
 			$format_sortie = 'svg';
@@ -1793,8 +1790,7 @@ 
 		$date = test_espace_prive() ? ('?' . $date) : '';
 
 		return _image_ecrire_tag($image, ['src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight]);
-	}
-	else {
+	} else {
 		# BMP, tiff ... les redacteurs osent tout!
 		return $img;
 	}

ecrire/inc/traduire.php

Indentation +192 added lines, -192 removed lines

@@ -18,7 +18,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -39,18 +39,18 @@ 
  *     Liste des fichiers de langue trouvés, dans l'ordre des chemins
  */
 function find_langs_in_path($file, $dirname = 'lang') {
-	static $dirs = [];
-	$liste = [];
-	foreach (creer_chemin() as $dir) {
-		if (!isset($dirs[$a = $dir . $dirname])) {
-			$dirs[$a] = (is_dir($a) || !$a);
-		}
-		if ($dirs[$a] && is_readable($a .= $file)) {
-			$liste[] = $a;
-		}
-	}
+    static $dirs = [];
+    $liste = [];
+    foreach (creer_chemin() as $dir) {
+        if (!isset($dirs[$a = $dir . $dirname])) {
+            $dirs[$a] = (is_dir($a) || !$a);
+        }
+        if ($dirs[$a] && is_readable($a .= $file)) {
+            $liste[] = $a;
+        }
+    }
 
-	return array_reverse($liste);
+    return array_reverse($liste);
 }
 
 /**
@@ -65,23 +65,23 @@ 
  *     Liste des fichiers touvés pour ce module et cette langue.
  **/
 function chercher_module_lang($module, $lang = '') {
-	if ($lang) {
-		$lang = '_' . $lang;
-	}
+    if ($lang) {
+        $lang = '_' . $lang;
+    }
 
-	// 1) dans un repertoire nomme lang/ se trouvant sur le chemin
-	if (
-		$f = ($module == 'local'
-		? find_in_path($module . $lang . '.php', 'lang/')
-		: find_langs_in_path($module . $lang . '.php', 'lang/'))
-	) {
-		return is_array($f) ? $f : [$f];
-	}
+    // 1) dans un repertoire nomme lang/ se trouvant sur le chemin
+    if (
+        $f = ($module == 'local'
+        ? find_in_path($module . $lang . '.php', 'lang/')
+        : find_langs_in_path($module . $lang . '.php', 'lang/'))
+    ) {
+        return is_array($f) ? $f : [$f];
+    }
 
-	// 2) directement dans le chemin (old style, uniquement pour local)
-	return ($module == 'local' || strpos($module, '/'))
-		? (($f = find_in_path($module . $lang . '.php')) ? [$f] : false)
-		: false;
+    // 2) directement dans le chemin (old style, uniquement pour local)
+    return ($module == 'local' || strpos($module, '/'))
+        ? (($f = find_in_path($module . $lang . '.php')) ? [$f] : false)
+        : false;
 }
 
 /**
@@ -103,33 +103,33 @@ 
  * @return void
  **/
 function charger_langue($lang, $module = 'spip') {
-	static $langs = [];
-	$var = 'i18n_' . $module . '_' . $lang;
-	if (!isset($langs[$lang])) {
-		$langs[$lang] = [];
-		if ($lang) {
-			$langs[$lang][] = $lang;
-			if (str_contains($lang, '_')) {
-				$l = explode('_', $lang);
-				$langs[$lang][] = reset($l);
-			}
-		}
-		$langs[$lang][] = $GLOBALS['meta']['langue_site'];
-		$langs[$lang][] = _LANGUE_PAR_DEFAUT;
-	}
-	foreach ($langs[$lang] as $l) {
-		if ($fichiers_lang = chercher_module_lang($module, $l)) {
-			$GLOBALS['idx_lang'] = 'i18n_' . $module . '_' . $l;
-			$GLOBALS[$GLOBALS['idx_lang']] = lire_fichier_langue(array_shift($fichiers_lang));
-			surcharger_langue($fichiers_lang);
-			if ($l !== $lang) {
-				$GLOBALS[$var] = &$GLOBALS['i18n_' . $module . '_' . $l];
-			}
-			$GLOBALS['lang_' . $var] = $l;
-			#spip_log("module de langue : {$module}_$l.php", 'traduire');
-			break;
-		}
-	}
+    static $langs = [];
+    $var = 'i18n_' . $module . '_' . $lang;
+    if (!isset($langs[$lang])) {
+        $langs[$lang] = [];
+        if ($lang) {
+            $langs[$lang][] = $lang;
+            if (str_contains($lang, '_')) {
+                $l = explode('_', $lang);
+                $langs[$lang][] = reset($l);
+            }
+        }
+        $langs[$lang][] = $GLOBALS['meta']['langue_site'];
+        $langs[$lang][] = _LANGUE_PAR_DEFAUT;
+    }
+    foreach ($langs[$lang] as $l) {
+        if ($fichiers_lang = chercher_module_lang($module, $l)) {
+            $GLOBALS['idx_lang'] = 'i18n_' . $module . '_' . $l;
+            $GLOBALS[$GLOBALS['idx_lang']] = lire_fichier_langue(array_shift($fichiers_lang));
+            surcharger_langue($fichiers_lang);
+            if ($l !== $lang) {
+                $GLOBALS[$var] = &$GLOBALS['i18n_' . $module . '_' . $l];
+            }
+            $GLOBALS['lang_' . $var] = $l;
+            #spip_log("module de langue : {$module}_$l.php", 'traduire');
+            break;
+        }
+    }
 }
 
 /**
@@ -142,21 +142,21 @@ 
  * @return array<string, string>
  */
 function lire_fichier_langue(string $fichier): array {
-	$idx_lang_before = $GLOBALS['idx_lang'] ?? null;
-	$idx_lang_tmp = ($GLOBALS['idx_lang'] ?? 'lang') . '@temporaire';
-	$GLOBALS['idx_lang'] = $idx_lang_tmp;
-	$idx_lang = include $fichier;
-	$GLOBALS['idx_lang'] = $idx_lang_before;
-	if (!is_array($idx_lang)) {
-		if (isset($GLOBALS[$idx_lang_tmp]) && is_array($GLOBALS[$idx_lang_tmp])) {
-			$idx_lang = $GLOBALS[$idx_lang_tmp];
-		} else {
-			$idx_lang = [];
-			spip_log(sprintf('Fichier de langue incorrect : %s', $fichier), _LOG_ERREUR);
-		}
-		unset($GLOBALS[$idx_lang_tmp]);
-	}
-	return $idx_lang;
+    $idx_lang_before = $GLOBALS['idx_lang'] ?? null;
+    $idx_lang_tmp = ($GLOBALS['idx_lang'] ?? 'lang') . '@temporaire';
+    $GLOBALS['idx_lang'] = $idx_lang_tmp;
+    $idx_lang = include $fichier;
+    $GLOBALS['idx_lang'] = $idx_lang_before;
+    if (!is_array($idx_lang)) {
+        if (isset($GLOBALS[$idx_lang_tmp]) && is_array($GLOBALS[$idx_lang_tmp])) {
+            $idx_lang = $GLOBALS[$idx_lang_tmp];
+        } else {
+            $idx_lang = [];
+            spip_log(sprintf('Fichier de langue incorrect : %s', $fichier), _LOG_ERREUR);
+        }
+        unset($GLOBALS[$idx_lang_tmp]);
+    }
+    return $idx_lang;
 }
 
 /**
@@ -176,29 +176,29 @@ 
  *    Liste des chemins de fichiers de langue à surcharger.
  **/
 function surcharger_langue($fichiers) {
-	static $surcharges = [];
-	if (!isset($GLOBALS['idx_lang'])) {
-		return;
-	}
+    static $surcharges = [];
+    if (!isset($GLOBALS['idx_lang'])) {
+        return;
+    }
 
-	if (!is_array($fichiers)) {
-		$fichiers = [$fichiers];
-	}
-	if ($fichiers === []) {
-		return;
-	}
-	foreach ($fichiers as $fichier) {
-		if (!isset($surcharges[$fichier])) {
-			$surcharges[$fichier] = lire_fichier_langue($fichier);
-		}
-		if (is_array($surcharges[$fichier])) {
-			$GLOBALS[$GLOBALS['idx_lang']] ??= [];
-			$GLOBALS[$GLOBALS['idx_lang']] = array_merge(
-				$GLOBALS[$GLOBALS['idx_lang']],
-				$surcharges[$fichier]
-			);
-		}
-	}
+    if (!is_array($fichiers)) {
+        $fichiers = [$fichiers];
+    }
+    if ($fichiers === []) {
+        return;
+    }
+    foreach ($fichiers as $fichier) {
+        if (!isset($surcharges[$fichier])) {
+            $surcharges[$fichier] = lire_fichier_langue($fichier);
+        }
+        if (is_array($surcharges[$fichier])) {
+            $GLOBALS[$GLOBALS['idx_lang']] ??= [];
+            $GLOBALS[$GLOBALS['idx_lang']] = array_merge(
+                $GLOBALS[$GLOBALS['idx_lang']],
+                $surcharges[$fichier]
+            );
+        }
+    }
 }
 
 /**
@@ -239,99 +239,99 @@ 
  *     - Description : traduction et description (texte, module, langue)
  **/
 function inc_traduire_dist($ori, $lang, $raw = false) {
-	static $deja_vu = [];
-	static $local = [];
+    static $deja_vu = [];
+    static $local = [];
 
-	if (isset($deja_vu[$lang][$ori]) && _request('var_mode') != 'traduction') {
-		return $raw ? $deja_vu[$lang][$ori] : $deja_vu[$lang][$ori]->texte;
-	}
+    if (isset($deja_vu[$lang][$ori]) && _request('var_mode') != 'traduction') {
+        return $raw ? $deja_vu[$lang][$ori] : $deja_vu[$lang][$ori]->texte;
+    }
 
-	// modules demandes explicitement <xxx|yyy|zzz:code> cf MODULES_IDIOMES
-	if (strpos($ori, ':')) {
-		[$modules, $code] = explode(':', $ori, 2);
-		$modules = explode('|', $modules);
-		$ori_complet = $ori;
-	} else {
-		$modules = ['spip', 'ecrire'];
-		$code = $ori;
-		$ori_complet = implode('|', $modules) . ':' . $ori;
-	}
+    // modules demandes explicitement <xxx|yyy|zzz:code> cf MODULES_IDIOMES
+    if (strpos($ori, ':')) {
+        [$modules, $code] = explode(':', $ori, 2);
+        $modules = explode('|', $modules);
+        $ori_complet = $ori;
+    } else {
+        $modules = ['spip', 'ecrire'];
+        $code = $ori;
+        $ori_complet = implode('|', $modules) . ':' . $ori;
+    }
 
-	$desc = new Description();
+    $desc = new Description();
 
-	// parcourir tous les modules jusqu'a ce qu'on trouve
-	foreach ($modules as $module) {
-		$var = 'i18n_' . $module . '_' . $lang;
+    // parcourir tous les modules jusqu'a ce qu'on trouve
+    foreach ($modules as $module) {
+        $var = 'i18n_' . $module . '_' . $lang;
 
-		if (empty($GLOBALS[$var])) {
-			charger_langue($lang, $module);
-			// surcharges persos -- on cherche
-			// (lang/)local_xx.php et/ou (lang/)local.php ...
-			if (!isset($local['local_' . $lang])) {
-				// redéfinir la langue en cours pour les surcharges (chercher_langue a pu le changer)
-				$GLOBALS['idx_lang'] = $var;
-				// ... (lang/)local_xx.php
-				$local['local_' . $lang] = chercher_module_lang('local', $lang);
-			}
-			if ($local['local_' . $lang]) {
-				surcharger_langue($local['local_' . $lang]);
-			}
-			// ... puis (lang/)local.php
-			if (!isset($local['local'])) {
-				$local['local'] = chercher_module_lang('local');
-			}
-			if ($local['local']) {
-				surcharger_langue($local['local']);
-			}
-		}
+        if (empty($GLOBALS[$var])) {
+            charger_langue($lang, $module);
+            // surcharges persos -- on cherche
+            // (lang/)local_xx.php et/ou (lang/)local.php ...
+            if (!isset($local['local_' . $lang])) {
+                // redéfinir la langue en cours pour les surcharges (chercher_langue a pu le changer)
+                $GLOBALS['idx_lang'] = $var;
+                // ... (lang/)local_xx.php
+                $local['local_' . $lang] = chercher_module_lang('local', $lang);
+            }
+            if ($local['local_' . $lang]) {
+                surcharger_langue($local['local_' . $lang]);
+            }
+            // ... puis (lang/)local.php
+            if (!isset($local['local'])) {
+                $local['local'] = chercher_module_lang('local');
+            }
+            if ($local['local']) {
+                surcharger_langue($local['local']);
+            }
+        }
 
-		if (isset($GLOBALS[$var][$code])) {
-			$desc->code = $code;
-			$desc->module = $module;
-			$desc->langue = $GLOBALS['lang_' . $var] ?? $lang;
-			$desc->texte = $GLOBALS[$var][$code];
-			break;
-		}
-	}
+        if (isset($GLOBALS[$var][$code])) {
+            $desc->code = $code;
+            $desc->module = $module;
+            $desc->langue = $GLOBALS['lang_' . $var] ?? $lang;
+            $desc->texte = $GLOBALS[$var][$code];
+            break;
+        }
+    }
 
-	if (!$desc->corrections) {
-		$desc->corrections = true;
-		// Retour aux sources si la chaine est absente dans la langue cible ;
-		// on essaie d'abord la langue du site, puis a defaut la langue fr
-		if (
-			($desc->texte === null || !strlen($desc->texte))
-			&& $lang !== _LANGUE_PAR_DEFAUT
-		) {
-			if ($lang !== $GLOBALS['meta']['langue_site']) {
-				$desc = inc_traduire_dist($ori, $GLOBALS['meta']['langue_site'], true);
-			} else {
-				$desc = inc_traduire_dist($ori, _LANGUE_PAR_DEFAUT, true);
-			}
-		}
+    if (!$desc->corrections) {
+        $desc->corrections = true;
+        // Retour aux sources si la chaine est absente dans la langue cible ;
+        // on essaie d'abord la langue du site, puis a defaut la langue fr
+        if (
+            ($desc->texte === null || !strlen($desc->texte))
+            && $lang !== _LANGUE_PAR_DEFAUT
+        ) {
+            if ($lang !== $GLOBALS['meta']['langue_site']) {
+                $desc = inc_traduire_dist($ori, $GLOBALS['meta']['langue_site'], true);
+            } else {
+                $desc = inc_traduire_dist($ori, _LANGUE_PAR_DEFAUT, true);
+            }
+        }
 
-		// Supprimer la mention <NEW> ou <MODIF>
-		if ($desc->texte && str_starts_with($desc->texte, '<')) {
-			$desc->texte = str_replace(['<NEW>', '<MODIF>'], [], $desc->texte);
-		}
+        // Supprimer la mention <NEW> ou <MODIF>
+        if ($desc->texte && str_starts_with($desc->texte, '<')) {
+            $desc->texte = str_replace(['<NEW>', '<MODIF>'], [], $desc->texte);
+        }
 
-		// Si on n'est pas en utf-8, la chaine peut l'etre...
-		// le cas echeant on la convertit en entites html &#xxx;
-		if (
-			(!isset($GLOBALS['meta']['charset']) || $GLOBALS['meta']['charset'] !== 'utf-8')
-			&& preg_match(',[\x7f-\xff],S', $desc->texte)
-		) {
-			include_spip('inc/charsets');
-			$desc->texte = charset2unicode($desc->texte, 'utf-8');
-		}
-	}
+        // Si on n'est pas en utf-8, la chaine peut l'etre...
+        // le cas echeant on la convertit en entites html &#xxx;
+        if (
+            (!isset($GLOBALS['meta']['charset']) || $GLOBALS['meta']['charset'] !== 'utf-8')
+            && preg_match(',[\x7f-\xff],S', $desc->texte)
+        ) {
+            include_spip('inc/charsets');
+            $desc->texte = charset2unicode($desc->texte, 'utf-8');
+        }
+    }
 
-	if (_request('var_mode') == 'traduction') {
-		$desc = definir_details_traduction($desc, $ori_complet);
-	} else {
-		$deja_vu[$lang][$ori] = $desc;
-	}
+    if (_request('var_mode') == 'traduction') {
+        $desc = definir_details_traduction($desc, $ori_complet);
+    } else {
+        $deja_vu[$lang][$ori] = $desc;
+    }
 
-	return $raw ? $desc : $desc->texte;
+    return $raw ? $desc : $desc->texte;
 }
 
 /**
@@ -343,23 +343,23 @@ 
  * @return Description
  */
 function definir_details_traduction($desc, $modules) {
-	if (!$desc->mode && $desc->texte) {
-		// ne pas modifier 2 fois l'affichage
-		$desc->mode = 'traduction';
-		$classe = 'debug-traduction' . ($desc->module == 'ecrire' ? '-prive' : '');
-		$desc->texte = '<span '
-			. 'lang=' . $desc->langue
-			. ' class=' . $classe
-			. ' data-module=' . $desc->module
-			. ' data-code=' . $desc->code
-			. ' title=' . $modules . '(' . $desc->langue . ')>'
-			. $desc->texte
-			. '</span>';
-		$desc->texte = str_replace(
-			["$desc->module:", "$desc->module|"],
-			["*$desc->module*:", "*$desc->module*|"],
-			$desc->texte
-		);
-	}
-	return $desc;
+    if (!$desc->mode && $desc->texte) {
+        // ne pas modifier 2 fois l'affichage
+        $desc->mode = 'traduction';
+        $classe = 'debug-traduction' . ($desc->module == 'ecrire' ? '-prive' : '');
+        $desc->texte = '<span '
+            . 'lang=' . $desc->langue
+            . ' class=' . $classe
+            . ' data-module=' . $desc->module
+            . ' data-code=' . $desc->code
+            . ' title=' . $modules . '(' . $desc->langue . ')>'
+            . $desc->texte
+            . '</span>';
+        $desc->texte = str_replace(
+            ["$desc->module:", "$desc->module|"],
+            ["*$desc->module*:", "*$desc->module*|"],
+            $desc->texte
+        );
+    }
+    return $desc;
 }

Spacing +23 added lines, -23 removed lines

@@ -42,7 +42,7 @@ 
 	static $dirs = [];
 	$liste = [];
 	foreach (creer_chemin() as $dir) {
-		if (!isset($dirs[$a = $dir . $dirname])) {
+		if (!isset($dirs[$a = $dir.$dirname])) {
 			$dirs[$a] = (is_dir($a) || !$a);
 		}
 		if ($dirs[$a] && is_readable($a .= $file)) {
@@ -66,21 +66,21 @@ 
  **/
 function chercher_module_lang($module, $lang = '') {
 	if ($lang) {
-		$lang = '_' . $lang;
+		$lang = '_'.$lang;
 	}
 
 	// 1) dans un repertoire nomme lang/ se trouvant sur le chemin
 	if (
 		$f = ($module == 'local'
-		? find_in_path($module . $lang . '.php', 'lang/')
-		: find_langs_in_path($module . $lang . '.php', 'lang/'))
+		? find_in_path($module.$lang.'.php', 'lang/')
+		: find_langs_in_path($module.$lang.'.php', 'lang/'))
 	) {
 		return is_array($f) ? $f : [$f];
 	}
 
 	// 2) directement dans le chemin (old style, uniquement pour local)
 	return ($module == 'local' || strpos($module, '/'))
-		? (($f = find_in_path($module . $lang . '.php')) ? [$f] : false)
+		? (($f = find_in_path($module.$lang.'.php')) ? [$f] : false)
 		: false;
 }
 
@@ -104,7 +104,7 @@ 
  **/
 function charger_langue($lang, $module = 'spip') {
 	static $langs = [];
-	$var = 'i18n_' . $module . '_' . $lang;
+	$var = 'i18n_'.$module.'_'.$lang;
 	if (!isset($langs[$lang])) {
 		$langs[$lang] = [];
 		if ($lang) {
@@ -119,13 +119,13 @@ 
 	}
 	foreach ($langs[$lang] as $l) {
 		if ($fichiers_lang = chercher_module_lang($module, $l)) {
-			$GLOBALS['idx_lang'] = 'i18n_' . $module . '_' . $l;
+			$GLOBALS['idx_lang'] = 'i18n_'.$module.'_'.$l;
 			$GLOBALS[$GLOBALS['idx_lang']] = lire_fichier_langue(array_shift($fichiers_lang));
 			surcharger_langue($fichiers_lang);
 			if ($l !== $lang) {
-				$GLOBALS[$var] = &$GLOBALS['i18n_' . $module . '_' . $l];
+				$GLOBALS[$var] = &$GLOBALS['i18n_'.$module.'_'.$l];
 			}
-			$GLOBALS['lang_' . $var] = $l;
+			$GLOBALS['lang_'.$var] = $l;
 			#spip_log("module de langue : {$module}_$l.php", 'traduire');
 			break;
 		}
@@ -143,7 +143,7 @@ 
  */
 function lire_fichier_langue(string $fichier): array {
 	$idx_lang_before = $GLOBALS['idx_lang'] ?? null;
-	$idx_lang_tmp = ($GLOBALS['idx_lang'] ?? 'lang') . '@temporaire';
+	$idx_lang_tmp = ($GLOBALS['idx_lang'] ?? 'lang').'@temporaire';
 	$GLOBALS['idx_lang'] = $idx_lang_tmp;
 	$idx_lang = include $fichier;
 	$GLOBALS['idx_lang'] = $idx_lang_before;
@@ -254,27 +254,27 @@ 
 	} else {
 		$modules = ['spip', 'ecrire'];
 		$code = $ori;
-		$ori_complet = implode('|', $modules) . ':' . $ori;
+		$ori_complet = implode('|', $modules).':'.$ori;
 	}
 
 	$desc = new Description();
 
 	// parcourir tous les modules jusqu'a ce qu'on trouve
 	foreach ($modules as $module) {
-		$var = 'i18n_' . $module . '_' . $lang;
+		$var = 'i18n_'.$module.'_'.$lang;
 
 		if (empty($GLOBALS[$var])) {
 			charger_langue($lang, $module);
 			// surcharges persos -- on cherche
 			// (lang/)local_xx.php et/ou (lang/)local.php ...
-			if (!isset($local['local_' . $lang])) {
+			if (!isset($local['local_'.$lang])) {
 				// redéfinir la langue en cours pour les surcharges (chercher_langue a pu le changer)
 				$GLOBALS['idx_lang'] = $var;
 				// ... (lang/)local_xx.php
-				$local['local_' . $lang] = chercher_module_lang('local', $lang);
+				$local['local_'.$lang] = chercher_module_lang('local', $lang);
 			}
-			if ($local['local_' . $lang]) {
-				surcharger_langue($local['local_' . $lang]);
+			if ($local['local_'.$lang]) {
+				surcharger_langue($local['local_'.$lang]);
 			}
 			// ... puis (lang/)local.php
 			if (!isset($local['local'])) {
@@ -288,7 +288,7 @@ 
 		if (isset($GLOBALS[$var][$code])) {
 			$desc->code = $code;
 			$desc->module = $module;
-			$desc->langue = $GLOBALS['lang_' . $var] ?? $lang;
+			$desc->langue = $GLOBALS['lang_'.$var] ?? $lang;
 			$desc->texte = $GLOBALS[$var][$code];
 			break;
 		}
@@ -346,13 +346,13 @@ 
 	if (!$desc->mode && $desc->texte) {
 		// ne pas modifier 2 fois l'affichage
 		$desc->mode = 'traduction';
-		$classe = 'debug-traduction' . ($desc->module == 'ecrire' ? '-prive' : '');
+		$classe = 'debug-traduction'.($desc->module == 'ecrire' ? '-prive' : '');
 		$desc->texte = '<span '
-			. 'lang=' . $desc->langue
-			. ' class=' . $classe
-			. ' data-module=' . $desc->module
-			. ' data-code=' . $desc->code
-			. ' title=' . $modules . '(' . $desc->langue . ')>'
+			. 'lang='.$desc->langue
+			. ' class='.$classe
+			. ' data-module='.$desc->module
+			. ' data-code='.$desc->code
+			. ' title='.$modules.'('.$desc->langue.')>'
 			. $desc->texte
 			. '</span>';
 		$desc->texte = str_replace(

ecrire/inc/math.php

Indentation +95 added lines, -95 removed lines

@@ -11,7 +11,7 @@ 
 
 //
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 //
@@ -20,52 +20,52 @@ 
 
 function produire_image_math($tex) {
 
-	switch ($GLOBALS['traiter_math']) {
-		// Attention: mathml desactiv'e pour l'instant
-		case 'mathml':
-			$ext = '.xhtml';
-			$server = $GLOBALS['mathml_server'];
-			break;
-		case 'tex':
-			$ext = '.png';
-			$server = $GLOBALS['tex_server'];
-			break;
-		default:
-			return $tex;
-	}
-
-	// Regarder dans le repertoire local des images TeX et blocs MathML
-	if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
-		@mkdir($dir_tex, _SPIP_CHMOD);
-	}
-	$fichier = $dir_tex . md5(trim($tex)) . $ext;
-
-
-	// Aller chercher l'image sur le serveur
-	if (!@file_exists($fichier) && $server) {
-		spip_log($url = $server . '?' . rawurlencode($tex));
-		include_spip('inc/distant');
-		recuperer_url($url, ['file' => $fichier]);
-	}
-
-
-	// Composer la reponse selon presence ou non de l'image
-	$tex = entites_html($tex);
-	if (@file_exists($fichier)) {
-		// MathML
-		if ($GLOBALS['traiter_math'] == 'mathml') {
-			return implode('', file($fichier));
-		} // TeX
-		else {
-			[, , , $size] = @spip_getimagesize($fichier);
-			$alt = "alt=\"$tex\" title=\"$tex\"";
-
-			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
-		}
-	} else // pas de fichier
-	{
-		return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
-	}
+    switch ($GLOBALS['traiter_math']) {
+        // Attention: mathml desactiv'e pour l'instant
+        case 'mathml':
+            $ext = '.xhtml';
+            $server = $GLOBALS['mathml_server'];
+            break;
+        case 'tex':
+            $ext = '.png';
+            $server = $GLOBALS['tex_server'];
+            break;
+        default:
+            return $tex;
+    }
+
+    // Regarder dans le repertoire local des images TeX et blocs MathML
+    if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
+        @mkdir($dir_tex, _SPIP_CHMOD);
+    }
+    $fichier = $dir_tex . md5(trim($tex)) . $ext;
+
+
+    // Aller chercher l'image sur le serveur
+    if (!@file_exists($fichier) && $server) {
+        spip_log($url = $server . '?' . rawurlencode($tex));
+        include_spip('inc/distant');
+        recuperer_url($url, ['file' => $fichier]);
+    }
+
+
+    // Composer la reponse selon presence ou non de l'image
+    $tex = entites_html($tex);
+    if (@file_exists($fichier)) {
+        // MathML
+        if ($GLOBALS['traiter_math'] == 'mathml') {
+            return implode('', file($fichier));
+        } // TeX
+        else {
+            [, , , $size] = @spip_getimagesize($fichier);
+            $alt = "alt=\"$tex\" title=\"$tex\"";
+
+            return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
+        }
+    } else // pas de fichier
+    {
+        return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
+    }
 }
 
 
@@ -100,52 +100,52 @@ 
  */
 function traiter_math($letexte, $source = '', $defaire_amp = false) {
 
-	$texte_a_voir = $letexte;
-	while (($debut = strpos($texte_a_voir, '<math>')) !== false) {
-		if (!$fin = strpos($texte_a_voir, '</math>')) {
-			$fin = strlen($texte_a_voir);
-		}
-
-		$texte_debut = substr($texte_a_voir, 0, $debut);
-		$texte_milieu = substr(
-			$texte_a_voir,
-			$debut + strlen('<math>'),
-			$fin - $debut - strlen('<math>')
-		);
-		$texte_fin = substr(
-			$texte_a_voir,
-			$fin + strlen('</math>'),
-			strlen($texte_a_voir)
-		);
-
-		// Les doubles $$x^2$$ en mode 'div'
-		while ((preg_match(',[$][$]([^$]+)[$][$],', $texte_milieu, $regs))) {
-			$expression = $regs[1];
-			if ($defaire_amp) {
-				$expression = str_replace('&amp;', '&', $expression);
-			}
-			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
-			$pos = strpos($texte_milieu, (string) $regs[0]);
-			$texte_milieu = substr($texte_milieu, 0, $pos)
-				. code_echappement($echap, $source)
-				. substr($texte_milieu, $pos + strlen($regs[0]));
-		}
-
-		// Les simples $x^2$ en mode 'span'
-		while ((preg_match(',[$]([^$]+)[$],', $texte_milieu, $regs))) {
-			$expression = $regs[1];
-			if ($defaire_amp) {
-				$expression = str_replace('&amp;', '&', $expression);
-			}
-			$echap = produire_image_math($expression);
-			$pos = strpos($texte_milieu, (string) $regs[0]);
-			$texte_milieu = substr($texte_milieu, 0, $pos)
-				. code_echappement($echap, $source)
-				. substr($texte_milieu, $pos + strlen($regs[0]));
-		}
-
-		$texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
-	}
-
-	return $texte_a_voir;
+    $texte_a_voir = $letexte;
+    while (($debut = strpos($texte_a_voir, '<math>')) !== false) {
+        if (!$fin = strpos($texte_a_voir, '</math>')) {
+            $fin = strlen($texte_a_voir);
+        }
+
+        $texte_debut = substr($texte_a_voir, 0, $debut);
+        $texte_milieu = substr(
+            $texte_a_voir,
+            $debut + strlen('<math>'),
+            $fin - $debut - strlen('<math>')
+        );
+        $texte_fin = substr(
+            $texte_a_voir,
+            $fin + strlen('</math>'),
+            strlen($texte_a_voir)
+        );
+
+        // Les doubles $$x^2$$ en mode 'div'
+        while ((preg_match(',[$][$]([^$]+)[$][$],', $texte_milieu, $regs))) {
+            $expression = $regs[1];
+            if ($defaire_amp) {
+                $expression = str_replace('&amp;', '&', $expression);
+            }
+            $echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
+            $pos = strpos($texte_milieu, (string) $regs[0]);
+            $texte_milieu = substr($texte_milieu, 0, $pos)
+                . code_echappement($echap, $source)
+                . substr($texte_milieu, $pos + strlen($regs[0]));
+        }
+
+        // Les simples $x^2$ en mode 'span'
+        while ((preg_match(',[$]([^$]+)[$],', $texte_milieu, $regs))) {
+            $expression = $regs[1];
+            if ($defaire_amp) {
+                $expression = str_replace('&amp;', '&', $expression);
+            }
+            $echap = produire_image_math($expression);
+            $pos = strpos($texte_milieu, (string) $regs[0]);
+            $texte_milieu = substr($texte_milieu, 0, $pos)
+                . code_echappement($echap, $source)
+                . substr($texte_milieu, $pos + strlen($regs[0]));
+        }
+
+        $texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
+    }
+
+    return $texte_a_voir;
 }

Spacing +6 added lines, -6 removed lines

@@ -35,15 +35,15 @@ 
 	}
 
 	// Regarder dans le repertoire local des images TeX et blocs MathML
-	if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
+	if (!@is_dir($dir_tex = _DIR_VAR.'cache-TeX/')) {
 		@mkdir($dir_tex, _SPIP_CHMOD);
 	}
-	$fichier = $dir_tex . md5(trim($tex)) . $ext;
+	$fichier = $dir_tex.md5(trim($tex)).$ext;
 
 
 	// Aller chercher l'image sur le serveur
 	if (!@file_exists($fichier) && $server) {
-		spip_log($url = $server . '?' . rawurlencode($tex));
+		spip_log($url = $server.'?'.rawurlencode($tex));
 		include_spip('inc/distant');
 		recuperer_url($url, ['file' => $fichier]);
 	}
@@ -57,7 +57,7 @@ 
 			return implode('', file($fichier));
 		} // TeX
 		else {
-			[, , , $size] = @spip_getimagesize($fichier);
+			[,,, $size] = @spip_getimagesize($fichier);
 			$alt = "alt=\"$tex\" title=\"$tex\"";
 
 			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
@@ -124,7 +124,7 @@ 
 			if ($defaire_amp) {
 				$expression = str_replace('&amp;', '&', $expression);
 			}
-			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
+			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($expression)."</p>\n";
 			$pos = strpos($texte_milieu, (string) $regs[0]);
 			$texte_milieu = substr($texte_milieu, 0, $pos)
 				. code_echappement($echap, $source)
@@ -144,7 +144,7 @@ 
 				. substr($texte_milieu, $pos + strlen($regs[0]));
 		}
 
-		$texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
+		$texte_a_voir = $texte_debut.$texte_milieu.$texte_fin;
 	}
 
 	return $texte_a_voir;

ecrire/inc/cvt_configurer.php

Indentation +148 added lines, -148 removed lines

@@ -19,7 +19,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/config');
@@ -31,31 +31,31 @@ 
  * @return array|false
  */
 function cvtconf_formulaire_charger($flux) {
-	if (
-		($form = $flux['args']['form'])
-		&& str_starts_with($form, 'configurer_') // un #FORMULAIRE_CONFIGURER_XXX
-	) {
-		// Pour tous les formulaires CONFIGURER, ayant une fonction charger ou pas, on teste si autorisé
-		include_spip('inc/autoriser');
-		if (!autoriser('configurer', '_' . substr($form, 11))) {
-			return false;
-		}
+    if (
+        ($form = $flux['args']['form'])
+        && str_starts_with($form, 'configurer_') // un #FORMULAIRE_CONFIGURER_XXX
+    ) {
+        // Pour tous les formulaires CONFIGURER, ayant une fonction charger ou pas, on teste si autorisé
+        include_spip('inc/autoriser');
+        if (!autoriser('configurer', '_' . substr($form, 11))) {
+            return false;
+        }
 
-		// S'il n'y a pas de fonction charger(), on génère un contexte automatiquement
-		if (!charger_fonction('charger', "formulaires/$form/", true)) {
-			$flux['data'] = cvtconf_formulaires_configurer_recense($form);
-			$flux['data']['editable'] = true;
-			if (_request('var_mode') == 'configurer' && autoriser('webmestre')) {
-				if (!_AJAX) {
-					var_dump($flux['data']);
-				}
-				// reinjecter pour la trace au traitement
-				$flux['data']['_hidden'] = "<input type='hidden' name='var_mode' value='configurer' />";
-			}
-		}
-	}
+        // S'il n'y a pas de fonction charger(), on génère un contexte automatiquement
+        if (!charger_fonction('charger', "formulaires/$form/", true)) {
+            $flux['data'] = cvtconf_formulaires_configurer_recense($form);
+            $flux['data']['editable'] = true;
+            if (_request('var_mode') == 'configurer' && autoriser('webmestre')) {
+                if (!_AJAX) {
+                    var_dump($flux['data']);
+                }
+                // reinjecter pour la trace au traitement
+                $flux['data']['_hidden'] = "<input type='hidden' name='var_mode' value='configurer' />";
+            }
+        }
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -65,16 +65,16 @@ 
  * @return array
  */
 function cvtconf_formulaire_traiter($flux) {
-	if (
-		($form = $flux['args']['form'])
-		&& str_starts_with($form, 'configurer_') // un #FORMULAIRE_CONFIGURER_XXX
-		&& !charger_fonction('traiter', "formulaires/$form/", true) // sans fonction traiter()
-	) {
-		$trace = cvtconf_formulaires_configurer_enregistre($form, $flux['args']['args']);
-		$flux['data'] = ['message_ok' => _T('config_info_enregistree') . $trace, 'editable' => true];
-	}
+    if (
+        ($form = $flux['args']['form'])
+        && str_starts_with($form, 'configurer_') // un #FORMULAIRE_CONFIGURER_XXX
+        && !charger_fonction('traiter', "formulaires/$form/", true) // sans fonction traiter()
+    ) {
+        $trace = cvtconf_formulaires_configurer_enregistre($form, $flux['args']['args']);
+        $flux['data'] = ['message_ok' => _T('config_info_enregistree') . $trace, 'editable' => true];
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -90,32 +90,32 @@ 
  * @return string
  */
 function cvtconf_formulaires_configurer_enregistre($form, $args) {
-	$valeurs = [];
-	// charger les valeurs
-	// ce qui permet de prendre en charge une fonction charger() existante
-	// qui prend alors la main sur l'auto detection
-	if ($charger_valeurs = charger_fonction('charger', "formulaires/$form/", true)) {
-		$valeurs = $charger_valeurs(...$args);
-	}
-	$valeurs = pipeline(
-		'formulaire_charger',
-		[
-			'args' => ['form' => $form, 'args' => $args, 'je_suis_poste' => false],
-			'data' => $valeurs
-		]
-	);
-	// ne pas stocker editable !
-	unset($valeurs['editable']);
+    $valeurs = [];
+    // charger les valeurs
+    // ce qui permet de prendre en charge une fonction charger() existante
+    // qui prend alors la main sur l'auto detection
+    if ($charger_valeurs = charger_fonction('charger', "formulaires/$form/", true)) {
+        $valeurs = $charger_valeurs(...$args);
+    }
+    $valeurs = pipeline(
+        'formulaire_charger',
+        [
+            'args' => ['form' => $form, 'args' => $args, 'je_suis_poste' => false],
+            'data' => $valeurs
+        ]
+    );
+    // ne pas stocker editable !
+    unset($valeurs['editable']);
 
-	// recuperer les valeurs postees
-	$store = [];
-	foreach ($valeurs as $k => $v) {
-		if (!str_starts_with($k, '_')) {
-			$store[$k] = _request($k);
-		}
-	}
+    // recuperer les valeurs postees
+    $store = [];
+    foreach ($valeurs as $k => $v) {
+        if (!str_starts_with($k, '_')) {
+            $store[$k] = _request($k);
+        }
+    }
 
-	return cvtconf_configurer_stocker($form, $valeurs, $store);
+    return cvtconf_configurer_stocker($form, $valeurs, $store);
 }
 
 /**
@@ -131,31 +131,31 @@ 
  * @return array
  */
 function cvtconf_definir_configurer_conteneur($form, $valeurs) {
-	// stocker en base
-	// par defaut, dans un casier serialize dans spip_meta (idem CFG)
-	$casier = substr($form, 11);
-	$table = 'meta';
-	$prefixe = '';
-	$stockage = '';
+    // stocker en base
+    // par defaut, dans un casier serialize dans spip_meta (idem CFG)
+    $casier = substr($form, 11);
+    $table = 'meta';
+    $prefixe = '';
+    $stockage = '';
 
-	if (isset($valeurs['_meta_casier'])) {
-		$casier = $valeurs['_meta_casier'];
-	}
-	if (isset($valeurs['_meta_prefixe'])) {
-		$prefixe = $valeurs['_meta_prefixe'];
-	}
-	if (isset($valeurs['_meta_stockage'])) {
-		$stockage = $valeurs['_meta_stockage'] . '::';
-	}
+    if (isset($valeurs['_meta_casier'])) {
+        $casier = $valeurs['_meta_casier'];
+    }
+    if (isset($valeurs['_meta_prefixe'])) {
+        $prefixe = $valeurs['_meta_prefixe'];
+    }
+    if (isset($valeurs['_meta_stockage'])) {
+        $stockage = $valeurs['_meta_stockage'] . '::';
+    }
 
-	// si on indique juste une table, il faut vider les autres proprietes
-	// car par defaut on utilise ni casier ni prefixe dans ce cas
-	if (isset($valeurs['_meta_table'])) {
-		$table = $valeurs['_meta_table'];
-		$casier = ($valeurs['_meta_casier'] ?? '');
-	}
+    // si on indique juste une table, il faut vider les autres proprietes
+    // car par defaut on utilise ni casier ni prefixe dans ce cas
+    if (isset($valeurs['_meta_table'])) {
+        $table = $valeurs['_meta_table'];
+        $casier = ($valeurs['_meta_casier'] ?? '');
+    }
 
-	return [$table, $casier, $prefixe, $stockage];
+    return [$table, $casier, $prefixe, $stockage];
 }
 
 /**
@@ -166,49 +166,49 @@ 
  * @return array
  */
 function cvtconf_formulaires_configurer_recense($form) {
-	$contenu = null;
-	$valeurs = ['editable' => ' '];
+    $contenu = null;
+    $valeurs = ['editable' => ' '];
 
-	// sinon cas analyse du squelette
-	if (
-		($f = find_in_path($form . '.' . _EXTENSION_SQUELETTES, 'formulaires/'))
-		&& lire_fichier($f, $contenu)
-	) {
-		for ($i = 0; $i < 2; $i++) {
-			// a la seconde iteration, evaluer le fond avec les valeurs deja trouvees
-			// permet de trouver aussi les name="#GET{truc}"
-			if ($i == 1) {
-				$contenu = recuperer_fond("formulaires/$form", $valeurs);
-			}
+    // sinon cas analyse du squelette
+    if (
+        ($f = find_in_path($form . '.' . _EXTENSION_SQUELETTES, 'formulaires/'))
+        && lire_fichier($f, $contenu)
+    ) {
+        for ($i = 0; $i < 2; $i++) {
+            // a la seconde iteration, evaluer le fond avec les valeurs deja trouvees
+            // permet de trouver aussi les name="#GET{truc}"
+            if ($i == 1) {
+                $contenu = recuperer_fond("formulaires/$form", $valeurs);
+            }
 
-			$balises = array_merge(
-				extraire_balises($contenu, 'input'),
-				extraire_balises($contenu, 'textarea'),
-				extraire_balises($contenu, 'select')
-			);
+            $balises = array_merge(
+                extraire_balises($contenu, 'input'),
+                extraire_balises($contenu, 'textarea'),
+                extraire_balises($contenu, 'select')
+            );
 
-			foreach ($balises as $b) {
-				if (
-					($n = extraire_attribut($b, 'name'))
-					&& preg_match(',^([\w\-]+)(\[\w*\])?$,', $n, $r)
-					&& !in_array($n, ['formulaire_action', 'formulaire_action_args', 'formulaire_action_sign'])
-					&& extraire_attribut($b, 'type') !== 'submit'
-				) {
-					$valeurs[$r[1]] = '';
-					// recuperer les valeurs _meta_xx qui peuvent etre fournies
-					// en input hidden dans le squelette
-					if (str_starts_with($r[1], '_meta_')) {
-						$valeurs[$r[1]] = extraire_attribut($b, 'value');
-					}
-				}
-			}
-		}
-	}
+            foreach ($balises as $b) {
+                if (
+                    ($n = extraire_attribut($b, 'name'))
+                    && preg_match(',^([\w\-]+)(\[\w*\])?$,', $n, $r)
+                    && !in_array($n, ['formulaire_action', 'formulaire_action_args', 'formulaire_action_sign'])
+                    && extraire_attribut($b, 'type') !== 'submit'
+                ) {
+                    $valeurs[$r[1]] = '';
+                    // recuperer les valeurs _meta_xx qui peuvent etre fournies
+                    // en input hidden dans le squelette
+                    if (str_starts_with($r[1], '_meta_')) {
+                        $valeurs[$r[1]] = extraire_attribut($b, 'value');
+                    }
+                }
+            }
+        }
+    }
 
 
-	cvtconf_configurer_lire_meta($form, $valeurs);
+    cvtconf_configurer_lire_meta($form, $valeurs);
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -220,26 +220,26 @@ 
  * @return string
  */
 function cvtconf_configurer_stocker($form, $valeurs, $store) {
-	$trace = '';
-	[$table, $casier, $prefixe, $stockage] = cvtconf_definir_configurer_conteneur($form, $valeurs);
-	// stocker en base
-	// par defaut, dans un casier serialize dans spip_meta (idem CFG)
-	if (!isset($GLOBALS[$table])) {
-		lire_metas($table);
-	}
+    $trace = '';
+    [$table, $casier, $prefixe, $stockage] = cvtconf_definir_configurer_conteneur($form, $valeurs);
+    // stocker en base
+    // par defaut, dans un casier serialize dans spip_meta (idem CFG)
+    if (!isset($GLOBALS[$table])) {
+        lire_metas($table);
+    }
 
-	$prefixe = ($prefixe ? $prefixe . '_' : '');
-	$table = ($table) ? "/$table/" : '';
-	$casier = ($casier) ? rtrim($casier, '/') . '/' : ''; // slash final, sinon rien
+    $prefixe = ($prefixe ? $prefixe . '_' : '');
+    $table = ($table) ? "/$table/" : '';
+    $casier = ($casier) ? rtrim($casier, '/') . '/' : ''; // slash final, sinon rien
 
-	foreach ($store as $k => $v) {
-		ecrire_config("$stockage$table$prefixe$casier$k", $v);
-		if (_request('var_mode') == 'configurer' && autoriser('webmestre')) {
-			$trace .= "<br />table $table : " . $prefixe . $k . " = $v;";
-		}
-	}
+    foreach ($store as $k => $v) {
+        ecrire_config("$stockage$table$prefixe$casier$k", $v);
+        if (_request('var_mode') == 'configurer' && autoriser('webmestre')) {
+            $trace .= "<br />table $table : " . $prefixe . $k . " = $v;";
+        }
+    }
 
-	return $trace;
+    return $trace;
 }
 
 /**
@@ -249,21 +249,21 @@ 
  * @param array $valeurs
  */
 function cvtconf_configurer_lire_meta($form, &$valeurs) {
-	[$table, $casier, $prefixe, $stockage] = cvtconf_definir_configurer_conteneur($form, $valeurs);
+    [$table, $casier, $prefixe, $stockage] = cvtconf_definir_configurer_conteneur($form, $valeurs);
 
-	$table = ($table) ? "/$table/" : '';
-	$prefixe = ($prefixe ? $prefixe . '_' : '');
-	if ($casier) {
-		$meta = lire_config("$stockage$table$prefixe$casier");
-		$prefixe = '';
-	} else {
-		$table = rtrim($table, '/');
-		$meta = lire_config("$stockage$table");
-	}
+    $table = ($table) ? "/$table/" : '';
+    $prefixe = ($prefixe ? $prefixe . '_' : '');
+    if ($casier) {
+        $meta = lire_config("$stockage$table$prefixe$casier");
+        $prefixe = '';
+    } else {
+        $table = rtrim($table, '/');
+        $meta = lire_config("$stockage$table");
+    }
 
-	foreach (array_keys($valeurs) as $k) {
-		if (!str_starts_with($k, '_')) {
-			$valeurs[$k] = ($meta[$prefixe . $k] ?? null);
-		}
-	}
+    foreach (array_keys($valeurs) as $k) {
+        if (!str_starts_with($k, '_')) {
+            $valeurs[$k] = ($meta[$prefixe . $k] ?? null);
+        }
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -37,7 +37,7 @@ 
 	) {
 		// Pour tous les formulaires CONFIGURER, ayant une fonction charger ou pas, on teste si autorisé
 		include_spip('inc/autoriser');
-		if (!autoriser('configurer', '_' . substr($form, 11))) {
+		if (!autoriser('configurer', '_'.substr($form, 11))) {
 			return false;
 		}
 
@@ -71,7 +71,7 @@ 
 		&& !charger_fonction('traiter', "formulaires/$form/", true) // sans fonction traiter()
 	) {
 		$trace = cvtconf_formulaires_configurer_enregistre($form, $flux['args']['args']);
-		$flux['data'] = ['message_ok' => _T('config_info_enregistree') . $trace, 'editable' => true];
+		$flux['data'] = ['message_ok' => _T('config_info_enregistree').$trace, 'editable' => true];
 	}
 
 	return $flux;
@@ -145,7 +145,7 @@ 
 		$prefixe = $valeurs['_meta_prefixe'];
 	}
 	if (isset($valeurs['_meta_stockage'])) {
-		$stockage = $valeurs['_meta_stockage'] . '::';
+		$stockage = $valeurs['_meta_stockage'].'::';
 	}
 
 	// si on indique juste une table, il faut vider les autres proprietes
@@ -171,7 +171,7 @@ 
 
 	// sinon cas analyse du squelette
 	if (
-		($f = find_in_path($form . '.' . _EXTENSION_SQUELETTES, 'formulaires/'))
+		($f = find_in_path($form.'.'._EXTENSION_SQUELETTES, 'formulaires/'))
 		&& lire_fichier($f, $contenu)
 	) {
 		for ($i = 0; $i < 2; $i++) {
@@ -228,14 +228,14 @@ 
 		lire_metas($table);
 	}
 
-	$prefixe = ($prefixe ? $prefixe . '_' : '');
+	$prefixe = ($prefixe ? $prefixe.'_' : '');
 	$table = ($table) ? "/$table/" : '';
-	$casier = ($casier) ? rtrim($casier, '/') . '/' : ''; // slash final, sinon rien
+	$casier = ($casier) ? rtrim($casier, '/').'/' : ''; // slash final, sinon rien
 
 	foreach ($store as $k => $v) {
 		ecrire_config("$stockage$table$prefixe$casier$k", $v);
 		if (_request('var_mode') == 'configurer' && autoriser('webmestre')) {
-			$trace .= "<br />table $table : " . $prefixe . $k . " = $v;";
+			$trace .= "<br />table $table : ".$prefixe.$k." = $v;";
 		}
 	}
 
@@ -252,7 +252,7 @@ 
 	[$table, $casier, $prefixe, $stockage] = cvtconf_definir_configurer_conteneur($form, $valeurs);
 
 	$table = ($table) ? "/$table/" : '';
-	$prefixe = ($prefixe ? $prefixe . '_' : '');
+	$prefixe = ($prefixe ? $prefixe.'_' : '');
 	if ($casier) {
 		$meta = lire_config("$stockage$table$prefixe$casier");
 		$prefixe = '';
@@ -263,7 +263,7 @@ 
 
 	foreach (array_keys($valeurs) as $k) {
 		if (!str_starts_with($k, '_')) {
-			$valeurs[$k] = ($meta[$prefixe . $k] ?? null);
+			$valeurs[$k] = ($meta[$prefixe.$k] ?? null);
 		}
 	}
 }