spip / SPIP

ecrire/inc/utils.php

Braces +804 added lines, -393 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;
+}
 
 //
 // Utilitaires indispensables autour du serveur Http.
@@ -32,19 +34,28 @@ 
 function charger_fonction($nom, $dossier='exec', $continue=false) {
 	static $echecs = array();
 
-	if (strlen($dossier) AND substr($dossier,-1) != '/') $dossier .= '/';
+	if (strlen($dossier) AND substr($dossier,-1) != '/') {
+	    $dossier .= '/';
+	}
 	$f = str_replace('/','_',$dossier) . $nom;
 
-	if (function_exists($f))
-		return $f;
-	if (function_exists($g = $f . '_dist'))
-		return $g;
+	if (function_exists($f)) {
+			return $f;
+	}
+	if (function_exists($g = $f . '_dist')) {
+			return $g;
+	}
 
-	if (isset($echecs[$f])) return $echecs[$f];
+	if (isset($echecs[$f])) {
+	    return $echecs[$f];
+	}
 	// Sinon charger le fichier de declaration si plausible
 
 	if (!preg_match(',^\w+$,', $f)){
-		if ($continue) return false; //appel interne, on passe
+		if ($continue) {
+		    return false;
+		}
+		//appel interne, on passe
 		include_spip('inc/minipres');
 		echo minipres();
 		exit;
@@ -55,12 +66,19 @@ 
 	if (!$inc = include_spip($dossier.($d = strtolower($nom)))
 		// si le fichier truc/machin/nom.php n'existe pas,
 		// la fonction peut etre definie dans truc/machin.php qui regroupe plusieurs petites fonctions
-		AND strlen(dirname($dossier)) AND dirname($dossier)!='.')
-		include_spip(substr($dossier,0,-1));
-	if (function_exists($f)) return $f;
-	if (function_exists($g)) return $g;
+		AND strlen(dirname($dossier)) AND dirname($dossier)!='.') {
+			include_spip(substr($dossier,0,-1));
+	}
+	if (function_exists($f)) {
+	    return $f;
+	}
+	if (function_exists($g)) {
+	    return $g;
+	}
 
-	if ($continue) return $echecs[$f] = false;
+	if ($continue) {
+	    return $echecs[$f] = false;
+	}
 
 	// Echec : message d'erreur
 	spip_log("fonction $nom ($f ou $g) indisponible" .
@@ -115,14 +133,15 @@ 
 // http://doc.spip.org/@minipipe
 function minipipe($fonc,&$val){
 	// fonction
-	if (function_exists($fonc))
-		$val = call_user_func($fonc, $val);
+	if (function_exists($fonc)) {
+			$val = call_user_func($fonc, $val);
+	}
 	// Class::Methode
 	else if (preg_match("/^(\w*)::(\w*)$/S", $fonc, $regs)
 	AND $methode = array($regs[1], $regs[2])
-	AND is_callable($methode))
-		$val = call_user_func($methode, $val);
-	else {
+	AND is_callable($methode)) {
+			$val = call_user_func($methode, $val);
+	} else {
 		spip_log("Erreur - '$fonc' non definie !");
 	}
 	return $val;
@@ -140,8 +159,9 @@ 
 			// generer les fichiers php precompiles
 			// de chargement des plugins et des pipelines
 			actualise_plugins_actifs();
-			if (!($ok = @is_readable($charger)))
-				spip_log("fichier $charger pas cree");
+			if (!($ok = @is_readable($charger))) {
+							spip_log("fichier $charger pas cree");
+			}
 		}
 
 		if ($ok) {
@@ -164,8 +184,9 @@ 
 	// array_key_exists pour php 4.1.0
 	if (is_array($val)
 	  AND count($val)==2
-	  AND (array_key_exists('data',$val)))
-		$val = $val['data'];
+	  AND (array_key_exists('data',$val))) {
+			$val = $val['data'];
+	}
 	return $val;
 }
 
@@ -190,10 +211,12 @@ 
 	static $pre = array();
 	static $log;
 	preg_match('/^([a-z_]*)\.?(\d)?$/iS', (string) $name, $regs);
-	if (!isset($regs[1]) OR !$logname = $regs[1])
-		$logname = null;
-	if (!isset($regs[2]) OR !$niveau = $regs[2])
-		$niveau = _LOG_INFO;
+	if (!isset($regs[1]) OR !$logname = $regs[1]) {
+			$logname = null;
+	}
+	if (!isset($regs[2]) OR !$niveau = $regs[2]) {
+			$niveau = _LOG_INFO;
+	}
 
 	if ($niveau <= (defined('_LOG_FILTRE_GRAVITE') ? _LOG_FILTRE_GRAVITE : _LOG_INFO_IMPORTANTE)) {
 		if (!$pre){
@@ -208,7 +231,9 @@ 
 				_LOG_DEBUG=>'debug:');
 			$log = charger_fonction('log', 'inc');
 		}
-		if (!is_string($message)) $message = var_export($message, true);
+		if (!is_string($message)) {
+		    $message = var_export($message, true);
+		}
 		$log($pre[$niveau].' '.$message, $logname);
 	}
 }
@@ -226,12 +251,17 @@ 
 // http://doc.spip.org/@_request
 function _request($var, $c=false) {
 
-	if (is_array($c))
-		return isset($c[$var]) ? $c[$var] : NULL;
+	if (is_array($c)) {
+			return isset($c[$var]) ? $c[$var] : NULL;
+	}
 
-	if (isset($_GET[$var])) $a = $_GET[$var];
-	elseif (isset($_POST[$var])) $a = $_POST[$var];
-	else return NULL;
+	if (isset($_GET[$var])) {
+	    $a = $_GET[$var];
+	} elseif (isset($_POST[$var])) {
+	    $a = $_POST[$var];
+	} else {
+	    return NULL;
+	}
 
 	// Si on est en ajax et en POST tout a ete encode
 	// via encodeURIComponent, il faut donc repasser
@@ -259,15 +289,17 @@ 
 function set_request($var, $val = NULL, $c=false) {
 	if (is_array($c)) {
 		unset($c[$var]);
-		if ($val !== NULL)
-			$c[$var] = $val;
+		if ($val !== NULL) {
+					$c[$var] = $val;
+		}
 		return $c;
 	}
 
 	unset($_GET[$var]);
 	unset($_POST[$var]);
-	if ($val !== NULL)
-		$_GET[$var] = $val;
+	if ($val !== NULL) {
+			$_GET[$var] = $val;
+	}
 
 	return false; # n'affecte pas $c
 }
@@ -314,22 +346,26 @@ 
  */
 function parametre_url($url, $c, $v=NULL, $sep='&amp;') {
 	// requete erronnee : plusieurs variable dans $c et aucun $v
-	if (strpos($c,"|")!==false AND is_null($v))
-		return null;
+	if (strpos($c,"|")!==false AND is_null($v)) {
+			return null;
+	}
 
 	// lever l'#ancre
 	if (preg_match(',^([^#]*)(#.*)$,', $url, $r)) {
 		$url = $r[1];
 		$ancre = $r[2];
-	} else
-		$ancre = '';
+	} else {
+			$ancre = '';
+	}
 
 	// eclater
 	$url = preg_split(',[?]|&amp;|&,', $url);
 
 	// recuperer la base
 	$a = array_shift($url);
-	if (!$a) $a= './';
+	if (!$a) {
+	    $a= './';
+	}
 
 	$regexp = ',^(' . str_replace('[]','\[\]',$c) . '[[]?[]]?)(=.*)?$,';
 	$ajouts = array_flip(explode('|',$c));
@@ -357,15 +393,17 @@ 
 	// traiter les parametres pas encore trouves
 	if ($v === NULL
 	AND $args = func_get_args()
-	AND count($args)==2)
-		return $v;
-	elseif ($testv) {
+	AND count($args)==2) {
+			return $v;
+	} elseif ($testv) {
 		foreach($ajouts as $k => $n) {
-		  if (!is_array($v))
-		    $url[] = $k .'=' . $u;
-		  else {
+		  if (!is_array($v)) {
+		  		    $url[] = $k .'=' . $u;
+		  } else {
 		  	$id = (substr($k,-2) == '[]') ? $k : ($k ."[]");
-		    foreach ($v as $w) $url[]= $id .'=' . $w;
+		    foreach ($v as $w) {
+		        $url[]= $id .'=' . $w;
+		    }
 		  }
 		}
 	}
@@ -374,8 +412,9 @@ 
 	$url = array_filter($url);
 
 	// recomposer l'adresse
-	if ($url)
-		$a .= '?' . join($sep, $url);
+	if ($url) {
+			$a .= '?' . join($sep, $url);
+	}
 
 	return $a . $ancre;
 }
@@ -390,8 +429,9 @@ 
 		$url = $r[1];
 	}
 	if (preg_match('/[^-_a-zA-Z0-9]+/S',$ancre)){
-		if (!function_exists('translitteration'))
-			include_spip('inc/charsets');
+		if (!function_exists('translitteration')) {
+					include_spip('inc/charsets');
+		}
 		$ancre = preg_replace(array('/^[^-_a-zA-Z0-9]+/', '/[^-_a-zA-Z0-9]/'), array('', '-'),
 						translitteration($ancre));
 	}
@@ -409,8 +449,12 @@ 
 {
 	static $done = false;
 	static $propre = '';
-	if (!is_null($reset)) return $propre=$reset;
-	if ($done) return $propre;
+	if (!is_null($reset)) {
+	    return $propre=$reset;
+	}
+	if ($done) {
+	    return $propre;
+	}
 	$done = true;
 
 	$uri1 = $GLOBALS['REQUEST_URI'];
@@ -442,12 +486,14 @@ 
 			$GLOBALS['profondeur_url']<(_DIR_RESTREINT?1:2)
 			// sinon c'est OK si _SET_HTML_BASE a ete force a false
 			OR (defined('_SET_HTML_BASE') AND !_SET_HTML_BASE))
-		)
-		$url = preg_replace(',^[^?]*/,', '', $url);
+		) {
+			$url = preg_replace(',^[^?]*/,', '', $url);
+	}
 	// ajouter le cas echeant les variables _POST['id_...']
-	foreach ($_POST as $v => $c)
-		if (substr($v,0,3) == 'id_')
+	foreach ($_POST as $v => $c) {
+			if (substr($v,0,3) == 'id_')
 			$url = parametre_url($url, $v, $c, '&');
+	}
 
 	// supprimer les variables sans interet
 	if (test_espace_prive()) {
@@ -465,8 +511,9 @@ 
 	$url = str_replace(array("'", '"', '<', '[', ']'), array('%27', '%22', '%3C', '%5B', '%5D'), $url);
 
 	// &amp; ?
-	if ($amp != '&amp;')
-		$url = str_replace('&amp;', $amp, $url);
+	if ($amp != '&amp;') {
+			$url = str_replace('&amp;', $amp, $url);
+	}
 
 	// Si ca demarre par ? ou vide, donner './'
 	$url = preg_replace(',^([?].*)?$,', './\1', $url);
@@ -507,8 +554,9 @@ 
 	$o = array('class'=>'', 'force'=>true);
 	if ($options){
 		// support de l'ancien argument $class
-		if (is_string($options))
-			$options = array('class'=>$options);
+		if (is_string($options)) {
+					$options = array('class'=>$options);
+		}
 		$o = array_merge($o,$options);
 	}
 
@@ -531,16 +579,18 @@ 
 	$text = $traduire($texte, $lang);
 
 	if (!strlen($text)){
-		if (!$o['force'])
-			return '';
+		if (!$o['force']) {
+					return '';
+		}
 
 		$text = $texte;
 
 		// pour les chaines non traduites, assurer un service minimum
-		if (!$GLOBALS['test_i18n'] AND (_request('var_mode') != 'traduction'))
-			$text = str_replace('_', ' ',
+		if (!$GLOBALS['test_i18n'] AND (_request('var_mode') != 'traduction')) {
+					$text = str_replace('_', ' ',
 				 (($n = strpos($text,':')) === false ? $texte :
 					substr($texte, $n+1)));
+		}
 		$o['class'] = null;
 
 	}
@@ -556,29 +606,34 @@ 
 	$f = $text;
 	if (is_array($args)) {
 		foreach ($args as $name => $value) {
-			if ($class)
-				$value = "<span class='$class'>$value</span>";
+			if ($class) {
+							$value = "<span class='$class'>$value</span>";
+			}
 			$t = str_replace ("@$name@", $value, $text);
 			if ($text !== $t) {unset($args[$name]); $text = $t;}
 		}
 		// Si des variables n'ont pas ete inserees, le signaler
 		// (chaines de langues pas a jour)
-		if ($args) spip_log("$f:  variables inutilisees " . join(', ', array_keys($args)),_LOG_DEBUG);
+		if ($args) {
+		    spip_log("$f:  variables inutilisees " . join(', ', array_keys($args)),_LOG_DEBUG);
+		}
 	}
 
-	if (($GLOBALS['test_i18n'] OR (_request('var_mode') == 'traduction')) AND $class===null)
-		return "<span class=debug-traduction-erreur>$text</span>";
-	else
-		return $text;
-}
+	if (($GLOBALS['test_i18n'] OR (_request('var_mode') == 'traduction')) AND $class===null) {
+			return "<span class=debug-traduction-erreur>$text</span>";
+	} else {
+			return $text;
+	}
+	}
 
 // Afficher "ecrire/data/" au lieu de "data/" dans les messages
 // ou tmp/ au lieu de ../tmp/
 // http://doc.spip.org/@joli_repertoire
 function joli_repertoire($rep) {
 	$a = substr($rep,0,1);
-	if ($a<>'.' AND $a<>'/')
-		$rep = (_DIR_RESTREINT?'':_DIR_RESTREINT_ABS).$rep;
+	if ($a<>'.' AND $a<>'/') {
+			$rep = (_DIR_RESTREINT?'':_DIR_RESTREINT_ABS).$rep;
+	}
 	$rep = preg_replace(',(^\.\.\/),', '', $rep);
 	return $rep;
 }
@@ -593,7 +648,10 @@ 
 	$a=time(); $b=microtime();
 	// microtime peut contenir les microsecondes et le temps
 	$b=explode(' ',$b);
