spip / SPIP

ecrire/genie/optimiser.php

Doc Comments +3 added lines

@@ -65,6 +65,9 @@
 // L'index du SELECT doit s'appeler "id"
 
 // http://doc.spip.org/@optimiser_sansref
+/**
+ * @param string $id
+ */
 function optimiser_sansref($table, $id, $sel, $and="")
 {
 	$in = array();

Indentation +74 added lines, -74 removed lines

@@ -17,44 +17,44 @@ 
 // http://doc.spip.org/@genie_optimiser_dist
 function genie_optimiser_dist($t) {
 
-	optimiser_base_une_table();
-	optimiser_base();
-
-	// la date souhaitee pour le tour suivant = apres-demain a 4h du mat ;
-	// sachant qu'on a un delai de 48h, on renvoie aujourd'hui a 4h du mat
-	// avec une periode de flou entre 2h et 6h pour ne pas saturer un hebergeur
-	// qui aurait beaucoup de sites SPIP
-	return -(mktime(2,0,0) + rand(0, 3600*4));
+    optimiser_base_une_table();
+    optimiser_base();
+
+    // la date souhaitee pour le tour suivant = apres-demain a 4h du mat ;
+    // sachant qu'on a un delai de 48h, on renvoie aujourd'hui a 4h du mat
+    // avec une periode de flou entre 2h et 6h pour ne pas saturer un hebergeur
+    // qui aurait beaucoup de sites SPIP
+    return -(mktime(2,0,0) + rand(0, 3600*4));
 }
 
 // heure de reference pour le garbage collector = 24h auparavant
 // http://doc.spip.org/@optimiser_base
 function optimiser_base($attente = 86400) {
-	optimiser_base_disparus($attente);
+    optimiser_base_disparus($attente);
 }
 
 
 // http://doc.spip.org/@optimiser_base_une_table
 function optimiser_base_une_table() {
 
-	$tables = array();
-	$result = sql_showbase();
-
-	// on n'optimise qu'une seule table a chaque fois,
-	// pour ne pas vautrer le systeme
-	// lire http://dev.mysql.com/doc/refman/5.0/fr/optimize-table.html
-	while ($row = sql_fetch($result))
-		$tables[] = array_shift($row);
-
-	if ($tables) {
-		$table_op = intval($GLOBALS['meta']['optimiser_table']+1) % sizeof($tables);
-		ecrire_meta('optimiser_table', $table_op);
-		$q = $tables[$table_op];
-		spip_log("debut d'optimisation de la table $q");
-		if (sql_optimize($q))
-			spip_log("fin d'optimisation de la table $q");
-		else spip_log("Pas d'optimiseur necessaire");
-	}
+    $tables = array();
+    $result = sql_showbase();
+
+    // on n'optimise qu'une seule table a chaque fois,
+    // pour ne pas vautrer le systeme
+    // lire http://dev.mysql.com/doc/refman/5.0/fr/optimize-table.html
+    while ($row = sql_fetch($result))
+        $tables[] = array_shift($row);
+
+    if ($tables) {
+        $table_op = intval($GLOBALS['meta']['optimiser_table']+1) % sizeof($tables);
+        ecrire_meta('optimiser_table', $table_op);
+        $q = $tables[$table_op];
+        spip_log("debut d'optimisation de la table $q");
+        if (sql_optimize($q))
+            spip_log("fin d'optimisation de la table $q");
+        else spip_log("Pas d'optimiseur necessaire");
+    }
 }
 
 // mysql < 4.0 refuse les requetes DELETE multi table 
@@ -67,15 +67,15 @@ 
 // http://doc.spip.org/@optimiser_sansref
 function optimiser_sansref($table, $id, $sel, $and="")
 {
-	$in = array();
-	while ($row = sql_fetch($sel)) $in[$row['id']]=true;
-	sql_free($sel);
-
-	if ($in) {
-		sql_delete($table,  sql_in($id,array_keys($in)) . ($and?" AND $and":""));
-		spip_log("Numeros des entrees $id supprimees dans la table $table: $in");
-	}
-	return count($in);
+    $in = array();
+    while ($row = sql_fetch($sel)) $in[$row['id']]=true;
+    sql_free($sel);
+
+    if ($in) {
+        sql_delete($table,  sql_in($id,array_keys($in)) . ($and?" AND $and":""));
+        spip_log("Numeros des entrees $id supprimees dans la table $table: $in");
+    }
+    return count($in);
 }
 
 // Nomenclature des liens morts entre les tables,
@@ -86,63 +86,63 @@ 
 // http://doc.spip.org/@optimiser_base_disparus
 function optimiser_base_disparus($attente = 86400) {
 
-	# format = 20060610110141, si on veut forcer une optimisation tout de suite
-	$mydate = sql_quote(date("Y-m-d H:i:s", time() - $attente));
+    # format = 20060610110141, si on veut forcer une optimisation tout de suite
+    $mydate = sql_quote(date("Y-m-d H:i:s", time() - $attente));
 
-	$n = 0;
+    $n = 0;
 
-	//
-	// Rubriques 
-	//
+    //
+    // Rubriques 
+    //
 
-	# les articles qui sont dans une id_rubrique inexistante
-	# attention on controle id_rubrique>0 pour ne pas tuer les articles
-	# specialement affectes a une rubrique non-existante (plugin,
-	# cf. http://trac.rezo.net/trac/spip/ticket/1549 )
-	$res = sql_select("A.id_article AS id",
-		        "spip_articles AS A
+    # les articles qui sont dans une id_rubrique inexistante
+    # attention on controle id_rubrique>0 pour ne pas tuer les articles
+    # specialement affectes a une rubrique non-existante (plugin,
+    # cf. http://trac.rezo.net/trac/spip/ticket/1549 )
+    $res = sql_select("A.id_article AS id",
+                "spip_articles AS A
 		        LEFT JOIN spip_rubriques AS R
 		          ON A.id_rubrique=R.id_rubrique",
-			 "A.id_rubrique > 0
+                "A.id_rubrique > 0
 			 AND R.id_rubrique IS NULL
 		         AND A.maj < $mydate");
 
-	$n+= optimiser_sansref('spip_articles', 'id_article', $res);
+    $n+= optimiser_sansref('spip_articles', 'id_article', $res);
 
-	// les articles a la poubelle
-	sql_delete("spip_articles", "statut='poubelle' AND maj < $mydate");
+    // les articles a la poubelle
+    sql_delete("spip_articles", "statut='poubelle' AND maj < $mydate");
 
-	//
-	// Auteurs
-	//
+    //
+    // Auteurs
+    //
 
-	include_spip('action/editer_liens');
-	// optimiser les liens de tous les auteurs vers des objets effaces
-	// et depuis des auteurs effaces
-	$n+= objet_optimiser_liens(array('auteur'=>'*'),'*');
+    include_spip('action/editer_liens');
+    // optimiser les liens de tous les auteurs vers des objets effaces
+    // et depuis des auteurs effaces
+    $n+= objet_optimiser_liens(array('auteur'=>'*'),'*');
 
-	# effacer les auteurs poubelle qui ne sont lies a rien
-	$res = sql_select("A.id_auteur AS id",
-		      	"spip_auteurs AS A
+    # effacer les auteurs poubelle qui ne sont lies a rien
+    $res = sql_select("A.id_auteur AS id",
+                    "spip_auteurs AS A
 		      	LEFT JOIN spip_auteurs_liens AS L
 		          ON L.id_auteur=A.id_auteur",
-			"L.id_auteur IS NULL
+            "L.id_auteur IS NULL
 		       	AND A.statut='5poubelle' AND A.maj < $mydate");
 
-	$n+= optimiser_sansref('spip_auteurs', 'id_auteur', $res);
+    $n+= optimiser_sansref('spip_auteurs', 'id_auteur', $res);
 
-	# supprimer les auteurs 'nouveau' qui n'ont jamais donne suite
-	# au mail de confirmation (45 jours pour repondre, ca devrait suffire)
-	sql_delete("spip_auteurs", "statut='nouveau' AND maj < ". sql_quote(date('Y-m-d', time()-45*24*3600)));
+    # supprimer les auteurs 'nouveau' qui n'ont jamais donne suite
+    # au mail de confirmation (45 jours pour repondre, ca devrait suffire)
+    sql_delete("spip_auteurs", "statut='nouveau' AND maj < ". sql_quote(date('Y-m-d', time()-45*24*3600)));
 
 
-	$n = pipeline('optimiser_base_disparus', array(
-			'args'=>array(
-				'attente' => $attente,
-				'date' => $mydate),
-			'data'=>$n
-	));
+    $n = pipeline('optimiser_base_disparus', array(
+            'args'=>array(
+                'attente' => $attente,
+                'date' => $mydate),
+            'data'=>$n
+    ));
 	
-	if (!$n) spip_log("Optimisation des tables: aucun lien mort");
+    if (!$n) spip_log("Optimisation des tables: aucun lien mort");
 }
 ?>

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 	// sachant qu'on a un delai de 48h, on renvoie aujourd'hui a 4h du mat
 	// avec une periode de flou entre 2h et 6h pour ne pas saturer un hebergeur
 	// qui aurait beaucoup de sites SPIP
-	return -(mktime(2,0,0) + rand(0, 3600*4));
+	return -(mktime(2, 0, 0) + rand(0, 3600 * 4));
 }
 
 // heure de reference pour le garbage collector = 24h auparavant
@@ -47,7 +47,7 @@ 
 		$tables[] = array_shift($row);
 
 	if ($tables) {
-		$table_op = intval($GLOBALS['meta']['optimiser_table']+1) % sizeof($tables);
+		$table_op = intval($GLOBALS['meta']['optimiser_table'] + 1) % sizeof($tables);
 		ecrire_meta('optimiser_table', $table_op);
 		$q = $tables[$table_op];
 		spip_log("debut d'optimisation de la table $q");
@@ -65,14 +65,14 @@ 
 // L'index du SELECT doit s'appeler "id"
 
 // http://doc.spip.org/@optimiser_sansref
-function optimiser_sansref($table, $id, $sel, $and="")
+function optimiser_sansref($table, $id, $sel, $and = "")
 {
 	$in = array();
-	while ($row = sql_fetch($sel)) $in[$row['id']]=true;
+	while ($row = sql_fetch($sel)) $in[$row['id']] = true;
 	sql_free($sel);
 
 	if ($in) {
-		sql_delete($table,  sql_in($id,array_keys($in)) . ($and?" AND $and":""));
+		sql_delete($table, sql_in($id, array_keys($in)).($and ? " AND $and" : ""));
 		spip_log("Numeros des entrees $id supprimees dans la table $table: $in");
 	}
 	return count($in);
@@ -107,7 +107,7 @@ 
 			 AND R.id_rubrique IS NULL
 		         AND A.maj < $mydate");
 
-	$n+= optimiser_sansref('spip_articles', 'id_article', $res);
+	$n += optimiser_sansref('spip_articles', 'id_article', $res);
 
 	// les articles a la poubelle
 	sql_delete("spip_articles", "statut='poubelle' AND maj < $mydate");
@@ -119,7 +119,7 @@ 
 	include_spip('action/editer_liens');
 	// optimiser les liens de tous les auteurs vers des objets effaces
 	// et depuis des auteurs effaces
-	$n+= objet_optimiser_liens(array('auteur'=>'*'),'*');
+	$n += objet_optimiser_liens(array('auteur'=>'*'), '*');
 
 	# effacer les auteurs poubelle qui ne sont lies a rien
 	$res = sql_select("A.id_auteur AS id",
@@ -129,11 +129,11 @@ 
 			"L.id_auteur IS NULL
 		       	AND A.statut='5poubelle' AND A.maj < $mydate");
 
-	$n+= optimiser_sansref('spip_auteurs', 'id_auteur', $res);
+	$n += optimiser_sansref('spip_auteurs', 'id_auteur', $res);
 
 	# supprimer les auteurs 'nouveau' qui n'ont jamais donne suite
 	# au mail de confirmation (45 jours pour repondre, ca devrait suffire)
-	sql_delete("spip_auteurs", "statut='nouveau' AND maj < ". sql_quote(date('Y-m-d', time()-45*24*3600)));
+	sql_delete("spip_auteurs", "statut='nouveau' AND maj < ".sql_quote(date('Y-m-d', time() - 45 * 24 * 3600)));
 
 
 	$n = pipeline('optimiser_base_disparus', array(

Braces +18 added lines, -9 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('base/abstract_sql');
 
@@ -43,17 +45,20 @@ 
 	// on n'optimise qu'une seule table a chaque fois,
 	// pour ne pas vautrer le systeme
 	// lire http://dev.mysql.com/doc/refman/5.0/fr/optimize-table.html
-	while ($row = sql_fetch($result))
-		$tables[] = array_shift($row);
+	while ($row = sql_fetch($result)) {
+			$tables[] = array_shift($row);
+	}
 
 	if ($tables) {
 		$table_op = intval($GLOBALS['meta']['optimiser_table']+1) % sizeof($tables);
 		ecrire_meta('optimiser_table', $table_op);
 		$q = $tables[$table_op];
 		spip_log("debut d'optimisation de la table $q");
-		if (sql_optimize($q))
-			spip_log("fin d'optimisation de la table $q");
-		else spip_log("Pas d'optimiseur necessaire");
+		if (sql_optimize($q)) {
+					spip_log("fin d'optimisation de la table $q");
+		} else {
+		    spip_log("Pas d'optimiseur necessaire");
+		}
 	}
 }
 
@@ -68,7 +73,9 @@ 
 function optimiser_sansref($table, $id, $sel, $and="")
 {
 	$in = array();
-	while ($row = sql_fetch($sel)) $in[$row['id']]=true;
+	while ($row = sql_fetch($sel)) {
+	    $in[$row['id']]=true;
+	}
 	sql_free($sel);
 
 	if ($in) {
@@ -143,6 +150,8 @@ 
 			'data'=>$n
 	));
 	
-	if (!$n) spip_log("Optimisation des tables: aucun lien mort");
-}
+	if (!$n) {
+	    spip_log("Optimisation des tables: aucun lien mort");
+	}
+	}
 ?>

ecrire/inc/acces.php

Doc Comments +6 added lines

@@ -105,6 +105,9 @@ 
 // Inclure les arguments significatifs pour le hachage
 // cas particulier du statut pour compatibilite ancien rss/suivi_revisions
 
+/**
+ * @param string $op
+ */
 function param_low_sec($op, $args=array(), $lang='', $mime='rss')
 {
 	$a = $b = '';
@@ -202,6 +205,9 @@ 
 // Installe ou verifie un .htaccess, y compris sa prise en compte par Apache
 //
 // http://doc.spip.org/@verifier_htaccess
+/**
+ * @param string|null $rep
+ */
 function verifier_htaccess($rep, $force=false) {
 	$htaccess = rtrim($rep,"/") . "/" . _ACCESS_FILE_NAME;
 	if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force)

Indentation +155 added lines, -155 removed lines

@@ -14,31 +14,31 @@ 
 
 // http://doc.spip.org/@creer_pass_aleatoire
 function creer_pass_aleatoire($longueur = 8, $sel = "") {
-	$seed = (double) (microtime() + 1) * time();
-	mt_srand($seed);
-	srand($seed);
-	$s = '';
-	$pass = '';
-	for ($i = 0; $i < $longueur; $i++) {
-		if (!$s) {
-			$s = mt_rand();
-			if (!$s) $s = rand();
-			$s = substr(md5(uniqid($s).$sel), 0, 16);
-		}
-		$r = unpack("Cr", pack("H2", $s.$s));
-		$x = $r['r'] & 63;
-		if ($x < 10) $x = chr($x + 48);
-		else if ($x < 36) $x = chr($x + 55);
-		else if ($x < 62) $x = chr($x + 61);
-		else if ($x == 63) $x = '/';
-		else $x = '.';
-		$pass .= $x;
-		$s = substr($s, 2);
-	}
-	$pass = preg_replace("@[./]@", "a", $pass);
-	$pass = preg_replace("@[I1l]@", "L", $pass);
-	$pass = preg_replace("@[0O]@", "o", $pass);
-	return $pass;
+    $seed = (double) (microtime() + 1) * time();
+    mt_srand($seed);
+    srand($seed);
+    $s = '';
+    $pass = '';
+    for ($i = 0; $i < $longueur; $i++) {
+        if (!$s) {
+            $s = mt_rand();
+            if (!$s) $s = rand();
+            $s = substr(md5(uniqid($s).$sel), 0, 16);
+        }
+        $r = unpack("Cr", pack("H2", $s.$s));
+        $x = $r['r'] & 63;
+        if ($x < 10) $x = chr($x + 48);
+        else if ($x < 36) $x = chr($x + 55);
+        else if ($x < 62) $x = chr($x + 61);
+        else if ($x == 63) $x = '/';
+        else $x = '.';
+        $pass .= $x;
+        $s = substr($s, 2);
+    }
+    $pass = preg_replace("@[./]@", "a", $pass);
+    $pass = preg_replace("@[I1l]@", "L", $pass);
+    $pass = preg_replace("@[0O]@", "o", $pass);
+    return $pass;
 }
 
 /**
@@ -49,18 +49,18 @@ 
  * @return string
  */
 function creer_uniqid() {
-	static $seeded;
-
-	if (!$seeded) {
-		$seed = (double) (microtime() + 1) * time();
-		mt_srand($seed);
-		srand($seed);
-		$seeded = true;
-	}
-
-	$s = mt_rand();
-	if (!$s) $s = rand();
-	return uniqid($s, 1);
+    static $seeded;
+
+    if (!$seeded) {
+        $seed = (double) (microtime() + 1) * time();
+        mt_srand($seed);
+        srand($seed);
+        $seeded = true;
+    }
+
+    $s = mt_rand();
+    if (!$s) $s = rand();
+    return uniqid($s, 1);
 }
 
 //
@@ -69,15 +69,15 @@ 
 
 // http://doc.spip.org/@renouvelle_alea
 function renouvelle_alea() {
-	if (!isset($GLOBALS['meta']['alea_ephemere'])){
-		include_spip('base/abstract_sql');
-		$GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
-	}
-	ecrire_meta('alea_ephemere_ancien', @$GLOBALS['meta']['alea_ephemere'], 'non');
-	$GLOBALS['meta']['alea_ephemere'] = md5(creer_uniqid());
-	ecrire_meta('alea_ephemere', $GLOBALS['meta']['alea_ephemere'], 'non');
-	ecrire_meta('alea_ephemere_date', time(), 'non');
-	spip_log("renouvellement de l'alea_ephemere");
+    if (!isset($GLOBALS['meta']['alea_ephemere'])){
+        include_spip('base/abstract_sql');
+        $GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
+    }
+    ecrire_meta('alea_ephemere_ancien', @$GLOBALS['meta']['alea_ephemere'], 'non');
+    $GLOBALS['meta']['alea_ephemere'] = md5(creer_uniqid());
+    ecrire_meta('alea_ephemere', $GLOBALS['meta']['alea_ephemere'], 'non');
+    ecrire_meta('alea_ephemere_date', time(), 'non');
+    spip_log("renouvellement de l'alea_ephemere");
 }
 
 //
@@ -86,20 +86,20 @@ 
 //
 // http://doc.spip.org/@low_sec
 function low_sec($id_auteur) {
-	// Pas d'id_auteur : low_sec
-	if (!$id_auteur = intval($id_auteur)) {
-		if (!$low_sec = $GLOBALS['meta']['low_sec']) {
-			ecrire_meta('low_sec', $low_sec = creer_pass_aleatoire());
-		}
-	}
-	else {
-		$low_sec = sql_getfetsel("low_sec", "spip_auteurs", "id_auteur = $id_auteur");
-		if (!$low_sec) {
-			$low_sec = creer_pass_aleatoire();
-			sql_updateq("spip_auteurs", array("low_sec" => $low_sec), "id_auteur = $id_auteur");
-		}
-	}
-	return $low_sec;
+    // Pas d'id_auteur : low_sec
+    if (!$id_auteur = intval($id_auteur)) {
+        if (!$low_sec = $GLOBALS['meta']['low_sec']) {
+            ecrire_meta('low_sec', $low_sec = creer_pass_aleatoire());
+        }
+    }
+    else {
+        $low_sec = sql_getfetsel("low_sec", "spip_auteurs", "id_auteur = $id_auteur");
+        if (!$low_sec) {
+            $low_sec = creer_pass_aleatoire();
+            sql_updateq("spip_auteurs", array("low_sec" => $low_sec), "id_auteur = $id_auteur");
+        }
+    }
+    return $low_sec;
 }
 
 // Inclure les arguments significatifs pour le hachage
@@ -107,46 +107,46 @@ 
 
 function param_low_sec($op, $args=array(), $lang='', $mime='rss')
 {
-	$a = $b = '';
-	foreach ($args as $val => $var)
-		if ($var) {
-			if ($val<>'statut') $a .= ':' . $val.'-'.$var;
-			$b .= $val.'='.$var . '&';
-		}
-	$a = substr($a,1);
-	$id = intval(@$GLOBALS['connect_id_auteur']);
-	return $b
-	  . "op="
-	  . $op
-	  . "&id="
-	  . $id
-	  . "&cle="
-	  . afficher_low_sec($id, "$mime $op $a")
-	  . (!$a ? '' : "&args=$a")
-	  . (!$lang ? '' : "&lang=$lang");
+    $a = $b = '';
+    foreach ($args as $val => $var)
+        if ($var) {
+            if ($val<>'statut') $a .= ':' . $val.'-'.$var;
+            $b .= $val.'='.$var . '&';
+        }
+    $a = substr($a,1);
+    $id = intval(@$GLOBALS['connect_id_auteur']);
+    return $b
+        . "op="
+        . $op
+        . "&id="
+        . $id
+        . "&cle="
+        . afficher_low_sec($id, "$mime $op $a")
+        . (!$a ? '' : "&args=$a")
+        . (!$lang ? '' : "&lang=$lang");
 }
 
 // http://doc.spip.org/@afficher_low_sec
 function afficher_low_sec ($id_auteur, $action='') {
-	return substr(md5($action.low_sec($id_auteur)),0,8);
+    return substr(md5($action.low_sec($id_auteur)),0,8);
 }
 
 // http://doc.spip.org/@verifier_low_sec
 function verifier_low_sec ($id_auteur, $cle, $action='') {
-	return ($cle == afficher_low_sec($id_auteur, $action));
+    return ($cle == afficher_low_sec($id_auteur, $action));
 }
 
 // http://doc.spip.org/@effacer_low_sec
 function effacer_low_sec($id_auteur) {
-	if (!$id_auteur = intval($id_auteur)) return; // jamais trop prudent ;)
-	sql_updateq("spip_auteurs", array("low_sec" => ''), "id_auteur = $id_auteur");
+    if (!$id_auteur = intval($id_auteur)) return; // jamais trop prudent ;)
+    sql_updateq("spip_auteurs", array("low_sec" => ''), "id_auteur = $id_auteur");
 }
 
 // http://doc.spip.org/@initialiser_sel
 function initialiser_sel() {
-	global $htsalt;
-	if (CRYPT_MD5) $htsalt = '$1$'.creer_pass_aleatoire();
-	else return "";
+    global $htsalt;
+    if (CRYPT_MD5) $htsalt = '$1$'.creer_pass_aleatoire();
+    else return "";
 }
 
 // Cette fonction ne sert qu'a la connexion en mode http_auth.non LDAP
@@ -154,48 +154,48 @@ 
 // Voir le plugin "acces restreint"
 // http://doc.spip.org/@ecrire_acces
 function ecrire_acces() {
-	$htaccess = _DIR_RESTREINT . _ACCESS_FILE_NAME;
-	$htpasswd = _DIR_TMP . _AUTH_USER_FILE;
-
-	// Cette variable de configuration peut etre posee par un plugin
-	// par exemple acces_restreint ;
-	// si .htaccess existe, outrepasser spip_meta
-	if (($GLOBALS['meta']['creer_htpasswd'] != 'oui')
-	AND !@file_exists($htaccess)) {
-		spip_unlink($htpasswd);
-		spip_unlink($htpasswd."-admin");
-		return;
-	}
-
-	# remarque : ici on laisse passer les "nouveau" de maniere a leur permettre
-	# de devenir redacteur le cas echeant (auth http)... a nettoyer
-	// attention, il faut au prealable se connecter a la base (necessaire car utilise par install)
-	// TODO: factoriser avec auth/spip qui fait deja ce job et generaliser le test spip_connect_ldap()
-
-	if (spip_connect_ldap()) return;
-	$p1 = ''; // login:htpass pour tous
-	$p2 = ''; // login:htpass pour les admins
-	$s = sql_select("login, htpass, statut", "spip_auteurs", sql_in("statut",  array('1comite','0minirezo','nouveau')));
-	while ($t = sql_fetch($s)) {
-		if (strlen($t['login']) AND strlen($t['htpass'])) {
-			$p1 .= $t['login'].':'.$t['htpass']."\n";
-			if ($t['statut'] == '0minirezo')
-				$p2 .= $t['login'].':'.$t['htpass']."\n";
-		}
-	}
-	if ($p1) {
-	  ecrire_fichier($htpasswd, $p1);
-	  ecrire_fichier($htpasswd.'-admin', $p2);
-	  spip_log("Ecriture de $htpasswd et $htpasswd-admin");
-	}
+    $htaccess = _DIR_RESTREINT . _ACCESS_FILE_NAME;
+    $htpasswd = _DIR_TMP . _AUTH_USER_FILE;
+
+    // Cette variable de configuration peut etre posee par un plugin
+    // par exemple acces_restreint ;
+    // si .htaccess existe, outrepasser spip_meta
+    if (($GLOBALS['meta']['creer_htpasswd'] != 'oui')
+    AND !@file_exists($htaccess)) {
+        spip_unlink($htpasswd);
+        spip_unlink($htpasswd."-admin");
+        return;
+    }
+
+    # remarque : ici on laisse passer les "nouveau" de maniere a leur permettre
+    # de devenir redacteur le cas echeant (auth http)... a nettoyer
+    // attention, il faut au prealable se connecter a la base (necessaire car utilise par install)
+    // TODO: factoriser avec auth/spip qui fait deja ce job et generaliser le test spip_connect_ldap()
+
+    if (spip_connect_ldap()) return;
+    $p1 = ''; // login:htpass pour tous
+    $p2 = ''; // login:htpass pour les admins
+    $s = sql_select("login, htpass, statut", "spip_auteurs", sql_in("statut",  array('1comite','0minirezo','nouveau')));
+    while ($t = sql_fetch($s)) {
+        if (strlen($t['login']) AND strlen($t['htpass'])) {
+            $p1 .= $t['login'].':'.$t['htpass']."\n";
+            if ($t['statut'] == '0minirezo')
+                $p2 .= $t['login'].':'.$t['htpass']."\n";
+        }
+    }
+    if ($p1) {
+        ecrire_fichier($htpasswd, $p1);
+        ecrire_fichier($htpasswd.'-admin', $p2);
+        spip_log("Ecriture de $htpasswd et $htpasswd-admin");
+    }
 }
 
 
 // http://doc.spip.org/@generer_htpass
 function generer_htpass($pass) {
-	global $htsalt;
-	if (function_exists('crypt'))
-		return crypt($pass, $htsalt);
+    global $htsalt;
+    if (function_exists('crypt'))
+        return crypt($pass, $htsalt);
 }
 
 //
@@ -203,46 +203,46 @@ 
 //
 // http://doc.spip.org/@verifier_htaccess
 function verifier_htaccess($rep, $force=false) {
-	$htaccess = rtrim($rep,"/") . "/" . _ACCESS_FILE_NAME;
-	if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force)
-		return true;
-	if ($ht = @fopen($htaccess, "w")) {
-		fputs($ht, "deny from all\n");
-		fclose($ht);
-		@chmod($htaccess, _SPIP_CHMOD & 0666);
-		$t = rtrim($rep,"/") . "/.ok";
-		if ($ht = @fopen($t, "w")) {
-			@fclose($ht);
-			include_spip('inc/distant');
-			$t = substr($t,strlen(_DIR_RACINE));
-			$t = url_de_base() . $t;
-			$ht = recuperer_lapage($t, false, 'HEAD', 0);
-			// htaccess inoperant si on a recupere des entetes HTTP
-			// (ignorer la reussite si connexion par fopen)
-			$ht = !(isset($ht[0]) AND $ht[0]);
-		}
-	}
-	spip_log("Creation de $htaccess " . ($ht ? " reussie" : " manquee"));
-	return $ht;
+    $htaccess = rtrim($rep,"/") . "/" . _ACCESS_FILE_NAME;
+    if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force)
+        return true;
+    if ($ht = @fopen($htaccess, "w")) {
+        fputs($ht, "deny from all\n");
+        fclose($ht);
+        @chmod($htaccess, _SPIP_CHMOD & 0666);
+        $t = rtrim($rep,"/") . "/.ok";
+        if ($ht = @fopen($t, "w")) {
+            @fclose($ht);
+            include_spip('inc/distant');
+            $t = substr($t,strlen(_DIR_RACINE));
+            $t = url_de_base() . $t;
+            $ht = recuperer_lapage($t, false, 'HEAD', 0);
+            // htaccess inoperant si on a recupere des entetes HTTP
+            // (ignorer la reussite si connexion par fopen)
+            $ht = !(isset($ht[0]) AND $ht[0]);
+        }
+    }
+    spip_log("Creation de $htaccess " . ($ht ? " reussie" : " manquee"));
+    return $ht;
 }	
 
 
 
 // http://doc.spip.org/@gerer_htaccess
 function gerer_htaccess() {
-	// Cette variable de configuration peut etre posee par un plugin
-	// par exemple acces_restreint
-	$f = ($GLOBALS['meta']['creer_htaccess'] === 'oui');
-	$dirs = sql_allfetsel('extension', 'spip_types_documents');
-	$dirs[] = array('extension' => 'distant');
-	foreach($dirs as $e) {
-		if (is_dir($dir = _DIR_IMG . $e['extension'])) {
-			if ($f)
-				verifier_htaccess($dir);
-			else spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
-		}
-	}
-	return $GLOBALS['meta']['creer_htaccess'];
+    // Cette variable de configuration peut etre posee par un plugin
+    // par exemple acces_restreint
+    $f = ($GLOBALS['meta']['creer_htaccess'] === 'oui');
+    $dirs = sql_allfetsel('extension', 'spip_types_documents');
+    $dirs[] = array('extension' => 'distant');
+    foreach($dirs as $e) {
+        if (is_dir($dir = _DIR_IMG . $e['extension'])) {
+            if ($f)
+                verifier_htaccess($dir);
+            else spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
+        }
+    }
+    return $GLOBALS['meta']['creer_htaccess'];
 }
 
 initialiser_sel();

Spacing +20 added lines, -20 removed lines

@@ -69,7 +69,7 @@ 
 
 // http://doc.spip.org/@renouvelle_alea
 function renouvelle_alea() {
-	if (!isset($GLOBALS['meta']['alea_ephemere'])){
+	if (!isset($GLOBALS['meta']['alea_ephemere'])) {
 		include_spip('base/abstract_sql');
 		$GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
 	}
@@ -105,15 +105,15 @@ 
 // Inclure les arguments significatifs pour le hachage
 // cas particulier du statut pour compatibilite ancien rss/suivi_revisions
 
-function param_low_sec($op, $args=array(), $lang='', $mime='rss')
+function param_low_sec($op, $args = array(), $lang = '', $mime = 'rss')
 {
 	$a = $b = '';
 	foreach ($args as $val => $var)
 		if ($var) {
-			if ($val<>'statut') $a .= ':' . $val.'-'.$var;
-			$b .= $val.'='.$var . '&';
+			if ($val <> 'statut') $a .= ':'.$val.'-'.$var;
+			$b .= $val.'='.$var.'&';
 		}
-	$a = substr($a,1);
+	$a = substr($a, 1);
 	$id = intval(@$GLOBALS['connect_id_auteur']);
 	return $b
 	  . "op="
@@ -127,12 +127,12 @@ 
 }
 
 // http://doc.spip.org/@afficher_low_sec
-function afficher_low_sec ($id_auteur, $action='') {
-	return substr(md5($action.low_sec($id_auteur)),0,8);
+function afficher_low_sec($id_auteur, $action = '') {
+	return substr(md5($action.low_sec($id_auteur)), 0, 8);
 }
 
 // http://doc.spip.org/@verifier_low_sec
-function verifier_low_sec ($id_auteur, $cle, $action='') {
+function verifier_low_sec($id_auteur, $cle, $action = '') {
 	return ($cle == afficher_low_sec($id_auteur, $action));
 }
 
@@ -154,8 +154,8 @@ 
 // Voir le plugin "acces restreint"
 // http://doc.spip.org/@ecrire_acces
 function ecrire_acces() {
-	$htaccess = _DIR_RESTREINT . _ACCESS_FILE_NAME;
-	$htpasswd = _DIR_TMP . _AUTH_USER_FILE;
+	$htaccess = _DIR_RESTREINT._ACCESS_FILE_NAME;
+	$htpasswd = _DIR_TMP._AUTH_USER_FILE;
 
 	// Cette variable de configuration peut etre posee par un plugin
 	// par exemple acces_restreint ;
@@ -175,7 +175,7 @@ 
 	if (spip_connect_ldap()) return;
 	$p1 = ''; // login:htpass pour tous
 	$p2 = ''; // login:htpass pour les admins
-	$s = sql_select("login, htpass, statut", "spip_auteurs", sql_in("statut",  array('1comite','0minirezo','nouveau')));
+	$s = sql_select("login, htpass, statut", "spip_auteurs", sql_in("statut", array('1comite', '0minirezo', 'nouveau')));
 	while ($t = sql_fetch($s)) {
 		if (strlen($t['login']) AND strlen($t['htpass'])) {
 			$p1 .= $t['login'].':'.$t['htpass']."\n";
@@ -202,27 +202,27 @@ 
 // Installe ou verifie un .htaccess, y compris sa prise en compte par Apache
 //
 // http://doc.spip.org/@verifier_htaccess
-function verifier_htaccess($rep, $force=false) {
-	$htaccess = rtrim($rep,"/") . "/" . _ACCESS_FILE_NAME;
+function verifier_htaccess($rep, $force = false) {
+	$htaccess = rtrim($rep, "/")."/"._ACCESS_FILE_NAME;
 	if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force)
 		return true;
 	if ($ht = @fopen($htaccess, "w")) {
 		fputs($ht, "deny from all\n");
 		fclose($ht);
 		@chmod($htaccess, _SPIP_CHMOD & 0666);
-		$t = rtrim($rep,"/") . "/.ok";
+		$t = rtrim($rep, "/")."/.ok";
 		if ($ht = @fopen($t, "w")) {
 			@fclose($ht);
 			include_spip('inc/distant');
-			$t = substr($t,strlen(_DIR_RACINE));
-			$t = url_de_base() . $t;
+			$t = substr($t, strlen(_DIR_RACINE));
+			$t = url_de_base().$t;
 			$ht = recuperer_lapage($t, false, 'HEAD', 0);
 			// htaccess inoperant si on a recupere des entetes HTTP
 			// (ignorer la reussite si connexion par fopen)
 			$ht = !(isset($ht[0]) AND $ht[0]);
 		}
 	}
-	spip_log("Creation de $htaccess " . ($ht ? " reussie" : " manquee"));
+	spip_log("Creation de $htaccess ".($ht ? " reussie" : " manquee"));
 	return $ht;
 }	
 
@@ -235,11 +235,11 @@ 
 	$f = ($GLOBALS['meta']['creer_htaccess'] === 'oui');
 	$dirs = sql_allfetsel('extension', 'spip_types_documents');
 	$dirs[] = array('extension' => 'distant');
-	foreach($dirs as $e) {
-		if (is_dir($dir = _DIR_IMG . $e['extension'])) {
+	foreach ($dirs as $e) {
+		if (is_dir($dir = _DIR_IMG.$e['extension'])) {
 			if ($f)
 				verifier_htaccess($dir);
-			else spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
+			else spip_unlink($dir.'/'._ACCESS_FILE_NAME);
 		}
 	}
 	return $GLOBALS['meta']['creer_htaccess'];

Braces +52 added lines, -27 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;
+}
 
 // http://doc.spip.org/@creer_pass_aleatoire
 function creer_pass_aleatoire($longueur = 8, $sel = "") {
@@ -22,16 +24,24 @@ 
 	for ($i = 0; $i < $longueur; $i++) {
 		if (!$s) {
 			$s = mt_rand();
-			if (!$s) $s = rand();
+			if (!$s) {
+			    $s = rand();
+			}
 			$s = substr(md5(uniqid($s).$sel), 0, 16);
 		}
 		$r = unpack("Cr", pack("H2", $s.$s));
 		$x = $r['r'] & 63;
-		if ($x < 10) $x = chr($x + 48);
-		else if ($x < 36) $x = chr($x + 55);
-		else if ($x < 62) $x = chr($x + 61);
-		else if ($x == 63) $x = '/';
-		else $x = '.';
+		if ($x < 10) {
+		    $x = chr($x + 48);
+		} else if ($x < 36) {
+		    $x = chr($x + 55);
+		} else if ($x < 62) {
+		    $x = chr($x + 61);
+		} else if ($x == 63) {
+		    $x = '/';
+		} else {
+		    $x = '.';
+		}
 		$pass .= $x;
 		$s = substr($s, 2);
 	}
@@ -59,7 +69,9 @@ 
 	}
 
 	$s = mt_rand();
-	if (!$s) $s = rand();
+	if (!$s) {
+	    $s = rand();
+	}
 	return uniqid($s, 1);
 }
 
@@ -91,8 +103,7 @@ 
 		if (!$low_sec = $GLOBALS['meta']['low_sec']) {
 			ecrire_meta('low_sec', $low_sec = creer_pass_aleatoire());
 		}
-	}
-	else {
+	} else {
 		$low_sec = sql_getfetsel("low_sec", "spip_auteurs", "id_auteur = $id_auteur");
 		if (!$low_sec) {
 			$low_sec = creer_pass_aleatoire();
@@ -108,9 +119,10 @@ 
 function param_low_sec($op, $args=array(), $lang='', $mime='rss')
 {
 	$a = $b = '';
-	foreach ($args as $val => $var)
-		if ($var) {
+	foreach ($args as $val => $var) {
+			if ($var) {
 			if ($val<>'statut') $a .= ':' . $val.'-'.$var;
+	}
 			$b .= $val.'='.$var . '&';
 		}
 	$a = substr($a,1);
@@ -138,16 +150,22 @@ 
 
 // http://doc.spip.org/@effacer_low_sec
 function effacer_low_sec($id_auteur) {
-	if (!$id_auteur = intval($id_auteur)) return; // jamais trop prudent ;)
+	if (!$id_auteur = intval($id_auteur)) {
+	    return;
+	}
+	// jamais trop prudent ;)
 	sql_updateq("spip_auteurs", array("low_sec" => ''), "id_auteur = $id_auteur");
 }
 
 // http://doc.spip.org/@initialiser_sel
 function initialiser_sel() {
 	global $htsalt;
-	if (CRYPT_MD5) $htsalt = '$1$'.creer_pass_aleatoire();
-	else return "";
-}
+	if (CRYPT_MD5) {
+	    $htsalt = '$1$'.creer_pass_aleatoire();
+	} else {
+	    return "";
+	}
+	}
 
 // Cette fonction ne sert qu'a la connexion en mode http_auth.non LDAP
 // Son role est de creer le fichier htpasswd
@@ -172,15 +190,18 @@ 
 	// attention, il faut au prealable se connecter a la base (necessaire car utilise par install)
 	// TODO: factoriser avec auth/spip qui fait deja ce job et generaliser le test spip_connect_ldap()
 
-	if (spip_connect_ldap()) return;
+	if (spip_connect_ldap()) {
+	    return;
+	}
 	$p1 = ''; // login:htpass pour tous
 	$p2 = ''; // login:htpass pour les admins
 	$s = sql_select("login, htpass, statut", "spip_auteurs", sql_in("statut",  array('1comite','0minirezo','nouveau')));
 	while ($t = sql_fetch($s)) {
 		if (strlen($t['login']) AND strlen($t['htpass'])) {
 			$p1 .= $t['login'].':'.$t['htpass']."\n";
-			if ($t['statut'] == '0minirezo')
-				$p2 .= $t['login'].':'.$t['htpass']."\n";
+			if ($t['statut'] == '0minirezo') {
+							$p2 .= $t['login'].':'.$t['htpass']."\n";
+			}
 		}
 	}
 	if ($p1) {
@@ -194,9 +215,10 @@ 
 // http://doc.spip.org/@generer_htpass
 function generer_htpass($pass) {
 	global $htsalt;
-	if (function_exists('crypt'))
-		return crypt($pass, $htsalt);
-}
+	if (function_exists('crypt')) {
+			return crypt($pass, $htsalt);
+	}
+	}
 
 //
 // Installe ou verifie un .htaccess, y compris sa prise en compte par Apache
@@ -204,8 +226,9 @@ 
 // http://doc.spip.org/@verifier_htaccess
 function verifier_htaccess($rep, $force=false) {
 	$htaccess = rtrim($rep,"/") . "/" . _ACCESS_FILE_NAME;
-	if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force)
-		return true;
+	if (((@file_exists($htaccess)) OR defined('_TEST_DIRS')) AND !$force) {
+			return true;
+	}
 	if ($ht = @fopen($htaccess, "w")) {
 		fputs($ht, "deny from all\n");
 		fclose($ht);
@@ -237,9 +260,11 @@ 
 	$dirs[] = array('extension' => 'distant');
 	foreach($dirs as $e) {
 		if (is_dir($dir = _DIR_IMG . $e['extension'])) {
-			if ($f)
-				verifier_htaccess($dir);
-			else spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
+			if ($f) {
+							verifier_htaccess($dir);
+			} else {
+			    spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
+			}
 		}
 	}
 	return $GLOBALS['meta']['creer_htaccess'];

ecrire/inc/actions.php

Doc Comments +12 added lines, -2 removed lines

@@ -24,11 +24,11 @@ 
  *   -1 : renvoyer action, arg et hash sous forme de array()
  *   true ou false : renvoyer une url, avec & (false) ou & (true)
  *   string : renvoyer un formulaire
- * @param string|int $att
+ * @param string $att
  *   id_auteur pour lequel generer l'action en mode url ou array()
  *   atributs du formulaire en mode formulaire
  * @param bool $public
- * @return array|string
+ * @return string
  */
 function generer_action_auteur($action, $arg, $redirect = "", $mode = false, $att = '', $public = false){
 	$securiser_action = charger_fonction('securiser_action', 'inc');
@@ -36,6 +36,10 @@ 
 }
 
 // http://doc.spip.org/@redirige_action_auteur
+/**
+ * @param string $action
+ * @param string $ret
+ */
 function redirige_action_auteur($action, $arg, $ret, $gra = '', $mode = false, $atts = '', $public = false) {
 
 	$r = ($public ? _DIR_RESTREINT_ABS : _DIR_RESTREINT) .generer_url_ecrire($ret, $gra, true, true);
@@ -44,6 +48,12 @@ 
 }
 
 // http://doc.spip.org/@redirige_action_post
+/**
+ * @param string $action
+ * @param string $arg
+ * @param string $ret
+ * @param string $gra
+ */
 function redirige_action_post($action, $arg, $ret, $gra, $corps, $att = ''){
 	$r = _DIR_RESTREINT.generer_url_ecrire($ret, $gra, false, true);
 	return generer_action_auteur($action, $arg, $r, $corps, $att." method='post'");

Indentation +31 added lines, -31 removed lines

@@ -31,22 +31,22 @@ 
  * @return array|string
  */
 function generer_action_auteur($action, $arg, $redirect = "", $mode = false, $att = '', $public = false){
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	return $securiser_action($action, $arg, $redirect, $mode, $att, $public);
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    return $securiser_action($action, $arg, $redirect, $mode, $att, $public);
 }
 
 // http://doc.spip.org/@redirige_action_auteur
 function redirige_action_auteur($action, $arg, $ret, $gra = '', $mode = false, $atts = '', $public = false) {
 
-	$r = ($public ? _DIR_RESTREINT_ABS : _DIR_RESTREINT) .generer_url_ecrire($ret, $gra, true, true);
+    $r = ($public ? _DIR_RESTREINT_ABS : _DIR_RESTREINT) .generer_url_ecrire($ret, $gra, true, true);
 
-	return generer_action_auteur($action, $arg, $r, $mode, $atts, $public);
+    return generer_action_auteur($action, $arg, $r, $mode, $atts, $public);
 }
 
 // http://doc.spip.org/@redirige_action_post
 function redirige_action_post($action, $arg, $ret, $gra, $corps, $att = ''){
-	$r = _DIR_RESTREINT.generer_url_ecrire($ret, $gra, false, true);
-	return generer_action_auteur($action, $arg, $r, $corps, $att." method='post'");
+    $r = _DIR_RESTREINT.generer_url_ecrire($ret, $gra, false, true);
+    return generer_action_auteur($action, $arg, $r, $corps, $att." method='post'");
 }
 
 
@@ -63,34 +63,34 @@ 
  *   passer "text/html" comme $content_type
  */
 function ajax_retour($corps, $content_type = null){
-	$xml = false;
-	if (is_null($content_type) OR $content_type===true){
-		$xml = true;
-		$content_type = 'text/html';
-	}
-	elseif (!$content_type OR !is_string($content_type) OR strpos($content_type,'/')===false) {
-		$content_type = 'text/html';
-	}
+    $xml = false;
+    if (is_null($content_type) OR $content_type===true){
+        $xml = true;
+        $content_type = 'text/html';
+    }
+    elseif (!$content_type OR !is_string($content_type) OR strpos($content_type,'/')===false) {
+        $content_type = 'text/html';
+    }
 	
-	$e = "";
-	if (isset($_COOKIE['spip_admin'])
-	    AND ((_request('var_mode')=='debug') OR !empty($GLOBALS['tableau_des_temps'])))
-		$e = erreur_squelette();
-	if (isset($GLOBALS['transformer_xml']) OR (isset($GLOBALS['exec']) AND $GLOBALS['exec']=='valider_xml')){
-		$debut = _DOCTYPE_ECRIRE
-		         ."<html><head><title>Debug Spip Ajax</title></head>"
-		         ."<body><div>\n\n"
-		         ."<!-- %%%%%%%%%%%%%%%%%%% Ajax %%%%%%%%%%%%%%%%%%% -->\n";
+    $e = "";
+    if (isset($_COOKIE['spip_admin'])
+        AND ((_request('var_mode')=='debug') OR !empty($GLOBALS['tableau_des_temps'])))
+        $e = erreur_squelette();
+    if (isset($GLOBALS['transformer_xml']) OR (isset($GLOBALS['exec']) AND $GLOBALS['exec']=='valider_xml')){
+        $debut = _DOCTYPE_ECRIRE
+                    ."<html><head><title>Debug Spip Ajax</title></head>"
+                    ."<body><div>\n\n"
+                    ."<!-- %%%%%%%%%%%%%%%%%%% Ajax %%%%%%%%%%%%%%%%%%% -->\n";
 
-		$fin = '</div></body></html>';
+        $fin = '</div></body></html>';
 
-	} else {
-		$c = $GLOBALS['meta']["charset"];
-		header('Content-Type: '.$content_type.'; charset='.$c);
-		$debut = (($xml AND strlen(trim($corps))) ? '<'."?xml version='1.0' encoding='".$c."'?".">\n" : '');
-		$fin = "";
-	}
-	echo $debut, $corps, $fin, $e;
+    } else {
+        $c = $GLOBALS['meta']["charset"];
+        header('Content-Type: '.$content_type.'; charset='.$c);
+        $debut = (($xml AND strlen(trim($corps))) ? '<'."?xml version='1.0' encoding='".$c."'?".">\n" : '');
+        $fin = "";
+    }
+    echo $debut, $corps, $fin, $e;
 }
 
 ?>

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
  * @param bool $public
  * @return array|string
  */
-function generer_action_auteur($action, $arg, $redirect = "", $mode = false, $att = '', $public = false){
+function generer_action_auteur($action, $arg, $redirect = "", $mode = false, $att = '', $public = false) {
 	$securiser_action = charger_fonction('securiser_action', 'inc');
 	return $securiser_action($action, $arg, $redirect, $mode, $att, $public);
 }
@@ -38,13 +38,13 @@ 
 // http://doc.spip.org/@redirige_action_auteur
 function redirige_action_auteur($action, $arg, $ret, $gra = '', $mode = false, $atts = '', $public = false) {
 
-	$r = ($public ? _DIR_RESTREINT_ABS : _DIR_RESTREINT) .generer_url_ecrire($ret, $gra, true, true);
+	$r = ($public ? _DIR_RESTREINT_ABS : _DIR_RESTREINT).generer_url_ecrire($ret, $gra, true, true);
 
 	return generer_action_auteur($action, $arg, $r, $mode, $atts, $public);
 }
 
 // http://doc.spip.org/@redirige_action_post
-function redirige_action_post($action, $arg, $ret, $gra, $corps, $att = ''){
+function redirige_action_post($action, $arg, $ret, $gra, $corps, $att = '') {
 	$r = _DIR_RESTREINT.generer_url_ecrire($ret, $gra, false, true);
 	return generer_action_auteur($action, $arg, $r, $corps, $att." method='post'");
 }
@@ -62,21 +62,21 @@ 
  *   La valeur speciale false fournit text/html sans entete xml. Elle equivaut a
  *   passer "text/html" comme $content_type
  */
-function ajax_retour($corps, $content_type = null){
+function ajax_retour($corps, $content_type = null) {
 	$xml = false;
-	if (is_null($content_type) OR $content_type===true){
+	if (is_null($content_type) OR $content_type === true) {
 		$xml = true;
 		$content_type = 'text/html';
 	}
-	elseif (!$content_type OR !is_string($content_type) OR strpos($content_type,'/')===false) {
+	elseif (!$content_type OR !is_string($content_type) OR strpos($content_type, '/') === false) {
 		$content_type = 'text/html';
 	}
 	
 	$e = "";
 	if (isset($_COOKIE['spip_admin'])
-	    AND ((_request('var_mode')=='debug') OR !empty($GLOBALS['tableau_des_temps'])))
+	    AND ((_request('var_mode') == 'debug') OR !empty($GLOBALS['tableau_des_temps'])))
 		$e = erreur_squelette();
-	if (isset($GLOBALS['transformer_xml']) OR (isset($GLOBALS['exec']) AND $GLOBALS['exec']=='valider_xml')){
+	if (isset($GLOBALS['transformer_xml']) OR (isset($GLOBALS['exec']) AND $GLOBALS['exec'] == 'valider_xml')) {
 		$debut = _DOCTYPE_ECRIRE
 		         ."<html><head><title>Debug Spip Ajax</title></head>"
 		         ."<body><div>\n\n"

Braces +7 added lines, -5 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * retourne une URL ou un formulaire securises
@@ -67,15 +69,15 @@ 
 	if (is_null($content_type) OR $content_type===true){
 		$xml = true;
 		$content_type = 'text/html';
-	}
-	elseif (!$content_type OR !is_string($content_type) OR strpos($content_type,'/')===false) {
+	} elseif (!$content_type OR !is_string($content_type) OR strpos($content_type,'/')===false) {
 		$content_type = 'text/html';
 	}
 	
 	$e = "";
 	if (isset($_COOKIE['spip_admin'])
-	    AND ((_request('var_mode')=='debug') OR !empty($GLOBALS['tableau_des_temps'])))
-		$e = erreur_squelette();
+	    AND ((_request('var_mode')=='debug') OR !empty($GLOBALS['tableau_des_temps']))) {
+			$e = erreur_squelette();
+	}
 	if (isset($GLOBALS['transformer_xml']) OR (isset($GLOBALS['exec']) AND $GLOBALS['exec']=='valider_xml')){
 		$debut = _DOCTYPE_ECRIRE
 		         ."<html><head><title>Debug Spip Ajax</title></head>"

ecrire/inc/aider.php

Doc Comments +6 added lines

@@ -92,6 +92,9 @@ 
 	return aider_icone($url);
 }
 
+/**
+ * @param null|string $url
+ */
 function aider_icone($url, $clic= '')
 {
 	global $spip_lang, $spip_lang_rtl;
@@ -118,6 +121,9 @@ 
 // Les sections d'un fichier aide sont reperees ainsi:
 define('_SECTIONS_AIDE', ',<h([12])(?:\s+class="spip")?'. '>([^/]+?)(?:/(.+?))?</h\1>,ism');
 
+/**
+ * @param string $path
+ */
 function aide_fichier($path, $help_server) {
 
 	$md5 = md5(serialize($help_server));

Indentation +152 added lines, -152 removed lines

@@ -22,34 +22,34 @@ 
 // Le tableau ci-dessous donne le repere correspondant a ces informations.
 
 $GLOBALS['aider_index'] = array(
-	'editer_article.html' => array (
-		'surtitre' => 'arttitre',
-		'titre' => 'arttitre',
-		'soustitre' => 'arttitre',
-		'id_parent' => 'artrub',
-		'descriptif' => 'artdesc',
-		'virtuel' => 'artvirt',
-		'chapo' => 'arttitre',
-		'text_area' => 'arttexte'),
+    'editer_article.html' => array (
+        'surtitre' => 'arttitre',
+        'titre' => 'arttitre',
+        'soustitre' => 'arttitre',
+        'id_parent' => 'artrub',
+        'descriptif' => 'artdesc',
+        'virtuel' => 'artvirt',
+        'chapo' => 'arttitre',
+        'text_area' => 'arttexte'),
 
-	'editer_breve.html' => array(
-		'id_parent' => 'brevesrub',
-		'lien_titre' => 'breveslien',
-		'statut' => 'brevesstatut'),
+    'editer_breve.html' => array(
+        'id_parent' => 'brevesrub',
+        'lien_titre' => 'breveslien',
+        'statut' => 'brevesstatut'),
 
-	'editer_groupe_mot.html' => array(
-		'titre' => 'motsgroupes'),
+    'editer_groupe_mot.html' => array(
+        'titre' => 'motsgroupes'),
 
-	'editer_mot.html' => array(
-		'titre' => 'mots',
-		'id_groupe' => 'motsgroupes'),
+    'editer_mot.html' => array(
+        'titre' => 'mots',
+        'id_groupe' => 'motsgroupes'),
 
-	'editer_rubrique.html' => array(
-		'titre' => 'arttitre',
-		'id_parent' => 'rubrub',
-		'text_area' => 'raccourcis')
+    'editer_rubrique.html' => array(
+        'titre' => 'arttitre',
+        'id_parent' => 'rubrub',
+        'text_area' => 'raccourcis')
 
-				);
+                );
 
 
 
@@ -68,51 +68,51 @@ 
  * @return 
 **/
 function inc_aider_dist($aide='', $skel='', $env=array(), $aide_spip_directe = false) {
-	global $spip_lang, $aider_index;
-
-	if (($skel = basename($skel))
-	AND isset($aider_index[$skel])
-	AND isset($aider_index[$skel][$aide]))
-		$aide = $aider_index[$skel][$aide];
-
-	if ($aide_spip_directe) {
-		// on suppose que spip.net est le premier present
-		// dans la liste des serveurs. C'est forcement le cas
-		// a l'installation tout du moins
-		$help_server = $GLOBALS['help_server'];
-		$url = array_shift($help_server) . '/';
-		$url = parametre_url($url, 'exec', 'aide');
-		$url = parametre_url($url, 'aide', $aide);
-		$url = parametre_url($url, 'var_lang', $spip_lang);
-	} else {
-		$args = "aide=$aide&var_lang=$spip_lang";
-		$url = generer_url_ecrire("aide", $args);
-	}
+    global $spip_lang, $aider_index;
+
+    if (($skel = basename($skel))
+    AND isset($aider_index[$skel])
+    AND isset($aider_index[$skel][$aide]))
+        $aide = $aider_index[$skel][$aide];
+
+    if ($aide_spip_directe) {
+        // on suppose que spip.net est le premier present
+        // dans la liste des serveurs. C'est forcement le cas
+        // a l'installation tout du moins
+        $help_server = $GLOBALS['help_server'];
+        $url = array_shift($help_server) . '/';
+        $url = parametre_url($url, 'exec', 'aide');
+        $url = parametre_url($url, 'aide', $aide);
+        $url = parametre_url($url, 'var_lang', $spip_lang);
+    } else {
+        $args = "aide=$aide&var_lang=$spip_lang";
+        $url = generer_url_ecrire("aide", $args);
+    }
 	
-	return aider_icone($url);
+    return aider_icone($url);
 }
 
 function aider_icone($url, $clic= '')
 {
-	global $spip_lang, $spip_lang_rtl;
-
-	if (!$clic) {
-		$t = _T('titre_image_aide');
-		$clic = http_img_pack("aide".aide_lang_dir($spip_lang,$spip_lang_rtl)."-16.png",
-			_T('info_image_aide'),
-			" title=\"$t\" class='aide'");
-	}
-	return "\n&nbsp;&nbsp;<a class='aide popin'\nhref='"
-	.  $url
-	. "' target='_blank'>"
-	. $clic
-	. "</a>";
+    global $spip_lang, $spip_lang_rtl;
+
+    if (!$clic) {
+        $t = _T('titre_image_aide');
+        $clic = http_img_pack("aide".aide_lang_dir($spip_lang,$spip_lang_rtl)."-16.png",
+            _T('info_image_aide'),
+            " title=\"$t\" class='aide'");
+    }
+    return "\n&nbsp;&nbsp;<a class='aide popin'\nhref='"
+    .  $url
+    . "' target='_blank'>"
+    . $clic
+    . "</a>";
 }
 
 // en hebreu le ? ne doit pas etre inverse
 // http://doc.spip.org/@aide_lang_dir
 function aide_lang_dir($spip_lang,$spip_lang_rtl) {
-	return ($spip_lang<>'he') ? $spip_lang_rtl : '';
+    return ($spip_lang<>'he') ? $spip_lang_rtl : '';
 }
 
 // Les sections d'un fichier aide sont reperees ainsi:
@@ -120,85 +120,85 @@ 
 
 function aide_fichier($path, $help_server) {
 
-	$md5 = md5(serialize($help_server));
-	$fichier_aide = _DIR_AIDE . substr($md5,0,16) . "-" . $path;
-	$lastm = @filemtime($fichier_aide);
-	$lastversion = @filemtime(_DIR_RESTREINT . 'inc_version.php');
-	$here = @(is_readable($fichier_aide) AND ($lastm >= $lastversion));
-	$contenu = '';
-
-	if ($here) {
-		lire_fichier($fichier_aide, $contenu);
-		return array($contenu, $lastm);
-	}
-
-	// mettre en cache (tant pis si echec)
-	sous_repertoire(_DIR_AIDE,'','',true);
-	$contenu = array();
-	include_spip('inc/distant');
-	foreach ($help_server as $k => $server) {
-		// Remplacer les liens aux images par leur gestionnaire de cache
-		$url = "$server/$path";
-		$local = _DIR_AIDE.substr(md5($url),0,8)."-".preg_replace(",[^\w.]+,i","_",$url);
-		$local = _DIR_RACINE . copie_locale($url, 'modif', $local);
-
-		lire_fichier($local,$page);
-		$page = aide_fixe_img($page,$server);
-		// les liens internes ne doivent pas etre deguises en externes
-		$url = parse_url($url);
-		$re = '@(<a\b[^>]*\s+href=["\'])' .
-		  '(?:' . $url['scheme'] . '://' . $url['host'] . ')?' .
-		  $url['path'] . '([^"\']*)@ims';
-		$page = preg_replace($re,'\\1\\2', $page);
-
-		preg_match_all(_SECTIONS_AIDE, $page, $sections, PREG_SET_ORDER);
-		// Fusionner les aides ayant meme nom de section
-		$vus = array();
-		foreach ($sections as $section) {
-			list($tout,$prof, $sujet,) = $section;
-			if (in_array($sujet, $vus)) continue;
-			$corps = aide_section($sujet, $page, $prof);
-			foreach ($contenu as $k => $s) {
-			  if ($sujet == $k) {
-			    // Section deja vue qu'il faut completer
-			    // Si le complement a des sous-sections,
-			    // ne pas en tenir compte quand on les rencontrera
-			    // lors des prochains passages dans la boucle
-			    preg_match_all(_SECTIONS_AIDE, $corps, $s, PREG_PATTERN_ORDER);
-			    if ($s) {$vus = array_merge($vus, $s[2]);}
-			    $contenu[$k] .= $corps;
-			    $corps = '';
-			    break;
-			  }
-			}
-			// Si totalement nouveau, inserer le titre
-			// mais pas le corps s'il contient des sous-sections:
-			// elles vont venir dans les passages suivants
-			if ($corps) {
-			  $corps = aide_section($sujet, $page);
-			  $contenu[$sujet] = $tout . "\n" . $corps;
-			}
-		}
-	}
-
-	$contenu = '<div>' . join('',$contenu) . '</div>';
-
-	// Renvoyer les liens vraiment externes dans une autre fenetre
-	$contenu = preg_replace('@<a href="(http://[^"]+)"([^>]*)>@',
-				'<a href="\\1"\\2 target="_blank">',
-				$contenu);
-
-	// Correction typo dans la langue demandee
-	#changer_typo($lang_aide);
-	$contenu = '<body>' . $contenu . '</body>';
-
-	if (strlen($contenu) <= 100) return array(false, false);
-	ecrire_fichier ($fichier_aide, $contenu);
-	return array($contenu, time());
+    $md5 = md5(serialize($help_server));
+    $fichier_aide = _DIR_AIDE . substr($md5,0,16) . "-" . $path;
+    $lastm = @filemtime($fichier_aide);
+    $lastversion = @filemtime(_DIR_RESTREINT . 'inc_version.php');
+    $here = @(is_readable($fichier_aide) AND ($lastm >= $lastversion));
+    $contenu = '';
+
+    if ($here) {
+        lire_fichier($fichier_aide, $contenu);
+        return array($contenu, $lastm);
+    }
+
+    // mettre en cache (tant pis si echec)
+    sous_repertoire(_DIR_AIDE,'','',true);
+    $contenu = array();
+    include_spip('inc/distant');
+    foreach ($help_server as $k => $server) {
+        // Remplacer les liens aux images par leur gestionnaire de cache
+        $url = "$server/$path";
+        $local = _DIR_AIDE.substr(md5($url),0,8)."-".preg_replace(",[^\w.]+,i","_",$url);
+        $local = _DIR_RACINE . copie_locale($url, 'modif', $local);
+
+        lire_fichier($local,$page);
+        $page = aide_fixe_img($page,$server);
+        // les liens internes ne doivent pas etre deguises en externes
+        $url = parse_url($url);
+        $re = '@(<a\b[^>]*\s+href=["\'])' .
+            '(?:' . $url['scheme'] . '://' . $url['host'] . ')?' .
+            $url['path'] . '([^"\']*)@ims';
+        $page = preg_replace($re,'\\1\\2', $page);
+
+        preg_match_all(_SECTIONS_AIDE, $page, $sections, PREG_SET_ORDER);
+        // Fusionner les aides ayant meme nom de section
+        $vus = array();
+        foreach ($sections as $section) {
+            list($tout,$prof, $sujet,) = $section;
+            if (in_array($sujet, $vus)) continue;
+            $corps = aide_section($sujet, $page, $prof);
+            foreach ($contenu as $k => $s) {
+                if ($sujet == $k) {
+                // Section deja vue qu'il faut completer
+                // Si le complement a des sous-sections,
+                // ne pas en tenir compte quand on les rencontrera
+                // lors des prochains passages dans la boucle
+                preg_match_all(_SECTIONS_AIDE, $corps, $s, PREG_PATTERN_ORDER);
+                if ($s) {$vus = array_merge($vus, $s[2]);}
+                $contenu[$k] .= $corps;
+                $corps = '';
+                break;
+                }
+            }
+            // Si totalement nouveau, inserer le titre
+            // mais pas le corps s'il contient des sous-sections:
+            // elles vont venir dans les passages suivants
+            if ($corps) {
+                $corps = aide_section($sujet, $page);
+                $contenu[$sujet] = $tout . "\n" . $corps;
+            }
+        }
+    }
+
+    $contenu = '<div>' . join('',$contenu) . '</div>';
+
+    // Renvoyer les liens vraiment externes dans une autre fenetre
+    $contenu = preg_replace('@<a href="(http://[^"]+)"([^>]*)>@',
+                '<a href="\\1"\\2 target="_blank">',
+                $contenu);
+
+    // Correction typo dans la langue demandee
+    #changer_typo($lang_aide);
+    $contenu = '<body>' . $contenu . '</body>';
+
+    if (strlen($contenu) <= 100) return array(false, false);
+    ecrire_fichier ($fichier_aide, $contenu);
+    return array($contenu, time());
 }
 
 function generer_url_aide_img($args){
-	return generer_url_action('aide_img', $args, false, true);
+    return generer_url_action('aide_img', $args, false, true);
 }
 
 
@@ -208,18 +208,18 @@ 
 
 // Remplacer les URL des images par l'URL du gestionnaire de cache local
 function aide_fixe_img($contenu, $server){
-	$html = "";
-	$re = "@(<img([^<>]* +)?\s*src=['\"])((AIDE|IMG|local)/([-_a-zA-Z0-9]*/?)([^'\"<>]*))@imsS";
-	while (preg_match($re, $contenu, $r)) {
-		$p = strpos($contenu, $r[0]);
-		$i = $server . '/' . $r[3];
-		$html .= substr($contenu, 0, $p) .  $r[1] . $i;
-		$contenu = substr($contenu, $p + strlen($r[0]));
-	}
-	$html .= $contenu;
-
-	// traiter les vieilles doc
-	return  preg_replace(_REPLACE_IMG_PACK,"\\1"._DIR_IMG_PACK, $html);
+    $html = "";
+    $re = "@(<img([^<>]* +)?\s*src=['\"])((AIDE|IMG|local)/([-_a-zA-Z0-9]*/?)([^'\"<>]*))@imsS";
+    while (preg_match($re, $contenu, $r)) {
+        $p = strpos($contenu, $r[0]);
+        $i = $server . '/' . $r[3];
+        $html .= substr($contenu, 0, $p) .  $r[1] . $i;
+        $contenu = substr($contenu, $p + strlen($r[0]));
+    }
+    $html .= $contenu;
+
+    // traiter les vieilles doc
+    return  preg_replace(_REPLACE_IMG_PACK,"\\1"._DIR_IMG_PACK, $html);
 }
 
 
@@ -228,14 +228,14 @@ 
 // et se termine par la prochaine balise h2 ou h1 ou le /body final.
 
 function aide_section($aide, $contenu, $prof=2){
-	$maxprof = ($prof >=2) ? "12" : "1";
-	$r = "@<h$prof" . '(?: class="spip")?' . '>\s*' . $aide
-	  ."\s*(?:/.+?)?</h$prof>(.*?)<(?:(?:h[$maxprof])|/body)@ism";
+    $maxprof = ($prof >=2) ? "12" : "1";
+    $r = "@<h$prof" . '(?: class="spip")?' . '>\s*' . $aide
+        ."\s*(?:/.+?)?</h$prof>(.*?)<(?:(?:h[$maxprof])|/body)@ism";
 
-	if (preg_match($r, $contenu, $m))
-	  return $m[1];
+    if (preg_match($r, $contenu, $m))
+        return $m[1];
 #	spip_log("aide inconnue $r dans " . substr($contenu, 0, 150));
-	return '';
+    return '';
 }
 
 

Spacing +33 added lines, -33 removed lines

@@ -22,7 +22,7 @@ 
 // Le tableau ci-dessous donne le repere correspondant a ces informations.
 
 $GLOBALS['aider_index'] = array(
-	'editer_article.html' => array (
+	'editer_article.html' => array(
 		'surtitre' => 'arttitre',
 		'titre' => 'arttitre',
 		'soustitre' => 'arttitre',
@@ -67,7 +67,7 @@ 
  * 		true : Le lien est realise sur spip.net/aide/ directement ...
  * @return 
 **/
-function inc_aider_dist($aide='', $skel='', $env=array(), $aide_spip_directe = false) {
+function inc_aider_dist($aide = '', $skel = '', $env = array(), $aide_spip_directe = false) {
 	global $spip_lang, $aider_index;
 
 	if (($skel = basename($skel))
@@ -80,7 +80,7 @@ 
 		// dans la liste des serveurs. C'est forcement le cas
 		// a l'installation tout du moins
 		$help_server = $GLOBALS['help_server'];
-		$url = array_shift($help_server) . '/';
+		$url = array_shift($help_server).'/';
 		$url = parametre_url($url, 'exec', 'aide');
 		$url = parametre_url($url, 'aide', $aide);
 		$url = parametre_url($url, 'var_lang', $spip_lang);
@@ -92,13 +92,13 @@ 
 	return aider_icone($url);
 }
 
-function aider_icone($url, $clic= '')
+function aider_icone($url, $clic = '')
 {
 	global $spip_lang, $spip_lang_rtl;
 
 	if (!$clic) {
 		$t = _T('titre_image_aide');
-		$clic = http_img_pack("aide".aide_lang_dir($spip_lang,$spip_lang_rtl)."-16.png",
+		$clic = http_img_pack("aide".aide_lang_dir($spip_lang, $spip_lang_rtl)."-16.png",
 			_T('info_image_aide'),
 			" title=\"$t\" class='aide'");
 	}
@@ -111,19 +111,19 @@ 
 
 // en hebreu le ? ne doit pas etre inverse
 // http://doc.spip.org/@aide_lang_dir
-function aide_lang_dir($spip_lang,$spip_lang_rtl) {
-	return ($spip_lang<>'he') ? $spip_lang_rtl : '';
+function aide_lang_dir($spip_lang, $spip_lang_rtl) {
+	return ($spip_lang <> 'he') ? $spip_lang_rtl : '';
 }
 
 // Les sections d'un fichier aide sont reperees ainsi:
-define('_SECTIONS_AIDE', ',<h([12])(?:\s+class="spip")?'. '>([^/]+?)(?:/(.+?))?</h\1>,ism');
+define('_SECTIONS_AIDE', ',<h([12])(?:\s+class="spip")?'.'>([^/]+?)(?:/(.+?))?</h\1>,ism');
 
 function aide_fichier($path, $help_server) {
 
 	$md5 = md5(serialize($help_server));
-	$fichier_aide = _DIR_AIDE . substr($md5,0,16) . "-" . $path;
+	$fichier_aide = _DIR_AIDE.substr($md5, 0, 16)."-".$path;
 	$lastm = @filemtime($fichier_aide);
-	$lastversion = @filemtime(_DIR_RESTREINT . 'inc_version.php');
+	$lastversion = @filemtime(_DIR_RESTREINT.'inc_version.php');
 	$here = @(is_readable($fichier_aide) AND ($lastm >= $lastversion));
 	$contenu = '';
 
@@ -133,29 +133,29 @@ 
 	}
 
 	// mettre en cache (tant pis si echec)
-	sous_repertoire(_DIR_AIDE,'','',true);
+	sous_repertoire(_DIR_AIDE, '', '', true);
 	$contenu = array();
 	include_spip('inc/distant');
 	foreach ($help_server as $k => $server) {
 		// Remplacer les liens aux images par leur gestionnaire de cache
 		$url = "$server/$path";
-		$local = _DIR_AIDE.substr(md5($url),0,8)."-".preg_replace(",[^\w.]+,i","_",$url);
-		$local = _DIR_RACINE . copie_locale($url, 'modif', $local);
+		$local = _DIR_AIDE.substr(md5($url), 0, 8)."-".preg_replace(",[^\w.]+,i", "_", $url);
+		$local = _DIR_RACINE.copie_locale($url, 'modif', $local);
 
-		lire_fichier($local,$page);
-		$page = aide_fixe_img($page,$server);
+		lire_fichier($local, $page);
+		$page = aide_fixe_img($page, $server);
 		// les liens internes ne doivent pas etre deguises en externes
 		$url = parse_url($url);
-		$re = '@(<a\b[^>]*\s+href=["\'])' .
-		  '(?:' . $url['scheme'] . '://' . $url['host'] . ')?' .
-		  $url['path'] . '([^"\']*)@ims';
-		$page = preg_replace($re,'\\1\\2', $page);
+		$re = '@(<a\b[^>]*\s+href=["\'])'.
+		  '(?:'.$url['scheme'].'://'.$url['host'].')?'.
+		  $url['path'].'([^"\']*)@ims';
+		$page = preg_replace($re, '\\1\\2', $page);
 
 		preg_match_all(_SECTIONS_AIDE, $page, $sections, PREG_SET_ORDER);
 		// Fusionner les aides ayant meme nom de section
 		$vus = array();
 		foreach ($sections as $section) {
-			list($tout,$prof, $sujet,) = $section;
+			list($tout, $prof, $sujet,) = $section;
 			if (in_array($sujet, $vus)) continue;
 			$corps = aide_section($sujet, $page, $prof);
 			foreach ($contenu as $k => $s) {
@@ -165,7 +165,7 @@ 
 			    // ne pas en tenir compte quand on les rencontrera
 			    // lors des prochains passages dans la boucle
 			    preg_match_all(_SECTIONS_AIDE, $corps, $s, PREG_PATTERN_ORDER);
-			    if ($s) {$vus = array_merge($vus, $s[2]);}
+			    if ($s) {$vus = array_merge($vus, $s[2]); }
 			    $contenu[$k] .= $corps;
 			    $corps = '';
 			    break;
@@ -176,12 +176,12 @@ 
 			// elles vont venir dans les passages suivants
 			if ($corps) {
 			  $corps = aide_section($sujet, $page);
-			  $contenu[$sujet] = $tout . "\n" . $corps;
+			  $contenu[$sujet] = $tout."\n".$corps;
 			}
 		}
 	}
 
-	$contenu = '<div>' . join('',$contenu) . '</div>';
+	$contenu = '<div>'.join('', $contenu).'</div>';
 
 	// Renvoyer les liens vraiment externes dans une autre fenetre
 	$contenu = preg_replace('@<a href="(http://[^"]+)"([^>]*)>@',
@@ -190,14 +190,14 @@ 
 
 	// Correction typo dans la langue demandee
 	#changer_typo($lang_aide);
-	$contenu = '<body>' . $contenu . '</body>';
+	$contenu = '<body>'.$contenu.'</body>';
 
 	if (strlen($contenu) <= 100) return array(false, false);
-	ecrire_fichier ($fichier_aide, $contenu);
+	ecrire_fichier($fichier_aide, $contenu);
 	return array($contenu, time());
 }
 
-function generer_url_aide_img($args){
+function generer_url_aide_img($args) {
 	return generer_url_action('aide_img', $args, false, true);
 }
 
@@ -207,19 +207,19 @@ 
 define('_REPLACE_IMG_PACK', "@(<img([^<>]* +)?\s*src=['\"])img_pack\/@ims");
 
 // Remplacer les URL des images par l'URL du gestionnaire de cache local
-function aide_fixe_img($contenu, $server){
+function aide_fixe_img($contenu, $server) {
 	$html = "";
 	$re = "@(<img([^<>]* +)?\s*src=['\"])((AIDE|IMG|local)/([-_a-zA-Z0-9]*/?)([^'\"<>]*))@imsS";
 	while (preg_match($re, $contenu, $r)) {
 		$p = strpos($contenu, $r[0]);
-		$i = $server . '/' . $r[3];
-		$html .= substr($contenu, 0, $p) .  $r[1] . $i;
+		$i = $server.'/'.$r[3];
+		$html .= substr($contenu, 0, $p).$r[1].$i;
 		$contenu = substr($contenu, $p + strlen($r[0]));
 	}
 	$html .= $contenu;
 
 	// traiter les vieilles doc
-	return  preg_replace(_REPLACE_IMG_PACK,"\\1"._DIR_IMG_PACK, $html);
+	return  preg_replace(_REPLACE_IMG_PACK, "\\1"._DIR_IMG_PACK, $html);
 }
 
 
@@ -227,9 +227,9 @@ 
 // qui commence par son nom entouree d'une balise h2
 // et se termine par la prochaine balise h2 ou h1 ou le /body final.
 
-function aide_section($aide, $contenu, $prof=2){
-	$maxprof = ($prof >=2) ? "12" : "1";
-	$r = "@<h$prof" . '(?: class="spip")?' . '>\s*' . $aide
+function aide_section($aide, $contenu, $prof = 2) {
+	$maxprof = ($prof >= 2) ? "12" : "1";
+	$r = "@<h$prof".'(?: class="spip")?'.'>\s*'.$aide
 	  ."\s*(?:/.+?)?</h$prof>(.*?)<(?:(?:h[$maxprof])|/body)@ism";
 
 	if (preg_match($r, $contenu, $m))

Braces +16 added lines, -8 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/filtres');
 
@@ -72,8 +74,9 @@ 
 
 	if (($skel = basename($skel))
 	AND isset($aider_index[$skel])
-	AND isset($aider_index[$skel][$aide]))
-		$aide = $aider_index[$skel][$aide];
+	AND isset($aider_index[$skel][$aide])) {
+			$aide = $aider_index[$skel][$aide];
+	}
 
 	if ($aide_spip_directe) {
 		// on suppose que spip.net est le premier present
@@ -156,7 +159,9 @@ 
 		$vus = array();
 		foreach ($sections as $section) {
 			list($tout,$prof, $sujet,) = $section;
-			if (in_array($sujet, $vus)) continue;
+			if (in_array($sujet, $vus)) {
+			    continue;
+			}
 			$corps = aide_section($sujet, $page, $prof);
 			foreach ($contenu as $k => $s) {
 			  if ($sujet == $k) {
@@ -192,7 +197,9 @@ 
 	#changer_typo($lang_aide);
 	$contenu = '<body>' . $contenu . '</body>';
 
-	if (strlen($contenu) <= 100) return array(false, false);
+	if (strlen($contenu) <= 100) {
+	    return array(false, false);
+	}
 	ecrire_fichier ($fichier_aide, $contenu);
 	return array($contenu, time());
 }
@@ -232,9 +239,10 @@ 
 	$r = "@<h$prof" . '(?: class="spip")?' . '>\s*' . $aide
 	  ."\s*(?:/.+?)?</h$prof>(.*?)<(?:(?:h[$maxprof])|/body)@ism";
 
-	if (preg_match($r, $contenu, $m))
-	  return $m[1];
-#	spip_log("aide inconnue $r dans " . substr($contenu, 0, 150));
+	if (preg_match($r, $contenu, $m)) {
+		  return $m[1];
+	}
+	#	spip_log("aide inconnue $r dans " . substr($contenu, 0, 150));
 	return '';
 }
 

ecrire/inc/auth.php

Doc Comments +4 added lines

@@ -448,6 +448,10 @@
 	return $securiser_action('auth', "$auth_methode/$login", $redirect, true);
 }
 
+/**
+ * @param string $auth_methode
+ * @param string $login
+ */
 function auth_terminer_identifier_login($auth_methode, $login, $serveur=''){
 	$args = func_get_args();
 	$auteur = auth_administrer('terminer_identifier_login',$args);

Indentation +369 added lines, -369 removed lines

@@ -32,27 +32,27 @@ 
  */
 function inc_auth_dist() {
 
-	global $connect_login ;
+    global $connect_login ;
 
-	$row = auth_mode();
+    $row = auth_mode();
 
-	if ($row) return auth_init_droits($row);
+    if ($row) return auth_init_droits($row);
 
-	if (!$connect_login) return auth_a_loger();
+    if (!$connect_login) return auth_a_loger();
 
-	// Cas ou l'auteur a ete identifie mais on n'a pas d'info sur lui
-	// C'est soit parce que la base est inutilisable,
-	// soit parce que la table des auteurs a changee (restauration etc)
-	// Pas la peine d'insister.
-	// Renvoyer le nom fautif et une URL de remise a zero
+    // Cas ou l'auteur a ete identifie mais on n'a pas d'info sur lui
+    // C'est soit parce que la base est inutilisable,
+    // soit parce que la table des auteurs a changee (restauration etc)
+    // Pas la peine d'insister.
+    // Renvoyer le nom fautif et une URL de remise a zero
 
-	if (spip_connect())
-		return array('login' => $connect_login,
-			'site' => generer_url_public('', "action=logout&logout=prive"));
+    if (spip_connect())
+        return array('login' => $connect_login,
+            'site' => generer_url_public('', "action=logout&logout=prive"));
 
-	$n = intval(sql_errno());
-	spip_log("Erreur base de donnees $n " . sql_error());
-	return $n ? $n : 1;
+    $n = intval(sql_errno());
+    spip_log("Erreur base de donnees $n " . sql_error());
+    return $n ? $n : 1;
 }
 
 /**
@@ -65,34 +65,34 @@ 
  */
 function auth_echec($raison)
 {
-	include_spip('inc/minipres');
-	include_spip('inc/headers');
-	// pas authentifie. Pourquoi ?
-	if (is_string($raison)) {
-		// redirection vers une page d'authentification
-		// on ne revient pas de cette fonction
-		// sauf si pb de header
-		$raison = redirige_formulaire($raison);
-	} elseif (is_int($raison)) {
-		// erreur SQL a afficher
-		$raison = minipres(_T('info_travaux_titre'), _T('titre_probleme_technique'). "<p><tt>".sql_errno()." ".sql_error()."</tt></p>");
-	} elseif (@$raison['statut']) {
-		// un simple visiteur n'a pas acces a l'espace prive
-		spip_log("connexion refusee a " . @$raison['id_auteur']);
-		$raison = minipres(_T('avis_erreur_connexion'),_T('avis_erreur_visiteur'));
-	} else {
-		// auteur en fin de droits ...
-		$h = $raison['site'];
-		$raison = minipres(_T('avis_erreur_connexion'),
-				"<br /><br /><p>"
-				. _T('texte_inc_auth_1',
-				array('auth_login' => $raison['login']))
-				. " <a href='$h'>"
-				.  _T('texte_inc_auth_2')
-				. "</a>"
-				. _T('texte_inc_auth_3'));
-	}
-	return $raison;
+    include_spip('inc/minipres');
+    include_spip('inc/headers');
+    // pas authentifie. Pourquoi ?
+    if (is_string($raison)) {
+        // redirection vers une page d'authentification
+        // on ne revient pas de cette fonction
+        // sauf si pb de header
+        $raison = redirige_formulaire($raison);
+    } elseif (is_int($raison)) {
+        // erreur SQL a afficher
+        $raison = minipres(_T('info_travaux_titre'), _T('titre_probleme_technique'). "<p><tt>".sql_errno()." ".sql_error()."</tt></p>");
+    } elseif (@$raison['statut']) {
+        // un simple visiteur n'a pas acces a l'espace prive
+        spip_log("connexion refusee a " . @$raison['id_auteur']);
+        $raison = minipres(_T('avis_erreur_connexion'),_T('avis_erreur_visiteur'));
+    } else {
+        // auteur en fin de droits ...
+        $h = $raison['site'];
+        $raison = minipres(_T('avis_erreur_connexion'),
+                "<br /><br /><p>"
+                . _T('texte_inc_auth_1',
+                array('auth_login' => $raison['login']))
+                . " <a href='$h'>"
+                .  _T('texte_inc_auth_2')
+                . "</a>"
+                . _T('texte_inc_auth_3'));
+    }
+    return $raison;
 }
 
 /**
@@ -103,78 +103,78 @@ 
  */
 function auth_mode()
 {
-	global $auth_can_disconnect, $ignore_auth_http, $ignore_remote_user;
-	global $connect_login ;
-
-	//
-	// Initialiser variables (eviter hacks par URL)
-	//
-
-	$connect_login = '';
-	$id_auteur = NULL;
-	$auth_can_disconnect = false;
-
-	//
-	// Recuperer les donnees d'identification
-	//
-
-	// Session valide en cours ?
-	if (isset($_COOKIE['spip_session'])) {
-		$session = charger_fonction('session', 'inc');
-		if ($id_auteur = $session()
-		OR $id_auteur===0 // reprise sur restauration
-		) {
-			$auth_can_disconnect = true;
-			$connect_login = $GLOBALS['visiteur_session']['login'];
-		} else unset($_COOKIE['spip_session']);
-	}
-
-	// Essayer auth http si significatif
-	// (ignorer les login d'intranet independants de spip)
-	if (!$ignore_auth_http) {
-		if (
-			(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])
-						AND $r = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']))
-			OR
-			// Si auth http differtente de basic, PHP_AUTH_PW
-			// est indisponible mais tentons quand meme pour
-			// autocreation via LDAP
-			(isset($_SERVER['REMOTE_USER'])
-						AND $r = lire_php_auth($_SERVER['PHP_AUTH_USER'] = $_SERVER['REMOTE_USER'], ''))
-			) {
-				if (!$id_auteur) {
-					$_SERVER['PHP_AUTH_PW'] = '';
-					$auth_can_disconnect = true;
-					$GLOBALS['visiteur_session'] = $r;
-					$connect_login = $GLOBALS['visiteur_session']['login'];
-					$id_auteur = $r['id_auteur'];
-				} else {
-				  // cas de la session en plus de PHP_AUTH
-				  /*				  if ($id_auteur != $r['id_auteur']){
+    global $auth_can_disconnect, $ignore_auth_http, $ignore_remote_user;
+    global $connect_login ;
+
+    //
+    // Initialiser variables (eviter hacks par URL)
+    //
+
+    $connect_login = '';
+    $id_auteur = NULL;
+    $auth_can_disconnect = false;
+
+    //
+    // Recuperer les donnees d'identification
+    //
+
+    // Session valide en cours ?
+    if (isset($_COOKIE['spip_session'])) {
+        $session = charger_fonction('session', 'inc');
+        if ($id_auteur = $session()
+        OR $id_auteur===0 // reprise sur restauration
+        ) {
+            $auth_can_disconnect = true;
+            $connect_login = $GLOBALS['visiteur_session']['login'];
+        } else unset($_COOKIE['spip_session']);
+    }
+
+    // Essayer auth http si significatif
+    // (ignorer les login d'intranet independants de spip)
+    if (!$ignore_auth_http) {
+        if (
+            (isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])
+                        AND $r = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']))
+            OR
+            // Si auth http differtente de basic, PHP_AUTH_PW
+            // est indisponible mais tentons quand meme pour
+            // autocreation via LDAP
+            (isset($_SERVER['REMOTE_USER'])
+                        AND $r = lire_php_auth($_SERVER['PHP_AUTH_USER'] = $_SERVER['REMOTE_USER'], ''))
+            ) {
+                if (!$id_auteur) {
+                    $_SERVER['PHP_AUTH_PW'] = '';
+                    $auth_can_disconnect = true;
+                    $GLOBALS['visiteur_session'] = $r;
+                    $connect_login = $GLOBALS['visiteur_session']['login'];
+                    $id_auteur = $r['id_auteur'];
+                } else {
+                    // cas de la session en plus de PHP_AUTH
+                    /*				  if ($id_auteur != $r['id_auteur']){
 				    spip_log("vol de session $id_auteur" . join(', ', $r));
 					unset($_COOKIE['spip_session']);
 					$id_auteur = '';
 					} */
-				}
-		}
-		// Authentification .htaccess old style, car .htaccess semble
-		// souvent definir *aussi* PHP_AUTH_USER et PHP_AUTH_PW
-		else if (isset($_SERVER['REMOTE_USER']))
-			$connect_login = $_SERVER['REMOTE_USER'];
-	}
-
-	$where = (is_numeric($id_auteur)
-	/*AND $id_auteur>0*/ // reprise lors des restaurations
-	) ?
-	  "id_auteur=$id_auteur" :
-	  (!strlen($connect_login) ? '' : "login=" . sql_quote($connect_login,'','text'));
-
-	if (!$where) return '';
-
-	// Trouver les autres infos dans la table auteurs.
-	// le champ 'quand' est utilise par l'agenda
-
-	return sql_fetsel("*, en_ligne AS quand", "spip_auteurs", "$where AND statut!='5poubelle'");
+                }
+        }
+        // Authentification .htaccess old style, car .htaccess semble
+        // souvent definir *aussi* PHP_AUTH_USER et PHP_AUTH_PW
+        else if (isset($_SERVER['REMOTE_USER']))
+            $connect_login = $_SERVER['REMOTE_USER'];
+    }
+
+    $where = (is_numeric($id_auteur)
+    /*AND $id_auteur>0*/ // reprise lors des restaurations
+    ) ?
+      "id_auteur=$id_auteur" :
+      (!strlen($connect_login) ? '' : "login=" . sql_quote($connect_login,'','text'));
+
+    if (!$where) return '';
+
+    // Trouver les autres infos dans la table auteurs.
+    // le champ 'quand' est utilise par l'agenda
+
+    return sql_fetsel("*, en_ligne AS quand", "spip_auteurs", "$where AND statut!='5poubelle'");
 }
 
 /**
@@ -190,73 +190,73 @@ 
  */
 function auth_init_droits($row)
 {
-	global $connect_statut, $connect_toutes_rubriques, $connect_id_rubrique, $connect_login, $connect_id_auteur;
-
-	if ($row['statut']=='nouveau'){
-		include_spip('action/inscrire_auteur');
-		$row = confirmer_statut_inscription($row);
-	}
-
-	$connect_id_auteur = $row['id_auteur'];
-	$connect_login = $row['login'];
-	$connect_statut = $row['statut'];
-
-	$GLOBALS['visiteur_session'] = array_merge((array)$GLOBALS['visiteur_session'], $row);
-
-	// au cas ou : ne pas memoriser les champs sensibles
-	unset($GLOBALS['visiteur_session']['pass']);
-	unset($GLOBALS['visiteur_session']['htpass']);
-	unset($GLOBALS['visiteur_session']['alea_actuel']);
-	unset($GLOBALS['visiteur_session']['alea_futur']);
-
-	// creer la session au besoin
-	if (!isset($_COOKIE['spip_session'])) {
-		$session = charger_fonction('session', 'inc');
-		$spip_session = $session($row);
-	}
-
-	// reinjecter les preferences_auteur apres le reset de spip_session
-	// car utilisees au retour par auth_loger()
-	$r = @unserialize($row['prefs']);
-	$GLOBALS['visiteur_session']['prefs'] = ($r ? $r : array());
-	// si prefs pas definies, les definir par defaut
-	if (!isset($GLOBALS['visiteur_session']['prefs']['couleur'])){
-		$GLOBALS['visiteur_session']['prefs']['couleur'] = 1;
-		$GLOBALS['visiteur_session']['prefs']['display'] = 2;
-		$GLOBALS['visiteur_session']['prefs']["display_navigation"] = "navigation_avec_icones";
-		$GLOBALS['visiteur_session']['prefs']["display_outils"] = "oui";
-	}
-
-	$GLOBALS['visiteur_session'] = pipeline('preparer_visiteur_session',array('args'=>array('row'=>$row),'data'=>$GLOBALS['visiteur_session']));
-
-	// Etablir les droits selon le codage attendu
-	// dans ecrire/index.php ecrire/prive.php
-
-	// Pas autorise a acceder a ecrire ? renvoyer le tableau
-	// A noter : le premier appel a autoriser() a le bon gout
-	// d'initialiser $GLOBALS['visiteur_session']['restreint'],
-	// qui ne figure pas dans le fichier de session
-	include_spip('inc/autoriser');
-
-	if (!autoriser('ecrire'))
-		return $row;
-
-	// autoriser('ecrire') ne laisse passer que les Admin et les Redac
-
-	auth_trace($row);
-
-	// Administrateurs
-	if (in_array($connect_statut, explode(',', _STATUT_AUTEUR_RUBRIQUE))) {
-		if (is_array($GLOBALS['visiteur_session']['restreint'])) {
-			$connect_id_rubrique = $GLOBALS['visiteur_session']['restreint'];
-		}
-		if ($connect_statut == '0minirezo') { 
-			$connect_toutes_rubriques = !$connect_id_rubrique;
-		}
-	}
-	// Pour les redacteurs, inc_version a fait l'initialisation minimale
-
-	return ''; // i.e. pas de pb.
+    global $connect_statut, $connect_toutes_rubriques, $connect_id_rubrique, $connect_login, $connect_id_auteur;
+
+    if ($row['statut']=='nouveau'){
+        include_spip('action/inscrire_auteur');
+        $row = confirmer_statut_inscription($row);
+    }
+
+    $connect_id_auteur = $row['id_auteur'];
+    $connect_login = $row['login'];
+    $connect_statut = $row['statut'];
+
+    $GLOBALS['visiteur_session'] = array_merge((array)$GLOBALS['visiteur_session'], $row);
+
+    // au cas ou : ne pas memoriser les champs sensibles
+    unset($GLOBALS['visiteur_session']['pass']);
+    unset($GLOBALS['visiteur_session']['htpass']);
+    unset($GLOBALS['visiteur_session']['alea_actuel']);
+    unset($GLOBALS['visiteur_session']['alea_futur']);
+
+    // creer la session au besoin
+    if (!isset($_COOKIE['spip_session'])) {
+        $session = charger_fonction('session', 'inc');
+        $spip_session = $session($row);
+    }
+
+    // reinjecter les preferences_auteur apres le reset de spip_session
+    // car utilisees au retour par auth_loger()
+    $r = @unserialize($row['prefs']);
+    $GLOBALS['visiteur_session']['prefs'] = ($r ? $r : array());
+    // si prefs pas definies, les definir par defaut
+    if (!isset($GLOBALS['visiteur_session']['prefs']['couleur'])){
+        $GLOBALS['visiteur_session']['prefs']['couleur'] = 1;
+        $GLOBALS['visiteur_session']['prefs']['display'] = 2;
+        $GLOBALS['visiteur_session']['prefs']["display_navigation"] = "navigation_avec_icones";
+        $GLOBALS['visiteur_session']['prefs']["display_outils"] = "oui";
+    }
+
+    $GLOBALS['visiteur_session'] = pipeline('preparer_visiteur_session',array('args'=>array('row'=>$row),'data'=>$GLOBALS['visiteur_session']));
+
+    // Etablir les droits selon le codage attendu
+    // dans ecrire/index.php ecrire/prive.php
+
+    // Pas autorise a acceder a ecrire ? renvoyer le tableau
+    // A noter : le premier appel a autoriser() a le bon gout
+    // d'initialiser $GLOBALS['visiteur_session']['restreint'],
+    // qui ne figure pas dans le fichier de session
+    include_spip('inc/autoriser');
+
+    if (!autoriser('ecrire'))
+        return $row;
+
+    // autoriser('ecrire') ne laisse passer que les Admin et les Redac
+
+    auth_trace($row);
+
+    // Administrateurs
+    if (in_array($connect_statut, explode(',', _STATUT_AUTEUR_RUBRIQUE))) {
+        if (is_array($GLOBALS['visiteur_session']['restreint'])) {
+            $connect_id_rubrique = $GLOBALS['visiteur_session']['restreint'];
+        }
+        if ($connect_statut == '0minirezo') { 
+            $connect_toutes_rubriques = !$connect_id_rubrique;
+        }
+    }
+    // Pour les redacteurs, inc_version a fait l'initialisation minimale
+
+    return ''; // i.e. pas de pb.
 }
 
 /**
@@ -266,21 +266,21 @@ 
  */
 function auth_a_loger()
 {
-	$redirect = generer_url_public('login',
-	"url=" . rawurlencode(self('&',true)), '&');
-
-	// un echec au "bonjour" (login initial) quand le statut est
-	// inconnu signale sans doute un probleme de cookies
-	if (isset($_GET['bonjour']))
-		$redirect = parametre_url($redirect,
-			'var_erreur',
-			(!isset($GLOBALS['visiteur_session']['statut'])
-					? 'cookie'
-					: 'statut'
-			 ),
-					  '&'
-					  );
-	return $redirect;
+    $redirect = generer_url_public('login',
+    "url=" . rawurlencode(self('&',true)), '&');
+
+    // un echec au "bonjour" (login initial) quand le statut est
+    // inconnu signale sans doute un probleme de cookies
+    if (isset($_GET['bonjour']))
+        $redirect = parametre_url($redirect,
+            'var_erreur',
+            (!isset($GLOBALS['visiteur_session']['statut'])
+                    ? 'cookie'
+                    : 'statut'
+                ),
+                        '&'
+                        );
+    return $redirect;
 }
 
 /**
@@ -292,19 +292,19 @@ 
  */
 function auth_trace($row, $date=null)
 {
-	// Indiquer la connexion. A la minute pres ca suffit.
-	if (!is_numeric($connect_quand = $row['quand']))
-		$connect_quand = strtotime($connect_quand);
+    // Indiquer la connexion. A la minute pres ca suffit.
+    if (!is_numeric($connect_quand = $row['quand']))
+        $connect_quand = strtotime($connect_quand);
 
-	if (is_null($date))
-		$date = date('Y-m-d H:i:s');
+    if (is_null($date))
+        $date = date('Y-m-d H:i:s');
 
-	if (abs(strtotime($date) - $connect_quand)  >= 60) {
-		sql_updateq("spip_auteurs", array("en_ligne" => $date), "id_auteur=" .$row['id_auteur']);
-		$row['en_ligne'] = $date;
-	}
+    if (abs(strtotime($date) - $connect_quand)  >= 60) {
+        sql_updateq("spip_auteurs", array("en_ligne" => $date), "id_auteur=" .$row['id_auteur']);
+        $row['en_ligne'] = $date;
+    }
 
-	pipeline('trig_auth_trace',array('args'=>array('row'=>$row,'date'=>$date)));
+    pipeline('trig_auth_trace',array('args'=>array('row'=>$row,'date'=>$date)));
 }
 
 
@@ -320,23 +320,23 @@ 
  * @return mixed
  */
 function auth_administrer($fonction,$args,$defaut=false){
-	$auth_methode = array_shift($args);
-	$auth_methode = $auth_methode ? $auth_methode : 'spip'; // valeur par defaut au cas ou
-	if ($auth = charger_fonction($auth_methode,'auth',true)
-		AND function_exists($f="auth_{$auth_methode}_$fonction")
-	)
-		$res = call_user_func_array($f, $args);
-	else
-		$res = $defaut;
-	$res = pipeline('auth_administrer',array(
-		'args' => array(
-			'fonction' => $fonction,
-			'methode' => $auth_methode,
-			'args' => $args
-		),
-		'data' => $res
-	));
-	return $res;
+    $auth_methode = array_shift($args);
+    $auth_methode = $auth_methode ? $auth_methode : 'spip'; // valeur par defaut au cas ou
+    if ($auth = charger_fonction($auth_methode,'auth',true)
+        AND function_exists($f="auth_{$auth_methode}_$fonction")
+    )
+        $res = call_user_func_array($f, $args);
+    else
+        $res = $defaut;
+    $res = pipeline('auth_administrer',array(
+        'args' => array(
+            'fonction' => $fonction,
+            'methode' => $auth_methode,
+            'args' => $args
+        ),
+        'data' => $res
+    ));
+    return $res;
 }
 
 /**
@@ -346,9 +346,9 @@ 
  * @return array
  */
 function auth_formulaire_login($flux){
-	foreach ($GLOBALS['liste_des_authentifications'] as $methode)
-		$flux = auth_administrer('formulaire_login',array($methode,$flux),$flux);
-	return $flux;
+    foreach ($GLOBALS['liste_des_authentifications'] as $methode)
+        $flux = auth_administrer('formulaire_login',array($methode,$flux),$flux);
+    return $flux;
 }
 
 
@@ -363,19 +363,19 @@ 
  * @return string/bool
  */
 function auth_retrouver_login($login, $serveur=''){
-	if (!spip_connect($serveur)) {
-		include_spip('inc/minipres');
-		echo minipres(_T('info_travaux_titre'),
-			      _T('titre_probleme_technique'));
-		exit;
-	}
-
-	foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
-		if ($auteur = auth_administrer('retrouver_login',array($methode, $login, $serveur))) {
-			return $auteur;
-		}
-	}
-	return false;
+    if (!spip_connect($serveur)) {
+        include_spip('inc/minipres');
+        echo minipres(_T('info_travaux_titre'),
+                    _T('titre_probleme_technique'));
+        exit;
+    }
+
+    foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
+        if ($auteur = auth_administrer('retrouver_login',array($methode, $login, $serveur))) {
+            return $auteur;
+        }
+    }
+    return false;
 }
 
 
@@ -391,28 +391,28 @@ 
  * @return array
  */
 function auth_informer_login($login, $serveur=''){
-	if (!$login
-		OR !$login = auth_retrouver_login($login, $serveur)
-		OR !$row = sql_fetsel('*','spip_auteurs','login='.sql_quote($login,$serveur,'text'),'','','','',$serveur)
-		)
-		return array();
-
-	$prefs = unserialize($row['prefs']);
-	$infos = array(
-		'id_auteur'=>$row['id_auteur'],
-		'login'=>$row['login'],
-		'cnx' => ($prefs['cnx'] == 'perma') ? '1' : '0',
-		'logo' => recuperer_fond('formulaires/inc-logo_auteur', $row),
-	);
-
-	// desactiver le hash md5 si pas auteur spip ?
-	if ($row['source']!=='spip'){
-		$row['alea_actuel']= '';
-		$row['alea_futur']= '';
-	}
-	verifier_visiteur();
-
-	return auth_administrer('informer_login',array($row['source'],$infos, $row, $serveur),$infos);
+    if (!$login
+        OR !$login = auth_retrouver_login($login, $serveur)
+        OR !$row = sql_fetsel('*','spip_auteurs','login='.sql_quote($login,$serveur,'text'),'','','','',$serveur)
+        )
+        return array();
+
+    $prefs = unserialize($row['prefs']);
+    $infos = array(
+        'id_auteur'=>$row['id_auteur'],
+        'login'=>$row['login'],
+        'cnx' => ($prefs['cnx'] == 'perma') ? '1' : '0',
+        'logo' => recuperer_fond('formulaires/inc-logo_auteur', $row),
+    );
+
+    // desactiver le hash md5 si pas auteur spip ?
+    if ($row['source']!=='spip'){
+        $row['alea_actuel']= '';
+        $row['alea_futur']= '';
+    }
+    verifier_visiteur();
+
+    return auth_administrer('informer_login',array($row['source'],$infos, $row, $serveur),$infos);
 }
 
 
@@ -426,20 +426,20 @@ 
  * @return mixed
  */
 function auth_identifier_login($login, $password, $serveur=''){
-	$erreur = "";
-	foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
-		if ($auth = charger_fonction($methode, 'auth',true)){
-			$auteur = $auth($login, $password, $serveur);
-			if (is_array($auteur) AND count($auteur)) {
-				spip_log("connexion de $login par methode $methode");
-				$auteur['auth'] = $methode;
-				return $auteur;
-			}
-			elseif (is_string($auteur))
-				$erreur .= "$auteur ";
-		}
-	}
-	return $erreur;
+    $erreur = "";
+    foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
+        if ($auth = charger_fonction($methode, 'auth',true)){
+            $auteur = $auth($login, $password, $serveur);
+            if (is_array($auteur) AND count($auteur)) {
+                spip_log("connexion de $login par methode $methode");
+                $auteur['auth'] = $methode;
+                return $auteur;
+            }
+            elseif (is_string($auteur))
+                $erreur .= "$auteur ";
+        }
+    }
+    return $erreur;
 }
 
 /**
@@ -453,14 +453,14 @@ 
  * @return string
  */
 function auth_url_retour_login($auth_methode, $login, $redirect='', $serveur=''){
-	$securiser_action = charger_fonction('securiser_action','inc');
-	return $securiser_action('auth', "$auth_methode/$login", $redirect, true);
+    $securiser_action = charger_fonction('securiser_action','inc');
+    return $securiser_action('auth', "$auth_methode/$login", $redirect, true);
 }
 
 function auth_terminer_identifier_login($auth_methode, $login, $serveur=''){
-	$args = func_get_args();
-	$auteur = auth_administrer('terminer_identifier_login',$args);
-	return $auteur;
+    $args = func_get_args();
+    $auteur = auth_administrer('terminer_identifier_login',$args);
+    return $auteur;
 }
 
 /**
@@ -470,41 +470,41 @@ 
  * @return bool
  */
 function auth_loger($auteur){
-	if (!is_array($auteur) OR !count($auteur))
-		return false;
-
-	// initialiser et poser le cookie de session
-	unset($_COOKIE['spip_session']);
-	auth_init_droits($auteur);
-
-	// initialiser les prefs
-	$p = $GLOBALS['visiteur_session']['prefs'];
-	$p['cnx'] = ($auteur['cookie'] == 'oui') ? 'perma' : '';
-
-	sql_updateq('spip_auteurs',
-	            array('prefs' => serialize($p)),
-	            "id_auteur=" . $auteur['id_auteur']);
-
-	// Si on est admin, poser le cookie de correspondance
-	include_spip('inc/cookie');
-	if ($auteur['statut'] == '0minirezo') {
-		spip_setcookie('spip_admin', '@'.$auteur['login'],
-		time() + 7 * 24 * 3600);
-	}
-	// sinon le supprimer ...
-	else {
-		spip_setcookie('spip_admin', '',1);
-	}
-
-	//  bloquer ici le visiteur qui tente d'abuser de ses droits
-	verifier_visiteur();
-	return true;
+    if (!is_array($auteur) OR !count($auteur))
+        return false;
+
+    // initialiser et poser le cookie de session
+    unset($_COOKIE['spip_session']);
+    auth_init_droits($auteur);
+
+    // initialiser les prefs
+    $p = $GLOBALS['visiteur_session']['prefs'];
+    $p['cnx'] = ($auteur['cookie'] == 'oui') ? 'perma' : '';
+
+    sql_updateq('spip_auteurs',
+                array('prefs' => serialize($p)),
+                "id_auteur=" . $auteur['id_auteur']);
+
+    // Si on est admin, poser le cookie de correspondance
+    include_spip('inc/cookie');
+    if ($auteur['statut'] == '0minirezo') {
+        spip_setcookie('spip_admin', '@'.$auteur['login'],
+        time() + 7 * 24 * 3600);
+    }
+    // sinon le supprimer ...
+    else {
+        spip_setcookie('spip_admin', '',1);
+    }
+
+    //  bloquer ici le visiteur qui tente d'abuser de ses droits
+    verifier_visiteur();
+    return true;
 }
 
 
 function auth_deloger(){
-	$logout = charger_fonction('logout','action');
-	$logout();
+    $logout = charger_fonction('logout','action');
+    $logout();
 }
 
 /**
@@ -516,8 +516,8 @@ 
  * @return bool
  */
 function auth_autoriser_modifier_login($auth_methode, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('autoriser_modifier_login',$args);
+    $args = func_get_args();
+    return auth_administrer('autoriser_modifier_login',$args);
 }
 
 /**
@@ -532,8 +532,8 @@ 
  *  message d'erreur ou chaine vide si pas d'erreur
  */
 function auth_verifier_login($auth_methode, $new_login, $id_auteur=0, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('verifier_login',$args,'');
+    $args = func_get_args();
+    return auth_administrer('verifier_login',$args,'');
 }
 
 /**
@@ -546,8 +546,8 @@ 
  * @return bool
  */
 function auth_modifier_login($auth_methode, $new_login, $id_auteur, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('modifier_login',$args);
+    $args = func_get_args();
+    return auth_administrer('modifier_login',$args);
 }
 
 /**
@@ -560,8 +560,8 @@ 
  *	succes ou echec
  */
 function auth_autoriser_modifier_pass($auth_methode, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('autoriser_modifier_pass',$args);
+    $args = func_get_args();
+    return auth_administrer('autoriser_modifier_pass',$args);
 }
 
 /**
@@ -577,8 +577,8 @@ 
  *	message d'erreur ou chaine vide si pas d'erreur
  */
 function auth_verifier_pass($auth_methode, $login, $new_pass, $id_auteur=0, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('verifier_pass',$args,'');
+    $args = func_get_args();
+    return auth_administrer('verifier_pass',$args,'');
 }
 
 /**
@@ -594,8 +594,8 @@ 
  *	succes ou echec
  */
 function auth_modifier_pass($auth_methode, $login, $new_pass, $id_auteur, $serveur=''){
-	$args = func_get_args();
-	return auth_administrer('modifier_pass',$args);
+    $args = func_get_args();
+    return auth_administrer('modifier_pass',$args);
 }
 
 /**
@@ -611,18 +611,18 @@ 
  * @return void
  */
 function auth_synchroniser_distant($auth_methode=true, $id_auteur=0, $champs=array(), $options = array(), $serveur=''){
-	$args = func_get_args();
-	if ($auth_methode===true OR (isset($options['all']) AND $options['all']==true)){
-		$options['all'] = true; // ajouter une option all=>true pour chaque auth
-		$args = array(true, $id_auteur, $champs, $options, $serveur);
-		foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
-			array_shift($args);
-			array_unshift($args,$methode);
-			auth_administrer('synchroniser_distant',$args);
-		}
-	}
-	else
-		auth_administrer('synchroniser_distant',$args);
+    $args = func_get_args();
+    if ($auth_methode===true OR (isset($options['all']) AND $options['all']==true)){
+        $options['all'] = true; // ajouter une option all=>true pour chaque auth
+        $args = array(true, $id_auteur, $champs, $options, $serveur);
+        foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
+            array_shift($args);
+            array_unshift($args,$methode);
+            auth_administrer('synchroniser_distant',$args);
+        }
+    }
+    else
+        auth_administrer('synchroniser_distant',$args);
 }
 
 
@@ -635,27 +635,27 @@ 
  */
 function lire_php_auth($login, $pw, $serveur=''){
 
-	$row = sql_fetsel('*', 'spip_auteurs', 'login=' . sql_quote($login,$serveur,'text'),'','','','',$serveur);
-
-	if (!$row) {
-		if (spip_connect_ldap($serveur)
-		AND $auth_ldap = charger_fonction('ldap', 'auth', true))
-			return $auth_ldap($login, $pw, $serveur, true);
-		return false;
-	}
-	// su pas de source definie
-	// ou auth/xxx introuvable, utiliser 'spip'
-	if (!$auth_methode = $row['source']
-		OR !$auth = charger_fonction($auth_methode, 'auth', true))
-		$auth = charger_fonction('spip', 'auth', true);
-
-	$auteur='';
-	if ($auth)
-		$auteur = $auth($login, $pw, $serveur, true);
-	// verifier que ce n'est pas un message d'erreur
-	if (is_array($auteur) AND count($auteur))
-		return $auteur;
-	return false;
+    $row = sql_fetsel('*', 'spip_auteurs', 'login=' . sql_quote($login,$serveur,'text'),'','','','',$serveur);
+
+    if (!$row) {
+        if (spip_connect_ldap($serveur)
+        AND $auth_ldap = charger_fonction('ldap', 'auth', true))
+            return $auth_ldap($login, $pw, $serveur, true);
+        return false;
+    }
+    // su pas de source definie
+    // ou auth/xxx introuvable, utiliser 'spip'
+    if (!$auth_methode = $row['source']
+        OR !$auth = charger_fonction($auth_methode, 'auth', true))
+        $auth = charger_fonction('spip', 'auth', true);
+
+    $auteur='';
+    if ($auth)
+        $auteur = $auth($login, $pw, $serveur, true);
+    // verifier que ce n'est pas un message d'erreur
+    if (is_array($auteur) AND count($auteur))
+        return $auteur;
+    return false;
 }
 
 /**
@@ -669,21 +669,21 @@ 
  * @param string $lien
  */
 function ask_php_auth($pb, $raison, $retour='', $url='', $re='', $lien='') {
-	@Header("WWW-Authenticate: Basic realm=\"espace prive\"");
-	@Header("HTTP/1.0 401 Unauthorized");
-	$corps = "";
-	$public = generer_url_public();
-	$ecrire = generer_url_ecrire();
-	$retour = $retour?$retour:_T('icone_retour');
-	$corps .= "<p>$raison</p>[<a href='$public'>$retour</a>] ";
-	if ($url) {
-		$corps .= "[<a href='" . generer_url_action('cookie',"essai_auth_http=oui&$url") . "'>$re</a>]";
-	}
-
-	if ($lien)
-		$corps .= " [<a href='$ecrire'>"._T('login_espace_prive')."</a>]";
-	include_spip('inc/minipres');
-	echo minipres($pb,$corps);
-	exit;
+    @Header("WWW-Authenticate: Basic realm=\"espace prive\"");
+    @Header("HTTP/1.0 401 Unauthorized");
+    $corps = "";
+    $public = generer_url_public();
+    $ecrire = generer_url_ecrire();
+    $retour = $retour?$retour:_T('icone_retour');
+    $corps .= "<p>$raison</p>[<a href='$public'>$retour</a>] ";
+    if ($url) {
+        $corps .= "[<a href='" . generer_url_action('cookie',"essai_auth_http=oui&$url") . "'>$re</a>]";
+    }
+
+    if ($lien)
+        $corps .= " [<a href='$ecrire'>"._T('login_espace_prive')."</a>]";
+    include_spip('inc/minipres');
+    echo minipres($pb,$corps);
+    exit;
 }
 ?>

Spacing +66 added lines, -67 removed lines

@@ -32,7 +32,7 @@ 
  */
 function inc_auth_dist() {
 
-	global $connect_login ;
+	global $connect_login;
 
 	$row = auth_mode();
 
@@ -51,7 +51,7 @@ 
 			'site' => generer_url_public('', "action=logout&logout=prive"));
 
 	$n = intval(sql_errno());
-	spip_log("Erreur base de donnees $n " . sql_error());
+	spip_log("Erreur base de donnees $n ".sql_error());
 	return $n ? $n : 1;
 }
 
@@ -75,11 +75,11 @@ 
 		$raison = redirige_formulaire($raison);
 	} elseif (is_int($raison)) {
 		// erreur SQL a afficher
-		$raison = minipres(_T('info_travaux_titre'), _T('titre_probleme_technique'). "<p><tt>".sql_errno()." ".sql_error()."</tt></p>");
+		$raison = minipres(_T('info_travaux_titre'), _T('titre_probleme_technique')."<p><tt>".sql_errno()." ".sql_error()."</tt></p>");
 	} elseif (@$raison['statut']) {
 		// un simple visiteur n'a pas acces a l'espace prive
-		spip_log("connexion refusee a " . @$raison['id_auteur']);
-		$raison = minipres(_T('avis_erreur_connexion'),_T('avis_erreur_visiteur'));
+		spip_log("connexion refusee a ".@$raison['id_auteur']);
+		$raison = minipres(_T('avis_erreur_connexion'), _T('avis_erreur_visiteur'));
 	} else {
 		// auteur en fin de droits ...
 		$h = $raison['site'];
@@ -104,7 +104,7 @@ 
 function auth_mode()
 {
 	global $auth_can_disconnect, $ignore_auth_http, $ignore_remote_user;
-	global $connect_login ;
+	global $connect_login;
 
 	//
 	// Initialiser variables (eviter hacks par URL)
@@ -122,7 +122,7 @@ 
 	if (isset($_COOKIE['spip_session'])) {
 		$session = charger_fonction('session', 'inc');
 		if ($id_auteur = $session()
-		OR $id_auteur===0 // reprise sur restauration
+		OR $id_auteur === 0 // reprise sur restauration
 		) {
 			$auth_can_disconnect = true;
 			$connect_login = $GLOBALS['visiteur_session']['login'];
@@ -166,8 +166,7 @@ 
 	$where = (is_numeric($id_auteur)
 	/*AND $id_auteur>0*/ // reprise lors des restaurations
 	) ?
-	  "id_auteur=$id_auteur" :
-	  (!strlen($connect_login) ? '' : "login=" . sql_quote($connect_login,'','text'));
+	  "id_auteur=$id_auteur" : (!strlen($connect_login) ? '' : "login=".sql_quote($connect_login, '', 'text'));
 
 	if (!$where) return '';
 
@@ -192,7 +191,7 @@ 
 {
 	global $connect_statut, $connect_toutes_rubriques, $connect_id_rubrique, $connect_login, $connect_id_auteur;
 
-	if ($row['statut']=='nouveau'){
+	if ($row['statut'] == 'nouveau') {
 		include_spip('action/inscrire_auteur');
 		$row = confirmer_statut_inscription($row);
 	}
@@ -201,7 +200,7 @@ 
 	$connect_login = $row['login'];
 	$connect_statut = $row['statut'];
 
-	$GLOBALS['visiteur_session'] = array_merge((array)$GLOBALS['visiteur_session'], $row);
+	$GLOBALS['visiteur_session'] = array_merge((array) $GLOBALS['visiteur_session'], $row);
 
 	// au cas ou : ne pas memoriser les champs sensibles
 	unset($GLOBALS['visiteur_session']['pass']);
@@ -220,14 +219,14 @@ 
 	$r = @unserialize($row['prefs']);
 	$GLOBALS['visiteur_session']['prefs'] = ($r ? $r : array());
 	// si prefs pas definies, les definir par defaut
-	if (!isset($GLOBALS['visiteur_session']['prefs']['couleur'])){
+	if (!isset($GLOBALS['visiteur_session']['prefs']['couleur'])) {
 		$GLOBALS['visiteur_session']['prefs']['couleur'] = 1;
 		$GLOBALS['visiteur_session']['prefs']['display'] = 2;
 		$GLOBALS['visiteur_session']['prefs']["display_navigation"] = "navigation_avec_icones";
 		$GLOBALS['visiteur_session']['prefs']["display_outils"] = "oui";
 	}
 
-	$GLOBALS['visiteur_session'] = pipeline('preparer_visiteur_session',array('args'=>array('row'=>$row),'data'=>$GLOBALS['visiteur_session']));
+	$GLOBALS['visiteur_session'] = pipeline('preparer_visiteur_session', array('args'=>array('row'=>$row), 'data'=>$GLOBALS['visiteur_session']));
 
 	// Etablir les droits selon le codage attendu
 	// dans ecrire/index.php ecrire/prive.php
@@ -267,7 +266,7 @@ 
 function auth_a_loger()
 {
 	$redirect = generer_url_public('login',
-	"url=" . rawurlencode(self('&',true)), '&');
+	"url=".rawurlencode(self('&', true)), '&');
 
 	// un echec au "bonjour" (login initial) quand le statut est
 	// inconnu signale sans doute un probleme de cookies
@@ -290,7 +289,7 @@ 
  * @param array $row
  * @param null|string $date
  */
-function auth_trace($row, $date=null)
+function auth_trace($row, $date = null)
 {
 	// Indiquer la connexion. A la minute pres ca suffit.
 	if (!is_numeric($connect_quand = $row['quand']))
@@ -299,12 +298,12 @@ 
 	if (is_null($date))
 		$date = date('Y-m-d H:i:s');
 
-	if (abs(strtotime($date) - $connect_quand)  >= 60) {
-		sql_updateq("spip_auteurs", array("en_ligne" => $date), "id_auteur=" .$row['id_auteur']);
+	if (abs(strtotime($date) - $connect_quand) >= 60) {
+		sql_updateq("spip_auteurs", array("en_ligne" => $date), "id_auteur=".$row['id_auteur']);
 		$row['en_ligne'] = $date;
 	}
 
-	pipeline('trig_auth_trace',array('args'=>array('row'=>$row,'date'=>$date)));
+	pipeline('trig_auth_trace', array('args'=>array('row'=>$row, 'date'=>$date)));
 }
 
 
@@ -319,16 +318,16 @@ 
  * @param mixed $defaut
  * @return mixed
  */
-function auth_administrer($fonction,$args,$defaut=false){
+function auth_administrer($fonction, $args, $defaut = false) {
 	$auth_methode = array_shift($args);
 	$auth_methode = $auth_methode ? $auth_methode : 'spip'; // valeur par defaut au cas ou
-	if ($auth = charger_fonction($auth_methode,'auth',true)
-		AND function_exists($f="auth_{$auth_methode}_$fonction")
+	if ($auth = charger_fonction($auth_methode, 'auth', true)
+		AND function_exists($f = "auth_{$auth_methode}_$fonction")
 	)
 		$res = call_user_func_array($f, $args);
 	else
 		$res = $defaut;
-	$res = pipeline('auth_administrer',array(
+	$res = pipeline('auth_administrer', array(
 		'args' => array(
 			'fonction' => $fonction,
 			'methode' => $auth_methode,
@@ -345,9 +344,9 @@ 
  * @param array $flux
  * @return array
  */
-function auth_formulaire_login($flux){
+function auth_formulaire_login($flux) {
 	foreach ($GLOBALS['liste_des_authentifications'] as $methode)
-		$flux = auth_administrer('formulaire_login',array($methode,$flux),$flux);
+		$flux = auth_administrer('formulaire_login', array($methode, $flux), $flux);
 	return $flux;
 }
 
@@ -362,7 +361,7 @@ 
  * @param string $serveur
  * @return string/bool
  */
-function auth_retrouver_login($login, $serveur=''){
+function auth_retrouver_login($login, $serveur = '') {
 	if (!spip_connect($serveur)) {
 		include_spip('inc/minipres');
 		echo minipres(_T('info_travaux_titre'),
@@ -371,7 +370,7 @@ 
 	}
 
 	foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
-		if ($auteur = auth_administrer('retrouver_login',array($methode, $login, $serveur))) {
+		if ($auteur = auth_administrer('retrouver_login', array($methode, $login, $serveur))) {
 			return $auteur;
 		}
 	}
@@ -390,10 +389,10 @@ 
  * @param string $serveur
  * @return array
  */
-function auth_informer_login($login, $serveur=''){
+function auth_informer_login($login, $serveur = '') {
 	if (!$login
 		OR !$login = auth_retrouver_login($login, $serveur)
-		OR !$row = sql_fetsel('*','spip_auteurs','login='.sql_quote($login,$serveur,'text'),'','','','',$serveur)
+		OR !$row = sql_fetsel('*', 'spip_auteurs', 'login='.sql_quote($login, $serveur, 'text'), '', '', '', '', $serveur)
 		)
 		return array();
 
@@ -406,13 +405,13 @@ 
 	);
 
 	// desactiver le hash md5 si pas auteur spip ?
-	if ($row['source']!=='spip'){
-		$row['alea_actuel']= '';
-		$row['alea_futur']= '';
+	if ($row['source'] !== 'spip') {
+		$row['alea_actuel'] = '';
+		$row['alea_futur'] = '';
 	}
 	verifier_visiteur();
 
-	return auth_administrer('informer_login',array($row['source'],$infos, $row, $serveur),$infos);
+	return auth_administrer('informer_login', array($row['source'], $infos, $row, $serveur), $infos);
 }
 
 
@@ -425,10 +424,10 @@ 
  * @param string $serveur
  * @return mixed
  */
-function auth_identifier_login($login, $password, $serveur=''){
+function auth_identifier_login($login, $password, $serveur = '') {
 	$erreur = "";
 	foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
-		if ($auth = charger_fonction($methode, 'auth',true)){
+		if ($auth = charger_fonction($methode, 'auth', true)) {
 			$auteur = $auth($login, $password, $serveur);
 			if (is_array($auteur) AND count($auteur)) {
 				spip_log("connexion de $login par methode $methode");
@@ -452,14 +451,14 @@ 
  * @param string $serveur
  * @return string
  */
-function auth_url_retour_login($auth_methode, $login, $redirect='', $serveur=''){
-	$securiser_action = charger_fonction('securiser_action','inc');
+function auth_url_retour_login($auth_methode, $login, $redirect = '', $serveur = '') {
+	$securiser_action = charger_fonction('securiser_action', 'inc');
 	return $securiser_action('auth', "$auth_methode/$login", $redirect, true);
 }
 
-function auth_terminer_identifier_login($auth_methode, $login, $serveur=''){
+function auth_terminer_identifier_login($auth_methode, $login, $serveur = '') {
 	$args = func_get_args();
-	$auteur = auth_administrer('terminer_identifier_login',$args);
+	$auteur = auth_administrer('terminer_identifier_login', $args);
 	return $auteur;
 }
 
@@ -469,7 +468,7 @@ 
  * @param array $auteur
  * @return bool
  */
-function auth_loger($auteur){
+function auth_loger($auteur) {
 	if (!is_array($auteur) OR !count($auteur))
 		return false;
 
@@ -483,7 +482,7 @@ 
 
 	sql_updateq('spip_auteurs',
 	            array('prefs' => serialize($p)),
-	            "id_auteur=" . $auteur['id_auteur']);
+	            "id_auteur=".$auteur['id_auteur']);
 
 	// Si on est admin, poser le cookie de correspondance
 	include_spip('inc/cookie');
@@ -493,7 +492,7 @@ 
 	}
 	// sinon le supprimer ...
 	else {
-		spip_setcookie('spip_admin', '',1);
+		spip_setcookie('spip_admin', '', 1);
 	}
 
 	//  bloquer ici le visiteur qui tente d'abuser de ses droits
@@ -502,8 +501,8 @@ 
 }
 
 
-function auth_deloger(){
-	$logout = charger_fonction('logout','action');
+function auth_deloger() {
+	$logout = charger_fonction('logout', 'action');
 	$logout();
 }
 
@@ -515,9 +514,9 @@ 
  * @param string $serveur
  * @return bool
  */
-function auth_autoriser_modifier_login($auth_methode, $serveur=''){
+function auth_autoriser_modifier_login($auth_methode, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('autoriser_modifier_login',$args);
+	return auth_administrer('autoriser_modifier_login', $args);
 }
 
 /**
@@ -531,9 +530,9 @@ 
  * @return string
  *  message d'erreur ou chaine vide si pas d'erreur
  */
-function auth_verifier_login($auth_methode, $new_login, $id_auteur=0, $serveur=''){
+function auth_verifier_login($auth_methode, $new_login, $id_auteur = 0, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('verifier_login',$args,'');
+	return auth_administrer('verifier_login', $args, '');
 }
 
 /**
@@ -545,9 +544,9 @@ 
  * @param string $serveur
  * @return bool
  */
-function auth_modifier_login($auth_methode, $new_login, $id_auteur, $serveur=''){
+function auth_modifier_login($auth_methode, $new_login, $id_auteur, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('modifier_login',$args);
+	return auth_administrer('modifier_login', $args);
 }
 
 /**
@@ -559,9 +558,9 @@ 
  * @return bool
  *	succes ou echec
  */
-function auth_autoriser_modifier_pass($auth_methode, $serveur=''){
+function auth_autoriser_modifier_pass($auth_methode, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('autoriser_modifier_pass',$args);
+	return auth_administrer('autoriser_modifier_pass', $args);
 }
 
 /**
@@ -576,9 +575,9 @@ 
  * @return string
  *	message d'erreur ou chaine vide si pas d'erreur
  */
-function auth_verifier_pass($auth_methode, $login, $new_pass, $id_auteur=0, $serveur=''){
+function auth_verifier_pass($auth_methode, $login, $new_pass, $id_auteur = 0, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('verifier_pass',$args,'');
+	return auth_administrer('verifier_pass', $args, '');
 }
 
 /**
@@ -593,9 +592,9 @@ 
  * @return bool
  *	succes ou echec
  */
-function auth_modifier_pass($auth_methode, $login, $new_pass, $id_auteur, $serveur=''){
+function auth_modifier_pass($auth_methode, $login, $new_pass, $id_auteur, $serveur = '') {
 	$args = func_get_args();
-	return auth_administrer('modifier_pass',$args);
+	return auth_administrer('modifier_pass', $args);
 }
 
 /**
@@ -610,19 +609,19 @@ 
  * @param string $serveur
  * @return void
  */
-function auth_synchroniser_distant($auth_methode=true, $id_auteur=0, $champs=array(), $options = array(), $serveur=''){
+function auth_synchroniser_distant($auth_methode = true, $id_auteur = 0, $champs = array(), $options = array(), $serveur = '') {
 	$args = func_get_args();
-	if ($auth_methode===true OR (isset($options['all']) AND $options['all']==true)){
+	if ($auth_methode === true OR (isset($options['all']) AND $options['all'] == true)) {
 		$options['all'] = true; // ajouter une option all=>true pour chaque auth
 		$args = array(true, $id_auteur, $champs, $options, $serveur);
 		foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
 			array_shift($args);
-			array_unshift($args,$methode);
-			auth_administrer('synchroniser_distant',$args);
+			array_unshift($args, $methode);
+			auth_administrer('synchroniser_distant', $args);
 		}
 	}
 	else
-		auth_administrer('synchroniser_distant',$args);
+		auth_administrer('synchroniser_distant', $args);
 }
 
 
@@ -633,9 +632,9 @@ 
  * @param string $serveur
  * @return array
  */
-function lire_php_auth($login, $pw, $serveur=''){
+function lire_php_auth($login, $pw, $serveur = '') {
 
-	$row = sql_fetsel('*', 'spip_auteurs', 'login=' . sql_quote($login,$serveur,'text'),'','','','',$serveur);
+	$row = sql_fetsel('*', 'spip_auteurs', 'login='.sql_quote($login, $serveur, 'text'), '', '', '', '', $serveur);
 
 	if (!$row) {
 		if (spip_connect_ldap($serveur)
@@ -649,7 +648,7 @@ 
 		OR !$auth = charger_fonction($auth_methode, 'auth', true))
 		$auth = charger_fonction('spip', 'auth', true);
 
-	$auteur='';
+	$auteur = '';
 	if ($auth)
 		$auteur = $auth($login, $pw, $serveur, true);
 	// verifier que ce n'est pas un message d'erreur
@@ -668,22 +667,22 @@ 
  * @param string $re
  * @param string $lien
  */
-function ask_php_auth($pb, $raison, $retour='', $url='', $re='', $lien='') {
+function ask_php_auth($pb, $raison, $retour = '', $url = '', $re = '', $lien = '') {
 	@Header("WWW-Authenticate: Basic realm=\"espace prive\"");
 	@Header("HTTP/1.0 401 Unauthorized");
 	$corps = "";
 	$public = generer_url_public();
 	$ecrire = generer_url_ecrire();
-	$retour = $retour?$retour:_T('icone_retour');
+	$retour = $retour ? $retour : _T('icone_retour');
 	$corps .= "<p>$raison</p>[<a href='$public'>$retour</a>] ";
 	if ($url) {
-		$corps .= "[<a href='" . generer_url_action('cookie',"essai_auth_http=oui&$url") . "'>$re</a>]";
+		$corps .= "[<a href='".generer_url_action('cookie', "essai_auth_http=oui&$url")."'>$re</a>]";
 	}
 
 	if ($lien)
 		$corps .= " [<a href='$ecrire'>"._T('login_espace_prive')."</a>]";
 	include_spip('inc/minipres');
-	echo minipres($pb,$corps);
+	echo minipres($pb, $corps);
 	exit;
 }
 ?>

Braces +67 added lines, -42 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('base/abstract_sql');
 
@@ -36,9 +38,13 @@ 
 
 	$row = auth_mode();
 
-	if ($row) return auth_init_droits($row);
+	if ($row) {
+	    return auth_init_droits($row);
+	}
 
-	if (!$connect_login) return auth_a_loger();
+	if (!$connect_login) {
+	    return auth_a_loger();
+	}
 
 	// Cas ou l'auteur a ete identifie mais on n'a pas d'info sur lui
 	// C'est soit parce que la base est inutilisable,
@@ -46,9 +52,10 @@ 
 	// Pas la peine d'insister.
 	// Renvoyer le nom fautif et une URL de remise a zero
 
-	if (spip_connect())
-		return array('login' => $connect_login,
+	if (spip_connect()) {
+			return array('login' => $connect_login,
 			'site' => generer_url_public('', "action=logout&logout=prive"));
+	}
 
 	$n = intval(sql_errno());
 	spip_log("Erreur base de donnees $n " . sql_error());
@@ -126,7 +133,9 @@ 
 		) {
 			$auth_can_disconnect = true;
 			$connect_login = $GLOBALS['visiteur_session']['login'];
-		} else unset($_COOKIE['spip_session']);
+		} else {
+		    unset($_COOKIE['spip_session']);
+		}
 	}
 
 	// Essayer auth http si significatif
@@ -159,8 +168,9 @@ 
 		}
 		// Authentification .htaccess old style, car .htaccess semble
 		// souvent definir *aussi* PHP_AUTH_USER et PHP_AUTH_PW
-		else if (isset($_SERVER['REMOTE_USER']))
-			$connect_login = $_SERVER['REMOTE_USER'];
+		else if (isset($_SERVER['REMOTE_USER'])) {
+					$connect_login = $_SERVER['REMOTE_USER'];
+		}
 	}
 
 	$where = (is_numeric($id_auteur)
@@ -169,7 +179,9 @@ 
 	  "id_auteur=$id_auteur" :
 	  (!strlen($connect_login) ? '' : "login=" . sql_quote($connect_login,'','text'));
 
-	if (!$where) return '';
+	if (!$where) {
+	    return '';
+	}
 
 	// Trouver les autres infos dans la table auteurs.
 	// le champ 'quand' est utilise par l'agenda
@@ -238,8 +250,9 @@ 
 	// qui ne figure pas dans le fichier de session
 	include_spip('inc/autoriser');
 
-	if (!autoriser('ecrire'))
-		return $row;
+	if (!autoriser('ecrire')) {
+			return $row;
+	}
 
 	// autoriser('ecrire') ne laisse passer que les Admin et les Redac
 
@@ -271,8 +284,8 @@ 
 
 	// un echec au "bonjour" (login initial) quand le statut est
 	// inconnu signale sans doute un probleme de cookies
-	if (isset($_GET['bonjour']))
-		$redirect = parametre_url($redirect,
+	if (isset($_GET['bonjour'])) {
+			$redirect = parametre_url($redirect,
 			'var_erreur',
 			(!isset($GLOBALS['visiteur_session']['statut'])
 					? 'cookie'
@@ -280,6 +293,7 @@ 
 			 ),
 					  '&'
 					  );
+	}
 	return $redirect;
 }
 
@@ -293,11 +307,13 @@ 
 function auth_trace($row, $date=null)
 {
 	// Indiquer la connexion. A la minute pres ca suffit.
-	if (!is_numeric($connect_quand = $row['quand']))
-		$connect_quand = strtotime($connect_quand);
+	if (!is_numeric($connect_quand = $row['quand'])) {
+			$connect_quand = strtotime($connect_quand);
+	}
 
-	if (is_null($date))
-		$date = date('Y-m-d H:i:s');
+	if (is_null($date)) {
+			$date = date('Y-m-d H:i:s');
+	}
 
 	if (abs(strtotime($date) - $connect_quand)  >= 60) {
 		sql_updateq("spip_auteurs", array("en_ligne" => $date), "id_auteur=" .$row['id_auteur']);
@@ -324,10 +340,11 @@ 
 	$auth_methode = $auth_methode ? $auth_methode : 'spip'; // valeur par defaut au cas ou
 	if ($auth = charger_fonction($auth_methode,'auth',true)
 		AND function_exists($f="auth_{$auth_methode}_$fonction")
-	)
-		$res = call_user_func_array($f, $args);
-	else
-		$res = $defaut;
+	) {
+			$res = call_user_func_array($f, $args);
+	} else {
+			$res = $defaut;
+	}
 	$res = pipeline('auth_administrer',array(
 		'args' => array(
 			'fonction' => $fonction,
@@ -346,8 +363,9 @@ 
  * @return array
  */
 function auth_formulaire_login($flux){
-	foreach ($GLOBALS['liste_des_authentifications'] as $methode)
-		$flux = auth_administrer('formulaire_login',array($methode,$flux),$flux);
+	foreach ($GLOBALS['liste_des_authentifications'] as $methode) {
+			$flux = auth_administrer('formulaire_login',array($methode,$flux),$flux);
+	}
 	return $flux;
 }
 
@@ -394,8 +412,9 @@ 
 	if (!$login
 		OR !$login = auth_retrouver_login($login, $serveur)
 		OR !$row = sql_fetsel('*','spip_auteurs','login='.sql_quote($login,$serveur,'text'),'','','','',$serveur)
-		)
-		return array();
+		) {
+			return array();
+	}
 
 	$prefs = unserialize($row['prefs']);
 	$infos = array(
@@ -434,9 +453,9 @@ 
 				spip_log("connexion de $login par methode $methode");
 				$auteur['auth'] = $methode;
 				return $auteur;
+			} elseif (is_string($auteur)) {
+							$erreur .= "$auteur ";
 			}
-			elseif (is_string($auteur))
-				$erreur .= "$auteur ";
 		}
 	}
 	return $erreur;
@@ -470,8 +489,9 @@ 
  * @return bool
  */
 function auth_loger($auteur){
-	if (!is_array($auteur) OR !count($auteur))
-		return false;
+	if (!is_array($auteur) OR !count($auteur)) {
+			return false;
+	}
 
 	// initialiser et poser le cookie de session
 	unset($_COOKIE['spip_session']);
@@ -620,10 +640,10 @@ 
 			array_unshift($args,$methode);
 			auth_administrer('synchroniser_distant',$args);
 		}
+	} else {
+			auth_administrer('synchroniser_distant',$args);
+	}
 	}
-	else
-		auth_administrer('synchroniser_distant',$args);
-}
 
 
 /**
@@ -639,22 +659,26 @@ 
 
 	if (!$row) {
 		if (spip_connect_ldap($serveur)
-		AND $auth_ldap = charger_fonction('ldap', 'auth', true))
-			return $auth_ldap($login, $pw, $serveur, true);
+		AND $auth_ldap = charger_fonction('ldap', 'auth', true)) {
+					return $auth_ldap($login, $pw, $serveur, true);
+		}
 		return false;
 	}
 	// su pas de source definie
 	// ou auth/xxx introuvable, utiliser 'spip'
 	if (!$auth_methode = $row['source']
-		OR !$auth = charger_fonction($auth_methode, 'auth', true))
-		$auth = charger_fonction('spip', 'auth', true);
+		OR !$auth = charger_fonction($auth_methode, 'auth', true)) {
+			$auth = charger_fonction('spip', 'auth', true);
+	}
 
 	$auteur='';
-	if ($auth)
-		$auteur = $auth($login, $pw, $serveur, true);
+	if ($auth) {
+			$auteur = $auth($login, $pw, $serveur, true);
+	}
 	// verifier que ce n'est pas un message d'erreur
-	if (is_array($auteur) AND count($auteur))
-		return $auteur;
+	if (is_array($auteur) AND count($auteur)) {
+			return $auteur;
+	}
 	return false;
 }
 
@@ -680,8 +704,9 @@ 
 		$corps .= "[<a href='" . generer_url_action('cookie',"essai_auth_http=oui&$url") . "'>$re</a>]";
 	}
 
-	if ($lien)
-		$corps .= " [<a href='$ecrire'>"._T('login_espace_prive')."</a>]";
+	if ($lien) {
+			$corps .= " [<a href='$ecrire'>"._T('login_espace_prive')."</a>]";
+	}
 	include_spip('inc/minipres');
 	echo minipres($pb,$corps);
 	exit;

ecrire/inc/bandeau.php

Doc Comments -3 removed lines

@@ -132,9 +132,6 @@
 /**
  * Construire tout le bandeau superieur de l'espace prive
  *
- * @param unknown_type $rubrique
- * @param unknown_type $sous_rubrique
- * @param unknown_type $largeur
  * @return unknown
  */
 function inc_bandeau_dist() {

Indentation +81 added lines, -81 removed lines

@@ -15,27 +15,27 @@ 
 include_spip('inc/boutons');
 
 function definir_barre_contexte($contexte = null){
-	if (is_null($contexte))
-		$contexte = $_GET;
-	elseif(is_string($contexte))
-		$contexte = unserialize($contexte);
-	if (!isset($contexte['id_rubrique']) AND isset($contexte['exec'])){
-		if (!function_exists('trouver_objet_exec'))
-			include_spip('inc/pipelines_ecrire');
-		if ($e=trouver_objet_exec($contexte['exec'])){
-			$_id = $e['id_table_objet'];
-			if (isset($contexte[$_id]) AND $id=intval($contexte[$_id])){
-				$table = $e['table_objet_sql'];
-				$row = sql_fetsel('*',$table,"$_id=".intval($id));
-				if (isset($row['id_rubrique'])){
-					$contexte['id_rubrique'] = $row['id_rubrique'];
-					if (isset($row['id_secteur']))
-						$contexte['id_secteur'] = $row['id_secteur'];
-				}
-			}
-		}
-	}
-	return $contexte;
+    if (is_null($contexte))
+        $contexte = $_GET;
+    elseif(is_string($contexte))
+        $contexte = unserialize($contexte);
+    if (!isset($contexte['id_rubrique']) AND isset($contexte['exec'])){
+        if (!function_exists('trouver_objet_exec'))
+            include_spip('inc/pipelines_ecrire');
+        if ($e=trouver_objet_exec($contexte['exec'])){
+            $_id = $e['id_table_objet'];
+            if (isset($contexte[$_id]) AND $id=intval($contexte[$_id])){
+                $table = $e['table_objet_sql'];
+                $row = sql_fetsel('*',$table,"$_id=".intval($id));
+                if (isset($row['id_rubrique'])){
+                    $contexte['id_rubrique'] = $row['id_rubrique'];
+                    if (isset($row['id_secteur']))
+                        $contexte['id_secteur'] = $row['id_secteur'];
+                }
+            }
+        }
+    }
+    return $contexte;
 }
 
 /**
@@ -51,55 +51,55 @@ 
  */
 function definir_barre_boutons($contexte=array(),$icones = true, $autorise = true) {
     include_spip('inc/autoriser');
-	$boutons_admin=array();
+    $boutons_admin=array();
 
-	// les boutons du core, issus de prive/navigation.xml
-	$liste_boutons = array();
+    // les boutons du core, issus de prive/navigation.xml
+    $liste_boutons = array();
 
-	// ajouter les boutons issus des plugin via plugin.xml
-	if (function_exists('boutons_plugins')
-	  AND is_array($liste_boutons_plugins = boutons_plugins()))
-		$liste_boutons = &$liste_boutons_plugins;
+    // ajouter les boutons issus des plugin via plugin.xml
+    if (function_exists('boutons_plugins')
+      AND is_array($liste_boutons_plugins = boutons_plugins()))
+        $liste_boutons = &$liste_boutons_plugins;
 
-	foreach($liste_boutons as $id => $infos){
-		$parent = "";
-		// les boutons principaux ne sont pas soumis a autorisation
-		if (!isset($infos['parent']) OR !($parent = $infos['parent']) OR !$autorise OR autoriser('menu',"_$id",0,NULL,array('contexte'=>$contexte))){
-			if ($parent
-			  AND $parent = preg_replace(',^bando_,','menu_',$parent)
-			  AND isset($boutons_admin[$parent])){
-				if (!is_array($boutons_admin[$parent]->sousmenu))
-					$boutons_admin[$parent]->sousmenu = array();
-				$position = (isset($infos['position']) AND strlen($infos['position'])) ? intval($infos['position']) : count($boutons_admin[$parent]->sousmenu);
-				if ($position<0) $position = count($boutons_admin[$parent]->sousmenu)+1+$position;
-				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu,0,$position)
-				+ array($id=> new Bouton(
-					($icones AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
-					$infos['titre'],	// titre
-					(isset($infos['action']) AND $infos['action'])?$infos['action']:null,
-					(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
-					))
-				+ array_slice($boutons_admin[$parent]->sousmenu,$position,100);
-			}
-			if (!$parent
-			// provisoire, eviter les vieux boutons
-			AND (!in_array($id,array('forum','statistiques_visites')))
+    foreach($liste_boutons as $id => $infos){
+        $parent = "";
+        // les boutons principaux ne sont pas soumis a autorisation
+        if (!isset($infos['parent']) OR !($parent = $infos['parent']) OR !$autorise OR autoriser('menu',"_$id",0,NULL,array('contexte'=>$contexte))){
+            if ($parent
+              AND $parent = preg_replace(',^bando_,','menu_',$parent)
+              AND isset($boutons_admin[$parent])){
+                if (!is_array($boutons_admin[$parent]->sousmenu))
+                    $boutons_admin[$parent]->sousmenu = array();
+                $position = (isset($infos['position']) AND strlen($infos['position'])) ? intval($infos['position']) : count($boutons_admin[$parent]->sousmenu);
+                if ($position<0) $position = count($boutons_admin[$parent]->sousmenu)+1+$position;
+                $boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu,0,$position)
+                + array($id=> new Bouton(
+                    ($icones AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
+                    $infos['titre'],	// titre
+                    (isset($infos['action']) AND $infos['action'])?$infos['action']:null,
+                    (isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
+                    ))
+                + array_slice($boutons_admin[$parent]->sousmenu,$position,100);
+            }
+            if (!$parent
+            // provisoire, eviter les vieux boutons
+            AND (!in_array($id,array('forum','statistiques_visites')))
 
-			) {
-				$position = (isset($infos['position']) and $infos['position']) ? $infos['position'] : count($boutons_admin);
-				$boutons_admin = array_slice($boutons_admin,0,$position)
-				+array($id=> new Bouton(
-					($icones AND isset($infos['icone']) AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
-					$infos['titre'],	// titre
-					(isset($infos['action']) AND $infos['action'])?$infos['action']:null,
-					(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
-					))
-				+ array_slice($boutons_admin,$position,100);
-			}
-		}
-	}
+            ) {
+                $position = (isset($infos['position']) and $infos['position']) ? $infos['position'] : count($boutons_admin);
+                $boutons_admin = array_slice($boutons_admin,0,$position)
+                +array($id=> new Bouton(
+                    ($icones AND isset($infos['icone']) AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
+                    $infos['titre'],	// titre
+                    (isset($infos['action']) AND $infos['action'])?$infos['action']:null,
+                    (isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
+                    ))
+                + array_slice($boutons_admin,$position,100);
+            }
+        }
+    }
 
-	return pipeline('ajouter_menus', $boutons_admin);
+    return pipeline('ajouter_menus', $boutons_admin);
 }
 
 /**
@@ -111,21 +111,21 @@ 
  */
 // http://doc.spip.org/@bandeau_creer_url
 function bandeau_creer_url($url, $args="", $contexte=null){
-	if (!preg_match(',[\/\?],',$url)) {
-		$url = generer_url_ecrire($url,$args,true);
-		// recuperer les parametres du contexte demande par l'url sous la forme
-		// &truc=@machin@
-		// @machin@ etant remplace par _request('machin')
-		$url = str_replace('&amp;','&',$url);
-		while (preg_match(",[&?]([a-z_]+)=@([a-z_]+)@,i",$url,$matches)){
-			if ($matches[2]=='id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique']))
-				$contexte['id_secteur'] = sql_getfetsel('id_secteur','spip_rubriques','id_rubrique='.intval($contexte['id_rubrique']));
-			$val = _request($matches[2],$contexte);
-			$url = parametre_url($url,$matches[1],$val?$val:'','&');
-		}
-		$url = str_replace('&','&amp;',$url);
-	}
-	return $url;
+    if (!preg_match(',[\/\?],',$url)) {
+        $url = generer_url_ecrire($url,$args,true);
+        // recuperer les parametres du contexte demande par l'url sous la forme
+        // &truc=@machin@
+        // @machin@ etant remplace par _request('machin')
+        $url = str_replace('&amp;','&',$url);
+        while (preg_match(",[&?]([a-z_]+)=@([a-z_]+)@,i",$url,$matches)){
+            if ($matches[2]=='id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique']))
+                $contexte['id_secteur'] = sql_getfetsel('id_secteur','spip_rubriques','id_rubrique='.intval($contexte['id_rubrique']));
+            $val = _request($matches[2],$contexte);
+            $url = parametre_url($url,$matches[1],$val?$val:'','&');
+        }
+        $url = str_replace('&','&amp;',$url);
+    }
+    return $url;
 }
 
 
@@ -138,7 +138,7 @@ 
  * @return unknown
  */
 function inc_bandeau_dist() {
-	return recuperer_fond('prive/squelettes/inclure/barre-nav',$_GET);
+    return recuperer_fond('prive/squelettes/inclure/barre-nav',$_GET);
 }
 
 ?>

Spacing +38 added lines, -38 removed lines

@@ -14,20 +14,20 @@ 
 
 include_spip('inc/boutons');
 
-function definir_barre_contexte($contexte = null){
+function definir_barre_contexte($contexte = null) {
 	if (is_null($contexte))
 		$contexte = $_GET;
-	elseif(is_string($contexte))
+	elseif (is_string($contexte))
 		$contexte = unserialize($contexte);
-	if (!isset($contexte['id_rubrique']) AND isset($contexte['exec'])){
+	if (!isset($contexte['id_rubrique']) AND isset($contexte['exec'])) {
 		if (!function_exists('trouver_objet_exec'))
 			include_spip('inc/pipelines_ecrire');
-		if ($e=trouver_objet_exec($contexte['exec'])){
+		if ($e = trouver_objet_exec($contexte['exec'])) {
 			$_id = $e['id_table_objet'];
-			if (isset($contexte[$_id]) AND $id=intval($contexte[$_id])){
+			if (isset($contexte[$_id]) AND $id = intval($contexte[$_id])) {
 				$table = $e['table_objet_sql'];
-				$row = sql_fetsel('*',$table,"$_id=".intval($id));
-				if (isset($row['id_rubrique'])){
+				$row = sql_fetsel('*', $table, "$_id=".intval($id));
+				if (isset($row['id_rubrique'])) {
 					$contexte['id_rubrique'] = $row['id_rubrique'];
 					if (isset($row['id_secteur']))
 						$contexte['id_secteur'] = $row['id_secteur'];
@@ -49,9 +49,9 @@ 
  * @param bool $autorise // ne renvoyer que les boutons autorises
  * @return array
  */
-function definir_barre_boutons($contexte=array(),$icones = true, $autorise = true) {
+function definir_barre_boutons($contexte = array(), $icones = true, $autorise = true) {
     include_spip('inc/autoriser');
-	$boutons_admin=array();
+	$boutons_admin = array();
 
 	// les boutons du core, issus de prive/navigation.xml
 	$liste_boutons = array();
@@ -61,40 +61,40 @@ 
 	  AND is_array($liste_boutons_plugins = boutons_plugins()))
 		$liste_boutons = &$liste_boutons_plugins;
 
-	foreach($liste_boutons as $id => $infos){
+	foreach ($liste_boutons as $id => $infos) {
 		$parent = "";
 		// les boutons principaux ne sont pas soumis a autorisation
-		if (!isset($infos['parent']) OR !($parent = $infos['parent']) OR !$autorise OR autoriser('menu',"_$id",0,NULL,array('contexte'=>$contexte))){
+		if (!isset($infos['parent']) OR !($parent = $infos['parent']) OR !$autorise OR autoriser('menu', "_$id", 0, NULL, array('contexte'=>$contexte))) {
 			if ($parent
-			  AND $parent = preg_replace(',^bando_,','menu_',$parent)
-			  AND isset($boutons_admin[$parent])){
+			  AND $parent = preg_replace(',^bando_,', 'menu_', $parent)
+			  AND isset($boutons_admin[$parent])) {
 				if (!is_array($boutons_admin[$parent]->sousmenu))
 					$boutons_admin[$parent]->sousmenu = array();
 				$position = (isset($infos['position']) AND strlen($infos['position'])) ? intval($infos['position']) : count($boutons_admin[$parent]->sousmenu);
-				if ($position<0) $position = count($boutons_admin[$parent]->sousmenu)+1+$position;
-				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu,0,$position)
+				if ($position < 0) $position = count($boutons_admin[$parent]->sousmenu) + 1 + $position;
+				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu, 0, $position)
 				+ array($id=> new Bouton(
-					($icones AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
-					$infos['titre'],	// titre
-					(isset($infos['action']) AND $infos['action'])?$infos['action']:null,
-					(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
+					($icones AND $infos['icone']) ? find_in_theme($infos['icone']) : '', // icone
+					$infos['titre'], // titre
+					(isset($infos['action']) AND $infos['action']) ? $infos['action'] : null,
+					(isset($infos['parametres']) AND $infos['parametres']) ? $infos['parametres'] : null
 					))
-				+ array_slice($boutons_admin[$parent]->sousmenu,$position,100);
+				+ array_slice($boutons_admin[$parent]->sousmenu, $position, 100);
 			}
 			if (!$parent
 			// provisoire, eviter les vieux boutons
-			AND (!in_array($id,array('forum','statistiques_visites')))
+			AND (!in_array($id, array('forum', 'statistiques_visites')))
 
 			) {
 				$position = (isset($infos['position']) and $infos['position']) ? $infos['position'] : count($boutons_admin);
-				$boutons_admin = array_slice($boutons_admin,0,$position)
+				$boutons_admin = array_slice($boutons_admin, 0, $position)
 				+array($id=> new Bouton(
-					($icones AND isset($infos['icone']) AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
-					$infos['titre'],	// titre
-					(isset($infos['action']) AND $infos['action'])?$infos['action']:null,
-					(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:null
+					($icones AND isset($infos['icone']) AND $infos['icone']) ? find_in_theme($infos['icone']) : '', // icone
+					$infos['titre'], // titre
+					(isset($infos['action']) AND $infos['action']) ? $infos['action'] : null,
+					(isset($infos['parametres']) AND $infos['parametres']) ? $infos['parametres'] : null
 					))
-				+ array_slice($boutons_admin,$position,100);
+				+ array_slice($boutons_admin, $position, 100);
 			}
 		}
 	}
@@ -110,20 +110,20 @@ 
  * @return string
  */
 // http://doc.spip.org/@bandeau_creer_url
-function bandeau_creer_url($url, $args="", $contexte=null){
-	if (!preg_match(',[\/\?],',$url)) {
-		$url = generer_url_ecrire($url,$args,true);
+function bandeau_creer_url($url, $args = "", $contexte = null) {
+	if (!preg_match(',[\/\?],', $url)) {
+		$url = generer_url_ecrire($url, $args, true);
 		// recuperer les parametres du contexte demande par l'url sous la forme
 		// &truc=@machin@
 		// @machin@ etant remplace par _request('machin')
-		$url = str_replace('&amp;','&',$url);
-		while (preg_match(",[&?]([a-z_]+)=@([a-z_]+)@,i",$url,$matches)){
-			if ($matches[2]=='id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique']))
-				$contexte['id_secteur'] = sql_getfetsel('id_secteur','spip_rubriques','id_rubrique='.intval($contexte['id_rubrique']));
-			$val = _request($matches[2],$contexte);
-			$url = parametre_url($url,$matches[1],$val?$val:'','&');
+		$url = str_replace('&amp;', '&', $url);
+		while (preg_match(",[&?]([a-z_]+)=@([a-z_]+)@,i", $url, $matches)) {
+			if ($matches[2] == 'id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique']))
+				$contexte['id_secteur'] = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.intval($contexte['id_rubrique']));
+			$val = _request($matches[2], $contexte);
+			$url = parametre_url($url, $matches[1], $val ? $val : '', '&');
 		}
-		$url = str_replace('&','&amp;',$url);
+		$url = str_replace('&', '&amp;', $url);
 	}
 	return $url;
 }
@@ -138,7 +138,7 @@ 
  * @return unknown
  */
 function inc_bandeau_dist() {
-	return recuperer_fond('prive/squelettes/inclure/barre-nav',$_GET);
+	return recuperer_fond('prive/squelettes/inclure/barre-nav', $_GET);
 }
 
 ?>

Braces +26 added lines, -16 removed lines

@@ -10,18 +10,22 @@ 
  *  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');
 
 function definir_barre_contexte($contexte = null){
-	if (is_null($contexte))
-		$contexte = $_GET;
-	elseif(is_string($contexte))
-		$contexte = unserialize($contexte);
+	if (is_null($contexte)) {
+			$contexte = $_GET;
+	} elseif(is_string($contexte)) {
+			$contexte = unserialize($contexte);
+	}
 	if (!isset($contexte['id_rubrique']) AND isset($contexte['exec'])){
-		if (!function_exists('trouver_objet_exec'))
-			include_spip('inc/pipelines_ecrire');
+		if (!function_exists('trouver_objet_exec')) {
+					include_spip('inc/pipelines_ecrire');
+		}
 		if ($e=trouver_objet_exec($contexte['exec'])){
 			$_id = $e['id_table_objet'];
 			if (isset($contexte[$_id]) AND $id=intval($contexte[$_id])){
@@ -29,8 +33,9 @@ 
 				$row = sql_fetsel('*',$table,"$_id=".intval($id));
 				if (isset($row['id_rubrique'])){
 					$contexte['id_rubrique'] = $row['id_rubrique'];
-					if (isset($row['id_secteur']))
-						$contexte['id_secteur'] = $row['id_secteur'];
+					if (isset($row['id_secteur'])) {
+											$contexte['id_secteur'] = $row['id_secteur'];
+					}
 				}
 			}
 		}
@@ -58,8 +63,9 @@ 
 
 	// ajouter les boutons issus des plugin via plugin.xml
 	if (function_exists('boutons_plugins')
-	  AND is_array($liste_boutons_plugins = boutons_plugins()))
-		$liste_boutons = &$liste_boutons_plugins;
+	  AND is_array($liste_boutons_plugins = boutons_plugins())) {
+			$liste_boutons = &$liste_boutons_plugins;
+	}
 
 	foreach($liste_boutons as $id => $infos){
 		$parent = "";
@@ -68,10 +74,13 @@ 
 			if ($parent
 			  AND $parent = preg_replace(',^bando_,','menu_',$parent)
 			  AND isset($boutons_admin[$parent])){
-				if (!is_array($boutons_admin[$parent]->sousmenu))
-					$boutons_admin[$parent]->sousmenu = array();
+				if (!is_array($boutons_admin[$parent]->sousmenu)) {
+									$boutons_admin[$parent]->sousmenu = array();
+				}
 				$position = (isset($infos['position']) AND strlen($infos['position'])) ? intval($infos['position']) : count($boutons_admin[$parent]->sousmenu);
-				if ($position<0) $position = count($boutons_admin[$parent]->sousmenu)+1+$position;
+				if ($position<0) {
+				    $position = count($boutons_admin[$parent]->sousmenu)+1+$position;
+				}
 				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu,0,$position)
 				+ array($id=> new Bouton(
 					($icones AND $infos['icone'])?find_in_theme($infos['icone']):'',  // icone
@@ -118,8 +127,9 @@ 
 		// @machin@ etant remplace par _request('machin')
 		$url = str_replace('&amp;','&',$url);
 		while (preg_match(",[&?]([a-z_]+)=@([a-z_]+)@,i",$url,$matches)){
-			if ($matches[2]=='id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique']))
-				$contexte['id_secteur'] = sql_getfetsel('id_secteur','spip_rubriques','id_rubrique='.intval($contexte['id_rubrique']));
+			if ($matches[2]=='id_secteur' AND !isset($contexte['id_secteur']) AND isset($contexte['id_rubrique'])) {
+							$contexte['id_secteur'] = sql_getfetsel('id_secteur','spip_rubriques','id_rubrique='.intval($contexte['id_rubrique']));
+			}
 			$val = _request($matches[2],$contexte);
 			$url = parametre_url($url,$matches[1],$val?$val:'','&');
 		}

ecrire/inc/boutons.php

Doc Comments +4 added lines

@@ -73,6 +73,10 @@
 
 
 // http://doc.spip.org/@barre_onglets
+/**
+ * @param string $rubrique
+ * @param string $ongletCourant
+ */
 function barre_onglets($rubrique, $ongletCourant, $class="barre_onglet"){
 	include_spip('inc/presentation');
 

Indentation +46 added lines, -46 removed lines

@@ -18,24 +18,24 @@ 
  */
 // http://doc.spip.org/@Bouton
 class Bouton {
-  var $icone;         /* l'icone a mettre dans le bouton */
-  var $libelle;       /* le nom de l'entree d'i18n associe */
-  var $url= null;     /* l'url de la page (null => ?exec=nom) */
-  var $urlArg= null;  /* arguments supplementaires de l'url */
-  var $url2= null;    /* url jscript */
-  var $target= null;  /* pour ouvrir dans une fenetre a part */
-  var $sousmenu= null;/* sous barre de boutons / onglets */
+    var $icone;         /* l'icone a mettre dans le bouton */
+    var $libelle;       /* le nom de l'entree d'i18n associe */
+    var $url= null;     /* l'url de la page (null => ?exec=nom) */
+    var $urlArg= null;  /* arguments supplementaires de l'url */
+    var $url2= null;    /* url jscript */
+    var $target= null;  /* pour ouvrir dans une fenetre a part */
+    var $sousmenu= null;/* sous barre de boutons / onglets */
 
 // http://doc.spip.org/@Bouton
-  function Bouton($icone, $libelle, $url=null, $urlArg=null,
-				  $url2=null, $target=null) {
-	$this->icone  = $icone;
-	$this->libelle= $libelle;
-	$this->url    = $url;
-	$this->urlArg = $urlArg;
-	$this->url2   = $url2;
-	$this->target = $target;
-  }
+    function Bouton($icone, $libelle, $url=null, $urlArg=null,
+                    $url2=null, $target=null) {
+    $this->icone  = $icone;
+    $this->libelle= $libelle;
+    $this->url    = $url;
+    $this->urlArg = $urlArg;
+    $this->url2   = $url2;
+    $this->target = $target;
+    }
 }
 
 
@@ -46,44 +46,44 @@ 
 // http://doc.spip.org/@definir_barre_onglets
 function definir_barre_onglets($script) {
 
-	$onglets=array();
-	$liste_onglets = array();
-
-	// ajouter les onglets issus des plugin via plugin.xml
-	if (function_exists('onglets_plugins'))
-		$liste_onglets = onglets_plugins();
-
-
-	foreach($liste_onglets as $id => $infos){
-		if (($parent = $infos['parent'])
-			&& $parent == $script
-			&& autoriser('onglet',"_$id")) {
-				$onglets[$id] = new Bouton(
-					find_in_theme($infos['icone']),  // icone
-					$infos['titre'],	// titre
-					(isset($infos['action']) and $infos['action'])
-						? generer_url_ecrire($infos['action'],(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:'')
-						: null
-					);
-		}
-	}
-
-	return pipeline('ajouter_onglets', array('data'=>$onglets,'args'=>$script));
+    $onglets=array();
+    $liste_onglets = array();
+
+    // ajouter les onglets issus des plugin via plugin.xml
+    if (function_exists('onglets_plugins'))
+        $liste_onglets = onglets_plugins();
+
+
+    foreach($liste_onglets as $id => $infos){
+        if (($parent = $infos['parent'])
+            && $parent == $script
+            && autoriser('onglet',"_$id")) {
+                $onglets[$id] = new Bouton(
+                    find_in_theme($infos['icone']),  // icone
+                    $infos['titre'],	// titre
+                    (isset($infos['action']) and $infos['action'])
+                        ? generer_url_ecrire($infos['action'],(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:'')
+                        : null
+                    );
+        }
+    }
+
+    return pipeline('ajouter_onglets', array('data'=>$onglets,'args'=>$script));
 }
 
 
 // http://doc.spip.org/@barre_onglets
 function barre_onglets($rubrique, $ongletCourant, $class="barre_onglet"){
-	include_spip('inc/presentation');
+    include_spip('inc/presentation');
 
-	$res = '';
+    $res = '';
 
-	foreach(definir_barre_onglets($rubrique) as $exec => $onglet) {
-		$url= $onglet->url ? $onglet->url : generer_url_ecrire($exec);
-		$res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
-	}
+    foreach(definir_barre_onglets($rubrique) as $exec => $onglet) {
+        $url= $onglet->url ? $onglet->url : generer_url_ecrire($exec);
+        $res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
+    }
 
-	return  !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
+    return  !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
 }
 
 

Spacing +21 added lines, -21 removed lines

@@ -18,19 +18,19 @@ 
  */
 // http://doc.spip.org/@Bouton
 class Bouton {
-  var $icone;         /* l'icone a mettre dans le bouton */
-  var $libelle;       /* le nom de l'entree d'i18n associe */
-  var $url= null;     /* l'url de la page (null => ?exec=nom) */
-  var $urlArg= null;  /* arguments supplementaires de l'url */
-  var $url2= null;    /* url jscript */
-  var $target= null;  /* pour ouvrir dans une fenetre a part */
-  var $sousmenu= null;/* sous barre de boutons / onglets */
+  var $icone; /* l'icone a mettre dans le bouton */
+  var $libelle; /* le nom de l'entree d'i18n associe */
+  var $url = null; /* l'url de la page (null => ?exec=nom) */
+  var $urlArg = null; /* arguments supplementaires de l'url */
+  var $url2 = null; /* url jscript */
+  var $target = null; /* pour ouvrir dans une fenetre a part */
+  var $sousmenu = null; /* sous barre de boutons / onglets */
 
 // http://doc.spip.org/@Bouton
-  function Bouton($icone, $libelle, $url=null, $urlArg=null,
-				  $url2=null, $target=null) {
+  function Bouton($icone, $libelle, $url = null, $urlArg = null,
+				  $url2 = null, $target = null) {
 	$this->icone  = $icone;
-	$this->libelle= $libelle;
+	$this->libelle = $libelle;
 	$this->url    = $url;
 	$this->urlArg = $urlArg;
 	$this->url2   = $url2;
@@ -46,7 +46,7 @@ 
 // http://doc.spip.org/@definir_barre_onglets
 function definir_barre_onglets($script) {
 
-	$onglets=array();
+	$onglets = array();
 	$liste_onglets = array();
 
 	// ajouter les onglets issus des plugin via plugin.xml
@@ -54,36 +54,36 @@ 
 		$liste_onglets = onglets_plugins();
 
 
-	foreach($liste_onglets as $id => $infos){
+	foreach ($liste_onglets as $id => $infos) {
 		if (($parent = $infos['parent'])
 			&& $parent == $script
-			&& autoriser('onglet',"_$id")) {
+			&& autoriser('onglet', "_$id")) {
 				$onglets[$id] = new Bouton(
-					find_in_theme($infos['icone']),  // icone
-					$infos['titre'],	// titre
+					find_in_theme($infos['icone']), // icone
+					$infos['titre'], // titre
 					(isset($infos['action']) and $infos['action'])
-						? generer_url_ecrire($infos['action'],(isset($infos['parametres']) AND $infos['parametres'])?$infos['parametres']:'')
+						? generer_url_ecrire($infos['action'], (isset($infos['parametres']) AND $infos['parametres']) ? $infos['parametres'] : '')
 						: null
 					);
 		}
 	}
 
-	return pipeline('ajouter_onglets', array('data'=>$onglets,'args'=>$script));
+	return pipeline('ajouter_onglets', array('data'=>$onglets, 'args'=>$script));
 }
 
 
 // http://doc.spip.org/@barre_onglets
-function barre_onglets($rubrique, $ongletCourant, $class="barre_onglet"){
+function barre_onglets($rubrique, $ongletCourant, $class = "barre_onglet") {
 	include_spip('inc/presentation');
 
 	$res = '';
 
-	foreach(definir_barre_onglets($rubrique) as $exec => $onglet) {
-		$url= $onglet->url ? $onglet->url : generer_url_ecrire($exec);
+	foreach (definir_barre_onglets($rubrique) as $exec => $onglet) {
+		$url = $onglet->url ? $onglet->url : generer_url_ecrire($exec);
 		$res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
 	}
 
-	return  !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
+	return  !$res ? '' : (debut_onglet($class).$res.fin_onglet());
 }
 
 

Braces +6 added lines, -3 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;
+}
 
 /**
  * une classe definissant un bouton dans la barre du haut de l'interface
@@ -50,8 +52,9 @@ 
 	$liste_onglets = array();
 
 	// ajouter les onglets issus des plugin via plugin.xml
-	if (function_exists('onglets_plugins'))
-		$liste_onglets = onglets_plugins();
+	if (function_exists('onglets_plugins')) {
+			$liste_onglets = onglets_plugins();
+	}
 
 
 	foreach($liste_onglets as $id => $infos){

ecrire/inc/chercher_rubrique.php

Doc Comments +11 added lines

@@ -62,6 +62,11 @@ 
 }
 
 // http://doc.spip.org/@sous_menu_rubriques
+/**
+ * @param integer $root
+ * @param integer $niv
+ * @param integer $exclus
+ */
 function sous_menu_rubriques($id_rubrique, $root, $niv, &$data, &$enfants, $exclus, $restreint, $type) {
 	static $decalage_secteur;
 
@@ -206,6 +211,12 @@ 
 // getElement en mode Ajax est trop couteux).
 
 // http://doc.spip.org/@construire_selecteur
+/**
+ * @param string $url
+ * @param string $js
+ * @param string $idom
+ * @param string $name
+ */
 function construire_selecteur($url, $js, $idom, $name, $init='', $id=0)
 {
 	$icone = (strpos($idom, 'auteur')!==false) ? 'auteur-24.png' : 'rechercher-20.png';

Indentation +138 added lines, -138 removed lines

@@ -24,18 +24,18 @@ 
 // $idem : en mode rubrique = la rubrique soi-meme
 // http://doc.spip.org/@inc_chercher_rubrique_dist
 function inc_chercher_rubrique_dist ($id_rubrique, $type, $restreint, $idem=0, $do='aff') {
-	if (sql_countsel('spip_rubriques')<1)
-		return '';
+    if (sql_countsel('spip_rubriques')<1)
+        return '';
 
-	// Mode sans Ajax :
-	// - soit parce que le cookie ajax n'est pas la
-	// - soit parce qu'il y a peu de rubriques
-	if (_SPIP_AJAX < 1
-	OR $type == 'breve'
-	OR sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES)
-		return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
+    // Mode sans Ajax :
+    // - soit parce que le cookie ajax n'est pas la
+    // - soit parce qu'il y a peu de rubriques
+    if (_SPIP_AJAX < 1
+    OR $type == 'breve'
+    OR sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES)
+        return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
 
-	else return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
+    else return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
 
 }
 
@@ -44,119 +44,119 @@ 
 
 // http://doc.spip.org/@style_menu_rubriques
 function style_menu_rubriques($i) {
-	global $browser_name, $spip_lang_left;
-
-	$espace = '';
-	if (preg_match(",mozilla,i", $browser_name)) {
-		$style = "padding-$spip_lang_left: 16px; "
-		. "margin-$spip_lang_left: ".(($i-1)*16)."px;";
-	} else {
-		$style = '';
-		for ($count = 0; $count <= $i; $count ++)
-			$espace .= "&nbsp;&nbsp;&nbsp;&nbsp;";
-	}
-	if ($i ==1)
-		$espace= "";
-	$class = "niveau_$i";
-	return array($class,$style,$espace);
+    global $browser_name, $spip_lang_left;
+
+    $espace = '';
+    if (preg_match(",mozilla,i", $browser_name)) {
+        $style = "padding-$spip_lang_left: 16px; "
+        . "margin-$spip_lang_left: ".(($i-1)*16)."px;";
+    } else {
+        $style = '';
+        for ($count = 0; $count <= $i; $count ++)
+            $espace .= "&nbsp;&nbsp;&nbsp;&nbsp;";
+    }
+    if ($i ==1)
+        $espace= "";
+    $class = "niveau_$i";
+    return array($class,$style,$espace);
 }
 
 // http://doc.spip.org/@sous_menu_rubriques
 function sous_menu_rubriques($id_rubrique, $root, $niv, &$data, &$enfants, $exclus, $restreint, $type) {
-	static $decalage_secteur;
+    static $decalage_secteur;
 
-	// Si on a demande l'exclusion ne pas descendre dans la rubrique courante
-	if ($exclus > 0
-	AND $root == $exclus) return '';
+    // Si on a demande l'exclusion ne pas descendre dans la rubrique courante
+    if ($exclus > 0
+    AND $root == $exclus) return '';
 
-	// en fonction du niveau faire un affichage plus ou moins kikoo
+    // en fonction du niveau faire un affichage plus ou moins kikoo
 
-	// selected ?
-	$selected = ($root == $id_rubrique) ? ' selected="selected"' : '';
+    // selected ?
+    $selected = ($root == $id_rubrique) ? ' selected="selected"' : '';
 
-	// le style en fonction de la profondeur
-	list($class, $style, $espace) = style_menu_rubriques($niv);
+    // le style en fonction de la profondeur
+    list($class, $style, $espace) = style_menu_rubriques($niv);
 
-	$class .= " selec_rub";
+    $class .= " selec_rub";
 
-	// creer l'<option> pour la rubrique $root
+    // creer l'<option> pour la rubrique $root
 
-	if (isset($data[$root])) # pas de racine sauf pour les rubriques
-	{
-		$r = "<option$selected value='$root' class='$class' style='$style'>$espace"
-		.$data[$root]
-		.'</option>'."\n";
-	} else 	$r = '';
+    if (isset($data[$root])) # pas de racine sauf pour les rubriques
+    {
+        $r = "<option$selected value='$root' class='$class' style='$style'>$espace"
+        .$data[$root]
+        .'</option>'."\n";
+    } else 	$r = '';
 	
-	// et le sous-menu pour ses enfants
-	$sous = '';
-	if (isset($enfants[$root]))
-		foreach ($enfants[$root] as $sousrub)
-			$sous .= sous_menu_rubriques($id_rubrique, $sousrub,
-				$niv+1, $data, $enfants, $exclus, $restreint, $type);
-
-	// si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
-	if ($restreint AND $root!=$id_rubrique AND !autoriser('publierdans','rubrique',$root))
-		return $sous;
-
-	// et voila le travail
-	return $r.$sous;
+    // et le sous-menu pour ses enfants
+    $sous = '';
+    if (isset($enfants[$root]))
+        foreach ($enfants[$root] as $sousrub)
+            $sous .= sous_menu_rubriques($id_rubrique, $sousrub,
+                $niv+1, $data, $enfants, $exclus, $restreint, $type);
+
+    // si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
+    if ($restreint AND $root!=$id_rubrique AND !autoriser('publierdans','rubrique',$root))
+        return $sous;
+
+    // et voila le travail
+    return $r.$sous;
 }
 
 // Le selecteur de rubriques en mode classique (menu)
 // http://doc.spip.org/@selecteur_rubrique_html
 function selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem=0) {
-	$data = array();
-	if ($type == 'rubrique' AND autoriser('publierdans','rubrique',0))
-		$data[0] = _T('info_racine_site');
-	# premier choix = neant
-	# si auteur (rubriques restreintes)
-	# ou si creation avec id_rubrique=0
-	elseif ($type == 'auteur' OR !$id_rubrique)
-		$data[0] = '&nbsp;';
-
-	//
-	// creer une structure contenant toute l'arborescence
-	//
-
-	include_spip('base/abstract_sql');
-	$q = sql_select("id_rubrique, id_parent, titre, statut, lang, langue_choisie", "spip_rubriques", ($type == 'breve' ?  ' id_parent=0 ' : ''), '', "0+titre,titre");
-	while ($r = sql_fetch($q)) {
-		if (autoriser('voir','rubrique',$r['id_rubrique'])){
-			// titre largeur maxi a 50
-			$titre = couper(supprimer_tags(typo($r['titre']))." ", 50);
-			if ($GLOBALS['meta']['multi_rubriques'] == 'oui'
-			AND ($r['langue_choisie'] == "oui" OR $r['id_parent'] == 0))
-				$titre .= ' ['.traduire_nom_langue($r['lang']).']';
-			$data[$r['id_rubrique']] = $titre;
-			$enfants[$r['id_parent']][] = $r['id_rubrique'];
-			if ($id_rubrique == $r['id_rubrique']) $id_parent = $r['id_parent'];
-		}
-	}
-
-	// si une seule rubrique comme choix possible,
-	// inutile de mettre le selecteur sur un choix vide par defaut
-	// sauf si le selecteur s'adresse a une rubrique puisque on peut la mettre a la racine dans ce cas
-	if (count($data)==2
-	  AND isset($data[0])
-	  AND !in_array($type,array('auteur','rubrique'))
-	  AND !$id_rubrique)
-		unset($data[0]);
-
-
-	$opt = sous_menu_rubriques($id_rubrique,0, 0,$data,$enfants,$idem, $restreint, $type);
-	$att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
-
-	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,',$opt,$r))
-	  $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2] ;
-	else 
-	  $r = "<select".$att." size='1'>\n$opt</select>\n";
-
-	# message pour neuneus (a supprimer ?)
+    $data = array();
+    if ($type == 'rubrique' AND autoriser('publierdans','rubrique',0))
+        $data[0] = _T('info_racine_site');
+    # premier choix = neant
+    # si auteur (rubriques restreintes)
+    # ou si creation avec id_rubrique=0
+    elseif ($type == 'auteur' OR !$id_rubrique)
+        $data[0] = '&nbsp;';
+
+    //
+    // creer une structure contenant toute l'arborescence
+    //
+
+    include_spip('base/abstract_sql');
+    $q = sql_select("id_rubrique, id_parent, titre, statut, lang, langue_choisie", "spip_rubriques", ($type == 'breve' ?  ' id_parent=0 ' : ''), '', "0+titre,titre");
+    while ($r = sql_fetch($q)) {
+        if (autoriser('voir','rubrique',$r['id_rubrique'])){
+            // titre largeur maxi a 50
+            $titre = couper(supprimer_tags(typo($r['titre']))." ", 50);
+            if ($GLOBALS['meta']['multi_rubriques'] == 'oui'
+            AND ($r['langue_choisie'] == "oui" OR $r['id_parent'] == 0))
+                $titre .= ' ['.traduire_nom_langue($r['lang']).']';
+            $data[$r['id_rubrique']] = $titre;
+            $enfants[$r['id_parent']][] = $r['id_rubrique'];
+            if ($id_rubrique == $r['id_rubrique']) $id_parent = $r['id_parent'];
+        }
+    }
+
+    // si une seule rubrique comme choix possible,
+    // inutile de mettre le selecteur sur un choix vide par defaut
+    // sauf si le selecteur s'adresse a une rubrique puisque on peut la mettre a la racine dans ce cas
+    if (count($data)==2
+      AND isset($data[0])
+      AND !in_array($type,array('auteur','rubrique'))
+      AND !$id_rubrique)
+        unset($data[0]);
+
+
+    $opt = sous_menu_rubriques($id_rubrique,0, 0,$data,$enfants,$idem, $restreint, $type);
+    $att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
+
+    if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,',$opt,$r))
+        $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2] ;
+    else 
+        $r = "<select".$att." size='1'>\n$opt</select>\n";
+
+    # message pour neuneus (a supprimer ?)
 #	if ($type != 'auteur' AND $type != 'breve')
 #		$r .= "\n<br />"._T('texte_rappel_selection_champs');
 
-	return $r;
+    return $r;
 }
 
 /**
@@ -179,24 +179,24 @@ 
  */
 function selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem=0, $do) {
 
-	if ($id_rubrique) {
-		$titre = sql_fetsel("titre", "spip_rubriques", "id_rubrique=$id_rubrique");
-		$titre = $titre['titre'];
-	} else if ($type == 'auteur')
-		$titre = '&nbsp;';
-	else
-		$titre = _T('info_racine_site');
+    if ($id_rubrique) {
+        $titre = sql_fetsel("titre", "spip_rubriques", "id_rubrique=$id_rubrique");
+        $titre = $titre['titre'];
+    } else if ($type == 'auteur')
+        $titre = '&nbsp;';
+    else
+        $titre = _T('info_racine_site');
 
-	$titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
-	$init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
+    $titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
+    $init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
 
-	$url = generer_url_ecrire('selectionner',"id=$id_rubrique&type=$type&do=$do"
-	. (!$idem ? '' : "&exclus=$idem")
-	. ($restreint ? "" : "&racine=oui")
-	. (isset($GLOBALS['var_profile']) ? '&var_profile=1' : ''));
+    $url = generer_url_ecrire('selectionner',"id=$id_rubrique&type=$type&do=$do"
+    . (!$idem ? '' : "&exclus=$idem")
+    . ($restreint ? "" : "&racine=oui")
+    . (isset($GLOBALS['var_profile']) ? '&var_profile=1' : ''));
 
 
-	return construire_selecteur($url, '', 'selection_rubrique', 'id_parent', $init, $id_rubrique);
+    return construire_selecteur($url, '', 'selection_rubrique', 'id_parent', $init, $id_rubrique);
 }
 
 // construit un bloc comportant une icone clicable avec image animee a cote
@@ -208,26 +208,26 @@ 
 // http://doc.spip.org/@construire_selecteur
 function construire_selecteur($url, $js, $idom, $name, $init='', $id=0)
 {
-	$icone = (strpos($idom, 'auteur')!==false) ? 'auteur-24.png' : 'rechercher-20.png';
-	return
- 	"<div class='rubrique_actuelle'><a onclick=\""
-	.  $js
-	. "return charger_node_url_si_vide('"
-	. $url
-	. "', this.parentNode.nextSibling, this.nextSibling,'',event)\"><img src='"
-	. chemin_image($icone)
-	. "'\nstyle='vertical-align: middle;' alt='"._T('titre_image_selecteur')."' /></a><img src='"
-	. chemin_image('searching.gif') 
-	. "' id='img_"
-	.  $idom
-	. "'\nstyle='visibility: hidden;' alt='*' />"
-	. "<input id='titreparent' name='titreparent'"
-	. $init
-	. " />" 
-	. "<input type='hidden' id='$name' name='$name' value='"
-	. $id
-	. "' /><div class='nettoyeur'></div></div><div id='"
-	. $idom
-	. "'\nstyle='display: none;'></div>";
+    $icone = (strpos($idom, 'auteur')!==false) ? 'auteur-24.png' : 'rechercher-20.png';
+    return
+        "<div class='rubrique_actuelle'><a onclick=\""
+    .  $js
+    . "return charger_node_url_si_vide('"
+    . $url
+    . "', this.parentNode.nextSibling, this.nextSibling,'',event)\"><img src='"
+    . chemin_image($icone)
+    . "'\nstyle='vertical-align: middle;' alt='"._T('titre_image_selecteur')."' /></a><img src='"
+    . chemin_image('searching.gif') 
+    . "' id='img_"
+    .  $idom
+    . "'\nstyle='visibility: hidden;' alt='*' />"
+    . "<input id='titreparent' name='titreparent'"
+    . $init
+    . " />" 
+    . "<input type='hidden' id='$name' name='$name' value='"
+    . $id
+    . "' /><div class='nettoyeur'></div></div><div id='"
+    . $idom
+    . "'\nstyle='display: none;'></div>";
 }
 ?>

Spacing +23 added lines, -23 removed lines

@@ -23,8 +23,8 @@ 
 //    n'importe ou (defaut), et les breves dans les secteurs.
 // $idem : en mode rubrique = la rubrique soi-meme
 // http://doc.spip.org/@inc_chercher_rubrique_dist
-function inc_chercher_rubrique_dist ($id_rubrique, $type, $restreint, $idem=0, $do='aff') {
-	if (sql_countsel('spip_rubriques')<1)
+function inc_chercher_rubrique_dist($id_rubrique, $type, $restreint, $idem = 0, $do = 'aff') {
+	if (sql_countsel('spip_rubriques') < 1)
 		return '';
 
 	// Mode sans Ajax :
@@ -49,16 +49,16 @@ 
 	$espace = '';
 	if (preg_match(",mozilla,i", $browser_name)) {
 		$style = "padding-$spip_lang_left: 16px; "
-		. "margin-$spip_lang_left: ".(($i-1)*16)."px;";
+		. "margin-$spip_lang_left: ".(($i - 1) * 16)."px;";
 	} else {
 		$style = '';
-		for ($count = 0; $count <= $i; $count ++)
+		for ($count = 0; $count <= $i; $count++)
 			$espace .= "&nbsp;&nbsp;&nbsp;&nbsp;";
 	}
-	if ($i ==1)
-		$espace= "";
+	if ($i == 1)
+		$espace = "";
 	$class = "niveau_$i";
-	return array($class,$style,$espace);
+	return array($class, $style, $espace);
 }
 
 // http://doc.spip.org/@sous_menu_rubriques
@@ -93,10 +93,10 @@ 
 	if (isset($enfants[$root]))
 		foreach ($enfants[$root] as $sousrub)
 			$sous .= sous_menu_rubriques($id_rubrique, $sousrub,
-				$niv+1, $data, $enfants, $exclus, $restreint, $type);
+				$niv + 1, $data, $enfants, $exclus, $restreint, $type);
 
 	// si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
-	if ($restreint AND $root!=$id_rubrique AND !autoriser('publierdans','rubrique',$root))
+	if ($restreint AND $root != $id_rubrique AND !autoriser('publierdans', 'rubrique', $root))
 		return $sous;
 
 	// et voila le travail
@@ -105,9 +105,9 @@ 
 
 // Le selecteur de rubriques en mode classique (menu)
 // http://doc.spip.org/@selecteur_rubrique_html
-function selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem=0) {
+function selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem = 0) {
 	$data = array();
-	if ($type == 'rubrique' AND autoriser('publierdans','rubrique',0))
+	if ($type == 'rubrique' AND autoriser('publierdans', 'rubrique', 0))
 		$data[0] = _T('info_racine_site');
 	# premier choix = neant
 	# si auteur (rubriques restreintes)
@@ -120,9 +120,9 @@ 
 	//
 
 	include_spip('base/abstract_sql');
-	$q = sql_select("id_rubrique, id_parent, titre, statut, lang, langue_choisie", "spip_rubriques", ($type == 'breve' ?  ' id_parent=0 ' : ''), '', "0+titre,titre");
+	$q = sql_select("id_rubrique, id_parent, titre, statut, lang, langue_choisie", "spip_rubriques", ($type == 'breve' ? ' id_parent=0 ' : ''), '', "0+titre,titre");
 	while ($r = sql_fetch($q)) {
-		if (autoriser('voir','rubrique',$r['id_rubrique'])){
+		if (autoriser('voir', 'rubrique', $r['id_rubrique'])) {
 			// titre largeur maxi a 50
 			$titre = couper(supprimer_tags(typo($r['titre']))." ", 50);
 			if ($GLOBALS['meta']['multi_rubriques'] == 'oui'
@@ -137,18 +137,18 @@ 
 	// si une seule rubrique comme choix possible,
 	// inutile de mettre le selecteur sur un choix vide par defaut
 	// sauf si le selecteur s'adresse a une rubrique puisque on peut la mettre a la racine dans ce cas
-	if (count($data)==2
+	if (count($data) == 2
 	  AND isset($data[0])
-	  AND !in_array($type,array('auteur','rubrique'))
+	  AND !in_array($type, array('auteur', 'rubrique'))
 	  AND !$id_rubrique)
 		unset($data[0]);
 
 
-	$opt = sous_menu_rubriques($id_rubrique,0, 0,$data,$enfants,$idem, $restreint, $type);
+	$opt = sous_menu_rubriques($id_rubrique, 0, 0, $data, $enfants, $idem, $restreint, $type);
 	$att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
 
-	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,',$opt,$r))
-	  $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2] ;
+	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,', $opt, $r))
+	  $r = "<input$att type='hidden' value='".$r[1]."' />".$r[2];
 	else 
 	  $r = "<select".$att." size='1'>\n$opt</select>\n";
 
@@ -177,7 +177,7 @@ 
  * @param string $do
  * @return string
  */
-function selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem=0, $do) {
+function selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem = 0, $do) {
 
 	if ($id_rubrique) {
 		$titre = sql_fetsel("titre", "spip_rubriques", "id_rubrique=$id_rubrique");
@@ -188,9 +188,9 @@ 
 		$titre = _T('info_racine_site');
 
 	$titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
-	$init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
+	$init = " disabled='disabled' type='text' value=\"".$titre."\"\nstyle='width:300px;'";
 
-	$url = generer_url_ecrire('selectionner',"id=$id_rubrique&type=$type&do=$do"
+	$url = generer_url_ecrire('selectionner', "id=$id_rubrique&type=$type&do=$do"
 	. (!$idem ? '' : "&exclus=$idem")
 	. ($restreint ? "" : "&racine=oui")
 	. (isset($GLOBALS['var_profile']) ? '&var_profile=1' : ''));
@@ -206,9 +206,9 @@ 
 // getElement en mode Ajax est trop couteux).
 
 // http://doc.spip.org/@construire_selecteur
-function construire_selecteur($url, $js, $idom, $name, $init='', $id=0)
+function construire_selecteur($url, $js, $idom, $name, $init = '', $id = 0)
 {
-	$icone = (strpos($idom, 'auteur')!==false) ? 'auteur-24.png' : 'rechercher-20.png';
+	$icone = (strpos($idom, 'auteur') !== false) ? 'auteur-24.png' : 'rechercher-20.png';
 	return
  	"<div class='rubrique_actuelle'><a onclick=\""
 	.  $js

Braces +57 added lines, -35 removed lines

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

ecrire/inc/config.php

Doc Comments +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
  * @param string $cfg
  * @param mixed $def
  * @param bool $unserialize
- * @return mixed
+ * @return string
  */
 function lire_config_metapack_dist($cfg='', $def=null, $unserialize=true) {
 	return lire_config($cfg, $def, $unserialize);

Indentation +339 added lines, -339 removed lines

@@ -21,7 +21,7 @@ 
  */
 // http://doc.spip.org/@inc_config_dist
 function inc_config_dist() {
-	actualise_metas(liste_metas());
+    actualise_metas(liste_metas());
 }
 
 /**
@@ -32,27 +32,27 @@ 
  * @return array
  */
 function expliquer_config($cfg){
-	// par defaut, sur la table des meta
-	$table = 'meta';
-	$casier = null;
-	$sous_casier = array();
-	if (strlen($cfg)){
-		$cfg = explode('/',$cfg);
-		// si le premier argument est vide, c'est une syntaxe /table/ ou un appel vide ''
-		if (!reset($cfg) AND count($cfg)>1) {
-			array_shift($cfg);
-			$table = array_shift($cfg);
-			if (!isset($GLOBALS[$table]))
-				lire_metas($table);
-		}
-		// si on a demande #CONFIG{/meta,'',0}
-		if (count($cfg))
-			$casier = array_shift($cfg);
-
-		if (count($cfg))
-			$sous_casier = $cfg;
-	}
-	return array($table,$casier,$sous_casier);
+    // par defaut, sur la table des meta
+    $table = 'meta';
+    $casier = null;
+    $sous_casier = array();
+    if (strlen($cfg)){
+        $cfg = explode('/',$cfg);
+        // si le premier argument est vide, c'est une syntaxe /table/ ou un appel vide ''
+        if (!reset($cfg) AND count($cfg)>1) {
+            array_shift($cfg);
+            $table = array_shift($cfg);
+            if (!isset($GLOBALS[$table]))
+                lire_metas($table);
+        }
+        // si on a demande #CONFIG{/meta,'',0}
+        if (count($cfg))
+            $casier = array_shift($cfg);
+
+        if (count($cfg))
+            $sous_casier = $cfg;
+    }
+    return array($table,$casier,$sous_casier);
 }
 
 /**
@@ -75,56 +75,56 @@ 
  * @return string
  */
 function lire_config($cfg='', $def=null, $unserialize=true) {
-	// lire le stockage sous la forme /table/valeur
-	// ou valeur qui est en fait implicitement /meta/valeur
-	// ou casier/valeur qui est en fait implicitement /meta/casier/valeur
-
-	// traiter en priorite le cas simple et frequent
-	// de lecture direct $GLOBALS['meta']['truc'], si $cfg ne contient ni / ni :
-	if ($cfg AND strpbrk($cfg,'/:')===false){
-		$r = isset($GLOBALS['meta'][$cfg])?
-		  ((!$unserialize
-			// ne pas essayer de deserialiser autre chose qu'une chaine
-			OR !is_string($GLOBALS['meta'][$cfg])
-			// ne pas essayer de deserialiser si ce n'est visiblement pas une chaine serializee
-			OR strpos($GLOBALS['meta'][$cfg],':')===false
-			OR ($t=unserialize($GLOBALS['meta'][$cfg]))===false)?$GLOBALS['meta'][$cfg]:$t)
-		  :$def;
-		return $r;
-	}
-
-	// Brancher sur methodes externes si besoin
-	if ($cfg AND $p=strpos($cfg,'::')){
-		$methode = substr($cfg,0,$p);
-		$lire_config = charger_fonction($methode, 'lire_config');
-		return $lire_config(substr($cfg,$p+2),$def,$unserialize);
-	}
-
-	list($table,$casier,$sous_casier) = expliquer_config($cfg);
-
-	if (!isset($GLOBALS[$table]))
-			return $def;
-
-	$r = $GLOBALS[$table];
-
-	// si on a demande #CONFIG{/meta,'',0}
-	if (!$casier)
-		return $unserialize ? $r : serialize($r);
-
-	// casier principal :
-	// le deserializer si demande
-	// ou si on a besoin
-	// d'un sous casier
-	$r = isset($r[$casier])?$r[$casier]:null;
-	if (($unserialize OR count($sous_casier)) AND $r AND is_string($r))
-		$r = (($t=unserialize($r))===false?$r:$t);
-
-	// aller chercher le sous_casier
-	while(!is_null($r) AND $casier = array_shift($sous_casier))
-		$r = isset($r[$casier])?$r[$casier]:null;
-
-	if (is_null($r)) return $def;
-	return $r;
+    // lire le stockage sous la forme /table/valeur
+    // ou valeur qui est en fait implicitement /meta/valeur
+    // ou casier/valeur qui est en fait implicitement /meta/casier/valeur
+
+    // traiter en priorite le cas simple et frequent
+    // de lecture direct $GLOBALS['meta']['truc'], si $cfg ne contient ni / ni :
+    if ($cfg AND strpbrk($cfg,'/:')===false){
+        $r = isset($GLOBALS['meta'][$cfg])?
+          ((!$unserialize
+            // ne pas essayer de deserialiser autre chose qu'une chaine
+            OR !is_string($GLOBALS['meta'][$cfg])
+            // ne pas essayer de deserialiser si ce n'est visiblement pas une chaine serializee
+            OR strpos($GLOBALS['meta'][$cfg],':')===false
+            OR ($t=unserialize($GLOBALS['meta'][$cfg]))===false)?$GLOBALS['meta'][$cfg]:$t)
+          :$def;
+        return $r;
+    }
+
+    // Brancher sur methodes externes si besoin
+    if ($cfg AND $p=strpos($cfg,'::')){
+        $methode = substr($cfg,0,$p);
+        $lire_config = charger_fonction($methode, 'lire_config');
+        return $lire_config(substr($cfg,$p+2),$def,$unserialize);
+    }
+
+    list($table,$casier,$sous_casier) = expliquer_config($cfg);
+
+    if (!isset($GLOBALS[$table]))
+            return $def;
+
+    $r = $GLOBALS[$table];
+
+    // si on a demande #CONFIG{/meta,'',0}
+    if (!$casier)
+        return $unserialize ? $r : serialize($r);
+
+    // casier principal :
+    // le deserializer si demande
+    // ou si on a besoin
+    // d'un sous casier
+    $r = isset($r[$casier])?$r[$casier]:null;
+    if (($unserialize OR count($sous_casier)) AND $r AND is_string($r))
+        $r = (($t=unserialize($r))===false?$r:$t);
+
+    // aller chercher le sous_casier
+    while(!is_null($r) AND $casier = array_shift($sous_casier))
+        $r = isset($r[$casier])?$r[$casier]:null;
+
+    if (is_null($r)) return $def;
+    return $r;
 }
 
 /**
@@ -138,7 +138,7 @@ 
  * @return mixed
  */
 function lire_config_metapack_dist($cfg='', $def=null, $unserialize=true) {
-	return lire_config($cfg, $def, $unserialize);
+    return lire_config($cfg, $def, $unserialize);
 }
 
 
@@ -150,93 +150,93 @@ 
  * @return bool
  */
 function ecrire_config($cfg,$store) {
-	// Brancher sur methodes externes si besoin
-	if ($cfg AND $p=strpos($cfg,'::')){
-		$methode = substr($cfg,0,$p);
-		$ecrire_config = charger_fonction($methode, 'ecrire_config');
-		return $ecrire_config(substr($cfg,$p+2),$store);
-	}
+    // Brancher sur methodes externes si besoin
+    if ($cfg AND $p=strpos($cfg,'::')){
+        $methode = substr($cfg,0,$p);
+        $ecrire_config = charger_fonction($methode, 'ecrire_config');
+        return $ecrire_config(substr($cfg,$p+2),$store);
+    }
 	
-	list($table,$casier,$sous_casier) = expliquer_config($cfg);
-	// il faut au moins un casier pour ecrire
-	if (!$casier) return false;
-
-	// trouvons ou creons le pointeur sur le casier
-	$st = isset($GLOBALS[$table][$casier])?$GLOBALS[$table][$casier]:null;
-	if (!is_array($st) AND ($sous_casier OR is_array($store))) {
-		$st = unserialize($st);
-		if ($st===false) {
-			// ne rien creer si c'est une demande d'effacement
-			if (is_null($store))
-				return false;
-			$st=array();
-		}
-	}
-
-	// si on a affaire a un sous caiser
-	// il faut ecrire au bon endroit sans perdre les autres sous casier freres
-	if ($c = $sous_casier) {
-		$sc = &$st;
-		$pointeurs = array();
-		while (count($c) AND $cc=array_shift($c)) {
-			// creer l'entree si elle n'existe pas
-			if (!isset($sc[$cc])) {
-				// si on essaye d'effacer une config qui n'existe pas
-				// ne rien creer mais sortir
-				if (is_null($store))
-					return false;
-				$sc[$cc] = array();
-			}
-			$pointeurs[$cc] = &$sc;
-			$sc = &$sc[$cc];
-		}
-
-		// si c'est une demande d'effacement
-		if (is_null($store)){
-			$c = $sous_casier;
-			$sous = array_pop($c);
-			// effacer, et remonter pour effacer les parents vides
-			do {
-				unset($pointeurs[$sous][$sous]);
-			} while ($sous = array_pop($c) AND !count($pointeurs[$sous][$sous]));
+    list($table,$casier,$sous_casier) = expliquer_config($cfg);
+    // il faut au moins un casier pour ecrire
+    if (!$casier) return false;
+
+    // trouvons ou creons le pointeur sur le casier
+    $st = isset($GLOBALS[$table][$casier])?$GLOBALS[$table][$casier]:null;
+    if (!is_array($st) AND ($sous_casier OR is_array($store))) {
+        $st = unserialize($st);
+        if ($st===false) {
+            // ne rien creer si c'est une demande d'effacement
+            if (is_null($store))
+                return false;
+            $st=array();
+        }
+    }
+
+    // si on a affaire a un sous caiser
+    // il faut ecrire au bon endroit sans perdre les autres sous casier freres
+    if ($c = $sous_casier) {
+        $sc = &$st;
+        $pointeurs = array();
+        while (count($c) AND $cc=array_shift($c)) {
+            // creer l'entree si elle n'existe pas
+            if (!isset($sc[$cc])) {
+                // si on essaye d'effacer une config qui n'existe pas
+                // ne rien creer mais sortir
+                if (is_null($store))
+                    return false;
+                $sc[$cc] = array();
+            }
+            $pointeurs[$cc] = &$sc;
+            $sc = &$sc[$cc];
+        }
+
+        // si c'est une demande d'effacement
+        if (is_null($store)){
+            $c = $sous_casier;
+            $sous = array_pop($c);
+            // effacer, et remonter pour effacer les parents vides
+            do {
+                unset($pointeurs[$sous][$sous]);
+            } while ($sous = array_pop($c) AND !count($pointeurs[$sous][$sous]));
 			
-			// si on a vide tous les sous casiers,
-			// et que le casier est vide
-			// vider aussi la meta
-			if (!$sous AND !count($st))
-				$st = null;
-		}
-		// dans tous les autres cas, on ecrase
-		else
-			$sc = $store;
-
-		// Maintenant que $st est modifiee
-		// reprenons la comme valeur a stocker dans le casier principal
-		$store = $st;
-	}
-
-	if (is_null($store)) {
-		if (is_null($st) AND !$sous_casier)
-			return false; // la config n'existait deja pas !
-		effacer_meta ($casier, $table);
-		if (!count($GLOBALS[$table])
-			OR count($GLOBALS[$table])==1 AND isset($GLOBALS[$table]['charset'])) {
-			effacer_meta('charset', $table); // enlevons cette meta
-			supprimer_table_meta($table); // supprimons la table (si elle est bien vide)
-		}
-	}
-	// les meta ne peuvent etre que des chaines : il faut serializer le reste
-	else {
-		if (!isset($GLOBALS[$table]))
-			installer_table_meta($table);
-		// si ce n'est pas une chaine
-		// il faut serializer
-		if (!is_string($store))
-			$store=serialize($store);
-		ecrire_meta($casier, $store, null, $table);
-	}
-	// verifier que lire_config($cfg)==$store ?
-	return true;
+            // si on a vide tous les sous casiers,
+            // et que le casier est vide
+            // vider aussi la meta
+            if (!$sous AND !count($st))
+                $st = null;
+        }
+        // dans tous les autres cas, on ecrase
+        else
+            $sc = $store;
+
+        // Maintenant que $st est modifiee
+        // reprenons la comme valeur a stocker dans le casier principal
+        $store = $st;
+    }
+
+    if (is_null($store)) {
+        if (is_null($st) AND !$sous_casier)
+            return false; // la config n'existait deja pas !
+        effacer_meta ($casier, $table);
+        if (!count($GLOBALS[$table])
+            OR count($GLOBALS[$table])==1 AND isset($GLOBALS[$table]['charset'])) {
+            effacer_meta('charset', $table); // enlevons cette meta
+            supprimer_table_meta($table); // supprimons la table (si elle est bien vide)
+        }
+    }
+    // les meta ne peuvent etre que des chaines : il faut serializer le reste
+    else {
+        if (!isset($GLOBALS[$table]))
+            installer_table_meta($table);
+        // si ce n'est pas une chaine
+        // il faut serializer
+        if (!is_string($store))
+            $store=serialize($store);
+        ecrire_meta($casier, $store, null, $table);
+    }
+    // verifier que lire_config($cfg)==$store ?
+    return true;
 }
 
 
@@ -250,12 +250,12 @@ 
  * @return bool
  */
 function ecrire_config_metapack_dist($cfg,$store) {
-	// cas particulier en metapack::
-	// si on ecrit une chaine deja serializee, il faut la reserializer pour la rendre
-	// intacte en sortie ...
-	if (is_string($store) AND strpos($store,':') AND unserialize($store))
-		$store = serialize($store);
-	return ecrire_config($cfg, $store);
+    // cas particulier en metapack::
+    // si on ecrit une chaine deja serializee, il faut la reserializer pour la rendre
+    // intacte en sortie ...
+    if (is_string($store) AND strpos($store,':') AND unserialize($store))
+        $store = serialize($store);
+    return ecrire_config($cfg, $store);
 }
 
 /**
@@ -264,75 +264,75 @@ 
  * @return bool
  */
 function effacer_config($cfg){
-	ecrire_config($cfg, null);
-	return true;
+    ecrire_config($cfg, null);
+    return true;
 }
 
 
 function lister_configurer($exclure = array()){
-	return array();
+    return array();
 	
-	// lister les pages de config deja dans les menus
-	$deja = array();
-	foreach($exclure as $id=>$b) {
-		$url = ($b['url'] ? $b['url'] : $id);
-		if (!$b['url'] or !isset($exclure[$url])) {
-			if (strncmp($url,'configurer_',11)==0) {
-				$deja[$url] = $b;
-			} elseif($b['url']=='configurer' AND preg_match(',cfg=([a-z0-9_]+),i',$b['args'],$match)) {
-				$deja["configurer_".$match[1]] = $b;
-			}
-		}
+    // lister les pages de config deja dans les menus
+    $deja = array();
+    foreach($exclure as $id=>$b) {
+        $url = ($b['url'] ? $b['url'] : $id);
+        if (!$b['url'] or !isset($exclure[$url])) {
+            if (strncmp($url,'configurer_',11)==0) {
+                $deja[$url] = $b;
+            } elseif($b['url']=='configurer' AND preg_match(',cfg=([a-z0-9_]+),i',$b['args'],$match)) {
+                $deja["configurer_".$match[1]] = $b;
+            }
+        }
 		
-	}
-	$exclure = $exclure + $deja;
-
-	$icone_defaut = "images/configuration-16.png";
-	$liste = array();
-	$skels = array();
-	$forms = array();
-
-	// trouver toutes les pages configurer_xxx de l'espace prive
-	// et construire un tableau des entrees qui ne sont pas dans $deja
-	$pages = find_all_in_path("prive/squelettes/contenu/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
-
-	foreach($pages as $page) {
-		$configurer = basename($page,"."._EXTENSION_SQUELETTES);
-		if (!isset($exclure[$configurer])) {
-			$liste[$configurer] = array(
-					'parent' => 'bando_configuration',
-					'url' => $configurer,
-					'titre' => _T("configurer:{$configurer}_titre"),
-					'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
-			);
-		}
-		$skels[$configurer] = $page;
-	}
-
-	// analyser la liste des $skels pour voir les #FORMULAIRE_CONFIGURER_ inclus
-	foreach($skels as $file) {
-		$forms = array_merge($forms, lister_formulaires_configurer($file));
-	}
-	$forms = array_flip($forms);
-
-	// trouver tous les formulaires/configurer_
-	// et construire un tableau des entrees
-	$pages = find_all_in_path("formulaires/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
-	foreach($pages as $page) {
-		$configurer = basename($page,"."._EXTENSION_SQUELETTES);
-		if (!isset($forms[$configurer])
-		  AND !isset($liste[$configurer])
-			AND !isset($exclure[$configurer]))
-			$liste[$configurer] = array(
-					'parent' => 'bando_configuration',
-					'url' => 'configurer',
-					'args' => 'cfg='.substr($configurer,11),
-					'titre' => _T("configurer:{$configurer}_titre"),
-					'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
-			);
-	}
-
-	return $liste;
+    }
+    $exclure = $exclure + $deja;
+
+    $icone_defaut = "images/configuration-16.png";
+    $liste = array();
+    $skels = array();
+    $forms = array();
+
+    // trouver toutes les pages configurer_xxx de l'espace prive
+    // et construire un tableau des entrees qui ne sont pas dans $deja
+    $pages = find_all_in_path("prive/squelettes/contenu/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
+
+    foreach($pages as $page) {
+        $configurer = basename($page,"."._EXTENSION_SQUELETTES);
+        if (!isset($exclure[$configurer])) {
+            $liste[$configurer] = array(
+                    'parent' => 'bando_configuration',
+                    'url' => $configurer,
+                    'titre' => _T("configurer:{$configurer}_titre"),
+                    'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
+            );
+        }
+        $skels[$configurer] = $page;
+    }
+
+    // analyser la liste des $skels pour voir les #FORMULAIRE_CONFIGURER_ inclus
+    foreach($skels as $file) {
+        $forms = array_merge($forms, lister_formulaires_configurer($file));
+    }
+    $forms = array_flip($forms);
+
+    // trouver tous les formulaires/configurer_
+    // et construire un tableau des entrees
+    $pages = find_all_in_path("formulaires/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
+    foreach($pages as $page) {
+        $configurer = basename($page,"."._EXTENSION_SQUELETTES);
+        if (!isset($forms[$configurer])
+          AND !isset($liste[$configurer])
+            AND !isset($exclure[$configurer]))
+            $liste[$configurer] = array(
+                    'parent' => 'bando_configuration',
+                    'url' => 'configurer',
+                    'args' => 'cfg='.substr($configurer,11),
+                    'titre' => _T("configurer:{$configurer}_titre"),
+                    'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
+            );
+    }
+
+    return $liste;
 }
 
 
@@ -344,92 +344,92 @@ 
  * @return array liste des formulaires trouves
 **/
 function lister_formulaires_configurer($file) {
-	$forms = array();
+    $forms = array();
 	
-	lire_fichier($file, $skel);
-	if (preg_match_all(",#FORMULAIRE_(CONFIGURER_[A-Z0-9_]*),", $skel, $matches,PREG_SET_ORDER)) {
-		$matches = array_map('end',$matches);
-		$matches = array_map('strtolower',$matches);
-		$forms = array_merge($forms,$matches);
-	}
-
-	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
-	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
-	$config = basename(substr($file,0,-strlen("."._EXTENSION_SQUELETTES)));
-	spip_log('Calcul de '."prive/squelettes/contenu/$config");
-	$fond = recuperer_fond("prive/squelettes/contenu/$config", array("exec" => $config));
+    lire_fichier($file, $skel);
+    if (preg_match_all(",#FORMULAIRE_(CONFIGURER_[A-Z0-9_]*),", $skel, $matches,PREG_SET_ORDER)) {
+        $matches = array_map('end',$matches);
+        $matches = array_map('strtolower',$matches);
+        $forms = array_merge($forms,$matches);
+    }
+
+    // evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
+    // et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
+    $config = basename(substr($file,0,-strlen("."._EXTENSION_SQUELETTES)));
+    spip_log('Calcul de '."prive/squelettes/contenu/$config");
+    $fond = recuperer_fond("prive/squelettes/contenu/$config", array("exec" => $config));
 	
-	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...
-	// et donc que l'on puisse les referencer aussi !
-	$fond = pipeline('affiche_milieu', array('args'=>array("exec" => $config),'data'=>$fond));
-
-	// recuperer les noms des formulaires presents.
-	if (is_array($inputs = extraire_balises($fond,"input"))) {
-		foreach($inputs as $i) {
-			if (extraire_attribut($i,'name')=='formulaire_action') {
-				$forms[] = ($c=extraire_attribut($i,'value'));
-			}
-		}
-	}
-	return $forms;
+    // passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...
+    // et donc que l'on puisse les referencer aussi !
+    $fond = pipeline('affiche_milieu', array('args'=>array("exec" => $config),'data'=>$fond));
+
+    // recuperer les noms des formulaires presents.
+    if (is_array($inputs = extraire_balises($fond,"input"))) {
+        foreach($inputs as $i) {
+            if (extraire_attribut($i,'name')=='formulaire_action') {
+                $forms[] = ($c=extraire_attribut($i,'value'));
+            }
+        }
+    }
+    return $forms;
 }
 
 
 // http://doc.spip.org/@liste_metas
 function liste_metas()
 {
-	return pipeline('configurer_liste_metas', array(
-		'nom_site' => _T('info_mon_site_spip'),
-		'slogan_site' => '',
-		'adresse_site' => preg_replace(",/$,", "", url_de_base()),
-		'descriptif_site' => '',
-		'activer_logos' => 'oui',
-		'activer_logos_survol' => 'non',
-		'articles_surtitre' => 'non',
-		'articles_soustitre' => 'non',
-		'articles_descriptif' => 'non',
-		'articles_chapeau' => 'non',
-		'articles_texte' => 'oui',
-		'articles_ps' => 'non',
-		'articles_redac' => 'non',
-		'post_dates' => 'non',
-		'articles_urlref' => 'non',
-		'articles_redirection' => 'non',
-		'creer_preview' => 'non',
-		'taille_preview' => 150,
-		'articles_modif' => 'non',
-
-		'rubriques_descriptif' => 'non',
-		'rubriques_texte' => 'oui',
-
-		'accepter_inscriptions' => 'non',
-		'accepter_visiteurs' => 'non',
-		'prevenir_auteurs' => 'non',
-		'suivi_edito' => 'non',
-		'adresse_suivi' =>'',
-		'adresse_suivi_inscription' =>'',
-		'adresse_neuf' => '',
-		'jours_neuf' => '',
-		'quoi_de_neuf' => 'non',
-		'preview' => ',0minirezo,1comite,',
-
-		'syndication_integrale' => 'oui',
-		'charset' => _DEFAULT_CHARSET,
-		'dir_img' => substr(_DIR_IMG,strlen(_DIR_RACINE)),
-
-		'multi_rubriques' => 'non',
-		'multi_secteurs' => 'non',
-		'gerer_trad' => 'non',
-		'langues_multilingue' => '',
-
-		'version_html_max' => 'html4',
-
-		'type_urls' => 'page',
-
-		'email_envoi' => '',
-		'email_webmaster' => '',
-		'auto_compress_http'=>'non',
-	));
+    return pipeline('configurer_liste_metas', array(
+        'nom_site' => _T('info_mon_site_spip'),
+        'slogan_site' => '',
+        'adresse_site' => preg_replace(",/$,", "", url_de_base()),
+        'descriptif_site' => '',
+        'activer_logos' => 'oui',
+        'activer_logos_survol' => 'non',
+        'articles_surtitre' => 'non',
+        'articles_soustitre' => 'non',
+        'articles_descriptif' => 'non',
+        'articles_chapeau' => 'non',
+        'articles_texte' => 'oui',
+        'articles_ps' => 'non',
+        'articles_redac' => 'non',
+        'post_dates' => 'non',
+        'articles_urlref' => 'non',
+        'articles_redirection' => 'non',
+        'creer_preview' => 'non',
+        'taille_preview' => 150,
+        'articles_modif' => 'non',
+
+        'rubriques_descriptif' => 'non',
+        'rubriques_texte' => 'oui',
+
+        'accepter_inscriptions' => 'non',
+        'accepter_visiteurs' => 'non',
+        'prevenir_auteurs' => 'non',
+        'suivi_edito' => 'non',
+        'adresse_suivi' =>'',
+        'adresse_suivi_inscription' =>'',
+        'adresse_neuf' => '',
+        'jours_neuf' => '',
+        'quoi_de_neuf' => 'non',
+        'preview' => ',0minirezo,1comite,',
+
+        'syndication_integrale' => 'oui',
+        'charset' => _DEFAULT_CHARSET,
+        'dir_img' => substr(_DIR_IMG,strlen(_DIR_RACINE)),
+
+        'multi_rubriques' => 'non',
+        'multi_secteurs' => 'non',
+        'gerer_trad' => 'non',
+        'langues_multilingue' => '',
+
+        'version_html_max' => 'html4',
+
+        'type_urls' => 'page',
+
+        'email_envoi' => '',
+        'email_webmaster' => '',
+        'auto_compress_http'=>'non',
+    ));
 }
 
 // mets les meta a des valeurs conventionnelles quand elles sont vides
@@ -438,21 +438,21 @@ 
 // http://doc.spip.org/@actualise_metas
 function actualise_metas($liste_meta)
 {
-	$meta_serveur =
-		array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
-	'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
-	// verifier le impt=non
-	sql_updateq('spip_meta',array('impt'=>'non'),sql_in('nom',$meta_serveur));
-
-	while (list($nom, $valeur) = each($liste_meta)) {
-		if (!$GLOBALS['meta'][$nom]) {
-			ecrire_meta($nom, $valeur);
-		}
-	}
-
-	include_spip('inc/rubriques');
-	$langues = calculer_langues_utilisees();
-	ecrire_meta('langues_utilisees', $langues);
+    $meta_serveur =
+        array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
+    'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
+    // verifier le impt=non
+    sql_updateq('spip_meta',array('impt'=>'non'),sql_in('nom',$meta_serveur));
+
+    while (list($nom, $valeur) = each($liste_meta)) {
+        if (!$GLOBALS['meta'][$nom]) {
+            ecrire_meta($nom, $valeur);
+        }
+    }
+
+    include_spip('inc/rubriques');
+    $langues = calculer_langues_utilisees();
+    ecrire_meta('langues_utilisees', $langues);
 }
 
 
@@ -464,17 +464,17 @@ 
 // http://doc.spip.org/@appliquer_modifs_config
 function appliquer_modifs_config($purger_skel=false) {
 
-	foreach(liste_metas() as $i => $v) {
-		if (($x =_request($i))!==NULL)
-			ecrire_meta($i, $x);
-		elseif  (!isset($GLOBALS['meta'][$i]))
-			ecrire_meta($i, $v);
-	}
-
-	if ($purger_skel) {
-		include_spip('inc/invalideur');
-		purger_repertoire(_DIR_SKELS);
-	}
+    foreach(liste_metas() as $i => $v) {
+        if (($x =_request($i))!==NULL)
+            ecrire_meta($i, $x);
+        elseif  (!isset($GLOBALS['meta'][$i]))
+            ecrire_meta($i, $v);
+    }
+
+    if ($purger_skel) {
+        include_spip('inc/invalideur');
+        purger_repertoire(_DIR_SKELS);
+    }
 }
 
 /**
@@ -485,16 +485,16 @@ 
  * @return void
  */
 function appliquer_adresse_site($adresse_site){
-	if ($adresse_site!==NULL){
-		if (!strlen($adresse_site)) {$GLOBALS['profondeur_url']=_DIR_RESTREINT?0:1;$adresse_site = url_de_base();}
-		$adresse_site = preg_replace(",/?\s*$,", "", $adresse_site);
+    if ($adresse_site!==NULL){
+        if (!strlen($adresse_site)) {$GLOBALS['profondeur_url']=_DIR_RESTREINT?0:1;$adresse_site = url_de_base();}
+        $adresse_site = preg_replace(",/?\s*$,", "", $adresse_site);
 
-		if (!preg_match(",^[\w]+://,Uims",$adresse_site))
-			$adresse_site = "http://$adresse_site";
+        if (!preg_match(",^[\w]+://,Uims",$adresse_site))
+            $adresse_site = "http://$adresse_site";
 
-		ecrire_meta('adresse_site',$adresse_site);
-	}
-	return $adresse_site;
+        ecrire_meta('adresse_site',$adresse_site);
+    }
+    return $adresse_site;
 }
 
 ?>

Spacing +70 added lines, -70 removed lines

@@ -31,15 +31,15 @@ 
  * @param string $cfg
  * @return array
  */
-function expliquer_config($cfg){
+function expliquer_config($cfg) {
 	// par defaut, sur la table des meta
 	$table = 'meta';
 	$casier = null;
 	$sous_casier = array();
-	if (strlen($cfg)){
-		$cfg = explode('/',$cfg);
+	if (strlen($cfg)) {
+		$cfg = explode('/', $cfg);
 		// si le premier argument est vide, c'est une syntaxe /table/ ou un appel vide ''
-		if (!reset($cfg) AND count($cfg)>1) {
+		if (!reset($cfg) AND count($cfg) > 1) {
 			array_shift($cfg);
 			$table = array_shift($cfg);
 			if (!isset($GLOBALS[$table]))
@@ -52,7 +52,7 @@ 
 		if (count($cfg))
 			$sous_casier = $cfg;
 	}
-	return array($table,$casier,$sous_casier);
+	return array($table, $casier, $sous_casier);
 }
 
 /**
@@ -74,33 +74,33 @@ 
  * @param  boolean $unserialize  n'affecte que le dépôt 'meta'
  * @return string
  */
-function lire_config($cfg='', $def=null, $unserialize=true) {
+function lire_config($cfg = '', $def = null, $unserialize = true) {
 	// lire le stockage sous la forme /table/valeur
 	// ou valeur qui est en fait implicitement /meta/valeur
 	// ou casier/valeur qui est en fait implicitement /meta/casier/valeur
 
 	// traiter en priorite le cas simple et frequent
 	// de lecture direct $GLOBALS['meta']['truc'], si $cfg ne contient ni / ni :
-	if ($cfg AND strpbrk($cfg,'/:')===false){
-		$r = isset($GLOBALS['meta'][$cfg])?
+	if ($cfg AND strpbrk($cfg, '/:') === false) {
+		$r = isset($GLOBALS['meta'][$cfg]) ?
 		  ((!$unserialize
 			// ne pas essayer de deserialiser autre chose qu'une chaine
 			OR !is_string($GLOBALS['meta'][$cfg])
 			// ne pas essayer de deserialiser si ce n'est visiblement pas une chaine serializee
-			OR strpos($GLOBALS['meta'][$cfg],':')===false
-			OR ($t=unserialize($GLOBALS['meta'][$cfg]))===false)?$GLOBALS['meta'][$cfg]:$t)
+			OR strpos($GLOBALS['meta'][$cfg], ':') === false
+			OR ($t = unserialize($GLOBALS['meta'][$cfg])) === false) ? $GLOBALS['meta'][$cfg] : $t)
 		  :$def;
 		return $r;
 	}
 
 	// Brancher sur methodes externes si besoin
-	if ($cfg AND $p=strpos($cfg,'::')){
-		$methode = substr($cfg,0,$p);
+	if ($cfg AND $p = strpos($cfg, '::')) {
+		$methode = substr($cfg, 0, $p);
 		$lire_config = charger_fonction($methode, 'lire_config');
-		return $lire_config(substr($cfg,$p+2),$def,$unserialize);
+		return $lire_config(substr($cfg, $p + 2), $def, $unserialize);
 	}
 
-	list($table,$casier,$sous_casier) = expliquer_config($cfg);
+	list($table, $casier, $sous_casier) = expliquer_config($cfg);
 
 	if (!isset($GLOBALS[$table]))
 			return $def;
@@ -115,13 +115,13 @@ 
 	// le deserializer si demande
 	// ou si on a besoin
 	// d'un sous casier
-	$r = isset($r[$casier])?$r[$casier]:null;
+	$r = isset($r[$casier]) ? $r[$casier] : null;
 	if (($unserialize OR count($sous_casier)) AND $r AND is_string($r))
-		$r = (($t=unserialize($r))===false?$r:$t);
+		$r = (($t = unserialize($r)) === false ? $r : $t);
 
 	// aller chercher le sous_casier
-	while(!is_null($r) AND $casier = array_shift($sous_casier))
-		$r = isset($r[$casier])?$r[$casier]:null;
+	while (!is_null($r) AND $casier = array_shift($sous_casier))
+		$r = isset($r[$casier]) ? $r[$casier] : null;
 
 	if (is_null($r)) return $def;
 	return $r;
@@ -137,7 +137,7 @@ 
  * @param bool $unserialize
  * @return mixed
  */
-function lire_config_metapack_dist($cfg='', $def=null, $unserialize=true) {
+function lire_config_metapack_dist($cfg = '', $def = null, $unserialize = true) {
 	return lire_config($cfg, $def, $unserialize);
 }
 
@@ -149,27 +149,27 @@ 
  * @param mixed $store
  * @return bool
  */
-function ecrire_config($cfg,$store) {
+function ecrire_config($cfg, $store) {
 	// Brancher sur methodes externes si besoin
-	if ($cfg AND $p=strpos($cfg,'::')){
-		$methode = substr($cfg,0,$p);
+	if ($cfg AND $p = strpos($cfg, '::')) {
+		$methode = substr($cfg, 0, $p);
 		$ecrire_config = charger_fonction($methode, 'ecrire_config');
-		return $ecrire_config(substr($cfg,$p+2),$store);
+		return $ecrire_config(substr($cfg, $p + 2), $store);
 	}
 	
-	list($table,$casier,$sous_casier) = expliquer_config($cfg);
+	list($table, $casier, $sous_casier) = expliquer_config($cfg);
 	// il faut au moins un casier pour ecrire
 	if (!$casier) return false;
 
 	// trouvons ou creons le pointeur sur le casier
-	$st = isset($GLOBALS[$table][$casier])?$GLOBALS[$table][$casier]:null;
+	$st = isset($GLOBALS[$table][$casier]) ? $GLOBALS[$table][$casier] : null;
 	if (!is_array($st) AND ($sous_casier OR is_array($store))) {
 		$st = unserialize($st);
-		if ($st===false) {
+		if ($st === false) {
 			// ne rien creer si c'est une demande d'effacement
 			if (is_null($store))
 				return false;
-			$st=array();
+			$st = array();
 		}
 	}
 
@@ -178,7 +178,7 @@ 
 	if ($c = $sous_casier) {
 		$sc = &$st;
 		$pointeurs = array();
-		while (count($c) AND $cc=array_shift($c)) {
+		while (count($c) AND $cc = array_shift($c)) {
 			// creer l'entree si elle n'existe pas
 			if (!isset($sc[$cc])) {
 				// si on essaye d'effacer une config qui n'existe pas
@@ -192,7 +192,7 @@ 
 		}
 
 		// si c'est une demande d'effacement
-		if (is_null($store)){
+		if (is_null($store)) {
 			$c = $sous_casier;
 			$sous = array_pop($c);
 			// effacer, et remonter pour effacer les parents vides
@@ -218,9 +218,9 @@ 
 	if (is_null($store)) {
 		if (is_null($st) AND !$sous_casier)
 			return false; // la config n'existait deja pas !
-		effacer_meta ($casier, $table);
+		effacer_meta($casier, $table);
 		if (!count($GLOBALS[$table])
-			OR count($GLOBALS[$table])==1 AND isset($GLOBALS[$table]['charset'])) {
+			OR count($GLOBALS[$table]) == 1 AND isset($GLOBALS[$table]['charset'])) {
 			effacer_meta('charset', $table); // enlevons cette meta
 			supprimer_table_meta($table); // supprimons la table (si elle est bien vide)
 		}
@@ -232,7 +232,7 @@ 
 		// si ce n'est pas une chaine
 		// il faut serializer
 		if (!is_string($store))
-			$store=serialize($store);
+			$store = serialize($store);
 		ecrire_meta($casier, $store, null, $table);
 	}
 	// verifier que lire_config($cfg)==$store ?
@@ -249,11 +249,11 @@ 
  * @param mixed $store
  * @return bool
  */
-function ecrire_config_metapack_dist($cfg,$store) {
+function ecrire_config_metapack_dist($cfg, $store) {
 	// cas particulier en metapack::
 	// si on ecrit une chaine deja serializee, il faut la reserializer pour la rendre
 	// intacte en sortie ...
-	if (is_string($store) AND strpos($store,':') AND unserialize($store))
+	if (is_string($store) AND strpos($store, ':') AND unserialize($store))
 		$store = serialize($store);
 	return ecrire_config($cfg, $store);
 }
@@ -263,23 +263,23 @@ 
  * @param string $cfg
  * @return bool
  */
-function effacer_config($cfg){
+function effacer_config($cfg) {
 	ecrire_config($cfg, null);
 	return true;
 }
 
 
-function lister_configurer($exclure = array()){
+function lister_configurer($exclure = array()) {
 	return array();
 	
 	// lister les pages de config deja dans les menus
 	$deja = array();
-	foreach($exclure as $id=>$b) {
+	foreach ($exclure as $id=>$b) {
 		$url = ($b['url'] ? $b['url'] : $id);
 		if (!$b['url'] or !isset($exclure[$url])) {
-			if (strncmp($url,'configurer_',11)==0) {
+			if (strncmp($url, 'configurer_', 11) == 0) {
 				$deja[$url] = $b;
-			} elseif($b['url']=='configurer' AND preg_match(',cfg=([a-z0-9_]+),i',$b['args'],$match)) {
+			} elseif ($b['url'] == 'configurer' AND preg_match(',cfg=([a-z0-9_]+),i', $b['args'], $match)) {
 				$deja["configurer_".$match[1]] = $b;
 			}
 		}
@@ -296,21 +296,21 @@ 
 	// et construire un tableau des entrees qui ne sont pas dans $deja
 	$pages = find_all_in_path("prive/squelettes/contenu/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
 
-	foreach($pages as $page) {
-		$configurer = basename($page,"."._EXTENSION_SQUELETTES);
+	foreach ($pages as $page) {
+		$configurer = basename($page, "."._EXTENSION_SQUELETTES);
 		if (!isset($exclure[$configurer])) {
 			$liste[$configurer] = array(
 					'parent' => 'bando_configuration',
 					'url' => $configurer,
 					'titre' => _T("configurer:{$configurer}_titre"),
-					'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
+					'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
 			);
 		}
 		$skels[$configurer] = $page;
 	}
 
 	// analyser la liste des $skels pour voir les #FORMULAIRE_CONFIGURER_ inclus
-	foreach($skels as $file) {
+	foreach ($skels as $file) {
 		$forms = array_merge($forms, lister_formulaires_configurer($file));
 	}
 	$forms = array_flip($forms);
@@ -318,17 +318,17 @@ 
 	// trouver tous les formulaires/configurer_
 	// et construire un tableau des entrees
 	$pages = find_all_in_path("formulaires/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
-	foreach($pages as $page) {
-		$configurer = basename($page,"."._EXTENSION_SQUELETTES);
+	foreach ($pages as $page) {
+		$configurer = basename($page, "."._EXTENSION_SQUELETTES);
 		if (!isset($forms[$configurer])
 		  AND !isset($liste[$configurer])
 			AND !isset($exclure[$configurer]))
 			$liste[$configurer] = array(
 					'parent' => 'bando_configuration',
 					'url' => 'configurer',
-					'args' => 'cfg='.substr($configurer,11),
+					'args' => 'cfg='.substr($configurer, 11),
 					'titre' => _T("configurer:{$configurer}_titre"),
-					'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
+					'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
 			);
 	}
 
@@ -347,27 +347,27 @@ 
 	$forms = array();
 	
 	lire_fichier($file, $skel);
-	if (preg_match_all(",#FORMULAIRE_(CONFIGURER_[A-Z0-9_]*),", $skel, $matches,PREG_SET_ORDER)) {
-		$matches = array_map('end',$matches);
-		$matches = array_map('strtolower',$matches);
-		$forms = array_merge($forms,$matches);
+	if (preg_match_all(",#FORMULAIRE_(CONFIGURER_[A-Z0-9_]*),", $skel, $matches, PREG_SET_ORDER)) {
+		$matches = array_map('end', $matches);
+		$matches = array_map('strtolower', $matches);
+		$forms = array_merge($forms, $matches);
 	}
 
 	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
 	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
-	$config = basename(substr($file,0,-strlen("."._EXTENSION_SQUELETTES)));
+	$config = basename(substr($file, 0, -strlen("."._EXTENSION_SQUELETTES)));
 	spip_log('Calcul de '."prive/squelettes/contenu/$config");
 	$fond = recuperer_fond("prive/squelettes/contenu/$config", array("exec" => $config));
 	
 	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...
 	// et donc que l'on puisse les referencer aussi !
-	$fond = pipeline('affiche_milieu', array('args'=>array("exec" => $config),'data'=>$fond));
+	$fond = pipeline('affiche_milieu', array('args'=>array("exec" => $config), 'data'=>$fond));
 
 	// recuperer les noms des formulaires presents.
-	if (is_array($inputs = extraire_balises($fond,"input"))) {
-		foreach($inputs as $i) {
-			if (extraire_attribut($i,'name')=='formulaire_action') {
-				$forms[] = ($c=extraire_attribut($i,'value'));
+	if (is_array($inputs = extraire_balises($fond, "input"))) {
+		foreach ($inputs as $i) {
+			if (extraire_attribut($i, 'name') == 'formulaire_action') {
+				$forms[] = ($c = extraire_attribut($i, 'value'));
 			}
 		}
 	}
@@ -415,7 +415,7 @@ 
 
 		'syndication_integrale' => 'oui',
 		'charset' => _DEFAULT_CHARSET,
-		'dir_img' => substr(_DIR_IMG,strlen(_DIR_RACINE)),
+		'dir_img' => substr(_DIR_IMG, strlen(_DIR_RACINE)),
 
 		'multi_rubriques' => 'non',
 		'multi_secteurs' => 'non',
@@ -439,10 +439,10 @@ 
 function actualise_metas($liste_meta)
 {
 	$meta_serveur =
-		array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
-	'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
+		array('version_installee', 'adresse_site', 'alea_ephemere_ancien', 'alea_ephemere', 'alea_ephemere_date', 'langue_site', 'langues_proposees', 'date_calcul_rubriques', 'derniere_modif', 'optimiser_table', 'drapeau_edition', 'creer_preview', 'taille_preview', 'creer_htpasswd', 'creer_htaccess', 'gd_formats_read', 'gd_formats',
+	'netpbm_formats', 'formats_graphiques', 'image_process', 'plugin_header', 'plugin');
 	// verifier le impt=non
-	sql_updateq('spip_meta',array('impt'=>'non'),sql_in('nom',$meta_serveur));
+	sql_updateq('spip_meta', array('impt'=>'non'), sql_in('nom', $meta_serveur));
 
 	while (list($nom, $valeur) = each($liste_meta)) {
 		if (!$GLOBALS['meta'][$nom]) {
@@ -462,12 +462,12 @@ 
 //
 
 // http://doc.spip.org/@appliquer_modifs_config
-function appliquer_modifs_config($purger_skel=false) {
+function appliquer_modifs_config($purger_skel = false) {
 
-	foreach(liste_metas() as $i => $v) {
-		if (($x =_request($i))!==NULL)
+	foreach (liste_metas() as $i => $v) {
+		if (($x = _request($i)) !== NULL)
 			ecrire_meta($i, $x);
-		elseif  (!isset($GLOBALS['meta'][$i]))
+		elseif (!isset($GLOBALS['meta'][$i]))
 			ecrire_meta($i, $v);
 	}
 
@@ -484,15 +484,15 @@ 
  * @param  $adresse_site
  * @return void
  */
-function appliquer_adresse_site($adresse_site){
-	if ($adresse_site!==NULL){
-		if (!strlen($adresse_site)) {$GLOBALS['profondeur_url']=_DIR_RESTREINT?0:1;$adresse_site = url_de_base();}
+function appliquer_adresse_site($adresse_site) {
+	if ($adresse_site !== NULL) {
+		if (!strlen($adresse_site)) {$GLOBALS['profondeur_url'] = _DIR_RESTREINT ? 0 : 1; $adresse_site = url_de_base(); }
 		$adresse_site = preg_replace(",/?\s*$,", "", $adresse_site);
 
-		if (!preg_match(",^[\w]+://,Uims",$adresse_site))
+		if (!preg_match(",^[\w]+://,Uims", $adresse_site))
 			$adresse_site = "http://$adresse_site";
 
-		ecrire_meta('adresse_site',$adresse_site);
+		ecrire_meta('adresse_site', $adresse_site);
 	}
 	return $adresse_site;
 }

Braces +66 added lines, -41 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;
+}
 
 
 /**
@@ -42,15 +44,18 @@ 
 		if (!reset($cfg) AND count($cfg)>1) {
 			array_shift($cfg);
 			$table = array_shift($cfg);
-			if (!isset($GLOBALS[$table]))
-				lire_metas($table);
+			if (!isset($GLOBALS[$table])) {
+							lire_metas($table);
+			}
 		}
 		// si on a demande #CONFIG{/meta,'',0}
-		if (count($cfg))
-			$casier = array_shift($cfg);
+		if (count($cfg)) {
+					$casier = array_shift($cfg);
+		}
 
-		if (count($cfg))
-			$sous_casier = $cfg;
+		if (count($cfg)) {
+					$sous_casier = $cfg;
+		}
 	}
 	return array($table,$casier,$sous_casier);
 }
@@ -102,28 +107,34 @@ 
 
 	list($table,$casier,$sous_casier) = expliquer_config($cfg);
 
-	if (!isset($GLOBALS[$table]))
-			return $def;
+	if (!isset($GLOBALS[$table])) {
+				return $def;
+	}
 
 	$r = $GLOBALS[$table];
 
 	// si on a demande #CONFIG{/meta,'',0}
-	if (!$casier)
-		return $unserialize ? $r : serialize($r);
+	if (!$casier) {
+			return $unserialize ? $r : serialize($r);
+	}
 
 	// casier principal :
 	// le deserializer si demande
 	// ou si on a besoin
 	// d'un sous casier
 	$r = isset($r[$casier])?$r[$casier]:null;
-	if (($unserialize OR count($sous_casier)) AND $r AND is_string($r))
-		$r = (($t=unserialize($r))===false?$r:$t);
+	if (($unserialize OR count($sous_casier)) AND $r AND is_string($r)) {
+			$r = (($t=unserialize($r))===false?$r:$t);
+	}
 
 	// aller chercher le sous_casier
-	while(!is_null($r) AND $casier = array_shift($sous_casier))
-		$r = isset($r[$casier])?$r[$casier]:null;
+	while(!is_null($r) AND $casier = array_shift($sous_casier)) {
+			$r = isset($r[$casier])?$r[$casier]:null;
+	}
 
-	if (is_null($r)) return $def;
+	if (is_null($r)) {
+	    return $def;
+	}
 	return $r;
 }
 
@@ -159,7 +170,9 @@ 
 	
 	list($table,$casier,$sous_casier) = expliquer_config($cfg);
 	// il faut au moins un casier pour ecrire
-	if (!$casier) return false;
+	if (!$casier) {
+	    return false;
+	}
 
 	// trouvons ou creons le pointeur sur le casier
 	$st = isset($GLOBALS[$table][$casier])?$GLOBALS[$table][$casier]:null;
@@ -167,8 +180,9 @@ 
 		$st = unserialize($st);
 		if ($st===false) {
 			// ne rien creer si c'est une demande d'effacement
-			if (is_null($store))
-				return false;
+			if (is_null($store)) {
+							return false;
+			}
 			$st=array();
 		}
 	}
@@ -183,8 +197,9 @@ 
 			if (!isset($sc[$cc])) {
 				// si on essaye d'effacer une config qui n'existe pas
 				// ne rien creer mais sortir
-				if (is_null($store))
-					return false;
+				if (is_null($store)) {
+									return false;
+				}
 				$sc[$cc] = array();
 			}
 			$pointeurs[$cc] = &$sc;
@@ -203,12 +218,14 @@ 
 			// si on a vide tous les sous casiers,
 			// et que le casier est vide
 			// vider aussi la meta
-			if (!$sous AND !count($st))
-				$st = null;
+			if (!$sous AND !count($st)) {
+							$st = null;
+			}
 		}
 		// dans tous les autres cas, on ecrase
-		else
-			$sc = $store;
+		else {
+					$sc = $store;
+		}
 
 		// Maintenant que $st est modifiee
 		// reprenons la comme valeur a stocker dans le casier principal
@@ -216,8 +233,10 @@ 
 	}
 
 	if (is_null($store)) {
-		if (is_null($st) AND !$sous_casier)
-			return false; // la config n'existait deja pas !
+		if (is_null($st) AND !$sous_casier) {
+					return false;
+		}
+		// la config n'existait deja pas !
 		effacer_meta ($casier, $table);
 		if (!count($GLOBALS[$table])
 			OR count($GLOBALS[$table])==1 AND isset($GLOBALS[$table]['charset'])) {
@@ -227,12 +246,14 @@ 
 	}
 	// les meta ne peuvent etre que des chaines : il faut serializer le reste
 	else {
-		if (!isset($GLOBALS[$table]))
-			installer_table_meta($table);
+		if (!isset($GLOBALS[$table])) {
+					installer_table_meta($table);
+		}
 		// si ce n'est pas une chaine
 		// il faut serializer
-		if (!is_string($store))
-			$store=serialize($store);
+		if (!is_string($store)) {
+					$store=serialize($store);
+		}
 		ecrire_meta($casier, $store, null, $table);
 	}
 	// verifier que lire_config($cfg)==$store ?
@@ -253,8 +274,9 @@ 
 	// cas particulier en metapack::
 	// si on ecrit une chaine deja serializee, il faut la reserializer pour la rendre
 	// intacte en sortie ...
-	if (is_string($store) AND strpos($store,':') AND unserialize($store))
-		$store = serialize($store);
+	if (is_string($store) AND strpos($store,':') AND unserialize($store)) {
+			$store = serialize($store);
+	}
 	return ecrire_config($cfg, $store);
 }
 
@@ -322,14 +344,15 @@ 
 		$configurer = basename($page,"."._EXTENSION_SQUELETTES);
 		if (!isset($forms[$configurer])
 		  AND !isset($liste[$configurer])
-			AND !isset($exclure[$configurer]))
-			$liste[$configurer] = array(
+			AND !isset($exclure[$configurer])) {
+					$liste[$configurer] = array(
 					'parent' => 'bando_configuration',
 					'url' => 'configurer',
 					'args' => 'cfg='.substr($configurer,11),
 					'titre' => _T("configurer:{$configurer}_titre"),
 					'icone' => find_in_theme($i="images/{$configurer}-16.png")?$i:$icone_defaut,
 			);
+		}
 	}
 
 	return $liste;
@@ -465,10 +488,11 @@ 
 function appliquer_modifs_config($purger_skel=false) {
 
 	foreach(liste_metas() as $i => $v) {
-		if (($x =_request($i))!==NULL)
-			ecrire_meta($i, $x);
-		elseif  (!isset($GLOBALS['meta'][$i]))
-			ecrire_meta($i, $v);
+		if (($x =_request($i))!==NULL) {
+					ecrire_meta($i, $x);
+		} elseif  (!isset($GLOBALS['meta'][$i])) {
+					ecrire_meta($i, $v);
+		}
 	}
 
 	if ($purger_skel) {
@@ -489,8 +513,9 @@ 
 		if (!strlen($adresse_site)) {$GLOBALS['profondeur_url']=_DIR_RESTREINT?0:1;$adresse_site = url_de_base();}
 		$adresse_site = preg_replace(",/?\s*$,", "", $adresse_site);
 
-		if (!preg_match(",^[\w]+://,Uims",$adresse_site))
-			$adresse_site = "http://$adresse_site";
+		if (!preg_match(",^[\w]+://,Uims",$adresse_site)) {
+					$adresse_site = "http://$adresse_site";
+		}
 
 		ecrire_meta('adresse_site',$adresse_site);
 	}