spip / SPIP

ecrire/inc/admin.php

Doc Comments +3 added lines

@@ -177,6 +177,9 @@
 }
 
 // http://doc.spip.org/@copy_request
+/**
+ * @param string $suite
+ */
 function copy_request($script, $suite, $submit='')
 {
 	include_spip('inc/filtres');

Indentation +119 added lines, -119 removed lines

@@ -19,22 +19,22 @@ 
 // http://doc.spip.org/@inc_admin_dist
 function inc_admin_dist($script, $titre, $comment='', $anonymous=false)
 {
-	$reprise = true;
-	if (!isset($GLOBALS['meta'][$script])
-	OR  !isset($GLOBALS['meta']['admin'])) {
-		$reprise = false;
-		$res = debut_admin($script, $titre, $comment); 
-		if ($res) return $res;
-		spip_log("meta: $script " . join(',', $_POST));
-		ecrire_meta($script, serialize($_POST));
-	} 
-
-	$res = admin_verifie_session($script,$anonymous);
-	if ($res) return $res;
-	$base = charger_fonction($script, 'base');
-	$base($titre,$reprise);
-	fin_admin($script);
-	return '';
+    $reprise = true;
+    if (!isset($GLOBALS['meta'][$script])
+    OR  !isset($GLOBALS['meta']['admin'])) {
+        $reprise = false;
+        $res = debut_admin($script, $titre, $comment); 
+        if ($res) return $res;
+        spip_log("meta: $script " . join(',', $_POST));
+        ecrire_meta($script, serialize($_POST));
+    } 
+
+    $res = admin_verifie_session($script,$anonymous);
+    if ($res) return $res;
+    $base = charger_fonction($script, 'base');
+    $base($titre,$reprise);
+    fin_admin($script);
+    return '';
 }
 
 // Gestion dans la meta "admin" du script d'administation demande,
@@ -55,46 +55,46 @@ 
 // http://doc.spip.org/@admin_verifie_session
 function admin_verifie_session($script, $anonymous=false) {
 
-	include_spip('base/abstract_sql');
-	$pref = sprintf("_%d_",$GLOBALS['visiteur_session']['id_auteur']);
-	$signal = fichier_admin($script, "$script$pref");
-	$valeur = sql_getfetsel('valeur', 'spip_meta', "nom='admin'");
-	if ($valeur === NULL) {
-		ecrire_meta('admin', $signal, 'non');
-	} else {
-		if (!$anonymous AND ($valeur != $signal)) {
-			if (!preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']["admin"], $l)
-				OR intval($l[2])!=$GLOBALS['visiteur_session']['id_auteur']) {
-				include_spip('inc/minipres');
-				spip_log("refus de lancer $script, priorite a $valeur");
-				return minipres(_T('info_travaux_texte'));
-			}
-		}
-	}
-	$journal = "spip";
-	if (autoriser('configurer')) // c'est une action webmestre, soit par ftp soit par statut webmestre
-		$journal = 'webmestre';
-	// on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
-
-	spip_log("admin $pref" . ($valeur ? " (reprise)" : ' (init)'),$journal);
-	return '';
+    include_spip('base/abstract_sql');
+    $pref = sprintf("_%d_",$GLOBALS['visiteur_session']['id_auteur']);
+    $signal = fichier_admin($script, "$script$pref");
+    $valeur = sql_getfetsel('valeur', 'spip_meta', "nom='admin'");
+    if ($valeur === NULL) {
+        ecrire_meta('admin', $signal, 'non');
+    } else {
+        if (!$anonymous AND ($valeur != $signal)) {
+            if (!preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']["admin"], $l)
+                OR intval($l[2])!=$GLOBALS['visiteur_session']['id_auteur']) {
+                include_spip('inc/minipres');
+                spip_log("refus de lancer $script, priorite a $valeur");
+                return minipres(_T('info_travaux_texte'));
+            }
+        }
+    }
+    $journal = "spip";
+    if (autoriser('configurer')) // c'est une action webmestre, soit par ftp soit par statut webmestre
+        $journal = 'webmestre';
+    // on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
+
+    spip_log("admin $pref" . ($valeur ? " (reprise)" : ' (init)'),$journal);
+    return '';
 }
 
 // http://doc.spip.org/@dir_admin
 function dir_admin()
 {
-	if (autoriser('configurer')) {
-		return _DIR_TMP;
-	} else {
-		return  _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
-	}
+    if (autoriser('configurer')) {
+        return _DIR_TMP;
+    } else {
+        return  _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
+    }
 }
 
 // http://doc.spip.org/@fichier_admin
 function fichier_admin($action, $pref='admin_') {
 
-	return $pref . 
-	  substr(md5($action.(time() & ~2047).$GLOBALS['visiteur_session']['login']), 0, 10);
+    return $pref . 
+        substr(md5($action.(time() & ~2047).$GLOBALS['visiteur_session']['login']), 0, 10);
 }
 
 // demande la creation d'un repertoire et sort
@@ -103,89 +103,89 @@ 
 // http://doc.spip.org/@debut_admin
 function debut_admin($script, $action='', $corps='') {
 
-	if ((!$action) || !(autoriser('webmestre') OR autoriser('chargerftp'))) {
-		include_spip('inc/minipres');
-		return minipres();
-	} else {
-		$dir = dir_admin();
-		$signal = fichier_admin($script);
-		if (@file_exists($dir . $signal)) {
-			spip_log ("Action admin: $action");
-			return '';
-		}
-		include_spip('inc/minipres');
-
-	// Si on est un super-admin, un bouton de validation suffit
-	// sauf dans les cas destroy
-		if ((autoriser('webmestre') OR $script === 'repair')
-		AND $script != 'delete_all') {
-			if (_request('validation_admin') == $signal) {
-				spip_log ("Action super-admin: $action");
-				return '';
-			}
-			$corps .= '<input type="hidden" name="validation_admin" value="'.$signal.'" />';
-			$suivant = _T('bouton_valider');
-			$js = '';
-		} else {
-			// cet appel permet d'assurer un copier-coller du nom du repertoire a creer dans tmp (esj)
-			// l'insertion du script a cet endroit n'est pas xhtml licite mais evite de l'embarquer dans toutes les pages minipres
-			$corps .= http_script('',  "spip_barre.js");
-
-			$corps .= "<fieldset><legend>"
-			. _T('info_authentification_ftp')
-			. aide("ftp_auth")
-			. "</legend>\n<label for='fichier'>"
-			. _T('info_creer_repertoire')
-			. "</label>\n"
+    if ((!$action) || !(autoriser('webmestre') OR autoriser('chargerftp'))) {
+        include_spip('inc/minipres');
+        return minipres();
+    } else {
+        $dir = dir_admin();
+        $signal = fichier_admin($script);
+        if (@file_exists($dir . $signal)) {
+            spip_log ("Action admin: $action");
+            return '';
+        }
+        include_spip('inc/minipres');
+
+    // Si on est un super-admin, un bouton de validation suffit
+    // sauf dans les cas destroy
+        if ((autoriser('webmestre') OR $script === 'repair')
+        AND $script != 'delete_all') {
+            if (_request('validation_admin') == $signal) {
+                spip_log ("Action super-admin: $action");
+                return '';
+            }
+            $corps .= '<input type="hidden" name="validation_admin" value="'.$signal.'" />';
+            $suivant = _T('bouton_valider');
+            $js = '';
+        } else {
+            // cet appel permet d'assurer un copier-coller du nom du repertoire a creer dans tmp (esj)
+            // l'insertion du script a cet endroit n'est pas xhtml licite mais evite de l'embarquer dans toutes les pages minipres
+            $corps .= http_script('',  "spip_barre.js");
+
+            $corps .= "<fieldset><legend>"
+            . _T('info_authentification_ftp')
+            . aide("ftp_auth")
+            . "</legend>\n<label for='fichier'>"
+            . _T('info_creer_repertoire')
+            . "</label>\n"
             . "<span id='signal' class='formo'>".$signal."</span>"
             . "<input type='hidden' id='fichier' name='fichier' value='" 
-			. $signal
-			. "' />"
-			. _T('info_creer_repertoire_2', array('repertoire' => joli_repertoire($dir)))
-			. "</fieldset>";
-
-			$suivant = _T('bouton_recharger_page');
-
-			// code volontairement tordu:
-			// provoquer la copie dans le presse papier du nom du repertoire
-			// en remettant a vide le champ pour que ca marche aussi en cas
-			// de JavaScript inactif.
-			$js = " onload='var range=document.createRange(); var signal = document.getElementById(\"signal\"); var userSelection = window.getSelection(); range.setStart(signal,0); range.setEnd(signal,1); userSelection.addRange(range);'";
-
-		}
-
-		// admin/xxx correspond
-		// a exec/base_xxx de preference
-		// et exec/xxx sinon (compat)
-		if (tester_url_ecrire("base_$script"))
-			$script = "base_$script";
-		$form = copy_request($script, $corps, $suivant);
-		$info_action = _T('info_action', array('action' => "$action"));
-		return minipres($info_action, $form, $js);
-	}
+            . $signal
+            . "' />"
+            . _T('info_creer_repertoire_2', array('repertoire' => joli_repertoire($dir)))
+            . "</fieldset>";
+
+            $suivant = _T('bouton_recharger_page');
+
+            // code volontairement tordu:
+            // provoquer la copie dans le presse papier du nom du repertoire
+            // en remettant a vide le champ pour que ca marche aussi en cas
+            // de JavaScript inactif.
+            $js = " onload='var range=document.createRange(); var signal = document.getElementById(\"signal\"); var userSelection = window.getSelection(); range.setStart(signal,0); range.setEnd(signal,1); userSelection.addRange(range);'";
+
+        }
+
+        // admin/xxx correspond
+        // a exec/base_xxx de preference
+        // et exec/xxx sinon (compat)
+        if (tester_url_ecrire("base_$script"))
+            $script = "base_$script";
+        $form = copy_request($script, $corps, $suivant);
+        $info_action = _T('info_action', array('action' => "$action"));
+        return minipres($info_action, $form, $js);
+    }
 }
 
 // http://doc.spip.org/@fin_admin
 function fin_admin($action) {
-	$signal = dir_admin() . fichier_admin($action);
-	spip_unlink($signal);
-	if ($action != 'delete_all') {
-		effacer_meta($action);
-		effacer_meta('admin');
-		spip_log("efface les meta admin et $action ");
-	}
+    $signal = dir_admin() . fichier_admin($action);
+    spip_unlink($signal);
+    if ($action != 'delete_all') {
+        effacer_meta($action);
+        effacer_meta('admin');
+        spip_log("efface les meta admin et $action ");
+    }
 }
 
 // http://doc.spip.org/@copy_request
 function copy_request($script, $suite, $submit='')
 {
-	include_spip('inc/filtres');
-	foreach(array_merge($_POST,$_GET) as $n => $c) {
-		if (!in_array($n,array('fichier','exec','validation_admin')) AND !is_array($c))
-		$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='" .
-			entites_html($c) .
-			"'  />";
-	}
-	return  generer_form_ecrire($script, $suite, '', $submit);
+    include_spip('inc/filtres');
+    foreach(array_merge($_POST,$_GET) as $n => $c) {
+        if (!in_array($n,array('fichier','exec','validation_admin')) AND !is_array($c))
+        $suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='" .
+            entites_html($c) .
+            "'  />";
+    }
+    return  generer_form_ecrire($script, $suite, '', $submit);
 }
 ?>

Spacing +22 added lines, -22 removed lines

@@ -17,7 +17,7 @@ 
 // si elle y est deja c'est qu'il y a eu suspension du script, on reprend.
 
 // http://doc.spip.org/@inc_admin_dist
-function inc_admin_dist($script, $titre, $comment='', $anonymous=false)
+function inc_admin_dist($script, $titre, $comment = '', $anonymous = false)
 {
 	$reprise = true;
 	if (!isset($GLOBALS['meta'][$script])
@@ -25,14 +25,14 @@ 
 		$reprise = false;
 		$res = debut_admin($script, $titre, $comment); 
 		if ($res) return $res;
-		spip_log("meta: $script " . join(',', $_POST));
+		spip_log("meta: $script ".join(',', $_POST));
 		ecrire_meta($script, serialize($_POST));
 	} 
 
-	$res = admin_verifie_session($script,$anonymous);
+	$res = admin_verifie_session($script, $anonymous);
 	if ($res) return $res;
 	$base = charger_fonction($script, 'base');
-	$base($titre,$reprise);
+	$base($titre, $reprise);
 	fin_admin($script);
 	return '';
 }
@@ -53,10 +53,10 @@ 
 // 	c'est que l'operation se passe mal, on la stoppe
 
 // http://doc.spip.org/@admin_verifie_session
-function admin_verifie_session($script, $anonymous=false) {
+function admin_verifie_session($script, $anonymous = false) {
 
 	include_spip('base/abstract_sql');
-	$pref = sprintf("_%d_",$GLOBALS['visiteur_session']['id_auteur']);
+	$pref = sprintf("_%d_", $GLOBALS['visiteur_session']['id_auteur']);
 	$signal = fichier_admin($script, "$script$pref");
 	$valeur = sql_getfetsel('valeur', 'spip_meta', "nom='admin'");
 	if ($valeur === NULL) {
@@ -64,7 +64,7 @@ 
 	} else {
 		if (!$anonymous AND ($valeur != $signal)) {
 			if (!preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']["admin"], $l)
-				OR intval($l[2])!=$GLOBALS['visiteur_session']['id_auteur']) {
+				OR intval($l[2]) != $GLOBALS['visiteur_session']['id_auteur']) {
 				include_spip('inc/minipres');
 				spip_log("refus de lancer $script, priorite a $valeur");
 				return minipres(_T('info_travaux_texte'));
@@ -76,7 +76,7 @@ 
 		$journal = 'webmestre';
 	// on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
 
-	spip_log("admin $pref" . ($valeur ? " (reprise)" : ' (init)'),$journal);
+	spip_log("admin $pref".($valeur ? " (reprise)" : ' (init)'), $journal);
 	return '';
 }
 
@@ -86,14 +86,14 @@ 
 	if (autoriser('configurer')) {
 		return _DIR_TMP;
 	} else {
-		return  _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
+		return  _DIR_TRANSFERT.$GLOBALS['visiteur_session']['login'].'/';
 	}
 }
 
 // http://doc.spip.org/@fichier_admin
-function fichier_admin($action, $pref='admin_') {
+function fichier_admin($action, $pref = 'admin_') {
 
-	return $pref . 
+	return $pref. 
 	  substr(md5($action.(time() & ~2047).$GLOBALS['visiteur_session']['login']), 0, 10);
 }
 
@@ -101,7 +101,7 @@ 
 // ou retourne sans rien faire si repertoire deja la.
 
 // http://doc.spip.org/@debut_admin
-function debut_admin($script, $action='', $corps='') {
+function debut_admin($script, $action = '', $corps = '') {
 
 	if ((!$action) || !(autoriser('webmestre') OR autoriser('chargerftp'))) {
 		include_spip('inc/minipres');
@@ -109,8 +109,8 @@ 
 	} else {
 		$dir = dir_admin();
 		$signal = fichier_admin($script);
-		if (@file_exists($dir . $signal)) {
-			spip_log ("Action admin: $action");
+		if (@file_exists($dir.$signal)) {
+			spip_log("Action admin: $action");
 			return '';
 		}
 		include_spip('inc/minipres');
@@ -120,7 +120,7 @@ 
 		if ((autoriser('webmestre') OR $script === 'repair')
 		AND $script != 'delete_all') {
 			if (_request('validation_admin') == $signal) {
-				spip_log ("Action super-admin: $action");
+				spip_log("Action super-admin: $action");
 				return '';
 			}
 			$corps .= '<input type="hidden" name="validation_admin" value="'.$signal.'" />';
@@ -129,7 +129,7 @@ 
 		} else {
 			// cet appel permet d'assurer un copier-coller du nom du repertoire a creer dans tmp (esj)
 			// l'insertion du script a cet endroit n'est pas xhtml licite mais evite de l'embarquer dans toutes les pages minipres
-			$corps .= http_script('',  "spip_barre.js");
+			$corps .= http_script('', "spip_barre.js");
 
 			$corps .= "<fieldset><legend>"
 			. _T('info_authentification_ftp')
@@ -167,7 +167,7 @@ 
 
 // http://doc.spip.org/@fin_admin
 function fin_admin($action) {
-	$signal = dir_admin() . fichier_admin($action);
+	$signal = dir_admin().fichier_admin($action);
 	spip_unlink($signal);
 	if ($action != 'delete_all') {
 		effacer_meta($action);
@@ -177,13 +177,13 @@ 
 }
 
 // http://doc.spip.org/@copy_request
-function copy_request($script, $suite, $submit='')
+function copy_request($script, $suite, $submit = '')
 {
 	include_spip('inc/filtres');
-	foreach(array_merge($_POST,$_GET) as $n => $c) {
-		if (!in_array($n,array('fichier','exec','validation_admin')) AND !is_array($c))
-		$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='" .
-			entites_html($c) .
+	foreach (array_merge($_POST, $_GET) as $n => $c) {
+		if (!in_array($n, array('fichier', 'exec', 'validation_admin')) AND !is_array($c))
+		$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='".
+			entites_html($c).
 			"'  />";
 	}
 	return  generer_form_ecrire($script, $suite, '', $submit);

Braces +18 added lines, -8 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // demande/verifie le droit de creation de repertoire par le demandeur;
 // memorise dans les meta que ce script est en cours d'execution
@@ -24,13 +26,17 @@ 
 	OR  !isset($GLOBALS['meta']['admin'])) {
 		$reprise = false;
 		$res = debut_admin($script, $titre, $comment); 
-		if ($res) return $res;
+		if ($res) {
+		    return $res;
+		}
 		spip_log("meta: $script " . join(',', $_POST));
 		ecrire_meta($script, serialize($_POST));
 	} 
 
 	$res = admin_verifie_session($script,$anonymous);
-	if ($res) return $res;
+	if ($res) {
+	    return $res;
+	}
 	$base = charger_fonction($script, 'base');
 	$base($titre,$reprise);
 	fin_admin($script);
@@ -72,8 +78,10 @@ 
 		}
 	}
 	$journal = "spip";
-	if (autoriser('configurer')) // c'est une action webmestre, soit par ftp soit par statut webmestre
+	if (autoriser('configurer')) {
+	    // c'est une action webmestre, soit par ftp soit par statut webmestre
 		$journal = 'webmestre';
+	}
 	// on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
 
 	spip_log("admin $pref" . ($valeur ? " (reprise)" : ' (init)'),$journal);
@@ -157,8 +165,9 @@ 
 		// admin/xxx correspond
 		// a exec/base_xxx de preference
 		// et exec/xxx sinon (compat)
-		if (tester_url_ecrire("base_$script"))
-			$script = "base_$script";
+		if (tester_url_ecrire("base_$script")) {
+					$script = "base_$script";
+		}
 		$form = copy_request($script, $corps, $suivant);
 		$info_action = _T('info_action', array('action' => "$action"));
 		return minipres($info_action, $form, $js);
@@ -181,10 +190,11 @@ 
 {
 	include_spip('inc/filtres');
 	foreach(array_merge($_POST,$_GET) as $n => $c) {
-		if (!in_array($n,array('fichier','exec','validation_admin')) AND !is_array($c))
-		$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='" .
+		if (!in_array($n,array('fichier','exec','validation_admin')) AND !is_array($c)) {
+				$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='" .
 			entites_html($c) .
 			"'  />";
+		}
 	}
 	return  generer_form_ecrire($script, $suite, '', $submit);
 }

ecrire/inc/charsets.php

Doc Comments +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
  * @param string $charset
  *     Charset à charger
  *     Par défaut (AUTO), utilise le charset du site
- * @return string|bool
+ * @return string|false
  *     Nom du charset
  *     false si le charset n'est pas décrit dans le répertoire charsets/
 **/
@@ -644,7 +644,7 @@ 
  * 
  * @param int $num
  *    Numéro de l'entité unicode
- * @return char
+ * @return string
  *    Caractère utf8 si trouvé, '' sinon
 **/
 function caractere_utf_8($num) {

Indentation +585 added lines, -585 removed lines

@@ -38,30 +38,30 @@ 
  *     false si le charset n'est pas décrit dans le répertoire charsets/
 **/
 function load_charset ($charset = 'AUTO') {
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
-	$charset = trim(strtolower($charset));
-	if (isset($GLOBALS['CHARSET'][$charset]))
-		return $charset;
-
-	if ($charset == 'utf-8') {
-		$GLOBALS['CHARSET'][$charset] = array();
-		return $charset;
-	}
+    if ($charset == 'AUTO')
+        $charset = $GLOBALS['meta']['charset'];
+    $charset = trim(strtolower($charset));
+    if (isset($GLOBALS['CHARSET'][$charset]))
+        return $charset;
+
+    if ($charset == 'utf-8') {
+        $GLOBALS['CHARSET'][$charset] = array();
+        return $charset;
+    }
 	
-	// Quelques synonymes
-	if ($charset == '') $charset = 'iso-8859-1';
-	else if ($charset == 'windows-1250') $charset = 'cp1250';
-	else if ($charset == 'windows-1251') $charset = 'cp1251';
-	else if ($charset == 'windows-1256') $charset = 'cp1256';
-
-	if (find_in_path($charset . '.php', 'charsets/', true)) {
-		return $charset;
-	} else {
-		spip_log("Erreur: pas de fichier de conversion 'charsets/$charset'");
-		$GLOBALS['CHARSET'][$charset] = array();
-		return false;
-	}
+    // Quelques synonymes
+    if ($charset == '') $charset = 'iso-8859-1';
+    else if ($charset == 'windows-1250') $charset = 'cp1250';
+    else if ($charset == 'windows-1251') $charset = 'cp1251';
+    else if ($charset == 'windows-1256') $charset = 'cp1256';
+
+    if (find_in_path($charset . '.php', 'charsets/', true)) {
+        return $charset;
+    } else {
+        spip_log("Erreur: pas de fichier de conversion 'charsets/$charset'");
+        $GLOBALS['CHARSET'][$charset] = array();
+        return false;
+    }
 }
 
 
@@ -72,27 +72,27 @@ 
  *     true si toutes les fonctions mb nécessaires sont présentes
 **/
 function init_mb_string() {
-	static $mb;
-
-	// verifier que tout est present (fonctions mb_string pour php >= 4.0.6)
-	// et que le charset interne est connu de mb_string
-	if (!$mb) {
-		if (function_exists('mb_internal_encoding')
-		AND function_exists('mb_detect_order')
-		AND function_exists('mb_substr')
-		AND function_exists('mb_strlen')
-		AND function_exists('mb_encode_mimeheader')
-		AND function_exists('mb_encode_numericentity')
-		AND function_exists('mb_decode_numericentity')
-		AND mb_detect_order($GLOBALS['meta']['charset'])
-		) {
-			mb_internal_encoding('utf-8');
-			$mb = 1;
-		} else
-			$mb = -1;
-	}
-
-	return ($mb == 1);
+    static $mb;
+
+    // verifier que tout est present (fonctions mb_string pour php >= 4.0.6)
+    // et que le charset interne est connu de mb_string
+    if (!$mb) {
+        if (function_exists('mb_internal_encoding')
+        AND function_exists('mb_detect_order')
+        AND function_exists('mb_substr')
+        AND function_exists('mb_strlen')
+        AND function_exists('mb_encode_mimeheader')
+        AND function_exists('mb_encode_numericentity')
+        AND function_exists('mb_decode_numericentity')
+        AND mb_detect_order($GLOBALS['meta']['charset'])
+        ) {
+            mb_internal_encoding('utf-8');
+            $mb = 1;
+        } else
+            $mb = -1;
+    }
+
+    return ($mb == 1);
 }
 
 /**
@@ -107,19 +107,19 @@ 
  *     true si iconv fonctionne correctement
 **/
 function test_iconv() {
-	static $iconv_ok;
-
-	if (!$iconv_ok) {
-		if (!function_exists('iconv'))
-			$iconv_ok = -1;
-		else {
-			if (utf_32_to_unicode(@iconv('utf-8', 'utf-32', 'chaine de test')) == 'chaine de test')
-				$iconv_ok = 1;
-			else
-				$iconv_ok = -1;
-		}
-	}
-	return ($iconv_ok == 1);
+    static $iconv_ok;
+
+    if (!$iconv_ok) {
+        if (!function_exists('iconv'))
+            $iconv_ok = -1;
+        else {
+            if (utf_32_to_unicode(@iconv('utf-8', 'utf-32', 'chaine de test')) == 'chaine de test')
+                $iconv_ok = 1;
+            else
+                $iconv_ok = -1;
+        }
+    }
+    return ($iconv_ok == 1);
 }
 
 
@@ -132,14 +132,14 @@ 
  *     true si PCRE supporte l'UTF-8 correctement
 **/
 function test_pcre_unicode() {
-	static $pcre_ok = 0;
-
-	if (!$pcre_ok) {
-		$s = " ".chr(195).chr(169)."t".chr(195).chr(169)." ";
-		if (preg_match(',\W...\W,u', $s)) $pcre_ok = 1;
-		else $pcre_ok = -1;
-	}
-	return $pcre_ok == 1;
+    static $pcre_ok = 0;
+
+    if (!$pcre_ok) {
+        $s = " ".chr(195).chr(169)."t".chr(195).chr(169)." ";
+        if (preg_match(',\W...\W,u', $s)) $pcre_ok = 1;
+        else $pcre_ok = -1;
+    }
+    return $pcre_ok == 1;
 }
 
 /**
@@ -155,22 +155,22 @@ 
  *    Plage de caractères
 **/
 function pcre_lettres_unicode() {
-	static $plage_unicode;
-
-	if (!$plage_unicode) {
-		if (test_pcre_unicode()) {
-			// cf. http://www.unicode.org/charts/
-			$plage_unicode = '\w' // iso-latin
-				. '\x{100}-\x{24f}' // europeen etendu
-				. '\x{300}-\x{1cff}' // des tas de trucs
-			;
-		}
-		else {
-			// fallback a trois sous
-			$plage_unicode = '\w';
-		}
-	}
-	return $plage_unicode;
+    static $plage_unicode;
+
+    if (!$plage_unicode) {
+        if (test_pcre_unicode()) {
+            // cf. http://www.unicode.org/charts/
+            $plage_unicode = '\w' // iso-latin
+                . '\x{100}-\x{24f}' // europeen etendu
+                . '\x{300}-\x{1cff}' // des tas de trucs
+            ;
+        }
+        else {
+            // fallback a trois sous
+            $plage_unicode = '\w';
+        }
+    }
+    return $plage_unicode;
 }
 
 
@@ -188,7 +188,7 @@ 
  *    Plage de caractères
 **/
 function plage_punct_unicode() {
-	return '\xE2(\x80[\x80-\xBF]|\x81[\x80-\xAF])';
+    return '\xE2(\x80[\x80-\xBF]|\x81[\x80-\xAF])';
 }
 
 /**
@@ -208,65 +208,65 @@ 
  *     Texte corrigé
 **/
 function corriger_caracteres_windows($texte, $charset='AUTO', $charset_cible='unicode') {
-	static $trans;
+    static $trans;
 	
-	if (is_array($texte)) {
-		return array_map('corriger_caracteres_windows', $texte);
-	}
+    if (is_array($texte)) {
+        return array_map('corriger_caracteres_windows', $texte);
+    }
 	
-	if ($charset=='AUTO') $charset = $GLOBALS['meta']['charset'];
-	if ($charset == 'utf-8') {
-		$p = chr(194);
-		if (strpos($texte,$p)==false)
-			return $texte;
-	} else if ($charset == 'iso-8859-1') {
-		$p = '';
-	} else
-		return $texte;
-
-	if (!isset($trans[$charset][$charset_cible])) {
-		$trans[$charset][$charset_cible] = array(
-			$p.chr(128) => "€",
-			$p.chr(129) => ' ', # pas affecte
-			$p.chr(130) => "‚",
-			$p.chr(131) => "ƒ",
-			$p.chr(132) => "„",
-			$p.chr(133) => "…",
-			$p.chr(134) => "†",
-			$p.chr(135) => "‡",
-			$p.chr(136) => "ˆ",
-			$p.chr(137) => "‰",
-			$p.chr(138) => "Š",
-			$p.chr(139) => "‹",
-			$p.chr(140) => "Œ",
-			$p.chr(141) => ' ', # pas affecte
-			$p.chr(142) => "Ž",
-			$p.chr(143) => ' ', # pas affecte
-			$p.chr(144) => ' ', # pas affecte
-			$p.chr(145) => "‘",
-			$p.chr(146) => "’",
-			$p.chr(147) => "“",
-			$p.chr(148) => "”",
-			$p.chr(149) => "•",
-			$p.chr(150) => "–",
-			$p.chr(151) => "—",
-			$p.chr(152) => "˜",
-			$p.chr(153) => "™", 
-			$p.chr(154) => "š",
-			$p.chr(155) => "›",
-			$p.chr(156) => "œ",
-			$p.chr(157) => ' ', # pas affecte
-			$p.chr(158) => "ž",
-			$p.chr(159) => "Ÿ",
-		);
-		if ($charset_cible!='unicode'){
-			foreach($trans[$charset][$charset_cible] as $k=>$c)
-				$trans[$charset][$charset_cible][$k] = unicode2charset($c, $charset_cible);
-		}
-	}
-
-	return @str_replace(array_keys($trans[$charset][$charset_cible]),
-			   array_values($trans[$charset][$charset_cible]),$texte);
+    if ($charset=='AUTO') $charset = $GLOBALS['meta']['charset'];
+    if ($charset == 'utf-8') {
+        $p = chr(194);
+        if (strpos($texte,$p)==false)
+            return $texte;
+    } else if ($charset == 'iso-8859-1') {
+        $p = '';
+    } else
+        return $texte;
+
+    if (!isset($trans[$charset][$charset_cible])) {
+        $trans[$charset][$charset_cible] = array(
+            $p.chr(128) => "€",
+            $p.chr(129) => ' ', # pas affecte
+            $p.chr(130) => "‚",
+            $p.chr(131) => "ƒ",
+            $p.chr(132) => "„",
+            $p.chr(133) => "…",
+            $p.chr(134) => "†",
+            $p.chr(135) => "‡",
+            $p.chr(136) => "ˆ",
+            $p.chr(137) => "‰",
+            $p.chr(138) => "Š",
+            $p.chr(139) => "‹",
+            $p.chr(140) => "Œ",
+            $p.chr(141) => ' ', # pas affecte
+            $p.chr(142) => "Ž",
+            $p.chr(143) => ' ', # pas affecte
+            $p.chr(144) => ' ', # pas affecte
+            $p.chr(145) => "‘",
+            $p.chr(146) => "’",
+            $p.chr(147) => "“",
+            $p.chr(148) => "”",
+            $p.chr(149) => "•",
+            $p.chr(150) => "–",
+            $p.chr(151) => "—",
+            $p.chr(152) => "˜",
+            $p.chr(153) => "™", 
+            $p.chr(154) => "š",
+            $p.chr(155) => "›",
+            $p.chr(156) => "œ",
+            $p.chr(157) => ' ', # pas affecte
+            $p.chr(158) => "ž",
+            $p.chr(159) => "Ÿ",
+        );
+        if ($charset_cible!='unicode'){
+            foreach($trans[$charset][$charset_cible] as $k=>$c)
+                $trans[$charset][$charset_cible][$k] = unicode2charset($c, $charset_cible);
+        }
+    }
+
+    return @str_replace(array_keys($trans[$charset][$charset_cible]),
+                array_values($trans[$charset][$charset_cible]),$texte);
 }
 
 
@@ -284,22 +284,22 @@ 
  *     Texte converti
 **/
 function html2unicode($texte, $secure=false) {
-	if (strpos($texte,'&') === false) return $texte;
-	static $trans = array();
-	if (!$trans) {
-		global $CHARSET;
-		load_charset('html');
-		foreach ($CHARSET['html'] as $key => $val) {
-			$trans["&$key;"] = $val;
-		}
-	}
-
-	if ($secure)
-		return str_replace(array_keys($trans),array_values($trans),$texte);
-	else
-		return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'),array('&', '"', '<', '>'),
-		  str_replace(array_keys($trans),array_values($trans),$texte)			
-		);
+    if (strpos($texte,'&') === false) return $texte;
+    static $trans = array();
+    if (!$trans) {
+        global $CHARSET;
+        load_charset('html');
+        foreach ($CHARSET['html'] as $key => $val) {
+            $trans["&$key;"] = $val;
+        }
+    }
+
+    if ($secure)
+        return str_replace(array_keys($trans),array_values($trans),$texte);
+    else
+        return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'),array('&', '"', '<', '>'),
+            str_replace(array_keys($trans),array_values($trans),$texte)			
+        );
 }
 
 
@@ -314,16 +314,16 @@ 
  *     Texte converti
 **/
 function mathml2unicode($texte) {
-	static $trans;
-	if (!$trans) {
-		global $CHARSET;
-		load_charset('mathml');
+    static $trans;
+    if (!$trans) {
+        global $CHARSET;
+        load_charset('mathml');
 		
-		foreach ($CHARSET['mathml'] as $key => $val)
-			$trans["&$key;"] = $val;
-	}
+        foreach ($CHARSET['mathml'] as $key => $val)
+            $trans["&$key;"] = $val;
+    }
 
-	return str_replace(array_keys($trans),array_values($trans),$texte);
+    return str_replace(array_keys($trans),array_values($trans),$texte);
 }
 
 
@@ -345,58 +345,58 @@ 
  *     Texte converti en unicode
 **/
 function charset2unicode($texte, $charset='AUTO' /* $forcer: obsolete*/) {
-	static $trans;
-
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
-
-	if ($charset == '') $charset = 'iso-8859-1';
-	$charset = strtolower($charset);
-
-	switch ($charset) {
-	case 'utf-8':
-	case 'utf8':
-		return utf_8_to_unicode($texte);
-
-	case 'iso-8859-1':
-		$texte = corriger_caracteres_windows($texte, 'iso-8859-1');
-		// pas de break; ici, on suit sur default:
-
-	default:
-		// mbstring presente ?
-		if (init_mb_string()) {
-			if ($order = mb_detect_order() # mb_string connait-il $charset?
-			AND mb_detect_order($charset)) {
-				$s = mb_convert_encoding($texte, 'utf-8', $charset);
-				if ($s && $s != $texte) return utf_8_to_unicode($s);
-			}
-			mb_detect_order($order); # remettre comme precedemment
-		}
-
-		// Sinon, peut-etre connaissons-nous ce charset ?
-		if (!isset($trans[$charset])) {
-			global $CHARSET;
-			if ($cset = load_charset($charset)
-			AND is_array($CHARSET[$cset]))
-				foreach ($CHARSET[$cset] as $key => $val) {
-					$trans[$charset][chr($key)] = '&#'.$val.';';
-			}
-		}
-		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
-
-		// Sinon demander a iconv (malgre le fait qu'il coupe quand un
-		// caractere n'appartient pas au charset, mais c'est un probleme
-		// surtout en utf-8, gere ci-dessus)
-		if (test_iconv()) {
-			$s = iconv($charset, 'utf-32le', $texte);
-			if ($s) return utf_32_to_unicode($s);
-		}
-
-		// Au pire ne rien faire
-		spip_log("erreur charset '$charset' non supporte");
-		return $texte;
-	}
+    static $trans;
+
+    if ($charset == 'AUTO')
+        $charset = $GLOBALS['meta']['charset'];
+
+    if ($charset == '') $charset = 'iso-8859-1';
+    $charset = strtolower($charset);
+
+    switch ($charset) {
+    case 'utf-8':
+    case 'utf8':
+        return utf_8_to_unicode($texte);
+
+    case 'iso-8859-1':
+        $texte = corriger_caracteres_windows($texte, 'iso-8859-1');
+        // pas de break; ici, on suit sur default:
+
+    default:
+        // mbstring presente ?
+        if (init_mb_string()) {
+            if ($order = mb_detect_order() # mb_string connait-il $charset?
+            AND mb_detect_order($charset)) {
+                $s = mb_convert_encoding($texte, 'utf-8', $charset);
+                if ($s && $s != $texte) return utf_8_to_unicode($s);
+            }
+            mb_detect_order($order); # remettre comme precedemment
+        }
+
+        // Sinon, peut-etre connaissons-nous ce charset ?
+        if (!isset($trans[$charset])) {
+            global $CHARSET;
+            if ($cset = load_charset($charset)
+            AND is_array($CHARSET[$cset]))
+                foreach ($CHARSET[$cset] as $key => $val) {
+                    $trans[$charset][chr($key)] = '&#'.$val.';';
+            }
+        }
+        if (count($trans[$charset]))
+            return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+
+        // Sinon demander a iconv (malgre le fait qu'il coupe quand un
+        // caractere n'appartient pas au charset, mais c'est un probleme
+        // surtout en utf-8, gere ci-dessus)
+        if (test_iconv()) {
+            $s = iconv($charset, 'utf-32le', $texte);
+            if ($s) return utf_32_to_unicode($s);
+        }
+
+        // Au pire ne rien faire
+        spip_log("erreur charset '$charset' non supporte");
+        return $texte;
+    }
 }
 
 
@@ -415,43 +415,43 @@ 
  *     Texte transformé dans le charset souhaité
 **/
 function unicode2charset($texte, $charset='AUTO') {
-	static $CHARSET_REVERSE;
-	static $trans = array();
+    static $CHARSET_REVERSE;
+    static $trans = array();
 	
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
-
-	switch($charset) {
-	case 'utf-8':
-		return unicode_to_utf_8($texte);
-		break;
-
-	default:
-		$charset = load_charset($charset);
-
-		if (!is_array($CHARSET_REVERSE[$charset])) {
-			$CHARSET_REVERSE[$charset] = array_flip($GLOBALS['CHARSET'][$charset]);
-		}
-
-		if (!isset($trans[$charset])){
-			$trans[$charset]=array();
-			$t = &$trans[$charset];
-			for($e=128;$e<255;$e++){
-				$h = dechex($e);
-				if ($s = isset($CHARSET_REVERSE[$charset][$e])){
-					$s = $CHARSET_REVERSE[$charset][$e];
-					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
-					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
-				}
-				else{
-					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
-					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
-				}
-			}
-		}
-		$texte = str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
-		return $texte;
-	}
+    if ($charset == 'AUTO')
+        $charset = $GLOBALS['meta']['charset'];
+
+    switch($charset) {
+    case 'utf-8':
+        return unicode_to_utf_8($texte);
+        break;
+
+    default:
+        $charset = load_charset($charset);
+
+        if (!is_array($CHARSET_REVERSE[$charset])) {
+            $CHARSET_REVERSE[$charset] = array_flip($GLOBALS['CHARSET'][$charset]);
+        }
+
+        if (!isset($trans[$charset])){
+            $trans[$charset]=array();
+            $t = &$trans[$charset];
+            for($e=128;$e<255;$e++){
+                $h = dechex($e);
+                if ($s = isset($CHARSET_REVERSE[$charset][$e])){
+                    $s = $CHARSET_REVERSE[$charset][$e];
+                    $t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
+                    $t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
+                }
+                else{
+                    $t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
+                    $t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
+                }
+            }
+        }
+        $texte = str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+        return $texte;
+    }
 }
 
 
@@ -469,32 +469,32 @@ 
  *     Texte transformé dans le charset site
 **/
 function importer_charset($texte, $charset = 'AUTO') {
-	static $trans = array();
-	// on traite le cas le plus frequent iso-8859-1 vers utf directement pour aller plus vite !
-	if (($charset == 'iso-8859-1') && ($GLOBALS['meta']['charset']=='utf-8')){
-		$texte = corriger_caracteres_windows($texte, 'iso-8859-1',$GLOBALS['meta']['charset']);
-		if (init_mb_string()) {
-			if ($order = mb_detect_order() # mb_string connait-il $charset?
-			AND mb_detect_order($charset)) {
-				$s = mb_convert_encoding($texte, 'utf-8', $charset);
-			}
-			mb_detect_order($order); # remettre comme precedemment
-			return $s;
-		}
-		// Sinon, peut-etre connaissons-nous ce charset ?
-		if (!isset($trans[$charset])) {
-			global $CHARSET;
-			if ($cset = load_charset($charset)
-			AND is_array($CHARSET[$cset]))
-				foreach ($CHARSET[$cset] as $key => $val) {
-					$trans[$charset][chr($key)] = unicode2charset('&#'.$val.';');
-			}
-		}
-		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
-		return $texte;
-	}
-	return unicode2charset(charset2unicode($texte, $charset));
+    static $trans = array();
+    // on traite le cas le plus frequent iso-8859-1 vers utf directement pour aller plus vite !
+    if (($charset == 'iso-8859-1') && ($GLOBALS['meta']['charset']=='utf-8')){
+        $texte = corriger_caracteres_windows($texte, 'iso-8859-1',$GLOBALS['meta']['charset']);
+        if (init_mb_string()) {
+            if ($order = mb_detect_order() # mb_string connait-il $charset?
+            AND mb_detect_order($charset)) {
+                $s = mb_convert_encoding($texte, 'utf-8', $charset);
+            }
+            mb_detect_order($order); # remettre comme precedemment
+            return $s;
+        }
+        // Sinon, peut-etre connaissons-nous ce charset ?
+        if (!isset($trans[$charset])) {
+            global $CHARSET;
+            if ($cset = load_charset($charset)
+            AND is_array($CHARSET[$cset]))
+                foreach ($CHARSET[$cset] as $key => $val) {
+                    $trans[$charset][chr($key)] = unicode2charset('&#'.$val.';');
+            }
+        }
+        if (count($trans[$charset]))
+            return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+        return $texte;
+    }
+    return unicode2charset(charset2unicode($texte, $charset));
 }
 
 
@@ -510,89 +510,89 @@ 
 **/
 function utf_8_to_unicode($source) {
 
-	// mb_string : methode rapide
-	if (init_mb_string()) {
-		$convmap = array(0x7F, 0xFFFFFF, 0x0, 0xFFFFFF);
-		return mb_encode_numericentity($source, $convmap, 'UTF-8');
-	}
-
-	// Sinon methode pas a pas
-	static $decrement;
-	static $shift;
-
-	// Cf. php.net, par Ronen. Adapte pour compatibilite < php4
-	if (!is_array($decrement)) {
-		// array used to figure what number to decrement from character order value
-		// according to number of characters used to map unicode to ascii by utf-8
-		$decrement[4] = 240;
-		$decrement[3] = 224;
-		$decrement[2] = 192;
-		$decrement[1] = 0;
-		// the number of bits to shift each charNum by
-		$shift[1][0] = 0;
-		$shift[2][0] = 6;
-		$shift[2][1] = 0;
-		$shift[3][0] = 12;
-		$shift[3][1] = 6;
-		$shift[3][2] = 0;
-		$shift[4][0] = 18;
-		$shift[4][1] = 12;
-		$shift[4][2] = 6;
-		$shift[4][3] = 0;
-	}
-
-	$pos = 0;
-	$len = strlen ($source);
-	$encodedString = '';
-	while ($pos < $len) {
-		$char = '';
-		$ischar = false;
-		$asciiPos = ord (substr ($source, $pos, 1));
-		if (($asciiPos >= 240) && ($asciiPos <= 255)) {
-			// 4 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 4);
-			$pos += 4;
-		}
-		else if (($asciiPos >= 224) && ($asciiPos <= 239)) {
-			// 3 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 3);
-			$pos += 3;
-		}
-		else if (($asciiPos >= 192) && ($asciiPos <= 223)) {
-			// 2 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 2);
-			$pos += 2;
-		}
-		else {
-			// 1 char (lower ascii)
-			$thisLetter = substr ($source, $pos, 1);
-			$pos += 1;
-			$char = $thisLetter;
-			$ischar = true;
-		}
-
-		if ($ischar)
-			$encodedString .= $char;
-		else {	// process the string representing the letter to a unicode entity
-			$thisLen = strlen ($thisLetter);
-			$thisPos = 0;
-			$decimalCode = 0;
-			while ($thisPos < $thisLen) {
-				$thisCharOrd = ord (substr ($thisLetter, $thisPos, 1));
-				if ($thisPos == 0) {
-					$charNum = intval ($thisCharOrd - $decrement[$thisLen]);
-					$decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
-				} else {
-					$charNum = intval ($thisCharOrd - 128);
-					$decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
-				}
-				$thisPos++;
-			}
-			$encodedLetter = "&#". preg_replace('/^0+/', '', $decimalCode) . ';';
-			$encodedString .= $encodedLetter;
-		}
-	}
-	return $encodedString;
+    // mb_string : methode rapide
+    if (init_mb_string()) {
+        $convmap = array(0x7F, 0xFFFFFF, 0x0, 0xFFFFFF);
+        return mb_encode_numericentity($source, $convmap, 'UTF-8');
+    }
+
+    // Sinon methode pas a pas
+    static $decrement;
+    static $shift;
+
+    // Cf. php.net, par Ronen. Adapte pour compatibilite < php4
+    if (!is_array($decrement)) {
+        // array used to figure what number to decrement from character order value
+        // according to number of characters used to map unicode to ascii by utf-8
+        $decrement[4] = 240;
+        $decrement[3] = 224;
+        $decrement[2] = 192;
+        $decrement[1] = 0;
+        // the number of bits to shift each charNum by
+        $shift[1][0] = 0;
+        $shift[2][0] = 6;
+        $shift[2][1] = 0;
+        $shift[3][0] = 12;
+        $shift[3][1] = 6;
+        $shift[3][2] = 0;
+        $shift[4][0] = 18;
+        $shift[4][1] = 12;
+        $shift[4][2] = 6;
+        $shift[4][3] = 0;
+    }
+
+    $pos = 0;
+    $len = strlen ($source);
+    $encodedString = '';
+    while ($pos < $len) {
+        $char = '';
+        $ischar = false;
+        $asciiPos = ord (substr ($source, $pos, 1));
+        if (($asciiPos >= 240) && ($asciiPos <= 255)) {
+            // 4 chars representing one unicode character
+            $thisLetter = substr ($source, $pos, 4);
+            $pos += 4;
+        }
+        else if (($asciiPos >= 224) && ($asciiPos <= 239)) {
+            // 3 chars representing one unicode character
+            $thisLetter = substr ($source, $pos, 3);
+            $pos += 3;
+        }
+        else if (($asciiPos >= 192) && ($asciiPos <= 223)) {
+            // 2 chars representing one unicode character
+            $thisLetter = substr ($source, $pos, 2);
+            $pos += 2;
+        }
+        else {
+            // 1 char (lower ascii)
+            $thisLetter = substr ($source, $pos, 1);
+            $pos += 1;
+            $char = $thisLetter;
+            $ischar = true;
+        }
+
+        if ($ischar)
+            $encodedString .= $char;
+        else {	// process the string representing the letter to a unicode entity
+            $thisLen = strlen ($thisLetter);
+            $thisPos = 0;
+            $decimalCode = 0;
+            while ($thisPos < $thisLen) {
+                $thisCharOrd = ord (substr ($thisLetter, $thisPos, 1));
+                if ($thisPos == 0) {
+                    $charNum = intval ($thisCharOrd - $decrement[$thisLen]);
+                    $decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
+                } else {
+                    $charNum = intval ($thisCharOrd - 128);
+                    $decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
+                }
+                $thisPos++;
+            }
+            $encodedLetter = "&#". preg_replace('/^0+/', '', $decimalCode) . ';';
+            $encodedString .= $encodedLetter;
+        }
+    }
+    return $encodedString;
 }
 
 /**
@@ -611,27 +611,27 @@ 
 **/
 function utf_32_to_unicode($source) {
 
-	// mb_string : methode rapide
-	if (init_mb_string()) {
-		$convmap = array(0x7F, 0xFFFFFF, 0x0, 0xFFFFFF);
-		$source = mb_encode_numericentity($source, $convmap, 'UTF-32LE');
-		return str_replace(chr(0), '', $source);
-	}
-
-	// Sinon methode lente
-	$texte = '';
-	while ($source) {
-		$words = unpack("V*", substr($source, 0, 1024));
-		$source = substr($source, 1024);
-		foreach ($words as $word) {
-			if ($word < 128)
-				$texte .= chr($word);
-			// ignorer le BOM - http://www.unicode.org/faq/utf_bom.html
-			else if ($word != 65279)
-				$texte .= '&#'.$word.';';
-		}
-	}
-	return $texte;
+    // mb_string : methode rapide
+    if (init_mb_string()) {
+        $convmap = array(0x7F, 0xFFFFFF, 0x0, 0xFFFFFF);
+        $source = mb_encode_numericentity($source, $convmap, 'UTF-32LE');
+        return str_replace(chr(0), '', $source);
+    }
+
+    // Sinon methode lente
+    $texte = '';
+    while ($source) {
+        $words = unpack("V*", substr($source, 0, 1024));
+        $source = substr($source, 1024);
+        foreach ($words as $word) {
+            if ($word < 128)
+                $texte .= chr($word);
+            // ignorer le BOM - http://www.unicode.org/faq/utf_bom.html
+            else if ($word != 65279)
+                $texte .= '&#'.$word.';';
+        }
+    }
+    return $texte;
 
 }
 
@@ -648,15 +648,15 @@ 
  *    Caractère utf8 si trouvé, '' sinon
 **/
 function caractere_utf_8($num) {
-	if($num<128)
-		return chr($num);
-	if($num<2048)
-		return chr(($num>>6)+192).chr(($num&63)+128);
-	if($num<65536)
-		return chr(($num>>12)+224).chr((($num>>6)&63)+128).chr(($num&63)+128);
-	if($num<1114112)
-		return chr($num>>18+240).chr((($num>>12)&63)+128).chr(($num>>6)&63+128). chr($num&63+128);
-	return '';
+    if($num<128)
+        return chr($num);
+    if($num<2048)
+        return chr(($num>>6)+192).chr(($num&63)+128);
+    if($num<65536)
+        return chr(($num>>12)+224).chr((($num>>6)&63)+128).chr(($num&63)+128);
+    if($num<1114112)
+        return chr($num>>18+240).chr((($num>>12)&63)+128).chr(($num>>6)&63+128). chr($num&63+128);
+    return '';
 }
 
 /**
@@ -669,25 +669,25 @@ 
 **/
 function unicode_to_utf_8($texte) {
 
-	// 1. Entites &#128; et suivantes
-	$vu = array();
-	if (preg_match_all(',&#0*([1-9][0-9][0-9]+);,S',
-	$texte, $regs, PREG_SET_ORDER))
-	foreach ($regs as $reg) {
-		if ($reg[1]>127 AND !isset($vu[$reg[0]]))
-			$vu[$reg[0]] = caractere_utf_8($reg[1]);
-	}
-	//$texte = str_replace(array_keys($vu), array_values($vu), $texte);
-
-	// 2. Entites > &#xFF;
-	//$vu = array();
-	if (preg_match_all(',&#x0*([1-9a-f][0-9a-f][0-9a-f]+);,iS',
-	$texte, $regs, PREG_SET_ORDER))
-	foreach ($regs as $reg) {
-		if (!isset($vu[$reg[0]]))
-			$vu[$reg[0]] = caractere_utf_8(hexdec($reg[1]));
-	}
-	return str_replace(array_keys($vu), array_values($vu), $texte);
+    // 1. Entites &#128; et suivantes
+    $vu = array();
+    if (preg_match_all(',&#0*([1-9][0-9][0-9]+);,S',
+    $texte, $regs, PREG_SET_ORDER))
+    foreach ($regs as $reg) {
+        if ($reg[1]>127 AND !isset($vu[$reg[0]]))
+            $vu[$reg[0]] = caractere_utf_8($reg[1]);
+    }
+    //$texte = str_replace(array_keys($vu), array_values($vu), $texte);
+
+    // 2. Entites > &#xFF;
+    //$vu = array();
+    if (preg_match_all(',&#x0*([1-9a-f][0-9a-f][0-9a-f]+);,iS',
+    $texte, $regs, PREG_SET_ORDER))
+    foreach ($regs as $reg) {
+        if (!isset($vu[$reg[0]]))
+            $vu[$reg[0]] = caractere_utf_8(hexdec($reg[1]));
+    }
+    return str_replace(array_keys($vu), array_values($vu), $texte);
 
 }
 
@@ -700,14 +700,14 @@ 
  *     Texte converti
 **/
 function unicode_to_javascript($texte) {
-	$vu = array();
-	while (preg_match(',&#0*([0-9]+);,S', $texte, $regs) AND !isset($vu[$regs[1]])) {
-		$num = $regs[1];
-		$vu[$num] = true;
-		$s = '\u'.sprintf("%04x", $num);
-		$texte = str_replace($regs[0], $s, $texte);
-	}
-	return $texte;
+    $vu = array();
+    while (preg_match(',&#0*([0-9]+);,S', $texte, $regs) AND !isset($vu[$regs[1]])) {
+        $num = $regs[1];
+        $vu[$num] = true;
+        $s = '\u'.sprintf("%04x", $num);
+        $texte = str_replace($regs[0], $s, $texte);
+    }
+    return $texte;
 }
 
 /**
@@ -719,9 +719,9 @@ 
  *     Texte converti
 **/
 function javascript_to_unicode ($texte) {
-	while (preg_match(",%u([0-9A-F][0-9A-F][0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],"&#".hexdec($regs[1]).";", $texte);
-	return $texte;
+    while (preg_match(",%u([0-9A-F][0-9A-F][0-9A-F][0-9A-F]),", $texte, $regs))
+        $texte = str_replace($regs[0],"&#".hexdec($regs[1]).";", $texte);
+    return $texte;
 }
 
 /**
@@ -733,31 +733,31 @@ 
  *     Texte converti
 **/
 function javascript_to_binary ($texte) {
-	while (preg_match(",%([0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],chr(hexdec($regs[1])), $texte);
-	return $texte;
+    while (preg_match(",%([0-9A-F][0-9A-F]),", $texte, $regs))
+        $texte = str_replace($regs[0],chr(hexdec($regs[1])), $texte);
+    return $texte;
 }
 
 
 // http://doc.spip.org/@translitteration_rapide
 function translitteration_rapide($texte, $charset='AUTO', $complexe='') {
-	static $trans;
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
-	if (!strlen($texte))
-		return $texte;
-
-	$table_translit ='translit'.$complexe;
-
-	// 2. Translitterer grace a la table predefinie
-	if (!$trans[$complexe]) {
-		global $CHARSET;
-		load_charset($table_translit);
-		foreach ($CHARSET[$table_translit] as $key => $val)
-			$trans[$complexe][caractere_utf_8($key)] = $val;
-	}
-
-	return str_replace(array_keys($trans[$complexe]),array_values($trans[$complexe]),$texte);
+    static $trans;
+    if ($charset == 'AUTO')
+        $charset = $GLOBALS['meta']['charset'];
+    if (!strlen($texte))
+        return $texte;
+
+    $table_translit ='translit'.$complexe;
+
+    // 2. Translitterer grace a la table predefinie
+    if (!$trans[$complexe]) {
+        global $CHARSET;
+        load_charset($table_translit);
+        foreach ($CHARSET[$table_translit] as $key => $val)
+            $trans[$complexe][caractere_utf_8($key)] = $val;
+    }
+
+    return str_replace(array_keys($trans[$complexe]),array_values($trans[$complexe]),$texte);
 }
 
 //
@@ -766,32 +766,32 @@ 
 //
 // http://doc.spip.org/@translitteration
 function translitteration($texte, $charset='AUTO', $complexe='') {
-	// 0. Supprimer les caracteres illegaux
-	include_spip('inc/filtres');
-	$texte = corriger_caracteres($texte);
+    // 0. Supprimer les caracteres illegaux
+    include_spip('inc/filtres');
+    $texte = corriger_caracteres($texte);
 
-	// 1. Passer le charset et les &eacute en utf-8
-	$texte = unicode_to_utf_8(html2unicode(charset2unicode($texte, $charset, true)));
+    // 1. Passer le charset et les &eacute en utf-8
+    $texte = unicode_to_utf_8(html2unicode(charset2unicode($texte, $charset, true)));
 
-	return translitteration_rapide($texte,$charset,$complexe);
+    return translitteration_rapide($texte,$charset,$complexe);
 }
 
 // &agrave; est retourne sous la forme "a`" et pas "a"
 // mais si $chiffre=true, on retourne "a8" (vietnamien)
 // http://doc.spip.org/@translitteration_complexe
 function translitteration_complexe($texte, $chiffres=false) {
-	$texte = translitteration($texte,'AUTO','complexe');
+    $texte = translitteration($texte,'AUTO','complexe');
 
-	if ($chiffres) {
-		$texte = preg_replace("/[aeiuoyd]['`?~.^+(-]{1,2}/eS",
-			"translitteration_chiffree('\\0')", $texte);
-	}
+    if ($chiffres) {
+        $texte = preg_replace("/[aeiuoyd]['`?~.^+(-]{1,2}/eS",
+            "translitteration_chiffree('\\0')", $texte);
+    }
 	
-	return $texte;
+    return $texte;
 }
 // http://doc.spip.org/@translitteration_chiffree
 function translitteration_chiffree($car) {
-	return strtr($car, "'`?~.^+(-", "123456789");
+    return strtr($car, "'`?~.^+(-", "123456789");
 }
 
 
@@ -804,7 +804,7 @@ 
  *    true s'il a un BOM
 **/
 function bom_utf8($texte) {
-	return (substr($texte, 0,3) == chr(0xEF).chr(0xBB).chr(0xBF));
+    return (substr($texte, 0,3) == chr(0xEF).chr(0xBB).chr(0xBF));
 }
 
 /**
@@ -821,18 +821,18 @@ 
  *     true si c'est le cas
 **/
 function is_utf8($string) {
-	return !strlen(
-	preg_replace(
-	  ',[\x09\x0A\x0D\x20-\x7E]'            # ASCII
-	. '|[\xC2-\xDF][\x80-\xBF]'             # non-overlong 2-byte
-	. '|\xE0[\xA0-\xBF][\x80-\xBF]'         # excluding overlongs
-	. '|[\xE1-\xEC\xEE\xEF][\x80-\xBF]{2}'  # straight 3-byte
-	. '|\xED[\x80-\x9F][\x80-\xBF]'         # excluding surrogates
-	. '|\xF0[\x90-\xBF][\x80-\xBF]{2}'      # planes 1-3
-	. '|[\xF1-\xF3][\x80-\xBF]{3}'          # planes 4-15
-	. '|\xF4[\x80-\x8F][\x80-\xBF]{2}'      # plane 16
-	. ',sS',
-	'', $string));
+    return !strlen(
+    preg_replace(
+        ',[\x09\x0A\x0D\x20-\x7E]'            # ASCII
+    . '|[\xC2-\xDF][\x80-\xBF]'             # non-overlong 2-byte
+    . '|\xE0[\xA0-\xBF][\x80-\xBF]'         # excluding overlongs
+    . '|[\xE1-\xEC\xEE\xEF][\x80-\xBF]{2}'  # straight 3-byte
+    . '|\xED[\x80-\x9F][\x80-\xBF]'         # excluding surrogates
+    . '|\xF0[\x90-\xBF][\x80-\xBF]{2}'      # planes 1-3
+    . '|[\xF1-\xF3][\x80-\xBF]{3}'          # planes 4-15
+    . '|\xF4[\x80-\x8F][\x80-\xBF]{2}'      # plane 16
+    . ',sS',
+    '', $string));
 }
 
 /**
@@ -844,10 +844,10 @@ 
  *     true si c'est le cas
 **/
 function is_ascii($string) {
-	return !strlen(
-	preg_replace(
-	',[\x09\x0A\x0D\x20-\x7E],sS',
-	'', $string));
+    return !strlen(
+    preg_replace(
+    ',[\x09\x0A\x0D\x20-\x7E],sS',
+    '', $string));
 }
 
 // Transcode une page (attrapee sur le web, ou un squelette) en essayant
@@ -855,49 +855,49 @@ 
 // http://doc.spip.org/@transcoder_page
 function transcoder_page($texte, $headers='') {
 
-	// Si tout est < 128 pas la peine d'aller plus loin
-	if (is_ascii($texte)) {
-		#spip_log('charset: ascii');
-		return $texte;
-	}
-
-	// Reconnaitre le BOM utf-8 (0xEFBBBF)
-	if (bom_utf8($texte)) {
-		$charset = 'utf-8';
-		$texte = substr($texte,3);
-	}
-
-	// charset precise par le contenu (xml)
-	else if (preg_match(
-	',<[?]xml[^>]*encoding[^>]*=[^>]*([-_a-z0-9]+?),UimsS', $texte, $regs))
-		$charset = trim(strtolower($regs[1]));
-	// charset precise par le contenu (html)
-	else if (preg_match(
-	',<(meta|html|body)[^>]*charset[^>]*=[^>]*([-_a-z0-9]+?),UimsS',
-	$texte, $regs)
-	# eviter #CHARSET des squelettes
-	AND (($tmp = trim(strtolower($regs[2]))) != 'charset'))
-		$charset = $tmp;
-	// charset de la reponse http
-	else if (preg_match(',charset=([-_a-z0-9]+),i', $headers, $regs))
-		$charset = trim(strtolower($regs[1]));
-	else $charset = '';
-	// normaliser les noms du shif-jis japonais
-	if (preg_match(',^(x|shift)[_-]s?jis$,i', $charset))
-		$charset = 'shift-jis';
-
-	if ($charset) {
-		spip_log("charset: $charset");
-	} else {
-		// valeur par defaut
-		if (is_utf8($texte))
-			$charset = 'utf-8';
-		else
-			$charset = 'iso-8859-1';
-		spip_log("charset probable: $charset");
-	}
-
-	return importer_charset($texte, $charset);
+    // Si tout est < 128 pas la peine d'aller plus loin
+    if (is_ascii($texte)) {
+        #spip_log('charset: ascii');
+        return $texte;
+    }
+
+    // Reconnaitre le BOM utf-8 (0xEFBBBF)
+    if (bom_utf8($texte)) {
+        $charset = 'utf-8';
+        $texte = substr($texte,3);
+    }
+
+    // charset precise par le contenu (xml)
+    else if (preg_match(
+    ',<[?]xml[^>]*encoding[^>]*=[^>]*([-_a-z0-9]+?),UimsS', $texte, $regs))
+        $charset = trim(strtolower($regs[1]));
+    // charset precise par le contenu (html)
+    else if (preg_match(
+    ',<(meta|html|body)[^>]*charset[^>]*=[^>]*([-_a-z0-9]+?),UimsS',
+    $texte, $regs)
+    # eviter #CHARSET des squelettes
+    AND (($tmp = trim(strtolower($regs[2]))) != 'charset'))
+        $charset = $tmp;
+    // charset de la reponse http
+    else if (preg_match(',charset=([-_a-z0-9]+),i', $headers, $regs))
+        $charset = trim(strtolower($regs[1]));
+    else $charset = '';
+    // normaliser les noms du shif-jis japonais
+    if (preg_match(',^(x|shift)[_-]s?jis$,i', $charset))
+        $charset = 'shift-jis';
+
+    if ($charset) {
+        spip_log("charset: $charset");
+    } else {
+        // valeur par defaut
+        if (is_utf8($texte))
+            $charset = 'utf-8';
+        else
+            $charset = 'iso-8859-1';
+        spip_log("charset probable: $charset");
+    }
+
+    return importer_charset($texte, $charset);
 }
 
 
@@ -920,24 +920,24 @@ 
  *     Le texte coupé
 **/
 function spip_substr($c, $start=0, $length = NULL) {
-	// Si ce n'est pas utf-8, utiliser substr
-	if ($GLOBALS['meta']['charset'] != 'utf-8') {
-		if ($length)
-			return substr($c, $start, $length);
-		else
-			substr($c, $start);
-	}
-
-	// Si utf-8, voir si on dispose de mb_string
-	if (init_mb_string()) {
-		if ($length)
-			return mb_substr($c, $start, $length);
-		else
-			return mb_substr($c, $start);
-	}
-
-	// Version manuelle (cf. ci-dessous)
-	return spip_substr_manuelle($c, $start, $length);
+    // Si ce n'est pas utf-8, utiliser substr
+    if ($GLOBALS['meta']['charset'] != 'utf-8') {
+        if ($length)
+            return substr($c, $start, $length);
+        else
+            substr($c, $start);
+    }
+
+    // Si utf-8, voir si on dispose de mb_string
+    if (init_mb_string()) {
+        if ($length)
+            return mb_substr($c, $start, $length);
+        else
+            return mb_substr($c, $start);
+    }
+
+    // Version manuelle (cf. ci-dessous)
+    return spip_substr_manuelle($c, $start, $length);
 }
 
 
@@ -957,32 +957,32 @@ 
 **/
 function spip_substr_manuelle($c, $start, $length = NULL) {
 
-	// Cas pathologique
-	if ($length === 0)
-		return '';
-
-	// S'il y a un demarrage, on se positionne
-	if ($start > 0)
-		$c = substr($c, strlen(spip_substr_manuelle($c, 0, $start)));
-	elseif ($start < 0)
-		return spip_substr_manuelle($c, spip_strlen($c)+$start, $length);
-
-	if (!$length)
-		return $c;
-
-	if ($length > 0) {
-		// on prend n fois la longueur desiree, pour etre surs d'avoir tout
-		// (un caractere utf-8 prenant au maximum n bytes)
-		$n = 0; while (preg_match(',[\x80-\xBF]{'.(++$n).'},', $c));
-		$c = substr($c, 0, $n*$length);
-		// puis, tant qu'on est trop long, on coupe...
-		while (($l = spip_strlen($c)) > $length)
-			$c = substr($c, 0, $length - $l);
-		return $c;
-	}
-
-	// $length < 0
-	return spip_substr_manuelle($c, 0, spip_strlen($c)+$length);
+    // Cas pathologique
+    if ($length === 0)
+        return '';
+
+    // S'il y a un demarrage, on se positionne
+    if ($start > 0)
+        $c = substr($c, strlen(spip_substr_manuelle($c, 0, $start)));
+    elseif ($start < 0)
+        return spip_substr_manuelle($c, spip_strlen($c)+$start, $length);
+
+    if (!$length)
+        return $c;
+
+    if ($length > 0) {
+        // on prend n fois la longueur desiree, pour etre surs d'avoir tout
+        // (un caractere utf-8 prenant au maximum n bytes)
+        $n = 0; while (preg_match(',[\x80-\xBF]{'.(++$n).'},', $c));
+        $c = substr($c, 0, $n*$length);
+        // puis, tant qu'on est trop long, on coupe...
+        while (($l = spip_strlen($c)) > $length)
+            $c = substr($c, 0, $length - $l);
+        return $c;
+    }
+
+    // $length < 0
+    return spip_substr_manuelle($c, 0, spip_strlen($c)+$length);
 }
 
 /**
@@ -996,15 +996,15 @@ 
  *     La chaîne avec une majuscule sur le premier mot 
  */
 function spip_ucfirst($c){
-	// Si ce n'est pas utf-8, utiliser ucfirst 
-	if ($GLOBALS['meta']['charset'] != 'utf-8')
-		return ucfirst($c);
-	// Si on n'a pas mb_* on utilise ucfirst
-	if (!init_mb_string())
-		return ucfirst($c);
+    // Si ce n'est pas utf-8, utiliser ucfirst 
+    if ($GLOBALS['meta']['charset'] != 'utf-8')
+        return ucfirst($c);
+    // Si on n'a pas mb_* on utilise ucfirst
+    if (!init_mb_string())
+        return ucfirst($c);
 	
-	$lettre1 = mb_strtoupper(spip_substr($c,0,1));
-	return $lettre1.spip_substr($c,1);
+    $lettre1 = mb_strtoupper(spip_substr($c,0,1));
+    return $lettre1.spip_substr($c,1);
 }
 
 /**
@@ -1018,21 +1018,21 @@ 
  *     Longueur de la chaîne
  */
 function spip_strlen($c) {
-	// On transforme les sauts de ligne pour ne pas compter deux caractères
-	$c = str_replace("\r\n", "\n", $c);
+    // On transforme les sauts de ligne pour ne pas compter deux caractères
+    $c = str_replace("\r\n", "\n", $c);
 	
-	// Si ce n'est pas utf-8, utiliser strlen
-	if ($GLOBALS['meta']['charset'] != 'utf-8')
-		return strlen($c);
-
-	// Sinon, utiliser mb_strlen() si disponible
-	if (init_mb_string())
-		return mb_strlen($c);
-
-	// Methode manuelle : on supprime les bytes 10......,
-	// on compte donc les ascii (0.......) et les demarrages
-	// de caracteres utf-8 (11......)
-	return strlen(preg_replace(',[\x80-\xBF],S', '', $c));
+    // Si ce n'est pas utf-8, utiliser strlen
+    if ($GLOBALS['meta']['charset'] != 'utf-8')
+        return strlen($c);
+
+    // Sinon, utiliser mb_strlen() si disponible
+    if (init_mb_string())
+        return mb_strlen($c);
+
+    // Methode manuelle : on supprime les bytes 10......,
+    // on compte donc les ascii (0.......) et les demarrages
+    // de caracteres utf-8 (11......)
+    return strlen(preg_replace(',[\x80-\xBF],S', '', $c));
 }
 
 // Initialisation
@@ -1043,12 +1043,12 @@ 
 // en utf-8 chr(195).chr(160) = a` alors qu'en iso-latin chr(160) = nbsp
 if (!isset($GLOBALS['meta']['pcre_u'])
   OR (isset($_GET['var_mode']) AND !isset($_GET['var_profile']))) {
-	include_spip('inc/meta');
-	ecrire_meta('pcre_u',
-		$u = ($GLOBALS['meta']['charset'] == 'utf-8'
-		AND test_pcre_unicode())
-			? 'u' :''
-	);
+    include_spip('inc/meta');
+    ecrire_meta('pcre_u',
+        $u = ($GLOBALS['meta']['charset'] == 'utf-8'
+        AND test_pcre_unicode())
+            ? 'u' :''
+    );
 }
 
 ?>

Switch Indentation +71 added lines, -71 removed lines

@@ -354,48 +354,48 @@ 
 	$charset = strtolower($charset);
 
 	switch ($charset) {
-	case 'utf-8':
-	case 'utf8':
-		return utf_8_to_unicode($texte);
-
-	case 'iso-8859-1':
-		$texte = corriger_caracteres_windows($texte, 'iso-8859-1');
-		// pas de break; ici, on suit sur default:
-
-	default:
-		// mbstring presente ?
-		if (init_mb_string()) {
-			if ($order = mb_detect_order() # mb_string connait-il $charset?
-			AND mb_detect_order($charset)) {
-				$s = mb_convert_encoding($texte, 'utf-8', $charset);
-				if ($s && $s != $texte) return utf_8_to_unicode($s);
-			}
-			mb_detect_order($order); # remettre comme precedemment
-		}
-
-		// Sinon, peut-etre connaissons-nous ce charset ?
-		if (!isset($trans[$charset])) {
-			global $CHARSET;
-			if ($cset = load_charset($charset)
-			AND is_array($CHARSET[$cset]))
-				foreach ($CHARSET[$cset] as $key => $val) {
-					$trans[$charset][chr($key)] = '&#'.$val.';';
-			}
-		}
-		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
-
-		// Sinon demander a iconv (malgre le fait qu'il coupe quand un
-		// caractere n'appartient pas au charset, mais c'est un probleme
-		// surtout en utf-8, gere ci-dessus)
-		if (test_iconv()) {
-			$s = iconv($charset, 'utf-32le', $texte);
-			if ($s) return utf_32_to_unicode($s);
-		}
-
-		// Au pire ne rien faire
-		spip_log("erreur charset '$charset' non supporte");
-		return $texte;
+	    case 'utf-8':
+	    case 'utf8':
+		    return utf_8_to_unicode($texte);
+
+	    case 'iso-8859-1':
+		    $texte = corriger_caracteres_windows($texte, 'iso-8859-1');
+		    // pas de break; ici, on suit sur default:
+
+	    default:
+		    // mbstring presente ?
+		    if (init_mb_string()) {
+			    if ($order = mb_detect_order() # mb_string connait-il $charset?
+			    AND mb_detect_order($charset)) {
+				    $s = mb_convert_encoding($texte, 'utf-8', $charset);
+				    if ($s && $s != $texte) return utf_8_to_unicode($s);
+			    }
+			    mb_detect_order($order); # remettre comme precedemment
+		    }
+
+		    // Sinon, peut-etre connaissons-nous ce charset ?
+		    if (!isset($trans[$charset])) {
+			    global $CHARSET;
+			    if ($cset = load_charset($charset)
+			    AND is_array($CHARSET[$cset]))
+				    foreach ($CHARSET[$cset] as $key => $val) {
+					    $trans[$charset][chr($key)] = '&#'.$val.';';
+			    }
+		    }
+		    if (count($trans[$charset]))
+			    return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+
+		    // Sinon demander a iconv (malgre le fait qu'il coupe quand un
+		    // caractere n'appartient pas au charset, mais c'est un probleme
+		    // surtout en utf-8, gere ci-dessus)
+		    if (test_iconv()) {
+			    $s = iconv($charset, 'utf-32le', $texte);
+			    if ($s) return utf_32_to_unicode($s);
+		    }
+
+		    // Au pire ne rien faire
+		    spip_log("erreur charset '$charset' non supporte");
+		    return $texte;
 	}
 }
 
@@ -422,35 +422,35 @@ 
 		$charset = $GLOBALS['meta']['charset'];
 
 	switch($charset) {
-	case 'utf-8':
-		return unicode_to_utf_8($texte);
-		break;
-
-	default:
-		$charset = load_charset($charset);
-
-		if (!is_array($CHARSET_REVERSE[$charset])) {
-			$CHARSET_REVERSE[$charset] = array_flip($GLOBALS['CHARSET'][$charset]);
-		}
-
-		if (!isset($trans[$charset])){
-			$trans[$charset]=array();
-			$t = &$trans[$charset];
-			for($e=128;$e<255;$e++){
-				$h = dechex($e);
-				if ($s = isset($CHARSET_REVERSE[$charset][$e])){
-					$s = $CHARSET_REVERSE[$charset][$e];
-					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
-					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
-				}
-				else{
-					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
-					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
-				}
-			}
-		}
-		$texte = str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
-		return $texte;
+	    case 'utf-8':
+		    return unicode_to_utf_8($texte);
+		    break;
+
+	    default:
+		    $charset = load_charset($charset);
+
+		    if (!is_array($CHARSET_REVERSE[$charset])) {
+			    $CHARSET_REVERSE[$charset] = array_flip($GLOBALS['CHARSET'][$charset]);
+		    }
+
+		    if (!isset($trans[$charset])){
+			    $trans[$charset]=array();
+			    $t = &$trans[$charset];
+			    for($e=128;$e<255;$e++){
+				    $h = dechex($e);
+				    if ($s = isset($CHARSET_REVERSE[$charset][$e])){
+					    $s = $CHARSET_REVERSE[$charset][$e];
+					    $t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
+					    $t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
+				    }
+				    else{
+					    $t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
+					    $t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
+				    }
+			    }
+		    }
+		    $texte = str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+		    return $texte;
 	}
 }
 

Spacing +68 added lines, -68 removed lines

@@ -37,7 +37,7 @@ 
  *     Nom du charset
  *     false si le charset n'est pas décrit dans le répertoire charsets/
 **/
-function load_charset ($charset = 'AUTO') {
+function load_charset($charset = 'AUTO') {
 	if ($charset == 'AUTO')
 		$charset = $GLOBALS['meta']['charset'];
 	$charset = trim(strtolower($charset));
@@ -55,7 +55,7 @@ 
 	else if ($charset == 'windows-1251') $charset = 'cp1251';
 	else if ($charset == 'windows-1256') $charset = 'cp1256';
 
-	if (find_in_path($charset . '.php', 'charsets/', true)) {
+	if (find_in_path($charset.'.php', 'charsets/', true)) {
 		return $charset;
 	} else {
 		spip_log("Erreur: pas de fichier de conversion 'charsets/$charset'");
@@ -207,17 +207,17 @@ 
  * @return string
  *     Texte corrigé
 **/
-function corriger_caracteres_windows($texte, $charset='AUTO', $charset_cible='unicode') {
+function corriger_caracteres_windows($texte, $charset = 'AUTO', $charset_cible = 'unicode') {
 	static $trans;
 	
 	if (is_array($texte)) {
 		return array_map('corriger_caracteres_windows', $texte);
 	}
 	
-	if ($charset=='AUTO') $charset = $GLOBALS['meta']['charset'];
+	if ($charset == 'AUTO') $charset = $GLOBALS['meta']['charset'];
 	if ($charset == 'utf-8') {
 		$p = chr(194);
-		if (strpos($texte,$p)==false)
+		if (strpos($texte, $p) == false)
 			return $texte;
 	} else if ($charset == 'iso-8859-1') {
 		$p = '';
@@ -259,14 +259,14 @@ 
 			$p.chr(158) => "ž",
 			$p.chr(159) => "Ÿ",
 		);
-		if ($charset_cible!='unicode'){
-			foreach($trans[$charset][$charset_cible] as $k=>$c)
+		if ($charset_cible != 'unicode') {
+			foreach ($trans[$charset][$charset_cible] as $k=>$c)
 				$trans[$charset][$charset_cible][$k] = unicode2charset($c, $charset_cible);
 		}
 	}
 
 	return @str_replace(array_keys($trans[$charset][$charset_cible]),
-			   array_values($trans[$charset][$charset_cible]),$texte);
+			   array_values($trans[$charset][$charset_cible]), $texte);
 }
 
 
@@ -283,8 +283,8 @@ 
  * @return string
  *     Texte converti
 **/
-function html2unicode($texte, $secure=false) {
-	if (strpos($texte,'&') === false) return $texte;
+function html2unicode($texte, $secure = false) {
+	if (strpos($texte, '&') === false) return $texte;
 	static $trans = array();
 	if (!$trans) {
 		global $CHARSET;
@@ -295,10 +295,10 @@ 
 	}
 
 	if ($secure)
-		return str_replace(array_keys($trans),array_values($trans),$texte);
+		return str_replace(array_keys($trans), array_values($trans), $texte);
 	else
-		return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'),array('&', '"', '<', '>'),
-		  str_replace(array_keys($trans),array_values($trans),$texte)			
+		return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'), array('&', '"', '<', '>'),
+		  str_replace(array_keys($trans), array_values($trans), $texte)			
 		);
 }
 
@@ -323,7 +323,7 @@ 
 			$trans["&$key;"] = $val;
 	}
 
-	return str_replace(array_keys($trans),array_values($trans),$texte);
+	return str_replace(array_keys($trans), array_values($trans), $texte);
 }
 
 
@@ -344,7 +344,7 @@ 
  * @return string
  *     Texte converti en unicode
 **/
-function charset2unicode($texte, $charset='AUTO' /* $forcer: obsolete*/) {
+function charset2unicode($texte, $charset = 'AUTO' /* $forcer: obsolete*/) {
 	static $trans;
 
 	if ($charset == 'AUTO')
@@ -383,7 +383,7 @@ 
 			}
 		}
 		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+			return str_replace(array_keys($trans[$charset]), array_values($trans[$charset]), $texte);
 
 		// Sinon demander a iconv (malgre le fait qu'il coupe quand un
 		// caractere n'appartient pas au charset, mais c'est un probleme
@@ -414,14 +414,14 @@ 
  * @return string
  *     Texte transformé dans le charset souhaité
 **/
-function unicode2charset($texte, $charset='AUTO') {
+function unicode2charset($texte, $charset = 'AUTO') {
 	static $CHARSET_REVERSE;
 	static $trans = array();
 	
 	if ($charset == 'AUTO')
 		$charset = $GLOBALS['meta']['charset'];
 
-	switch($charset) {
+	switch ($charset) {
 	case 'utf-8':
 		return unicode_to_utf_8($texte);
 		break;
@@ -433,23 +433,23 @@ 
 			$CHARSET_REVERSE[$charset] = array_flip($GLOBALS['CHARSET'][$charset]);
 		}
 
-		if (!isset($trans[$charset])){
-			$trans[$charset]=array();
+		if (!isset($trans[$charset])) {
+			$trans[$charset] = array();
 			$t = &$trans[$charset];
-			for($e=128;$e<255;$e++){
+			for ($e = 128; $e < 255; $e++) {
 				$h = dechex($e);
-				if ($s = isset($CHARSET_REVERSE[$charset][$e])){
+				if ($s = isset($CHARSET_REVERSE[$charset][$e])) {
 					$s = $CHARSET_REVERSE[$charset][$e];
 					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
 					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
 				}
-				else{
+				else {
 					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
 					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
 				}
 			}
 		}
-		$texte = str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+		$texte = str_replace(array_keys($trans[$charset]), array_values($trans[$charset]), $texte);
 		return $texte;
 	}
 }
@@ -471,8 +471,8 @@ 
 function importer_charset($texte, $charset = 'AUTO') {
 	static $trans = array();
 	// on traite le cas le plus frequent iso-8859-1 vers utf directement pour aller plus vite !
-	if (($charset == 'iso-8859-1') && ($GLOBALS['meta']['charset']=='utf-8')){
-		$texte = corriger_caracteres_windows($texte, 'iso-8859-1',$GLOBALS['meta']['charset']);
+	if (($charset == 'iso-8859-1') && ($GLOBALS['meta']['charset'] == 'utf-8')) {
+		$texte = corriger_caracteres_windows($texte, 'iso-8859-1', $GLOBALS['meta']['charset']);
 		if (init_mb_string()) {
 			if ($order = mb_detect_order() # mb_string connait-il $charset?
 			AND mb_detect_order($charset)) {
@@ -491,7 +491,7 @@ 
 			}
 		}
 		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
+			return str_replace(array_keys($trans[$charset]), array_values($trans[$charset]), $texte);
 		return $texte;
 	}
 	return unicode2charset(charset2unicode($texte, $charset));
@@ -542,30 +542,30 @@ 
 	}
 
 	$pos = 0;
-	$len = strlen ($source);
+	$len = strlen($source);
 	$encodedString = '';
 	while ($pos < $len) {
 		$char = '';
 		$ischar = false;
-		$asciiPos = ord (substr ($source, $pos, 1));
+		$asciiPos = ord(substr($source, $pos, 1));
 		if (($asciiPos >= 240) && ($asciiPos <= 255)) {
 			// 4 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 4);
+			$thisLetter = substr($source, $pos, 4);
 			$pos += 4;
 		}
 		else if (($asciiPos >= 224) && ($asciiPos <= 239)) {
 			// 3 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 3);
+			$thisLetter = substr($source, $pos, 3);
 			$pos += 3;
 		}
 		else if (($asciiPos >= 192) && ($asciiPos <= 223)) {
 			// 2 chars representing one unicode character
-			$thisLetter = substr ($source, $pos, 2);
+			$thisLetter = substr($source, $pos, 2);
 			$pos += 2;
 		}
 		else {
 			// 1 char (lower ascii)
-			$thisLetter = substr ($source, $pos, 1);
+			$thisLetter = substr($source, $pos, 1);
 			$pos += 1;
 			$char = $thisLetter;
 			$ischar = true;
@@ -574,21 +574,21 @@ 
 		if ($ischar)
 			$encodedString .= $char;
 		else {	// process the string representing the letter to a unicode entity
-			$thisLen = strlen ($thisLetter);
+			$thisLen = strlen($thisLetter);
 			$thisPos = 0;
 			$decimalCode = 0;
 			while ($thisPos < $thisLen) {
-				$thisCharOrd = ord (substr ($thisLetter, $thisPos, 1));
+				$thisCharOrd = ord(substr($thisLetter, $thisPos, 1));
 				if ($thisPos == 0) {
-					$charNum = intval ($thisCharOrd - $decrement[$thisLen]);
+					$charNum = intval($thisCharOrd - $decrement[$thisLen]);
 					$decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
 				} else {
-					$charNum = intval ($thisCharOrd - 128);
+					$charNum = intval($thisCharOrd - 128);
 					$decimalCode += ($charNum << $shift[$thisLen][$thisPos]);
 				}
 				$thisPos++;
 			}
-			$encodedLetter = "&#". preg_replace('/^0+/', '', $decimalCode) . ';';
+			$encodedLetter = "&#".preg_replace('/^0+/', '', $decimalCode).';';
 			$encodedString .= $encodedLetter;
 		}
 	}
@@ -648,14 +648,14 @@ 
  *    Caractère utf8 si trouvé, '' sinon
 **/
 function caractere_utf_8($num) {
-	if($num<128)
+	if ($num < 128)
 		return chr($num);
-	if($num<2048)
-		return chr(($num>>6)+192).chr(($num&63)+128);
-	if($num<65536)
-		return chr(($num>>12)+224).chr((($num>>6)&63)+128).chr(($num&63)+128);
-	if($num<1114112)
-		return chr($num>>18+240).chr((($num>>12)&63)+128).chr(($num>>6)&63+128). chr($num&63+128);
+	if ($num < 2048)
+		return chr(($num >> 6) + 192).chr(($num & 63) + 128);
+	if ($num < 65536)
+		return chr(($num >> 12) + 224).chr((($num >> 6) & 63) + 128).chr(($num & 63) + 128);
+	if ($num < 1114112)
+		return chr($num >> 18 + 240).chr((($num >> 12) & 63) + 128).chr(($num >> 6) & 63 + 128).chr($num & 63 + 128);
 	return '';
 }
 
@@ -674,7 +674,7 @@ 
 	if (preg_match_all(',&#0*([1-9][0-9][0-9]+);,S',
 	$texte, $regs, PREG_SET_ORDER))
 	foreach ($regs as $reg) {
-		if ($reg[1]>127 AND !isset($vu[$reg[0]]))
+		if ($reg[1] > 127 AND !isset($vu[$reg[0]]))
 			$vu[$reg[0]] = caractere_utf_8($reg[1]);
 	}
 	//$texte = str_replace(array_keys($vu), array_values($vu), $texte);
@@ -718,9 +718,9 @@ 
  * @return string
  *     Texte converti
 **/
-function javascript_to_unicode ($texte) {
+function javascript_to_unicode($texte) {
 	while (preg_match(",%u([0-9A-F][0-9A-F][0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],"&#".hexdec($regs[1]).";", $texte);
+		$texte = str_replace($regs[0], "&#".hexdec($regs[1]).";", $texte);
 	return $texte;
 }
 
@@ -732,22 +732,22 @@ 
  * @return string
  *     Texte converti
 **/
-function javascript_to_binary ($texte) {
+function javascript_to_binary($texte) {
 	while (preg_match(",%([0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],chr(hexdec($regs[1])), $texte);
+		$texte = str_replace($regs[0], chr(hexdec($regs[1])), $texte);
 	return $texte;
 }
 
 
 // http://doc.spip.org/@translitteration_rapide
-function translitteration_rapide($texte, $charset='AUTO', $complexe='') {
+function translitteration_rapide($texte, $charset = 'AUTO', $complexe = '') {
 	static $trans;
 	if ($charset == 'AUTO')
 		$charset = $GLOBALS['meta']['charset'];
 	if (!strlen($texte))
 		return $texte;
 
-	$table_translit ='translit'.$complexe;
+	$table_translit = 'translit'.$complexe;
 
 	// 2. Translitterer grace a la table predefinie
 	if (!$trans[$complexe]) {
@@ -757,7 +757,7 @@ 
 			$trans[$complexe][caractere_utf_8($key)] = $val;
 	}
 
-	return str_replace(array_keys($trans[$complexe]),array_values($trans[$complexe]),$texte);
+	return str_replace(array_keys($trans[$complexe]), array_values($trans[$complexe]), $texte);
 }
 
 //
@@ -765,7 +765,7 @@ 
 // Attention les caracteres non reconnus sont renvoyes en utf-8
 //
 // http://doc.spip.org/@translitteration
-function translitteration($texte, $charset='AUTO', $complexe='') {
+function translitteration($texte, $charset = 'AUTO', $complexe = '') {
 	// 0. Supprimer les caracteres illegaux
 	include_spip('inc/filtres');
 	$texte = corriger_caracteres($texte);
@@ -773,14 +773,14 @@ 
 	// 1. Passer le charset et les &eacute en utf-8
 	$texte = unicode_to_utf_8(html2unicode(charset2unicode($texte, $charset, true)));
 
-	return translitteration_rapide($texte,$charset,$complexe);
+	return translitteration_rapide($texte, $charset, $complexe);
 }
 
 // &agrave; est retourne sous la forme "a`" et pas "a"
 // mais si $chiffre=true, on retourne "a8" (vietnamien)
 // http://doc.spip.org/@translitteration_complexe
-function translitteration_complexe($texte, $chiffres=false) {
-	$texte = translitteration($texte,'AUTO','complexe');
+function translitteration_complexe($texte, $chiffres = false) {
+	$texte = translitteration($texte, 'AUTO', 'complexe');
 
 	if ($chiffres) {
 		$texte = preg_replace("/[aeiuoyd]['`?~.^+(-]{1,2}/eS",
@@ -804,7 +804,7 @@ 
  *    true s'il a un BOM
 **/
 function bom_utf8($texte) {
-	return (substr($texte, 0,3) == chr(0xEF).chr(0xBB).chr(0xBF));
+	return (substr($texte, 0, 3) == chr(0xEF).chr(0xBB).chr(0xBF));
 }
 
 /**
@@ -853,7 +853,7 @@ 
 // Transcode une page (attrapee sur le web, ou un squelette) en essayant
 // par tous les moyens de deviner son charset (y compris headers HTTP)
 // http://doc.spip.org/@transcoder_page
-function transcoder_page($texte, $headers='') {
+function transcoder_page($texte, $headers = '') {
 
 	// Si tout est < 128 pas la peine d'aller plus loin
 	if (is_ascii($texte)) {
@@ -864,7 +864,7 @@ 
 	// Reconnaitre le BOM utf-8 (0xEFBBBF)
 	if (bom_utf8($texte)) {
 		$charset = 'utf-8';
-		$texte = substr($texte,3);
+		$texte = substr($texte, 3);
 	}
 
 	// charset precise par le contenu (xml)
@@ -919,7 +919,7 @@ 
  * @return string
  *     Le texte coupé
 **/
-function spip_substr($c, $start=0, $length = NULL) {
+function spip_substr($c, $start = 0, $length = NULL) {
 	// Si ce n'est pas utf-8, utiliser substr
 	if ($GLOBALS['meta']['charset'] != 'utf-8') {
 		if ($length)
@@ -965,7 +965,7 @@ 
 	if ($start > 0)
 		$c = substr($c, strlen(spip_substr_manuelle($c, 0, $start)));
 	elseif ($start < 0)
-		return spip_substr_manuelle($c, spip_strlen($c)+$start, $length);
+		return spip_substr_manuelle($c, spip_strlen($c) + $start, $length);
 
 	if (!$length)
 		return $c;
@@ -974,7 +974,7 @@ 
 		// on prend n fois la longueur desiree, pour etre surs d'avoir tout
 		// (un caractere utf-8 prenant au maximum n bytes)
 		$n = 0; while (preg_match(',[\x80-\xBF]{'.(++$n).'},', $c));
-		$c = substr($c, 0, $n*$length);
+		$c = substr($c, 0, $n * $length);
 		// puis, tant qu'on est trop long, on coupe...
 		while (($l = spip_strlen($c)) > $length)
 			$c = substr($c, 0, $length - $l);
@@ -982,7 +982,7 @@ 
 	}
 
 	// $length < 0
-	return spip_substr_manuelle($c, 0, spip_strlen($c)+$length);
+	return spip_substr_manuelle($c, 0, spip_strlen($c) + $length);
 }
 
 /**
@@ -995,7 +995,7 @@ 
  * @return string
  *     La chaîne avec une majuscule sur le premier mot 
  */
-function spip_ucfirst($c){
+function spip_ucfirst($c) {
 	// Si ce n'est pas utf-8, utiliser ucfirst 
 	if ($GLOBALS['meta']['charset'] != 'utf-8')
 		return ucfirst($c);
@@ -1003,8 +1003,8 @@ 
 	if (!init_mb_string())
 		return ucfirst($c);
 	
-	$lettre1 = mb_strtoupper(spip_substr($c,0,1));
-	return $lettre1.spip_substr($c,1);
+	$lettre1 = mb_strtoupper(spip_substr($c, 0, 1));
+	return $lettre1.spip_substr($c, 1);
 }
 
 /**
@@ -1047,7 +1047,7 @@ 
 	ecrire_meta('pcre_u',
 		$u = ($GLOBALS['meta']['charset'] == 'utf-8'
 		AND test_pcre_unicode())
-			? 'u' :''
+			? 'u' : ''
 	);
 }
 

Braces +187 added lines, -128 removed lines

@@ -21,7 +21,9 @@ 
 **/
 
 // securité
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 
 /**
@@ -38,11 +40,13 @@ 
  *     false si le charset n'est pas décrit dans le répertoire charsets/
 **/
 function load_charset ($charset = 'AUTO') {
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
+	if ($charset == 'AUTO') {
+			$charset = $GLOBALS['meta']['charset'];
+	}
 	$charset = trim(strtolower($charset));
-	if (isset($GLOBALS['CHARSET'][$charset]))
-		return $charset;
+	if (isset($GLOBALS['CHARSET'][$charset])) {
+			return $charset;
+	}
 
 	if ($charset == 'utf-8') {
 		$GLOBALS['CHARSET'][$charset] = array();
@@ -50,10 +54,15 @@ 
 	}
 	
 	// Quelques synonymes
-	if ($charset == '') $charset = 'iso-8859-1';
-	else if ($charset == 'windows-1250') $charset = 'cp1250';
-	else if ($charset == 'windows-1251') $charset = 'cp1251';
-	else if ($charset == 'windows-1256') $charset = 'cp1256';
+	if ($charset == '') {
+	    $charset = 'iso-8859-1';
+	} else if ($charset == 'windows-1250') {
+	    $charset = 'cp1250';
+	} else if ($charset == 'windows-1251') {
+	    $charset = 'cp1251';
+	} else if ($charset == 'windows-1256') {
+	    $charset = 'cp1256';
+	}
 
 	if (find_in_path($charset . '.php', 'charsets/', true)) {
 		return $charset;
@@ -88,8 +97,9 @@ 
 		) {
 			mb_internal_encoding('utf-8');
 			$mb = 1;
-		} else
-			$mb = -1;
+		} else {
+					$mb = -1;
+		}
 	}
 
 	return ($mb == 1);
@@ -110,13 +120,14 @@ 
 	static $iconv_ok;
 
 	if (!$iconv_ok) {
-		if (!function_exists('iconv'))
-			$iconv_ok = -1;
-		else {
-			if (utf_32_to_unicode(@iconv('utf-8', 'utf-32', 'chaine de test')) == 'chaine de test')
-				$iconv_ok = 1;
-			else
-				$iconv_ok = -1;
+		if (!function_exists('iconv')) {
+					$iconv_ok = -1;
+		} else {
+			if (utf_32_to_unicode(@iconv('utf-8', 'utf-32', 'chaine de test')) == 'chaine de test') {
+							$iconv_ok = 1;
+			} else {
+							$iconv_ok = -1;
+			}
 		}
 	}
 	return ($iconv_ok == 1);
@@ -136,8 +147,11 @@ 
 
 	if (!$pcre_ok) {
 		$s = " ".chr(195).chr(169)."t".chr(195).chr(169)." ";
-		if (preg_match(',\W...\W,u', $s)) $pcre_ok = 1;
-		else $pcre_ok = -1;
+		if (preg_match(',\W...\W,u', $s)) {
+		    $pcre_ok = 1;
+		} else {
+		    $pcre_ok = -1;
+		}
 	}
 	return $pcre_ok == 1;
 }
@@ -164,8 +178,7 @@ 
 				. '\x{100}-\x{24f}' // europeen etendu
 				. '\x{300}-\x{1cff}' // des tas de trucs
 			;
-		}
-		else {
+		} else {
 			// fallback a trois sous
 			$plage_unicode = '\w';
 		}
@@ -214,15 +227,19 @@ 
 		return array_map('corriger_caracteres_windows', $texte);
 	}
 	
-	if ($charset=='AUTO') $charset = $GLOBALS['meta']['charset'];
+	if ($charset=='AUTO') {
+	    $charset = $GLOBALS['meta']['charset'];
+	}
 	if ($charset == 'utf-8') {
 		$p = chr(194);
-		if (strpos($texte,$p)==false)
-			return $texte;
+		if (strpos($texte,$p)==false) {
+					return $texte;
+		}
 	} else if ($charset == 'iso-8859-1') {
 		$p = '';
-	} else
-		return $texte;
+	} else {
+			return $texte;
+	}
 
 	if (!isset($trans[$charset][$charset_cible])) {
 		$trans[$charset][$charset_cible] = array(
@@ -260,8 +277,9 @@ 
 			$p.chr(159) => "Ÿ",
 		);
 		if ($charset_cible!='unicode'){
-			foreach($trans[$charset][$charset_cible] as $k=>$c)
-				$trans[$charset][$charset_cible][$k] = unicode2charset($c, $charset_cible);
+			foreach($trans[$charset][$charset_cible] as $k=>$c) {
+							$trans[$charset][$charset_cible][$k] = unicode2charset($c, $charset_cible);
+			}
 		}
 	}
 
@@ -284,7 +302,9 @@ 
  *     Texte converti
 **/
 function html2unicode($texte, $secure=false) {
-	if (strpos($texte,'&') === false) return $texte;
+	if (strpos($texte,'&') === false) {
+	    return $texte;
+	}
 	static $trans = array();
 	if (!$trans) {
 		global $CHARSET;
@@ -294,13 +314,14 @@ 
 		}
 	}
 
-	if ($secure)
-		return str_replace(array_keys($trans),array_values($trans),$texte);
-	else
-		return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'),array('&', '"', '<', '>'),
+	if ($secure) {
+			return str_replace(array_keys($trans),array_values($trans),$texte);
+	} else {
+			return str_replace(array('&amp;', '&quot;', '&lt;', '&gt;'),array('&', '"', '<', '>'),
 		  str_replace(array_keys($trans),array_values($trans),$texte)			
 		);
-}
+	}
+	}
 
 
 /**
@@ -319,8 +340,9 @@ 
 		global $CHARSET;
 		load_charset('mathml');
 		
-		foreach ($CHARSET['mathml'] as $key => $val)
-			$trans["&$key;"] = $val;
+		foreach ($CHARSET['mathml'] as $key => $val) {
+					$trans["&$key;"] = $val;
+		}
 	}
 
 	return str_replace(array_keys($trans),array_values($trans),$texte);
@@ -347,10 +369,13 @@ 
 function charset2unicode($texte, $charset='AUTO' /* $forcer: obsolete*/) {
 	static $trans;
 
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
+	if ($charset == 'AUTO') {
+			$charset = $GLOBALS['meta']['charset'];
+	}
 
-	if ($charset == '') $charset = 'iso-8859-1';
+	if ($charset == '') {
+	    $charset = 'iso-8859-1';
+	}
 	$charset = strtolower($charset);
 
 	switch ($charset) {
@@ -368,7 +393,9 @@ 
 			if ($order = mb_detect_order() # mb_string connait-il $charset?
 			AND mb_detect_order($charset)) {
 				$s = mb_convert_encoding($texte, 'utf-8', $charset);
-				if ($s && $s != $texte) return utf_8_to_unicode($s);
+				if ($s && $s != $texte) {
+				    return utf_8_to_unicode($s);
+				}
 			}
 			mb_detect_order($order); # remettre comme precedemment
 		}
@@ -377,20 +404,24 @@ 
 		if (!isset($trans[$charset])) {
 			global $CHARSET;
 			if ($cset = load_charset($charset)
-			AND is_array($CHARSET[$cset]))
-				foreach ($CHARSET[$cset] as $key => $val) {
+			AND is_array($CHARSET[$cset])) {
+							foreach ($CHARSET[$cset] as $key => $val) {
 					$trans[$charset][chr($key)] = '&#'.$val.';';
 			}
+			}
+		}
+		if (count($trans[$charset])) {
+					return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
 		}
-		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
 
 		// Sinon demander a iconv (malgre le fait qu'il coupe quand un
 		// caractere n'appartient pas au charset, mais c'est un probleme
 		// surtout en utf-8, gere ci-dessus)
 		if (test_iconv()) {
 			$s = iconv($charset, 'utf-32le', $texte);
-			if ($s) return utf_32_to_unicode($s);
+			if ($s) {
+			    return utf_32_to_unicode($s);
+			}
 		}
 
 		// Au pire ne rien faire
@@ -418,8 +449,9 @@ 
 	static $CHARSET_REVERSE;
 	static $trans = array();
 	
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
+	if ($charset == 'AUTO') {
+			$charset = $GLOBALS['meta']['charset'];
+	}
 
 	switch($charset) {
 	case 'utf-8':
@@ -442,8 +474,7 @@ 
 					$s = $CHARSET_REVERSE[$charset][$e];
 					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($s);
 					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($s);
-				}
-				else{
+				} else{
 					$t['&#'.$e.';'] = $t['&#0'.$e.';'] = $t['&#00'.$e.';'] = chr($e);
 					$t['&#x'.$h.';'] = $t['&#x0'.$h.';'] = $t['&#x00'.$h.';'] = chr($e);
 				}
@@ -485,13 +516,15 @@ 
 		if (!isset($trans[$charset])) {
 			global $CHARSET;
 			if ($cset = load_charset($charset)
-			AND is_array($CHARSET[$cset]))
-				foreach ($CHARSET[$cset] as $key => $val) {
+			AND is_array($CHARSET[$cset])) {
+							foreach ($CHARSET[$cset] as $key => $val) {
 					$trans[$charset][chr($key)] = unicode2charset('&#'.$val.';');
 			}
+			}
+		}
+		if (count($trans[$charset])) {
+					return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
 		}
-		if (count($trans[$charset]))
-			return str_replace(array_keys($trans[$charset]),array_values($trans[$charset]),$texte);
 		return $texte;
 	}
 	return unicode2charset(charset2unicode($texte, $charset));
@@ -552,18 +585,15 @@ 
 			// 4 chars representing one unicode character
 			$thisLetter = substr ($source, $pos, 4);
 			$pos += 4;
-		}
-		else if (($asciiPos >= 224) && ($asciiPos <= 239)) {
+		} else if (($asciiPos >= 224) && ($asciiPos <= 239)) {
 			// 3 chars representing one unicode character
 			$thisLetter = substr ($source, $pos, 3);
 			$pos += 3;
-		}
-		else if (($asciiPos >= 192) && ($asciiPos <= 223)) {
+		} else if (($asciiPos >= 192) && ($asciiPos <= 223)) {
 			// 2 chars representing one unicode character
 			$thisLetter = substr ($source, $pos, 2);
 			$pos += 2;
-		}
-		else {
+		} else {
 			// 1 char (lower ascii)
 			$thisLetter = substr ($source, $pos, 1);
 			$pos += 1;
@@ -571,9 +601,9 @@ 
 			$ischar = true;
 		}
 
-		if ($ischar)
-			$encodedString .= $char;
-		else {	// process the string representing the letter to a unicode entity
+		if ($ischar) {
+					$encodedString .= $char;
+		} else {	// process the string representing the letter to a unicode entity
 			$thisLen = strlen ($thisLetter);
 			$thisPos = 0;
 			$decimalCode = 0;
@@ -624,11 +654,13 @@ 
 		$words = unpack("V*", substr($source, 0, 1024));
 		$source = substr($source, 1024);
 		foreach ($words as $word) {
-			if ($word < 128)
-				$texte .= chr($word);
+			if ($word < 128) {
+							$texte .= chr($word);
+			}
 			// ignorer le BOM - http://www.unicode.org/faq/utf_bom.html
-			else if ($word != 65279)
-				$texte .= '&#'.$word.';';
+			else if ($word != 65279) {
+							$texte .= '&#'.$word.';';
+			}
 		}
 	}
 	return $texte;
@@ -648,14 +680,18 @@ 
  *    Caractère utf8 si trouvé, '' sinon
 **/
 function caractere_utf_8($num) {
-	if($num<128)
-		return chr($num);
-	if($num<2048)
-		return chr(($num>>6)+192).chr(($num&63)+128);
-	if($num<65536)
-		return chr(($num>>12)+224).chr((($num>>6)&63)+128).chr(($num&63)+128);
-	if($num<1114112)
-		return chr($num>>18+240).chr((($num>>12)&63)+128).chr(($num>>6)&63+128). chr($num&63+128);
+	if($num<128) {
+			return chr($num);
+	}
+	if($num<2048) {
+			return chr(($num>>6)+192).chr(($num&63)+128);
+	}
+	if($num<65536) {
+			return chr(($num>>12)+224).chr((($num>>6)&63)+128).chr(($num&63)+128);
+	}
+	if($num<1114112) {
+			return chr($num>>18+240).chr((($num>>12)&63)+128).chr(($num>>6)&63+128). chr($num&63+128);
+	}
 	return '';
 }
 
@@ -672,21 +708,23 @@ 
 	// 1. Entites &#128; et suivantes
 	$vu = array();
 	if (preg_match_all(',&#0*([1-9][0-9][0-9]+);,S',
-	$texte, $regs, PREG_SET_ORDER))
-	foreach ($regs as $reg) {
+	$texte, $regs, PREG_SET_ORDER)) {
+		foreach ($regs as $reg) {
 		if ($reg[1]>127 AND !isset($vu[$reg[0]]))
 			$vu[$reg[0]] = caractere_utf_8($reg[1]);
 	}
+	}
 	//$texte = str_replace(array_keys($vu), array_values($vu), $texte);
 
 	// 2. Entites > &#xFF;
 	//$vu = array();
 	if (preg_match_all(',&#x0*([1-9a-f][0-9a-f][0-9a-f]+);,iS',
-	$texte, $regs, PREG_SET_ORDER))
-	foreach ($regs as $reg) {
+	$texte, $regs, PREG_SET_ORDER)) {
+		foreach ($regs as $reg) {
 		if (!isset($vu[$reg[0]]))
 			$vu[$reg[0]] = caractere_utf_8(hexdec($reg[1]));
 	}
+	}
 	return str_replace(array_keys($vu), array_values($vu), $texte);
 
 }
@@ -719,8 +757,9 @@ 
  *     Texte converti
 **/
 function javascript_to_unicode ($texte) {
-	while (preg_match(",%u([0-9A-F][0-9A-F][0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],"&#".hexdec($regs[1]).";", $texte);
+	while (preg_match(",%u([0-9A-F][0-9A-F][0-9A-F][0-9A-F]),", $texte, $regs)) {
+			$texte = str_replace($regs[0],"&#".hexdec($regs[1]).";", $texte);
+	}
 	return $texte;
 }
 
@@ -733,8 +772,9 @@ 
  *     Texte converti
 **/
 function javascript_to_binary ($texte) {
-	while (preg_match(",%([0-9A-F][0-9A-F]),", $texte, $regs))
-		$texte = str_replace($regs[0],chr(hexdec($regs[1])), $texte);
+	while (preg_match(",%([0-9A-F][0-9A-F]),", $texte, $regs)) {
+			$texte = str_replace($regs[0],chr(hexdec($regs[1])), $texte);
+	}
 	return $texte;
 }
 
@@ -742,10 +782,12 @@ 
 // http://doc.spip.org/@translitteration_rapide
 function translitteration_rapide($texte, $charset='AUTO', $complexe='') {
 	static $trans;
-	if ($charset == 'AUTO')
-		$charset = $GLOBALS['meta']['charset'];
-	if (!strlen($texte))
-		return $texte;
+	if ($charset == 'AUTO') {
+			$charset = $GLOBALS['meta']['charset'];
+	}
+	if (!strlen($texte)) {
+			return $texte;
+	}
 
 	$table_translit ='translit'.$complexe;
 
@@ -753,8 +795,9 @@ 
 	if (!$trans[$complexe]) {
 		global $CHARSET;
 		load_charset($table_translit);
-		foreach ($CHARSET[$table_translit] as $key => $val)
-			$trans[$complexe][caractere_utf_8($key)] = $val;
+		foreach ($CHARSET[$table_translit] as $key => $val) {
+					$trans[$complexe][caractere_utf_8($key)] = $val;
+		}
 	}
 
 	return str_replace(array_keys($trans[$complexe]),array_values($trans[$complexe]),$texte);
@@ -869,31 +912,37 @@ 
 
 	// charset precise par le contenu (xml)
 	else if (preg_match(
-	',<[?]xml[^>]*encoding[^>]*=[^>]*([-_a-z0-9]+?),UimsS', $texte, $regs))
-		$charset = trim(strtolower($regs[1]));
+	',<[?]xml[^>]*encoding[^>]*=[^>]*([-_a-z0-9]+?),UimsS', $texte, $regs)) {
+			$charset = trim(strtolower($regs[1]));
+	}
 	// charset precise par le contenu (html)
 	else if (preg_match(
 	',<(meta|html|body)[^>]*charset[^>]*=[^>]*([-_a-z0-9]+?),UimsS',
 	$texte, $regs)
 	# eviter #CHARSET des squelettes
-	AND (($tmp = trim(strtolower($regs[2]))) != 'charset'))
-		$charset = $tmp;
+	AND (($tmp = trim(strtolower($regs[2]))) != 'charset')) {
+			$charset = $tmp;
+	}
 	// charset de la reponse http
-	else if (preg_match(',charset=([-_a-z0-9]+),i', $headers, $regs))
-		$charset = trim(strtolower($regs[1]));
-	else $charset = '';
+	else if (preg_match(',charset=([-_a-z0-9]+),i', $headers, $regs)) {
+			$charset = trim(strtolower($regs[1]));
+	} else {
+	    $charset = '';
+	}
 	// normaliser les noms du shif-jis japonais
-	if (preg_match(',^(x|shift)[_-]s?jis$,i', $charset))
-		$charset = 'shift-jis';
+	if (preg_match(',^(x|shift)[_-]s?jis$,i', $charset)) {
+			$charset = 'shift-jis';
+	}
 
 	if ($charset) {
 		spip_log("charset: $charset");
 	} else {
 		// valeur par defaut
-		if (is_utf8($texte))
-			$charset = 'utf-8';
-		else
-			$charset = 'iso-8859-1';
+		if (is_utf8($texte)) {
+					$charset = 'utf-8';
+		} else {
+					$charset = 'iso-8859-1';
+		}
 		spip_log("charset probable: $charset");
 	}
 
@@ -922,18 +971,20 @@ 
 function spip_substr($c, $start=0, $length = NULL) {
 	// Si ce n'est pas utf-8, utiliser substr
 	if ($GLOBALS['meta']['charset'] != 'utf-8') {
-		if ($length)
-			return substr($c, $start, $length);
-		else
-			substr($c, $start);
+		if ($length) {
+					return substr($c, $start, $length);
+		} else {
+					substr($c, $start);
+		}
 	}
 
 	// Si utf-8, voir si on dispose de mb_string
 	if (init_mb_string()) {
-		if ($length)
-			return mb_substr($c, $start, $length);
-		else
-			return mb_substr($c, $start);
+		if ($length) {
+					return mb_substr($c, $start, $length);
+		} else {
+					return mb_substr($c, $start);
+		}
 	}
 
 	// Version manuelle (cf. ci-dessous)
@@ -958,17 +1009,20 @@ 
 function spip_substr_manuelle($c, $start, $length = NULL) {
 
 	// Cas pathologique
-	if ($length === 0)
-		return '';
+	if ($length === 0) {
+			return '';
+	}
 
 	// S'il y a un demarrage, on se positionne
-	if ($start > 0)
-		$c = substr($c, strlen(spip_substr_manuelle($c, 0, $start)));
-	elseif ($start < 0)
-		return spip_substr_manuelle($c, spip_strlen($c)+$start, $length);
+	if ($start > 0) {
+			$c = substr($c, strlen(spip_substr_manuelle($c, 0, $start)));
+	} elseif ($start < 0) {
+			return spip_substr_manuelle($c, spip_strlen($c)+$start, $length);
+	}
 
-	if (!$length)
-		return $c;
+	if (!$length) {
+			return $c;
+	}
 
 	if ($length > 0) {
 		// on prend n fois la longueur desiree, pour etre surs d'avoir tout
@@ -976,8 +1030,9 @@ 
 		$n = 0; while (preg_match(',[\x80-\xBF]{'.(++$n).'},', $c));
 		$c = substr($c, 0, $n*$length);
 		// puis, tant qu'on est trop long, on coupe...
-		while (($l = spip_strlen($c)) > $length)
-			$c = substr($c, 0, $length - $l);
+		while (($l = spip_strlen($c)) > $length) {
+					$c = substr($c, 0, $length - $l);
+		}
 		return $c;
 	}
 
@@ -997,11 +1052,13 @@ 
  */
 function spip_ucfirst($c){
 	// Si ce n'est pas utf-8, utiliser ucfirst 
-	if ($GLOBALS['meta']['charset'] != 'utf-8')
-		return ucfirst($c);
+	if ($GLOBALS['meta']['charset'] != 'utf-8') {
+			return ucfirst($c);
+	}
 	// Si on n'a pas mb_* on utilise ucfirst
-	if (!init_mb_string())
-		return ucfirst($c);
+	if (!init_mb_string()) {
+			return ucfirst($c);
+	}
 	
 	$lettre1 = mb_strtoupper(spip_substr($c,0,1));
 	return $lettre1.spip_substr($c,1);
@@ -1022,12 +1079,14 @@ 
 	$c = str_replace("\r\n", "\n", $c);
 	
 	// Si ce n'est pas utf-8, utiliser strlen
-	if ($GLOBALS['meta']['charset'] != 'utf-8')
-		return strlen($c);
+	if ($GLOBALS['meta']['charset'] != 'utf-8') {
+			return strlen($c);
+	}
 
 	// Sinon, utiliser mb_strlen() si disponible
-	if (init_mb_string())
-		return mb_strlen($c);
+	if (init_mb_string()) {
+			return mb_strlen($c);
+	}
 
 	// Methode manuelle : on supprime les bytes 10......,
 	// on compte donc les ascii (0.......) et les demarrages

ecrire/inc/cvt_autosave.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 /**
  * Nettoyer les echappements
- * @param $val
+ * @param string $val
  * @return string
  */
 function autosave_clean_value($val){

Indentation +49 added lines, -49 removed lines

@@ -18,7 +18,7 @@ 
  * @return string
  */
 function autosave_clean_value($val){
-	return stripslashes(urldecode($val));
+    return stripslashes(urldecode($val));
 }
 
 /**
@@ -29,37 +29,37 @@ 
  * @return array
  */
 function cvtautosave_formulaire_charger($flux){
-	if (is_array($flux['data'])
-	  AND isset($flux['data']['_autosave_id'])
-	  AND $cle_autosave = $flux['data']['_autosave_id']){
+    if (is_array($flux['data'])
+      AND isset($flux['data']['_autosave_id'])
+      AND $cle_autosave = $flux['data']['_autosave_id']){
 
-		$form = $flux['args']['form'];
-		$je_suis_poste = $flux['args']['je_suis_poste'];
+        $form = $flux['args']['form'];
+        $je_suis_poste = $flux['args']['je_suis_poste'];
 
-		$cle_autosave = serialize($cle_autosave);
-		$cle_autosave = $form."_".md5($cle_autosave);
+        $cle_autosave = serialize($cle_autosave);
+        $cle_autosave = $form."_".md5($cle_autosave);
 
-		// si on a un backup en session et qu'on est au premier chargement, non poste
-		// on restitue les donnees
-		if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])
-		  AND !$je_suis_poste) {
-			parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
-			foreach ($vars as $key=>$val) {
-				if (isset($flux['data'][$key]))
-					$flux['data'][$key] = (is_string($val)?autosave_clean_value($val):array_map('autosave_clean_value',$val));
-			}
-		}
+        // si on a un backup en session et qu'on est au premier chargement, non poste
+        // on restitue les donnees
+        if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])
+          AND !$je_suis_poste) {
+            parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
+            foreach ($vars as $key=>$val) {
+                if (isset($flux['data'][$key]))
+                    $flux['data'][$key] = (is_string($val)?autosave_clean_value($val):array_map('autosave_clean_value',$val));
+            }
+        }
 
-		/**
-		 * Envoyer le input hidden et le bout de js qui l'utilisera
-		 */
-		$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
-		  .'<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
+        /**
+         * Envoyer le input hidden et le bout de js qui l'utilisera
+         */
+        $flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
+            .'<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
 		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"'.$GLOBALS['meta']['adresse_site'].'/"}).addClass("autosaveon");
 			});/*]]>*/</script>';
 
-	}
-	return $flux;
+    }
+    return $flux;
 }
 
 /**
@@ -70,32 +70,32 @@ 
  * @return
  */
 function cvtautosave_formulaire_traiter($flux){
-	// si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
-	// on elimine les donnees de la session
-	if ($cle_autosave = _request('autosave')){
-		include_spip('inc/session');
-		session_set('session_autosave_'.$cle_autosave, null);
-	}
+    // si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
+    // on elimine les donnees de la session
+    if ($cle_autosave = _request('autosave')){
+        include_spip('inc/session');
+        session_set('session_autosave_'.$cle_autosave, null);
+    }
 
-	if (isset($GLOBALS['visiteur_session']) AND $GLOBALS['visiteur_session']){
-		// delai par defaut avant purge d'un backup de form : 72H
-		if (!defined('_AUTOSAVE_GB_DELAY')) define('_AUTOSAVE_GB_DELAY',72*3600);
-		$time_too_old = time() - _AUTOSAVE_GB_DELAY;
-		// purger aussi toutes les vieilles autosave
-		$session = $GLOBALS['visiteur_session'];
-		foreach($session as $k=>$v){
-			if (strncmp($k,'session_autosave_',17)==0){
-				$timestamp = 0;
-				if (preg_match(",&__timestamp=(\d+)$,",$v,$m)){
-					$timestamp = intval($m[1]);
-				}
-				if ($timestamp<$time_too_old)
-					session_set($k, null);
-			}
-		}
-	}
+    if (isset($GLOBALS['visiteur_session']) AND $GLOBALS['visiteur_session']){
+        // delai par defaut avant purge d'un backup de form : 72H
+        if (!defined('_AUTOSAVE_GB_DELAY')) define('_AUTOSAVE_GB_DELAY',72*3600);
+        $time_too_old = time() - _AUTOSAVE_GB_DELAY;
+        // purger aussi toutes les vieilles autosave
+        $session = $GLOBALS['visiteur_session'];
+        foreach($session as $k=>$v){
+            if (strncmp($k,'session_autosave_',17)==0){
+                $timestamp = 0;
+                if (preg_match(",&__timestamp=(\d+)$,",$v,$m)){
+                    $timestamp = intval($m[1]);
+                }
+                if ($timestamp<$time_too_old)
+                    session_set($k, null);
+            }
+        }
+    }
 	
-	return $flux;
+    return $flux;
 }
 
 

Braces +12 added lines, -6 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Nettoyer les echappements
@@ -45,8 +47,9 @@ 
 		  AND !$je_suis_poste) {
 			parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
 			foreach ($vars as $key=>$val) {
-				if (isset($flux['data'][$key]))
-					$flux['data'][$key] = (is_string($val)?autosave_clean_value($val):array_map('autosave_clean_value',$val));
+				if (isset($flux['data'][$key])) {
+									$flux['data'][$key] = (is_string($val)?autosave_clean_value($val):array_map('autosave_clean_value',$val));
+				}
 			}
 		}
 
@@ -79,7 +82,9 @@ 
 
 	if (isset($GLOBALS['visiteur_session']) AND $GLOBALS['visiteur_session']){
 		// delai par defaut avant purge d'un backup de form : 72H
-		if (!defined('_AUTOSAVE_GB_DELAY')) define('_AUTOSAVE_GB_DELAY',72*3600);
+		if (!defined('_AUTOSAVE_GB_DELAY')) {
+		    define('_AUTOSAVE_GB_DELAY',72*3600);
+		}
 		$time_too_old = time() - _AUTOSAVE_GB_DELAY;
 		// purger aussi toutes les vieilles autosave
 		$session = $GLOBALS['visiteur_session'];
@@ -89,8 +94,9 @@ 
 				if (preg_match(",&__timestamp=(\d+)$,",$v,$m)){
 					$timestamp = intval($m[1]);
 				}
-				if ($timestamp<$time_too_old)
-					session_set($k, null);
+				if ($timestamp<$time_too_old) {
+									session_set($k, null);
+				}
 			}
 		}
 	}

Spacing +12 added lines, -12 removed lines

@@ -17,7 +17,7 @@ 
  * @param $val
  * @return string
  */
-function autosave_clean_value($val){
+function autosave_clean_value($val) {
 	return stripslashes(urldecode($val));
 }
 
@@ -28,10 +28,10 @@ 
  * @param array $flux
  * @return array
  */
-function cvtautosave_formulaire_charger($flux){
+function cvtautosave_formulaire_charger($flux) {
 	if (is_array($flux['data'])
 	  AND isset($flux['data']['_autosave_id'])
-	  AND $cle_autosave = $flux['data']['_autosave_id']){
+	  AND $cle_autosave = $flux['data']['_autosave_id']) {
 
 		$form = $flux['args']['form'];
 		$je_suis_poste = $flux['args']['je_suis_poste'];
@@ -46,7 +46,7 @@ 
 			parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
 			foreach ($vars as $key=>$val) {
 				if (isset($flux['data'][$key]))
-					$flux['data'][$key] = (is_string($val)?autosave_clean_value($val):array_map('autosave_clean_value',$val));
+					$flux['data'][$key] = (is_string($val) ?autosave_clean_value($val) : array_map('autosave_clean_value', $val));
 			}
 		}
 
@@ -69,27 +69,27 @@ 
  * @param $flux
  * @return
  */
-function cvtautosave_formulaire_traiter($flux){
+function cvtautosave_formulaire_traiter($flux) {
 	// si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
 	// on elimine les donnees de la session
-	if ($cle_autosave = _request('autosave')){
+	if ($cle_autosave = _request('autosave')) {
 		include_spip('inc/session');
 		session_set('session_autosave_'.$cle_autosave, null);
 	}
 
-	if (isset($GLOBALS['visiteur_session']) AND $GLOBALS['visiteur_session']){
+	if (isset($GLOBALS['visiteur_session']) AND $GLOBALS['visiteur_session']) {
 		// delai par defaut avant purge d'un backup de form : 72H
-		if (!defined('_AUTOSAVE_GB_DELAY')) define('_AUTOSAVE_GB_DELAY',72*3600);
+		if (!defined('_AUTOSAVE_GB_DELAY')) define('_AUTOSAVE_GB_DELAY', 72 * 3600);
 		$time_too_old = time() - _AUTOSAVE_GB_DELAY;
 		// purger aussi toutes les vieilles autosave
 		$session = $GLOBALS['visiteur_session'];
-		foreach($session as $k=>$v){
-			if (strncmp($k,'session_autosave_',17)==0){
+		foreach ($session as $k=>$v) {
+			if (strncmp($k, 'session_autosave_', 17) == 0) {
 				$timestamp = 0;
-				if (preg_match(",&__timestamp=(\d+)$,",$v,$m)){
+				if (preg_match(",&__timestamp=(\d+)$,", $v, $m)) {
 					$timestamp = intval($m[1]);
 				}
-				if ($timestamp<$time_too_old)
+				if ($timestamp < $time_too_old)
 					session_set($k, null);
 			}
 		}

ecrire/inc/documents.php

Doc Comments +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
  * http://doc.spip.org/@get_spip_doc
  *
  * @param string $fichier
- * @return bool|string
+ * @return string|false
  */
 function get_spip_doc($fichier) {
 	// fichier distant
@@ -111,7 +111,7 @@ 
  * @param string $ext
  * @param string $orig
  * @param string $source
- * @return bool|mixed|string
+ * @return string|false
  */
 function copier_document($ext, $orig, $source) {
 
@@ -142,7 +142,7 @@ 
  * http://doc.spip.org/@determine_upload
  *
  * @param string $type
- * @return bool|string
+ * @return false|string
  */
 function determine_upload($type='') {
 	if(!function_exists('autoriser'))
@@ -172,7 +172,7 @@ 
  * @param string $source
  * @param string $dest
  * @param bool $move
- * @return bool|mixed|string
+ * @return string|false
  */
 function deplacer_fichier_upload($source, $dest, $move=false) {
 	// Securite

Indentation +140 added lines, -140 removed lines

@@ -22,10 +22,10 @@ 
  * @return string
  */
 function set_spip_doc($fichier) {
-	if (strpos($fichier, _DIR_IMG) === 0)
-		return substr($fichier, strlen(_DIR_IMG));
-	else
-		return $fichier; // ex: fichier distant
+    if (strpos($fichier, _DIR_IMG) === 0)
+        return substr($fichier, strlen(_DIR_IMG));
+    else
+        return $fichier; // ex: fichier distant
 }
 
 /**
@@ -37,22 +37,22 @@ 
  * @return bool|string
  */
 function get_spip_doc($fichier) {
-	// fichier distant
-	if (preg_match(',^\w+://,', $fichier))
-		return $fichier;
-
-	// gestion d'erreurs, fichier=''
-	if (!strlen($fichier))
-		return false;
-
-	$fichier = (
-		    strncmp($fichier,_DIR_IMG, strlen(_DIR_IMG))!=0
-		)
-		? _DIR_IMG . $fichier
-		: $fichier ;
-
-	// fichier normal
-	return $fichier;
+    // fichier distant
+    if (preg_match(',^\w+://,', $fichier))
+        return $fichier;
+
+    // gestion d'erreurs, fichier=''
+    if (!strlen($fichier))
+        return false;
+
+    $fichier = (
+            strncmp($fichier,_DIR_IMG, strlen(_DIR_IMG))!=0
+        )
+        ? _DIR_IMG . $fichier
+        : $fichier ;
+
+    // fichier normal
+    return $fichier;
 }
 
 /**
@@ -64,21 +64,21 @@ 
  * @return string
  */
 function creer_repertoire_documents($ext) {
-	$rep = sous_repertoire(_DIR_IMG, $ext);
+    $rep = sous_repertoire(_DIR_IMG, $ext);
 
-	if (!$ext OR !$rep) {
-		spip_log("creer_repertoire_documents '$rep' interdit");
-		exit;
-	}
+    if (!$ext OR !$rep) {
+        spip_log("creer_repertoire_documents '$rep' interdit");
+        exit;
+    }
 
-	// Cette variable de configuration peut etre posee par un plugin
-	// par exemple acces_restreint
-	if (isset($GLOBALS['meta']["creer_htaccess"]) AND $GLOBALS['meta']["creer_htaccess"] == 'oui') {
-		include_spip('inc/acces');
-		verifier_htaccess($rep);
-	}
+    // Cette variable de configuration peut etre posee par un plugin
+    // par exemple acces_restreint
+    if (isset($GLOBALS['meta']["creer_htaccess"]) AND $GLOBALS['meta']["creer_htaccess"] == 'oui') {
+        include_spip('inc/acces');
+        verifier_htaccess($rep);
+    }
 
-	return $rep;
+    return $rep;
 }
 
 /**
@@ -89,16 +89,16 @@ 
  * @param string $nom
  */
 function effacer_repertoire_temporaire($nom) {
-	$d = opendir($nom);
-	while (($f = readdir($d)) !== false) {
-		if (is_file("$nom/$f"))
-			spip_unlink("$nom/$f");
-		else if ($f <> '.' AND $f <> '..'
-		AND is_dir("$nom/$f"))
-			effacer_repertoire_temporaire("$nom/$f");
-	}
-	closedir($d);
-	@rmdir($nom);
+    $d = opendir($nom);
+    while (($f = readdir($d)) !== false) {
+        if (is_file("$nom/$f"))
+            spip_unlink("$nom/$f");
+        else if ($f <> '.' AND $f <> '..'
+        AND is_dir("$nom/$f"))
+            effacer_repertoire_temporaire("$nom/$f");
+    }
+    closedir($d);
+    @rmdir($nom);
 }
 
 //
@@ -115,25 +115,25 @@ 
  */
 function copier_document($ext, $orig, $source) {
 
-	$orig = preg_replace(',\.\.+,', '.', $orig); // pas de .. dans le nom du doc
-	$dir = creer_repertoire_documents($ext);
-	$dest = preg_replace("/[^.=\w-]+/", "_",
-			translitteration(preg_replace("/\.([^.]+)$/", "",
-						      preg_replace("/<[^>]*>/", '', basename($orig)))));
+    $orig = preg_replace(',\.\.+,', '.', $orig); // pas de .. dans le nom du doc
+    $dir = creer_repertoire_documents($ext);
+    $dest = preg_replace("/[^.=\w-]+/", "_",
+            translitteration(preg_replace("/\.([^.]+)$/", "",
+                                preg_replace("/<[^>]*>/", '', basename($orig)))));
 
-	// ne pas accepter de noms de la forme -r90.jpg qui sont reserves
-	// pour les images transformees par rotation (action/documenter)
-	$dest = preg_replace(',-r(90|180|270)$,', '', $dest);
+    // ne pas accepter de noms de la forme -r90.jpg qui sont reserves
+    // pour les images transformees par rotation (action/documenter)
+    $dest = preg_replace(',-r(90|180|270)$,', '', $dest);
 
-	// Si le document "source" est deja au bon endroit, ne rien faire
-	if ($source == ($dir . $dest . '.' . $ext))
-		return $source;
+    // Si le document "source" est deja au bon endroit, ne rien faire
+    if ($source == ($dir . $dest . '.' . $ext))
+        return $source;
 
-	// sinon tourner jusqu'a trouver un numero correct
-	$n = 0;
-	while (@file_exists($newFile = $dir . $dest .($n++ ? ('-'.$n) : '').'.'.$ext));
+    // sinon tourner jusqu'a trouver un numero correct
+    $n = 0;
+    while (@file_exists($newFile = $dir . $dest .($n++ ? ('-'.$n) : '').'.'.$ext));
 
-	return deplacer_fichier_upload($source, $newFile);
+    return deplacer_fichier_upload($source, $newFile);
 }
 
 /**
@@ -145,23 +145,23 @@ 
  * @return bool|string
  */
 function determine_upload($type='') {
-	if(!function_exists('autoriser'))
-		include_spip('inc/autoriser');
+    if(!function_exists('autoriser'))
+        include_spip('inc/autoriser');
 	
-	if (!autoriser('chargerftp')
-	OR $type == 'logos') # on ne le permet pas pour les logos
-		return false;
-
-	$repertoire = _DIR_TRANSFERT;
-	if (!@is_dir($repertoire)) {
-		$repertoire = str_replace(_DIR_TMP, '', $repertoire);
-		$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
-	}
-
-	if (!$GLOBALS['visiteur_session']['restreint'])
-		return $repertoire;
-	else
-		return sous_repertoire($repertoire, $GLOBALS['visiteur_session']['login']);
+    if (!autoriser('chargerftp')
+    OR $type == 'logos') # on ne le permet pas pour les logos
+        return false;
+
+    $repertoire = _DIR_TRANSFERT;
+    if (!@is_dir($repertoire)) {
+        $repertoire = str_replace(_DIR_TMP, '', $repertoire);
+        $repertoire = sous_repertoire(_DIR_TMP, $repertoire);
+    }
+
+    if (!$GLOBALS['visiteur_session']['restreint'])
+        return $repertoire;
+    else
+        return sous_repertoire($repertoire, $GLOBALS['visiteur_session']['login']);
 }
 
 /**
@@ -175,28 +175,28 @@ 
  * @return bool|mixed|string
  */
 function deplacer_fichier_upload($source, $dest, $move=false) {
-	// Securite
-	if (substr($dest,0,strlen(_DIR_RACINE))==_DIR_RACINE)
-		$dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest,strlen(_DIR_RACINE)));
-	else
-		$dest = preg_replace(',\.\.+,', '.', $dest);
-
-	if ($move)	$ok = @rename($source, $dest);
-	else				$ok = @copy($source, $dest);
-	if (!$ok) $ok = @move_uploaded_file($source, $dest);
-	if ($ok)
-		@chmod($dest, _SPIP_CHMOD & ~0111);
-	else {
-		$f = @fopen($dest,'w');
-		if ($f) {
-			fclose ($f);
-		} else {
-			include_spip('inc/flock');
-			raler_fichier($dest);
-		}
-		spip_unlink($dest);
-	}
-	return $ok ? $dest : false;
+    // Securite
+    if (substr($dest,0,strlen(_DIR_RACINE))==_DIR_RACINE)
+        $dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest,strlen(_DIR_RACINE)));
+    else
+        $dest = preg_replace(',\.\.+,', '.', $dest);
+
+    if ($move)	$ok = @rename($source, $dest);
+    else				$ok = @copy($source, $dest);
+    if (!$ok) $ok = @move_uploaded_file($source, $dest);
+    if ($ok)
+        @chmod($dest, _SPIP_CHMOD & ~0111);
+    else {
+        $f = @fopen($dest,'w');
+        if ($f) {
+            fclose ($f);
+        } else {
+            include_spip('inc/flock');
+            raler_fichier($dest);
+        }
+        spip_unlink($dest);
+    }
+    return $ok ? $dest : false;
 }
 
 
@@ -206,50 +206,50 @@ 
 // pour les autres erreurs affiche le message d'erreur et meurt
 // http://doc.spip.org/@check_upload_error
 function check_upload_error($error, $msg='', $return=false) {
-	global $spip_lang_right;
-
-	if (!$error) return false;
-
-	spip_log("Erreur upload $error -- cf. http://php.net/manual/fr/features.file-upload.errors.php");
-
-	switch ($error) {
-
-		case 4: /* UPLOAD_ERR_NO_FILE */
-			return true;
-
-		# on peut affiner les differents messages d'erreur
-		case 1: /* UPLOAD_ERR_INI_SIZE */
-			$msg = _T('upload_limit',
-			array('max' => ini_get('upload_max_filesize')));
-			break;
-		case 2: /* UPLOAD_ERR_FORM_SIZE */
-			$msg = _T('upload_limit',
-			array('max' => ini_get('upload_max_filesize')));
-			break;
-		case 3: /* UPLOAD_ERR_PARTIAL  */
-			$msg = _T('upload_limit',
-			array('max' => ini_get('upload_max_filesize')));
-			break;
-
-		default: /* autre */
-			if (!$msg)
-			$msg = _T('pass_erreur').' '. $error
-			. '<br />' . propre("[->http://php.net/manual/fr/features.file-upload.errors.php]");
-			break;
-	}
-
-	spip_log ("erreur upload $error");
-	if ($return)
-		return $msg;
-
-  if(_request("iframe")=="iframe") {
-	  echo "<div class='upload_answer upload_error'>$msg</div>";
-	  exit;
-	}
-
-	include_spip('inc/minipres');
-	echo minipres($msg,
-		      "<div style='text-align: $spip_lang_right'><a href='"  . rawurldecode($GLOBALS['redirect']) . "'><button type='button'>" . _T('ecrire:bouton_suivant') . "</button></a></div>");
-	exit;
+    global $spip_lang_right;
+
+    if (!$error) return false;
+
+    spip_log("Erreur upload $error -- cf. http://php.net/manual/fr/features.file-upload.errors.php");
+
+    switch ($error) {
+
+        case 4: /* UPLOAD_ERR_NO_FILE */
+            return true;
+
+        # on peut affiner les differents messages d'erreur
+        case 1: /* UPLOAD_ERR_INI_SIZE */
+            $msg = _T('upload_limit',
+            array('max' => ini_get('upload_max_filesize')));
+            break;
+        case 2: /* UPLOAD_ERR_FORM_SIZE */
+            $msg = _T('upload_limit',
+            array('max' => ini_get('upload_max_filesize')));
+            break;
+        case 3: /* UPLOAD_ERR_PARTIAL  */
+            $msg = _T('upload_limit',
+            array('max' => ini_get('upload_max_filesize')));
+            break;
+
+        default: /* autre */
+            if (!$msg)
+            $msg = _T('pass_erreur').' '. $error
+            . '<br />' . propre("[->http://php.net/manual/fr/features.file-upload.errors.php]");
+            break;
+    }
+
+    spip_log ("erreur upload $error");
+    if ($return)
+        return $msg;
+
+    if(_request("iframe")=="iframe") {
+        echo "<div class='upload_answer upload_error'>$msg</div>";
+        exit;
+    }
+
+    include_spip('inc/minipres');
+    echo minipres($msg,
+                "<div style='text-align: $spip_lang_right'><a href='"  . rawurldecode($GLOBALS['redirect']) . "'><button type='button'>" . _T('ecrire:bouton_suivant') . "</button></a></div>");
+    exit;
 }
 ?>

Braces +62 added lines, -40 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * donne le chemin du fichier relatif a _DIR_IMG
@@ -22,10 +24,12 @@ 
  * @return string
  */
 function set_spip_doc($fichier) {
-	if (strpos($fichier, _DIR_IMG) === 0)
-		return substr($fichier, strlen(_DIR_IMG));
-	else
-		return $fichier; // ex: fichier distant
+	if (strpos($fichier, _DIR_IMG) === 0) {
+			return substr($fichier, strlen(_DIR_IMG));
+	} else {
+			return $fichier;
+	}
+	// ex: fichier distant
 }
 
 /**
@@ -38,12 +42,14 @@ 
  */
 function get_spip_doc($fichier) {
 	// fichier distant
-	if (preg_match(',^\w+://,', $fichier))
-		return $fichier;
+	if (preg_match(',^\w+://,', $fichier)) {
+			return $fichier;
+	}
 
 	// gestion d'erreurs, fichier=''
-	if (!strlen($fichier))
-		return false;
+	if (!strlen($fichier)) {
+			return false;
+	}
 
 	$fichier = (
 		    strncmp($fichier,_DIR_IMG, strlen(_DIR_IMG))!=0
@@ -91,11 +97,12 @@ 
 function effacer_repertoire_temporaire($nom) {
 	$d = opendir($nom);
 	while (($f = readdir($d)) !== false) {
-		if (is_file("$nom/$f"))
-			spip_unlink("$nom/$f");
-		else if ($f <> '.' AND $f <> '..'
-		AND is_dir("$nom/$f"))
-			effacer_repertoire_temporaire("$nom/$f");
+		if (is_file("$nom/$f")) {
+					spip_unlink("$nom/$f");
+		} else if ($f <> '.' AND $f <> '..'
+		AND is_dir("$nom/$f")) {
+					effacer_repertoire_temporaire("$nom/$f");
+		}
 	}
 	closedir($d);
 	@rmdir($nom);
@@ -126,8 +133,9 @@ 
 	$dest = preg_replace(',-r(90|180|270)$,', '', $dest);
 
 	// Si le document "source" est deja au bon endroit, ne rien faire
-	if ($source == ($dir . $dest . '.' . $ext))
-		return $source;
+	if ($source == ($dir . $dest . '.' . $ext)) {
+			return $source;
+	}
 
 	// sinon tourner jusqu'a trouver un numero correct
 	$n = 0;
@@ -145,12 +153,15 @@ 
  * @return bool|string
  */
 function determine_upload($type='') {
-	if(!function_exists('autoriser'))
-		include_spip('inc/autoriser');
+	if(!function_exists('autoriser')) {
+			include_spip('inc/autoriser');
+	}
 	
 	if (!autoriser('chargerftp')
-	OR $type == 'logos') # on ne le permet pas pour les logos
+	OR $type == 'logos') {
+	    # on ne le permet pas pour les logos
 		return false;
+	}
 
 	$repertoire = _DIR_TRANSFERT;
 	if (!@is_dir($repertoire)) {
@@ -158,11 +169,12 @@ 
 		$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
 	}
 
-	if (!$GLOBALS['visiteur_session']['restreint'])
-		return $repertoire;
-	else
-		return sous_repertoire($repertoire, $GLOBALS['visiteur_session']['login']);
-}
+	if (!$GLOBALS['visiteur_session']['restreint']) {
+			return $repertoire;
+	} else {
+			return sous_repertoire($repertoire, $GLOBALS['visiteur_session']['login']);
+	}
+	}
 
 /**
  * Deplacer ou copier un fichier
@@ -176,17 +188,23 @@ 
  */
 function deplacer_fichier_upload($source, $dest, $move=false) {
 	// Securite
-	if (substr($dest,0,strlen(_DIR_RACINE))==_DIR_RACINE)
-		$dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest,strlen(_DIR_RACINE)));
-	else
-		$dest = preg_replace(',\.\.+,', '.', $dest);
-
-	if ($move)	$ok = @rename($source, $dest);
-	else				$ok = @copy($source, $dest);
-	if (!$ok) $ok = @move_uploaded_file($source, $dest);
-	if ($ok)
-		@chmod($dest, _SPIP_CHMOD & ~0111);
-	else {
+	if (substr($dest,0,strlen(_DIR_RACINE))==_DIR_RACINE) {
+			$dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest,strlen(_DIR_RACINE)));
+	} else {
+			$dest = preg_replace(',\.\.+,', '.', $dest);
+	}
+
+	if ($move) {
+	    $ok = @rename($source, $dest);
+	} else {
+	    $ok = @copy($source, $dest);
+	}
+	if (!$ok) {
+	    $ok = @move_uploaded_file($source, $dest);
+	}
+	if ($ok) {
+			@chmod($dest, _SPIP_CHMOD & ~0111);
+	} else {
 		$f = @fopen($dest,'w');
 		if ($f) {
 			fclose ($f);
@@ -208,7 +226,9 @@ 
 function check_upload_error($error, $msg='', $return=false) {
 	global $spip_lang_right;
 
-	if (!$error) return false;
+	if (!$error) {
+	    return false;
+	}
 
 	spip_log("Erreur upload $error -- cf. http://php.net/manual/fr/features.file-upload.errors.php");
 
@@ -232,15 +252,17 @@ 
 			break;
 
 		default: /* autre */
-			if (!$msg)
-			$msg = _T('pass_erreur').' '. $error
+			if (!$msg) {
+						$msg = _T('pass_erreur').' '. $error
 			. '<br />' . propre("[->http://php.net/manual/fr/features.file-upload.errors.php]");
+			}
 			break;
 	}
 
 	spip_log ("erreur upload $error");
-	if ($return)
-		return $msg;
+	if ($return) {
+			return $msg;
+	}
 
   if(_request("iframe")=="iframe") {
 	  echo "<div class='upload_answer upload_error'>$msg</div>";

Spacing +17 added lines, -17 removed lines

@@ -46,10 +46,10 @@ 
 		return false;
 
 	$fichier = (
-		    strncmp($fichier,_DIR_IMG, strlen(_DIR_IMG))!=0
+		    strncmp($fichier, _DIR_IMG, strlen(_DIR_IMG)) != 0
 		)
 		? _DIR_IMG . $fichier
-		: $fichier ;
+		: $fichier;
 
 	// fichier normal
 	return $fichier;
@@ -126,12 +126,12 @@ 
 	$dest = preg_replace(',-r(90|180|270)$,', '', $dest);
 
 	// Si le document "source" est deja au bon endroit, ne rien faire
-	if ($source == ($dir . $dest . '.' . $ext))
+	if ($source == ($dir.$dest.'.'.$ext))
 		return $source;
 
 	// sinon tourner jusqu'a trouver un numero correct
 	$n = 0;
-	while (@file_exists($newFile = $dir . $dest .($n++ ? ('-'.$n) : '').'.'.$ext));
+	while (@file_exists($newFile = $dir.$dest.($n++ ? ('-'.$n) : '').'.'.$ext));
 
 	return deplacer_fichier_upload($source, $newFile);
 }
@@ -144,8 +144,8 @@ 
  * @param string $type
  * @return bool|string
  */
-function determine_upload($type='') {
-	if(!function_exists('autoriser'))
+function determine_upload($type = '') {
+	if (!function_exists('autoriser'))
 		include_spip('inc/autoriser');
 	
 	if (!autoriser('chargerftp')
@@ -174,10 +174,10 @@ 
  * @param bool $move
  * @return bool|mixed|string
  */
-function deplacer_fichier_upload($source, $dest, $move=false) {
+function deplacer_fichier_upload($source, $dest, $move = false) {
 	// Securite
-	if (substr($dest,0,strlen(_DIR_RACINE))==_DIR_RACINE)
-		$dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest,strlen(_DIR_RACINE)));
+	if (substr($dest, 0, strlen(_DIR_RACINE)) == _DIR_RACINE)
+		$dest = _DIR_RACINE.preg_replace(',\.\.+,', '.', substr($dest, strlen(_DIR_RACINE)));
 	else
 		$dest = preg_replace(',\.\.+,', '.', $dest);
 
@@ -187,9 +187,9 @@ 
 	if ($ok)
 		@chmod($dest, _SPIP_CHMOD & ~0111);
 	else {
-		$f = @fopen($dest,'w');
+		$f = @fopen($dest, 'w');
 		if ($f) {
-			fclose ($f);
+			fclose($f);
 		} else {
 			include_spip('inc/flock');
 			raler_fichier($dest);
@@ -205,7 +205,7 @@ 
 // et true si erreur = pas de fichier
 // pour les autres erreurs affiche le message d'erreur et meurt
 // http://doc.spip.org/@check_upload_error
-function check_upload_error($error, $msg='', $return=false) {
+function check_upload_error($error, $msg = '', $return = false) {
 	global $spip_lang_right;
 
 	if (!$error) return false;
@@ -233,23 +233,23 @@ 
 
 		default: /* autre */
 			if (!$msg)
-			$msg = _T('pass_erreur').' '. $error
-			. '<br />' . propre("[->http://php.net/manual/fr/features.file-upload.errors.php]");
+			$msg = _T('pass_erreur').' '.$error
+			. '<br />'.propre("[->http://php.net/manual/fr/features.file-upload.errors.php]");
 			break;
 	}
 
-	spip_log ("erreur upload $error");
+	spip_log("erreur upload $error");
 	if ($return)
 		return $msg;
 
-  if(_request("iframe")=="iframe") {
+  if (_request("iframe") == "iframe") {
 	  echo "<div class='upload_answer upload_error'>$msg</div>";
 	  exit;
 	}
 
 	include_spip('inc/minipres');
 	echo minipres($msg,
-		      "<div style='text-align: $spip_lang_right'><a href='"  . rawurldecode($GLOBALS['redirect']) . "'><button type='button'>" . _T('ecrire:bouton_suivant') . "</button></a></div>");
+		      "<div style='text-align: $spip_lang_right'><a href='".rawurldecode($GLOBALS['redirect'])."'><button type='button'>"._T('ecrire:bouton_suivant')."</button></a></div>");
 	exit;
 }
 ?>

ecrire/inc/drapeau_edition.php

Doc Comments +3 added lines

@@ -105,6 +105,9 @@
 }
 
 // http://doc.spip.org/@mention_qui_edite
+/**
+ * @param integer $id
+ */
 function mention_qui_edite ($id, $type='article') {
 	$modif = qui_edite($id, $type);
 	unset($modif[$GLOBALS['visiteur_session']['id_auteur']]);

Indentation +101 added lines, -101 removed lines

@@ -30,40 +30,40 @@ 
 
 // http://doc.spip.org/@lire_tableau_edition
 function lire_tableau_edition () {
-	$edition = @unserialize($GLOBALS['meta']['drapeau_edition']);
-	if (!$edition) return array();
-	$changed = false;
-
-	$bon_pour_le_service = time()-3600;
-	// parcourir le tableau et virer les vieux
-	foreach ($edition as $objet => $data) {
-		if (!is_array($data))
-			unset ($edition[$objet]); // vieille version
-		else foreach ($data as $id => $tab) {
-			if (!is_array($tab))
-			  unset ($edition[$objet][$tab]); // vieille version
-			else foreach ($tab as $n => $duo) {
-				if (current($duo) < $bon_pour_le_service) {
-					unset($edition[$objet][$id][$n]);
-					$changed = true;
-				}
-			}
-			if (!$edition[$objet][$id])
-				unset($edition[$objet][$id]);
-		}
-		if (!$edition[$objet])
-			unset($edition[$objet]);
-	}
-
-	if ($changed)
-		ecrire_tableau_edition($edition);
-
-	return $edition;
+    $edition = @unserialize($GLOBALS['meta']['drapeau_edition']);
+    if (!$edition) return array();
+    $changed = false;
+
+    $bon_pour_le_service = time()-3600;
+    // parcourir le tableau et virer les vieux
+    foreach ($edition as $objet => $data) {
+        if (!is_array($data))
+            unset ($edition[$objet]); // vieille version
+        else foreach ($data as $id => $tab) {
+            if (!is_array($tab))
+                unset ($edition[$objet][$tab]); // vieille version
+            else foreach ($tab as $n => $duo) {
+                if (current($duo) < $bon_pour_le_service) {
+                    unset($edition[$objet][$id][$n]);
+                    $changed = true;
+                }
+            }
+            if (!$edition[$objet][$id])
+                unset($edition[$objet][$id]);
+        }
+        if (!$edition[$objet])
+            unset($edition[$objet]);
+    }
+
+    if ($changed)
+        ecrire_tableau_edition($edition);
+
+    return $edition;
 }
 
 // http://doc.spip.org/@ecrire_tableau_edition
 function ecrire_tableau_edition($edition) {
-	ecrire_meta('drapeau_edition', serialize($edition));
+    ecrire_meta('drapeau_edition', serialize($edition));
 }
 
 /**
@@ -79,48 +79,48 @@ 
  * @return mixed
  */
 function signale_edition ($id, $auteur, $type='article') {
-	include_spip('base/objets');
-	include_spip('inc/filtres');
-	if (objet_info($type,'editable')!=='oui')
-		return;
-
-	$edition = lire_tableau_edition();
-	if (isset($auteur['id_auteur']) and $id_a = $auteur['id_auteur'])
-		$nom = $auteur['nom'];
-	else
-		$nom = $id_a = $GLOBALS['ip'];
-	if (!is_array($edition[$type][$id]))
-		$edition[$type][$id] = array();
-	$edition[$type][$id][$id_a][$nom] = time();
-	ecrire_tableau_edition($edition);
+    include_spip('base/objets');
+    include_spip('inc/filtres');
+    if (objet_info($type,'editable')!=='oui')
+        return;
+
+    $edition = lire_tableau_edition();
+    if (isset($auteur['id_auteur']) and $id_a = $auteur['id_auteur'])
+        $nom = $auteur['nom'];
+    else
+        $nom = $id_a = $GLOBALS['ip'];
+    if (!is_array($edition[$type][$id]))
+        $edition[$type][$id] = array();
+    $edition[$type][$id][$id_a][$nom] = time();
+    ecrire_tableau_edition($edition);
 }
 
 // Qui edite mon objet ?
 // http://doc.spip.org/@qui_edite
 function qui_edite ($id, $type='article') {
 
-	$edition = lire_tableau_edition();
+    $edition = lire_tableau_edition();
 
-	return $edition ? $edition[$type][$id] : array();
+    return $edition ? $edition[$type][$id] : array();
 }
 
 // http://doc.spip.org/@mention_qui_edite
 function mention_qui_edite ($id, $type='article') {
-	$modif = qui_edite($id, $type);
-	unset($modif[$GLOBALS['visiteur_session']['id_auteur']]);
-
-	if ($modif) {
-		$quand = 0;
-		foreach ($modif as $duo) {
-			$auteurs[] = typo(key($duo));
-			$quand = max($quand, current($duo));
-		}
-		// format lie a la chaine de langue 'avis_article_modifie'
-		return array(
-			'nom_auteur_modif' => join(' | ', $auteurs),
-			'date_diff' => ceil((time()-$quand) / 60)
-		);
-	}
+    $modif = qui_edite($id, $type);
+    unset($modif[$GLOBALS['visiteur_session']['id_auteur']]);
+
+    if ($modif) {
+        $quand = 0;
+        foreach ($modif as $duo) {
+            $auteurs[] = typo(key($duo));
+            $quand = max($quand, current($duo));
+        }
+        // format lie a la chaine de langue 'avis_article_modifie'
+        return array(
+            'nom_auteur_modif' => join(' | ', $auteurs),
+            'date_diff' => ceil((time()-$quand) / 60)
+        );
+    }
 }
 
 /**
@@ -132,55 +132,55 @@ 
  * @return array
  */
 function liste_drapeau_edition ($id_auteur) {
-	$edition = lire_tableau_edition();
-	$objets_ouverts = array();
-
-	foreach ($edition as $objet => $data)
-		foreach ($data as $id => $auteurs)
-		{
-			if (isset($auteurs[$id_auteur])
-			AND is_array($auteurs[$id_auteur]) // precaution
-			AND (array_pop($auteurs[$id_auteur]) > time()-3600)) {
-				$objets_ouverts[] = array(
-					'objet'=>$objet,
-					'id_objet' => $id,
-				);
-			}
-		}
-	return $objets_ouverts;
+    $edition = lire_tableau_edition();
+    $objets_ouverts = array();
+
+    foreach ($edition as $objet => $data)
+        foreach ($data as $id => $auteurs)
+        {
+            if (isset($auteurs[$id_auteur])
+            AND is_array($auteurs[$id_auteur]) // precaution
+            AND (array_pop($auteurs[$id_auteur]) > time()-3600)) {
+                $objets_ouverts[] = array(
+                    'objet'=>$objet,
+                    'id_objet' => $id,
+                );
+            }
+        }
+    return $objets_ouverts;
 }
 
 // Quand l'auteur veut liberer tous ses objets (tous types)
 // http://doc.spip.org/@debloquer_tous
 function debloquer_tous($id_auteur) {
-	$edition = lire_tableau_edition();
-	foreach ($edition as $objet => $data) {
-		foreach ($data as $id => $auteurs)
-		{
-			if (isset($auteurs[$id_auteur])) {
-				unset ($edition[$objet][$id][$id_auteur]);
-				ecrire_tableau_edition($edition);
-			}
-		}
-	}
+    $edition = lire_tableau_edition();
+    foreach ($edition as $objet => $data) {
+        foreach ($data as $id => $auteurs)
+        {
+            if (isset($auteurs[$id_auteur])) {
+                unset ($edition[$objet][$id][$id_auteur]);
+                ecrire_tableau_edition($edition);
+            }
+        }
+    }
 }
 
 // quand l'auteur libere un objet precis
 // http://doc.spip.org/@debloquer_edition
 function debloquer_edition($id_auteur, $id_objet, $type='article') {
-	$edition = lire_tableau_edition();
-
-	foreach ($edition as $objet => $data){
-		if ($objet == $type) {
-			foreach ($data as $id => $auteurs)
-			{
-				if ($id == $id_objet
-				AND isset($auteurs[$id_auteur])) {
-					unset ($edition[$objet][$id][$id_auteur]);
-					ecrire_tableau_edition($edition);
-				}
-			}
-		}
-	}
+    $edition = lire_tableau_edition();
+
+    foreach ($edition as $objet => $data){
+        if ($objet == $type) {
+            foreach ($data as $id => $auteurs)
+            {
+                if ($id == $id_objet
+                AND isset($auteurs[$id_auteur])) {
+                    unset ($edition[$objet][$id][$id_auteur]);
+                    ecrire_tableau_edition($edition);
+                }
+            }
+        }
+    }
 }
 ?>

Spacing +11 added lines, -11 removed lines

@@ -29,12 +29,12 @@ 
 
 
 // http://doc.spip.org/@lire_tableau_edition
-function lire_tableau_edition () {
+function lire_tableau_edition() {
 	$edition = @unserialize($GLOBALS['meta']['drapeau_edition']);
 	if (!$edition) return array();
 	$changed = false;
 
-	$bon_pour_le_service = time()-3600;
+	$bon_pour_le_service = time() - 3600;
 	// parcourir le tableau et virer les vieux
 	foreach ($edition as $objet => $data) {
 		if (!is_array($data))
@@ -78,10 +78,10 @@ 
  * @param string $type
  * @return mixed
  */
-function signale_edition ($id, $auteur, $type='article') {
+function signale_edition($id, $auteur, $type = 'article') {
 	include_spip('base/objets');
 	include_spip('inc/filtres');
-	if (objet_info($type,'editable')!=='oui')
+	if (objet_info($type, 'editable') !== 'oui')
 		return;
 
 	$edition = lire_tableau_edition();
@@ -97,7 +97,7 @@ 
 
 // Qui edite mon objet ?
 // http://doc.spip.org/@qui_edite
-function qui_edite ($id, $type='article') {
+function qui_edite($id, $type = 'article') {
 
 	$edition = lire_tableau_edition();
 
@@ -105,7 +105,7 @@ 
 }
 
 // http://doc.spip.org/@mention_qui_edite
-function mention_qui_edite ($id, $type='article') {
+function mention_qui_edite($id, $type = 'article') {
 	$modif = qui_edite($id, $type);
 	unset($modif[$GLOBALS['visiteur_session']['id_auteur']]);
 
@@ -118,7 +118,7 @@ 
 		// format lie a la chaine de langue 'avis_article_modifie'
 		return array(
 			'nom_auteur_modif' => join(' | ', $auteurs),
-			'date_diff' => ceil((time()-$quand) / 60)
+			'date_diff' => ceil((time() - $quand) / 60)
 		);
 	}
 }
@@ -131,7 +131,7 @@ 
  * @param $id_auteur
  * @return array
  */
-function liste_drapeau_edition ($id_auteur) {
+function liste_drapeau_edition($id_auteur) {
 	$edition = lire_tableau_edition();
 	$objets_ouverts = array();
 
@@ -140,7 +140,7 @@ 
 		{
 			if (isset($auteurs[$id_auteur])
 			AND is_array($auteurs[$id_auteur]) // precaution
-			AND (array_pop($auteurs[$id_auteur]) > time()-3600)) {
+			AND (array_pop($auteurs[$id_auteur]) > time() - 3600)) {
 				$objets_ouverts[] = array(
 					'objet'=>$objet,
 					'id_objet' => $id,
@@ -167,10 +167,10 @@ 
 
 // quand l'auteur libere un objet precis
 // http://doc.spip.org/@debloquer_edition
-function debloquer_edition($id_auteur, $id_objet, $type='article') {
+function debloquer_edition($id_auteur, $id_objet, $type = 'article') {
 	$edition = lire_tableau_edition();
 
-	foreach ($edition as $objet => $data){
+	foreach ($edition as $objet => $data) {
 		if ($objet == $type) {
 			foreach ($data as $id => $auteurs)
 			{

Braces +41 added lines, -23 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Drapeau d'edition : on regarde qui a ouvert quel objet editorial en 
 // edition, et on le signale aux autres redacteurs pour eviter de se marcher 
@@ -31,32 +33,44 @@ 
 // http://doc.spip.org/@lire_tableau_edition
 function lire_tableau_edition () {
 	$edition = @unserialize($GLOBALS['meta']['drapeau_edition']);
-	if (!$edition) return array();
+	if (!$edition) {
+	    return array();
+	}
 	$changed = false;
 
 	$bon_pour_le_service = time()-3600;
 	// parcourir le tableau et virer les vieux
 	foreach ($edition as $objet => $data) {
-		if (!is_array($data))
-			unset ($edition[$objet]); // vieille version
-		else foreach ($data as $id => $tab) {
+		if (!is_array($data)) {
+					unset ($edition[$objet]);
+		}
+		// vieille version
+		else {
+		    foreach ($data as $id => $tab) {
 			if (!is_array($tab))
-			  unset ($edition[$objet][$tab]); // vieille version
-			else foreach ($tab as $n => $duo) {
+			  unset ($edition[$objet][$tab]);
+		}
+		// vieille version
+			else {
+			    foreach ($tab as $n => $duo) {
 				if (current($duo) < $bon_pour_le_service) {
 					unset($edition[$objet][$id][$n]);
+			}
 					$changed = true;
 				}
 			}
-			if (!$edition[$objet][$id])
-				unset($edition[$objet][$id]);
+			if (!$edition[$objet][$id]) {
+							unset($edition[$objet][$id]);
+			}
+		}
+		if (!$edition[$objet]) {
+					unset($edition[$objet]);
 		}
-		if (!$edition[$objet])
-			unset($edition[$objet]);
 	}
 
-	if ($changed)
-		ecrire_tableau_edition($edition);
+	if ($changed) {
+			ecrire_tableau_edition($edition);
+	}
 
 	return $edition;
 }
@@ -81,16 +95,19 @@ 
 function signale_edition ($id, $auteur, $type='article') {
 	include_spip('base/objets');
 	include_spip('inc/filtres');
-	if (objet_info($type,'editable')!=='oui')
-		return;
+	if (objet_info($type,'editable')!=='oui') {
+			return;
+	}
 
 	$edition = lire_tableau_edition();
-	if (isset($auteur['id_auteur']) and $id_a = $auteur['id_auteur'])
-		$nom = $auteur['nom'];
-	else
-		$nom = $id_a = $GLOBALS['ip'];
-	if (!is_array($edition[$type][$id]))
-		$edition[$type][$id] = array();
+	if (isset($auteur['id_auteur']) and $id_a = $auteur['id_auteur']) {
+			$nom = $auteur['nom'];
+	} else {
+			$nom = $id_a = $GLOBALS['ip'];
+	}
+	if (!is_array($edition[$type][$id])) {
+			$edition[$type][$id] = array();
+	}
 	$edition[$type][$id][$id_a][$nom] = time();
 	ecrire_tableau_edition($edition);
 }
@@ -135,8 +152,8 @@ 
 	$edition = lire_tableau_edition();
 	$objets_ouverts = array();
 
-	foreach ($edition as $objet => $data)
-		foreach ($data as $id => $auteurs)
+	foreach ($edition as $objet => $data) {
+			foreach ($data as $id => $auteurs)
 		{
 			if (isset($auteurs[$id_auteur])
 			AND is_array($auteurs[$id_auteur]) // precaution
@@ -145,6 +162,7 @@ 
 					'objet'=>$objet,
 					'id_objet' => $id,
 				);
+	}
 			}
 		}
 	return $objets_ouverts;

ecrire/inc/envoyer_mail.php

Doc Comments +3 added lines

@@ -142,6 +142,9 @@
 	return @mail($destinataire, $sujet, $texte, $headers);
 }
 
+/**
+ * @param string $to
+ */
 function mail_normaliser_headers($headers, $from, $to, $texte, $parts="")
 {
 	$charset = $GLOBALS['meta']['charset'];

Indentation +126 added lines, -126 removed lines

@@ -17,27 +17,27 @@ 
 
 // http://doc.spip.org/@nettoyer_titre_email
 function nettoyer_titre_email($titre) {
-	return str_replace("\n", ' ', textebrut(corriger_typo($titre)));
+    return str_replace("\n", ' ', textebrut(corriger_typo($titre)));
 }
 
 // http://doc.spip.org/@nettoyer_caracteres_mail
 function nettoyer_caracteres_mail($t) {
 
-	$t = filtrer_entites($t);
+    $t = filtrer_entites($t);
 
-	if ($GLOBALS['meta']['charset'] <> 'utf-8') {
-		$t = str_replace(
-			array("&#8217;","&#8220;","&#8221;"),
-			array("'",      '"',      '"'),
-		$t);
-	}
+    if ($GLOBALS['meta']['charset'] <> 'utf-8') {
+        $t = str_replace(
+            array("&#8217;","&#8220;","&#8221;"),
+            array("'",      '"',      '"'),
+        $t);
+    }
 
-	$t = str_replace(
-		array("&mdash;", "&endash;"),
-		array("--","-" ),
-	$t);
+    $t = str_replace(
+        array("&mdash;", "&endash;"),
+        array("--","-" ),
+    $t);
 
-	return $t;
+    return $t;
 }
 
 /**
@@ -71,122 +71,122 @@ 
  */
 function inc_envoyer_mail_dist($destinataire, $sujet, $corps, $from = "", $headers = "") {
 
-	if (!email_valide($destinataire)) return false;
-	if ($destinataire == _T('info_mail_fournisseur')) return false; // tres fort
-
-	// Fournir si possible un Message-Id: conforme au RFC1036,
-	// sinon SpamAssassin denoncera un MSGID_FROM_MTA_HEADER
-
-	$email_envoi = $GLOBALS['meta']["email_envoi"];
-	if (!email_valide($email_envoi)) {
-		spip_log("Meta email_envoi invalide. Le mail sera probablement vu comme spam.");
-		$email_envoi = $destinataire;
-	}
-
-	if (is_array($corps)){
-		$texte = $corps['texte'];
-		$from = (isset($corps['from'])?$corps['from']:$from);
-		$headers = (isset($corps['headers'])?$corps['headers']:$headers);
-		if (is_array($headers))
-			$headers = implode("\n",$headers);
-		$parts = "";
-		if ($corps['pieces_jointes'] AND function_exists('mail_embarquer_pieces_jointes'))
-			$parts = mail_embarquer_pieces_jointes($corps['pieces_jointes']);
-	} else
-		$texte = $corps;
-
-	if (!$from) $from = $email_envoi;
-
-	// ceci est la RegExp NO_REAL_NAME faisant hurler SpamAssassin
-	if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from))
-		$from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';
-
-	// nettoyer les &eacute; &#8217, &emdash; etc...
-	// les 'cliquer ici' etc sont a eviter;  voir:
-	// http://mta.org.ua/spamassassin-2.55/stuff/wiki.CustomRulesets/20050914/rules/french_rules.cf
-	$texte = nettoyer_caracteres_mail($texte);
-	$sujet = nettoyer_caracteres_mail($sujet);
-
-	// encoder le sujet si possible selon la RFC
-	if (init_mb_string()) {
-		# un bug de mb_string casse mb_encode_mimeheader si l'encoding interne
-		# est UTF-8 et le charset iso-8859-1 (constate php5-mac ; php4.3-debian)
-		$charset = $GLOBALS['meta']['charset'];
-		mb_internal_encoding($charset);
-		$sujet = mb_encode_mimeheader($sujet, $charset, 'Q', "\n");
-		mb_internal_encoding('utf-8');
-	}
-
-	if (function_exists('wordwrap') && (preg_match(',multipart/mixed,',$headers) == 0))
-		$texte = wordwrap($texte);
-
-	list($headers, $texte) = mail_normaliser_headers($headers, $from, $destinataire, $texte, $parts);
-
-	if (_OS_SERVEUR == 'windows') {
-		$texte = preg_replace ("@\r*\n@","\r\n", $texte);
-		$headers = preg_replace ("@\r*\n@","\r\n", $headers);
-		$sujet = preg_replace ("@\r*\n@","\r\n", $sujet);
-	}
-
-	spip_log("mail $destinataire\n$sujet\n$headers",'mails');
-	// mode TEST : forcer l'email
-	if (defined('_TEST_EMAIL_DEST')) {
-		if (!_TEST_EMAIL_DEST)
-			return false;
-		else {
-			$texte = "Dest : $destinataire\r\n" . $texte;
-			$destinataire = _TEST_EMAIL_DEST;
-		}
-	}
-
-	return @mail($destinataire, $sujet, $texte, $headers);
+    if (!email_valide($destinataire)) return false;
+    if ($destinataire == _T('info_mail_fournisseur')) return false; // tres fort
+
+    // Fournir si possible un Message-Id: conforme au RFC1036,
+    // sinon SpamAssassin denoncera un MSGID_FROM_MTA_HEADER
+
+    $email_envoi = $GLOBALS['meta']["email_envoi"];
+    if (!email_valide($email_envoi)) {
+        spip_log("Meta email_envoi invalide. Le mail sera probablement vu comme spam.");
+        $email_envoi = $destinataire;
+    }
+
+    if (is_array($corps)){
+        $texte = $corps['texte'];
+        $from = (isset($corps['from'])?$corps['from']:$from);
+        $headers = (isset($corps['headers'])?$corps['headers']:$headers);
+        if (is_array($headers))
+            $headers = implode("\n",$headers);
+        $parts = "";
+        if ($corps['pieces_jointes'] AND function_exists('mail_embarquer_pieces_jointes'))
+            $parts = mail_embarquer_pieces_jointes($corps['pieces_jointes']);
+    } else
+        $texte = $corps;
+
+    if (!$from) $from = $email_envoi;
+
+    // ceci est la RegExp NO_REAL_NAME faisant hurler SpamAssassin
+    if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from))
+        $from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';
+
+    // nettoyer les &eacute; &#8217, &emdash; etc...
+    // les 'cliquer ici' etc sont a eviter;  voir:
+    // http://mta.org.ua/spamassassin-2.55/stuff/wiki.CustomRulesets/20050914/rules/french_rules.cf
+    $texte = nettoyer_caracteres_mail($texte);
+    $sujet = nettoyer_caracteres_mail($sujet);
+
+    // encoder le sujet si possible selon la RFC
+    if (init_mb_string()) {
+        # un bug de mb_string casse mb_encode_mimeheader si l'encoding interne
+        # est UTF-8 et le charset iso-8859-1 (constate php5-mac ; php4.3-debian)
+        $charset = $GLOBALS['meta']['charset'];
+        mb_internal_encoding($charset);
+        $sujet = mb_encode_mimeheader($sujet, $charset, 'Q', "\n");
+        mb_internal_encoding('utf-8');
+    }
+
+    if (function_exists('wordwrap') && (preg_match(',multipart/mixed,',$headers) == 0))
+        $texte = wordwrap($texte);
+
+    list($headers, $texte) = mail_normaliser_headers($headers, $from, $destinataire, $texte, $parts);
+
+    if (_OS_SERVEUR == 'windows') {
+        $texte = preg_replace ("@\r*\n@","\r\n", $texte);
+        $headers = preg_replace ("@\r*\n@","\r\n", $headers);
+        $sujet = preg_replace ("@\r*\n@","\r\n", $sujet);
+    }
+
+    spip_log("mail $destinataire\n$sujet\n$headers",'mails');
+    // mode TEST : forcer l'email
+    if (defined('_TEST_EMAIL_DEST')) {
+        if (!_TEST_EMAIL_DEST)
+            return false;
+        else {
+            $texte = "Dest : $destinataire\r\n" . $texte;
+            $destinataire = _TEST_EMAIL_DEST;
+        }
+    }
+
+    return @mail($destinataire, $sujet, $texte, $headers);
 }
 
 function mail_normaliser_headers($headers, $from, $to, $texte, $parts="")
 {
-	$charset = $GLOBALS['meta']['charset'];
-
-	// Ajouter le Content-Type et consort s'il n'y est pas deja
-	if (strpos($headers, "Content-Type: ") === false)
-		$type =
-		"Content-Type: text/plain;charset=\"$charset\";\n".
-		"Content-Transfer-Encoding: 8bit\n";
-	else $type = '';
-
-	// calculer un identifiant unique
-	preg_match('/@\S+/', $from, $domain);
-	$uniq = rand() . '_' . md5($to . $texte) . $domain[0];
-
-	// Si multi-part, s'en servir comme borne ...
-	if ($parts) {
-		$texte = "--$uniq\n$type\n" . $texte ."\n";
-		foreach ($parts as $part) {
-			$n = strlen($part[1]) . ($part[0] ? "\n" : '');
-			$e = join("\n", $part[0]);
-			$texte .= "\n--$uniq\nContent-Length: $n$e\n\n" . $part[1];
-		}
-		$texte .= "\n\n--$uniq--\n";
-		// Si boundary n'est pas entre guillemets,
-		// elle est comprise mais le charset est ignoree !
-		$type = "Content-Type: multipart/mixed; boundary=\"$uniq\"\n";
-	}
-
-	// .. et s'en servir pour plaire a SpamAssassin
-
-	$mid = 'Message-Id: <' . $uniq . ">";
-
-	// indispensable pour les sites qui collent d'office From: serveur-http
-	// sauf si deja mis par l'envoyeur
-	$rep = (strpos($headers,"Reply-To:")!==FALSE) ? '' : "Reply-To: $from\n";
-
-	// Nettoyer les en-tetes envoyees
-	// Ajouter le \n final
-	if (strlen($headers = trim($headers))) $headers .= "\n";
-
-	// Et mentionner l'indeboulonable nomenclature ratee 
-
-	$headers .= "From: $from\n$type$rep$mid\nMIME-Version: 1.0\n";
-
-	return array($headers, $texte);
+    $charset = $GLOBALS['meta']['charset'];
+
+    // Ajouter le Content-Type et consort s'il n'y est pas deja
+    if (strpos($headers, "Content-Type: ") === false)
+        $type =
+        "Content-Type: text/plain;charset=\"$charset\";\n".
+        "Content-Transfer-Encoding: 8bit\n";
+    else $type = '';
+
+    // calculer un identifiant unique
+    preg_match('/@\S+/', $from, $domain);
+    $uniq = rand() . '_' . md5($to . $texte) . $domain[0];
+
+    // Si multi-part, s'en servir comme borne ...
+    if ($parts) {
+        $texte = "--$uniq\n$type\n" . $texte ."\n";
+        foreach ($parts as $part) {
+            $n = strlen($part[1]) . ($part[0] ? "\n" : '');
+            $e = join("\n", $part[0]);
+            $texte .= "\n--$uniq\nContent-Length: $n$e\n\n" . $part[1];
+        }
+        $texte .= "\n\n--$uniq--\n";
+        // Si boundary n'est pas entre guillemets,
+        // elle est comprise mais le charset est ignoree !
+        $type = "Content-Type: multipart/mixed; boundary=\"$uniq\"\n";
+    }
+
+    // .. et s'en servir pour plaire a SpamAssassin
+
+    $mid = 'Message-Id: <' . $uniq . ">";
+
+    // indispensable pour les sites qui collent d'office From: serveur-http
+    // sauf si deja mis par l'envoyeur
+    $rep = (strpos($headers,"Reply-To:")!==FALSE) ? '' : "Reply-To: $from\n";
+
+    // Nettoyer les en-tetes envoyees
+    // Ajouter le \n final
+    if (strlen($headers = trim($headers))) $headers .= "\n";
+
+    // Et mentionner l'indeboulonable nomenclature ratee 
+
+    $headers .= "From: $from\n$type$rep$mid\nMIME-Version: 1.0\n";
+
+    return array($headers, $texte);
 }
 ?>

Spacing +21 added lines, -21 removed lines

@@ -27,14 +27,14 @@ 
 
 	if ($GLOBALS['meta']['charset'] <> 'utf-8') {
 		$t = str_replace(
-			array("&#8217;","&#8220;","&#8221;"),
-			array("'",      '"',      '"'),
+			array("&#8217;", "&#8220;", "&#8221;"),
+			array("'", '"', '"'),
 		$t);
 	}
 
 	$t = str_replace(
 		array("&mdash;", "&endash;"),
-		array("--","-" ),
+		array("--", "-"),
 	$t);
 
 	return $t;
@@ -83,12 +83,12 @@ 
 		$email_envoi = $destinataire;
 	}
 
-	if (is_array($corps)){
+	if (is_array($corps)) {
 		$texte = $corps['texte'];
-		$from = (isset($corps['from'])?$corps['from']:$from);
-		$headers = (isset($corps['headers'])?$corps['headers']:$headers);
+		$from = (isset($corps['from']) ? $corps['from'] : $from);
+		$headers = (isset($corps['headers']) ? $corps['headers'] : $headers);
 		if (is_array($headers))
-			$headers = implode("\n",$headers);
+			$headers = implode("\n", $headers);
 		$parts = "";
 		if ($corps['pieces_jointes'] AND function_exists('mail_embarquer_pieces_jointes'))
 			$parts = mail_embarquer_pieces_jointes($corps['pieces_jointes']);
@@ -99,7 +99,7 @@ 
 
 	// ceci est la RegExp NO_REAL_NAME faisant hurler SpamAssassin
 	if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from))
-		$from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';
+		$from .= ' ('.str_replace(')', '', translitteration(str_replace('@', ' at ', $from))).')';
 
 	// nettoyer les &eacute; &#8217, &emdash; etc...
 	// les 'cliquer ici' etc sont a eviter;  voir:
@@ -117,24 +117,24 @@ 
 		mb_internal_encoding('utf-8');
 	}
 
-	if (function_exists('wordwrap') && (preg_match(',multipart/mixed,',$headers) == 0))
+	if (function_exists('wordwrap') && (preg_match(',multipart/mixed,', $headers) == 0))
 		$texte = wordwrap($texte);
 
 	list($headers, $texte) = mail_normaliser_headers($headers, $from, $destinataire, $texte, $parts);
 
 	if (_OS_SERVEUR == 'windows') {
-		$texte = preg_replace ("@\r*\n@","\r\n", $texte);
-		$headers = preg_replace ("@\r*\n@","\r\n", $headers);
-		$sujet = preg_replace ("@\r*\n@","\r\n", $sujet);
+		$texte = preg_replace("@\r*\n@", "\r\n", $texte);
+		$headers = preg_replace("@\r*\n@", "\r\n", $headers);
+		$sujet = preg_replace("@\r*\n@", "\r\n", $sujet);
 	}
 
-	spip_log("mail $destinataire\n$sujet\n$headers",'mails');
+	spip_log("mail $destinataire\n$sujet\n$headers", 'mails');
 	// mode TEST : forcer l'email
 	if (defined('_TEST_EMAIL_DEST')) {
 		if (!_TEST_EMAIL_DEST)
 			return false;
 		else {
-			$texte = "Dest : $destinataire\r\n" . $texte;
+			$texte = "Dest : $destinataire\r\n".$texte;
 			$destinataire = _TEST_EMAIL_DEST;
 		}
 	}
@@ -142,7 +142,7 @@ 
 	return @mail($destinataire, $sujet, $texte, $headers);
 }
 
-function mail_normaliser_headers($headers, $from, $to, $texte, $parts="")
+function mail_normaliser_headers($headers, $from, $to, $texte, $parts = "")
 {
 	$charset = $GLOBALS['meta']['charset'];
 
@@ -155,15 +155,15 @@ 
 
 	// calculer un identifiant unique
 	preg_match('/@\S+/', $from, $domain);
-	$uniq = rand() . '_' . md5($to . $texte) . $domain[0];
+	$uniq = rand().'_'.md5($to.$texte).$domain[0];
 
 	// Si multi-part, s'en servir comme borne ...
 	if ($parts) {
-		$texte = "--$uniq\n$type\n" . $texte ."\n";
+		$texte = "--$uniq\n$type\n".$texte."\n";
 		foreach ($parts as $part) {
-			$n = strlen($part[1]) . ($part[0] ? "\n" : '');
+			$n = strlen($part[1]).($part[0] ? "\n" : '');
 			$e = join("\n", $part[0]);
-			$texte .= "\n--$uniq\nContent-Length: $n$e\n\n" . $part[1];
+			$texte .= "\n--$uniq\nContent-Length: $n$e\n\n".$part[1];
 		}
 		$texte .= "\n\n--$uniq--\n";
 		// Si boundary n'est pas entre guillemets,
@@ -173,11 +173,11 @@ 
 
 	// .. et s'en servir pour plaire a SpamAssassin
 
-	$mid = 'Message-Id: <' . $uniq . ">";
+	$mid = 'Message-Id: <'.$uniq.">";
 
 	// indispensable pour les sites qui collent d'office From: serveur-http
 	// sauf si deja mis par l'envoyeur
-	$rep = (strpos($headers,"Reply-To:")!==FALSE) ? '' : "Reply-To: $from\n";
+	$rep = (strpos($headers, "Reply-To:") !== FALSE) ? '' : "Reply-To: $from\n";
 
 	// Nettoyer les en-tetes envoyees
 	// Ajouter le \n final

Braces +39 added lines, -21 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/charsets');
 include_spip('inc/texte');
@@ -71,8 +73,13 @@ 
  */
 function inc_envoyer_mail_dist($destinataire, $sujet, $corps, $from = "", $headers = "") {
 
-	if (!email_valide($destinataire)) return false;
-	if ($destinataire == _T('info_mail_fournisseur')) return false; // tres fort
+	if (!email_valide($destinataire)) {
+	    return false;
+	}
+	if ($destinataire == _T('info_mail_fournisseur')) {
+	    return false;
+	}
+	// tres fort
 
 	// Fournir si possible un Message-Id: conforme au RFC1036,
 	// sinon SpamAssassin denoncera un MSGID_FROM_MTA_HEADER
@@ -87,19 +94,25 @@ 
 		$texte = $corps['texte'];
 		$from = (isset($corps['from'])?$corps['from']:$from);
 		$headers = (isset($corps['headers'])?$corps['headers']:$headers);
-		if (is_array($headers))
-			$headers = implode("\n",$headers);
+		if (is_array($headers)) {
+					$headers = implode("\n",$headers);
+		}
 		$parts = "";
-		if ($corps['pieces_jointes'] AND function_exists('mail_embarquer_pieces_jointes'))
-			$parts = mail_embarquer_pieces_jointes($corps['pieces_jointes']);
-	} else
-		$texte = $corps;
+		if ($corps['pieces_jointes'] AND function_exists('mail_embarquer_pieces_jointes')) {
+					$parts = mail_embarquer_pieces_jointes($corps['pieces_jointes']);
+		}
+	} else {
+			$texte = $corps;
+	}
 
-	if (!$from) $from = $email_envoi;
+	if (!$from) {
+	    $from = $email_envoi;
+	}
 
 	// ceci est la RegExp NO_REAL_NAME faisant hurler SpamAssassin
-	if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from))
-		$from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';
+	if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from)) {
+			$from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';
+	}
 
 	// nettoyer les &eacute; &#8217, &emdash; etc...
 	// les 'cliquer ici' etc sont a eviter;  voir:
@@ -117,8 +130,9 @@ 
 		mb_internal_encoding('utf-8');
 	}
 
-	if (function_exists('wordwrap') && (preg_match(',multipart/mixed,',$headers) == 0))
-		$texte = wordwrap($texte);
+	if (function_exists('wordwrap') && (preg_match(',multipart/mixed,',$headers) == 0)) {
+			$texte = wordwrap($texte);
+	}
 
 	list($headers, $texte) = mail_normaliser_headers($headers, $from, $destinataire, $texte, $parts);
 
@@ -131,9 +145,9 @@ 
 	spip_log("mail $destinataire\n$sujet\n$headers",'mails');
 	// mode TEST : forcer l'email
 	if (defined('_TEST_EMAIL_DEST')) {
-		if (!_TEST_EMAIL_DEST)
-			return false;
-		else {
+		if (!_TEST_EMAIL_DEST) {
+					return false;
+		} else {
 			$texte = "Dest : $destinataire\r\n" . $texte;
 			$destinataire = _TEST_EMAIL_DEST;
 		}
@@ -147,11 +161,13 @@ 
 	$charset = $GLOBALS['meta']['charset'];
 
 	// Ajouter le Content-Type et consort s'il n'y est pas deja
-	if (strpos($headers, "Content-Type: ") === false)
-		$type =
+	if (strpos($headers, "Content-Type: ") === false) {
+			$type =
 		"Content-Type: text/plain;charset=\"$charset\";\n".
 		"Content-Transfer-Encoding: 8bit\n";
-	else $type = '';
+	} else {
+	    $type = '';
+	}
 
 	// calculer un identifiant unique
 	preg_match('/@\S+/', $from, $domain);
@@ -181,7 +197,9 @@ 
 
 	// Nettoyer les en-tetes envoyees
 	// Ajouter le \n final
-	if (strlen($headers = trim($headers))) $headers .= "\n";
+	if (strlen($headers = trim($headers))) {
+	    $headers .= "\n";
+	}
 
 	// Et mentionner l'indeboulonable nomenclature ratee 
 

ecrire/inc/idna_convert.class.php

Doc Comments +5 added lines, -5 removed lines

@@ -88,7 +88,7 @@ 
      * the constructor
      *
      * @param array $options
-     * @return boolean
+     * @return boolean|null
      * @since 0.5.2
      */
     public function __construct($options = false)
@@ -393,7 +393,7 @@ 
     /**
      * Use this method to get the last error ocurred
      * @param    void
-     * @return   string   The last error, that occured
+     * @return   boolean   The last error, that occured
      */
     public function get_last_error()
     {
@@ -403,7 +403,7 @@ 
     /**
      * The actual decoding algorithm
      * @param string
-     * @return mixed
+     * @return false|string
      */
     protected function _decode($encoded)
     {
@@ -464,7 +464,7 @@ 
     /**
      * The actual encoding algorithm
      * @param  string
-     * @return mixed
+     * @return false|string
      */
     protected function _encode($decoded)
     {
@@ -699,7 +699,7 @@ 
      * Decomposes a Hangul syllable
      * (see http://www.unicode.org/unicode/reports/tr15/#Hangul
      * @param    integer  32bit UCS4 code point
-     * @return   array    Either Hangul Syllable decomposed or original 32bit value as one value array
+     * @return   integer[]    Either Hangul Syllable decomposed or original 32bit value as one value array
      */
     protected function _hangul_decompose($char)
     {

Spacing +43 added lines, -46 removed lines

@@ -73,16 +73,16 @@ 
     protected $_lcount = 19;
     protected $_vcount = 21;
     protected $_tcount = 28;
-    protected $_ncount = 588;   // _vcount * _tcount
+    protected $_ncount = 588; // _vcount * _tcount
     protected $_scount = 11172; // _lcount * _tcount * _vcount
     protected $_error = false;
     protected static $_mb_string_overload = null;
     // See {@link set_paramter()} for details of how to change the following
     // settings from within your script / application
-    protected $_api_encoding = 'utf8';   // Default input charset is UTF-8
-    protected $_allow_overlong = false;  // Overlong UTF-8 encodings are forbidden
-    protected $_strict_mode = false;     // Behave strict or not
-    protected $_idn_version = 2003;      // Can be either 2003 (old, default) or 2008
+    protected $_api_encoding = 'utf8'; // Default input charset is UTF-8
+    protected $_allow_overlong = false; // Overlong UTF-8 encodings are forbidden
+    protected $_strict_mode = false; // Behave strict or not
+    protected $_idn_version = 2003; // Can be either 2003 (old, default) or 2008
 
     /**
      * the constructor
@@ -140,7 +140,7 @@ 
                             $this->_api_encoding = $v;
                             break;
                         default:
-                            $this->_error('Set Parameter: Unknown parameter ' . $v . ' for option ' . $k);
+                            $this->_error('Set Parameter: Unknown parameter '.$v.' for option '.$k);
                             return false;
                     }
                     break;
@@ -154,7 +154,7 @@ 
                     if (in_array($v, array('2003', '2008'))) {
                         $this->_idn_version = $v;
                     } else {
-                        $this->_error('Set Parameter: Unknown parameter ' . $v . ' for option ' . $k);
+                        $this->_error('Set Parameter: Unknown parameter '.$v.' for option '.$k);
                     }
                     break;
                 case 'encode_german_sz': // Deprecated
@@ -165,7 +165,7 @@ 
                     }
                     break;
                 default:
-                    $this->_error('Set Parameter: Unknown option ' . $k);
+                    $this->_error('Set Parameter: Unknown option '.$k);
                     return false;
             }
         }
@@ -188,7 +188,7 @@ 
                 case 'ucs4_array':
                     break;
                 default:
-                    $this->_error('Unknown encoding ' . $one_time_encoding);
+                    $this->_error('Unknown encoding '.$one_time_encoding);
                     return false;
             }
         }
@@ -206,7 +206,7 @@ 
             list ($email_pref, $input) = explode('@', $input, 2);
             $arr = explode('.', $input);
             foreach ($arr as $k => $v) {
-                if (preg_match('!^' . preg_quote($this->_punycode_prefix, '!') . '!', $v)) {
+                if (preg_match('!^'.preg_quote($this->_punycode_prefix, '!').'!', $v)) {
                     $conv = $this->_decode($v);
                     if ($conv) {
                         $arr[$k] = $conv;
@@ -216,7 +216,7 @@ 
             $input = join('.', $arr);
             $arr = explode('.', $email_pref);
             foreach ($arr as $k => $v) {
-                if (preg_match('!^' . preg_quote($this->_punycode_prefix, '!') . '!', $v)) {
+                if (preg_match('!^'.preg_quote($this->_punycode_prefix, '!').'!', $v)) {
                     $conv = $this->_decode($v);
                     if ($conv) {
                         $arr[$k] = $conv;
@@ -224,7 +224,7 @@ 
                 }
             }
             $email_pref = join('.', $arr);
-            $return = $email_pref . '@' . $input;
+            $return = $email_pref.'@'.$input;
         } elseif (preg_match('![:\./]!', $input)) { // Or a complete domain name (with or without paths / parameters)
             // No no in strict mode
             if ($this->_strict_mode) {
@@ -241,13 +241,13 @@ 
                     }
                 }
                 $parsed['host'] = join('.', $arr);
-                $return = (empty($parsed['scheme']) ? '' : $parsed['scheme'] . (strtolower($parsed['scheme']) == 'mailto' ? ':' : '://')).
-                        (empty($parsed['user']) ? '' : $parsed['user'] . (empty($parsed['pass']) ? '' : ':' . $parsed['pass']) . '@').
+                $return = (empty($parsed['scheme']) ? '' : $parsed['scheme'].(strtolower($parsed['scheme']) == 'mailto' ? ':' : '://')).
+                        (empty($parsed['user']) ? '' : $parsed['user'].(empty($parsed['pass']) ? '' : ':'.$parsed['pass']).'@').
                         $parsed['host'].
-                        (empty($parsed['port']) ? '' : ':' . $parsed['port']).
+                        (empty($parsed['port']) ? '' : ':'.$parsed['port']).
                         (empty($parsed['path']) ? '' : $parsed['path']).
-                        (empty($parsed['query']) ? '' : '?' . $parsed['query']).
-                        (empty($parsed['fragment']) ? '' : '#' . $parsed['fragment']);
+                        (empty($parsed['query']) ? '' : '?'.$parsed['query']).
+                        (empty($parsed['fragment']) ? '' : '#'.$parsed['fragment']);
             } else { // parse_url seems to have failed, try without it
                 $arr = explode('.', $input);
                 foreach ($arr as $k => $v) {
@@ -265,8 +265,8 @@ 
         // The output is UTF-8 by default, other output formats need conversion here
         // If one time encoding is given, use this, else the objects property
         switch (($one_time_encoding) ? $one_time_encoding : $this->_api_encoding) {
-            case 'utf8':        return $return; // break;
-            case 'ucs4_string': return $this->_ucs4_to_ucs4_string($this->_utf8_to_ucs4($return));  // break;
+            case 'utf8' : return $return; // break;
+            case 'ucs4_string': return $this->_ucs4_to_ucs4_string($this->_utf8_to_ucs4($return)); // break;
             case 'ucs4_array':  return $this->_utf8_to_ucs4($return); // break;
             default:            $this->_error('Unsupported output format'); return false;
         }
@@ -283,15 +283,14 @@ 
         // Forcing conversion of input to UCS4 array
         // If one time encoding is given, use this, else the objects property
         switch ($one_time_encoding ? $one_time_encoding : $this->_api_encoding) {
-            case 'utf8':
-                $decoded = $this->_utf8_to_ucs4($decoded);
+            case 'utf8' : $decoded = $this->_utf8_to_ucs4($decoded);
                 break;
             case 'ucs4_string':
                 $decoded = $this->_ucs4_string_to_ucs4($decoded);
             case 'ucs4_array':
                 break;
             default:
-                $this->_error('Unsupported input format: ' . ($one_time_encoding ? $one_time_encoding : $this->_api_encoding));
+                $this->_error('Unsupported input format: '.($one_time_encoding ? $one_time_encoding : $this->_api_encoding));
                 return false;
         }
 
@@ -380,13 +379,13 @@ 
             }
         }
         $parsed['host'] = join('.', $arr);
-        $return = (empty($parsed['scheme']) ? '' : $parsed['scheme'] . (strtolower($parsed['scheme']) == 'mailto' ? ':' : '://')).
-                (empty($parsed['user']) ? '' : $parsed['user'] . (empty($parsed['pass']) ? '' : ':' . $parsed['pass']) . '@').
+        $return = (empty($parsed['scheme']) ? '' : $parsed['scheme'].(strtolower($parsed['scheme']) == 'mailto' ? ':' : '://')).
+                (empty($parsed['user']) ? '' : $parsed['user'].(empty($parsed['pass']) ? '' : ':'.$parsed['pass']).'@').
                 $parsed['host'].
-                (empty($parsed['port']) ? '' : ':' . $parsed['port']).
+                (empty($parsed['port']) ? '' : ':'.$parsed['port']).
                 (empty($parsed['path']) ? '' : $parsed['path']).
-                (empty($parsed['query']) ? '' : '?' . $parsed['query']).
-                (empty($parsed['fragment']) ? '' : '#' . $parsed['fragment']);
+                (empty($parsed['query']) ? '' : '?'.$parsed['query']).
+                (empty($parsed['fragment']) ? '' : '#'.$parsed['fragment']);
         return $return;
     }
 
@@ -409,11 +408,11 @@ 
     {
         $decoded = array();
         // find the Punycode prefix
-        if (!preg_match('!^' . preg_quote($this->_punycode_prefix, '!') . '!', $encoded)) {
+        if (!preg_match('!^'.preg_quote($this->_punycode_prefix, '!').'!', $encoded)) {
             $this->_error('This is not a punycode string');
             return false;
         }
-        $encode_test = preg_replace('!^' . preg_quote($this->_punycode_prefix, '!') . '!', '', $encoded);
+        $encode_test = preg_replace('!^'.preg_quote($this->_punycode_prefix, '!').'!', '', $encoded);
         // If nothing left after removing the prefix, it is hopeless
         if (!$encode_test) {
             $this->_error('The given encoded string was empty');
@@ -439,8 +438,7 @@ 
             for ($old_idx = $idx, $w = 1, $k = $this->_base; 1; $k += $this->_base) {
                 $digit = $this->_decode_digit($encoded{$enco_idx++});
                 $idx += $digit * $w;
-                $t = ($k <= $bias) ? $this->_tmin :
-                        (($k >= $bias + $this->_tmax) ? $this->_tmax : ($k - $bias));
+                $t = ($k <= $bias) ? $this->_tmin : (($k >= $bias + $this->_tmax) ? $this->_tmax : ($k - $bias));
                 if ($digit < $t) {
                     break;
                 }
@@ -513,7 +511,7 @@ 
             return $encoded; // All codepoints were basic ones
         }
         // Start with the prefix; copy it to output
-        $encoded = $this->_punycode_prefix . $encoded;
+        $encoded = $this->_punycode_prefix.$encoded;
         // If we have basic code points in output, add an hyphen to the end
         if ($codecount) {
             $encoded .= '-';
@@ -540,8 +538,7 @@ 
                     $delta++;
                 } elseif ($decoded[$i] == $cur_code) {
                     for ($q = $delta, $k = $this->_base; 1; $k += $this->_base) {
-                        $t = ($k <= $bias) ? $this->_tmin :
-                                (($k >= $bias + $this->_tmax) ? $this->_tmax : $k - $bias);
+                        $t = ($k <= $bias) ? $this->_tmin : (($k >= $bias + $this->_tmax) ? $this->_tmax : $k - $bias);
                         if ($q < $t) {
                             break;
                         }
@@ -628,12 +625,12 @@ 
             }
             // Try to find prohibited input
             if (in_array($v, self::$NP['prohibit']) || in_array($v, self::$NP['general_prohibited'])) {
-                $this->_error('NAMEPREP: Prohibited input U+' . sprintf('%08X', $v));
+                $this->_error('NAMEPREP: Prohibited input U+'.sprintf('%08X', $v));
                 return false;
             }
             foreach (self::$NP['prohibit_ranges'] as $range) {
                 if ($range[0] <= $v && $v <= $range[1]) {
-                    $this->_error('NAMEPREP: Prohibited input U+' . sprintf('%08X', $v));
+                    $this->_error('NAMEPREP: Prohibited input U+'.sprintf('%08X', $v));
                     return false;
                 }
             }
@@ -869,7 +866,7 @@ 
                 $output[$out_len] = $v;
                 ++$out_len;
                 if ('add' == $mode) {
-                    $this->_error('Conversion from UTF-8 to UCS-4 failed: malformed input at byte ' . $k);
+                    $this->_error('Conversion from UTF-8 to UCS-4 failed: malformed input at byte '.$k);
                     return false;
                 }
                 continue;
@@ -894,7 +891,7 @@ 
                     $next_byte = 4;
                     $v = ($v - 252) << 30;
                 } else {
-                    $this->_error('This might be UTF-8, but I don\'t understand it at byte ' . $k);
+                    $this->_error('This might be UTF-8, but I don\'t understand it at byte '.$k);
                     return false;
                 }
                 if ('add' == $mode) {
@@ -907,7 +904,7 @@ 
                 if (!$this->_allow_overlong && $test == 'range') {
                     $test = 'none';
                     if (($v < 0xA0 && $start_byte == 0xE0) || ($v < 0x90 && $start_byte == 0xF0) || ($v > 0x8F && $start_byte == 0xF4)) {
-                        $this->_error('Bogus UTF-8 character detected (out of legal range) at byte ' . $k);
+                        $this->_error('Bogus UTF-8 character detected (out of legal range) at byte '.$k);
                         return false;
                     }
                 }
@@ -916,7 +913,7 @@ 
                     $output[($out_len - 1)] += $v;
                     --$next_byte;
                 } else {
-                    $this->_error('Conversion from UTF-8 to UCS-4 failed: malformed input at byte ' . $k);
+                    $this->_error('Conversion from UTF-8 to UCS-4 failed: malformed input at byte '.$k);
                     return false;
                 }
                 if ($next_byte < 0) {
@@ -940,13 +937,13 @@ 
             if ($v < 128) { // 7bit are transferred literally
                 $output .= chr($v);
             } elseif ($v < (1 << 11)) { // 2 bytes
-                $output .= chr(192 + ($v >> 6)) . chr(128 + ($v & 63));
+                $output .= chr(192 + ($v >> 6)).chr(128 + ($v & 63));
             } elseif ($v < (1 << 16)) { // 3 bytes
-                $output .= chr(224 + ($v >> 12)) . chr(128 + (($v >> 6) & 63)) . chr(128 + ($v & 63));
+                $output .= chr(224 + ($v >> 12)).chr(128 + (($v >> 6) & 63)).chr(128 + ($v & 63));
             } elseif ($v < (1 << 21)) { // 4 bytes
-                $output .= chr(240 + ($v >> 18)) . chr(128 + (($v >> 12) & 63)) . chr(128 + (($v >> 6) & 63)) . chr(128 + ($v & 63));
+                $output .= chr(240 + ($v >> 18)).chr(128 + (($v >> 12) & 63)).chr(128 + (($v >> 6) & 63)).chr(128 + ($v & 63));
             } else {
-                $this->_error('Conversion from UCS-4 to UTF-8 failed: malformed input at byte ' . $k);
+                $this->_error('Conversion from UCS-4 to UTF-8 failed: malformed input at byte '.$k);
                 return false;
             }
         }
@@ -965,7 +962,7 @@ 
         // Take array values and split output to 4 bytes per value
         // The bit mask is 255, which reads &11111111
         foreach ($input as $v) {
-            $output .= chr(($v >> 24) & 255) . chr(($v >> 16) & 255) . chr(($v >> 8) & 255) . chr($v & 255);
+            $output .= chr(($v >> 24) & 255).chr(($v >> 16) & 255).chr(($v >> 8) & 255).chr($v & 255);
         }
         return $output;
     }
@@ -995,7 +992,7 @@ 
                 $out_len++;
                 $output[$out_len] = 0;
             }
-            $output[$out_len] += ord($input{$i}) << (8 * (3 - ($i % 4) ) );
+            $output[$out_len] += ord($input{$i}) << (8 * (3 - ($i % 4)));
         }
         return $output;
     }

ecrire/inc/lang.php

Doc Comments +8 added lines, -1 removed lines

@@ -30,7 +30,7 @@ 
  * 
  * @param string $lang
  *     La langue à utiliser
- * @return string|bool
+ * @return false|string
  *     string : La langue qui a été utilisée si trouvée
  *     false : aucune langue ne correspondait à la demande
 **/
@@ -154,6 +154,9 @@ 
 // pour 'changer_lang' (langue de l'article, espace prive), c'est en Ajax
 // 
 // http://doc.spip.org/@menu_langues
+/**
+ * @param string $nom_select
+ */
 function menu_langues($nom_select, $default='') {
 	include_spip('inc/actions');
 
@@ -184,6 +187,10 @@ 
 }
 
 // http://doc.spip.org/@select_langues
+/**
+ * @param string $change
+ * @param string $options
+ */
 function select_langues($nom_select, $change, $options, $label="")
 {
 	static $cpt = 0;

Indentation +210 added lines, -210 removed lines

@@ -35,28 +35,28 @@ 
  *     false : aucune langue ne correspondait à la demande
 **/
 function changer_langue($lang) {
-	global $spip_lang_rtl, $spip_lang_right, $spip_lang_left;
+    global $spip_lang_rtl, $spip_lang_right, $spip_lang_left;
 
-	$liste_langues = ',' . @$GLOBALS['meta']['langues_proposees']
-	. ',' . @$GLOBALS['meta']['langues_multilingue'] . ',';
+    $liste_langues = ',' . @$GLOBALS['meta']['langues_proposees']
+    . ',' . @$GLOBALS['meta']['langues_multilingue'] . ',';
 
-	// Si la langue demandee n'existe pas, on essaie d'autres variantes
-	// Exemple : 'pt-br' => 'pt_br' => 'pt'
-	$lang = str_replace('-', '_', trim($lang));
-	if (!$lang)
-		return false;
+    // Si la langue demandee n'existe pas, on essaie d'autres variantes
+    // Exemple : 'pt-br' => 'pt_br' => 'pt'
+    $lang = str_replace('-', '_', trim($lang));
+    if (!$lang)
+        return false;
 
-	if (strpos($liste_langues,",$lang,")!==false
-	OR ($lang = preg_replace(',_.*,', '', $lang)
-	AND strpos($liste_langues,",$lang,")!==false)) {
+    if (strpos($liste_langues,",$lang,")!==false
+    OR ($lang = preg_replace(',_.*,', '', $lang)
+    AND strpos($liste_langues,",$lang,")!==false)) {
 
-		$spip_lang_rtl =   lang_dir($lang, '', '_rtl');
-		$spip_lang_right = $spip_lang_rtl ? 'left' : 'right';
-		$spip_lang_left =  $spip_lang_rtl ? 'right' : 'left';
+        $spip_lang_rtl =   lang_dir($lang, '', '_rtl');
+        $spip_lang_right = $spip_lang_rtl ? 'left' : 'right';
+        $spip_lang_left =  $spip_lang_rtl ? 'right' : 'left';
 
-		return $GLOBALS['spip_lang'] = $lang;
-	} else
-		return false;
+        return $GLOBALS['spip_lang'] = $lang;
+    } else
+        return false;
 }
 
 //
@@ -69,8 +69,8 @@ 
 // par exemple le francais pour l'espagnol, l'anglais pour l'allemand, etc.
 
 function choisir_traduction ($trads, $lang='') {
-	$k = approcher_langue($trads, $lang);
-	return $k ? $trads[$k] : array_shift($trads);
+    $k = approcher_langue($trads, $lang);
+    return $k ? $trads[$k] : array_shift($trads);
 }
 
 // retourne son 2e argument si c'est un index du premier
@@ -78,25 +78,25 @@ 
 // la langue X etant consideree comme une approche de X_Y
 function approcher_langue ($trads, $lang='') {
 
-	if (!$lang) $lang = $GLOBALS['spip_lang']; 
-
-	if (isset($trads[$lang])) {
-		return $lang;
-	}
-	// cas des langues xx_yy
-	else {
-		$r = explode('_', $lang);
-		if (isset($trads[$r[0]]))
-			return $r[0];
-	}
-	return '';
+    if (!$lang) $lang = $GLOBALS['spip_lang']; 
+
+    if (isset($trads[$lang])) {
+        return $lang;
+    }
+    // cas des langues xx_yy
+    else {
+        $r = explode('_', $lang);
+        if (isset($trads[$r[0]]))
+            return $r[0];
+    }
+    return '';
 }
 
 // http://doc.spip.org/@traduire_nom_langue
 function traduire_nom_langue($lang) {
-	include_spip('inc/lang_liste');
-	include_spip('inc/charsets');
-	return html2unicode(isset($GLOBALS['codes_langues'][$lang]) ? $GLOBALS['codes_langues'][$lang] : $lang);
+    include_spip('inc/lang_liste');
+    include_spip('inc/charsets');
+    return html2unicode(isset($GLOBALS['codes_langues'][$lang]) ? $GLOBALS['codes_langues'][$lang] : $lang);
 }
 
 //
@@ -110,10 +110,10 @@ 
 // C'est utilise par #LANG_DIR, #LANG_LEFT, #LANG_RIGHT.
 // http://doc.spip.org/@lang_dir
 function lang_dir($lang='', $droitier='ltr', $gaucher='rtl') {
-	static $lang_rtl = array('ar', 'fa', 'ku', 'prs', 'ps', 'ur', 'he', 'heb', 'hbo', 'yi');
+    static $lang_rtl = array('ar', 'fa', 'ku', 'prs', 'ps', 'ur', 'he', 'heb', 'hbo', 'yi');
 
-	return in_array(($lang ? $lang : $GLOBALS['spip_lang']), $lang_rtl) ?
-		$gaucher : $droitier;
+    return in_array(($lang ? $lang : $GLOBALS['spip_lang']), $lang_rtl) ?
+        $gaucher : $droitier;
 }
 
 // typo francaise ou anglaise ?
@@ -123,28 +123,28 @@ 
 
 // http://doc.spip.org/@lang_typo
 function lang_typo($lang='') {
-	if (!$lang) {
-		$lang = isset($GLOBALS['lang_objet'])
-			? $GLOBALS['lang_objet']
-			: $GLOBALS['spip_lang'];
-	}
-	if ($lang == 'eo'
-	  OR $lang == 'fr'
-	  OR strncmp($lang, 'fr_', 3)==0
-	  OR $lang == 'cpf')
-		return 'fr';
-	else
-		return 'en';
+    if (!$lang) {
+        $lang = isset($GLOBALS['lang_objet'])
+            ? $GLOBALS['lang_objet']
+            : $GLOBALS['spip_lang'];
+    }
+    if ($lang == 'eo'
+      OR $lang == 'fr'
+      OR strncmp($lang, 'fr_', 3)==0
+      OR $lang == 'cpf')
+        return 'fr';
+    else
+        return 'en';
 }
 
 // gestion de la globale $lang_objet pour que les textes soient affiches
 // avec les memes typo et direction dans l'espace prive que dans le public
 // http://doc.spip.org/@changer_typo
 function changer_typo($lang = '') {
-	if ($lang)
-		$GLOBALS['lang_objet'] = $lang;
-	else
-		unset($GLOBALS['lang_objet']);
+    if ($lang)
+        $GLOBALS['lang_objet'] = $lang;
+    else
+        unset($GLOBALS['lang_objet']);
 }
 
 //
@@ -155,51 +155,51 @@ 
 // 
 // http://doc.spip.org/@menu_langues
 function menu_langues($nom_select, $default='') {
-	include_spip('inc/actions');
-
-	$langues = liste_options_langues($nom_select);
-	$ret = "";
-	if (!count($langues))
-		return '';
-
-	if (!$default) $default = $GLOBALS['spip_lang'];
-	foreach ($langues as $l) {
-		$selected = ($l == $default) ? ' selected=\'selected\'' : '';
-		$ret .= "<option value='$l'$selected>[".$l."] ".traduire_nom_langue($l)."</option>\n";
-	}
-
-	if (!test_espace_prive()) {
-		$cible = self();
-		$base = '';
-	} else {
-		$cible = self();
-		$base = spip_connect() ? 'base' : '';
-	}
-
-	$change = ' onchange="this.parentNode.parentNode.submit()"';
-	return generer_action_auteur('converser',$base, $cible,
-		(select_langues($nom_select, $change, $ret)
-		 . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='". _T('bouton_changer')."' /></div></noscript>"),
-				     " method='post'");
+    include_spip('inc/actions');
+
+    $langues = liste_options_langues($nom_select);
+    $ret = "";
+    if (!count($langues))
+        return '';
+
+    if (!$default) $default = $GLOBALS['spip_lang'];
+    foreach ($langues as $l) {
+        $selected = ($l == $default) ? ' selected=\'selected\'' : '';
+        $ret .= "<option value='$l'$selected>[".$l."] ".traduire_nom_langue($l)."</option>\n";
+    }
+
+    if (!test_espace_prive()) {
+        $cible = self();
+        $base = '';
+    } else {
+        $cible = self();
+        $base = spip_connect() ? 'base' : '';
+    }
+
+    $change = ' onchange="this.parentNode.parentNode.submit()"';
+    return generer_action_auteur('converser',$base, $cible,
+        (select_langues($nom_select, $change, $ret)
+            . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='". _T('bouton_changer')."' /></div></noscript>"),
+                        " method='post'");
 }
 
 // http://doc.spip.org/@select_langues
 function select_langues($nom_select, $change, $options, $label="")
 {
-	static $cpt = 0;
-	$id = "menu_langues" . $cpt++;
-	return
-		"<label for='$id'>".($label?$label:_T('info_langues'))."</label> ".
-		"<select name='$nom_select' id='$id' "
-	  . ((!test_espace_prive()) ?
-	     ("class='forml menu_langues'") :
-	     (($nom_select == 'var_lang_ecrire')  ?
-	      ("class='lang_ecrire'") :
-	      "class='fondl'"))
-	  . $change
-	  . ">\n"
-	  . $options
-	  . "</select>";
+    static $cpt = 0;
+    $id = "menu_langues" . $cpt++;
+    return
+        "<label for='$id'>".($label?$label:_T('info_langues'))."</label> ".
+        "<select name='$nom_select' id='$id' "
+        . ((!test_espace_prive()) ?
+         ("class='forml menu_langues'") :
+         (($nom_select == 'var_lang_ecrire')  ?
+          ("class='lang_ecrire'") :
+          "class='fondl'"))
+        . $change
+        . ">\n"
+        . $options
+        . "</select>";
 }
 
 /**
@@ -220,31 +220,31 @@ 
  */
 function liste_options_langues($nom_select) {
 
-	switch($nom_select) {
-		# #MENU_LANG
-		case 'var_lang':
-		# menu de changement de la langue d'un article
-		# les langues selectionnees dans la configuration "multilinguisme"
-		case 'changer_lang':
-			$langues = explode(',', $GLOBALS['meta']['langues_multilingue']);
-			break;
-	# menu de l'interface (privee, installation et panneau de login)
-	# les langues presentes sous forme de fichiers de langue
-	# on force la relecture du repertoire des langues pour etre synchrone.
-		case 'var_lang_ecrire':
-		default:
-			$GLOBALS['meta']['langues_proposees'] = '';
-			init_langues();
-			$langues = explode(',', $GLOBALS['meta']['langues_proposees']);
-			break;
+    switch($nom_select) {
+        # #MENU_LANG
+        case 'var_lang':
+        # menu de changement de la langue d'un article
+        # les langues selectionnees dans la configuration "multilinguisme"
+        case 'changer_lang':
+            $langues = explode(',', $GLOBALS['meta']['langues_multilingue']);
+            break;
+    # menu de l'interface (privee, installation et panneau de login)
+    # les langues presentes sous forme de fichiers de langue
+    # on force la relecture du repertoire des langues pour etre synchrone.
+        case 'var_lang_ecrire':
+        default:
+            $GLOBALS['meta']['langues_proposees'] = '';
+            init_langues();
+            $langues = explode(',', $GLOBALS['meta']['langues_proposees']);
+            break;
 
 # dernier choix possible : toutes les langues = langues_proposees 
 # + langues_multilingues ; mais, ne sert pas
 #			$langues = explode(',', $GLOBALS['all_langs']);
-	}
-	if (count($langues) <= 1) return array();
-	sort($langues);
-	return $langues;
+    }
+    if (count($langues) <= 1) return array();
+    sort($langues);
+    return $langues;
 }
 
 
@@ -259,35 +259,35 @@ 
  * @return void
 **/
 function verifier_lang_url() {
-	global $spip_lang;
-
-	// quelle langue est demandee ?
-	$lang_demandee = (test_espace_prive()?$GLOBALS['spip_lang']:$GLOBALS['meta']['langue_site']);
-	if (isset($_COOKIE['spip_lang_ecrire']))
-		$lang_demandee = $_COOKIE['spip_lang_ecrire'];
-	if (!test_espace_prive() AND isset($_COOKIE['spip_lang']))
-		$lang_demandee = $_COOKIE['spip_lang'];
-	if (isset($_GET['lang']))
-		$lang_demandee = $_GET['lang'];
-
-	// Renvoyer si besoin (et si la langue demandee existe)
-	if ($spip_lang != $lang_demandee
-	AND changer_langue($lang_demandee)
-	AND $lang_demandee != @$_GET['lang']) {
-		$destination = parametre_url(self(),'lang', $lang_demandee, '&');
-		// ici on a besoin des var_truc
-		foreach ($_GET as $var => $val) {
-			if (!strncmp('var_', $var, 4))
-				$destination = parametre_url($destination, $var, $val, '&');
-		}
-		include_spip('inc/headers');
-		redirige_par_entete($destination);
-	}
-
-	// Subtilite : si la langue demandee par cookie est la bonne
-	// alors on fait comme si $lang etait passee dans l'URL
-	// (pour criteres {lang}).
-	$GLOBALS['lang'] = $_GET['lang'] = $spip_lang;
+    global $spip_lang;
+
+    // quelle langue est demandee ?
+    $lang_demandee = (test_espace_prive()?$GLOBALS['spip_lang']:$GLOBALS['meta']['langue_site']);
+    if (isset($_COOKIE['spip_lang_ecrire']))
+        $lang_demandee = $_COOKIE['spip_lang_ecrire'];
+    if (!test_espace_prive() AND isset($_COOKIE['spip_lang']))
+        $lang_demandee = $_COOKIE['spip_lang'];
+    if (isset($_GET['lang']))
+        $lang_demandee = $_GET['lang'];
+
+    // Renvoyer si besoin (et si la langue demandee existe)
+    if ($spip_lang != $lang_demandee
+    AND changer_langue($lang_demandee)
+    AND $lang_demandee != @$_GET['lang']) {
+        $destination = parametre_url(self(),'lang', $lang_demandee, '&');
+        // ici on a besoin des var_truc
+        foreach ($_GET as $var => $val) {
+            if (!strncmp('var_', $var, 4))
+                $destination = parametre_url($destination, $var, $val, '&');
+        }
+        include_spip('inc/headers');
+        redirige_par_entete($destination);
+    }
+
+    // Subtilite : si la langue demandee par cookie est la bonne
+    // alors on fait comme si $lang etait passee dans l'URL
+    // (pour criteres {lang}).
+    $GLOBALS['lang'] = $_GET['lang'] = $spip_lang;
 }
 
 
@@ -303,20 +303,20 @@ 
  *     La langue sélectionnée
 **/
 function utiliser_langue_site() {
-	// s'il existe une langue du site (en gros tout le temps en théorie)
-	if (isset($GLOBALS['meta']['langue_site'])
-	  // et si spip_langue est pas encore définie (ce que va faire changer_langue())
-	  // ou qu'elle n'est pas identique à la langue du site
-	  AND (!isset($GLOBALS['spip_lang'])
-	    OR $GLOBALS['spip_lang']!=$GLOBALS['meta']['langue_site']))
-	{
-		return changer_langue($GLOBALS['meta']['langue_site']);//@:install
-	}
-	// en theorie là, la globale est définie, sinon c'est un problème.
-	if (!isset($GLOBALS['spip_lang'])) {
-		spip_log("La globale spip_lang est indéfinie dans utiliser_langue_site() !", _LOG_ERREUR);
-	}
-	return $GLOBALS['spip_lang'];
+    // s'il existe une langue du site (en gros tout le temps en théorie)
+    if (isset($GLOBALS['meta']['langue_site'])
+        // et si spip_langue est pas encore définie (ce que va faire changer_langue())
+        // ou qu'elle n'est pas identique à la langue du site
+      AND (!isset($GLOBALS['spip_lang'])
+        OR $GLOBALS['spip_lang']!=$GLOBALS['meta']['langue_site']))
+    {
+        return changer_langue($GLOBALS['meta']['langue_site']);//@:install
+    }
+    // en theorie là, la globale est définie, sinon c'est un problème.
+    if (!isset($GLOBALS['spip_lang'])) {
+        spip_log("La globale spip_lang est indéfinie dans utiliser_langue_site() !", _LOG_ERREUR);
+    }
+    return $GLOBALS['spip_lang'];
 }
 
 /**
@@ -333,23 +333,23 @@ 
 **/
 function utiliser_langue_visiteur() {
 
-	$l = (!test_espace_prive()  ? 'spip_lang' : 'spip_lang_ecrire');
-	if (isset($_COOKIE[$l]))
-		if (changer_langue($l = $_COOKIE[$l]))
-			return $l;
+    $l = (!test_espace_prive()  ? 'spip_lang' : 'spip_lang_ecrire');
+    if (isset($_COOKIE[$l]))
+        if (changer_langue($l = $_COOKIE[$l]))
+            return $l;
 
-	if (isset($GLOBALS['visiteur_session']['lang']))
-		if (changer_langue($l = $GLOBALS['visiteur_session']['lang']))
-			return $l;
+    if (isset($GLOBALS['visiteur_session']['lang']))
+        if (changer_langue($l = $GLOBALS['visiteur_session']['lang']))
+            return $l;
 
-	foreach (explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $s)  {
-		if (preg_match('#^([a-z]{2,3})(-[a-z]{2,3})?(;q=[0-9.]+)?$#i', trim($s), $r)) {
-			if (changer_langue($l=strtolower($r[1])))
-				return $l;
-		}
-	}
+    foreach (explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $s)  {
+        if (preg_match('#^([a-z]{2,3})(-[a-z]{2,3})?(;q=[0-9.]+)?$#i', trim($s), $r)) {
+            if (changer_langue($l=strtolower($r[1])))
+                return $l;
+        }
+    }
 
-	return utiliser_langue_site();
+    return utiliser_langue_site();
 }
 
 
@@ -367,16 +367,16 @@ 
  *     Retourne ecrire/lang/ s'il rien ne semble correspondre.
 **/
 function repertoire_lang($module='spip', $lang='fr') {
-	# valeur forcee (par ex.sur spip.net), old style, a faire disparaitre
-	if (defined('_DIR_LANG'))
-		return _DIR_LANG;
+    # valeur forcee (par ex.sur spip.net), old style, a faire disparaitre
+    if (defined('_DIR_LANG'))
+        return _DIR_LANG;
 
-	# regarder s'il existe une v.f. qq part
-	if ($f = find_in_path($module.'_'.$lang . '.php', 'lang/'))
-		return dirname($f).'/';
+    # regarder s'il existe une v.f. qq part
+    if ($f = find_in_path($module.'_'.$lang . '.php', 'lang/'))
+        return dirname($f).'/';
 
-	# sinon, je ne sais trop pas quoi dire...
-	return _DIR_RESTREINT . 'lang/';
+    # sinon, je ne sais trop pas quoi dire...
+    return _DIR_RESTREINT . 'lang/';
 }
 
 
@@ -395,34 +395,34 @@ 
 **/
 function init_langues() {
 
-	// liste des langues dans les meta, sauf a l'install
-	$all_langs = @$GLOBALS['meta']['langues_proposees'];
-
-	$tout = array();
-	if (!$all_langs) {
-		if ($d = @opendir(repertoire_lang())) {
-			while (($f = readdir($d)) !== false) {
-				if (preg_match(',^spip_([a-z_]+)\.php[3]?$,', $f, $regs))
-					$tout[] = $regs[1];
-			}
-			closedir($d);
-			sort($tout);
-			$tout = join(',', $tout);
-			// Si les langues n'ont pas change, ne rien faire
-			if ($tout != $all_langs) {
-				$GLOBALS['meta']['langues_proposees'] =	$tout;
-				include_spip('inc/meta');
-				ecrire_meta('langues_proposees', $tout);
-			} else $tout = '';
-		}
-	}
-	if (!isset($GLOBALS['meta']['langue_site'])) {
-		// Initialisation : le francais si dispo, sinon la premiere langue trouvee
-		$GLOBALS['meta']['langue_site'] = $tout =
-		(!$all_langs OR (strpos(','._LANGUE_PAR_DEFAUT.',',",$all_langs,")!==false))
-		  ? _LANGUE_PAR_DEFAUT :  substr($all_langs,0,strpos($all_langs,','));
-		ecrire_meta('langue_site', $tout);
-	}
+    // liste des langues dans les meta, sauf a l'install
+    $all_langs = @$GLOBALS['meta']['langues_proposees'];
+
+    $tout = array();
+    if (!$all_langs) {
+        if ($d = @opendir(repertoire_lang())) {
+            while (($f = readdir($d)) !== false) {
+                if (preg_match(',^spip_([a-z_]+)\.php[3]?$,', $f, $regs))
+                    $tout[] = $regs[1];
+            }
+            closedir($d);
+            sort($tout);
+            $tout = join(',', $tout);
+            // Si les langues n'ont pas change, ne rien faire
+            if ($tout != $all_langs) {
+                $GLOBALS['meta']['langues_proposees'] =	$tout;
+                include_spip('inc/meta');
+                ecrire_meta('langues_proposees', $tout);
+            } else $tout = '';
+        }
+    }
+    if (!isset($GLOBALS['meta']['langue_site'])) {
+        // Initialisation : le francais si dispo, sinon la premiere langue trouvee
+        $GLOBALS['meta']['langue_site'] = $tout =
+        (!$all_langs OR (strpos(','._LANGUE_PAR_DEFAUT.',',",$all_langs,")!==false))
+          ? _LANGUE_PAR_DEFAUT :  substr($all_langs,0,strpos($all_langs,','));
+        ecrire_meta('langue_site', $tout);
+    }
 }
 
 /**
@@ -437,9 +437,9 @@ 
 **/
 function html_lang_attributes()
 {
-	$lang = $GLOBALS['spip_lang'];
-	$dir = ($GLOBALS['spip_lang_rtl'] ? 'rtl' : 'ltr');
-	return  "<html class='$dir $lang no-js' xmlns='http://www.w3.org/1999/xhtml' lang='$lang' dir='$dir'>\n" ;
+    $lang = $GLOBALS['spip_lang'];
+    $dir = ($GLOBALS['spip_lang_rtl'] ? 'rtl' : 'ltr');
+    return  "<html class='$dir $lang no-js' xmlns='http://www.w3.org/1999/xhtml' lang='$lang' dir='$dir'>\n" ;
 }
 
 // initialise les globales (liste des langue, langue du site, spip_lang...)

Braces +70 added lines, -44 removed lines

@@ -15,7 +15,9 @@ 
  *
  * @package SPIP\Langue
 **/
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 
 
@@ -43,8 +45,9 @@ 
 	// Si la langue demandee n'existe pas, on essaie d'autres variantes
 	// Exemple : 'pt-br' => 'pt_br' => 'pt'
 	$lang = str_replace('-', '_', trim($lang));
-	if (!$lang)
-		return false;
+	if (!$lang) {
+			return false;
+	}
 
 	if (strpos($liste_langues,",$lang,")!==false
 	OR ($lang = preg_replace(',_.*,', '', $lang)
@@ -55,9 +58,10 @@ 
 		$spip_lang_left =  $spip_lang_rtl ? 'right' : 'left';
 
 		return $GLOBALS['spip_lang'] = $lang;
-	} else
-		return false;
-}
+	} else {
+			return false;
+	}
+	}
 
 //
 // Gestion des blocs multilingues
@@ -78,7 +82,9 @@ 
 // la langue X etant consideree comme une approche de X_Y
 function approcher_langue ($trads, $lang='') {
 
-	if (!$lang) $lang = $GLOBALS['spip_lang']; 
+	if (!$lang) {
+	    $lang = $GLOBALS['spip_lang'];
+	}
 
 	if (isset($trads[$lang])) {
 		return $lang;
@@ -86,8 +92,9 @@ 
 	// cas des langues xx_yy
 	else {
 		$r = explode('_', $lang);
-		if (isset($trads[$r[0]]))
-			return $r[0];
+		if (isset($trads[$r[0]])) {
+					return $r[0];
+		}
 	}
 	return '';
 }
@@ -131,21 +138,23 @@ 
 	if ($lang == 'eo'
 	  OR $lang == 'fr'
 	  OR strncmp($lang, 'fr_', 3)==0
-	  OR $lang == 'cpf')
-		return 'fr';
-	else
-		return 'en';
-}
+	  OR $lang == 'cpf') {
+			return 'fr';
+	} else {
+			return 'en';
+	}
+	}
 
 // gestion de la globale $lang_objet pour que les textes soient affiches
 // avec les memes typo et direction dans l'espace prive que dans le public
 // http://doc.spip.org/@changer_typo
 function changer_typo($lang = '') {
-	if ($lang)
-		$GLOBALS['lang_objet'] = $lang;
-	else
-		unset($GLOBALS['lang_objet']);
-}
+	if ($lang) {
+			$GLOBALS['lang_objet'] = $lang;
+	} else {
+			unset($GLOBALS['lang_objet']);
+	}
+	}
 
 //
 // Afficher un menu de selection de langue
@@ -159,10 +168,13 @@ 
 
 	$langues = liste_options_langues($nom_select);
 	$ret = "";
-	if (!count($langues))
-		return '';
+	if (!count($langues)) {
+			return '';
+	}
 
-	if (!$default) $default = $GLOBALS['spip_lang'];
+	if (!$default) {
+	    $default = $GLOBALS['spip_lang'];
+	}
 	foreach ($langues as $l) {
 		$selected = ($l == $default) ? ' selected=\'selected\'' : '';
 		$ret .= "<option value='$l'$selected>[".$l."] ".traduire_nom_langue($l)."</option>\n";
@@ -242,7 +254,9 @@ 
 # + langues_multilingues ; mais, ne sert pas
 #			$langues = explode(',', $GLOBALS['all_langs']);
 	}
-	if (count($langues) <= 1) return array();
+	if (count($langues) <= 1) {
+	    return array();
+	}
 	sort($langues);
 	return $langues;
 }
@@ -263,12 +277,15 @@ 
 
 	// quelle langue est demandee ?
 	$lang_demandee = (test_espace_prive()?$GLOBALS['spip_lang']:$GLOBALS['meta']['langue_site']);
-	if (isset($_COOKIE['spip_lang_ecrire']))
-		$lang_demandee = $_COOKIE['spip_lang_ecrire'];
-	if (!test_espace_prive() AND isset($_COOKIE['spip_lang']))
-		$lang_demandee = $_COOKIE['spip_lang'];
-	if (isset($_GET['lang']))
-		$lang_demandee = $_GET['lang'];
+	if (isset($_COOKIE['spip_lang_ecrire'])) {
+			$lang_demandee = $_COOKIE['spip_lang_ecrire'];
+	}
+	if (!test_espace_prive() AND isset($_COOKIE['spip_lang'])) {
+			$lang_demandee = $_COOKIE['spip_lang'];
+	}
+	if (isset($_GET['lang'])) {
+			$lang_demandee = $_GET['lang'];
+	}
 
 	// Renvoyer si besoin (et si la langue demandee existe)
 	if ($spip_lang != $lang_demandee
@@ -277,8 +294,9 @@ 
 		$destination = parametre_url(self(),'lang', $lang_demandee, '&');
 		// ici on a besoin des var_truc
 		foreach ($_GET as $var => $val) {
-			if (!strncmp('var_', $var, 4))
-				$destination = parametre_url($destination, $var, $val, '&');
+			if (!strncmp('var_', $var, 4)) {
+							$destination = parametre_url($destination, $var, $val, '&');
+			}
 		}
 		include_spip('inc/headers');
 		redirige_par_entete($destination);
@@ -334,18 +352,21 @@ 
 function utiliser_langue_visiteur() {
 
 	$l = (!test_espace_prive()  ? 'spip_lang' : 'spip_lang_ecrire');
-	if (isset($_COOKIE[$l]))
-		if (changer_langue($l = $_COOKIE[$l]))
+	if (isset($_COOKIE[$l])) {
+			if (changer_langue($l = $_COOKIE[$l]))
 			return $l;
+	}
 
-	if (isset($GLOBALS['visiteur_session']['lang']))
-		if (changer_langue($l = $GLOBALS['visiteur_session']['lang']))
+	if (isset($GLOBALS['visiteur_session']['lang'])) {
+			if (changer_langue($l = $GLOBALS['visiteur_session']['lang']))
 			return $l;
+	}
 
 	foreach (explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $s)  {
 		if (preg_match('#^([a-z]{2,3})(-[a-z]{2,3})?(;q=[0-9.]+)?$#i', trim($s), $r)) {
-			if (changer_langue($l=strtolower($r[1])))
-				return $l;
+			if (changer_langue($l=strtolower($r[1]))) {
+							return $l;
+			}
 		}
 	}
 
@@ -368,12 +389,14 @@ 
 **/
 function repertoire_lang($module='spip', $lang='fr') {
 	# valeur forcee (par ex.sur spip.net), old style, a faire disparaitre
-	if (defined('_DIR_LANG'))
-		return _DIR_LANG;
+	if (defined('_DIR_LANG')) {
+			return _DIR_LANG;
+	}
 
 	# regarder s'il existe une v.f. qq part
-	if ($f = find_in_path($module.'_'.$lang . '.php', 'lang/'))
-		return dirname($f).'/';
+	if ($f = find_in_path($module.'_'.$lang . '.php', 'lang/')) {
+			return dirname($f).'/';
+	}
 
 	# sinon, je ne sais trop pas quoi dire...
 	return _DIR_RESTREINT . 'lang/';
@@ -402,8 +425,9 @@ 
 	if (!$all_langs) {
 		if ($d = @opendir(repertoire_lang())) {
 			while (($f = readdir($d)) !== false) {
-				if (preg_match(',^spip_([a-z_]+)\.php[3]?$,', $f, $regs))
-					$tout[] = $regs[1];
+				if (preg_match(',^spip_([a-z_]+)\.php[3]?$,', $f, $regs)) {
+									$tout[] = $regs[1];
+				}
 			}
 			closedir($d);
 			sort($tout);
@@ -413,7 +437,9 @@ 
 				$GLOBALS['meta']['langues_proposees'] =	$tout;
 				include_spip('inc/meta');
 				ecrire_meta('langues_proposees', $tout);
-			} else $tout = '';
+			} else {
+			    $tout = '';
+			}
 		}
 	}
 	if (!isset($GLOBALS['meta']['langue_site'])) {

Spacing +34 added lines, -36 removed lines

@@ -37,8 +37,8 @@ 
 function changer_langue($lang) {
 	global $spip_lang_rtl, $spip_lang_right, $spip_lang_left;
 
-	$liste_langues = ',' . @$GLOBALS['meta']['langues_proposees']
-	. ',' . @$GLOBALS['meta']['langues_multilingue'] . ',';
+	$liste_langues = ','.@$GLOBALS['meta']['langues_proposees']
+	. ','.@$GLOBALS['meta']['langues_multilingue'].',';
 
 	// Si la langue demandee n'existe pas, on essaie d'autres variantes
 	// Exemple : 'pt-br' => 'pt_br' => 'pt'
@@ -46,13 +46,13 @@ 
 	if (!$lang)
 		return false;
 
-	if (strpos($liste_langues,",$lang,")!==false
+	if (strpos($liste_langues, ",$lang,") !== false
 	OR ($lang = preg_replace(',_.*,', '', $lang)
-	AND strpos($liste_langues,",$lang,")!==false)) {
+	AND strpos($liste_langues, ",$lang,") !== false)) {
 
-		$spip_lang_rtl =   lang_dir($lang, '', '_rtl');
+		$spip_lang_rtl = lang_dir($lang, '', '_rtl');
 		$spip_lang_right = $spip_lang_rtl ? 'left' : 'right';
-		$spip_lang_left =  $spip_lang_rtl ? 'right' : 'left';
+		$spip_lang_left = $spip_lang_rtl ? 'right' : 'left';
 
 		return $GLOBALS['spip_lang'] = $lang;
 	} else
@@ -68,7 +68,7 @@ 
 // ou permettre de choisir une langue "plus proche",
 // par exemple le francais pour l'espagnol, l'anglais pour l'allemand, etc.
 
-function choisir_traduction ($trads, $lang='') {
+function choisir_traduction($trads, $lang = '') {
 	$k = approcher_langue($trads, $lang);
 	return $k ? $trads[$k] : array_shift($trads);
 }
@@ -76,7 +76,7 @@ 
 // retourne son 2e argument si c'est un index du premier
 // ou un index approchant sinon et si possible, 
 // la langue X etant consideree comme une approche de X_Y
-function approcher_langue ($trads, $lang='') {
+function approcher_langue($trads, $lang = '') {
 
 	if (!$lang) $lang = $GLOBALS['spip_lang']; 
 
@@ -109,7 +109,7 @@ 
 // hebreu a priori), 'droitier' sinon.
 // C'est utilise par #LANG_DIR, #LANG_LEFT, #LANG_RIGHT.
 // http://doc.spip.org/@lang_dir
-function lang_dir($lang='', $droitier='ltr', $gaucher='rtl') {
+function lang_dir($lang = '', $droitier = 'ltr', $gaucher = 'rtl') {
 	static $lang_rtl = array('ar', 'fa', 'ku', 'prs', 'ps', 'ur', 'he', 'heb', 'hbo', 'yi');
 
 	return in_array(($lang ? $lang : $GLOBALS['spip_lang']), $lang_rtl) ?
@@ -122,7 +122,7 @@ 
 // sinon determiner la typo en fonction de la langue courante
 
 // http://doc.spip.org/@lang_typo
-function lang_typo($lang='') {
+function lang_typo($lang = '') {
 	if (!$lang) {
 		$lang = isset($GLOBALS['lang_objet'])
 			? $GLOBALS['lang_objet']
@@ -130,7 +130,7 @@ 
 	}
 	if ($lang == 'eo'
 	  OR $lang == 'fr'
-	  OR strncmp($lang, 'fr_', 3)==0
+	  OR strncmp($lang, 'fr_', 3) == 0
 	  OR $lang == 'cpf')
 		return 'fr';
 	else
@@ -154,7 +154,7 @@ 
 // pour 'changer_lang' (langue de l'article, espace prive), c'est en Ajax
 // 
 // http://doc.spip.org/@menu_langues
-function menu_langues($nom_select, $default='') {
+function menu_langues($nom_select, $default = '') {
 	include_spip('inc/actions');
 
 	$langues = liste_options_langues($nom_select);
@@ -177,25 +177,23 @@ 
 	}
 
 	$change = ' onchange="this.parentNode.parentNode.submit()"';
-	return generer_action_auteur('converser',$base, $cible,
+	return generer_action_auteur('converser', $base, $cible,
 		(select_langues($nom_select, $change, $ret)
-		 . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='". _T('bouton_changer')."' /></div></noscript>"),
+		 . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='"._T('bouton_changer')."' /></div></noscript>"),
 				     " method='post'");
 }
 
 // http://doc.spip.org/@select_langues
-function select_langues($nom_select, $change, $options, $label="")
+function select_langues($nom_select, $change, $options, $label = "")
 {
 	static $cpt = 0;
-	$id = "menu_langues" . $cpt++;
+	$id = "menu_langues".$cpt++;
 	return
-		"<label for='$id'>".($label?$label:_T('info_langues'))."</label> ".
+		"<label for='$id'>".($label ? $label : _T('info_langues'))."</label> ".
 		"<select name='$nom_select' id='$id' "
 	  . ((!test_espace_prive()) ?
-	     ("class='forml menu_langues'") :
-	     (($nom_select == 'var_lang_ecrire')  ?
-	      ("class='lang_ecrire'") :
-	      "class='fondl'"))
+	     ("class='forml menu_langues'") : (($nom_select == 'var_lang_ecrire') ?
+	      ("class='lang_ecrire'") : "class='fondl'"))
 	  . $change
 	  . ">\n"
 	  . $options
@@ -220,7 +218,7 @@ 
  */
 function liste_options_langues($nom_select) {
 
-	switch($nom_select) {
+	switch ($nom_select) {
 		# #MENU_LANG
 		case 'var_lang':
 		# menu de changement de la langue d'un article
@@ -262,7 +260,7 @@ 
 	global $spip_lang;
 
 	// quelle langue est demandee ?
-	$lang_demandee = (test_espace_prive()?$GLOBALS['spip_lang']:$GLOBALS['meta']['langue_site']);
+	$lang_demandee = (test_espace_prive() ? $GLOBALS['spip_lang'] : $GLOBALS['meta']['langue_site']);
 	if (isset($_COOKIE['spip_lang_ecrire']))
 		$lang_demandee = $_COOKIE['spip_lang_ecrire'];
 	if (!test_espace_prive() AND isset($_COOKIE['spip_lang']))
@@ -274,7 +272,7 @@ 
 	if ($spip_lang != $lang_demandee
 	AND changer_langue($lang_demandee)
 	AND $lang_demandee != @$_GET['lang']) {
-		$destination = parametre_url(self(),'lang', $lang_demandee, '&');
+		$destination = parametre_url(self(), 'lang', $lang_demandee, '&');
 		// ici on a besoin des var_truc
 		foreach ($_GET as $var => $val) {
 			if (!strncmp('var_', $var, 4))
@@ -308,9 +306,9 @@ 
 	  // et si spip_langue est pas encore définie (ce que va faire changer_langue())
 	  // ou qu'elle n'est pas identique à la langue du site
 	  AND (!isset($GLOBALS['spip_lang'])
-	    OR $GLOBALS['spip_lang']!=$GLOBALS['meta']['langue_site']))
+	    OR $GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site']))
 	{
-		return changer_langue($GLOBALS['meta']['langue_site']);//@:install
+		return changer_langue($GLOBALS['meta']['langue_site']); //@:install
 	}
 	// en theorie là, la globale est définie, sinon c'est un problème.
 	if (!isset($GLOBALS['spip_lang'])) {
@@ -333,7 +331,7 @@ 
 **/
 function utiliser_langue_visiteur() {
 
-	$l = (!test_espace_prive()  ? 'spip_lang' : 'spip_lang_ecrire');
+	$l = (!test_espace_prive() ? 'spip_lang' : 'spip_lang_ecrire');
 	if (isset($_COOKIE[$l]))
 		if (changer_langue($l = $_COOKIE[$l]))
 			return $l;
@@ -342,9 +340,9 @@ 
 		if (changer_langue($l = $GLOBALS['visiteur_session']['lang']))
 			return $l;
 
-	foreach (explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $s)  {
+	foreach (explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $s) {
 		if (preg_match('#^([a-z]{2,3})(-[a-z]{2,3})?(;q=[0-9.]+)?$#i', trim($s), $r)) {
-			if (changer_langue($l=strtolower($r[1])))
+			if (changer_langue($l = strtolower($r[1])))
 				return $l;
 		}
 	}
@@ -366,17 +364,17 @@ 
  *     Adresse du répertoire contenant le module
  *     Retourne ecrire/lang/ s'il rien ne semble correspondre.
 **/
-function repertoire_lang($module='spip', $lang='fr') {
+function repertoire_lang($module = 'spip', $lang = 'fr') {
 	# valeur forcee (par ex.sur spip.net), old style, a faire disparaitre
 	if (defined('_DIR_LANG'))
 		return _DIR_LANG;
 
 	# regarder s'il existe une v.f. qq part
-	if ($f = find_in_path($module.'_'.$lang . '.php', 'lang/'))
+	if ($f = find_in_path($module.'_'.$lang.'.php', 'lang/'))
 		return dirname($f).'/';
 
 	# sinon, je ne sais trop pas quoi dire...
-	return _DIR_RESTREINT . 'lang/';
+	return _DIR_RESTREINT.'lang/';
 }
 
 
@@ -410,7 +408,7 @@ 
 			$tout = join(',', $tout);
 			// Si les langues n'ont pas change, ne rien faire
 			if ($tout != $all_langs) {
-				$GLOBALS['meta']['langues_proposees'] =	$tout;
+				$GLOBALS['meta']['langues_proposees'] = $tout;
 				include_spip('inc/meta');
 				ecrire_meta('langues_proposees', $tout);
 			} else $tout = '';
@@ -419,8 +417,8 @@ 
 	if (!isset($GLOBALS['meta']['langue_site'])) {
 		// Initialisation : le francais si dispo, sinon la premiere langue trouvee
 		$GLOBALS['meta']['langue_site'] = $tout =
-		(!$all_langs OR (strpos(','._LANGUE_PAR_DEFAUT.',',",$all_langs,")!==false))
-		  ? _LANGUE_PAR_DEFAUT :  substr($all_langs,0,strpos($all_langs,','));
+		(!$all_langs OR (strpos(','._LANGUE_PAR_DEFAUT.',', ",$all_langs,") !== false))
+		  ? _LANGUE_PAR_DEFAUT :  substr($all_langs, 0, strpos($all_langs, ','));
 		ecrire_meta('langue_site', $tout);
 	}
 }
@@ -439,7 +437,7 @@ 
 {
 	$lang = $GLOBALS['spip_lang'];
 	$dir = ($GLOBALS['spip_lang_rtl'] ? 'rtl' : 'ltr');
-	return  "<html class='$dir $lang no-js' xmlns='http://www.w3.org/1999/xhtml' lang='$lang' dir='$dir'>\n" ;
+	return  "<html class='$dir $lang no-js' xmlns='http://www.w3.org/1999/xhtml' lang='$lang' dir='$dir'>\n";
 }
 
 // initialise les globales (liste des langue, langue du site, spip_lang...)

ecrire/inc/math.php

Doc Comments +3 added lines

@@ -19,6 +19,9 @@
 //
 
 // http://doc.spip.org/@image_math
+/**
+ * @param string $tex
+ */
 function produire_image_math($tex) {
 	global $traiter_math;
 	switch ($traiter_math) {

Indentation +76 added lines, -76 removed lines

@@ -20,56 +20,56 @@ 
 
 // http://doc.spip.org/@image_math
 function produire_image_math($tex) {
-	global $traiter_math;
-	switch ($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;
+    global $traiter_math;
+    switch ($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;
 	
 
-	if (!@file_exists($fichier)) {
-		// Aller chercher l'image sur le serveur
-		if ($server) {
-			spip_log($url = $server.'?'.rawurlencode($tex));
-			include_spip('inc/distant');
-			recuperer_page($url,$fichier);
-		}
-	}
+    if (!@file_exists($fichier)) {
+        // Aller chercher l'image sur le serveur
+        if ($server) {
+            spip_log($url = $server.'?'.rawurlencode($tex));
+            include_spip('inc/distant');
+            recuperer_page($url,$fichier);
+        }
+    }
 
 
-	// Composer la reponse selon presence ou non de l'image
-	$tex = entites_html($tex);
-	if (@file_exists($fichier)) {
+    // Composer la reponse selon presence ou non de l'image
+    $tex = entites_html($tex);
+    if (@file_exists($fichier)) {
 
-		// MathML
-		if ($traiter_math == 'mathml') {
-			return join(file("$fichier"),"");
-		}
+        // MathML
+        if ($traiter_math == 'mathml') {
+            return join(file("$fichier"),"");
+        }
 
-		// TeX
-		else {
-			list(,,,$size) = @getimagesize($fichier);
-			$alt = "alt=\"$tex\" title=\"$tex\""; 
-			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
-		}
+        // TeX
+        else {
+            list(,,,$size) = @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>";
+    }
+    else // pas de fichier
+        return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
 
 }
 
@@ -78,39 +78,39 @@ 
 // http://doc.spip.org/@traiter_math
 function traiter_math($letexte, $source='') {
 
-	$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))) {
-			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($regs[1])."</p>\n";
-			$pos = strpos($texte_milieu, $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))) {
-			$echap = produire_image_math($regs[1]);
-			$pos = strpos($texte_milieu, $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))) {
+            $echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($regs[1])."</p>\n";
+            $pos = strpos($texte_milieu, $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))) {
+            $echap = produire_image_math($regs[1]);
+            $pos = strpos($texte_milieu, $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 +14 added lines, -14 removed lines

@@ -37,8 +37,8 @@ 
 
 	// 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;
+		@mkdir($dir_tex, _SPIP_CHMOD);
+	$fichier = $dir_tex.md5(trim($tex)).$ext;
 	
 
 	if (!@file_exists($fichier)) {
@@ -46,7 +46,7 @@ 
 		if ($server) {
 			spip_log($url = $server.'?'.rawurlencode($tex));
 			include_spip('inc/distant');
-			recuperer_page($url,$fichier);
+			recuperer_page($url, $fichier);
 		}
 	}
 
@@ -57,7 +57,7 @@ 
 
 		// MathML
 		if ($traiter_math == 'mathml') {
-			return join(file("$fichier"),"");
+			return join(file("$fichier"), "");
 		}
 
 		// TeX
@@ -76,35 +76,35 @@ 
 
 // Fonction appelee par propre() s'il repere un mode <math>
 // http://doc.spip.org/@traiter_math
-function traiter_math($letexte, $source='') {
+function traiter_math($letexte, $source = '') {
 
 	$texte_a_voir = $letexte;
 	while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
-		if (!$fin = strpos($texte_a_voir,"</math>"))
+		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>"));
+			$debut + strlen("<math>"), $fin - $debut - strlen("<math>"));
 		$texte_fin = substr($texte_a_voir,
-			$fin+strlen("</math>"), strlen($texte_a_voir));
+			$fin + strlen("</math>"), strlen($texte_a_voir));
 
 		// Les doubles $$x^2$$ en mode 'div'
-		while((preg_match(",[$][$]([^$]+)[$][$],",$texte_milieu, $regs))) {
+		while ((preg_match(",[$][$]([^$]+)[$][$],", $texte_milieu, $regs))) {
 			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($regs[1])."</p>\n";
 			$pos = strpos($texte_milieu, $regs[0]);
-			$texte_milieu = substr($texte_milieu,0,$pos)
+			$texte_milieu = substr($texte_milieu, 0, $pos)
 				. code_echappement($echap, $source)
-				. substr($texte_milieu,$pos+strlen($regs[0]));
+				. substr($texte_milieu, $pos + strlen($regs[0]));
 		}
 
 		// Les simples $x^2$ en mode 'span'
-		while((preg_match(",[$]([^$]+)[$],",$texte_milieu, $regs))) {
+		while ((preg_match(",[$]([^$]+)[$],", $texte_milieu, $regs))) {
 			$echap = produire_image_math($regs[1]);
 			$pos = strpos($texte_milieu, $regs[0]);
-			$texte_milieu = substr($texte_milieu,0,$pos)
+			$texte_milieu = substr($texte_milieu, 0, $pos)
 				. code_echappement($echap, $source)
-				. substr($texte_milieu,$pos+strlen($regs[0]));
+				. substr($texte_milieu, $pos + strlen($regs[0]));
 		}
 
 		$texte_a_voir = $texte_debut.$texte_milieu.$texte_fin;

Braces +12 added lines, -7 removed lines

@@ -12,7 +12,9 @@ 
 
 
 //
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 //
 // Gestion du raccourci <math>...</math> en client-serveur
@@ -36,8 +38,9 @@ 
 	}
 
 	// 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);
+	if (!@is_dir($dir_tex = _DIR_VAR.'cache-TeX/')) {
+			@mkdir ($dir_tex, _SPIP_CHMOD);
+	}
 	$fichier = $dir_tex .md5(trim($tex)).$ext;
 	
 
@@ -67,9 +70,10 @@ 
 			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
 		}
 
-	}
-	else // pas de fichier
+	} else {
+	    // pas de fichier
 		return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
+	}
 
 }
 
@@ -80,8 +84,9 @@ 
 
 	$texte_a_voir = $letexte;
 	while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
-		if (!$fin = strpos($texte_a_voir,"</math>"))
-			$fin = strlen($texte_a_voir);
+		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,