-	if (count($b)==2) $a = end($b); // plus precis !
+	if (count($b)==2) {
+	    $a = end($b);
+	}
+	// plus precis !
 	$b = reset($b);
 	if (!isset($time[$t])) {
 		$time[$t] = $a + $b;
@@ -601,10 +659,12 @@ 
 		$p = ($a + $b - $time[$t]) * 1000;
 		unset($time[$t]);
 #			echo "'$p'";exit;
-		if ($raw) return $p;
-		if ($p < 1000)
-			$s = '';
-		else {
+		if ($raw) {
+		    return $p;
+		}
+		if ($p < 1000) {
+					$s = '';
+		} else {
 			$s = sprintf("%d ", $x = floor($p/1000));
 			$p -= ($x*1000);
 		}
@@ -619,8 +679,9 @@ 
 function spip_touch($fichier, $duree=0, $touch=true) {
 	if ($duree) {
 		clearstatcache();
-		if ((@$f=filemtime($fichier)) AND ($f >= time() - $duree))
-			return false;
+		if ((@$f=filemtime($fichier)) AND ($f >= time() - $duree)) {
+					return false;
+		}
 	}
 	if ($touch!==false) {
 		if (!@touch($fichier)) { spip_unlink($fichier); @touch($fichier); };
@@ -663,13 +724,20 @@ 
  */
 function cron ($taches=array(), $taches_old= array()) {
 	// si pas en mode cron force, laisser tomber.
-	if (!defined('_DIRECT_CRON_FORCE')) return false;
-	if (!is_array($taches)) $taches = $taches_old; // compat anciens appels
+	if (!defined('_DIRECT_CRON_FORCE')) {
+	    return false;
+	}
+	if (!is_array($taches)) {
+	    $taches = $taches_old;
+	}
+	// compat anciens appels
 	// si taches a inserer en base et base inaccessible, laisser tomber
 	// sinon on ne verifie pas la connexion tout de suite, car si ca se trouve
 	// queue_sleep_time_to_next_job() dira qu'il n'y a rien a faire
 	// et on evite d'ouvrir une connexion pour rien (utilisation de _DIRECT_CRON_FORCE dans mes_options.php)
-	if ($taches AND count($taches) AND !spip_connect())  return false;
+	if ($taches AND count($taches) AND !spip_connect()) {
+	    return false;
+	}
 	spip_log("cron !",'jq'._LOG_DEBUG);
 	if ($genie = charger_fonction('genie', 'inc', true)) {
 		return $genie($taches);
@@ -743,28 +811,31 @@ 
  */
 function queue_sleep_time_to_next_job($force=null) {
 	static $queue_next_job_time = -1;
-	if ($force===true)
-		$queue_next_job_time = -1;
-	elseif ($force)
-		$queue_next_job_time = $force;
+	if ($force===true) {
+			$queue_next_job_time = -1;
+	} elseif ($force) {
+			$queue_next_job_time = $force;
+	}
 
 	if ($queue_next_job_time==-1) {
 		define('_JQ_NEXT_JOB_TIME_FILENAME',_DIR_TMP . "job_queue_next.txt");
 		// utiliser un cache memoire si dispo
 		if (include_spip('inc/memoization') AND defined('_MEMOIZE_MEMORY') AND _MEMOIZE_MEMORY) {
 			$queue_next_job_time = cache_get(_JQ_NEXT_JOB_TIME_FILENAME);
-		}
-		else {
+		} else {
 			$queue_next_job_time = null;
-			if (lire_fichier(_JQ_NEXT_JOB_TIME_FILENAME, $contenu))
-				$queue_next_job_time = intval($contenu);
+			if (lire_fichier(_JQ_NEXT_JOB_TIME_FILENAME, $contenu)) {
+							$queue_next_job_time = intval($contenu);
+			}
 		}
 	}
 
-	if (is_null($queue_next_job_time))
-		return null;
-	if (!$_SERVER['REQUEST_TIME'])
-		$_SERVER['REQUEST_TIME'] = time();
+	if (is_null($queue_next_job_time)) {
+			return null;
+	}
+	if (!$_SERVER['REQUEST_TIME']) {
+			$_SERVER['REQUEST_TIME'] = time();
+	}
 	return $queue_next_job_time-$_SERVER['REQUEST_TIME'];
 }
 
@@ -786,14 +857,17 @@ 
 		$done[$src] = true;
 		$src = find_in_path($src, _JAVASCRIPT);
 		$src = " src='$src'";
+	} else {
+	    $src = '';
 	}
-	else $src = '';
-	if ($script)
-		$script = ("/*<![CDATA[*/\n" .
+	if ($script) {
+			$script = ("/*<![CDATA[*/\n" .
 		preg_replace(',</([^>]*)>,','<\/\1>', $script) .
 		"/*]]>*/");
-	if ($noscript)
-		$noscript = "<noscript>\n\t$noscript\n</noscript>\n";
+	}
+	if ($noscript) {
+			$noscript = "<noscript>\n\t$noscript\n</noscript>\n";
+	}
 
 	return ($src OR $script OR $noscript)
 	? "<script type='text/javascript'$src>$script</script>$noscript"
@@ -826,43 +900,53 @@ 
 			_DIR_RACINE.'prive/:'.
 			_DIR_RESTREINT;
 		// Ajouter squelettes/
-		if (@is_dir(_DIR_RACINE.'squelettes'))
-			$path = _DIR_RACINE.'squelettes/:' . $path;
+		if (@is_dir(_DIR_RACINE.'squelettes')) {
+					$path = _DIR_RACINE.'squelettes/:' . $path;
+		}
 		foreach (explode(':', $path) as $dir) {
-			if (strlen($dir) AND substr($dir,-1) != '/')
-				$dir .= "/";
+			if (strlen($dir) AND substr($dir,-1) != '/') {
+							$dir .= "/";
+			}
 			$path_base[] = $dir;
 		}
 		$path_full = $path_base;
 		// Et le(s) dossier(s) des squelettes nommes
-		if (strlen($GLOBALS['dossier_squelettes']))
-			foreach (array_reverse(explode(':', $GLOBALS['dossier_squelettes'])) as $d)
+		if (strlen($GLOBALS['dossier_squelettes'])) {
+					foreach (array_reverse(explode(':', $GLOBALS['dossier_squelettes'])) as $d)
 				array_unshift($path_full, ($d[0] == '/' ? '' : _DIR_RACINE) . $d . '/');
+		}
 		$GLOBALS['path_sig'] = md5(serialize($path_full));
 	}
-	if ($dir_path===NULL) return $path_full;
+	if ($dir_path===NULL) {
+	    return $path_full;
+	}
 
 	if (strlen($dir_path)){
 		$tete = "";
-		if (reset($path_base)==_DIR_RACINE.'squelettes/')
-			$tete = array_shift($path_base);
+		if (reset($path_base)==_DIR_RACINE.'squelettes/') {
+					$tete = array_shift($path_base);
+		}
 		$dirs = array_reverse(explode(':',$dir_path));
 		foreach($dirs as $dir_path){
 				#if ($dir_path{0}!='/')
 				#	$dir_path = $dir_path;
-				if (substr($dir_path,-1) != '/')
-					$dir_path .= "/";
-				if (!in_array($dir_path,$path_base))
-					array_unshift($path_base,$dir_path);
+				if (substr($dir_path,-1) != '/') {
+									$dir_path .= "/";
+				}
+				if (!in_array($dir_path,$path_base)) {
+									array_unshift($path_base,$dir_path);
+				}
+		}
+		if (strlen($tete)) {
+					array_unshift($path_base,$tete);
 		}
-		if (strlen($tete))
-			array_unshift($path_base,$tete);
 	}
 	$path_full = $path_base;
 	// Et le(s) dossier(s) des squelettes nommes
-	if (strlen($GLOBALS['dossier_squelettes']))
-		foreach (array_reverse(explode(':', $GLOBALS['dossier_squelettes'])) as $d)
+	if (strlen($GLOBALS['dossier_squelettes'])) {
+			foreach (array_reverse(explode(':', $GLOBALS['dossier_squelettes'])) as $d)
 			array_unshift($path_full, ($d[0] == '/' ? '' : _DIR_RACINE) . $d . '/');
+	}
 
 	$GLOBALS['path_sig'] = md5(serialize($path_full));
 	return $path_full;
@@ -887,8 +971,9 @@ 
 	static $themes = null;
 	if (is_null($themes)){
 		// si pas encore definie
-		if (!defined('_SPIP_THEME_PRIVE'))
-			define('_SPIP_THEME_PRIVE', 'spip');
+		if (!defined('_SPIP_THEME_PRIVE')) {
+					define('_SPIP_THEME_PRIVE', 'spip');
+		}
 		$themes = array(_SPIP_THEME_PRIVE);
 		// lors d'une installation neuve, prefs n'est pas definie.
 		if (isset($GLOBALS['visiteur_session']['prefs'])) {
@@ -896,24 +981,30 @@ 
 		} else {
 			$prefs = array();
 		}
-		if (is_string($prefs))
-			$prefs = unserialize($GLOBALS['visiteur_session']['prefs']);
+		if (is_string($prefs)) {
+					$prefs = unserialize($GLOBALS['visiteur_session']['prefs']);
+		}
 		if (
 			((isset($prefs['theme']) AND $theme = $prefs['theme'])
 			OR (isset($GLOBALS['theme_prive_defaut']) AND $theme = $GLOBALS['theme_prive_defaut']))
-			AND $theme != _SPIP_THEME_PRIVE)
-			array_unshift($themes,$theme); // placer le theme choisi en tete
+			AND $theme != _SPIP_THEME_PRIVE) {
+					array_unshift($themes,$theme);
+		}
+		// placer le theme choisi en tete
 	}
 	return $themes;
 }
 
 function find_in_theme($file, $subdir='', $include=false){
 	static $themefiles=array();
-	if (isset($themefiles["$subdir$file"])) return $themefiles["$subdir$file"];
+	if (isset($themefiles["$subdir$file"])) {
+	    return $themefiles["$subdir$file"];
+	}
 	$themes = lister_themes_prives();
 	foreach($themes as $theme){
-		if ($f = find_in_path($file,"prive/themes/$theme/$subdir",$include))
-			return $themefiles["$subdir$file"] = $f;
+		if ($f = find_in_path($file,"prive/themes/$theme/$subdir",$include)) {
+					return $themefiles["$subdir$file"] = $f;
+		}
 	}
 	spip_log("$file introuvable dans le theme prive ".reset($themes),'theme');
 	return $themefiles["$subdir$file"] = "";
@@ -928,18 +1019,23 @@ 
 function chemin_image($icone){
 	static $icone_renommer;
 	// gerer le cas d'un double appel en evitant de refaire le travail inutilement
-	if (strpos($icone,"/")!==false AND file_exists($icone)) return $icone;
+	if (strpos($icone,"/")!==false AND file_exists($icone)) {
+	    return $icone;
+	}
 	
 	// si c'est un nom d'image complet (article-24.png) essayer de le renvoyer direct
-	if (preg_match(',[.](png|gif|jpg)$,',$icone) AND $f = find_in_theme("images/$icone"))
-		return $f;
+	if (preg_match(',[.](png|gif|jpg)$,',$icone) AND $f = find_in_theme("images/$icone")) {
+			return $f;
+	}
 	// sinon passer par le module de renommage
-	if (is_null($icone_renommer))
-		$icone_renommer = charger_fonction('icone_renommer','inc',true);
+	if (is_null($icone_renommer)) {
+			$icone_renommer = charger_fonction('icone_renommer','inc',true);
+	}
 	if ($icone_renommer){
 		list($icone,$fonction) = $icone_renommer($icone,"");
-		if (file_exists($icone))
-			return $icone;
+		if (file_exists($icone)) {
+					return $icone;
+		}
 	}
 	return find_in_path ($icone, _NOM_IMG_PACK);
 }
@@ -965,8 +1061,9 @@ 
 	}
 
 	if (isset($GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file])) {
-		if (!$GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file])
-			return false;
+		if (!$GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file]) {
+					return false;
+		}
 		if ($include AND !isset($inc[$dirname][$file])) {
 			include_once _ROOT_CWD . $GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file];
 			$inc[$dirname][$file] = $inc[''][$dirname . $file] = true;
@@ -981,8 +1078,9 @@ 
 	}
 
 	foreach(creer_chemin() as $dir) {
-		if (!isset($dirs[$a = $dir . $dirname]))
-			$dirs[$a] = (is_dir(_ROOT_CWD . $a) || !$a) ;
+		if (!isset($dirs[$a = $dir . $dirname])) {
+					$dirs[$a] = (is_dir(_ROOT_CWD . $a) || !$a) ;
+		}
 		if ($dirs[$a]) {
 			if (file_exists(_ROOT_CWD . ($a .= $file))) {
 				if ($include AND !isset($inc[$dirname][$file])) {
@@ -991,7 +1089,9 @@ 
 				}
 				if (!defined('_SAUVER_CHEMIN')){
 					// si le chemin n'a pas encore ete charge, ne pas lever le flag, ne pas cacher
-					if (is_null($GLOBALS['path_files'])) return $a;
+					if (is_null($GLOBALS['path_files'])) {
+					    return $a;
+					}
 					define('_SAUVER_CHEMIN', true);
 				}
 				return $GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file] = $GLOBALS['path_files'][$GLOBALS['path_sig']][''][$dirname . $file] = $a;
@@ -1004,8 +1104,9 @@ 
 		if ($include==='required'){
 			echo '<pre>',
 			"<strong>Erreur Fatale</strong><br />";
-			if (function_exists('debug_print_backtrace'))
-				echo debug_print_backtrace();
+			if (function_exists('debug_print_backtrace')) {
+							echo debug_print_backtrace();
+			}
 			echo '</pre>';
 			die("Erreur interne: ne peut inclure $dirname$file");
 		}
@@ -1013,7 +1114,9 @@ 
 
 	if (!defined('_SAUVER_CHEMIN')){
 		// si le chemin n'a pas encore ete charge, ne pas lever le flag, ne pas cacher
-		if (is_null($GLOBALS['path_files'])) return false;
+		if (is_null($GLOBALS['path_files'])) {
+		    return false;
+		}
 		define('_SAUVER_CHEMIN', true);
 	}
 	return $GLOBALS['path_files'][$GLOBALS['path_sig']][$dirname][$file] = $GLOBALS['path_files'][$GLOBALS['path_sig']][''][$dirname . $file] = false;
@@ -1045,8 +1148,9 @@ 
 			// mais si semble corrompu on relit avec un verrou
 			if (!$GLOBALS['path_files']=unserialize($contenu)){
 				lire_fichier(_CACHE_CHEMIN,$contenu);
-				if (!$GLOBALS['path_files']=unserialize($contenu))
-					$GLOBALS['path_files'] = array();
+				if (!$GLOBALS['path_files']=unserialize($contenu)) {
+									$GLOBALS['path_files'] = array();
+				}
 			}
 		}
 	}
