spip / SPIP

ecrire/inc/precharger_article.php

Indentation +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 // new=oui = article a creer si on valide le formulaire
 // http://doc.spip.org/@inc_article_select_dist
 function inc_precharger_article_dist($id_article, $id_rubrique=0, $lier_trad=0) {
-	return precharger_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
+    return precharger_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
 }
 
 
@@ -32,7 +32,7 @@ 
 //
 // (fonction facultative si pas de changement dans les traitements)
 function inc_precharger_traduction_article_dist($id_article, $id_rubrique=0, $lier_trad=0) {
-	return precharger_traduction_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
+    return precharger_traduction_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
 }
 
 

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 // lier_trad = l'associer a l'article numero $lier_trad
 // new=oui = article a creer si on valide le formulaire
 // http://doc.spip.org/@inc_article_select_dist
-function inc_precharger_article_dist($id_article, $id_rubrique=0, $lier_trad=0) {
+function inc_precharger_article_dist($id_article, $id_rubrique = 0, $lier_trad = 0) {
 	return precharger_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
 }
 
@@ -31,7 +31,7 @@ 
 // on initialise les donnees de maniere specifique
 //
 // (fonction facultative si pas de changement dans les traitements)
-function inc_precharger_traduction_article_dist($id_article, $id_rubrique=0, $lier_trad=0) {
+function inc_precharger_traduction_article_dist($id_article, $id_rubrique = 0, $lier_trad = 0) {
 	return precharger_traduction_objet('article', $id_article, $id_rubrique, $lier_trad, 'titre');
 }
 

Braces +3 added lines, -1 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/precharger_objet');
 

ecrire/inc/mail.php

Indentation +6 added lines, -6 removed lines

@@ -17,13 +17,13 @@
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 if (!function_exists('envoyer_mail')) {
-	define('_FUNCTION_ENVOYER_MAIL', charger_fonction('envoyer_mail', 'inc'));
+    define('_FUNCTION_ENVOYER_MAIL', charger_fonction('envoyer_mail', 'inc'));
 // http://doc.spip.org/@envoyer_mail
-	function envoyer_mail() {
-		$args = func_get_args();
-		if (_FUNCTION_ENVOYER_MAIL)
-			return call_user_func_array(_FUNCTION_ENVOYER_MAIL, $args);
-	}
+    function envoyer_mail() {
+        $args = func_get_args();
+        if (_FUNCTION_ENVOYER_MAIL)
+            return call_user_func_array(_FUNCTION_ENVOYER_MAIL, $args);
+    }
 }
 
 

Braces +6 added lines, -3 removed lines

@@ -14,15 +14,18 @@
 ## Module de compatibilite ascendante : desormais inc/envoyer_mail
 ##
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 if (!function_exists('envoyer_mail')) {
 	define('_FUNCTION_ENVOYER_MAIL', charger_fonction('envoyer_mail', 'inc'));
 // http://doc.spip.org/@envoyer_mail
 	function envoyer_mail() {
 		$args = func_get_args();
-		if (_FUNCTION_ENVOYER_MAIL)
-			return call_user_func_array(_FUNCTION_ENVOYER_MAIL, $args);
+		if (_FUNCTION_ENVOYER_MAIL) {
+					return call_user_func_array(_FUNCTION_ENVOYER_MAIL, $args);
+		}
 	}
 }
 

ecrire/inc/minipres.php

Indentation +88 added lines, -88 removed lines

@@ -29,64 +29,64 @@ 
  * @return string
  */
 function install_debut_html($titre = 'AUTO', $onLoad = '', $all_inline = false) {
-	global $spip_lang_right,$spip_lang_left;
+    global $spip_lang_right,$spip_lang_left;
 	
-	utiliser_langue_visiteur();
-
-	http_no_cache();
-
-	if ($titre=='AUTO')
-		$titre=_T('info_installation_systeme_publication');
-
-	# le charset est en utf-8, pour recuperer le nom comme il faut
-	# lors de l'installation
-	if (!headers_sent())
-		header('Content-Type: text/html; charset=utf-8');
-
-	$css = "";
-	$files = array('reset.css','clear.css','minipres.css');
-	if ($all_inline){
-		// inliner les CSS (optimisation de la page minipres qui passe en un seul hit a la demande)
-		foreach ($files as $name){
-			$file = direction_css(find_in_theme($name));
-			if (function_exists("compacte"))
-				$file = compacte($file);
-			else
-				$file = url_absolue_css($file); // precaution
-			lire_fichier($file,$c);
-			$css .= $c;
-		}
-		$css = "<style type='text/css'>".$css."</style>";
-	}
-	else{
-		foreach ($files as $name){
-			$file = direction_css(find_in_theme($name));
-			$css .= "<link rel='stylesheet' href='$file' type='text/css' />\n";
-		}
-	}
-
-	// au cas ou minipres() est appele avant spip_initialisation_suite()
-	if (!defined('_DOCTYPE_ECRIRE')) define('_DOCTYPE_ECRIRE', '');
-	return  _DOCTYPE_ECRIRE.
-		html_lang_attributes().
-		"<head>\n".
-		"<title>".
-		textebrut($titre).
-		"</title>\n".
-		"<meta name='viewport' content='width=device-width' />\n".
-		$css .
+    utiliser_langue_visiteur();
+
+    http_no_cache();
+
+    if ($titre=='AUTO')
+        $titre=_T('info_installation_systeme_publication');
+
+    # le charset est en utf-8, pour recuperer le nom comme il faut
+    # lors de l'installation
+    if (!headers_sent())
+        header('Content-Type: text/html; charset=utf-8');
+
+    $css = "";
+    $files = array('reset.css','clear.css','minipres.css');
+    if ($all_inline){
+        // inliner les CSS (optimisation de la page minipres qui passe en un seul hit a la demande)
+        foreach ($files as $name){
+            $file = direction_css(find_in_theme($name));
+            if (function_exists("compacte"))
+                $file = compacte($file);
+            else
+                $file = url_absolue_css($file); // precaution
+            lire_fichier($file,$c);
+            $css .= $c;
+        }
+        $css = "<style type='text/css'>".$css."</style>";
+    }
+    else{
+        foreach ($files as $name){
+            $file = direction_css(find_in_theme($name));
+            $css .= "<link rel='stylesheet' href='$file' type='text/css' />\n";
+        }
+    }
+
+    // au cas ou minipres() est appele avant spip_initialisation_suite()
+    if (!defined('_DOCTYPE_ECRIRE')) define('_DOCTYPE_ECRIRE', '');
+    return  _DOCTYPE_ECRIRE.
+        html_lang_attributes().
+        "<head>\n".
+        "<title>".
+        textebrut($titre).
+        "</title>\n".
+        "<meta name='viewport' content='width=device-width' />\n".
+        $css .
 "</head>
 <body".$onLoad." class='minipres'>
 	<div id='minipres'>
 	<h1>".
-	  $titre .
-	  "</h1>
+        $titre .
+        "</h1>
 	<div>\n";
 }
 
 // http://doc.spip.org/@install_fin_html
 function install_fin_html() {
-	return "\n\t</div>\n\t</div>\n</body>\n</html>";
+    return "\n\t</div>\n\t</div>\n</body>\n</html>";
 }
 
 
@@ -105,45 +105,45 @@ 
  */
 function minipres($titre='', $corps="", $onload='', $all_inline = false)
 {
-	if (!defined('_AJAX')) define('_AJAX', false); // par securite
-	if (!$titre) {
-		if (!_AJAX)
-			http_status(403);
-		if (!$titre = _request('action')
-		AND !$titre = _request('exec')
-		AND !$titre = _request('page'))
-			$titre = '?';
-
-		$titre = spip_htmlspecialchars($titre);
-
-		$titre = ($titre == 'install')
-		  ?  _T('avis_espace_interdit')
-		  : $titre . '&nbsp;: '. _T('info_acces_interdit');
-
-		$statut = isset($GLOBALS['visiteur_session']['statut']) ? $GLOBALS['visiteur_session']['statut'] : '';
-		$nom    = isset($GLOBALS['visiteur_session']['nom']) ? $GLOBALS['visiteur_session']['nom'] : '';
-
-		if ($statut != '0minirezo') {
-			$titre = _T('info_acces_interdit');
-		}
-
-		$corps = generer_form_ecrire('accueil', '','',
-						$statut ? _T('public:accueil_site') : _T('public:lien_connecter')
-		);
-		spip_log($nom . " $titre " . $_SERVER['REQUEST_URI']);
-	}
-
-	if (!_AJAX)
-		return install_debut_html($titre, $onload, $all_inline)
-		. $corps
-		. install_fin_html();
-	else {
-		include_spip('inc/headers');
-		include_spip('inc/actions');
-		$url = self('&',true);
-		foreach ($_POST as $v => $c)
-			$url = parametre_url($url, $v, $c, '&');
-		ajax_retour("<div>".$titre . redirige_formulaire($url)."</div>",false);
-	}
+    if (!defined('_AJAX')) define('_AJAX', false); // par securite
+    if (!$titre) {
+        if (!_AJAX)
+            http_status(403);
+        if (!$titre = _request('action')
+        AND !$titre = _request('exec')
+        AND !$titre = _request('page'))
+            $titre = '?';
+
+        $titre = spip_htmlspecialchars($titre);
+
+        $titre = ($titre == 'install')
+          ?  _T('avis_espace_interdit')
+          : $titre . '&nbsp;: '. _T('info_acces_interdit');
+
+        $statut = isset($GLOBALS['visiteur_session']['statut']) ? $GLOBALS['visiteur_session']['statut'] : '';
+        $nom    = isset($GLOBALS['visiteur_session']['nom']) ? $GLOBALS['visiteur_session']['nom'] : '';
+
+        if ($statut != '0minirezo') {
+            $titre = _T('info_acces_interdit');
+        }
+
+        $corps = generer_form_ecrire('accueil', '','',
+                        $statut ? _T('public:accueil_site') : _T('public:lien_connecter')
+        );
+        spip_log($nom . " $titre " . $_SERVER['REQUEST_URI']);
+    }
+
+    if (!_AJAX)
+        return install_debut_html($titre, $onload, $all_inline)
+        . $corps
+        . install_fin_html();
+    else {
+        include_spip('inc/headers');
+        include_spip('inc/actions');
+        $url = self('&',true);
+        foreach ($_POST as $v => $c)
+            $url = parametre_url($url, $v, $c, '&');
+        ajax_retour("<div>".$titre . redirige_formulaire($url)."</div>",false);
+    }
 }
 ?>

Braces +35 added lines, -22 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/headers');
 include_spip('inc/texte'); //inclue inc/lang et inc/filtres
@@ -35,13 +37,15 @@ 
 
 	http_no_cache();
 
-	if ($titre=='AUTO')
-		$titre=_T('info_installation_systeme_publication');
+	if ($titre=='AUTO') {
+			$titre=_T('info_installation_systeme_publication');
+	}
 
 	# le charset est en utf-8, pour recuperer le nom comme il faut
 	# lors de l'installation
-	if (!headers_sent())
-		header('Content-Type: text/html; charset=utf-8');
+	if (!headers_sent()) {
+			header('Content-Type: text/html; charset=utf-8');
+	}
 
 	$css = "";
 	$files = array('reset.css','clear.css','minipres.css');
@@ -49,16 +53,17 @@ 
 		// inliner les CSS (optimisation de la page minipres qui passe en un seul hit a la demande)
 		foreach ($files as $name){
 			$file = direction_css(find_in_theme($name));
-			if (function_exists("compacte"))
-				$file = compacte($file);
-			else
-				$file = url_absolue_css($file); // precaution
+			if (function_exists("compacte")) {
+							$file = compacte($file);
+			} else {
+							$file = url_absolue_css($file);
+			}
+			// precaution
 			lire_fichier($file,$c);
 			$css .= $c;
 		}
 		$css = "<style type='text/css'>".$css."</style>";
-	}
-	else{
+	} else{
 		foreach ($files as $name){
 			$file = direction_css(find_in_theme($name));
 			$css .= "<link rel='stylesheet' href='$file' type='text/css' />\n";
@@ -66,7 +71,9 @@ 
 	}
 
 	// au cas ou minipres() est appele avant spip_initialisation_suite()
-	if (!defined('_DOCTYPE_ECRIRE')) define('_DOCTYPE_ECRIRE', '');
+	if (!defined('_DOCTYPE_ECRIRE')) {
+	    define('_DOCTYPE_ECRIRE', '');
+	}
 	return  _DOCTYPE_ECRIRE.
 		html_lang_attributes().
 		"<head>\n".
@@ -105,14 +112,19 @@ 
  */
 function minipres($titre='', $corps="", $onload='', $all_inline = false)
 {
-	if (!defined('_AJAX')) define('_AJAX', false); // par securite
+	if (!defined('_AJAX')) {
+	    define('_AJAX', false);
+	}
+	// par securite
 	if (!$titre) {
-		if (!_AJAX)
-			http_status(403);
+		if (!_AJAX) {
+					http_status(403);
+		}
 		if (!$titre = _request('action')
 		AND !$titre = _request('exec')
-		AND !$titre = _request('page'))
-			$titre = '?';
+		AND !$titre = _request('page')) {
+					$titre = '?';
+		}
 
 		$titre = spip_htmlspecialchars($titre);
 
@@ -133,16 +145,17 @@ 
 		spip_log($nom . " $titre " . $_SERVER['REQUEST_URI']);
 	}
 
-	if (!_AJAX)
-		return install_debut_html($titre, $onload, $all_inline)
+	if (!_AJAX) {
+			return install_debut_html($titre, $onload, $all_inline)
 		. $corps
 		. install_fin_html();
-	else {
+	} else {
 		include_spip('inc/headers');
 		include_spip('inc/actions');
 		$url = self('&',true);
-		foreach ($_POST as $v => $c)
-			$url = parametre_url($url, $v, $c, '&');
+		foreach ($_POST as $v => $c) {
+					$url = parametre_url($url, $v, $c, '&');
+		}
 		ajax_retour("<div>".$titre . redirige_formulaire($url)."</div>",false);
 	}
 }

Spacing +17 added lines, -17 removed lines

@@ -29,14 +29,14 @@ 
  * @return string
  */
 function install_debut_html($titre = 'AUTO', $onLoad = '', $all_inline = false) {
-	global $spip_lang_right,$spip_lang_left;
+	global $spip_lang_right, $spip_lang_left;
 	
 	utiliser_langue_visiteur();
 
 	http_no_cache();
 
-	if ($titre=='AUTO')
-		$titre=_T('info_installation_systeme_publication');
+	if ($titre == 'AUTO')
+		$titre = _T('info_installation_systeme_publication');
 
 	# le charset est en utf-8, pour recuperer le nom comme il faut
 	# lors de l'installation
@@ -44,22 +44,22 @@ 
 		header('Content-Type: text/html; charset=utf-8');
 
 	$css = "";
-	$files = array('reset.css','clear.css','minipres.css');
-	if ($all_inline){
+	$files = array('reset.css', 'clear.css', 'minipres.css');
+	if ($all_inline) {
 		// inliner les CSS (optimisation de la page minipres qui passe en un seul hit a la demande)
-		foreach ($files as $name){
+		foreach ($files as $name) {
 			$file = direction_css(find_in_theme($name));
 			if (function_exists("compacte"))
 				$file = compacte($file);
 			else
 				$file = url_absolue_css($file); // precaution
-			lire_fichier($file,$c);
+			lire_fichier($file, $c);
 			$css .= $c;
 		}
 		$css = "<style type='text/css'>".$css."</style>";
 	}
-	else{
-		foreach ($files as $name){
+	else {
+		foreach ($files as $name) {
 			$file = direction_css(find_in_theme($name));
 			$css .= "<link rel='stylesheet' href='$file' type='text/css' />\n";
 		}
@@ -74,12 +74,12 @@ 
 		textebrut($titre).
 		"</title>\n".
 		"<meta name='viewport' content='width=device-width' />\n".
-		$css .
+		$css.
 "</head>
 <body".$onLoad." class='minipres'>
 	<div id='minipres'>
 	<h1>".
-	  $titre .
+	  $titre.
 	  "</h1>
 	<div>\n";
 }
@@ -103,7 +103,7 @@ 
  *   inliner les css et js dans la page (limiter le nombre de hits)
  * @return string
  */
-function minipres($titre='', $corps="", $onload='', $all_inline = false)
+function minipres($titre = '', $corps = "", $onload = '', $all_inline = false)
 {
 	if (!defined('_AJAX')) define('_AJAX', false); // par securite
 	if (!$titre) {
@@ -118,7 +118,7 @@ 
 
 		$titre = ($titre == 'install')
 		  ?  _T('avis_espace_interdit')
-		  : $titre . '&nbsp;: '. _T('info_acces_interdit');
+		  : $titre.'&nbsp;: '._T('info_acces_interdit');
 
 		$statut = isset($GLOBALS['visiteur_session']['statut']) ? $GLOBALS['visiteur_session']['statut'] : '';
 		$nom    = isset($GLOBALS['visiteur_session']['nom']) ? $GLOBALS['visiteur_session']['nom'] : '';
@@ -127,10 +127,10 @@ 
 			$titre = _T('info_acces_interdit');
 		}
 
-		$corps = generer_form_ecrire('accueil', '','',
+		$corps = generer_form_ecrire('accueil', '', '',
 						$statut ? _T('public:accueil_site') : _T('public:lien_connecter')
 		);
-		spip_log($nom . " $titre " . $_SERVER['REQUEST_URI']);
+		spip_log($nom." $titre ".$_SERVER['REQUEST_URI']);
 	}
 
 	if (!_AJAX)
@@ -140,10 +140,10 @@ 
 	else {
 		include_spip('inc/headers');
 		include_spip('inc/actions');
-		$url = self('&',true);
+		$url = self('&', true);
 		foreach ($_POST as $v => $c)
 			$url = parametre_url($url, $v, $c, '&');
-		ajax_retour("<div>".$titre . redirige_formulaire($url)."</div>",false);
+		ajax_retour("<div>".$titre.redirige_formulaire($url)."</div>", false);
 	}
 }
 ?>

ecrire/inc/filtres_mini.php

Indentation +89 added lines, -89 removed lines

@@ -24,16 +24,16 @@ 
 //
 // http://doc.spip.org/@resolve_path
 function resolve_path($url) {
-	list($url, $query) = array_pad(explode('?', $url, 2), 2, null);
-	while (preg_match(',/\.?/,', $url, $regs)		# supprime // et /./
-	OR preg_match(',/[^/]*/\.\./,S', $url, $regs)	# supprime /toto/../
-	OR preg_match(',^/\.\./,S', $url, $regs))		# supprime les /../ du haut
-		$url = str_replace($regs[0], '/', $url);
+    list($url, $query) = array_pad(explode('?', $url, 2), 2, null);
+    while (preg_match(',/\.?/,', $url, $regs)		# supprime // et /./
+    OR preg_match(',/[^/]*/\.\./,S', $url, $regs)	# supprime /toto/../
+    OR preg_match(',^/\.\./,S', $url, $regs))		# supprime les /../ du haut
+        $url = str_replace($regs[0], '/', $url);
 
-	if ($query)
-		$url .= '?'.$query;
+    if ($query)
+        $url .= '?'.$query;
 
-	return '/'.preg_replace(',^/,S', '', $url);
+    return '/'.preg_replace(',^/,S', '', $url);
 }
 
 // 
@@ -44,46 +44,46 @@ 
 // http://doc.spip.org/@suivre_lien
 function suivre_lien($url, $lien) {
 
-	if (preg_match(',^(mailto|javascript|data):,iS', $lien))
-		return $lien;
-	if (preg_match(';^((?:[a-z]{3,7}:)?//.*?)(/.*)?$;iS', $lien, $r))
-		return $r[1].resolve_path($r[2]);
-
-	# L'url site spip est un lien absolu aussi
-	if ($lien == $GLOBALS['meta']['adresse_site']){
-		return $lien;
-	}
-
-	# lien relatif, il faut verifier l'url de base
-	# commencer par virer la chaine de get de l'url de base
-	if (preg_match(';^((?:[a-z]{3,7}:)?//[^/]+)(/.*?/?)?([^/#?]*)([?][^#]*)?(#.*)?$;S', $url, $regs)) {
-		$debut = $regs[1];
-		$dir = !strlen($regs[2]) ? '/' : $regs[2];
-		$mot = $regs[3];
-		$get = isset($regs[4])?$regs[4]:"";
-		$hash = isset($regs[5])?$regs[5]:"";
-	}
-	switch (substr($lien,0,1)) {
-		case '/':
-			return $debut . resolve_path($lien);
-		case '#':
-			return $debut . resolve_path($dir.$mot.$get.$lien);
-		case '':
-			return $debut . resolve_path($dir.$mot.$get.$hash);
-		default:
-			return $debut . resolve_path($dir.$lien);
-	}
+    if (preg_match(',^(mailto|javascript|data):,iS', $lien))
+        return $lien;
+    if (preg_match(';^((?:[a-z]{3,7}:)?//.*?)(/.*)?$;iS', $lien, $r))
+        return $r[1].resolve_path($r[2]);
+
+    # L'url site spip est un lien absolu aussi
+    if ($lien == $GLOBALS['meta']['adresse_site']){
+        return $lien;
+    }
+
+    # lien relatif, il faut verifier l'url de base
+    # commencer par virer la chaine de get de l'url de base
+    if (preg_match(';^((?:[a-z]{3,7}:)?//[^/]+)(/.*?/?)?([^/#?]*)([?][^#]*)?(#.*)?$;S', $url, $regs)) {
+        $debut = $regs[1];
+        $dir = !strlen($regs[2]) ? '/' : $regs[2];
+        $mot = $regs[3];
+        $get = isset($regs[4])?$regs[4]:"";
+        $hash = isset($regs[5])?$regs[5]:"";
+    }
+    switch (substr($lien,0,1)) {
+        case '/':
+            return $debut . resolve_path($lien);
+        case '#':
+            return $debut . resolve_path($dir.$mot.$get.$lien);
+        case '':
+            return $debut . resolve_path($dir.$mot.$get.$hash);
+        default:
+            return $debut . resolve_path($dir.$lien);
+    }
 }
 
 // un filtre pour transformer les URLs relatives en URLs absolues ;
 // ne s'applique qu'aux #URL_XXXX
 // http://doc.spip.org/@url_absolue
 function url_absolue($url, $base='') {
-	if (strlen($url = trim($url)) == 0)
-		return '';
-	if (!$base)
-		$base = url_de_base() . (_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
-	return suivre_lien($base, $url);
+    if (strlen($url = trim($url)) == 0)
+        return '';
+    if (!$base)
+        $base = url_de_base() . (_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
+    return suivre_lien($base, $url);
 }
 
 /**
@@ -93,33 +93,33 @@ 
  * @return string
  */
 function protocole_implicite($url_absolue){
-	return preg_replace(";^[a-z]{3,7}://;i","//",$url_absolue);
+    return preg_replace(";^[a-z]{3,7}://;i","//",$url_absolue);
 }
 
 // un filtre pour transformer les URLs relatives en URLs absolues ;
 // ne s'applique qu'aux textes contenant des liens
 // http://doc.spip.org/@liens_absolus
 function liens_absolus($texte, $base='') {
-	if (preg_match_all(',(<(a|link|image|img|script)\s[^<>]*(href|src)=[^<>]*>),imsS', 
-	$texte, $liens, PREG_SET_ORDER)) {
-		if (!function_exists('extraire_attribut')) {
-			include_spip('inc/filtres');
-		}
-		foreach ($liens as $lien) {
-			foreach(array('href', 'src') as $attr) {
-				$href = extraire_attribut($lien[0], $attr);
-				if (strlen($href)>0) {
-					$abs = url_absolue($href, $base);
-					if ($href != $abs and !preg_match('/^#/',$href)) {
-						$texte_lien = inserer_attribut($lien[0], $attr, $abs);
-						$texte = str_replace($lien[0],$texte_lien,$texte);
-					}
-				}
-			}
-		}
-	}
-
-	return $texte;
+    if (preg_match_all(',(<(a|link|image|img|script)\s[^<>]*(href|src)=[^<>]*>),imsS', 
+    $texte, $liens, PREG_SET_ORDER)) {
+        if (!function_exists('extraire_attribut')) {
+            include_spip('inc/filtres');
+        }
+        foreach ($liens as $lien) {
+            foreach(array('href', 'src') as $attr) {
+                $href = extraire_attribut($lien[0], $attr);
+                if (strlen($href)>0) {
+                    $abs = url_absolue($href, $base);
+                    if ($href != $abs and !preg_match('/^#/',$href)) {
+                        $texte_lien = inserer_attribut($lien[0], $attr, $abs);
+                        $texte = str_replace($lien[0],$texte_lien,$texte);
+                    }
+                }
+            }
+        }
+    }
+
+    return $texte;
 }
 
 //
@@ -127,10 +127,10 @@ 
 //
 // http://doc.spip.org/@abs_url
 function abs_url($texte, $base='') {
-	if ($GLOBALS['mode_abs_url'] == 'url')
-		return url_absolue($texte, $base);
-	else
-		return liens_absolus($texte, $base);
+    if ($GLOBALS['mode_abs_url'] == 'url')
+        return url_absolue($texte, $base);
+    else
+        return liens_absolus($texte, $base);
 }
 
 /**
@@ -143,17 +143,17 @@ 
 * @return string
 */
 function spip_htmlspecialchars($string, $flags=null, $encoding='ISO-8859-1', $double_encode = true){
-	if (is_null($flags)) {
-		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
-			$flags = ENT_COMPAT;
-		else
-			$flags = ENT_COMPAT|ENT_HTML401;
-	}
-
-	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlspecialchars($string,$flags,$encoding);
-	else
-		return htmlspecialchars($string,$flags,$encoding,$double_encode);
+    if (is_null($flags)) {
+        if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
+            $flags = ENT_COMPAT;
+        else
+            $flags = ENT_COMPAT|ENT_HTML401;
+    }
+
+    if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
+        return htmlspecialchars($string,$flags,$encoding);
+    else
+        return htmlspecialchars($string,$flags,$encoding,$double_encode);
 }
 
 /**
@@ -166,16 +166,16 @@ 
 * @return string
 */
 function spip_htmlentities($string,$flags=null,$encoding = 'ISO-8859-1',$double_encode = true){
-	if (is_null($flags)) {
-		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
-			$flags = ENT_COMPAT;
-		else
-			$flags = ENT_COMPAT|ENT_HTML401;
-	}
-
-	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlentities($string,$flags,$encoding);
-	else
-		return htmlentities($string,$flags,$encoding,$double_encode);
+    if (is_null($flags)) {
+        if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
+            $flags = ENT_COMPAT;
+        else
+            $flags = ENT_COMPAT|ENT_HTML401;
+    }
+
+    if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
+        return htmlentities($string,$flags,$encoding);
+    else
+        return htmlentities($string,$flags,$encoding,$double_encode);
 }
 ?>

Spacing +26 added lines, -26 removed lines

@@ -50,7 +50,7 @@ 
 		return $r[1].resolve_path($r[2]);
 
 	# L'url site spip est un lien absolu aussi
-	if ($lien == $GLOBALS['meta']['adresse_site']){
+	if ($lien == $GLOBALS['meta']['adresse_site']) {
 		return $lien;
 	}
 
@@ -60,29 +60,29 @@ 
 		$debut = $regs[1];
 		$dir = !strlen($regs[2]) ? '/' : $regs[2];
 		$mot = $regs[3];
-		$get = isset($regs[4])?$regs[4]:"";
-		$hash = isset($regs[5])?$regs[5]:"";
+		$get = isset($regs[4]) ? $regs[4] : "";
+		$hash = isset($regs[5]) ? $regs[5] : "";
 	}
-	switch (substr($lien,0,1)) {
+	switch (substr($lien, 0, 1)) {
 		case '/':
-			return $debut . resolve_path($lien);
+			return $debut.resolve_path($lien);
 		case '#':
-			return $debut . resolve_path($dir.$mot.$get.$lien);
+			return $debut.resolve_path($dir.$mot.$get.$lien);
 		case '':
-			return $debut . resolve_path($dir.$mot.$get.$hash);
+			return $debut.resolve_path($dir.$mot.$get.$hash);
 		default:
-			return $debut . resolve_path($dir.$lien);
+			return $debut.resolve_path($dir.$lien);
 	}
 }
 
 // un filtre pour transformer les URLs relatives en URLs absolues ;
 // ne s'applique qu'aux #URL_XXXX
 // http://doc.spip.org/@url_absolue
-function url_absolue($url, $base='') {
+function url_absolue($url, $base = '') {
 	if (strlen($url = trim($url)) == 0)
 		return '';
 	if (!$base)
-		$base = url_de_base() . (_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
+		$base = url_de_base().(_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
 	return suivre_lien($base, $url);
 }
 
@@ -92,27 +92,27 @@ 
  * @param string $url_absolue
  * @return string
  */
-function protocole_implicite($url_absolue){
-	return preg_replace(";^[a-z]{3,7}://;i","//",$url_absolue);
+function protocole_implicite($url_absolue) {
+	return preg_replace(";^[a-z]{3,7}://;i", "//", $url_absolue);
 }
 
 // un filtre pour transformer les URLs relatives en URLs absolues ;
 // ne s'applique qu'aux textes contenant des liens
 // http://doc.spip.org/@liens_absolus
-function liens_absolus($texte, $base='') {
+function liens_absolus($texte, $base = '') {
 	if (preg_match_all(',(<(a|link|image|img|script)\s[^<>]*(href|src)=[^<>]*>),imsS', 
 	$texte, $liens, PREG_SET_ORDER)) {
 		if (!function_exists('extraire_attribut')) {
 			include_spip('inc/filtres');
 		}
 		foreach ($liens as $lien) {
-			foreach(array('href', 'src') as $attr) {
+			foreach (array('href', 'src') as $attr) {
 				$href = extraire_attribut($lien[0], $attr);
-				if (strlen($href)>0) {
+				if (strlen($href) > 0) {
 					$abs = url_absolue($href, $base);
-					if ($href != $abs and !preg_match('/^#/',$href)) {
+					if ($href != $abs and !preg_match('/^#/', $href)) {
 						$texte_lien = inserer_attribut($lien[0], $attr, $abs);
-						$texte = str_replace($lien[0],$texte_lien,$texte);
+						$texte = str_replace($lien[0], $texte_lien, $texte);
 					}
 				}
 			}
@@ -126,7 +126,7 @@ 
 // Ce filtre public va traiter les URL ou les <a href>
 //
 // http://doc.spip.org/@abs_url
-function abs_url($texte, $base='') {
+function abs_url($texte, $base = '') {
 	if ($GLOBALS['mode_abs_url'] == 'url')
 		return url_absolue($texte, $base);
 	else
@@ -142,18 +142,18 @@ 
 * @param bool $double_encode
 * @return string
 */
-function spip_htmlspecialchars($string, $flags=null, $encoding='ISO-8859-1', $double_encode = true){
+function spip_htmlspecialchars($string, $flags = null, $encoding = 'ISO-8859-1', $double_encode = true) {
 	if (is_null($flags)) {
 		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
 			$flags = ENT_COMPAT;
 		else
-			$flags = ENT_COMPAT|ENT_HTML401;
+			$flags = ENT_COMPAT | ENT_HTML401;
 	}
 
 	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlspecialchars($string,$flags,$encoding);
+		return htmlspecialchars($string, $flags, $encoding);
 	else
-		return htmlspecialchars($string,$flags,$encoding,$double_encode);
+		return htmlspecialchars($string, $flags, $encoding, $double_encode);
 }
 
 /**
@@ -165,17 +165,17 @@ 
 * @param bool $double_encode
 * @return string
 */
-function spip_htmlentities($string,$flags=null,$encoding = 'ISO-8859-1',$double_encode = true){
+function spip_htmlentities($string, $flags = null, $encoding = 'ISO-8859-1', $double_encode = true) {
 	if (is_null($flags)) {
 		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
 			$flags = ENT_COMPAT;
 		else
-			$flags = ENT_COMPAT|ENT_HTML401;
+			$flags = ENT_COMPAT | ENT_HTML401;
 	}
 
 	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlentities($string,$flags,$encoding);
+		return htmlentities($string, $flags, $encoding);
 	else
-		return htmlentities($string,$flags,$encoding,$double_encode);
+		return htmlentities($string, $flags, $encoding, $double_encode);
 }
 ?>

Braces +49 added lines, -35 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 //
 // Filtres d'URLs
@@ -27,11 +29,14 @@ 
 	list($url, $query) = array_pad(explode('?', $url, 2), 2, null);
 	while (preg_match(',/\.?/,', $url, $regs)		# supprime // et /./
 	OR preg_match(',/[^/]*/\.\./,S', $url, $regs)	# supprime /toto/../
-	OR preg_match(',^/\.\./,S', $url, $regs))		# supprime les /../ du haut
+	OR preg_match(',^/\.\./,S', $url, $regs)) {
+	    # supprime les /../ du haut
 		$url = str_replace($regs[0], '/', $url);
+	}
 
-	if ($query)
-		$url .= '?'.$query;
+	if ($query) {
+			$url .= '?'.$query;
+	}
 
 	return '/'.preg_replace(',^/,S', '', $url);
 }
@@ -44,10 +49,12 @@ 
 // http://doc.spip.org/@suivre_lien
 function suivre_lien($url, $lien) {
 
-	if (preg_match(',^(mailto|javascript|data):,iS', $lien))
-		return $lien;
-	if (preg_match(';^((?:[a-z]{3,7}:)?//.*?)(/.*)?$;iS', $lien, $r))
-		return $r[1].resolve_path($r[2]);
+	if (preg_match(',^(mailto|javascript|data):,iS', $lien)) {
+			return $lien;
+	}
+	if (preg_match(';^((?:[a-z]{3,7}:)?//.*?)(/.*)?$;iS', $lien, $r)) {
+			return $r[1].resolve_path($r[2]);
+	}
 
 	# L'url site spip est un lien absolu aussi
 	if ($lien == $GLOBALS['meta']['adresse_site']){
@@ -79,10 +86,12 @@ 
 // ne s'applique qu'aux #URL_XXXX
 // http://doc.spip.org/@url_absolue
 function url_absolue($url, $base='') {
-	if (strlen($url = trim($url)) == 0)
-		return '';
-	if (!$base)
-		$base = url_de_base() . (_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
+	if (strlen($url = trim($url)) == 0) {
+			return '';
+	}
+	if (!$base) {
+			$base = url_de_base() . (_DIR_RACINE ? _DIR_RESTREINT_ABS : '');
+	}
 	return suivre_lien($base, $url);
 }
 
@@ -127,11 +136,12 @@ 
 //
 // http://doc.spip.org/@abs_url
 function abs_url($texte, $base='') {
-	if ($GLOBALS['mode_abs_url'] == 'url')
-		return url_absolue($texte, $base);
-	else
-		return liens_absolus($texte, $base);
-}
+	if ($GLOBALS['mode_abs_url'] == 'url') {
+			return url_absolue($texte, $base);
+	} else {
+			return liens_absolus($texte, $base);
+	}
+	}
 
 /**
 * htmlspecialchars wrapper (PHP >= 5.4 compat issue)
@@ -144,17 +154,19 @@ 
 */
 function spip_htmlspecialchars($string, $flags=null, $encoding='ISO-8859-1', $double_encode = true){
 	if (is_null($flags)) {
-		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
-			$flags = ENT_COMPAT;
-		else
-			$flags = ENT_COMPAT|ENT_HTML401;
+		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400) {
+					$flags = ENT_COMPAT;
+		} else {
+					$flags = ENT_COMPAT|ENT_HTML401;
+		}
 	}
 
-	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlspecialchars($string,$flags,$encoding);
-	else
-		return htmlspecialchars($string,$flags,$encoding,$double_encode);
-}
+	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203) {
+			return htmlspecialchars($string,$flags,$encoding);
+	} else {
+			return htmlspecialchars($string,$flags,$encoding,$double_encode);
+	}
+	}
 
 /**
 * htmlentities wrapper (PHP >= 5.4 compat issue)
@@ -167,15 +179,17 @@ 
 */
 function spip_htmlentities($string,$flags=null,$encoding = 'ISO-8859-1',$double_encode = true){
 	if (is_null($flags)) {
-		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400)
-			$flags = ENT_COMPAT;
-		else
-			$flags = ENT_COMPAT|ENT_HTML401;
+		if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50400) {
+					$flags = ENT_COMPAT;
+		} else {
+					$flags = ENT_COMPAT|ENT_HTML401;
+		}
 	}
 
-	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203)
-		return htmlentities($string,$flags,$encoding);
-	else
-		return htmlentities($string,$flags,$encoding,$double_encode);
-}
+	if (!defined('PHP_VERSION_ID') OR PHP_VERSION_ID < 50203) {
+			return htmlentities($string,$flags,$encoding);
+	} else {
+			return htmlentities($string,$flags,$encoding,$double_encode);
+	}
+	}
 ?>

ecrire/inc/cvt_multietapes.php

Indentation +122 added lines, -122 removed lines

@@ -72,39 +72,39 @@ 
  * @return array
  */
 function cvtmulti_recuperer_post_precedents($form){
-	include_spip('inc/filtres');
-	if ($form
-	  AND $c = _request('cvtm_prev_post')
-		AND $c = decoder_contexte_ajax($c, $form)){
-		#var_dump($c);
+    include_spip('inc/filtres');
+    if ($form
+      AND $c = _request('cvtm_prev_post')
+        AND $c = decoder_contexte_ajax($c, $form)){
+        #var_dump($c);
 		
-		# reinjecter dans la bonne variable pour permettre de retrouver
-		# toutes les saisies dans un seul tableau
-		if ($_SERVER['REQUEST_METHOD']=='POST')
-			$store = &$_POST;
-		else
-			$store = &$_GET;
-
-		foreach($c as $k=>$v)
-			// on ecrase pas si saisi a nouveau !
-			if (!isset($store[$k]))
-				$_REQUEST[$k] = $store[$k] = $v;
-			// mais si tableau des deux cotes, on merge avec priorite a la derniere saisie
-			elseif(is_array($store[$k])
-			  AND is_array($v)
-				AND $z = array_keys($v)
-				AND !is_numeric(reset($z))
-				AND $z = array_keys($store[$k])
-				AND !is_numeric(reset($z))
-				)
-				$_REQUEST[$k] = $store[$k] = array_merge($v,$store[$k]);
-
-		// vider pour eviter un second appel a verifier_n
-		// en cas de double implementation (unipotence)
-		set_request('cvtm_prev_post');
-		return array($c['_etape'],$c['_etapes']);
-	}
-	return false;
+        # reinjecter dans la bonne variable pour permettre de retrouver
+        # toutes les saisies dans un seul tableau
+        if ($_SERVER['REQUEST_METHOD']=='POST')
+            $store = &$_POST;
+        else
+            $store = &$_GET;
+
+        foreach($c as $k=>$v)
+            // on ecrase pas si saisi a nouveau !
+            if (!isset($store[$k]))
+                $_REQUEST[$k] = $store[$k] = $v;
+            // mais si tableau des deux cotes, on merge avec priorite a la derniere saisie
+            elseif(is_array($store[$k])
+              AND is_array($v)
+                AND $z = array_keys($v)
+                AND !is_numeric(reset($z))
+                AND $z = array_keys($store[$k])
+                AND !is_numeric(reset($z))
+                )
+                $_REQUEST[$k] = $store[$k] = array_merge($v,$store[$k]);
+
+        // vider pour eviter un second appel a verifier_n
+        // en cas de double implementation (unipotence)
+        set_request('cvtm_prev_post');
+        return array($c['_etape'],$c['_etapes']);
+    }
+    return false;
 }
 
 /**
@@ -117,25 +117,25 @@ 
  * @return array
  */
 function cvtmulti_sauver_post($form, $je_suis_poste, &$valeurs){
-	if (!isset($valeurs['_cvtm_prev_post'])){
-		$post = array('_etape'=>$valeurs['_etape'],'_etapes'=>$valeurs['_etapes']);
-		foreach(array_keys($valeurs) as $champ){
-			if (substr($champ,0,1)!=='_'){
-				if ($je_suis_poste || (isset($valeurs['_forcer_request']) && $valeurs['_forcer_request'])) {
-					if (($v = _request($champ))!==NULL)
-						$post[$champ] = $v;
-				}
-			}
-		}
-		include_spip('inc/filtres');
-		$c = encoder_contexte_ajax($post,$form);
-		if (!isset($valeurs['_hidden']))
-			$valeurs['_hidden'] = '';
-		$valeurs['_hidden'] .= "<input type='hidden' name='cvtm_prev_post' value='$c' />";
-		// marquer comme fait, pour eviter double encodage (unipotence)
-		$valeurs['_cvtm_prev_post'] = true;
-	}
-	return $valeurs;
+    if (!isset($valeurs['_cvtm_prev_post'])){
+        $post = array('_etape'=>$valeurs['_etape'],'_etapes'=>$valeurs['_etapes']);
+        foreach(array_keys($valeurs) as $champ){
+            if (substr($champ,0,1)!=='_'){
+                if ($je_suis_poste || (isset($valeurs['_forcer_request']) && $valeurs['_forcer_request'])) {
+                    if (($v = _request($champ))!==NULL)
+                        $post[$champ] = $v;
+                }
+            }
+        }
+        include_spip('inc/filtres');
+        $c = encoder_contexte_ajax($post,$form);
+        if (!isset($valeurs['_hidden']))
+            $valeurs['_hidden'] = '';
+        $valeurs['_hidden'] .= "<input type='hidden' name='cvtm_prev_post' value='$c' />";
+        // marquer comme fait, pour eviter double encodage (unipotence)
+        $valeurs['_cvtm_prev_post'] = true;
+    }
+    return $valeurs;
 }
 
 
@@ -147,22 +147,22 @@ 
  * @return <type> 
  */
 function cvtmulti_formulaire_charger($flux){
-	#var_dump($flux['data']['_etapes']);
-	if (is_array($flux['data'])
-	  AND isset($flux['data']['_etapes'])){
-		$form = $flux['args']['form'];
-		$je_suis_poste = $flux['args']['je_suis_poste'];
-		$nb_etapes = $flux['data']['_etapes'];
-		$etape = _request('_etape');
-		$etape = min(max($etape,1),$nb_etapes);
-		set_request('_etape',$etape);
-		$flux['data']['_etape'] = $etape;
-
-		// sauver les posts de cette etape pour les avoir a la prochaine etape
-		$flux['data'] = cvtmulti_sauver_post($form, $je_suis_poste, $flux['data']);
-		#var_dump($flux['data']);
-	}
-	return $flux;
+    #var_dump($flux['data']['_etapes']);
+    if (is_array($flux['data'])
+      AND isset($flux['data']['_etapes'])){
+        $form = $flux['args']['form'];
+        $je_suis_poste = $flux['args']['je_suis_poste'];
+        $nb_etapes = $flux['data']['_etapes'];
+        $etape = _request('_etape');
+        $etape = min(max($etape,1),$nb_etapes);
+        set_request('_etape',$etape);
+        $flux['data']['_etape'] = $etape;
+
+        // sauver les posts de cette etape pour les avoir a la prochaine etape
+        $flux['data'] = cvtmulti_sauver_post($form, $je_suis_poste, $flux['data']);
+        #var_dump($flux['data']);
+    }
+    return $flux;
 }
 
 
@@ -173,53 +173,53 @@ 
  * @return array
  */
 function cvtmulti_formulaire_verifier($flux){
-	#var_dump('Pipe verifier');
+    #var_dump('Pipe verifier');
 	
-	if ($form = $flux['args']['form']
-	  AND ($e = cvtmulti_recuperer_post_precedents($form))!==false){
-		// recuperer l'etape saisie et le nombre d'etapes total
-		list($etape,$etapes) = $e;
-		$etape_demandee = _request('aller_a_etape'); // possibilite de poster en entier dans aller_a_etape
-
-		// lancer les verifs pour chaque etape deja saisie de 1 a $etape
-		$erreurs = array();
-		$derniere_etape_ok = 0;
-		$e = 0;
-		while ($e<$etape AND $e<$etapes){
-			$e++;
-			$erreurs[$e] = array();
-			if ($verifier = charger_fonction("verifier_$e","formulaires/$form/",true))
-				$erreurs[$e] = call_user_func_array($verifier, $flux['args']['args']);
-			elseif ($verifier = charger_fonction("verifier_etape","formulaires/$form/",true)){
-				$args = $flux['args']['args'];
-				array_unshift($args, $e);
-				$erreurs[$e] = call_user_func_array($verifier, $args);
-			}
-			if ($derniere_etape_ok==$e-1 AND !count($erreurs[$e]))
-				$derniere_etape_ok = $e;
-			// possibilite de poster dans _retour_etape_x
-			if (!is_null(_request("_retour_etape_$e")))
-				$etape_demandee = $e;
-		}
-
-		// si la derniere etape OK etait la derniere
-		// on renvoie le flux inchange et ca declenche traiter
-		if ($derniere_etape_ok==$etapes AND !$etape_demandee){
-			return $flux;
-		}
-		else {
-			$etape = $derniere_etape_ok+1;
-			if ($etape_demandee>0 AND $etape_demandee<$etape)
-				$etape = $etape_demandee;
-			$etape = min($etape,$etapes);
-			#var_dump("prochaine etape $etape");
-			// retourner les erreurs de l'etape ciblee
-			$flux['data'] = isset($erreurs[$etape]) ? $erreurs[$etape] : array() ;
-			$flux['data']['_etapes'] = "etape suivante $etape";
-			set_request('_etape',$etape);
-		}
-	}
-	return $flux;
+    if ($form = $flux['args']['form']
+      AND ($e = cvtmulti_recuperer_post_precedents($form))!==false){
+        // recuperer l'etape saisie et le nombre d'etapes total
+        list($etape,$etapes) = $e;
+        $etape_demandee = _request('aller_a_etape'); // possibilite de poster en entier dans aller_a_etape
+
+        // lancer les verifs pour chaque etape deja saisie de 1 a $etape
+        $erreurs = array();
+        $derniere_etape_ok = 0;
+        $e = 0;
+        while ($e<$etape AND $e<$etapes){
+            $e++;
+            $erreurs[$e] = array();
+            if ($verifier = charger_fonction("verifier_$e","formulaires/$form/",true))
+                $erreurs[$e] = call_user_func_array($verifier, $flux['args']['args']);
+            elseif ($verifier = charger_fonction("verifier_etape","formulaires/$form/",true)){
+                $args = $flux['args']['args'];
+                array_unshift($args, $e);
+                $erreurs[$e] = call_user_func_array($verifier, $args);
+            }
+            if ($derniere_etape_ok==$e-1 AND !count($erreurs[$e]))
+                $derniere_etape_ok = $e;
+            // possibilite de poster dans _retour_etape_x
+            if (!is_null(_request("_retour_etape_$e")))
+                $etape_demandee = $e;
+        }
+
+        // si la derniere etape OK etait la derniere
+        // on renvoie le flux inchange et ca declenche traiter
+        if ($derniere_etape_ok==$etapes AND !$etape_demandee){
+            return $flux;
+        }
+        else {
+            $etape = $derniere_etape_ok+1;
+            if ($etape_demandee>0 AND $etape_demandee<$etape)
+                $etape = $etape_demandee;
+            $etape = min($etape,$etapes);
+            #var_dump("prochaine etape $etape");
+            // retourner les erreurs de l'etape ciblee
+            $flux['data'] = isset($erreurs[$etape]) ? $erreurs[$etape] : array() ;
+            $flux['data']['_etapes'] = "etape suivante $etape";
+            set_request('_etape',$etape);
+        }
+    }
+    return $flux;
 }
 
 /**
@@ -231,15 +231,15 @@ 
  * @return array
  */
 function cvtmulti_styliser($flux){
-	if (strncmp($flux['args']['fond'],'formulaires/',12)==0
-	  AND isset($flux['args']['contexte']['_etapes'])
-	  AND isset($flux['args']['contexte']['_etape'])
-	  AND ($e=$flux['args']['contexte']['_etape'])>1
-		AND $ext = $flux['args']['ext']
-		AND $f=$flux['data']
-		AND file_exists($f."_$e.$ext"))
-		$flux['data'] = $f."_$e";
-	return $flux;
+    if (strncmp($flux['args']['fond'],'formulaires/',12)==0
+      AND isset($flux['args']['contexte']['_etapes'])
+      AND isset($flux['args']['contexte']['_etape'])
+      AND ($e=$flux['args']['contexte']['_etape'])>1
+        AND $ext = $flux['args']['ext']
+        AND $f=$flux['data']
+        AND file_exists($f."_$e.$ext"))
+        $flux['data'] = $f."_$e";
+    return $flux;
 }
 
 

Spacing +35 added lines, -35 removed lines

@@ -71,38 +71,38 @@ 
  * @param string $form
  * @return array
  */
-function cvtmulti_recuperer_post_precedents($form){
+function cvtmulti_recuperer_post_precedents($form) {
 	include_spip('inc/filtres');
 	if ($form
 	  AND $c = _request('cvtm_prev_post')
-		AND $c = decoder_contexte_ajax($c, $form)){
+		AND $c = decoder_contexte_ajax($c, $form)) {
 		#var_dump($c);
 		
 		# reinjecter dans la bonne variable pour permettre de retrouver
 		# toutes les saisies dans un seul tableau
-		if ($_SERVER['REQUEST_METHOD']=='POST')
+		if ($_SERVER['REQUEST_METHOD'] == 'POST')
 			$store = &$_POST;
 		else
 			$store = &$_GET;
 
-		foreach($c as $k=>$v)
+		foreach ($c as $k=>$v)
 			// on ecrase pas si saisi a nouveau !
 			if (!isset($store[$k]))
 				$_REQUEST[$k] = $store[$k] = $v;
 			// mais si tableau des deux cotes, on merge avec priorite a la derniere saisie
-			elseif(is_array($store[$k])
+			elseif (is_array($store[$k])
 			  AND is_array($v)
 				AND $z = array_keys($v)
 				AND !is_numeric(reset($z))
 				AND $z = array_keys($store[$k])
 				AND !is_numeric(reset($z))
 				)
-				$_REQUEST[$k] = $store[$k] = array_merge($v,$store[$k]);
+				$_REQUEST[$k] = $store[$k] = array_merge($v, $store[$k]);
 
 		// vider pour eviter un second appel a verifier_n
 		// en cas de double implementation (unipotence)
 		set_request('cvtm_prev_post');
-		return array($c['_etape'],$c['_etapes']);
+		return array($c['_etape'], $c['_etapes']);
 	}
 	return false;
 }
@@ -116,19 +116,19 @@ 
  * @param array $valeurs
  * @return array
  */
-function cvtmulti_sauver_post($form, $je_suis_poste, &$valeurs){
-	if (!isset($valeurs['_cvtm_prev_post'])){
-		$post = array('_etape'=>$valeurs['_etape'],'_etapes'=>$valeurs['_etapes']);
-		foreach(array_keys($valeurs) as $champ){
-			if (substr($champ,0,1)!=='_'){
+function cvtmulti_sauver_post($form, $je_suis_poste, &$valeurs) {
+	if (!isset($valeurs['_cvtm_prev_post'])) {
+		$post = array('_etape'=>$valeurs['_etape'], '_etapes'=>$valeurs['_etapes']);
+		foreach (array_keys($valeurs) as $champ) {
+			if (substr($champ, 0, 1) !== '_') {
 				if ($je_suis_poste || (isset($valeurs['_forcer_request']) && $valeurs['_forcer_request'])) {
-					if (($v = _request($champ))!==NULL)
+					if (($v = _request($champ)) !== NULL)
 						$post[$champ] = $v;
 				}
 			}
 		}
 		include_spip('inc/filtres');
-		$c = encoder_contexte_ajax($post,$form);
+		$c = encoder_contexte_ajax($post, $form);
 		if (!isset($valeurs['_hidden']))
 			$valeurs['_hidden'] = '';
 		$valeurs['_hidden'] .= "<input type='hidden' name='cvtm_prev_post' value='$c' />";
@@ -146,16 +146,16 @@ 
  * @param <type> $flux
  * @return <type> 
  */
-function cvtmulti_formulaire_charger($flux){
+function cvtmulti_formulaire_charger($flux) {
 	#var_dump($flux['data']['_etapes']);
 	if (is_array($flux['data'])
-	  AND isset($flux['data']['_etapes'])){
+	  AND isset($flux['data']['_etapes'])) {
 		$form = $flux['args']['form'];
 		$je_suis_poste = $flux['args']['je_suis_poste'];
 		$nb_etapes = $flux['data']['_etapes'];
 		$etape = _request('_etape');
-		$etape = min(max($etape,1),$nb_etapes);
-		set_request('_etape',$etape);
+		$etape = min(max($etape, 1), $nb_etapes);
+		set_request('_etape', $etape);
 		$flux['data']['_etape'] = $etape;
 
 		// sauver les posts de cette etape pour les avoir a la prochaine etape
@@ -172,30 +172,30 @@ 
  * @param array $flux
  * @return array
  */
-function cvtmulti_formulaire_verifier($flux){
+function cvtmulti_formulaire_verifier($flux) {
 	#var_dump('Pipe verifier');
 	
 	if ($form = $flux['args']['form']
-	  AND ($e = cvtmulti_recuperer_post_precedents($form))!==false){
+	  AND ($e = cvtmulti_recuperer_post_precedents($form)) !== false) {
 		// recuperer l'etape saisie et le nombre d'etapes total
-		list($etape,$etapes) = $e;
+		list($etape, $etapes) = $e;
 		$etape_demandee = _request('aller_a_etape'); // possibilite de poster en entier dans aller_a_etape
 
 		// lancer les verifs pour chaque etape deja saisie de 1 a $etape
 		$erreurs = array();
 		$derniere_etape_ok = 0;
 		$e = 0;
-		while ($e<$etape AND $e<$etapes){
+		while ($e < $etape AND $e < $etapes) {
 			$e++;
 			$erreurs[$e] = array();
-			if ($verifier = charger_fonction("verifier_$e","formulaires/$form/",true))
+			if ($verifier = charger_fonction("verifier_$e", "formulaires/$form/", true))
 				$erreurs[$e] = call_user_func_array($verifier, $flux['args']['args']);
-			elseif ($verifier = charger_fonction("verifier_etape","formulaires/$form/",true)){
+			elseif ($verifier = charger_fonction("verifier_etape", "formulaires/$form/", true)) {
 				$args = $flux['args']['args'];
 				array_unshift($args, $e);
 				$erreurs[$e] = call_user_func_array($verifier, $args);
 			}
-			if ($derniere_etape_ok==$e-1 AND !count($erreurs[$e]))
+			if ($derniere_etape_ok == $e - 1 AND !count($erreurs[$e]))
 				$derniere_etape_ok = $e;
 			// possibilite de poster dans _retour_etape_x
 			if (!is_null(_request("_retour_etape_$e")))
@@ -204,19 +204,19 @@ 
 
 		// si la derniere etape OK etait la derniere
 		// on renvoie le flux inchange et ca declenche traiter
-		if ($derniere_etape_ok==$etapes AND !$etape_demandee){
+		if ($derniere_etape_ok == $etapes AND !$etape_demandee) {
 			return $flux;
 		}
 		else {
-			$etape = $derniere_etape_ok+1;
-			if ($etape_demandee>0 AND $etape_demandee<$etape)
+			$etape = $derniere_etape_ok + 1;
+			if ($etape_demandee > 0 AND $etape_demandee < $etape)
 				$etape = $etape_demandee;
-			$etape = min($etape,$etapes);
+			$etape = min($etape, $etapes);
 			#var_dump("prochaine etape $etape");
 			// retourner les erreurs de l'etape ciblee
-			$flux['data'] = isset($erreurs[$etape]) ? $erreurs[$etape] : array() ;
+			$flux['data'] = isset($erreurs[$etape]) ? $erreurs[$etape] : array();
 			$flux['data']['_etapes'] = "etape suivante $etape";
-			set_request('_etape',$etape);
+			set_request('_etape', $etape);
 		}
 	}
 	return $flux;
@@ -230,13 +230,13 @@ 
  * @param array $flux
  * @return array
  */
-function cvtmulti_styliser($flux){
-	if (strncmp($flux['args']['fond'],'formulaires/',12)==0
+function cvtmulti_styliser($flux) {
+	if (strncmp($flux['args']['fond'], 'formulaires/', 12) == 0
 	  AND isset($flux['args']['contexte']['_etapes'])
 	  AND isset($flux['args']['contexte']['_etape'])
-	  AND ($e=$flux['args']['contexte']['_etape'])>1
+	  AND ($e = $flux['args']['contexte']['_etape']) > 1
 		AND $ext = $flux['args']['ext']
-		AND $f=$flux['data']
+		AND $f = $flux['data']
 		AND file_exists($f."_$e.$ext"))
 		$flux['data'] = $f."_$e";
 	return $flux;

Braces +36 added lines, -26 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;
+}
 
 /*
  * CVT Multi etapes
@@ -80,15 +82,17 @@ 
 		
 		# reinjecter dans la bonne variable pour permettre de retrouver
 		# toutes les saisies dans un seul tableau
-		if ($_SERVER['REQUEST_METHOD']=='POST')
-			$store = &$_POST;
-		else
-			$store = &$_GET;
+		if ($_SERVER['REQUEST_METHOD']=='POST') {
+					$store = &$_POST;
+		} else {
+					$store = &$_GET;
+		}
 
-		foreach($c as $k=>$v)
-			// on ecrase pas si saisi a nouveau !
+		foreach($c as $k=>$v) {
+					// on ecrase pas si saisi a nouveau !
 			if (!isset($store[$k]))
 				$_REQUEST[$k] = $store[$k] = $v;
+		}
 			// mais si tableau des deux cotes, on merge avec priorite a la derniere saisie
 			elseif(is_array($store[$k])
 			  AND is_array($v)
@@ -96,8 +100,9 @@ 
 				AND !is_numeric(reset($z))
 				AND $z = array_keys($store[$k])
 				AND !is_numeric(reset($z))
-				)
-				$_REQUEST[$k] = $store[$k] = array_merge($v,$store[$k]);
+				) {
+							$_REQUEST[$k] = $store[$k] = array_merge($v,$store[$k]);
+			}
 
 		// vider pour eviter un second appel a verifier_n
 		// en cas de double implementation (unipotence)
@@ -122,15 +127,17 @@ 
 		foreach(array_keys($valeurs) as $champ){
 			if (substr($champ,0,1)!=='_'){
 				if ($je_suis_poste || (isset($valeurs['_forcer_request']) && $valeurs['_forcer_request'])) {
-					if (($v = _request($champ))!==NULL)
-						$post[$champ] = $v;
+					if (($v = _request($champ))!==NULL) {
+											$post[$champ] = $v;
+					}
 				}
 			}
 		}
 		include_spip('inc/filtres');
 		$c = encoder_contexte_ajax($post,$form);
-		if (!isset($valeurs['_hidden']))
-			$valeurs['_hidden'] = '';
+		if (!isset($valeurs['_hidden'])) {
+					$valeurs['_hidden'] = '';
+		}
 		$valeurs['_hidden'] .= "<input type='hidden' name='cvtm_prev_post' value='$c' />";
 		// marquer comme fait, pour eviter double encodage (unipotence)
 		$valeurs['_cvtm_prev_post'] = true;
@@ -188,29 +195,31 @@ 
 		while ($e<$etape AND $e<$etapes){
 			$e++;
 			$erreurs[$e] = array();
-			if ($verifier = charger_fonction("verifier_$e","formulaires/$form/",true))
-				$erreurs[$e] = call_user_func_array($verifier, $flux['args']['args']);
-			elseif ($verifier = charger_fonction("verifier_etape","formulaires/$form/",true)){
+			if ($verifier = charger_fonction("verifier_$e","formulaires/$form/",true)) {
+							$erreurs[$e] = call_user_func_array($verifier, $flux['args']['args']);
+			} elseif ($verifier = charger_fonction("verifier_etape","formulaires/$form/",true)){
 				$args = $flux['args']['args'];
 				array_unshift($args, $e);
 				$erreurs[$e] = call_user_func_array($verifier, $args);
 			}
-			if ($derniere_etape_ok==$e-1 AND !count($erreurs[$e]))
-				$derniere_etape_ok = $e;
+			if ($derniere_etape_ok==$e-1 AND !count($erreurs[$e])) {
+							$derniere_etape_ok = $e;
+			}
 			// possibilite de poster dans _retour_etape_x
-			if (!is_null(_request("_retour_etape_$e")))
-				$etape_demandee = $e;
+			if (!is_null(_request("_retour_etape_$e"))) {
+							$etape_demandee = $e;
+			}
 		}
 
 		// si la derniere etape OK etait la derniere
 		// on renvoie le flux inchange et ca declenche traiter
 		if ($derniere_etape_ok==$etapes AND !$etape_demandee){
 			return $flux;
-		}
-		else {
+		} else {
 			$etape = $derniere_etape_ok+1;
-			if ($etape_demandee>0 AND $etape_demandee<$etape)
-				$etape = $etape_demandee;
+			if ($etape_demandee>0 AND $etape_demandee<$etape) {
+							$etape = $etape_demandee;
+			}
 			$etape = min($etape,$etapes);
 			#var_dump("prochaine etape $etape");
 			// retourner les erreurs de l'etape ciblee
@@ -237,8 +246,9 @@ 
 	  AND ($e=$flux['args']['contexte']['_etape'])>1
 		AND $ext = $flux['args']['ext']
 		AND $f=$flux['data']
-		AND file_exists($f."_$e.$ext"))
-		$flux['data'] = $f."_$e";
+		AND file_exists($f."_$e.$ext")) {
+			$flux['data'] = $f."_$e";
+	}
 	return $flux;
 }
 

ecrire/inc/genie.php

Indentation +66 added lines, -66 removed lines

@@ -52,21 +52,21 @@ 
 
 // http://doc.spip.org/@inc_genie_dist
 function inc_genie_dist($taches = array()) {
-	include_spip('inc/queue');
+    include_spip('inc/queue');
 
-	if (_request('exec')=='job_queue')
-		return false;
+    if (_request('exec')=='job_queue')
+        return false;
 
-	$force_jobs = array();
-	// l'ancienne facon de lancer une tache cron immediatement
-	// etait de la passer en parametre a ing_genie_dist
-	// on reroute en ajoutant simplement le job a la queue, ASAP
-	foreach($taches as $function=>$period)
-		$force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time()-abs($period)), "genie/");
+    $force_jobs = array();
+    // l'ancienne facon de lancer une tache cron immediatement
+    // etait de la passer en parametre a ing_genie_dist
+    // on reroute en ajoutant simplement le job a la queue, ASAP
+    foreach($taches as $function=>$period)
+        $force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time()-abs($period)), "genie/");
 	
-	// et on passe la main a la gestion de la queue !
-	// en forcant eventuellement les jobs ajoute a l'instant
-	return queue_schedule(count($force_jobs)?$force_jobs:null);
+    // et on passe la main a la gestion de la queue !
+    // en forcant eventuellement les jobs ajoute a l'instant
+    return queue_schedule(count($force_jobs)?$force_jobs:null);
 }
 
 //
@@ -80,33 +80,33 @@ 
 // http://doc.spip.org/@taches_generales
 function taches_generales($taches_generales = array()) {
 
-	// verifier que toutes les taches cron sont planifiees
-	// c'est une tache cron !
-	$taches_generales['queue_watch'] = 3600*24;
+    // verifier que toutes les taches cron sont planifiees
+    // c'est une tache cron !
+    $taches_generales['queue_watch'] = 3600*24;
 
-	// MAJ des rubriques publiques (cas de la publication post-datee)
-	// est fait au coup par coup a present
-	//	$taches_generales['rubriques'] = 3600;
+    // MAJ des rubriques publiques (cas de la publication post-datee)
+    // est fait au coup par coup a present
+    //	$taches_generales['rubriques'] = 3600;
 
-	// Optimisation de la base
-	$taches_generales['optimiser'] = 3600*48;
+    // Optimisation de la base
+    $taches_generales['optimiser'] = 3600*48;
 
-	// cache (chaque 10 minutes => 1/16eme du repertoire cache,
-	// soit toutes les 2h40 sur le meme rep)
-	$taches_generales['invalideur'] = 600;
+    // cache (chaque 10 minutes => 1/16eme du repertoire cache,
+    // soit toutes les 2h40 sur le meme rep)
+    $taches_generales['invalideur'] = 600;
 
-	// nouveautes
-	if ($GLOBALS['meta']['adresse_neuf'] AND $GLOBALS['meta']['jours_neuf']
-	AND ($GLOBALS['meta']['quoi_de_neuf'] == 'oui'))
-		$taches_generales['mail']= 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
+    // nouveautes
+    if ($GLOBALS['meta']['adresse_neuf'] AND $GLOBALS['meta']['jours_neuf']
+    AND ($GLOBALS['meta']['quoi_de_neuf'] == 'oui'))
+        $taches_generales['mail']= 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
 
-	// maintenance (ajax, verifications diverses)
-	$taches_generales['maintenance'] = 3600 * 2;
+    // maintenance (ajax, verifications diverses)
+    $taches_generales['maintenance'] = 3600 * 2;
 
-	// verifier si une mise a jour de spip est disponible (2 fois par semaine suffit largement)
-	$taches_generales['mise_a_jour'] = 3*24*3600;
+    // verifier si une mise a jour de spip est disponible (2 fois par semaine suffit largement)
+    $taches_generales['mise_a_jour'] = 3*24*3600;
 
-	return pipeline('taches_generales_cron',$taches_generales);
+    return pipeline('taches_generales_cron',$taches_generales);
 }
 
 // Pas de fichier a part pour une fonction aussi petite:
@@ -116,13 +116,13 @@ 
 // http://doc.spip.org/@genie_invalideur_dist
 function genie_invalideur_dist($t) {
 
-	include_spip('inc/invalideur');
-	$encore = appliquer_quota_cache();
+    include_spip('inc/invalideur');
+    $encore = appliquer_quota_cache();
 
-	// si le cache est trop gonfle, redemander la main pour poursuivre
-	if ($encore)
-		return (0 - $t);
-	return 1;
+    // si le cache est trop gonfle, redemander la main pour poursuivre
+    if ($encore)
+        return (0 - $t);
+    return 1;
 }
 
 /**
@@ -134,18 +134,18 @@ 
  * @return int
  */
 function genie_queue_watch_dist(){
-	static $deja_la = false;
-	if ($deja_la) return; // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
-	$deja_la = true;
-	$taches = taches_generales();
-	$programmees = sql_allfetsel('fonction','spip_jobs',sql_in('fonction',array_keys($taches)));
-	$programmees = array_map('reset',$programmees);
-	foreach($taches as $tache=>$periode){
-		if (!in_array($tache,$programmees))
-			queue_genie_replan_job($tache,$periode,time()-round(rand(1,$periode)),0);
-	}
-	$deja_la = false;
-	return 1;
+    static $deja_la = false;
+    if ($deja_la) return; // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
+    $deja_la = true;
+    $taches = taches_generales();
+    $programmees = sql_allfetsel('fonction','spip_jobs',sql_in('fonction',array_keys($taches)));
+    $programmees = array_map('reset',$programmees);
+    foreach($taches as $tache=>$periode){
+        if (!in_array($tache,$programmees))
+            queue_genie_replan_job($tache,$periode,time()-round(rand(1,$periode)),0);
+    }
+    $deja_la = false;
+    return 1;
 }
 
 /**
@@ -166,22 +166,22 @@ 
  * @return void
  */
 function queue_genie_replan_job($function,$period,$last=0,$time=null, $priority=0){
-	static $done = array();
-	if (isset($done[$function])) return;
-	$done[$function] = true;
-	if (is_null($time)){
-		$time=time();
-		if ($last)
-			$time = max($last+$period,$time);
-	}
-	if (!$last)
-		$last = $time-$period;
-	spip_log("replan_job $function $period $last $time $priority",'queue');
-	include_spip('inc/queue');
-	// on replanifie un job cron
-	// uniquement si il n'y en a pas deja un avec le meme nom
-	// independament de l'argument
-	queue_add_job($function, _T('tache_cron_secondes', array('function'=>$function, 'nb'=>$period)), array($last), "genie/", 'function_only', $time, $priority);
+    static $done = array();
+    if (isset($done[$function])) return;
+    $done[$function] = true;
+    if (is_null($time)){
+        $time=time();
+        if ($last)
+            $time = max($last+$period,$time);
+    }
+    if (!$last)
+        $last = $time-$period;
+    spip_log("replan_job $function $period $last $time $priority",'queue');
+    include_spip('inc/queue');
+    // on replanifie un job cron
+    // uniquement si il n'y en a pas deja un avec le meme nom
+    // independament de l'argument
+    queue_add_job($function, _T('tache_cron_secondes', array('function'=>$function, 'nb'=>$period)), array($last), "genie/", 'function_only', $time, $priority);
 }
 
 

Spacing +21 added lines, -21 removed lines

@@ -54,19 +54,19 @@ 
 function inc_genie_dist($taches = array()) {
 	include_spip('inc/queue');
 
-	if (_request('exec')=='job_queue')
+	if (_request('exec') == 'job_queue')
 		return false;
 
 	$force_jobs = array();
 	// l'ancienne facon de lancer une tache cron immediatement
 	// etait de la passer en parametre a ing_genie_dist
 	// on reroute en ajoutant simplement le job a la queue, ASAP
-	foreach($taches as $function=>$period)
-		$force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time()-abs($period)), "genie/");
+	foreach ($taches as $function=>$period)
+		$force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time() - abs($period)), "genie/");
 	
 	// et on passe la main a la gestion de la queue !
 	// en forcant eventuellement les jobs ajoute a l'instant
-	return queue_schedule(count($force_jobs)?$force_jobs:null);
+	return queue_schedule(count($force_jobs) ? $force_jobs : null);
 }
 
 //
@@ -82,14 +82,14 @@ 
 
 	// verifier que toutes les taches cron sont planifiees
 	// c'est une tache cron !
-	$taches_generales['queue_watch'] = 3600*24;
+	$taches_generales['queue_watch'] = 3600 * 24;
 
 	// MAJ des rubriques publiques (cas de la publication post-datee)
 	// est fait au coup par coup a present
 	//	$taches_generales['rubriques'] = 3600;
 
 	// Optimisation de la base
-	$taches_generales['optimiser'] = 3600*48;
+	$taches_generales['optimiser'] = 3600 * 48;
 
 	// cache (chaque 10 minutes => 1/16eme du repertoire cache,
 	// soit toutes les 2h40 sur le meme rep)
@@ -98,15 +98,15 @@ 
 	// nouveautes
 	if ($GLOBALS['meta']['adresse_neuf'] AND $GLOBALS['meta']['jours_neuf']
 	AND ($GLOBALS['meta']['quoi_de_neuf'] == 'oui'))
-		$taches_generales['mail']= 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
+		$taches_generales['mail'] = 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
 
 	// maintenance (ajax, verifications diverses)
 	$taches_generales['maintenance'] = 3600 * 2;
 
 	// verifier si une mise a jour de spip est disponible (2 fois par semaine suffit largement)
-	$taches_generales['mise_a_jour'] = 3*24*3600;
+	$taches_generales['mise_a_jour'] = 3 * 24 * 3600;
 
-	return pipeline('taches_generales_cron',$taches_generales);
+	return pipeline('taches_generales_cron', $taches_generales);
 }
 
 // Pas de fichier a part pour une fonction aussi petite:
@@ -133,16 +133,16 @@ 
  *
  * @return int
  */
-function genie_queue_watch_dist(){
+function genie_queue_watch_dist() {
 	static $deja_la = false;
 	if ($deja_la) return; // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
 	$deja_la = true;
 	$taches = taches_generales();
-	$programmees = sql_allfetsel('fonction','spip_jobs',sql_in('fonction',array_keys($taches)));
-	$programmees = array_map('reset',$programmees);
-	foreach($taches as $tache=>$periode){
-		if (!in_array($tache,$programmees))
-			queue_genie_replan_job($tache,$periode,time()-round(rand(1,$periode)),0);
+	$programmees = sql_allfetsel('fonction', 'spip_jobs', sql_in('fonction', array_keys($taches)));
+	$programmees = array_map('reset', $programmees);
+	foreach ($taches as $tache=>$periode) {
+		if (!in_array($tache, $programmees))
+			queue_genie_replan_job($tache, $periode, time() - round(rand(1, $periode)), 0);
 	}
 	$deja_la = false;
 	return 1;
@@ -165,18 +165,18 @@ 
  *   priorite
  * @return void
  */
-function queue_genie_replan_job($function,$period,$last=0,$time=null, $priority=0){
+function queue_genie_replan_job($function, $period, $last = 0, $time = null, $priority = 0) {
 	static $done = array();
 	if (isset($done[$function])) return;
 	$done[$function] = true;
-	if (is_null($time)){
-		$time=time();
+	if (is_null($time)) {
+		$time = time();
 		if ($last)
-			$time = max($last+$period,$time);
+			$time = max($last + $period, $time);
 	}
 	if (!$last)
-		$last = $time-$period;
-	spip_log("replan_job $function $period $last $time $priority",'queue');
+		$last = $time - $period;
+	spip_log("replan_job $function $period $last $time $priority", 'queue');
 	include_spip('inc/queue');
 	// on replanifie un job cron
 	// uniquement si il n'y en a pas deja un avec le meme nom

Braces +31 added lines, -17 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;
+}
 
 // --------------------------
 // Gestion des taches de fond
@@ -54,15 +56,17 @@ 
 function inc_genie_dist($taches = array()) {
 	include_spip('inc/queue');
 
-	if (_request('exec')=='job_queue')
-		return false;
+	if (_request('exec')=='job_queue') {
+			return false;
+	}
 
 	$force_jobs = array();
 	// l'ancienne facon de lancer une tache cron immediatement
 	// etait de la passer en parametre a ing_genie_dist
 	// on reroute en ajoutant simplement le job a la queue, ASAP
-	foreach($taches as $function=>$period)
-		$force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time()-abs($period)), "genie/");
+	foreach($taches as $function=>$period) {
+			$force_jobs[] = queue_add_job($function, _T('tache_cron_asap', array('function'=>$function)), array(time()-abs($period)), "genie/");
+	}
 	
 	// et on passe la main a la gestion de la queue !
 	// en forcant eventuellement les jobs ajoute a l'instant
@@ -97,8 +101,9 @@ 
 
 	// nouveautes
 	if ($GLOBALS['meta']['adresse_neuf'] AND $GLOBALS['meta']['jours_neuf']
-	AND ($GLOBALS['meta']['quoi_de_neuf'] == 'oui'))
-		$taches_generales['mail']= 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
+	AND ($GLOBALS['meta']['quoi_de_neuf'] == 'oui')) {
+			$taches_generales['mail']= 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
+	}
 
 	// maintenance (ajax, verifications diverses)
 	$taches_generales['maintenance'] = 3600 * 2;
@@ -120,8 +125,9 @@ 
 	$encore = appliquer_quota_cache();
 
 	// si le cache est trop gonfle, redemander la main pour poursuivre
-	if ($encore)
-		return (0 - $t);
+	if ($encore) {
+			return (0 - $t);
+	}
 	return 1;
 }
 
@@ -135,14 +141,18 @@ 
  */
 function genie_queue_watch_dist(){
 	static $deja_la = false;
-	if ($deja_la) return; // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
+	if ($deja_la) {
+	    return;
+	}
+	// re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
 	$deja_la = true;
 	$taches = taches_generales();
 	$programmees = sql_allfetsel('fonction','spip_jobs',sql_in('fonction',array_keys($taches)));
 	$programmees = array_map('reset',$programmees);
 	foreach($taches as $tache=>$periode){
-		if (!in_array($tache,$programmees))
-			queue_genie_replan_job($tache,$periode,time()-round(rand(1,$periode)),0);
+		if (!in_array($tache,$programmees)) {
+					queue_genie_replan_job($tache,$periode,time()-round(rand(1,$periode)),0);
+		}
 	}
 	$deja_la = false;
 	return 1;
@@ -167,15 +177,19 @@ 
  */
 function queue_genie_replan_job($function,$period,$last=0,$time=null, $priority=0){
 	static $done = array();
-	if (isset($done[$function])) return;
+	if (isset($done[$function])) {
+	    return;
+	}
 	$done[$function] = true;
 	if (is_null($time)){
 		$time=time();
-		if ($last)
-			$time = max($last+$period,$time);
+		if ($last) {
+					$time = max($last+$period,$time);
+		}
+	}
+	if (!$last) {
+			$last = $time-$period;
 	}
-	if (!$last)
-		$last = $time-$period;
 	spip_log("replan_job $function $period $last $time $priority",'queue');
 	include_spip('inc/queue');
 	// on replanifie un job cron

ecrire/inc/icone_renommer.php

Indentation +58 added lines, -58 removed lines

@@ -16,69 +16,69 @@
 include_spip('base/objets');
 
 function inc_icone_renommer_dist($fond,$fonction){
-	$size = 24;
-	if (preg_match("/(?:-([0-9]{1,3}))?([.](gif|png))?$/i",$fond,$match)
-		AND ((isset($match[0]) AND $match[0]) OR (isset($match[1]) AND $match[1]))) {
-		if (isset($match[1]) AND $match[1]) {
-			$size = $match[1];
-		}
-		$type = substr($fond,0,-strlen($match[0]));
-		if (!isset($match[2]) OR !$match[2]) {
-			$fond .= ".png";
-		}
-	}
-	else {
-		$type = $fond;
-		$fond .= ".png";
-	}
+    $size = 24;
+    if (preg_match("/(?:-([0-9]{1,3}))?([.](gif|png))?$/i",$fond,$match)
+        AND ((isset($match[0]) AND $match[0]) OR (isset($match[1]) AND $match[1]))) {
+        if (isset($match[1]) AND $match[1]) {
+            $size = $match[1];
+        }
+        $type = substr($fond,0,-strlen($match[0]));
+        if (!isset($match[2]) OR !$match[2]) {
+            $fond .= ".png";
+        }
+    }
+    else {
+        $type = $fond;
+        $fond .= ".png";
+    }
 
-	$rtl = false;
-	if (preg_match(',[-_]rtl$,i',$type,$match)){
-		$rtl = true;
-		$type = substr($type,0,-strlen($match[0]));
-	}
+    $rtl = false;
+    if (preg_match(',[-_]rtl$,i',$type,$match)){
+        $rtl = true;
+        $type = substr($type,0,-strlen($match[0]));
+    }
 
-	// objet_type garde invariant tout ce qui ne commence par par id_, spip_
-	// et ne finit pas par un s, sauf si c'est une exception declaree
-	$type = objet_type($type, false);
+    // objet_type garde invariant tout ce qui ne commence par par id_, spip_
+    // et ne finit pas par un s, sauf si c'est une exception declaree
+    $type = objet_type($type, false);
 
-	$dir = "images/";
-	$f = "$type-$size.png";
-	if ($icone = find_in_theme($dir.$f)){
-		$dir = dirname($icone);
-		$fond = $icone;
+    $dir = "images/";
+    $f = "$type-$size.png";
+    if ($icone = find_in_theme($dir.$f)){
+        $dir = dirname($icone);
+        $fond = $icone;
 
-		if ($rtl
-			AND $fr = "$type-rtl-$size.png"
-			AND file_exists($dir.'/'.$fr))
-			$type = "$type-rtl";
+        if ($rtl
+            AND $fr = "$type-rtl-$size.png"
+            AND file_exists($dir.'/'.$fr))
+            $type = "$type-rtl";
 
-		$action = $fonction;
-		if ($action=="supprimer.gif"){
-			$action = "del";
-		}
-		elseif ($action=="creer.gif"){
-			$action = "new";
-		}
-		elseif ($action=="edit.gif"){
-			$action = "edit";
-		}
-		if (!in_array($action,array('del','new','edit')))
-			$action = "";
-		if ($action){
-			if ($fa = "$type-$action-$size.png"
-			AND file_exists($dir.'/'.$fa)){
-				$fond = $dir .'/'. $fa;
-				$fonction = "";
-			}
-			else {
-				$fonction = "$action-$size.png";
-			}
-		}
-		// c'est bon !
-		return array($fond,$fonction);
-	}
+        $action = $fonction;
+        if ($action=="supprimer.gif"){
+            $action = "del";
+        }
+        elseif ($action=="creer.gif"){
+            $action = "new";
+        }
+        elseif ($action=="edit.gif"){
+            $action = "edit";
+        }
+        if (!in_array($action,array('del','new','edit')))
+            $action = "";
+        if ($action){
+            if ($fa = "$type-$action-$size.png"
+            AND file_exists($dir.'/'.$fa)){
+                $fond = $dir .'/'. $fa;
+                $fonction = "";
+            }
+            else {
+                $fonction = "$action-$size.png";
+            }
+        }
+        // c'est bon !
+        return array($fond,$fonction);
+    }
 
-	return array($fond,$fonction);
+    return array($fond,$fonction);
 }
 ?>

Spacing +15 added lines, -15 removed lines

@@ -15,14 +15,14 @@ 
 include_spip('inc/boutons');
 include_spip('base/objets');
 
-function inc_icone_renommer_dist($fond,$fonction){
+function inc_icone_renommer_dist($fond, $fonction) {
 	$size = 24;
-	if (preg_match("/(?:-([0-9]{1,3}))?([.](gif|png))?$/i",$fond,$match)
+	if (preg_match("/(?:-([0-9]{1,3}))?([.](gif|png))?$/i", $fond, $match)
 		AND ((isset($match[0]) AND $match[0]) OR (isset($match[1]) AND $match[1]))) {
 		if (isset($match[1]) AND $match[1]) {
 			$size = $match[1];
 		}
-		$type = substr($fond,0,-strlen($match[0]));
+		$type = substr($fond, 0, -strlen($match[0]));
 		if (!isset($match[2]) OR !$match[2]) {
 			$fond .= ".png";
 		}
@@ -33,9 +33,9 @@ 
 	}
 
 	$rtl = false;
-	if (preg_match(',[-_]rtl$,i',$type,$match)){
+	if (preg_match(',[-_]rtl$,i', $type, $match)) {
 		$rtl = true;
-		$type = substr($type,0,-strlen($match[0]));
+		$type = substr($type, 0, -strlen($match[0]));
 	}
 
 	// objet_type garde invariant tout ce qui ne commence par par id_, spip_
@@ -44,7 +44,7 @@ 
 
 	$dir = "images/";
 	$f = "$type-$size.png";
-	if ($icone = find_in_theme($dir.$f)){
+	if ($icone = find_in_theme($dir.$f)) {
 		$dir = dirname($icone);
 		$fond = $icone;
 
@@ -54,21 +54,21 @@ 
 			$type = "$type-rtl";
 
 		$action = $fonction;
-		if ($action=="supprimer.gif"){
+		if ($action == "supprimer.gif") {
 			$action = "del";
 		}
-		elseif ($action=="creer.gif"){
+		elseif ($action == "creer.gif") {
 			$action = "new";
 		}
-		elseif ($action=="edit.gif"){
+		elseif ($action == "edit.gif") {
 			$action = "edit";
 		}
-		if (!in_array($action,array('del','new','edit')))
+		if (!in_array($action, array('del', 'new', 'edit')))
 			$action = "";
-		if ($action){
+		if ($action) {
 			if ($fa = "$type-$action-$size.png"
-			AND file_exists($dir.'/'.$fa)){
-				$fond = $dir .'/'. $fa;
+			AND file_exists($dir.'/'.$fa)) {
+				$fond = $dir.'/'.$fa;
 				$fonction = "";
 			}
 			else {
@@ -76,9 +76,9 @@ 
 			}
 		}
 		// c'est bon !
-		return array($fond,$fonction);
+		return array($fond, $fonction);
 	}
 
-	return array($fond,$fonction);
+	return array($fond, $fonction);
 }
 ?>

Braces +13 added lines, -13 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/boutons');
 include_spip('base/objets');
@@ -26,8 +28,7 @@ 
 		if (!isset($match[2]) OR !$match[2]) {
 			$fond .= ".png";
 		}
-	}
-	else {
+	} else {
 		$type = $fond;
 		$fond .= ".png";
 	}
@@ -50,28 +51,27 @@ 
 
 		if ($rtl
 			AND $fr = "$type-rtl-$size.png"
-			AND file_exists($dir.'/'.$fr))
-			$type = "$type-rtl";
+			AND file_exists($dir.'/'.$fr)) {
+					$type = "$type-rtl";
+		}
 
 		$action = $fonction;
 		if ($action=="supprimer.gif"){
 			$action = "del";
-		}
-		elseif ($action=="creer.gif"){
+		} elseif ($action=="creer.gif"){
 			$action = "new";
-		}
-		elseif ($action=="edit.gif"){
+		} elseif ($action=="edit.gif"){
 			$action = "edit";
 		}
-		if (!in_array($action,array('del','new','edit')))
-			$action = "";
+		if (!in_array($action,array('del','new','edit'))) {
+					$action = "";
+		}
 		if ($action){
 			if ($fa = "$type-$action-$size.png"
 			AND file_exists($dir.'/'.$fa)){
 				$fond = $dir .'/'. $fa;
 				$fonction = "";
-			}
-			else {
+			} else {
 				$fonction = "$action-$size.png";
 			}
 		}

ecrire/inc/log.php

Indentation +78 added lines, -78 removed lines

@@ -13,84 +13,84 @@
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 function inc_log_dist($message, $logname=NULL, $logdir=NULL, $logsuf=NULL) {
-	static $test_repertoire = array();
-	static $compteur = array();
-	static $debugverb = ""; // pour ne pas le recalculer au reappel
-	global $nombre_de_logs, $taille_des_logs;
-
-	if (is_null($logname) OR !is_string($logname))
-		$logname = defined('_FILE_LOG') ? _FILE_LOG : 'spip';
-	if (!isset($compteur[$logname])) $compteur[$logname] = 0;
-	if ($logname != 'maj'
-	AND defined('_MAX_LOG')
-	AND (
-		$compteur[$logname]++ > _MAX_LOG
-		OR !$nombre_de_logs
-		OR !$taille_des_logs
-	))
-		return;
-
-	$logfile = ($logdir===NULL ? _DIR_LOG : $logdir)
-	  . ($logname)
-	  . ($logsuf===NULL ? _FILE_LOG_SUFFIX : $logsuf);
-
-	if (!isset($test_repertoire[$d = dirname($logfile)])) {
-		$test_repertoire[$d] = false; // eviter une recursivite en cas d'erreur de sous_repertoire
-		$test_repertoire[$d] = (@is_dir($d)?true:(function_exists('sous_repertoire')?sous_repertoire($d, '', false, true):false));
-	}
-
-	// si spip_log() dans mes_options, ou repertoire log/ non present, poser dans tmp/
-	if (!defined('_DIR_LOG') OR !$test_repertoire[$d])
-		$logfile = _DIR_RACINE._NOM_TEMPORAIRES_INACCESSIBLES.$logname.'.log';
-
-	$rotate = 0;
-	$pid = '(pid '.@getmypid().')';
-
-	// accepter spip_log( Array )
-	if (!is_string($message)) $message = var_export($message, true);
-
-	if (!$debugverb AND defined('_LOG_FILELINE') AND _LOG_FILELINE){
-		$debug = debug_backtrace();
-		$l = $debug[1]['line'];
-		$fi = $debug[1]['file'];
-		if (strncmp($fi,_ROOT_RACINE,strlen(_ROOT_RACINE))==0)
-			$fi = substr($fi,strlen(_ROOT_RACINE));
-		$fu = isset($debug[2]['function']) ? $debug[2]['function'] : '';
-		$debugverb = "$fi:L$l:$fu"."():";
-	}
-
-	$m = date("Y-m-d H:i:s").' '.$GLOBALS['ip'].' '.$pid.' '
-	  //distinguer les logs prives et publics dans les grep
-		. $debugverb
-		. (test_espace_prive()?':Pri:':':Pub:')
-		.preg_replace("/\n*$/", "\n", $message);
-
-
-	if (@is_readable($logfile)
-	AND (!$s = @filesize($logfile) OR $s > $taille_des_logs * 1024)) {
-		$rotate = $nombre_de_logs;
-		$m .= "[-- rotate --]\n";
-	}
-
-	$f = @fopen($logfile, "ab");
-	if ($f) {
-		fputs($f, (defined('_LOG_BRUT') AND _LOG_BRUT) ? $m : str_replace('<','&lt;',$m));
-		fclose($f);
-	}
-
-	if ($rotate-- > 0
-	AND function_exists('spip_unlink')) {
-		spip_unlink($logfile . '.' . $rotate);
-		while ($rotate--) {
-			@rename($logfile . ($rotate ? '.' . $rotate : ''), $logfile . '.' . ($rotate + 1));
-		}
-	}
-
-	// Dupliquer les erreurs specifiques dans le log general
-	if ($logname !== _FILE_LOG
-	  AND defined('_FILE_LOG'))
-		inc_log_dist($logname=='maj' ? 'cf maj.log' : $message);
-	$debugverb = "";
+    static $test_repertoire = array();
+    static $compteur = array();
+    static $debugverb = ""; // pour ne pas le recalculer au reappel
+    global $nombre_de_logs, $taille_des_logs;
+
+    if (is_null($logname) OR !is_string($logname))
+        $logname = defined('_FILE_LOG') ? _FILE_LOG : 'spip';
+    if (!isset($compteur[$logname])) $compteur[$logname] = 0;
+    if ($logname != 'maj'
+    AND defined('_MAX_LOG')
+    AND (
+        $compteur[$logname]++ > _MAX_LOG
+        OR !$nombre_de_logs
+        OR !$taille_des_logs
+    ))
+        return;
+
+    $logfile = ($logdir===NULL ? _DIR_LOG : $logdir)
+        . ($logname)
+        . ($logsuf===NULL ? _FILE_LOG_SUFFIX : $logsuf);
+
+    if (!isset($test_repertoire[$d = dirname($logfile)])) {
+        $test_repertoire[$d] = false; // eviter une recursivite en cas d'erreur de sous_repertoire
+        $test_repertoire[$d] = (@is_dir($d)?true:(function_exists('sous_repertoire')?sous_repertoire($d, '', false, true):false));
+    }
+
+    // si spip_log() dans mes_options, ou repertoire log/ non present, poser dans tmp/
+    if (!defined('_DIR_LOG') OR !$test_repertoire[$d])
+        $logfile = _DIR_RACINE._NOM_TEMPORAIRES_INACCESSIBLES.$logname.'.log';
+
+    $rotate = 0;
+    $pid = '(pid '.@getmypid().')';
+
+    // accepter spip_log( Array )
+    if (!is_string($message)) $message = var_export($message, true);
+
+    if (!$debugverb AND defined('_LOG_FILELINE') AND _LOG_FILELINE){
+        $debug = debug_backtrace();
+        $l = $debug[1]['line'];
+        $fi = $debug[1]['file'];
+        if (strncmp($fi,_ROOT_RACINE,strlen(_ROOT_RACINE))==0)
+            $fi = substr($fi,strlen(_ROOT_RACINE));
+        $fu = isset($debug[2]['function']) ? $debug[2]['function'] : '';
+        $debugverb = "$fi:L$l:$fu"."():";
+    }
+
+    $m = date("Y-m-d H:i:s").' '.$GLOBALS['ip'].' '.$pid.' '
+        //distinguer les logs prives et publics dans les grep
+        . $debugverb
+        . (test_espace_prive()?':Pri:':':Pub:')
+        .preg_replace("/\n*$/", "\n", $message);
+
+
+    if (@is_readable($logfile)
+    AND (!$s = @filesize($logfile) OR $s > $taille_des_logs * 1024)) {
+        $rotate = $nombre_de_logs;
+        $m .= "[-- rotate --]\n";
+    }
+
+    $f = @fopen($logfile, "ab");
+    if ($f) {
+        fputs($f, (defined('_LOG_BRUT') AND _LOG_BRUT) ? $m : str_replace('<','&lt;',$m));
+        fclose($f);
+    }
+
+    if ($rotate-- > 0
+    AND function_exists('spip_unlink')) {
+        spip_unlink($logfile . '.' . $rotate);
+        while ($rotate--) {
+            @rename($logfile . ($rotate ? '.' . $rotate : ''), $logfile . '.' . ($rotate + 1));
+        }
+    }
+
+    // Dupliquer les erreurs specifiques dans le log general
+    if ($logname !== _FILE_LOG
+      AND defined('_FILE_LOG'))
+        inc_log_dist($logname=='maj' ? 'cf maj.log' : $message);
+    $debugverb = "";
 }
 
 ?>

Braces +24 added lines, -13 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 inc_log_dist($message, $logname=NULL, $logdir=NULL, $logsuf=NULL) {
 	static $test_repertoire = array();
@@ -18,17 +20,21 @@ 
 	static $debugverb = ""; // pour ne pas le recalculer au reappel
 	global $nombre_de_logs, $taille_des_logs;
 
-	if (is_null($logname) OR !is_string($logname))
-		$logname = defined('_FILE_LOG') ? _FILE_LOG : 'spip';
-	if (!isset($compteur[$logname])) $compteur[$logname] = 0;
+	if (is_null($logname) OR !is_string($logname)) {
+			$logname = defined('_FILE_LOG') ? _FILE_LOG : 'spip';
+	}
+	if (!isset($compteur[$logname])) {
+	    $compteur[$logname] = 0;
+	}
 	if ($logname != 'maj'
 	AND defined('_MAX_LOG')
 	AND (
 		$compteur[$logname]++ > _MAX_LOG
 		OR !$nombre_de_logs
 		OR !$taille_des_logs
-	))
-		return;
+	)) {
+			return;
+	}
 
 	$logfile = ($logdir===NULL ? _DIR_LOG : $logdir)
 	  . ($logname)
@@ -40,21 +46,25 @@ 
 	}
 
 	// si spip_log() dans mes_options, ou repertoire log/ non present, poser dans tmp/
-	if (!defined('_DIR_LOG') OR !$test_repertoire[$d])
-		$logfile = _DIR_RACINE._NOM_TEMPORAIRES_INACCESSIBLES.$logname.'.log';
+	if (!defined('_DIR_LOG') OR !$test_repertoire[$d]) {
+			$logfile = _DIR_RACINE._NOM_TEMPORAIRES_INACCESSIBLES.$logname.'.log';
+	}
 
 	$rotate = 0;
 	$pid = '(pid '.@getmypid().')';
 
 	// accepter spip_log( Array )
-	if (!is_string($message)) $message = var_export($message, true);
+	if (!is_string($message)) {
+	    $message = var_export($message, true);
+	}
 
 	if (!$debugverb AND defined('_LOG_FILELINE') AND _LOG_FILELINE){
 		$debug = debug_backtrace();
 		$l = $debug[1]['line'];
 		$fi = $debug[1]['file'];
-		if (strncmp($fi,_ROOT_RACINE,strlen(_ROOT_RACINE))==0)
-			$fi = substr($fi,strlen(_ROOT_RACINE));
+		if (strncmp($fi,_ROOT_RACINE,strlen(_ROOT_RACINE))==0) {
+					$fi = substr($fi,strlen(_ROOT_RACINE));
+		}
 		$fu = isset($debug[2]['function']) ? $debug[2]['function'] : '';
 		$debugverb = "$fi:L$l:$fu"."():";
 	}
@@ -88,8 +98,9 @@ 
 
 	// Dupliquer les erreurs specifiques dans le log general
 	if ($logname !== _FILE_LOG
-	  AND defined('_FILE_LOG'))
-		inc_log_dist($logname=='maj' ? 'cf maj.log' : $message);
+	  AND defined('_FILE_LOG')) {
+			inc_log_dist($logname=='maj' ? 'cf maj.log' : $message);
+	}
 	$debugverb = "";
 }
 

Spacing +12 added lines, -12 removed lines

@@ -12,7 +12,7 @@ 
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
-function inc_log_dist($message, $logname=NULL, $logdir=NULL, $logsuf=NULL) {
+function inc_log_dist($message, $logname = NULL, $logdir = NULL, $logsuf = NULL) {
 	static $test_repertoire = array();
 	static $compteur = array();
 	static $debugverb = ""; // pour ne pas le recalculer au reappel
@@ -30,13 +30,13 @@ 
 	))
 		return;
 
-	$logfile = ($logdir===NULL ? _DIR_LOG : $logdir)
+	$logfile = ($logdir === NULL ? _DIR_LOG : $logdir)
 	  . ($logname)
-	  . ($logsuf===NULL ? _FILE_LOG_SUFFIX : $logsuf);
+	  . ($logsuf === NULL ? _FILE_LOG_SUFFIX : $logsuf);
 
 	if (!isset($test_repertoire[$d = dirname($logfile)])) {
 		$test_repertoire[$d] = false; // eviter une recursivite en cas d'erreur de sous_repertoire
-		$test_repertoire[$d] = (@is_dir($d)?true:(function_exists('sous_repertoire')?sous_repertoire($d, '', false, true):false));
+		$test_repertoire[$d] = (@is_dir($d) ?true:(function_exists('sous_repertoire') ?sous_repertoire($d, '', false, true) : false));
 	}
 
 	// si spip_log() dans mes_options, ou repertoire log/ non present, poser dans tmp/
@@ -49,12 +49,12 @@ 
 	// accepter spip_log( Array )
 	if (!is_string($message)) $message = var_export($message, true);
 
-	if (!$debugverb AND defined('_LOG_FILELINE') AND _LOG_FILELINE){
+	if (!$debugverb AND defined('_LOG_FILELINE') AND _LOG_FILELINE) {
 		$debug = debug_backtrace();
 		$l = $debug[1]['line'];
 		$fi = $debug[1]['file'];
-		if (strncmp($fi,_ROOT_RACINE,strlen(_ROOT_RACINE))==0)
-			$fi = substr($fi,strlen(_ROOT_RACINE));
+		if (strncmp($fi, _ROOT_RACINE, strlen(_ROOT_RACINE)) == 0)
+			$fi = substr($fi, strlen(_ROOT_RACINE));
 		$fu = isset($debug[2]['function']) ? $debug[2]['function'] : '';
 		$debugverb = "$fi:L$l:$fu"."():";
 	}
@@ -62,7 +62,7 @@ 
 	$m = date("Y-m-d H:i:s").' '.$GLOBALS['ip'].' '.$pid.' '
 	  //distinguer les logs prives et publics dans les grep
 		. $debugverb
-		. (test_espace_prive()?':Pri:':':Pub:')
+		. (test_espace_prive() ? ':Pri:' : ':Pub:')
 		.preg_replace("/\n*$/", "\n", $message);
 
 
@@ -74,22 +74,22 @@ 
 
 	$f = @fopen($logfile, "ab");
 	if ($f) {
-		fputs($f, (defined('_LOG_BRUT') AND _LOG_BRUT) ? $m : str_replace('<','&lt;',$m));
+		fputs($f, (defined('_LOG_BRUT') AND _LOG_BRUT) ? $m : str_replace('<', '&lt;', $m));
 		fclose($f);
 	}
 
 	if ($rotate-- > 0
 	AND function_exists('spip_unlink')) {
-		spip_unlink($logfile . '.' . $rotate);
+		spip_unlink($logfile.'.'.$rotate);
 		while ($rotate--) {
-			@rename($logfile . ($rotate ? '.' . $rotate : ''), $logfile . '.' . ($rotate + 1));
+			@rename($logfile.($rotate ? '.'.$rotate : ''), $logfile.'.'.($rotate + 1));
 		}
 	}
 
 	// Dupliquer les erreurs specifiques dans le log general
 	if ($logname !== _FILE_LOG
 	  AND defined('_FILE_LOG'))
-		inc_log_dist($logname=='maj' ? 'cf maj.log' : $message);
+		inc_log_dist($logname == 'maj' ? 'cf maj.log' : $message);
 	$debugverb = "";
 }
 

ecrire/inc/session.php

Indentation +261 added lines, -261 removed lines

@@ -34,12 +34,12 @@ 
  */
 function inc_session_dist($auteur=false)
 {
-	if (is_numeric($auteur))
-		return supprimer_sessions($auteur, $auteur > 0);
-	else if (is_array($auteur))
-		return ajouter_session($auteur);
-	else
-		return verifier_session($auteur);
+    if (is_numeric($auteur))
+        return supprimer_sessions($auteur, $auteur > 0);
+    else if (is_array($auteur))
+        return ajouter_session($auteur);
+    else
+        return verifier_session($auteur);
 }
 
 
@@ -65,36 +65,36 @@ 
  */
 function supprimer_sessions($id_auteur, $toutes=true, $actives=true) {
 
-	$nb_files = 0;
-	$nb_max_files = (defined('_MAX_NB_SESSIONS_OUVERTES')?_MAX_NB_SESSIONS_OUVERTES:1000);
-	spip_log("supprimer sessions auteur $id_auteur");
-	if ($toutes OR $id_auteur!==$GLOBALS['visiteur_session']['id_auteur']) {
-		if ($dir = opendir(_DIR_SESSIONS)){
-			$t = $_SERVER['REQUEST_TIME']  - (4*_RENOUVELLE_ALEA); // 48h par defaut
-			$t_short = $_SERVER['REQUEST_TIME']  - max(_RENOUVELLE_ALEA/4,3*3600); // 3h par defaut
-			while(($f = readdir($dir)) !== false) {
-				$nb_files++;
-				if (preg_match(",^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,", $f, $regs)){
-					$f = _DIR_SESSIONS . $f;
-					if (($actives AND $regs[1] == $id_auteur) OR ($t > filemtime($f))){
-						spip_unlink($f);
-					}
-					// si il y a trop de sessions ouvertes, on purge les sessions anonymes de plus de 3H
-					// cd http://core.spip.org/issues/3276
-					elseif($nb_files>$nb_max_files AND !intval($regs[1]) AND ($t_short > filemtime($f))){
-						spip_unlink($f);
-					}
-				}
-			}
-		}
-	}
-	else {
-		verifier_session();
-		spip_unlink(fichier_session('alea_ephemere', true));
-	}
-
-	// forcer le recalcul de la session courante
-	spip_session(true);
+    $nb_files = 0;
+    $nb_max_files = (defined('_MAX_NB_SESSIONS_OUVERTES')?_MAX_NB_SESSIONS_OUVERTES:1000);
+    spip_log("supprimer sessions auteur $id_auteur");
+    if ($toutes OR $id_auteur!==$GLOBALS['visiteur_session']['id_auteur']) {
+        if ($dir = opendir(_DIR_SESSIONS)){
+            $t = $_SERVER['REQUEST_TIME']  - (4*_RENOUVELLE_ALEA); // 48h par defaut
+            $t_short = $_SERVER['REQUEST_TIME']  - max(_RENOUVELLE_ALEA/4,3*3600); // 3h par defaut
+            while(($f = readdir($dir)) !== false) {
+                $nb_files++;
+                if (preg_match(",^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,", $f, $regs)){
+                    $f = _DIR_SESSIONS . $f;
+                    if (($actives AND $regs[1] == $id_auteur) OR ($t > filemtime($f))){
+                        spip_unlink($f);
+                    }
+                    // si il y a trop de sessions ouvertes, on purge les sessions anonymes de plus de 3H
+                    // cd http://core.spip.org/issues/3276
+                    elseif($nb_files>$nb_max_files AND !intval($regs[1]) AND ($t_short > filemtime($f))){
+                        spip_unlink($f);
+                    }
+                }
+            }
+        }
+    }
+    else {
+        verifier_session();
+        spip_unlink(fichier_session('alea_ephemere', true));
+    }
+
+    // forcer le recalcul de la session courante
+    spip_session(true);
 }
 
 /**
@@ -106,84 +106,84 @@ 
  * @return bool|string
  */
 function ajouter_session($auteur) {
-	// Si le client a deja une session valide pour son id_auteur
-	// on conserve le meme fichier
-
-	// Attention un visiteur peut avoir une session et un id=0,
-	// => ne pas melanger les sessions des differents visiteurs
-	$id_auteur = intval($auteur['id_auteur']);
-
-	// Si ce n'est pas un inscrit (les inscrits ont toujours des choses en session)
-	// on va vérifier s'il y a vraiment des choses à écrire
-	if (!$id_auteur){
-		// On supprime les données de base pour voir le contenu réel de la session
-		$auteur_verif = $auteur;
-		if (isset($auteur_verif['id_auteur'])) unset($auteur_verif['id_auteur']);
-		if (isset($auteur_verif['hash_env'])) unset($auteur_verif['hash_env']);
-		if (isset($auteur_verif['ip_change'])) unset($auteur_verif['ip_change']);
-		if (isset($auteur_verif['date_session'])) unset($auteur_verif['date_session']);
+    // Si le client a deja une session valide pour son id_auteur
+    // on conserve le meme fichier
+
+    // Attention un visiteur peut avoir une session et un id=0,
+    // => ne pas melanger les sessions des differents visiteurs
+    $id_auteur = intval($auteur['id_auteur']);
+
+    // Si ce n'est pas un inscrit (les inscrits ont toujours des choses en session)
+    // on va vérifier s'il y a vraiment des choses à écrire
+    if (!$id_auteur){
+        // On supprime les données de base pour voir le contenu réel de la session
+        $auteur_verif = $auteur;
+        if (isset($auteur_verif['id_auteur'])) unset($auteur_verif['id_auteur']);
+        if (isset($auteur_verif['hash_env'])) unset($auteur_verif['hash_env']);
+        if (isset($auteur_verif['ip_change'])) unset($auteur_verif['ip_change']);
+        if (isset($auteur_verif['date_session'])) unset($auteur_verif['date_session']);
 		
-		// Les variables vraiment nulle ne sont pas à prendre en compte non plus
-		foreach($auteur_verif as $variable=>$valeur){
-			if ($valeur === null){
-				unset($auteur_verif[$variable]);
-			}
-		}
-		// Si après ça la session est vide et qu'on a pas de cookie session, on arrete
-		if (!$auteur_verif AND !isset($_COOKIE['spip_session'])){
-			return false;
-		}
-	}
-
-	if (!isset($_COOKIE['spip_session'])
-	  OR !preg_match(',^'.$id_auteur.'_,', $_COOKIE['spip_session'])){
-		$_COOKIE['spip_session'] = $id_auteur.'_'.md5(uniqid(rand(),true));
-	}
-
-	$fichier_session = fichier_session('alea_ephemere');
-
-	// Si la session est vide alors on supprime l'éventuel fichier et on arrête là
-	if (!$id_auteur AND !$auteur_verif){
-		if (@file_exists($fichier_session)) spip_unlink($fichier_session);
-		// unset le COOKIE de session
-		// car il est pris en compte dans spip_session() qui va croire a tort qu'on est pas un visiteur anonyme
-		unset($_COOKIE['spip_session']);
-		return false;
-	}
-
-	// Maintenant on sait qu'on a des choses à écrire
-	// On s'assure d'avoir au moins ces valeurs
-	$auteur['id_auteur'] = $id_auteur;
-	if (!isset($auteur['hash_env'])) $auteur['hash_env'] = hash_env();
-	if (!isset($auteur['ip_change'])) $auteur['ip_change'] = false;
-
-	if (!isset($auteur['date_session'])) $auteur['date_session'] = time();
-	if (is_string($auteur['prefs']))
-		$auteur['prefs'] = unserialize($auteur['prefs']);
-
-	if (!ecrire_fichier_session($fichier_session, $auteur)) {
-		spip_log('Echec ecriture fichier session '.$fichier_session,_LOG_HS);
-		include_spip('inc/minipres');
-		echo minipres();
-		exit;
-	} else {
-		include_spip('inc/cookie');
-		$duree = _RENOUVELLE_ALEA *
-		  (!isset($auteur['cookie'])
-		  	? 2 : (is_numeric($auteur['cookie'])
-				? $auteur['cookie'] : 20));
-		spip_setcookie(
-			'spip_session',
-			$_COOKIE['spip_session'],
-			time() + $duree
-			);
-		spip_log("ajoute session $fichier_session cookie $duree");
-
-		# on en profite pour purger les vieilles sessions abandonnees
-		supprimer_sessions(0, true, false);
-
-		return $_COOKIE['spip_session'];
-	}
+        // Les variables vraiment nulle ne sont pas à prendre en compte non plus
+        foreach($auteur_verif as $variable=>$valeur){
+            if ($valeur === null){
+                unset($auteur_verif[$variable]);
+            }
+        }
+        // Si après ça la session est vide et qu'on a pas de cookie session, on arrete
+        if (!$auteur_verif AND !isset($_COOKIE['spip_session'])){
+            return false;
+        }
+    }
+
+    if (!isset($_COOKIE['spip_session'])
+      OR !preg_match(',^'.$id_auteur.'_,', $_COOKIE['spip_session'])){
+        $_COOKIE['spip_session'] = $id_auteur.'_'.md5(uniqid(rand(),true));
+    }
+
+    $fichier_session = fichier_session('alea_ephemere');
+
+    // Si la session est vide alors on supprime l'éventuel fichier et on arrête là
+    if (!$id_auteur AND !$auteur_verif){
+        if (@file_exists($fichier_session)) spip_unlink($fichier_session);
+        // unset le COOKIE de session
+        // car il est pris en compte dans spip_session() qui va croire a tort qu'on est pas un visiteur anonyme
+        unset($_COOKIE['spip_session']);
+        return false;
+    }
+
+    // Maintenant on sait qu'on a des choses à écrire
+    // On s'assure d'avoir au moins ces valeurs
+    $auteur['id_auteur'] = $id_auteur;
+    if (!isset($auteur['hash_env'])) $auteur['hash_env'] = hash_env();
+    if (!isset($auteur['ip_change'])) $auteur['ip_change'] = false;
+
+    if (!isset($auteur['date_session'])) $auteur['date_session'] = time();
+    if (is_string($auteur['prefs']))
+        $auteur['prefs'] = unserialize($auteur['prefs']);
+
+    if (!ecrire_fichier_session($fichier_session, $auteur)) {
+        spip_log('Echec ecriture fichier session '.$fichier_session,_LOG_HS);
+        include_spip('inc/minipres');
+        echo minipres();
+        exit;
+    } else {
+        include_spip('inc/cookie');
+        $duree = _RENOUVELLE_ALEA *
+          (!isset($auteur['cookie'])
+              ? 2 : (is_numeric($auteur['cookie'])
+                ? $auteur['cookie'] : 20));
+        spip_setcookie(
+            'spip_session',
+            $_COOKIE['spip_session'],
+            time() + $duree
+            );
+        spip_log("ajoute session $fichier_session cookie $duree");
+
+        # on en profite pour purger les vieilles sessions abandonnees
+        supprimer_sessions(0, true, false);
+
+        return $_COOKIE['spip_session'];
+    }
 }
 
 
@@ -200,69 +200,69 @@ 
  * @return bool|int|null
  */
 function verifier_session($change=false) {
-	// si pas de cookie, c'est fichu
-
-	if (!isset($_COOKIE['spip_session']))
-		return false;
-
-	// Tester avec alea courant
-	$fichier_session = fichier_session('alea_ephemere', true);
-
-	if ($fichier_session AND @file_exists($fichier_session)) {
-		include($fichier_session);
-	} else {
-		// Sinon, tester avec alea precedent
-		$fichier_session = fichier_session('alea_ephemere_ancien', true);
-		if (!$fichier_session OR !@file_exists($fichier_session)) return false;
-
-		// Renouveler la session avec l'alea courant
-		include($fichier_session);
-		spip_log('renouvelle session '.$GLOBALS['visiteur_session']['id_auteur']);
-		spip_unlink($fichier_session);
-		ajouter_session($GLOBALS['visiteur_session']);
-	}
-
-	// Compatibilite ascendante : auteur_session est visiteur_session si
-	// c'est un auteur SPIP authentifie (tandis qu'un visiteur_session peut
-	// n'etre qu'identifie, sans aucune authentification).
-
-	if ($GLOBALS['visiteur_session']['id_auteur'])
-		$GLOBALS['auteur_session'] = &$GLOBALS['visiteur_session'];
-
-
-	// Si l'adresse IP change, inc/presentation mettra une balise image
-	// avec un URL de rappel demandant a changer le nom de la session.
-	// Seul celui qui a l'IP d'origine est rejoue
-	// ainsi un eventuel voleur de cookie ne pourrait pas deconnecter
-	// sa victime, mais se ferait deconnecter par elle.
-	if (hash_env() != $GLOBALS['visiteur_session']['hash_env']) {
-		if (!$GLOBALS['visiteur_session']['ip_change']) {
-			define('_SESSION_REJOUER',rejouer_session());
-			$GLOBALS['visiteur_session']['ip_change'] = true;
-			ajouter_session($GLOBALS['visiteur_session']);
-		} else if ($change) {
-			spip_log("session non rejouee, vol de cookie ?");
-		}
-	} else if ($change) {
-		spip_log("rejoue session $fichier_session ".$_COOKIE['spip_session']);
-		spip_unlink($fichier_session);
-		$GLOBALS['visiteur_session']['ip_change'] = false;
-		unset($_COOKIE['spip_session']);
-		ajouter_session($GLOBALS['visiteur_session']);
-	}
-
-	// Si la session a ete initiee il y a trop longtemps, elle est annulee
-	if (isset($GLOBALS['visiteur_session'])
-	AND defined('_AGE_SESSION_MAX')
-	AND _AGE_SESSION_MAX > 0
-	AND time() - @$GLOBALS['visiteur_session']['date_session'] > _AGE_SESSION_MAX) {
-		unset($GLOBALS['visiteur_session']);
-		return false;
-	}
-
-	return is_numeric($GLOBALS['visiteur_session']['id_auteur'])
-		? $GLOBALS['visiteur_session']['id_auteur']
-		: null;
+    // si pas de cookie, c'est fichu
+
+    if (!isset($_COOKIE['spip_session']))
+        return false;
+
+    // Tester avec alea courant
+    $fichier_session = fichier_session('alea_ephemere', true);
+
+    if ($fichier_session AND @file_exists($fichier_session)) {
+        include($fichier_session);
+    } else {
+        // Sinon, tester avec alea precedent
+        $fichier_session = fichier_session('alea_ephemere_ancien', true);
+        if (!$fichier_session OR !@file_exists($fichier_session)) return false;
+
+        // Renouveler la session avec l'alea courant
+        include($fichier_session);
+        spip_log('renouvelle session '.$GLOBALS['visiteur_session']['id_auteur']);
+        spip_unlink($fichier_session);
+        ajouter_session($GLOBALS['visiteur_session']);
+    }
+
+    // Compatibilite ascendante : auteur_session est visiteur_session si
+    // c'est un auteur SPIP authentifie (tandis qu'un visiteur_session peut
+    // n'etre qu'identifie, sans aucune authentification).
+
+    if ($GLOBALS['visiteur_session']['id_auteur'])
+        $GLOBALS['auteur_session'] = &$GLOBALS['visiteur_session'];
+
+
+    // Si l'adresse IP change, inc/presentation mettra une balise image
+    // avec un URL de rappel demandant a changer le nom de la session.
+    // Seul celui qui a l'IP d'origine est rejoue
+    // ainsi un eventuel voleur de cookie ne pourrait pas deconnecter
+    // sa victime, mais se ferait deconnecter par elle.
+    if (hash_env() != $GLOBALS['visiteur_session']['hash_env']) {
+        if (!$GLOBALS['visiteur_session']['ip_change']) {
+            define('_SESSION_REJOUER',rejouer_session());
+            $GLOBALS['visiteur_session']['ip_change'] = true;
+            ajouter_session($GLOBALS['visiteur_session']);
+        } else if ($change) {
+            spip_log("session non rejouee, vol de cookie ?");
+        }
+    } else if ($change) {
+        spip_log("rejoue session $fichier_session ".$_COOKIE['spip_session']);
+        spip_unlink($fichier_session);
+        $GLOBALS['visiteur_session']['ip_change'] = false;
+        unset($_COOKIE['spip_session']);
+        ajouter_session($GLOBALS['visiteur_session']);
+    }
+
+    // Si la session a ete initiee il y a trop longtemps, elle est annulee
+    if (isset($GLOBALS['visiteur_session'])
+    AND defined('_AGE_SESSION_MAX')
+    AND _AGE_SESSION_MAX > 0
+    AND time() - @$GLOBALS['visiteur_session']['date_session'] > _AGE_SESSION_MAX) {
+        unset($GLOBALS['visiteur_session']);
+        return false;
+    }
+
+    return is_numeric($GLOBALS['visiteur_session']['id_auteur'])
+        ? $GLOBALS['visiteur_session']['id_auteur']
+        : null;
 }
 
 /**
@@ -274,7 +274,7 @@ 
  * @return mixed
  */
 function session_get($nom) {
-	return isset($GLOBALS['visiteur_session'][$nom]) ? $GLOBALS['visiteur_session'][$nom] : null;
+    return isset($GLOBALS['visiteur_session'][$nom]) ? $GLOBALS['visiteur_session'][$nom] : null;
 }
 
 
@@ -288,18 +288,18 @@ 
  */
 function session_set($nom, $val=null) {
 
-	if (is_null($val)){
-		// rien a faire
-		if (!isset($GLOBALS['visiteur_session'][$nom])) return;
-		unset($GLOBALS['visiteur_session'][$nom]);
-	}
-	else {
-		// On ajoute la valeur dans la globale
-		$GLOBALS['visiteur_session'][$nom] = $val;
-	}
-
-	ajouter_session($GLOBALS['visiteur_session']);
-	actualiser_sessions($GLOBALS['visiteur_session']);
+    if (is_null($val)){
+        // rien a faire
+        if (!isset($GLOBALS['visiteur_session'][$nom])) return;
+        unset($GLOBALS['visiteur_session'][$nom]);
+    }
+    else {
+        // On ajoute la valeur dans la globale
+        $GLOBALS['visiteur_session'][$nom] = $val;
+    }
+
+    ajouter_session($GLOBALS['visiteur_session']);
+    actualiser_sessions($GLOBALS['visiteur_session']);
 }
 
 /**
@@ -313,33 +313,33 @@ 
  * @param array $auteur
  */
 function actualiser_sessions($auteur) {
-	if (!$id_auteur = intval($auteur['id_auteur']))
-		return;
-
-	// memoriser l'auteur courant (celui qui modifie la fiche)
-	$sauve = $GLOBALS['visiteur_session'];
-
-	// .. mettre a jour les sessions de l'auteur cible
-	// attention au $ final pour ne pas risquer d'embarquer un .php.jeton temporaire
-	// cree par une ecriture concurente d'une session (fichier atomique temporaire)
-	$sessions = preg_files(_DIR_SESSIONS, '/'.$id_auteur.'_.*\.php$');
-	foreach($sessions as $session) {
-		$GLOBALS['visiteur_session'] = array();
-		// a pu etre supprime entre le preg initial et le moment ou l'on arrive la (concurrence)
-		if (@file_exists($session)){
-			include $session; # $GLOBALS['visiteur_session'] est alors l'auteur cible
-
-			$auteur = array_merge($GLOBALS['visiteur_session'], $auteur);
-			ecrire_fichier_session($session, $auteur);
-		}
-	}
-
-	// restaurer l'auteur courant
-	$GLOBALS['visiteur_session'] = $sauve;
-
-	// si c'est le meme, rafraichir les valeurs
-	if (isset($sauve['id_auteur']) and $auteur['id_auteur'] == $sauve['id_auteur'])
-		verifier_session();
+    if (!$id_auteur = intval($auteur['id_auteur']))
+        return;
+
+    // memoriser l'auteur courant (celui qui modifie la fiche)
+    $sauve = $GLOBALS['visiteur_session'];
+
+    // .. mettre a jour les sessions de l'auteur cible
+    // attention au $ final pour ne pas risquer d'embarquer un .php.jeton temporaire
+    // cree par une ecriture concurente d'une session (fichier atomique temporaire)
+    $sessions = preg_files(_DIR_SESSIONS, '/'.$id_auteur.'_.*\.php$');
+    foreach($sessions as $session) {
+        $GLOBALS['visiteur_session'] = array();
+        // a pu etre supprime entre le preg initial et le moment ou l'on arrive la (concurrence)
+        if (@file_exists($session)){
+            include $session; # $GLOBALS['visiteur_session'] est alors l'auteur cible
+
+            $auteur = array_merge($GLOBALS['visiteur_session'], $auteur);
+            ecrire_fichier_session($session, $auteur);
+        }
+    }
+
+    // restaurer l'auteur courant
+    $GLOBALS['visiteur_session'] = $sauve;
+
+    // si c'est le meme, rafraichir les valeurs
+    if (isset($sauve['id_auteur']) and $auteur['id_auteur'] == $sauve['id_auteur'])
+        verifier_session();
 }
 
 /**
@@ -353,33 +353,33 @@ 
  */
 function ecrire_fichier_session($fichier, $auteur) {
 
-	$row = $auteur;
+    $row = $auteur;
 
-	// ne pas enregistrer ces elements de securite
-	// dans le fichier de session
-	unset($auteur['pass']);
-	unset($auteur['htpass']);
-	unset($auteur['low_sec']);
-	unset($auteur['alea_actuel']);
-	unset($auteur['alea_futur']);
+    // ne pas enregistrer ces elements de securite
+    // dans le fichier de session
+    unset($auteur['pass']);
+    unset($auteur['htpass']);
+    unset($auteur['low_sec']);
+    unset($auteur['alea_actuel']);
+    unset($auteur['alea_futur']);
 
-	$auteur = pipeline('preparer_fichier_session',array('args'=>array('row'=>$row),'data'=>$auteur));
+    $auteur = pipeline('preparer_fichier_session',array('args'=>array('row'=>$row),'data'=>$auteur));
 
-	// ne pas enregistrer les valeurs vraiment nulle dans le fichier
-	foreach($auteur as $variable=>$valeur){
-		if ($valeur === null){
-			unset($auteur[$variable]);
-		}
-	}
+    // ne pas enregistrer les valeurs vraiment nulle dans le fichier
+    foreach($auteur as $variable=>$valeur){
+        if ($valeur === null){
+            unset($auteur[$variable]);
+        }
+    }
 	
-	// enregistrer les autres donnees du visiteur
-	$texte = "<"."?php\n";
-	foreach ($auteur as $var => $val)
-		$texte .= '$GLOBALS[\'visiteur_session\'][\''.$var.'\'] = '
-			. var_export($val,true).";\n";
-	$texte .= "?".">\n";
-
-	return ecrire_fichier($fichier, $texte);
+    // enregistrer les autres donnees du visiteur
+    $texte = "<"."?php\n";
+    foreach ($auteur as $var => $val)
+        $texte .= '$GLOBALS[\'visiteur_session\'][\''.$var.'\'] = '
+            . var_export($val,true).";\n";
+    $texte .= "?".">\n";
+
+    return ecrire_fichier($fichier, $texte);
 }
 
 
@@ -394,24 +394,24 @@ 
  */
 function fichier_session($alea, $tantpis=false) {
 
-	if (!isset($GLOBALS['meta'][$alea])) {
-		include_spip('base/abstract_sql');
-		$GLOBALS['meta'][$alea]  = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea), '','', '', '', '', 'continue');
-	}
-
-	if (!$GLOBALS['meta'][$alea]) {
-		if (!$tantpis) {
-			spip_log("fichier session ($tantpis): $alea indisponible");
-			include_spip('inc/minipres');
-			echo minipres();
-		}
-		return ''; // echec mais $tanpis
-	}
-	else {
-		$repertoire = sous_repertoire(_DIR_SESSIONS,'',false,$tantpis);
-		$c = $_COOKIE['spip_session'];
-		return $repertoire . intval($c) .'_' . md5($c.' '.$GLOBALS['meta'][$alea]). '.php';
-	}
+    if (!isset($GLOBALS['meta'][$alea])) {
+        include_spip('base/abstract_sql');
+        $GLOBALS['meta'][$alea]  = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea), '','', '', '', '', 'continue');
+    }
+
+    if (!$GLOBALS['meta'][$alea]) {
+        if (!$tantpis) {
+            spip_log("fichier session ($tantpis): $alea indisponible");
+            include_spip('inc/minipres');
+            echo minipres();
+        }
+        return ''; // echec mais $tanpis
+    }
+    else {
+        $repertoire = sous_repertoire(_DIR_SESSIONS,'',false,$tantpis);
+        $c = $_COOKIE['spip_session'];
+        return $repertoire . intval($c) .'_' . md5($c.' '.$GLOBALS['meta'][$alea]). '.php';
+    }
 }
 
 
@@ -428,7 +428,7 @@ 
  */
 function rejouer_session()
 {
-	return '<img src="'.generer_url_action('cookie','change_session=oui', true).'" width="0" height="0" alt="" />';
+    return '<img src="'.generer_url_action('cookie','change_session=oui', true).'" width="0" height="0" alt="" />';
 }
 
 
@@ -440,9 +440,9 @@ 
  * @return string
  */
 function hash_env() {
-  static $res ='';
-  if ($res) return $res;
-  return $res = md5($GLOBALS['ip'] . $_SERVER['HTTP_USER_AGENT']);
+    static $res ='';
+    if ($res) return $res;
+    return $res = md5($GLOBALS['ip'] . $_SERVER['HTTP_USER_AGENT']);
 }
 
 ?>

Braces +66 added lines, -38 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;
+}
 
 /*
  * Gestion de l'authentification par sessions
@@ -34,13 +36,14 @@ 
  */
 function inc_session_dist($auteur=false)
 {
-	if (is_numeric($auteur))
-		return supprimer_sessions($auteur, $auteur > 0);
-	else if (is_array($auteur))
-		return ajouter_session($auteur);
-	else
-		return verifier_session($auteur);
-}
+	if (is_numeric($auteur)) {
+			return supprimer_sessions($auteur, $auteur > 0);
+	} else if (is_array($auteur)) {
+			return ajouter_session($auteur);
+	} else {
+			return verifier_session($auteur);
+	}
+	}
 
 
 /**
@@ -87,8 +90,7 @@ 
 				}
 			}
 		}
-	}
-	else {
+	} else {
 		verifier_session();
 		spip_unlink(fichier_session('alea_ephemere', true));
 	}
@@ -118,10 +120,18 @@ 
 	if (!$id_auteur){
 		// On supprime les données de base pour voir le contenu réel de la session
 		$auteur_verif = $auteur;
-		if (isset($auteur_verif['id_auteur'])) unset($auteur_verif['id_auteur']);
-		if (isset($auteur_verif['hash_env'])) unset($auteur_verif['hash_env']);
-		if (isset($auteur_verif['ip_change'])) unset($auteur_verif['ip_change']);
-		if (isset($auteur_verif['date_session'])) unset($auteur_verif['date_session']);
+		if (isset($auteur_verif['id_auteur'])) {
+		    unset($auteur_verif['id_auteur']);
+		}
+		if (isset($auteur_verif['hash_env'])) {
+		    unset($auteur_verif['hash_env']);
+		}
+		if (isset($auteur_verif['ip_change'])) {
+		    unset($auteur_verif['ip_change']);
+		}
+		if (isset($auteur_verif['date_session'])) {
+		    unset($auteur_verif['date_session']);
+		}
 		
 		// Les variables vraiment nulle ne sont pas à prendre en compte non plus
 		foreach($auteur_verif as $variable=>$valeur){
@@ -144,7 +154,9 @@ 
 
 	// Si la session est vide alors on supprime l'éventuel fichier et on arrête là
 	if (!$id_auteur AND !$auteur_verif){
-		if (@file_exists($fichier_session)) spip_unlink($fichier_session);
+		if (@file_exists($fichier_session)) {
+		    spip_unlink($fichier_session);
+		}
 		// unset le COOKIE de session
 		// car il est pris en compte dans spip_session() qui va croire a tort qu'on est pas un visiteur anonyme
 		unset($_COOKIE['spip_session']);
@@ -154,12 +166,19 @@ 
 	// Maintenant on sait qu'on a des choses à écrire
 	// On s'assure d'avoir au moins ces valeurs
 	$auteur['id_auteur'] = $id_auteur;
-	if (!isset($auteur['hash_env'])) $auteur['hash_env'] = hash_env();
-	if (!isset($auteur['ip_change'])) $auteur['ip_change'] = false;
+	if (!isset($auteur['hash_env'])) {
+	    $auteur['hash_env'] = hash_env();
+	}
+	if (!isset($auteur['ip_change'])) {
+	    $auteur['ip_change'] = false;
+	}
 
-	if (!isset($auteur['date_session'])) $auteur['date_session'] = time();
-	if (is_string($auteur['prefs']))
-		$auteur['prefs'] = unserialize($auteur['prefs']);
+	if (!isset($auteur['date_session'])) {
+	    $auteur['date_session'] = time();
+	}
+	if (is_string($auteur['prefs'])) {
+			$auteur['prefs'] = unserialize($auteur['prefs']);
+	}
 
 	if (!ecrire_fichier_session($fichier_session, $auteur)) {
 		spip_log('Echec ecriture fichier session '.$fichier_session,_LOG_HS);
@@ -202,8 +221,9 @@ 
 function verifier_session($change=false) {
 	// si pas de cookie, c'est fichu
 
-	if (!isset($_COOKIE['spip_session']))
-		return false;
+	if (!isset($_COOKIE['spip_session'])) {
+			return false;
+	}
 
 	// Tester avec alea courant
 	$fichier_session = fichier_session('alea_ephemere', true);
@@ -213,7 +233,9 @@ 
 	} else {
 		// Sinon, tester avec alea precedent
 		$fichier_session = fichier_session('alea_ephemere_ancien', true);
-		if (!$fichier_session OR !@file_exists($fichier_session)) return false;
+		if (!$fichier_session OR !@file_exists($fichier_session)) {
+		    return false;
+		}
 
 		// Renouveler la session avec l'alea courant
 		include($fichier_session);
@@ -226,8 +248,9 @@ 
 	// c'est un auteur SPIP authentifie (tandis qu'un visiteur_session peut
 	// n'etre qu'identifie, sans aucune authentification).
 
-	if ($GLOBALS['visiteur_session']['id_auteur'])
-		$GLOBALS['auteur_session'] = &$GLOBALS['visiteur_session'];
+	if ($GLOBALS['visiteur_session']['id_auteur']) {
+			$GLOBALS['auteur_session'] = &$GLOBALS['visiteur_session'];
+	}
 
 
 	// Si l'adresse IP change, inc/presentation mettra une balise image
@@ -290,10 +313,11 @@ 
 
 	if (is_null($val)){
 		// rien a faire
-		if (!isset($GLOBALS['visiteur_session'][$nom])) return;
+		if (!isset($GLOBALS['visiteur_session'][$nom])) {
+		    return;
+		}
 		unset($GLOBALS['visiteur_session'][$nom]);
-	}
-	else {
+	} else {
 		// On ajoute la valeur dans la globale
 		$GLOBALS['visiteur_session'][$nom] = $val;
 	}
@@ -313,8 +337,9 @@ 
  * @param array $auteur
  */
 function actualiser_sessions($auteur) {
-	if (!$id_auteur = intval($auteur['id_auteur']))
-		return;
+	if (!$id_auteur = intval($auteur['id_auteur'])) {
+			return;
+	}
 
 	// memoriser l'auteur courant (celui qui modifie la fiche)
 	$sauve = $GLOBALS['visiteur_session'];
@@ -338,9 +363,10 @@ 
 	$GLOBALS['visiteur_session'] = $sauve;
 
 	// si c'est le meme, rafraichir les valeurs
-	if (isset($sauve['id_auteur']) and $auteur['id_auteur'] == $sauve['id_auteur'])
-		verifier_session();
-}
+	if (isset($sauve['id_auteur']) and $auteur['id_auteur'] == $sauve['id_auteur']) {
+			verifier_session();
+	}
+	}
 
 /**
  * Ecrire le fichier d'une session
@@ -374,9 +400,10 @@ 
 	
 	// enregistrer les autres donnees du visiteur
 	$texte = "<"."?php\n";
-	foreach ($auteur as $var => $val)
-		$texte .= '$GLOBALS[\'visiteur_session\'][\''.$var.'\'] = '
+	foreach ($auteur as $var => $val) {
+			$texte .= '$GLOBALS[\'visiteur_session\'][\''.$var.'\'] = '
 			. var_export($val,true).";\n";
+	}
 	$texte .= "?".">\n";
 
 	return ecrire_fichier($fichier, $texte);
@@ -406,8 +433,7 @@ 
 			echo minipres();
 		}
 		return ''; // echec mais $tanpis
-	}
-	else {
+	} else {
 		$repertoire = sous_repertoire(_DIR_SESSIONS,'',false,$tantpis);
 		$c = $_COOKIE['spip_session'];
 		return $repertoire . intval($c) .'_' . md5($c.' '.$GLOBALS['meta'][$alea]). '.php';
@@ -441,7 +467,9 @@ 
  */
 function hash_env() {
   static $res ='';
-  if ($res) return $res;
+  if ($res) {
+      return $res;
+  }
   return $res = md5($GLOBALS['ip'] . $_SERVER['HTTP_USER_AGENT']);
 }
 

Spacing +37 added lines, -37 removed lines

@@ -32,7 +32,7 @@ 
  * @param int|array|bool $auteur
  * @return bool|null|void
  */
-function inc_session_dist($auteur=false)
+function inc_session_dist($auteur = false)
 {
 	if (is_numeric($auteur))
 		return supprimer_sessions($auteur, $auteur > 0);
@@ -63,25 +63,25 @@ 
  * 		false pour ne pas supprimer les sessions valides de $id_auteur.
  * 		false revient donc a uniquement supprimer les vieilles sessions !
  */
-function supprimer_sessions($id_auteur, $toutes=true, $actives=true) {
+function supprimer_sessions($id_auteur, $toutes = true, $actives = true) {
 
 	$nb_files = 0;
-	$nb_max_files = (defined('_MAX_NB_SESSIONS_OUVERTES')?_MAX_NB_SESSIONS_OUVERTES:1000);
+	$nb_max_files = (defined('_MAX_NB_SESSIONS_OUVERTES') ?_MAX_NB_SESSIONS_OUVERTES:1000);
 	spip_log("supprimer sessions auteur $id_auteur");
-	if ($toutes OR $id_auteur!==$GLOBALS['visiteur_session']['id_auteur']) {
-		if ($dir = opendir(_DIR_SESSIONS)){
-			$t = $_SERVER['REQUEST_TIME']  - (4*_RENOUVELLE_ALEA); // 48h par defaut
-			$t_short = $_SERVER['REQUEST_TIME']  - max(_RENOUVELLE_ALEA/4,3*3600); // 3h par defaut
-			while(($f = readdir($dir)) !== false) {
+	if ($toutes OR $id_auteur !== $GLOBALS['visiteur_session']['id_auteur']) {
+		if ($dir = opendir(_DIR_SESSIONS)) {
+			$t = $_SERVER['REQUEST_TIME'] - (4 * _RENOUVELLE_ALEA); // 48h par defaut
+			$t_short = $_SERVER['REQUEST_TIME'] - max(_RENOUVELLE_ALEA / 4, 3 * 3600); // 3h par defaut
+			while (($f = readdir($dir)) !== false) {
 				$nb_files++;
-				if (preg_match(",^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,", $f, $regs)){
-					$f = _DIR_SESSIONS . $f;
-					if (($actives AND $regs[1] == $id_auteur) OR ($t > filemtime($f))){
+				if (preg_match(",^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,", $f, $regs)) {
+					$f = _DIR_SESSIONS.$f;
+					if (($actives AND $regs[1] == $id_auteur) OR ($t > filemtime($f))) {
 						spip_unlink($f);
 					}
 					// si il y a trop de sessions ouvertes, on purge les sessions anonymes de plus de 3H
 					// cd http://core.spip.org/issues/3276
-					elseif($nb_files>$nb_max_files AND !intval($regs[1]) AND ($t_short > filemtime($f))){
+					elseif ($nb_files > $nb_max_files AND !intval($regs[1]) AND ($t_short > filemtime($f))) {
 						spip_unlink($f);
 					}
 				}
@@ -115,7 +115,7 @@ 
 
 	// Si ce n'est pas un inscrit (les inscrits ont toujours des choses en session)
 	// on va vérifier s'il y a vraiment des choses à écrire
-	if (!$id_auteur){
+	if (!$id_auteur) {
 		// On supprime les données de base pour voir le contenu réel de la session
 		$auteur_verif = $auteur;
 		if (isset($auteur_verif['id_auteur'])) unset($auteur_verif['id_auteur']);
@@ -124,26 +124,26 @@ 
 		if (isset($auteur_verif['date_session'])) unset($auteur_verif['date_session']);
 		
 		// Les variables vraiment nulle ne sont pas à prendre en compte non plus
-		foreach($auteur_verif as $variable=>$valeur){
-			if ($valeur === null){
+		foreach ($auteur_verif as $variable=>$valeur) {
+			if ($valeur === null) {
 				unset($auteur_verif[$variable]);
 			}
 		}
 		// Si après ça la session est vide et qu'on a pas de cookie session, on arrete
-		if (!$auteur_verif AND !isset($_COOKIE['spip_session'])){
+		if (!$auteur_verif AND !isset($_COOKIE['spip_session'])) {
 			return false;
 		}
 	}
 
 	if (!isset($_COOKIE['spip_session'])
-	  OR !preg_match(',^'.$id_auteur.'_,', $_COOKIE['spip_session'])){
-		$_COOKIE['spip_session'] = $id_auteur.'_'.md5(uniqid(rand(),true));
+	  OR !preg_match(',^'.$id_auteur.'_,', $_COOKIE['spip_session'])) {
+		$_COOKIE['spip_session'] = $id_auteur.'_'.md5(uniqid(rand(), true));
 	}
 
 	$fichier_session = fichier_session('alea_ephemere');
 
 	// Si la session est vide alors on supprime l'éventuel fichier et on arrête là
-	if (!$id_auteur AND !$auteur_verif){
+	if (!$id_auteur AND !$auteur_verif) {
 		if (@file_exists($fichier_session)) spip_unlink($fichier_session);
 		// unset le COOKIE de session
 		// car il est pris en compte dans spip_session() qui va croire a tort qu'on est pas un visiteur anonyme
@@ -162,7 +162,7 @@ 
 		$auteur['prefs'] = unserialize($auteur['prefs']);
 
 	if (!ecrire_fichier_session($fichier_session, $auteur)) {
-		spip_log('Echec ecriture fichier session '.$fichier_session,_LOG_HS);
+		spip_log('Echec ecriture fichier session '.$fichier_session, _LOG_HS);
 		include_spip('inc/minipres');
 		echo minipres();
 		exit;
@@ -199,7 +199,7 @@ 
  * @param bool $change
  * @return bool|int|null
  */
-function verifier_session($change=false) {
+function verifier_session($change = false) {
 	// si pas de cookie, c'est fichu
 
 	if (!isset($_COOKIE['spip_session']))
@@ -237,7 +237,7 @@ 
 	// sa victime, mais se ferait deconnecter par elle.
 	if (hash_env() != $GLOBALS['visiteur_session']['hash_env']) {
 		if (!$GLOBALS['visiteur_session']['ip_change']) {
-			define('_SESSION_REJOUER',rejouer_session());
+			define('_SESSION_REJOUER', rejouer_session());
 			$GLOBALS['visiteur_session']['ip_change'] = true;
 			ajouter_session($GLOBALS['visiteur_session']);
 		} else if ($change) {
@@ -286,9 +286,9 @@ 
  * @param null $val
  * @return void
  */
-function session_set($nom, $val=null) {
+function session_set($nom, $val = null) {
 
-	if (is_null($val)){
+	if (is_null($val)) {
 		// rien a faire
 		if (!isset($GLOBALS['visiteur_session'][$nom])) return;
 		unset($GLOBALS['visiteur_session'][$nom]);
@@ -323,10 +323,10 @@ 
 	// attention au $ final pour ne pas risquer d'embarquer un .php.jeton temporaire
 	// cree par une ecriture concurente d'une session (fichier atomique temporaire)
 	$sessions = preg_files(_DIR_SESSIONS, '/'.$id_auteur.'_.*\.php$');
-	foreach($sessions as $session) {
+	foreach ($sessions as $session) {
 		$GLOBALS['visiteur_session'] = array();
 		// a pu etre supprime entre le preg initial et le moment ou l'on arrive la (concurrence)
-		if (@file_exists($session)){
+		if (@file_exists($session)) {
 			include $session; # $GLOBALS['visiteur_session'] est alors l'auteur cible
 
 			$auteur = array_merge($GLOBALS['visiteur_session'], $auteur);
@@ -363,11 +363,11 @@ 
 	unset($auteur['alea_actuel']);
 	unset($auteur['alea_futur']);
 
-	$auteur = pipeline('preparer_fichier_session',array('args'=>array('row'=>$row),'data'=>$auteur));
+	$auteur = pipeline('preparer_fichier_session', array('args'=>array('row'=>$row), 'data'=>$auteur));
 
 	// ne pas enregistrer les valeurs vraiment nulle dans le fichier
-	foreach($auteur as $variable=>$valeur){
-		if ($valeur === null){
+	foreach ($auteur as $variable=>$valeur) {
+		if ($valeur === null) {
 			unset($auteur[$variable]);
 		}
 	}
@@ -376,7 +376,7 @@ 
 	$texte = "<"."?php\n";
 	foreach ($auteur as $var => $val)
 		$texte .= '$GLOBALS[\'visiteur_session\'][\''.$var.'\'] = '
-			. var_export($val,true).";\n";
+			. var_export($val, true).";\n";
 	$texte .= "?".">\n";
 
 	return ecrire_fichier($fichier, $texte);
@@ -392,11 +392,11 @@ 
  * @param bool $tantpis
  * @return string
  */
-function fichier_session($alea, $tantpis=false) {
+function fichier_session($alea, $tantpis = false) {
 
 	if (!isset($GLOBALS['meta'][$alea])) {
 		include_spip('base/abstract_sql');
-		$GLOBALS['meta'][$alea]  = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea), '','', '', '', '', 'continue');
+		$GLOBALS['meta'][$alea] = sql_getfetsel('valeur', 'spip_meta', "nom=".sql_quote($alea), '', '', '', '', '', 'continue');
 	}
 
 	if (!$GLOBALS['meta'][$alea]) {
@@ -408,9 +408,9 @@ 
 		return ''; // echec mais $tanpis
 	}
 	else {
-		$repertoire = sous_repertoire(_DIR_SESSIONS,'',false,$tantpis);
+		$repertoire = sous_repertoire(_DIR_SESSIONS, '', false, $tantpis);
 		$c = $_COOKIE['spip_session'];
-		return $repertoire . intval($c) .'_' . md5($c.' '.$GLOBALS['meta'][$alea]). '.php';
+		return $repertoire.intval($c).'_'.md5($c.' '.$GLOBALS['meta'][$alea]).'.php';
 	}
 }
 
@@ -428,7 +428,7 @@ 
  */
 function rejouer_session()
 {
-	return '<img src="'.generer_url_action('cookie','change_session=oui', true).'" width="0" height="0" alt="" />';
+	return '<img src="'.generer_url_action('cookie', 'change_session=oui', true).'" width="0" height="0" alt="" />';
 }
 
 
@@ -440,9 +440,9 @@ 
  * @return string
  */
 function hash_env() {
-  static $res ='';
+  static $res = '';
   if ($res) return $res;
-  return $res = md5($GLOBALS['ip'] . $_SERVER['HTTP_USER_AGENT']);
+  return $res = md5($GLOBALS['ip'].$_SERVER['HTTP_USER_AGENT']);
 }
 
 ?>