@@ -1054,9 +1158,10 @@ 
 
 function save_path_cache(){
 	if (defined('_SAUVER_CHEMIN')
-		AND _SAUVER_CHEMIN)
-		ecrire_fichier(_CACHE_CHEMIN,serialize($GLOBALS['path_files']));
-}
+		AND _SAUVER_CHEMIN) {
+			ecrire_fichier(_CACHE_CHEMIN,serialize($GLOBALS['path_files']));
+	}
+	}
 
 
 /**
@@ -1084,8 +1189,9 @@ 
 				// ne prendre que les fichiers pas deja trouves
 				// car find_in_path prend le premier qu'il trouve,
 				// les autres sont donc masques
-				if (!isset($liste_fichiers[$nom]))
-					$liste_fichiers[$nom] = $chemin;
+				if (!isset($liste_fichiers[$nom])) {
+									$liste_fichiers[$nom] = $chemin;
+				}
 			}
 		}
 	}
@@ -1128,13 +1234,18 @@ 
  */
 function generer_url_entite($id='', $entite='', $args='', $ancre='', $public=NULL, $type=NULL)
 {
-	if ($public === NULL) $public = !test_espace_prive();
+	if ($public === NULL) {
+	    $public = !test_espace_prive();
+	}
 	$entite = objet_type($entite); // cas particulier d'appels sur objet/id_objet...
 
 	if (!$public) {
-		if (!$entite) return '';
-		if (!function_exists('generer_url_ecrire_objet'))
-			include_spip('inc/urls');
+		if (!$entite) {
+		    return '';
+		}
+		if (!function_exists('generer_url_ecrire_objet')) {
+					include_spip('inc/urls');
+		}
 		$res = generer_url_ecrire_objet($entite,$id, $args, $ancre, false);
 	} else {
 		if ($type === NULL) {
@@ -1146,32 +1257,42 @@ 
 
 		$f = charger_fonction($type, 'urls', true);
 		// se rabattre sur les urls page si les urls perso non dispo
-		if (!$f) $f = charger_fonction('page', 'urls', true);
+		if (!$f) {
+		    $f = charger_fonction('page', 'urls', true);
+		}
 
 		// si $entite='', on veut la fonction de passage URL ==> id
 		// sinon on veut effectuer le passage id ==> URL
-		if (!$entite) return $f;
+		if (!$entite) {
+		    return $f;
+		}
 
 		// mais d'abord il faut tester le cas des urls sur une
 		// base distante
 		if (is_string($public)
-		AND $g = charger_fonction('connect', 'urls', true))
-			$f = $g;
+		AND $g = charger_fonction('connect', 'urls', true)) {
+					$f = $g;
+		}
 
 		$res = $f(intval($id), $entite, $args, $ancre, $public);
 
 	}
-	if ($res) return $res;
+	if ($res) {
+	    return $res;
+	}
 	// Sinon c'est un raccourci ou compat SPIP < 2
 	if (!function_exists($f = 'generer_url_' . $entite)) {
-		if (!function_exists($f .= '_dist')) $f = '';
+		if (!function_exists($f .= '_dist')) {
+		    $f = '';
+		}
 	}
 	if ($f) {
 		$url = $f($id, $args, $ancre);
-		if (strlen($args))
-			$url .= strstr($url, '?')
+		if (strlen($args)) {
+					$url .= strstr($url, '?')
 				? '&amp;'.$args
 				: '?'.$args;
+		}
 		return $url;
 	}
 	// On a ete gentil mais la ....
@@ -1182,12 +1303,14 @@ 
 function generer_url_ecrire_entite_edit($id, $entite, $args='', $ancre=''){
 	$exec = objet_info($entite,'url_edit');
 	$url = generer_url_ecrire($exec,$args);
-	if (intval($id))
-		$url = parametre_url($url,id_table_objet($entite),$id);
-	else
-		$url = parametre_url($url,'new','oui');
-	if ($ancre)
-		$url = ancre_url($url,$ancre);
+	if (intval($id)) {
+			$url = parametre_url($url,id_table_objet($entite),$id);
+	} else {
+			$url = parametre_url($url,'new','oui');
+	}
+	if ($ancre) {
+			$url = ancre_url($url,$ancre);
+	}
 	return $url;
 }
 
@@ -1207,8 +1330,9 @@ 
 	if (preg_match(',[^\x00-\x7E],sS', $url)){
 		$uri = '';
 		for ($i=0; $i < strlen($url); $i++) {
-			if (ord($a = $url[$i]) > 127)
-				$a = rawurlencode($a);
+			if (ord($a = $url[$i]) > 127) {
+							$a = rawurlencode($a);
+			}
 			$uri .= $a;
 		}
 		$url = $uri;
@@ -1219,7 +1343,9 @@ 
 // http://doc.spip.org/@generer_url_entite_absolue
 function generer_url_entite_absolue($id='', $entite='', $args='', $ancre='', $connect=NULL)
 {
-	if (!$connect) $connect = true;
+	if (!$connect) {
+	    $connect = true;
+	}
 	$h = generer_url_entite($id, $entite, $args, $ancre, $connect);
 	if (!preg_match(',^\w+:,', $h)) {
 		include_spip('inc/filtres_mini');
@@ -1232,7 +1358,9 @@ 
 // variables d'environnement comme $_SERVER[HTTPS] ou ini_get(register_globals)
 // http://doc.spip.org/@test_valeur_serveur
 function test_valeur_serveur($truc) {
-	if (!$truc) return false;
+	if (!$truc) {
+	    return false;
+	}
 	return (strtolower($truc) !== 'off');
 }
 
@@ -1259,13 +1387,20 @@ 
 function url_de_base($profondeur=null) {
 
 	static $url = array();
-	if (is_array($profondeur)) return $url = $profondeur;
-	if ($profondeur===false) return $url;
+	if (is_array($profondeur)) {
+	    return $url = $profondeur;
+	}
+	if ($profondeur===false) {
+	    return $url;
+	}
 
-	if (is_null($profondeur)) $profondeur = $GLOBALS['profondeur_url'];
+	if (is_null($profondeur)) {
+	    $profondeur = $GLOBALS['profondeur_url'];
+	}
 
-	if (isset($url[$profondeur]))
-		return $url[$profondeur];
+	if (isset($url[$profondeur])) {
+			return $url[$profondeur];
+	}
 
 	$http = (
 		(isset($_SERVER["SCRIPT_URI"]) AND
@@ -1286,8 +1421,12 @@ 
 	if (isset($_SERVER['SERVER_PORT'])
 		AND $port=$_SERVER['SERVER_PORT']
 		AND strpos($host,":")==false){
-		if ($http=="http" AND $port!=80) $host.=":$port";
-		if ($http=="https" AND $port!=443) $host.=":$port";
+		if ($http=="http" AND $port!=80) {
+		    $host.=":$port";
+		}
+		if ($http=="https" AND $port!=443) {
+		    $host.=":$port";
+		}
 	}
 	if (!$GLOBALS['REQUEST_URI']){
 		if (isset($_SERVER['REQUEST_URI'])) {
@@ -1295,8 +1434,9 @@ 
 		} else {
 			$GLOBALS['REQUEST_URI'] = $_SERVER['PHP_SELF'];
 			if ($_SERVER['QUERY_STRING']
-			AND !strpos($_SERVER['REQUEST_URI'], '?'))
-				$GLOBALS['REQUEST_URI'] .= '?'.$_SERVER['QUERY_STRING'];
+			AND !strpos($_SERVER['REQUEST_URI'], '?')) {
+							$GLOBALS['REQUEST_URI'] .= '?'.$_SERVER['QUERY_STRING'];
+			}
 		}
 	}
 
@@ -1345,18 +1485,22 @@ 
 
 // http://doc.spip.org/@generer_url_ecrire
 function generer_url_ecrire($script='', $args="", $no_entities=false, $rel=false) {
-	if (!$rel)
-		$rel = url_de_base() . _DIR_RESTREINT_ABS . _SPIP_ECRIRE_SCRIPT;
-	else if (!is_string($rel))
-		$rel = _DIR_RESTREINT ? _DIR_RESTREINT :
+	if (!$rel) {
+			$rel = url_de_base() . _DIR_RESTREINT_ABS . _SPIP_ECRIRE_SCRIPT;
+	} else if (!is_string($rel)) {
+			$rel = _DIR_RESTREINT ? _DIR_RESTREINT :
 			('./'  . _SPIP_ECRIRE_SCRIPT);
+	}
 
 	@list($script, $ancre) = explode('#', $script);
-	if ($script AND ($script<>'accueil' OR $rel))
-		$args = "?exec=$script" . (!$args ? '' : "&$args");
-	elseif ($args)
-		$args ="?$args";
-	if ($ancre) $args .= "#$ancre";
+	if ($script AND ($script<>'accueil' OR $rel)) {
+			$args = "?exec=$script" . (!$args ? '' : "&$args");
+	} elseif ($args) {
+			$args ="?$args";
+	}
+	if ($ancre) {
+	    $args .= "#$ancre";
+	}
 	return $rel . ($no_entities ? $args : str_replace('&', '&amp;', $args));
 }
 
@@ -1375,32 +1519,39 @@ 
 // http://doc.spip.org/@get_spip_script
 function get_spip_script($default='') {
 	# cas define('_SPIP_SCRIPT', '');
-	if (_SPIP_SCRIPT)
-		return _SPIP_SCRIPT;
-	else
-		return $default;
-}
+	if (_SPIP_SCRIPT) {
+			return _SPIP_SCRIPT;
+	} else {
+			return $default;
+	}
+	}
 
 // http://doc.spip.org/@generer_url_public
 function generer_url_public($script='', $args="", $no_entities=false, $rel=true, $action='') {
 	// si le script est une action (spip_pass, spip_inscription),
 	// standardiser vers la nouvelle API
 
-	if (!$action) $action = get_spip_script();
-	if ($script)
-		$action = parametre_url($action, _SPIP_PAGE, $script, '&');
+	if (!$action) {
+	    $action = get_spip_script();
+	}
+	if ($script) {
+			$action = parametre_url($action, _SPIP_PAGE, $script, '&');
+	}
 
 	if ($args) {
 		if (is_array($args)) {
 			$r = '';
-			foreach($args as $k => $v) $r .= '&' . $k . '=' . $v;
+			foreach($args as $k => $v) {
+			    $r .= '&' . $k . '=' . $v;
+			}
 			$args = substr($r,1);
 		}
 		$action .=
 			(strpos($action, '?') !== false ? '&' : '?') . $args;
 	}
-	if (!$no_entities)
-		$action = quote_amp($action);
+	if (!$no_entities) {
+			$action = quote_amp($action);
+	}
 
 	// ne pas generer une url avec /./?page= en cas d'url absolue et de _SPIP_SCRIPT vide
 	return ($rel ? _DIR_RACINE . $action : rtrim(url_de_base(),'/') . preg_replace(",^/[.]/,","/","/$action"));
@@ -1478,9 +1629,13 @@ 
 	  ? generer_url_ecrire(_request('exec'))
 	  :  generer_url_public('','',false,false);
 	$url = parametre_url($url,'action',$script);
-	if ($args) $url .= quote_amp('&'.$args);
+	if ($args) {
+	    $url .= quote_amp('&'.$args);
+	}
 
-	if ($no_entities) $url = str_replace('&amp;','&',$url);
+	if ($no_entities) {
+	    $url = str_replace('&amp;','&',$url);
+	}
 	return $url;
 }
 
@@ -1514,107 +1669,204 @@ 
  */
 function spip_initialisation_core($pi=NULL, $pa=NULL, $ti=NULL, $ta=NULL) {
 	static $too_late = 0;
-	if ($too_late++) return;
+	if ($too_late++) {
+	    return;
+	}
 	
 	// Declaration des repertoires
 
 	// le nom du repertoire plugins/ activables/desactivables
-	if (!defined('_DIR_PLUGINS')) define('_DIR_PLUGINS', _DIR_RACINE . "plugins/");
+	if (!defined('_DIR_PLUGINS')) {
+	    define('_DIR_PLUGINS', _DIR_RACINE . "plugins/");
+	}
 
 	// le nom du repertoire des extensions/ permanentes du core, toujours actives
-	if (!defined('_DIR_PLUGINS_DIST')) define('_DIR_PLUGINS_DIST', _DIR_RACINE . "plugins-dist/");
+	if (!defined('_DIR_PLUGINS_DIST')) {
+	    define('_DIR_PLUGINS_DIST', _DIR_RACINE . "plugins-dist/");
+	}
 
 	// le nom du repertoire des librairies
-	if (!defined('_DIR_LIB')) define('_DIR_LIB', _DIR_RACINE . "lib/");
+	if (!defined('_DIR_LIB')) {
+	    define('_DIR_LIB', _DIR_RACINE . "lib/");
+	}
 	
-	if (!defined('_DIR_IMG')) define('_DIR_IMG', $pa);
-	if (!defined('_DIR_LOGOS')) define('_DIR_LOGOS', $pa);
-	if (!defined('_DIR_IMG_ICONES')) define('_DIR_IMG_ICONES', _DIR_LOGOS . "icones/");
-
-	if (!defined('_DIR_DUMP')) define('_DIR_DUMP', $ti . "dump/");
-	if (!defined('_DIR_SESSIONS')) define('_DIR_SESSIONS', $ti . "sessions/");
-	if (!defined('_DIR_TRANSFERT')) define('_DIR_TRANSFERT', $ti . "upload/");
-	if (!defined('_DIR_CACHE')) define('_DIR_CACHE', $ti . "cache/");
-	if (!defined('_DIR_CACHE_XML')) define('_DIR_CACHE_XML', _DIR_CACHE . "xml/");
-	if (!defined('_DIR_SKELS')) define('_DIR_SKELS',  _DIR_CACHE . "skel/");
-	if (!defined('_DIR_AIDE')) define('_DIR_AIDE',  _DIR_CACHE . "aide/");
-	if (!defined('_DIR_TMP')) define('_DIR_TMP', $ti);
-
-	if (!defined('_DIR_VAR')) define('_DIR_VAR', $ta);
-
-	if (!defined('_DIR_ETC')) define('_DIR_ETC', $pi);
-	if (!defined('_DIR_CONNECT')) define('_DIR_CONNECT', $pi);
-	if (!defined('_DIR_CHMOD')) define('_DIR_CHMOD', $pi);
-
-	if (!isset($GLOBALS['test_dirs']))
-	  // Pas $pi car il est bon de le mettre hors ecriture apres intstall
+	if (!defined('_DIR_IMG')) {
+	    define('_DIR_IMG', $pa);
+	}
+	if (!defined('_DIR_LOGOS')) {
+	    define('_DIR_LOGOS', $pa);
+	}
+	if (!defined('_DIR_IMG_ICONES')) {
+	    define('_DIR_IMG_ICONES', _DIR_LOGOS . "icones/");
+	}
+
+	if (!defined('_DIR_DUMP')) {
+	    define('_DIR_DUMP', $ti . "dump/");
+	}
+	if (!defined('_DIR_SESSIONS')) {
+	    define('_DIR_SESSIONS', $ti . "sessions/");
+	}
+	if (!defined('_DIR_TRANSFERT')) {
+	    define('_DIR_TRANSFERT', $ti . "upload/");
+	}
+	if (!defined('_DIR_CACHE')) {
+	    define('_DIR_CACHE', $ti . "cache/");
+	}
+	if (!defined('_DIR_CACHE_XML')) {
+	    define('_DIR_CACHE_XML', _DIR_CACHE . "xml/");
+	}
+	if (!defined('_DIR_SKELS')) {
+	    define('_DIR_SKELS',  _DIR_CACHE . "skel/");
+	}
+	if (!defined('_DIR_AIDE')) {
+	    define('_DIR_AIDE',  _DIR_CACHE . "aide/");
+	}
+	if (!defined('_DIR_TMP')) {
+	    define('_DIR_TMP', $ti);
+	}
+
+	if (!defined('_DIR_VAR')) {
+	    define('_DIR_VAR', $ta);
+	}
+
+	if (!defined('_DIR_ETC')) {
+	    define('_DIR_ETC', $pi);
+	}
+	if (!defined('_DIR_CONNECT')) {
+	    define('_DIR_CONNECT', $pi);
+	}
+	if (!defined('_DIR_CHMOD')) {
+	    define('_DIR_CHMOD', $pi);
+	}
+
+	if (!isset($GLOBALS['test_dirs'])) {
+		  // Pas $pi car il est bon de le mettre hors ecriture apres intstall
 	  // il sera rajoute automatiquement si besoin a l'etape 2 de l'install
 		$GLOBALS['test_dirs'] =  array($pa, $ti, $ta);
+	}
 
 	// Declaration des fichiers
 
-	if (!defined('_CACHE_PLUGINS_PATH')) define('_CACHE_PLUGINS_PATH', _DIR_CACHE . "charger_plugins_chemins.php");
-	if (!defined('_CACHE_PLUGINS_OPT')) define('_CACHE_PLUGINS_OPT', _DIR_CACHE . "charger_plugins_options.php");
-	if (!defined('_CACHE_PLUGINS_FCT')) define('_CACHE_PLUGINS_FCT', _DIR_CACHE . "charger_plugins_fonctions.php");
-	if (!defined('_CACHE_PIPELINES')) define('_CACHE_PIPELINES',  _DIR_CACHE."charger_pipelines.php");
-	if (!defined('_CACHE_CHEMIN')) define('_CACHE_CHEMIN',  _DIR_CACHE."chemin.txt");
+	if (!defined('_CACHE_PLUGINS_PATH')) {
+	    define('_CACHE_PLUGINS_PATH', _DIR_CACHE . "charger_plugins_chemins.php");
+	}
+	if (!defined('_CACHE_PLUGINS_OPT')) {
+	    define('_CACHE_PLUGINS_OPT', _DIR_CACHE . "charger_plugins_options.php");
+	}
+	if (!defined('_CACHE_PLUGINS_FCT')) {
+	    define('_CACHE_PLUGINS_FCT', _DIR_CACHE . "charger_plugins_fonctions.php");
+	}
+	if (!defined('_CACHE_PIPELINES')) {
+	    define('_CACHE_PIPELINES',  _DIR_CACHE."charger_pipelines.php");
+	}
+	if (!defined('_CACHE_CHEMIN')) {
+	    define('_CACHE_CHEMIN',  _DIR_CACHE."chemin.txt");
+	}
 
 	# attention .php obligatoire pour ecrire_fichier_securise
-	if (!defined('_FILE_META')) define('_FILE_META', $ti . 'meta_cache.php');
-	if (!defined('_DIR_LOG')) define('_DIR_LOG', _DIR_TMP . 'log/');
-	if (!defined('_FILE_LOG')) define('_FILE_LOG', 'spip');
-	if (!defined('_FILE_LOG_SUFFIX')) define('_FILE_LOG_SUFFIX', '.log');
+	if (!defined('_FILE_META')) {
+	    define('_FILE_META', $ti . 'meta_cache.php');
+	}
+	if (!defined('_DIR_LOG')) {
+	    define('_DIR_LOG', _DIR_TMP . 'log/');
+	}
+	if (!defined('_FILE_LOG')) {
+	    define('_FILE_LOG', 'spip');
+	}
+	if (!defined('_FILE_LOG_SUFFIX')) {
+	    define('_FILE_LOG_SUFFIX', '.log');
+	}
 
 	// Le fichier de connexion a la base de donnees
 	// tient compte des anciennes versions (inc_connect...)
-	if (!defined('_FILE_CONNECT_INS')) define('_FILE_CONNECT_INS', 'connect');
-	if (!defined('_FILE_CONNECT')) define('_FILE_CONNECT',
+	if (!defined('_FILE_CONNECT_INS')) {
+	    define('_FILE_CONNECT_INS', 'connect');
+	}
+	if (!defined('_FILE_CONNECT')) {
+	    define('_FILE_CONNECT',
 		(@is_readable($f = _DIR_CONNECT . _FILE_CONNECT_INS . '.php') ? $f
 	:	(@is_readable($f = _DIR_RESTREINT . 'inc_connect.php') ? $f
 	:	false)));
+	}
 
 	// Le fichier de reglages des droits
-	if (!defined('_FILE_CHMOD_INS')) define('_FILE_CHMOD_INS', 'chmod');
-	if (!defined('_FILE_CHMOD')) define('_FILE_CHMOD',
+	if (!defined('_FILE_CHMOD_INS')) {
+	    define('_FILE_CHMOD_INS', 'chmod');
+	}
+	if (!defined('_FILE_CHMOD')) {
+	    define('_FILE_CHMOD',
 		(@is_readable($f = _DIR_CHMOD . _FILE_CHMOD_INS . '.php') ? $f
 	:	false));
+	}
 
-	if (!defined('_FILE_LDAP')) define('_FILE_LDAP', 'ldap.php');
+	if (!defined('_FILE_LDAP')) {
+	    define('_FILE_LDAP', 'ldap.php');
+	}
 
-	if (!defined('_FILE_TMP_SUFFIX')) define('_FILE_TMP_SUFFIX', '.tmp.php');
-	if (!defined('_FILE_CONNECT_TMP')) define('_FILE_CONNECT_TMP', _DIR_CONNECT . _FILE_CONNECT_INS . _FILE_TMP_SUFFIX);
-	if (!defined('_FILE_CHMOD_TMP')) define('_FILE_CHMOD_TMP', _DIR_CHMOD . _FILE_CHMOD_INS . _FILE_TMP_SUFFIX);
+	if (!defined('_FILE_TMP_SUFFIX')) {
+	    define('_FILE_TMP_SUFFIX', '.tmp.php');
+	}
+	if (!defined('_FILE_CONNECT_TMP')) {
+	    define('_FILE_CONNECT_TMP', _DIR_CONNECT . _FILE_CONNECT_INS . _FILE_TMP_SUFFIX);
+	}
+	if (!defined('_FILE_CHMOD_TMP')) {
+	    define('_FILE_CHMOD_TMP', _DIR_CHMOD . _FILE_CHMOD_INS . _FILE_TMP_SUFFIX);
+	}
 
 	// Definition des droits d'acces en ecriture
-	if (!defined('_SPIP_CHMOD') AND _FILE_CHMOD)
-		include_once _FILE_CHMOD;
+	if (!defined('_SPIP_CHMOD') AND _FILE_CHMOD) {
+			include_once _FILE_CHMOD;
+	}
 
 	// Se mefier des fichiers mal remplis!
-	if (!defined('_SPIP_CHMOD')) define('_SPIP_CHMOD', 0777);
+	if (!defined('_SPIP_CHMOD')) {
+	    define('_SPIP_CHMOD', 0777);
+	}
 
 	// Le charset par defaut lors de l'installation
-	if (!defined('_DEFAULT_CHARSET')) define('_DEFAULT_CHARSET', 'utf-8');
-	if (!defined('_ROOT_PLUGINS')) define('_ROOT_PLUGINS', _ROOT_RACINE . "plugins/");
-	if (!defined('_ROOT_PLUGINS_DIST')) define('_ROOT_PLUGINS_DIST', _ROOT_RACINE . "plugins-dist/");
-	if (!defined('_ROOT_PLUGINS_SUPPL') && defined('_DIR_PLUGINS_SUPPL') && _DIR_PLUGINS_SUPPL) define('_ROOT_PLUGINS_SUPPL', _ROOT_RACINE . str_replace(_DIR_RACINE,'',_DIR_PLUGINS_SUPPL));
+	if (!defined('_DEFAULT_CHARSET')) {
+	    define('_DEFAULT_CHARSET', 'utf-8');
+	}
+	if (!defined('_ROOT_PLUGINS')) {
+	    define('_ROOT_PLUGINS', _ROOT_RACINE . "plugins/");
+	}
+	if (!defined('_ROOT_PLUGINS_DIST')) {
+	    define('_ROOT_PLUGINS_DIST', _ROOT_RACINE . "plugins-dist/");
+	}
+	if (!defined('_ROOT_PLUGINS_SUPPL') && defined('_DIR_PLUGINS_SUPPL') && _DIR_PLUGINS_SUPPL) {
+	    define('_ROOT_PLUGINS_SUPPL', _ROOT_RACINE . str_replace(_DIR_RACINE,'',_DIR_PLUGINS_SUPPL));
+	}
 
 	// La taille des Log
-	if (!defined('_MAX_LOG')) define('_MAX_LOG', 100);
+	if (!defined('_MAX_LOG')) {
+	    define('_MAX_LOG', 100);
+	}
 
 	// Sommes-nous dans l'empire du Mal ?
 	// (ou sous le signe du Pingouin, ascendant GNU ?)
 	if (strpos($_SERVER['SERVER_SOFTWARE'], '(Win') !== false){
-		if (!defined('_OS_SERVEUR')) define('_OS_SERVEUR', 'windows');
-		if (!defined('_SPIP_LOCK_MODE')) define('_SPIP_LOCK_MODE',1); // utiliser le flock php
-	}
-	else {
-		if (!defined('_OS_SERVEUR')) define('_OS_SERVEUR', '');
-		if (!defined('_SPIP_LOCK_MODE')) define('_SPIP_LOCK_MODE',1); // utiliser le flock php
+		if (!defined('_OS_SERVEUR')) {
+		    define('_OS_SERVEUR', 'windows');
+		}
+		if (!defined('_SPIP_LOCK_MODE')) {
+		    define('_SPIP_LOCK_MODE',1);
+		}
+		// utiliser le flock php
+	} else {
+		if (!defined('_OS_SERVEUR')) {
+		    define('_OS_SERVEUR', '');
+		}
+		if (!defined('_SPIP_LOCK_MODE')) {
+		    define('_SPIP_LOCK_MODE',1);
+		}
+		// utiliser le flock php
 		#if (!defined('_SPIP_LOCK_MODE')) define('_SPIP_LOCK_MODE',2); // utiliser le nfslock de spip mais link() est tres souvent interdite
 	}
 
 	// Langue par defaut
-	if (!defined('_LANGUE_PAR_DEFAUT')) define('_LANGUE_PAR_DEFAUT','fr');
+	if (!defined('_LANGUE_PAR_DEFAUT')) {
+	    define('_LANGUE_PAR_DEFAUT','fr');
+	}
 
 	// PHP_VERSION_ID dispo depuis PHP 5.2.7
 	if (!defined('PHP_VERSION_ID')) {
@@ -1639,7 +1891,9 @@ 
 	//
 
 	// Ne pas se faire manger par un bug php qui accepte ?GLOBALS[truc]=toto
-	if (isset($_REQUEST['GLOBALS'])) die();
+	if (isset($_REQUEST['GLOBALS'])) {
+	    die();
+	}
 	// nettoyer les magic quotes \' et les caracteres nuls %00
 	spip_desinfecte($_GET);
 	spip_desinfecte($_POST);
@@ -1658,8 +1912,9 @@ 
 		// ne pas desinfecter les globales en profondeur car elle contient aussi les
 		// precedentes, qui seraient desinfectees 2 fois.
 		spip_desinfecte($GLOBALS,false);
-		if (include_spip('inc/php3'))
-			spip_register_globals(true);
+		if (include_spip('inc/php3')) {
+					spip_register_globals(true);
+		}
 
 		$avertir_register_globals = true;
 	}
@@ -1688,12 +1943,15 @@ 
 	} else {
 		$GLOBALS['REQUEST_URI'] = $_SERVER['PHP_SELF'];
 		if ($_SERVER['QUERY_STRING']
-		AND !strpos($_SERVER['REQUEST_URI'], '?'))
-			$GLOBALS['REQUEST_URI'] .= '?'.$_SERVER['QUERY_STRING'];
+		AND !strpos($_SERVER['REQUEST_URI'], '?')) {
+					$GLOBALS['REQUEST_URI'] .= '?'.$_SERVER['QUERY_STRING'];
+		}
 	}
 
 	// Duree de validite de l'alea pour les cookies et ce qui s'ensuit.
-	if (!defined('_RENOUVELLE_ALEA')) define('_RENOUVELLE_ALEA', 12 * 3600);
+	if (!defined('_RENOUVELLE_ALEA')) {
+	    define('_RENOUVELLE_ALEA', 12 * 3600);
+	}
 
 	// charger les meta si possible et renouveller l'alea au besoin
 	// charge aussi effacer_meta et ecrire_meta
@@ -1701,16 +1959,17 @@ 
 	$inc_meta();
 
 	// on a pas pu le faire plus tot
-	if  ($avertir_register_globals)
-		avertir_auteurs("register_globals",_L("Probl&egrave;me de s&eacute;curit&eacute; : register_globals=on; dans php.ini &agrave; corriger."));
+	if  ($avertir_register_globals) {
+			avertir_auteurs("register_globals",_L("Probl&egrave;me de s&eacute;curit&eacute; : register_globals=on; dans php.ini &agrave; corriger."));
+	}
 
 	// nombre de repertoires depuis la racine
 	// on compare a l'adresse de spip.php : $_SERVER["SCRIPT_NAME"]
 	// ou a defaut celle donnee en meta ; (mais si celle-ci est fausse
 	// le calcul est faux)
-	if (!_DIR_RESTREINT)
-		$GLOBALS['profondeur_url'] = 1;
-	else {
+	if (!_DIR_RESTREINT) {
+			$GLOBALS['profondeur_url'] = 1;
+	} else {
 		$uri = isset($_SERVER['REQUEST_URI']) ? explode('?', $_SERVER['REQUEST_URI']) : '';
 		$uri_ref = $_SERVER["SCRIPT_NAME"];
 		if (!$uri_ref
@@ -1723,13 +1982,13 @@ 
 			if (isset($GLOBALS['meta']['adresse_site'])) {
 				$uri_ref = parse_url($GLOBALS['meta']['adresse_site']);
 				$uri_ref = $uri_ref['path'].'/';
-			}
-		  else
-			  $uri_ref = "";
+			} else {
+		  			  $uri_ref = "";
+		  }
 		}
-		if (!$uri OR !$uri_ref)
-			$GLOBALS['profondeur_url'] = 0;
-		else {
+		if (!$uri OR !$uri_ref) {
+					$GLOBALS['profondeur_url'] = 0;
+		} else {
 			$GLOBALS['profondeur_url'] = max(0,
 				substr_count($uri[0], '/')
 				- substr_count($uri_ref,'/'));
@@ -1739,8 +1998,9 @@ 
 	if (_FILE_CONNECT) {
 		if (verifier_visiteur()=='0minirezo'
 			// si c'est un admin sans cookie admin, il faut ignorer le cache chemin !
-		  AND !isset($_COOKIE['spip_admin']))
-			clear_path_cache();
+		  AND !isset($_COOKIE['spip_admin'])) {
+					clear_path_cache();
+		}
 	}
 
 }
@@ -1752,74 +2012,140 @@ 
  */
 function spip_initialisation_suite() {
 	static $too_late = 0;
-	if ($too_late++) return;
+	if ($too_late++) {
+	    return;
+	}
 
 	// taille mini des login
-	if (!defined('_LOGIN_TROP_COURT')) define('_LOGIN_TROP_COURT', 4);
+	if (!defined('_LOGIN_TROP_COURT')) {
+	    define('_LOGIN_TROP_COURT', 4);
+	}
 
 	// la taille maxi des logos (0 : pas de limite)
-	if (!defined('_LOGO_MAX_SIZE')) define('_LOGO_MAX_SIZE', 0); # poids en ko
-	if (!defined('_LOGO_MAX_WIDTH')) define('_LOGO_MAX_WIDTH', 0); # largeur en pixels
-	if (!defined('_LOGO_MAX_HEIGHT')) define('_LOGO_MAX_HEIGHT', 0); # hauteur en pixels
+	if (!defined('_LOGO_MAX_SIZE')) {
+	    define('_LOGO_MAX_SIZE', 0);
+	}
+	# poids en ko
+	if (!defined('_LOGO_MAX_WIDTH')) {
+	    define('_LOGO_MAX_WIDTH', 0);
+	}
+	# largeur en pixels
+	if (!defined('_LOGO_MAX_HEIGHT')) {
+	    define('_LOGO_MAX_HEIGHT', 0);
+	}
+	# hauteur en pixels
 
-	if (!defined('_DOC_MAX_SIZE')) define('_DOC_MAX_SIZE', 0); # poids en ko
+	if (!defined('_DOC_MAX_SIZE')) {
+	    define('_DOC_MAX_SIZE', 0);
+	}
+	# poids en ko
 
-	if (!defined('_IMG_MAX_SIZE')) define('_IMG_MAX_SIZE', 0); # poids en ko
-	if (!defined('_IMG_MAX_WIDTH')) define('_IMG_MAX_WIDTH', 0); # largeur en pixels
-	if (!defined('_IMG_MAX_HEIGHT')) define('_IMG_MAX_HEIGHT', 0); # hauteur en pixels
-	if (!defined('_PASS_LONGUEUR_MINI')) define('_PASS_LONGUEUR_MINI',6);
+	if (!defined('_IMG_MAX_SIZE')) {
+	    define('_IMG_MAX_SIZE', 0);
+	}
+	# poids en ko
+	if (!defined('_IMG_MAX_WIDTH')) {
+	    define('_IMG_MAX_WIDTH', 0);
+	}
+	# largeur en pixels
+	if (!defined('_IMG_MAX_HEIGHT')) {
+	    define('_IMG_MAX_HEIGHT', 0);
+	}
+	# hauteur en pixels
+	if (!defined('_PASS_LONGUEUR_MINI')) {
+	    define('_PASS_LONGUEUR_MINI',6);
+	}
 
 
 	// Qualite des images calculees automatiquement. C'est un nombre entre 0 et 100, meme pour imagick (on ramene a 0..1 par la suite)
-        if (!defined('_IMG_QUALITE')) define('_IMG_QUALITE', 85); # valeur par defaut
-        if (!defined('_IMG_GD_QUALITE')) define('_IMG_GD_QUALITE', _IMG_QUALITE); # surcharge pour la lib GD
-        if (!defined('_IMG_CONVERT_QUALITE')) define('_IMG_CONVERT_QUALITE', _IMG_QUALITE); # surcharge pour imagick en ligne de commande
+        if (!defined('_IMG_QUALITE')) {
+            define('_IMG_QUALITE', 85);
+        }
+        # valeur par defaut
+        if (!defined('_IMG_GD_QUALITE')) {
+            define('_IMG_GD_QUALITE', _IMG_QUALITE);
+        }
+        # surcharge pour la lib GD
+        if (!defined('_IMG_CONVERT_QUALITE')) {
+            define('_IMG_CONVERT_QUALITE', _IMG_QUALITE);
+        }
+        # surcharge pour imagick en ligne de commande
 	// Historiquement la valeur pour imagick semble differente. Si ca n'est pas necessaire, il serait preferable de garder _IMG_QUALITE
-        if (!defined('_IMG_IMAGICK_QUALITE')) define('_IMG_IMAGICK_QUALITE', 75); # surcharge pour imagick en PHP
+        if (!defined('_IMG_IMAGICK_QUALITE')) {
+            define('_IMG_IMAGICK_QUALITE', 75);
+        }
+        # surcharge pour imagick en PHP
 
-	if (!defined('_COPIE_LOCALE_MAX_SIZE')) define('_COPIE_LOCALE_MAX_SIZE',16777216); // poids en octet
+	if (!defined('_COPIE_LOCALE_MAX_SIZE')) {
+	    define('_COPIE_LOCALE_MAX_SIZE',16777216);
+	}
+	// poids en octet
 
 	// qq chaines standard
-	if (!defined('_ACCESS_FILE_NAME')) define('_ACCESS_FILE_NAME', '.htaccess');
-	if (!defined('_AUTH_USER_FILE')) define('_AUTH_USER_FILE', '.htpasswd');
-	if (!defined('_SPIP_DUMP')) define('_SPIP_DUMP', 'dump@nom_site@@stamp@.xml');
-	if (!defined('_CACHE_RUBRIQUES')) define('_CACHE_RUBRIQUES', _DIR_TMP.'menu-rubriques-cache.txt');
-	if (!defined('_CACHE_RUBRIQUES_MAX')) define('_CACHE_RUBRIQUES_MAX', 500);
+	if (!defined('_ACCESS_FILE_NAME')) {
+	    define('_ACCESS_FILE_NAME', '.htaccess');
+	}
+	if (!defined('_AUTH_USER_FILE')) {
+	    define('_AUTH_USER_FILE', '.htpasswd');
+	}
+	if (!defined('_SPIP_DUMP')) {
+	    define('_SPIP_DUMP', 'dump@nom_site@@stamp@.xml');
+	}
+	if (!defined('_CACHE_RUBRIQUES')) {
+	    define('_CACHE_RUBRIQUES', _DIR_TMP.'menu-rubriques-cache.txt');
+	}
+	if (!defined('_CACHE_RUBRIQUES_MAX')) {
+	    define('_CACHE_RUBRIQUES_MAX', 500);
+	}
 
-	if (!defined('_EXTENSION_SQUELETTES')) define('_EXTENSION_SQUELETTES', 'html');
+	if (!defined('_EXTENSION_SQUELETTES')) {
+	    define('_EXTENSION_SQUELETTES', 'html');
+	}
 
-	if (!defined('_DOCTYPE_ECRIRE')) define('_DOCTYPE_ECRIRE',
+	if (!defined('_DOCTYPE_ECRIRE')) {
+	    define('_DOCTYPE_ECRIRE',
 		// "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN' 'http://www.w3.org/TR/html4/loose.dtd'>\n");
 		//"<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Transitional//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd'>\n");
 		//"<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Strict//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd'>\n");
 	       // "<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.1 //EN' 'http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd'>\n");
 		"<!DOCTYPE html>\n");
-	if (!defined('_DOCTYPE_AIDE')) define('_DOCTYPE_AIDE',
+	}
+	if (!defined('_DOCTYPE_AIDE')) {
+	    define('_DOCTYPE_AIDE',
 	       "<!DOCTYPE html PUBLIC '-//W3C//DTD HTML 4.01 Frameset//EN' 'http://www.w3.org/TR/1999/REC-html401-19991224/frameset.dtd'>");
+	}
 
 	// L'adresse de base du site ; on peut mettre '' si la racine est geree par
 	// le script de l'espace public, alias  index.php
-	if (!defined('_SPIP_SCRIPT')) define('_SPIP_SCRIPT', 'spip.php');
+	if (!defined('_SPIP_SCRIPT')) {
+	    define('_SPIP_SCRIPT', 'spip.php');
+	}
 	// argument page, personalisable en cas de conflit avec un autre script
-	if (!defined('_SPIP_PAGE')) define('_SPIP_PAGE', 'page');
+	if (!defined('_SPIP_PAGE')) {
+	    define('_SPIP_PAGE', 'page');
+	}
 
 	// le script de l'espace prive
 	// Mettre a "index.php" si DirectoryIndex ne le fait pas ou pb connexes:
 	// les anciens IIS n'acceptent pas les POST sur ecrire/ (#419)
 	// meme pb sur thttpd cf. http://forum.spip.org/fr_184153.html
 
-	if (!defined('_SPIP_ECRIRE_SCRIPT')) define('_SPIP_ECRIRE_SCRIPT', // true ? #decommenter ici et commenter la
+	if (!defined('_SPIP_ECRIRE_SCRIPT')) {
+	    define('_SPIP_ECRIRE_SCRIPT', // true ? #decommenter ici et commenter la
 	       preg_match(',IIS|thttpd,',$_SERVER['SERVER_SOFTWARE']) ?
 	       'index.php' : '');
+	}
 
 
-	if (!defined('_SPIP_AJAX'))
-		define('_SPIP_AJAX', ((!isset($_COOKIE['spip_accepte_ajax']))
+	if (!defined('_SPIP_AJAX')) {
+			define('_SPIP_AJAX', ((!isset($_COOKIE['spip_accepte_ajax']))
 			? 1
 		       : (($_COOKIE['spip_accepte_ajax'] != -1) ? 1 : 0)));
+	}
 
 	// La requete est-elle en ajax ?
-	if (!defined('_AJAX')) define('_AJAX',
+	if (!defined('_AJAX')) {
+	    define('_AJAX',
 		(isset($_SERVER['HTTP_X_REQUESTED_WITH']) # ajax jQuery
 		OR @$_REQUEST['var_ajax_redir'] # redirection 302 apres ajax jQuery
 		OR @$_REQUEST['var_ajaxcharset'] # compat ascendante pour plugins
@@ -1827,16 +2153,22 @@ 
 		)
 		AND !@$_REQUEST['var_noajax'] # horrible exception, car c'est pas parce que la requete est ajax jquery qu'il faut tuer tous les formulaires ajax qu'elle contient
 	);
+	}
 
 	# nombre de pixels maxi pour calcul de la vignette avec gd
 	# au dela de 5500000 on considere que php n'est pas limite en memoire pour cette operation
 	# les configurations limitees en memoire ont un seuil plutot vers 1MPixel
-	if (!defined('_IMG_GD_MAX_PIXELS')) define('_IMG_GD_MAX_PIXELS',
+	if (!defined('_IMG_GD_MAX_PIXELS')) {
+	    define('_IMG_GD_MAX_PIXELS',
 		(isset($GLOBALS['meta']['max_taille_vignettes'])&&$GLOBALS['meta']['max_taille_vignettes']<5500000)
 		 ? $GLOBALS['meta']['max_taille_vignettes']
 		 : 0);
+	}
 
-	if (!defined('_MEMORY_LIMIT_MIN')) define('_MEMORY_LIMIT_MIN',10); // en Mo
+	if (!defined('_MEMORY_LIMIT_MIN')) {
+	    define('_MEMORY_LIMIT_MIN',10);
+	}
+	// en Mo
 	// si on est dans l'espace prive et si le besoin est superieur a 8Mo (qui est vraiment le standard)
 	// on verifie que la memoire est suffisante pour le compactage css+js pour eviter la page blanche
 	// il y aura d'autres problemes et l'utilisateur n'ira pas tres loin, mais ce sera plus comprehensible qu'une page blanche
@@ -1852,16 +2184,22 @@ 
 			if ($memory<_MEMORY_LIMIT_MIN*1024*1024){
 				ini_set('memory_limit',$m=_MEMORY_LIMIT_MIN.'M');
 				if (trim(ini_get('memory_limit'))!=$m){
-					if (!defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE')) define('_INTERDIRE_COMPACTE_HEAD_ECRIRE',true); // evite une page blanche car on ne saura pas calculer la css dans ce hit
+					if (!defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE')) {
+					    define('_INTERDIRE_COMPACTE_HEAD_ECRIRE',true);
+					}
+					// evite une page blanche car on ne saura pas calculer la css dans ce hit
 				}
 			}
-		}
-		else
-			if (!defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE')) define('_INTERDIRE_COMPACTE_HEAD_ECRIRE',true); // evite une page blanche car on ne saura pas calculer la css dans ce hit
+		} else
+			if (!defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE')) {
+			    define('_INTERDIRE_COMPACTE_HEAD_ECRIRE',true);
+			}
+			// evite une page blanche car on ne saura pas calculer la css dans ce hit
 	}
 	// Protocoles a normaliser dans les chaines de langues
-	if (!defined('_PROTOCOLES_STD'))
-		define('_PROTOCOLES_STD', 'http|https|ftp|mailto|webcal');
+	if (!defined('_PROTOCOLES_STD')) {
+			define('_PROTOCOLES_STD', 'http|https|ftp|mailto|webcal');
+	}
 
 	init_var_mode();
 }
@@ -1877,7 +2215,9 @@ 
 			// tout le monde peut calcul/recalcul
 			if ($_GET['var_mode'] == 'calcul'
 			OR $_GET['var_mode'] == 'recalcul') {
-				if (!defined('_VAR_MODE')) define('_VAR_MODE',$_GET['var_mode']);
+				if (!defined('_VAR_MODE')) {
+				    define('_VAR_MODE',$_GET['var_mode']);
+				}
 			}
 			// preview, debug, blocs, urls et images necessitent une autorisation
 			else if (in_array($_GET['var_mode'],array('preview','debug','inclure','urls','images','traduction'))) {
@@ -1890,48 +2230,79 @@ 
 					switch($_GET['var_mode']){
 						case 'traduction':
 							// forcer le calcul pour passer dans traduire
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','calcul');
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','calcul');
+							}
 							// et ne pas enregistrer de cache pour ne pas trainer les surlignages sur d'autres pages
-							if (!defined('_VAR_NOCACHE')) define('_VAR_NOCACHE',true);
+							if (!defined('_VAR_NOCACHE')) {
+							    define('_VAR_NOCACHE',true);
+							}
 							break;
 						case 'preview':
 							// basculer sur les criteres de preview dans les boucles
-							if (!defined('_VAR_PREVIEW')) define('_VAR_PREVIEW',true);
+							if (!defined('_VAR_PREVIEW')) {
+							    define('_VAR_PREVIEW',true);
+							}
 							// forcer le calcul
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','calcul');
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','calcul');
+							}
 							// et ne pas enregistrer de cache
-							if (!defined('_VAR_NOCACHE')) define('_VAR_NOCACHE',true);
+							if (!defined('_VAR_NOCACHE')) {
+							    define('_VAR_NOCACHE',true);
+							}
 							break;
 						case 'inclure':
 							// forcer le compilo et ignorer les caches existants
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','calcul');
-							if (!defined('_VAR_INCLURE')) define('_VAR_INCLURE',true);
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','calcul');
+							}
+							if (!defined('_VAR_INCLURE')) {
+							    define('_VAR_INCLURE',true);
+							}
 							// et ne pas enregistrer de cache
-							if (!defined('_VAR_NOCACHE')) define('_VAR_NOCACHE',true);
+							if (!defined('_VAR_NOCACHE')) {
+							    define('_VAR_NOCACHE',true);
+							}
 							break;
 						case 'urls':
 							// forcer le compilo et ignorer les caches existants
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','calcul');
-							if (!defined('_VAR_URLS')) define('_VAR_URLS',true);
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','calcul');
+							}
+							if (!defined('_VAR_URLS')) {
+							    define('_VAR_URLS',true);
+							}
 							break;
 						case 'images':
 							// forcer le compilo et ignorer les caches existants
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','calcul');
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','calcul');
+							}
 							// indiquer qu'on doit recalculer les images
-							if (!defined('_VAR_IMAGES')) define('_VAR_IMAGES',true);
+							if (!defined('_VAR_IMAGES')) {
+							    define('_VAR_IMAGES',true);
+							}
 							break;
 						case 'debug':
-							if (!defined('_VAR_MODE')) define('_VAR_MODE','debug');
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE','debug');
+							}
 							// et ne pas enregistrer de cache
-							if (!defined('_VAR_NOCACHE')) define('_VAR_NOCACHE',true);
+							if (!defined('_VAR_NOCACHE')) {
+							    define('_VAR_NOCACHE',true);
+							}
 							break;
 						default :
-							if (!defined('_VAR_MODE')) define('_VAR_MODE',$_GET['var_mode']);
+							if (!defined('_VAR_MODE')) {
+							    define('_VAR_MODE',$_GET['var_mode']);
+							}
 							break;
 					}
-          if (isset($GLOBALS['visiteur_session']['nom']))
-					spip_log($GLOBALS['visiteur_session']['nom']
+          if (isset($GLOBALS['visiteur_session']['nom'])) {
+          					spip_log($GLOBALS['visiteur_session']['nom']
 						. " "._VAR_MODE);
+          }
 				}
 				// pas autorise ?
 				else {
@@ -1946,7 +2317,9 @@ 
 					// sinon tant pis
 				}
 			}
-			if (!defined('_VAR_MODE')) define('_VAR_MODE',false);
+			if (!defined('_VAR_MODE')) {
+			    define('_VAR_MODE',false);
+			}
 		}
 		$done = true;
 	}
@@ -1958,18 +2331,21 @@ 
 // http://doc.spip.org/@spip_desinfecte
 function spip_desinfecte(&$t,$deep = true) {
 	static $magic_quotes;
-	if (!isset($magic_quotes))
-		$magic_quotes = @get_magic_quotes_gpc();
+	if (!isset($magic_quotes)) {
+			$magic_quotes = @get_magic_quotes_gpc();
+	}
 
 	foreach ($t as $key => $val) {
 		if (is_string($t[$key])) {
-			if ($magic_quotes)
-				$t[$key] = stripslashes($t[$key]);
+			if ($magic_quotes) {
+							$t[$key] = stripslashes($t[$key]);
+			}
 			$t[$key] = str_replace(chr(0), '-', $t[$key]);
 		}
 		// traiter aussi les "texte_plus" de article_edit
-		else if ($deep AND is_array($t[$key]) AND $key!=='GLOBALS')
-			spip_desinfecte($t[$key],$deep);
+		else if ($deep AND is_array($t[$key]) AND $key!=='GLOBALS') {
+					spip_desinfecte($t[$key],$deep);
+		}
 	}
 }
 
@@ -2004,11 +2380,13 @@ 
 		$session = charger_fonction('session', 'inc');
 		$session();
 		include_spip('inc/texte');
-		foreach($variables_session as $var)
-			if (($a = _request($var)) !== null)
+		foreach($variables_session as $var) {
+					if (($a = _request($var)) !== null)
 				$GLOBALS['visiteur_session'][$var] = safehtml($a);
-		if (!isset($GLOBALS['visiteur_session']['id_auteur']))
-			$GLOBALS['visiteur_session']['id_auteur'] = 0;
+		}
+		if (!isset($GLOBALS['visiteur_session']['id_auteur'])) {
+					$GLOBALS['visiteur_session']['id_auteur'] = 0;
+		}
 		$session($GLOBALS['visiteur_session']);
 		return 0;
 	}
@@ -2047,15 +2425,17 @@ 
 // http://doc.spip.org/@lang_select
 function lang_select ($lang=NULL) {
 	static $pile_langues = array();
-	if (!function_exists('changer_langue'))
-		include_spip('inc/lang');
-	if ($lang === NULL)
-		$lang = array_pop($pile_langues);
-	else {
+	if (!function_exists('changer_langue')) {
+			include_spip('inc/lang');
+	}
+	if ($lang === NULL) {
+			$lang = array_pop($pile_langues);
+	} else {
 		array_push($pile_langues, $GLOBALS['spip_lang']);
 	}
-	if (isset($GLOBALS['spip_lang']) AND $lang == $GLOBALS['spip_lang'])
-		return $lang;
+	if (isset($GLOBALS['spip_lang']) AND $lang == $GLOBALS['spip_lang']) {
+			return $lang;
+	}
 	changer_langue($lang);
 	return $lang;
 }
@@ -2106,11 +2486,12 @@ 
 // http://doc.spip.org/@exec_info_dist
 function exec_info_dist() {
 	global $connect_statut;
-	if ($connect_statut == '0minirezo')
-		phpinfo();
-	else
-		echo "pas admin";
-}
+	if ($connect_statut == '0minirezo') {
+			phpinfo();
+	} else {
+			echo "pas admin";
+	}
+	}
 
 /**
  * Génère une erreur de squelette
@@ -2169,12 +2550,17 @@ 
  * 		ou tableau d'information sur le squelette.
  */
 function recuperer_fond($fond, $contexte=array(), $options = array(), $connect='') {
-	if (!function_exists('evaluer_fond'))
-		include_spip('public/assembler');
+	if (!function_exists('evaluer_fond')) {
+			include_spip('public/assembler');
+	}
 	// assurer la compat avec l'ancienne syntaxe
 	// (trim etait le 3eme argument, par defaut a true)
-	if (!is_array($options)) $options = array('trim'=>$options);
-	if (!isset($options['trim'])) $options['trim']=true;
+	if (!is_array($options)) {
+	    $options = array('trim'=>$options);
+	}
+	if (!isset($options['trim'])) {
+	    $options['trim']=true;
+	}
 
 	if (isset($contexte['connect'])){
 		$connect = ($connect ? $connect : $contexte['connect']);
@@ -2186,8 +2572,9 @@ 
 	$lang_select = '';
 	if (!isset($options['etoile']) OR !$options['etoile']){
 		// Si on a inclus sans fixer le critere de lang, on prend la langue courante
-		if (!isset($contexte['lang']))
-			$contexte['lang'] = $GLOBALS['spip_lang'];
+		if (!isset($contexte['lang'])) {
+					$contexte['lang'] = $GLOBALS['spip_lang'];
+		}
 
 		if ($contexte['lang'] != $GLOBALS['meta']['langue_site']) {
 			$lang_select = lang_select($contexte['lang']);
@@ -2212,25 +2599,30 @@ 
 			'data'=>$page
 		));
 		if (isset($options['ajax']) AND $options['ajax']){
-			if (!function_exists('encoder_contexte_ajax'))
-				include_spip('inc/filtres');
+			if (!function_exists('encoder_contexte_ajax')) {
+							include_spip('inc/filtres');
+			}
 			$page['texte'] = encoder_contexte_ajax(array_merge($contexte,array('fond'=>$f)),'',$page['texte'], $options['ajax']);
 		}
 
-		if (isset($options['raw']) AND $options['raw'])
-			$pages[] = $page;
-		else
-			$texte .= $options['trim'] ? rtrim($page['texte']) : $page['texte'];
+		if (isset($options['raw']) AND $options['raw']) {
+					$pages[] = $page;
+		} else {
+					$texte .= $options['trim'] ? rtrim($page['texte']) : $page['texte'];
+		}
 	}
 
 	$GLOBALS['_INC_PUBLIC']--;
 
-	if ($lang_select) lang_select();
-	if (isset($options['raw']) AND $options['raw'])
-		return is_array($fond)?$pages:reset($pages);
-	else
-		return $options['trim'] ? ltrim($texte) : $texte;
-}
+	if ($lang_select) {
+	    lang_select();
+	}
+	if (isset($options['raw']) AND $options['raw']) {
+			return is_array($fond)?$pages:reset($pages);
+	} else {
+			return $options['trim'] ? ltrim($texte) : $texte;
+	}
+	}
 
 /**
  * Trouve un squelette dans le repertoire modeles/
@@ -2256,7 +2648,9 @@ 
  */
 function trouver_fond($nom, $dir='', $pathinfo = false) {
 	$f = find_in_path($nom.'.'. _EXTENSION_SQUELETTES, $dir?rtrim($dir,'/').'/':'');
-	if (!$pathinfo) return $f;
+	if (!$pathinfo) {
+	    return $f;
+	}
 	// renvoyer un tableau detaille si $pathinfo==true
 	$p = pathinfo($f);
 	if (!isset($p['extension']) OR !$p['extension']) {
@@ -2271,16 +2665,21 @@ 
 
 function tester_url_ecrire($nom){
 	static $exec=array();
-	if (isset($exec[$nom])) return $exec[$nom];
+	if (isset($exec[$nom])) {
+	    return $exec[$nom];
+	}
 	// tester si c'est une page en squelette
-	if (trouver_fond($nom, 'prive/squelettes/contenu/'))
-		return $exec[$nom] = 'fond';
+	if (trouver_fond($nom, 'prive/squelettes/contenu/')) {
+			return $exec[$nom] = 'fond';
+	}
 	// compat skels orthogonaux version precedente
-	elseif (trouver_fond($nom, 'prive/exec/'))
-		return $exec[$nom] = 'fond_monobloc';
+	elseif (trouver_fond($nom, 'prive/exec/')) {
+			return $exec[$nom] = 'fond_monobloc';
+	}
 	// echafaudage d'un fond !
-	elseif(include_spip('public/styliser_par_z') AND z_echafaudable($nom))
-		return $exec[$nom] = 'fond';
+	elseif(include_spip('public/styliser_par_z') AND z_echafaudable($nom)) {
+			return $exec[$nom] = 'fond';
+	}
 	// attention, il ne faut pas inclure l'exec ici
 	// car sinon #URL_ECRIRE provoque des inclusions
 	// et des define intrusifs potentiels
@@ -2324,13 +2723,23 @@ 
 // http://doc.spip.org/@spip_fetch_array
 function spip_fetch_array($r, $t=NULL) {
 	if (!isset($t)) {
-		if ($r) return sql_fetch($r);
+		if ($r) {
+		    return sql_fetch($r);
+		}
 	} else {
-		if ($t=='SPIP_NUM') $t = MYSQL_NUM;
-		if ($t=='SPIP_BOTH') $t = MYSQL_BOTH;
-		if ($t=='SPIP_ASSOC') $t = MYSQL_ASSOC;
+		if ($t=='SPIP_NUM') {
+		    $t = MYSQL_NUM;
+		}
+		if ($t=='SPIP_BOTH') {
+		    $t = MYSQL_BOTH;
+		}
+		if ($t=='SPIP_ASSOC') {
+		    $t = MYSQL_ASSOC;
+		}
 		spip_log("appel deprecie de spip_fetch_array(..., $t)", 'vieilles_defs');
-		if ($r) return mysql_fetch_array($r, $t);
+		if ($r) {
+		    return mysql_fetch_array($r, $t);
+		}
 	}
 }
 
@@ -2347,11 +2756,13 @@ 
 function avertir_auteurs($nom,$message, $statut=''){
 	$alertes = $GLOBALS['meta']['message_alertes_auteurs'];
 	if (!$alertes
-		OR !is_array($alertes = unserialize($alertes)))
-		$alertes = array();
+		OR !is_array($alertes = unserialize($alertes))) {
+			$alertes = array();
+	}
 
-	if (!isset($alertes[$statut]))
-		$alertes[$statut] = array();
+	if (!isset($alertes[$statut])) {
+			$alertes[$statut] = array();
+	}
 	$alertes[$statut][$nom] = $message;
 	ecrire_meta("message_alertes_auteurs",serialize($alertes));
 }

ecrire/inc/texte.php

Braces +42 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/texte_mini');
 include_spip('inc/lien');
@@ -66,11 +68,12 @@ 
 // afficher joliment les <script>
 // http://doc.spip.org/@echappe_js
 function echappe_js($t,$class=' class="echappe-js"') {
-	if (preg_match_all(',<script.*?($|</script.),isS', $t, $r, PREG_SET_ORDER))
-	foreach ($r as $regs)
+	if (preg_match_all(',<script.*?($|</script.),isS', $t, $r, PREG_SET_ORDER)) {
+		foreach ($r as $regs)
 		$t = str_replace($regs[0],
 			"<code$class>".nl2br(spip_htmlspecialchars($regs[0])).'</code>',
 			$t);
+	}
 	return $t;
 }
 
@@ -105,7 +108,9 @@ 
 	static $dejavu = array();
 
 	// Attention, si ce n'est pas une chaine, laisser intact
-	if (!$arg OR !is_string($arg) OR !strstr($arg, '<')) return $arg; 
+	if (!$arg OR !is_string($arg) OR !strstr($arg, '<')) {
+	    return $arg;
+	}
 
 	if (is_null($mode_filtre) or !in_array($mode_filtre, array(-1, 0, 1))) {
 		$mode_filtre = $GLOBALS['filtrer_javascript'];
@@ -127,8 +132,9 @@ 
 	// Pour le js, trois modes : parano (-1), prive (0), ok (1)
 	switch ($mode_filtre) {
 		case 0:
-			if (!_DIR_RESTREINT)
-				$t = echappe_js($t);
+			if (!_DIR_RESTREINT) {
+							$t = echappe_js($t);
+			}
 			break;
 		case -1:
 			$t = echappe_js($t);
@@ -139,10 +145,12 @@ 
 	$t = preg_replace(',<(base\b),iS', '&lt;\1', $t);
 
 	// Reinserer les echappements des modeles
-	if (defined('_PROTEGE_JS_MODELES'))
-		$t = echappe_retour($t,"javascript"._PROTEGE_JS_MODELES);
-	if (defined('_PROTEGE_PHP_MODELES'))
-		$t = echappe_retour($t,"php"._PROTEGE_PHP_MODELES);
+	if (defined('_PROTEGE_JS_MODELES')) {
+			$t = echappe_retour($t,"javascript"._PROTEGE_JS_MODELES);
+	}
+	if (defined('_PROTEGE_PHP_MODELES')) {
+			$t = echappe_retour($t,"php"._PROTEGE_PHP_MODELES);
+	}
 
 	return $dejavu[$mode_filtre][$arg] = $t;
 }
@@ -153,7 +161,9 @@ 
 // http://doc.spip.org/@typo
 function typo($letexte, $echapper=true, $connect=null, $env=array()) {
 	// Plus vite !
-	if (!$letexte) return $letexte;
+	if (!$letexte) {
+	    return $letexte;
+	}
 
 	// les appels directs a cette fonction depuis le php de l'espace
 	// prive etant historiquement ecrit sans argment $connect
@@ -169,8 +179,9 @@ 
 	}
 
 	// Echapper les codes <html> etc
-	if ($echapper)
-		$letexte = echappe_html($letexte, 'TYPO');
+	if ($echapper) {
+			$letexte = echappe_html($letexte, 'TYPO');
+	}
 
 	//
 	// Installer les modeles, notamment images et documents ;
@@ -179,19 +190,23 @@ 
 	// cf. inc/lien
 
 	$letexte = traiter_modeles($mem = $letexte, false, $echapper ? 'TYPO' : '', $connect, null, $env);
-	if ($letexte != $mem) $echapper = true;
+	if ($letexte != $mem) {
+	    $echapper = true;
+	}
 	unset($mem);
 
 	$letexte = corriger_typo($letexte);
 	$letexte = echapper_faux_tags($letexte);
 
 	// reintegrer les echappements
-	if ($echapper)
-		$letexte = echappe_retour($letexte, 'TYPO');
+	if ($echapper) {
+			$letexte = echappe_retour($letexte, 'TYPO');
+	}
 
 	// Dans les appels directs hors squelette, securiser ici aussi
-	if ($interdire_script)
-		$letexte = interdire_scripts($letexte);
+	if ($interdire_script) {
+			$letexte = interdire_scripts($letexte);
+	}
 
 	// Dans l'espace prive on se mefie de tout contenu dangereux
 	// https://core.spip.net/issues/3371
@@ -212,7 +227,9 @@ 
 function corriger_typo($letexte, $lang='') {
 
 	// Plus vite !
-	if (!$letexte) return $letexte;
+	if (!$letexte) {
+	    return $letexte;
+	}
 
 	$letexte = pipeline('pre_typo', $letexte);
 
@@ -239,7 +256,9 @@ 
 	$letexte = $typographie($letexte);
 
 	// Les citations en une autre langue, s'il y a lieu
-	if (!$e) $letexte = echappe_retour($letexte, 'multi');
+	if (!$e) {
+	    $letexte = echappe_retour($letexte, 'multi');
+	}
 
 	// Retablir les caracteres proteges
 	$letexte = strtr($letexte, _TYPO_PROTECTEUR, _TYPO_PROTEGER);
@@ -295,7 +314,9 @@ 
 		$interdire_script = true;
 	}
 
-	if (!$t) return strval($t);
+	if (!$t) {
+	    return strval($t);
+	}
 
 	$t = echappe_html($t);
 	$t = expanser_liens($t,$connect, $env);

ecrire/inc/rechercher.php

Braces +30 added lines, -18 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 
 // Donne la liste des champs/tables ou l'on sait chercher/remplacer
@@ -216,17 +218,20 @@ 
 		if (is_string($tables)
 		AND $tables != '') {
 			$toutes = array();
-			foreach(explode(',', $tables) as $t)
-				if (isset($liste[$t]))
+			foreach(explode(',', $tables) as $t) {
+							if (isset($liste[$t]))
 					$toutes[$t] = $liste[$t];
+			}
 			$tables = $toutes;
 			unset($toutes);
-		} else
-			$tables = $liste;
+		} else {
+					$tables = $liste;
+		}
 	}
 
-	if (!strlen($recherche) OR !count($tables))
-		return array();
+	if (!strlen($recherche) OR !count($tables)) {
+			return array();
+	}
 
 	include_spip('inc/autoriser');
 
@@ -254,8 +259,9 @@ 
 	include_spip('inc/memoization');
 	foreach ($tables as $table => $champs) {
 		# lock via memoization, si dispo
-		if (function_exists('cache_lock'))
-			cache_lock($lock = 'recherche '.$table.' '.$recherche);
+		if (function_exists('cache_lock')) {
+					cache_lock($lock = 'recherche '.$table.' '.$recherche);
+		}
 
 		spip_timer('rech');
 
@@ -270,8 +276,9 @@ 
 
 		spip_log("recherche $table ($recherche) : ".count($results[$table])." resultats ".spip_timer('rech'),'recherche');
 
-		if (isset($lock))
-			cache_unlock($lock);
+		if (isset($lock)) {
+					cache_unlock($lock);
+		}
 	}
 
 	return $results;
@@ -293,8 +300,9 @@ 
 	$options['champs'] = true;
 
 
-	if (!is_array($tables))
-		$tables = liste_des_champs();
+	if (!is_array($tables)) {
+			$tables = liste_des_champs();
+	}
 
 	$results = recherche_en_base($recherche, $tables, $options);
 
@@ -307,17 +315,21 @@ 
 			OR autoriser('modifier', $table, $id)) {
 				$modifs = array();
 				foreach ($x['champs'] as $key => $val) {
-					if ($key == $_id_table) next;
+					if ($key == $_id_table) {
+					    next;
+					}
 					$repl = preg_replace($preg, $remplace, $val);
-					if ($repl <> $val)
-						$modifs[$key] = $repl;
+					if ($repl <> $val) {
+											$modifs[$key] = $repl;
+					}
 				}
-				if ($modifs)
-					objet_modifier_champs($table, $id,
+				if ($modifs) {
+									objet_modifier_champs($table, $id,
 						array(
 							'champs' => array_keys($modifs),
 						),
 						$modifs);
+				}
 			}
 		}
 	}

ecrire/inc/chercher_rubrique.php

Braces +57 added lines, -35 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;
+}
 
 define('_SPIP_SELECT_RUBRIQUES', 20); /* mettre 100000 pour desactiver ajax */
 
@@ -24,18 +26,20 @@ 
 // $idem : en mode rubrique = la rubrique soi-meme
 // http://doc.spip.org/@inc_chercher_rubrique_dist
 function inc_chercher_rubrique_dist ($id_rubrique, $type, $restreint, $idem=0, $do='aff') {
-	if (sql_countsel('spip_rubriques')<1)
-		return '';
+	if (sql_countsel('spip_rubriques')<1) {
+			return '';
+	}
 
 	// Mode sans Ajax :
 	// - soit parce que le cookie ajax n'est pas la
 	// - soit parce qu'il y a peu de rubriques
 	if (_SPIP_AJAX < 1
 	OR $type == 'breve'
-	OR sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES)
-		return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
-
-	else return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
+	OR sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES) {
+			return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
+	} else {
+	    return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
+	}
 
 }
 
@@ -52,11 +56,13 @@ 
 		. "margin-$spip_lang_left: ".(($i-1)*16)."px;";
 	} else {
 		$style = '';
-		for ($count = 0; $count <= $i; $count ++)
-			$espace .= "&nbsp;&nbsp;&nbsp;&nbsp;";
+		for ($count = 0; $count <= $i; $count ++) {
+					$espace .= "&nbsp;&nbsp;&nbsp;&nbsp;";
+		}
+	}
+	if ($i ==1) {
+			$espace= "";
 	}
-	if ($i ==1)
-		$espace= "";
 	$class = "niveau_$i";
 	return array($class,$style,$espace);
 }
@@ -67,7 +73,9 @@ 
 
 	// Si on a demande l'exclusion ne pas descendre dans la rubrique courante
 	if ($exclus > 0
-	AND $root == $exclus) return '';
+	AND $root == $exclus) {
+	    return '';
+	}
 
 	// en fonction du niveau faire un affichage plus ou moins kikoo
 
@@ -81,23 +89,29 @@ 
 
 	// creer l'<option> pour la rubrique $root
 
-	if (isset($data[$root])) # pas de racine sauf pour les rubriques
+	if (isset($data[$root])) {
+	    # pas de racine sauf pour les rubriques
 	{
 		$r = "<option$selected value='$root' class='$class' style='$style'>$espace"
 		.$data[$root]
 		.'</option>'."\n";
-	} else 	$r = '';
+	}
+	} else {
+	    $r = '';
+	}
 	
 	// et le sous-menu pour ses enfants
 	$sous = '';
-	if (isset($enfants[$root]))
-		foreach ($enfants[$root] as $sousrub)
+	if (isset($enfants[$root])) {
+			foreach ($enfants[$root] as $sousrub)
 			$sous .= sous_menu_rubriques($id_rubrique, $sousrub,
 				$niv+1, $data, $enfants, $exclus, $restreint, $type);
+	}
 
 	// si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
-	if ($restreint AND $root!=$id_rubrique AND !autoriser('publierdans','rubrique',$root))
-		return $sous;
+	if ($restreint AND $root!=$id_rubrique AND !autoriser('publierdans','rubrique',$root)) {
+			return $sous;
+	}
 
 	// et voila le travail
 	return $r.$sous;
@@ -107,13 +121,15 @@ 
 // http://doc.spip.org/@selecteur_rubrique_html
 function selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem=0) {
 	$data = array();
-	if ($type == 'rubrique' AND autoriser('publierdans','rubrique',0))
-		$data[0] = _T('info_racine_site');
+	if ($type == 'rubrique' AND autoriser('publierdans','rubrique',0)) {
+			$data[0] = _T('info_racine_site');
+	}
 	# premier choix = neant
 	# si auteur (rubriques restreintes)
 	# ou si creation avec id_rubrique=0
-	elseif ($type == 'auteur' OR !$id_rubrique)
-		$data[0] = '&nbsp;';
+	elseif ($type == 'auteur' OR !$id_rubrique) {
+			$data[0] = '&nbsp;';
+	}
 
 	//
 	// creer une structure contenant toute l'arborescence
@@ -126,11 +142,14 @@ 
 			// titre largeur maxi a 50
 			$titre = couper(supprimer_tags(typo($r['titre']))." ", 50);
 			if ($GLOBALS['meta']['multi_rubriques'] == 'oui'
-			AND ($r['langue_choisie'] == "oui" OR $r['id_parent'] == 0))
-				$titre .= ' ['.traduire_nom_langue($r['lang']).']';
+			AND ($r['langue_choisie'] == "oui" OR $r['id_parent'] == 0)) {
+							$titre .= ' ['.traduire_nom_langue($r['lang']).']';
+			}
 			$data[$r['id_rubrique']] = $titre;
 			$enfants[$r['id_parent']][] = $r['id_rubrique'];
-			if ($id_rubrique == $r['id_rubrique']) $id_parent = $r['id_parent'];
+			if ($id_rubrique == $r['id_rubrique']) {
+			    $id_parent = $r['id_parent'];
+			}
 		}
 	}
 
@@ -140,17 +159,19 @@ 
 	if (count($data)==2
 	  AND isset($data[0])
 	  AND !in_array($type,array('auteur','rubrique'))
-	  AND !$id_rubrique)
-		unset($data[0]);
+	  AND !$id_rubrique) {
+			unset($data[0]);
+	}
 
 
 	$opt = sous_menu_rubriques($id_rubrique,0, 0,$data,$enfants,$idem, $restreint, $type);
 	$att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
 
-	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,',$opt,$r))
-	  $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2] ;
-	else 
-	  $r = "<select".$att." size='1'>\n$opt</select>\n";
+	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,',$opt,$r)) {
+		  $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2] ;
+	} else {
+		  $r = "<select".$att." size='1'>\n$opt</select>\n";
+	}
 
 	# message pour neuneus (a supprimer ?)
 #	if ($type != 'auteur' AND $type != 'breve')
@@ -181,10 +202,11 @@ 
 
 	if ($id_rubrique) {
 		$titre = sql_getfetsel("titre", "spip_rubriques", "id_rubrique=".intval($id_rubrique));
-	} else if ($type == 'auteur')
-		$titre = '&nbsp;';
-	else
-		$titre = _T('info_racine_site');
+	} else if ($type == 'auteur') {
+			$titre = '&nbsp;';
+	} else {
+			$titre = _T('info_racine_site');
+	}
 
 	$titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
 	$init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";

ecrire/public/assembler.php

Braces +127 added lines, -83 removed lines

@@ -10,9 +10,13 @@ 
  *  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;
+}
 
-if (!defined('_CONTEXTE_IGNORE_VARIABLES')) define('_CONTEXTE_IGNORE_VARIABLES', "/(^var_|^PHPSESSID$)/");
+if (!defined('_CONTEXTE_IGNORE_VARIABLES')) {
+    define('_CONTEXTE_IGNORE_VARIABLES', "/(^var_|^PHPSESSID$)/");
+}
 //
 // calcule la page et les entetes
 // determine le contexte donne par l'URL (en tenant compte des reecritures) 
@@ -34,14 +38,17 @@ 
 	$cacher = charger_fonction('cacher', 'public', true);
 	// Les quatre derniers parametres sont modifies par la fonction:
 	// emplacement, validite, et, s'il est valide, contenu & age
-	if ($cacher)
-		$res = $cacher($GLOBALS['contexte'], $use_cache, $chemin_cache, $page, $lastmodified);
-	else
-		$use_cache = -1;
+	if ($cacher) {
+			$res = $cacher($GLOBALS['contexte'], $use_cache, $chemin_cache, $page, $lastmodified);
+	} else {
+			$use_cache = -1;
+	}
 	// Si un resultat est retourne, c'est un message d'impossibilite
 	if ($res) {return array('texte' => $res);}
 
-	if (!$chemin_cache || !$lastmodified) $lastmodified = time();
+	if (!$chemin_cache || !$lastmodified) {
+	    $lastmodified = time();
+	}
 
 	$headers_only = ($_SERVER['REQUEST_METHOD'] == 'HEAD');
 	$calculer_page = true;
@@ -83,8 +90,7 @@ 
 			// plus necessaire si on utilise bien la fonction urls_decoder_url
 			#unset($_SERVER['REDIRECT_url_propre']);
 			#unset($_ENV['url_propre']);
-		}
-		else {
+		} else {
 			// Compat ascendante :
 			// 1. $contexte est global
 			// (a evacuer car urls_decoder_url gere ce probleme ?)
@@ -94,8 +100,9 @@ 
 				list($fond,$contexte,$url_redirect) = urls_decoder_url(nettoyer_uri(),$fond,$contexte,true);
 			}
 			// squelette par defaut
-			if (!strlen($fond))
-				$fond = 'sommaire';
+			if (!strlen($fond)) {
+							$fond = 'sommaire';
+			}
 
 			// produire la page : peut mettre a jour $lastmodified
 			$produire_page = charger_fonction('produire_page','public');
@@ -109,7 +116,9 @@ 
 			}
 		}
 
-		if ($page AND $chemin_cache) $page['cache'] = $chemin_cache;
+		if ($page AND $chemin_cache) {
+		    $page['cache'] = $chemin_cache;
+		}
 
 		auto_content_type($page);
 
@@ -140,12 +149,14 @@ 
 	// a celui qu'on a refuse d'honorer plus haut (cf. #655)
 	if ($lastmodified
 	AND !isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])
-	AND !isset($page['entetes']["Last-Modified"]))
-		$page['entetes']["Last-Modified"]=gmdate("D, d M Y H:i:s", $lastmodified)." GMT";
+	AND !isset($page['entetes']["Last-Modified"])) {
+			$page['entetes']["Last-Modified"]=gmdate("D, d M Y H:i:s", $lastmodified)." GMT";
+	}
 
 	// fermer la connexion apres les headers si requete HEAD
-	if ($headers_only)
-		$page['entetes']["Connection"] = "close";
+	if ($headers_only) {
+			$page['entetes']["Connection"] = "close";
+	}
 
 	return $page;
 }
@@ -162,12 +173,14 @@ 
 
 	$contexte = array();
 	foreach($_GET as $var => $val) {
-		if (!preg_match(_CONTEXTE_IGNORE_VARIABLES,$var))
-			$contexte[$var] = $val;
+		if (!preg_match(_CONTEXTE_IGNORE_VARIABLES,$var)) {
+					$contexte[$var] = $val;
+		}
 	}
 	foreach($_POST as $var => $val) {
-		if (!preg_match(_CONTEXTE_IGNORE_VARIABLES,$var))
-			$contexte[$var] = $val;
+		if (!preg_match(_CONTEXTE_IGNORE_VARIABLES,$var)) {
+					$contexte[$var] = $val;
+		}
 	}
 
 	return $contexte;
@@ -182,8 +195,9 @@ 
  */
 function calculer_contexte_implicite(){
 	static $notes = null;
-	if (is_null($notes))
-		$notes = charger_fonction('notes','inc',true);
+	if (is_null($notes)) {
+			$notes = charger_fonction('notes','inc',true);
+	}
 	$contexte_implicite = array(
 		'squelettes' => $GLOBALS['dossier_squelettes'], // devrait etre 'chemin' => $GLOBALS['path_sig'], ?
 		'host' => $_SERVER['HTTP_HOST'],
@@ -222,14 +236,16 @@ 
 	unset($contexte['fond']);
 	$page = array('contexte_implicite'=>calculer_contexte_implicite());
 	$page['contexte_implicite']['cache'] = $fond;
-	if (is_null($cacher))
-		$cacher = charger_fonction('cacher', 'public', true);
+	if (is_null($cacher)) {
+			$cacher = charger_fonction('cacher', 'public', true);
+	}
 	// Les quatre derniers parametres sont modifies par la fonction:
 	// emplacement, validite, et, s'il est valide, contenu & age
-	if ($cacher)
-		$res = $cacher($contexte, $use_cache, $chemin_cache, $page, $lastinclude);
-	else
-		$use_cache = -1;
+	if ($cacher) {
+			$res = $cacher($contexte, $use_cache, $chemin_cache, $page, $lastinclude);
+	} else {
+			$use_cache = -1;
+	}
 	// $res = message d'erreur : on sort de la
 	if ($res) {return array('texte' => $res);}
 
@@ -237,8 +253,9 @@ 
 	// produire la page : peut mettre a jour $lastinclude
 	// le contexte_cache envoye a cacher() a ete conserve et est passe a produire
 	if ($use_cache) {
-		if (is_null($produire_page))
-			$produire_page = charger_fonction('produire_page','public');
+		if (is_null($produire_page)) {
+					$produire_page = charger_fonction('produire_page','public');
+		}
 		$page = $produire_page($fond, $contexte, $use_cache, $chemin_cache, $contexte, $page, $lastinclude, $connect);
 	}
 	// dans tous les cas, mettre a jour $lastmodified
@@ -263,8 +280,9 @@ 
  */
 function public_produire_page_dist($fond, $contexte, $use_cache, $chemin_cache, $contexte_cache, &$page, &$lastinclude, $connect=''){
 	static $parametrer,$cacher;
-	if (!$parametrer)
-		$parametrer = charger_fonction('parametrer', 'public');
+	if (!$parametrer) {
+			$parametrer = charger_fonction('parametrer', 'public');
+	}
 	$page = $parametrer($fond, $contexte, $chemin_cache, $connect);
 	// et on l'enregistre sur le disque
 	if ($chemin_cache
@@ -272,13 +290,15 @@ 
 	AND is_array($page)
 	AND count($page)
 	AND $page['entetes']['X-Spip-Cache'] > 0){
-		if (is_null($cacher))
-			$cacher = charger_fonction('cacher', 'public', true);
+		if (is_null($cacher)) {
+					$cacher = charger_fonction('cacher', 'public', true);
+		}
 		$lastinclude = time();
-		if ($cacher)
-			$cacher($contexte_cache, $use_cache, $chemin_cache, $page, $lastinclude);
-		else
-			$use_cache = -1;
+		if ($cacher) {
+					$cacher($contexte_cache, $use_cache, $chemin_cache, $page, $lastinclude);
+		} else {
+					$use_cache = -1;
+		}
 	}
 	return $page;
 }
@@ -296,8 +316,10 @@ 
 
 function inserer_balise_dynamique($contexte_exec, $contexte_compil)
 {
-	if (!is_array($contexte_exec))
-		echo $contexte_exec; // message d'erreur etc
+	if (!is_array($contexte_exec)) {
+			echo $contexte_exec;
+	}
+	// message d'erreur etc
 	else {
 		inclure_balise_dynamique($contexte_exec, true, $contexte_compil);
 	}
@@ -328,8 +350,9 @@ 
 			unset($page['entetes']['X-Spip-Cache']);
 			unset($page['entetes']['Content-Type']);
 			if (isset($GLOBALS['page']) AND is_array($GLOBALS['page'])) {
-				if (!is_array($GLOBALS['page']['entetes']))
-					$GLOBALS['page']['entetes'] = array();
+				if (!is_array($GLOBALS['page']['entetes'])) {
+									$GLOBALS['page']['entetes'] = array();
+				}
 				$GLOBALS['page']['entetes'] = 
 					array_merge($GLOBALS['page']['entetes'],$page['entetes']);
 			}
@@ -358,17 +381,20 @@ 
 		$ligne =  intval(isset($contexte_compil[3]) ? $contexte_compil[3] : $contexte_compil);
 		$GLOBALS['debug_objets']['resultat'][$ligne] = $texte;
 	}
-	if ($echo)
-		echo $texte;
-	else
-		return $texte;
+	if ($echo) {
+			echo $texte;
+	} else {
+			return $texte;
+	}
 
 }
 
 // http://doc.spip.org/@message_page_indisponible
 function message_page_indisponible ($page, $contexte) {
 	static $deja = false;
-	if ($deja) return "erreur";
+	if ($deja) {
+	    return "erreur";
+	}
 	$codes = array(
 		'404' => '404 Not Found',
 		'503' => '503 Service Unavailable',
@@ -377,8 +403,9 @@ 
 	$contexte['status'] = ($page !== false) ? '404' : '503';
 	$contexte['code'] = $codes[$contexte['status']];
 	$contexte['fond'] = '404'; // gere les 2 erreurs
-	if (!isset($contexte['lang']))
-		$contexte['lang'] = $GLOBALS['spip_lang'];
+	if (!isset($contexte['lang'])) {
+			$contexte['lang'] = $GLOBALS['spip_lang'];
+	}
 
 	$deja = true;
 	// passer aux plugins qui peuvent decider d'une page d'erreur plus pertinent
@@ -387,8 +414,9 @@ 
 
 	// produire la page d'erreur
 	$page = inclure_page($contexte['fond'], $contexte);
-	if (!$page)
-		$page = inclure_page('404', $contexte);
+	if (!$page) {
+			$page = inclure_page('404', $contexte);
+	}
 	$page['status'] = $contexte['status'];
 
 	return $page;
@@ -406,14 +434,17 @@ 
 				$contexte[$var] = $val;
 			} else {
 				$args = explode('=', $val);
-				if (count($args)>=2) // Flashvars=arg1=machin&arg2=truc genere plus de deux args
+				if (count($args)>=2) {
+				    // Flashvars=arg1=machin&arg2=truc genere plus de deux args
 					$contexte[trim($args[0])] = substr($val,strlen($args[0])+1);
-				else // notation abregee
+				} else {
+				    // notation abregee
 					$contexte[trim($val)] = trim($val);
+				}
 			}
+		} else {
+					$contexte[$var] = $val;
 		}
-		else
-			$contexte[$var] = $val;
 	}
 
 	return $contexte;
@@ -424,7 +455,10 @@ 
 function inclure_modele($type, $id, $params, $lien, $connect='', $env=array()) {
 
 	static $compteur;
-	if (++$compteur>10) return ''; # ne pas boucler indefiniment
+	if (++$compteur>10) {
+	    return '';
+	}
+	# ne pas boucler indefiniment
 
 	$type = strtolower($type);
 
@@ -468,8 +502,9 @@ 
 	$_id = id_table_objet($type);
 	$contexte['id'] = $contexte[$_id] = $id;
 
-	if (isset($class))
-		$contexte['class'] = $class;
+	if (isset($class)) {
+			$contexte['class'] = $class;
+	}
 
 	// Si un lien a ete passe en parametre, ex: [<modele1>->url]
 	if ($lien) {
@@ -499,8 +534,9 @@ 
 	'spip_lien_ok')) {
 		$retour = inserer_attribut($retour, 'class',
 			trim(str_replace(' spip_lien_ok ', ' ', " $classes ")));
-	} else if ($lien)
-		$retour = "<a href='".$lien['href']."' class='".$lien['class']."'>".$retour."</a>";
+	} else if ($lien) {
+			$retour = "<a href='".$lien['href']."' class='".$lien['class']."'>".$retour."</a>";
+	}
 
 	$compteur--;
 
@@ -518,7 +554,9 @@ 
 
 	$page = inclure_page($fond, $contexte, $connect);
 
-	if (!$page) return $page;
+	if (!$page) {
+	    return $page;
+	}
 	// eval $page et affecte $res
 	include _ROOT_RESTREINT."public/evaluer_page.php";
 	
@@ -528,8 +566,9 @@ 
 	// il faut bien lever ce drapeau apres avoir evalue le fond
 	// pour ne pas faire descendre le flag vers les inclusions appelees
 	if (isset($page['invalideurs'])
-	AND isset($page['invalideurs']['session']))
-		$GLOBALS['cache_utilise_session'] = $page['invalideurs']['session'];
+	AND isset($page['invalideurs']['session'])) {
+			$GLOBALS['cache_utilise_session'] = $page['invalideurs']['session'];
+	}
 
 	return $page;
 }
@@ -539,16 +578,17 @@ 
 function page_base_href(&$texte){
 	static $set_html_base = null;
 	if (is_null($set_html_base)){
-		if (!defined('_SET_HTML_BASE'))
-			// si la profondeur est superieure a 1
+		if (!defined('_SET_HTML_BASE')) {
+					// si la profondeur est superieure a 1
 			// est que ce n'est pas une url page ni une url action
 			// activer par defaut
 			$set_html_base = ((
 				$GLOBALS['profondeur_url'] >= (_DIR_RESTREINT?1:2)
 				AND _request(_SPIP_PAGE) !== 'login'
 				AND !_request('action'))?true:false);
-		else
-			$set_html_base = _SET_HTML_BASE;
+		} else {
+					$set_html_base = _SET_HTML_BASE;
+		}
 	}
 
 	if ($set_html_base
@@ -557,25 +597,26 @@ 
 	  AND ($poshead = strpos($texte,'</head>'))!==FALSE){
 		$head = substr($texte,0,$poshead);
 		$insert = false;
-		if (strpos($head, '<base')===false) 
-			$insert = true;
-		else {
+		if (strpos($head, '<base')===false) {
+					$insert = true;
+		} else {
 			// si aucun <base ...> n'a de href c'est bon quand meme !
 			$insert = true;
 			include_spip('inc/filtres');
 			$bases = extraire_balises($head,'base');
-			foreach ($bases as $base)
-				if (extraire_attribut($base,'href'))
+			foreach ($bases as $base) {
+							if (extraire_attribut($base,'href'))
 					$insert = false;
+			}
 		}
 		if ($insert) {
 			include_spip('inc/filtres_mini');
 			// ajouter un base qui reglera tous les liens relatifs
 			$base = url_absolue('./');
 			$bbase = "\n<base href=\"$base\" />";
-			if (($pos = strpos($head, '<head>')) !== false)
-				$head = substr_replace($head, $bbase, $pos+6, 0);
-			elseif(preg_match(",<head[^>]*>,i",$head,$r)){
+			if (($pos = strpos($head, '<head>')) !== false) {
+							$head = substr_replace($head, $bbase, $pos+6, 0);
+			} elseif(preg_match(",<head[^>]*>,i",$head,$r)){
 				$head = str_replace($r[0], $r[0].$bbase, $head);
 			}
 			$texte = $head . substr($texte,$poshead);
@@ -585,10 +626,12 @@ 
 			if (strpos($base,"'") or strpos($base,'"') or strpos($base,'<')) {
 				$base = str_replace(array("'",'"','<'),array("%27",'%22','%3C'), $base);
 			}
-			if (strpos($texte,"href='#")!==false)
-				$texte = str_replace("href='#","href='$base#",$texte);
-			if (strpos($texte, "href=\"#")!==false)
-				$texte = str_replace("href=\"#","href=\"$base#",$texte);
+			if (strpos($texte,"href='#")!==false) {
+							$texte = str_replace("href='#","href='$base#",$texte);
+			}
+			if (strpos($texte, "href=\"#")!==false) {
+							$texte = str_replace("href=\"#","href=\"$base#",$texte);
+			}
 		}
 	}
 }
@@ -598,9 +641,10 @@ 
 // et demarrent par X-Spip-...
 // http://doc.spip.org/@envoyer_entetes
 function envoyer_entetes($entetes) {
-	foreach ($entetes as $k => $v)
-	#	if (strncmp($k, 'X-Spip-', 7))
-			@header(strlen($v)?"$k: $v":$k); 
-}
+	foreach ($entetes as $k => $v) {
+		#	if (strncmp($k, 'X-Spip-', 7))
+			@header(strlen($v)?"$k: $v":$k);
+	}
+	}
 
 ?>

prive/formulaires/configurer_transcodeur.php

Braces +8 added lines, -5 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;
+}
 
 function formulaires_configurer_transcodeur_charger_dist(){
 	$valeurs = array(
@@ -24,10 +26,11 @@ 
 	include_spip('inc/charsets');
 	
 	$erreurs = array();
-	if (!$charset=_request('charset'))
-		$erreurs['charset'] = _T('info_obligatoire');
-	elseif ($charset!='utf-8' AND !load_charset($charset))
-		$erreurs['charset'] = _T('utf8_convert_erreur_orig', array('charset' => entites_html($charset)));
+	if (!$charset=_request('charset')) {
+			$erreurs['charset'] = _T('info_obligatoire');
+	} elseif ($charset!='utf-8' AND !load_charset($charset)) {
+			$erreurs['charset'] = _T('utf8_convert_erreur_orig', array('charset' => entites_html($charset)));
+	}
 
 	return $erreurs;
 }