spip / SPIP

ecrire/public/phraser_html.php

Spacing +143 added lines, -143 removed lines

@@ -27,21 +27,21 @@ 
 define('SPEC_BOUCLE', '/\s*\(\s*([^\s?)]+)(\s*[^)?]*)([?]?)\)/');
 define('NOM_DE_BOUCLE', "[0-9]+|[-_][-_.a-zA-Z0-9]*");
 # ecriture alambiquee pour rester compatible avec les hexadecimaux des vieux squelettes
-define('NOM_DE_CHAMP', "#((" . NOM_DE_BOUCLE . "):)?(([A-F]*[G-Z_][A-Z_0-9]*)|[A-Z_]+)(\*{0,2})");
-define('CHAMP_ETENDU', '/\[([^]\[]*)\(' . NOM_DE_CHAMP . '([^[)]*\)[^]\[]*)\]/S');
+define('NOM_DE_CHAMP', "#((".NOM_DE_BOUCLE."):)?(([A-F]*[G-Z_][A-Z_0-9]*)|[A-Z_]+)(\*{0,2})");
+define('CHAMP_ETENDU', '/\[([^]\[]*)\('.NOM_DE_CHAMP.'([^[)]*\)[^]\[]*)\]/S');
 
 define('BALISE_INCLURE', '/<INCLU[DR]E[[:space:]]*(\(([^)]*)\))?/S');
 define('BALISE_POLYGLOTTE', ',<multi>(.*)</multi>,Uims');
 define('BALISE_IDIOMES', ',<:(([a-z0-9_]+):)?([a-z0-9_]*)({([^\|=>]*=[^\|>]*)})?((\|[^>]*)?:>),iS');
-define('BALISE_IDIOMES_ARGS', '@^\s*([^= ]*)\s*=\s*((' . NOM_DE_CHAMP . '[{][^}]*})?[^,]*)\s*,?\s*@s');
+define('BALISE_IDIOMES_ARGS', '@^\s*([^= ]*)\s*=\s*(('.NOM_DE_CHAMP.'[{][^}]*})?[^,]*)\s*,?\s*@s');
 
 define('SQL_ARGS', '(\([^)]*\))');
-define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)' . SQL_ARGS . '?`?');
+define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)'.SQL_ARGS.'?`?');
 
 // http://doc.spip.org/@phraser_inclure
-function phraser_inclure($texte, $ligne, $result){
+function phraser_inclure($texte, $ligne, $result) {
 
-	while (preg_match(BALISE_INCLURE, $texte, $match)){
+	while (preg_match(BALISE_INCLURE, $texte, $match)) {
 		$p = strpos($texte, $match[0]);
 		$debut = substr($texte, 0, $p);
 		if ($p) $result = phraser_idiomes($debut, $ligne, $result);
@@ -52,35 +52,35 @@ 
 		$fichier = @$match[2];
 		# assurer ici la migration .php3 => .php
 		# et de l'ancienne syntaxe INCLURE(page.php3) devenue surperflue
-		if (preg_match(',^(.*[.]php)3$,', $fichier, $r)){
+		if (preg_match(',^(.*[.]php)3$,', $fichier, $r)) {
 			$fichier = $r[1];
 		}
-		$champ->texte = ($fichier!=='page.php') ? $fichier : '';
-		$texte = substr($texte, $p+strlen($match[0]));
+		$champ->texte = ($fichier !== 'page.php') ? $fichier : '';
+		$texte = substr($texte, $p + strlen($match[0]));
 		// on assimile {var=val} a une liste de un argument sans fonction
 		phraser_args($texte, "/>", "", $result, $champ);
-		if (!$champ->texte OR count($champ->param)>1){
+		if (!$champ->texte OR count($champ->param) > 1) {
 			if (!function_exists('normaliser_inclure'))
 				include_spip('public/normaliser');
 			normaliser_inclure($champ);
 		}
-		$texte = substr($champ->apres, strpos($champ->apres, '>')+1);
+		$texte = substr($champ->apres, strpos($champ->apres, '>') + 1);
 		$champ->apres = "";
 		$texte = preg_replace(',^</INCLU[DR]E>,', '', $texte);
 		$result[] = $champ;
 	}
 
-	return (($texte==="") ? $result : phraser_idiomes($texte, $ligne, $result));
+	return (($texte === "") ? $result : phraser_idiomes($texte, $ligne, $result));
 }
 
 // http://doc.spip.org/@phraser_polyglotte
-function phraser_polyglotte($texte, $ligne, $result){
+function phraser_polyglotte($texte, $ligne, $result) {
 
 	if (preg_match_all(BALISE_POLYGLOTTE, $texte, $m, PREG_SET_ORDER))
-		foreach ($m as $match){
+		foreach ($m as $match) {
 			$p = strpos($texte, $match[0]);
 			$debut = substr($texte, 0, $p);
-			if ($p){
+			if ($p) {
 				$champ = new Texte;
 				$champ->texte = $debut;
 				$champ->ligne = $ligne;
@@ -93,8 +93,8 @@ 
 			$ligne += substr_count($match[0], "\n");
 			$lang = '';
 			$bloc = $match[1];
-			$texte = substr($texte, $p+strlen($match[0]));
-			while (preg_match("/^[[:space:]]*([^[{]*)[[:space:]]*[[{]([a-z_]+)[]}](.*)$/si", $bloc, $regs)){
+			$texte = substr($texte, $p + strlen($match[0]));
+			while (preg_match("/^[[:space:]]*([^[{]*)[[:space:]]*[[{]([a-z_]+)[]}](.*)$/si", $bloc, $regs)) {
 				$trad = $regs[1];
 				if ($trad OR $lang)
 					$champ->traductions[$lang] = $trad;
@@ -104,7 +104,7 @@ 
 			$champ->traductions[$lang] = $bloc;
 			$result[] = $champ;
 		}
-	if ($texte!==""){
+	if ($texte !== "") {
 		$champ = new Texte;
 		$champ->texte = $texte;
 		$champ->ligne = $ligne;
@@ -120,13 +120,13 @@ 
 // chaine peut etre vide si =texte1 est present et arg1 est vide
 // sinon ce n'est pas un idiome
 // http://doc.spip.org/@phraser_idiomes
-function phraser_idiomes($texte, $ligne, $result){
-	while (preg_match(BALISE_IDIOMES, $texte, $match)){
+function phraser_idiomes($texte, $ligne, $result) {
+	while (preg_match(BALISE_IDIOMES, $texte, $match)) {
 		$p = strpos($texte, $match[0]);
-		$ko = (!$match[3] && ($match[5][0]!=='='));
-		$debut = substr($texte, 0, $p+($ko ? strlen($match[0]) : 0));
+		$ko = (!$match[3] && ($match[5][0] !== '='));
+		$debut = substr($texte, 0, $p + ($ko ? strlen($match[0]) : 0));
 		if ($debut) $result = phraser_champs($debut, $ligne, $result);
-		$texte = substr($texte, $p+strlen($match[0]));
+		$texte = substr($texte, $p + strlen($match[0]));
 		$ligne += substr_count($debut, "\n");
 		if ($ko) continue; // faux idiome
 		$champ = new Idiome;
@@ -135,7 +135,7 @@ 
 		// Stocker les arguments de la balise de traduction
 		$args = array();
 		$largs = $match[5];
-		while (preg_match(BALISE_IDIOMES_ARGS, $largs, $r)){
+		while (preg_match(BALISE_IDIOMES_ARGS, $largs, $r)) {
 			$args[$r[1]] = phraser_champs($r[2], 0, array());
 			$largs = substr($largs, strlen($r[0]));
 		}
@@ -146,17 +146,17 @@ 
 		phraser_args(@$match[7], ":", '', array(), $champ);
 		$result[] = $champ;
 	}
-	if ($texte!=="") $result = phraser_champs($texte, $ligne, $result);
+	if ($texte !== "") $result = phraser_champs($texte, $ligne, $result);
 
 	return $result;
 }
 
 // http://doc.spip.org/@phraser_champs
-function phraser_champs($texte, $ligne, $result){
-	while (preg_match("/" . NOM_DE_CHAMP . "/S", $texte, $match)){
+function phraser_champs($texte, $ligne, $result) {
+	while (preg_match("/".NOM_DE_CHAMP."/S", $texte, $match)) {
 		$p = strpos($texte, $match[0]);
-		$suite = substr($texte, $p+strlen($match[0]));
-		if ($match[5] || (strpos($suite[0], "[0-9]")===false)){
+		$suite = substr($texte, $p + strlen($match[0]));
+		if ($match[5] || (strpos($suite[0], "[0-9]") === false)) {
 			$debut = substr($texte, 0, $p);
 			if ($p) $result = phraser_polyglotte($debut, $ligne, $result);
 			$ligne += substr_count($debut, "\n");
@@ -167,7 +167,7 @@ 
 			$champ->nom_champ = $match[3];
 			$champ->etoile = $match[5];
 
-			if ($suite[0]=='{'){
+			if ($suite[0] == '{') {
 				phraser_arg($suite, '', array(), $champ);
 				// ce ltrim est une ereur de conception
 				// mais on le conserve par souci de compatibilite
@@ -185,11 +185,11 @@ 
 			$result[] = $champ;
 		} else {
 			// faux champ
-			$result = phraser_polyglotte(substr($texte, 0, $p+1), $ligne, $result);
-			$texte = (substr($texte, $p+1));
+			$result = phraser_polyglotte(substr($texte, 0, $p + 1), $ligne, $result);
+			$texte = (substr($texte, $p + 1));
 		}
 	}
-	if ($texte!=="") $result = phraser_polyglotte($texte, $ligne, $result);
+	if ($texte !== "") $result = phraser_polyglotte($texte, $ligne, $result);
 
 	return $result;
 }
@@ -200,10 +200,10 @@ 
 // on recommence tant qu'il y a des [...] en substituant a l'appel suivant
 
 // http://doc.spip.org/@phraser_champs_etendus
-function phraser_champs_etendus($texte, $ligne, $result){
-	if ($texte==="") return $result;
+function phraser_champs_etendus($texte, $ligne, $result) {
+	if ($texte === "") return $result;
 	$sep = '##';
-	while (strpos($texte, $sep)!==false)
+	while (strpos($texte, $sep) !== false)
 		$sep .= '#';
 
 	return array_merge($result, phraser_champs_interieurs($texte, $ligne, $sep, array()));
@@ -216,9 +216,9 @@ 
 // Tres chevelu
 
 // http://doc.spip.org/@phraser_args
-function phraser_args($texte, $fin, $sep, $result, &$pointeur_champ){
+function phraser_args($texte, $fin, $sep, $result, &$pointeur_champ) {
 	$texte = ltrim($texte);
-	while (($texte!=="") && strpos($fin, $texte[0])===false){
+	while (($texte !== "") && strpos($fin, $texte[0]) === false) {
 		$result = phraser_arg($texte, $sep, $result, $pointeur_champ);
 		$texte = ltrim($texte);
 	}
@@ -230,23 +230,23 @@ 
 }
 
 // http://doc.spip.org/@phraser_arg
-function phraser_arg(&$texte, $sep, $result, &$pointeur_champ){
+function phraser_arg(&$texte, $sep, $result, &$pointeur_champ) {
 	preg_match(",^(\|?[^}{)|]*)(.*)$,ms", $texte, $match);
 	$suite = ltrim($match[2]);
 	$fonc = trim($match[1]);
-	if ($fonc && $fonc[0]=="|") $fonc = ltrim(substr($fonc, 1));
+	if ($fonc && $fonc[0] == "|") $fonc = ltrim(substr($fonc, 1));
 	$res = array($fonc);
 	$err_f = '';
 	// cas du filtre sans argument ou du critere /
-	if (($suite && ($suite[0]!='{')) || ($fonc && $fonc[0]=='/')){
+	if (($suite && ($suite[0] != '{')) || ($fonc && $fonc[0] == '/')) {
 		// si pas d'argument, alors il faut une fonction ou un double |
-		if (!$match[1]){
+		if (!$match[1]) {
 			$err_f = array('zbug_erreur_filtre', array('filtre' => $texte));
 			erreur_squelette($err_f, $pointeur_champ);
 			$texte = '';
 		} else  $texte = $suite;
 		if ($err_f) $pointeur_champ->param = false;
-		elseif ($fonc!=='') $pointeur_champ->param[] = $res;
+		elseif ($fonc !== '') $pointeur_champ->param[] = $res;
 		// pour les balises avec faux filtres qui boudent ce dur larbeur
 		$pointeur_champ->fonctions[] = array($fonc, '');
 
@@ -254,14 +254,14 @@ 
 	}
 	$args = ltrim(substr($suite, 1)); // virer le '(' initial
 	$collecte = array();
-	while ($args && $args[0]!='}'){
-		if ($args[0]=='"')
+	while ($args && $args[0] != '}') {
+		if ($args[0] == '"')
 			preg_match('/^(")([^"]*)(")(.*)$/ms', $args, $regs);
-		else if ($args[0]=="'")
+		else if ($args[0] == "'")
 			preg_match("/^(')([^']*)(')(.*)$/ms", $args, $regs);
 		else {
 			preg_match("/^([[:space:]]*)([^,([{}]*([(\[{][^])}]*[])}])?[^,}]*)([,}].*)$/ms", $args, $regs);
-			if (!strlen($regs[2])){
+			if (!strlen($regs[2])) {
 				$err_f = array('zbug_erreur_filtre', array('filtre' => $args));
 				erreur_squelette($err_f, $pointeur_champ);
 				$champ = new Texte;
@@ -270,81 +270,81 @@ 
 			}
 		}
 		$arg = $regs[2];
-		if (trim($regs[1])){
+		if (trim($regs[1])) {
 			$champ = new Texte;
 			$champ->texte = $arg;
 			$champ->apres = $champ->avant = $regs[1];
 			$result[] = $champ;
 			$collecte[] = $champ;
-			$args = ltrim($regs[count($regs)-1]);
+			$args = ltrim($regs[count($regs) - 1]);
 		} else {
-			if (!preg_match("/" . NOM_DE_CHAMP . "([{|])/", $arg, $r)){
+			if (!preg_match("/".NOM_DE_CHAMP."([{|])/", $arg, $r)) {
 				// 0 est un aveu d'impuissance. A completer
 				$arg = phraser_champs_exterieurs($arg, 0, $sep, $result);
 
-				$args = ltrim($regs[count($regs)-1]);
+				$args = ltrim($regs[count($regs) - 1]);
 				$collecte = array_merge($collecte, $arg);
 				$result = array_merge($result, $arg);
 			} else {
 				$n = strpos($args, $r[0]);
 				$pred = substr($args, 0, $n);
 				$par = ',}';
-				if (preg_match('/^(.*)\($/', $pred, $m)){
+				if (preg_match('/^(.*)\($/', $pred, $m)) {
 					$pred = $m[1];
 					$par = ')';
 				}
-				if ($pred){
+				if ($pred) {
 					$champ = new Texte;
 					$champ->texte = $pred;
 					$champ->apres = $champ->avant = "";
 					$result[] = $champ;
 					$collecte[] = $champ;
 				}
-				$rec = substr($args, $n+strlen($r[0])-1);
+				$rec = substr($args, $n + strlen($r[0]) - 1);
 				$champ = new Champ;
 				$champ->nom_boucle = $r[2];
 				$champ->nom_champ = $r[3];
 				$champ->etoile = $r[5];
 				$next = $r[6];
-				while ($next=='{'){
+				while ($next == '{') {
 					phraser_arg($rec, $sep, array(), $champ);
 					$args = ltrim($rec);
 					$next = isset($args[0]) ? $args[0] : '';
 				}
-				while ($next=='|'){
+				while ($next == '|') {
 					phraser_args($rec, $par, $sep, array(), $champ);
 					$args = $champ->apres;
 					$champ->apres = '';
 					$next = isset($args[0]) ? $args[0] : '';
 				}
 				// Si erreur de syntaxe dans un sous-argument, propager.
-				if ($champ->param===false)
+				if ($champ->param === false)
 					$err_f = true;
 				else phraser_vieux($champ);
-				if ($par==')') $args = substr($args, 1);
+				if ($par == ')') $args = substr($args, 1);
 				$collecte[] = $champ;
 				$result[] = $champ;
 			}
 		}
-		if (isset($args[0]) AND $args[0]==','){
+		if (isset($args[0]) AND $args[0] == ',') {
 			$args = ltrim(substr($args, 1));
-			if ($collecte){
+			if ($collecte) {
 				$res[] = $collecte;
 				$collecte = array();
 			}
 		}
 	}
-	if ($collecte){
+	if ($collecte) {
 		$res[] = $collecte;
 		$collecte = array();
 	}
 	$texte = substr($args, 1);
-	$source = substr($suite, 0, strlen($suite)-strlen($texte));
+	$source = substr($suite, 0, strlen($suite) - strlen($texte));
 	// propager les erreurs, et ignorer les param vides
-	if ($pointeur_champ->param!==false){
+	if ($pointeur_champ->param !== false) {
 		if ($err_f)
 			$pointeur_champ->param = false;
-		elseif ($fonc!=='' || count($res)>1)
+		elseif ($fonc !== '' || count($res) > 1)
 			$pointeur_champ->param[] = $res;
 	}
 	// pour les balises avec faux filtres qui boudent ce dur larbeur
@@ -355,41 +355,41 @@ 
 
 
 // http://doc.spip.org/@phraser_champs_exterieurs
-function phraser_champs_exterieurs($texte, $ligne, $sep, $nested){
+function phraser_champs_exterieurs($texte, $ligne, $sep, $nested) {
 	$res = array();
-	while (($p = strpos($texte, "%$sep"))!==false){
-		if (!preg_match(',^%' . preg_quote($sep) . '([0-9]+)@,', substr($texte, $p), $m))
+	while (($p = strpos($texte, "%$sep")) !== false) {
+		if (!preg_match(',^%'.preg_quote($sep).'([0-9]+)@,', substr($texte, $p), $m))
 			break;
 		$debut = substr($texte, 0, $p);
-		$texte = substr($texte, $p+strlen($m[0]));
+		$texte = substr($texte, $p + strlen($m[0]));
 		if ($p)
 			$res = phraser_inclure($debut, $ligne, $res);
 		$ligne += substr_count($debut, "\n");
 		$res[] = $nested[$m[1]];
 	}
 
-	return (($texte==='') ? $res : phraser_inclure($texte, $ligne, $res));
+	return (($texte === '') ? $res : phraser_inclure($texte, $ligne, $res));
 }
 
 // http://doc.spip.org/@phraser_champs_interieurs
-function phraser_champs_interieurs($texte, $ligne, $sep, $result){
+function phraser_champs_interieurs($texte, $ligne, $sep, $result) {
 	$i = 0; // en fait count($result)
 	$x = "";
 
-	while (true){
+	while (true) {
 		$j = $i;
 		$n = $ligne;
-		while (preg_match(CHAMP_ETENDU, $texte, $match)){
+		while (preg_match(CHAMP_ETENDU, $texte, $match)) {
 			$p = strpos($texte, $match[0]);
 			$debut = substr($texte, 0, $p);
-			if ($p){
+			if ($p) {
 				$result[$i] = $debut;
 				$i++;
 			}
 			$nom = $match[4];
 			$champ = new Champ;
 			// ca ne marche pas encore en cas de champ imbrique
-			$champ->ligne = $x ? 0 : ($n+substr_count($debut, "\n"));
+			$champ->ligne = $x ? 0 : ($n + substr_count($debut, "\n"));
 			$champ->nom_boucle = $match[3];
 			$champ->nom_champ = $nom;
 			$champ->etoile = $match[6];
@@ -399,22 +399,22 @@ 
 			$champ->avant =
 				phraser_champs_exterieurs($match[1], $n, $sep, $result);
 			$debut = substr($champ->apres, 1);
-			if (!empty($debut)){
+			if (!empty($debut)) {
 				$n += substr_count(substr($texte, 0, strpos($texte, $debut)), "\n");
 			}
 			$champ->apres = phraser_champs_exterieurs($debut, $n, $sep, $result);
 
 			$result[$i] = $champ;
 			$i++;
-			$texte = substr($texte, $p+strlen($match[0]));
+			$texte = substr($texte, $p + strlen($match[0]));
 		}
-		if ($texte!==""){
+		if ($texte !== "") {
 			$result[$i] = $texte;
 			$i++;
 		}
 		$x = '';
 
-		while ($j<$i){
+		while ($j < $i) {
 			$z = $result[$j];
 			// j'aurais besoin de connaitre le nombre de lignes...
 			if (is_object($z))
@@ -430,18 +430,18 @@ 
 	}
 }
 
-function phraser_vieux(&$champ){
+function phraser_vieux(&$champ) {
 	$nom = $champ->nom_champ;
-	if ($nom=='EMBED_DOCUMENT'){
+	if ($nom == 'EMBED_DOCUMENT') {
 		if (!function_exists('phraser_vieux_emb'))
 			include_spip('public/normaliser');
 		phraser_vieux_emb($champ);
-	} elseif ($nom=='EXPOSER') {
+	} elseif ($nom == 'EXPOSER') {
 		if (!function_exists('phraser_vieux_exposer'))
 			include_spip('public/normaliser');
 		phraser_vieux_exposer($champ);
 	} elseif ($champ->param) {
-		if ($nom=='FORMULAIRE_RECHERCHE'){
+		if ($nom == 'FORMULAIRE_RECHERCHE') {
 			if (!function_exists('phraser_vieux_recherche'))
 				include_spip('public/normaliser');
 			phraser_vieux_recherche($champ);
@@ -449,11 +449,11 @@ 
 			if (!function_exists('phraser_vieux_logos'))
 				include_spip('public/normaliser');
 			phraser_vieux_logos($champ);
-		} elseif ($nom=='MODELE') {
+		} elseif ($nom == 'MODELE') {
 			if (!function_exists('phraser_vieux_modele'))
 				include_spip('public/normaliser');
 			phraser_vieux_modele($champ);
-		} elseif ($nom=='INCLURE' OR $nom=='INCLUDE') {
+		} elseif ($nom == 'INCLURE' OR $nom == 'INCLUDE') {
 			if (!function_exists('phraser_vieux_inclu'))
 				include_spip('public/normaliser');
 			phraser_vieux_inclu($champ);
@@ -484,22 +484,22 @@ 
  *     Elle sera complété de la liste de ses critères
  * @return void
  **/
-function phraser_criteres($params, &$result){
+function phraser_criteres($params, &$result) {
 
 	$err_ci = ''; // indiquera s'il y a eu une erreur
 	$args = array();
 	$type = $result->type_requete;
 	$doublons = array();
-	foreach ($params as $v){
+	foreach ($params as $v) {
 		$var = $v[1][0];
-		$param = ($var->type!='texte') ? "" : $var->texte;
-		if ((count($v)>2) && (!preg_match(",[^A-Za-z]IN[^A-Za-z],i", $param))){
+		$param = ($var->type != 'texte') ? "" : $var->texte;
+		if ((count($v) > 2) && (!preg_match(",[^A-Za-z]IN[^A-Za-z],i", $param))) {
 // plus d'un argument et pas le critere IN:
 // detecter comme on peut si c'est le critere implicite LIMIT debut, fin
 
-			if ($var->type!='texte'
+			if ($var->type != 'texte'
 				OR preg_match("/^(n|n-|(n-)?\d+)$/S", $param)
-			){
+			) {
 				$op = ',';
 				$not = "";
 			} else {
@@ -512,11 +512,11 @@ 
 				// et mettre son reliquat eventuel
 				// Recopier pour ne pas alterer le texte source
 				// utile au debusqueur
-				if ($m[4]){
+				if ($m[4]) {
 					// une maniere tres sale de supprimer les "' autour de {critere "xxx","yyy"}
-					if (preg_match(',^(["\'])(.*)\1$,', $m[4])){
+					if (preg_match(',^(["\'])(.*)\1$,', $m[4])) {
 						$c = null;
-						eval ('$c = ' . $m[4] . ';');
+						eval ('$c = '.$m[4].';');
 						if (isset($c))
 							$m[4] = $c;
 					}
@@ -534,10 +534,10 @@ 
 			$crit->param = $v;
 			$args[] = $crit;
 		} else {
-			if ($var->type!='texte'){
+			if ($var->type != 'texte') {
 				// cas 1 seul arg ne commencant pas par du texte brut:
 				// erreur ou critere infixe "/"
-				if (($v[1][1]->type!='texte') || (trim($v[1][1]->texte)!='/')){
+				if (($v[1][1]->type != 'texte') || (trim($v[1][1]->texte) != '/')) {
 					$err_ci = array('zbug_critere_inconnu',
 						array('critere' => $var->nom_champ));
 					erreur_squelette($err_ci, $result);
@@ -554,9 +554,9 @@ 
 				// les separateurs
 				if ($var->apres)
 					$result->separateur[] = $param;
-				elseif (($param=='tout') OR ($param=='tous'))
+				elseif (($param == 'tout') OR ($param == 'tous'))
 					$result->modificateur['tout'] = true;
-				elseif ($param=='plat')
+				elseif ($param == 'plat')
 					$result->modificateur['plat'] = true;
 
 				// Boucle hierarchie, analyser le critere id_rubrique
@@ -564,43 +564,43 @@ 
 				// ceux-ci pour avoir la rubrique mere...
 				// Les autres critères de la boucle hierarchie doivent être
 				// traités normalement.
-				elseif (strcasecmp($type, 'hierarchie')==0
+				elseif (strcasecmp($type, 'hierarchie') == 0
 					AND !preg_match(",^id_rubrique\b,", $param)
 					AND preg_match(",^id_\w+\s*$,", $param)
 				) {
 					$result->modificateur['tout'] = true;
-				} elseif (strcasecmp($type, 'hierarchie')==0 AND $param=="id_rubrique") {
+				} elseif (strcasecmp($type, 'hierarchie') == 0 AND $param == "id_rubrique") {
 					// rien a faire sur {id_rubrique} tout seul
 				} else {
 					// pas d'emplacement statique, faut un dynamique
 					/// mais il y a 2 cas qui ont les 2 !
-					if (($param=='unique') || (preg_match(',^!?doublons *,', $param))){
+					if (($param == 'unique') || (preg_match(',^!?doublons *,', $param))) {
 						// cette variable sera inseree dans le code
 						// et son nom sert d'indicateur des maintenant
 						$result->doublons = '$doublons_index';
-						if ($param=='unique') $param = 'doublons';
-					} elseif ($param=='recherche')
+						if ($param == 'unique') $param = 'doublons';
+					} elseif ($param == 'recherche')
 						// meme chose (a cause de #nom_de_boucle:URL_*)
 						$result->hash = ' ';
-					if (preg_match(',^ *([0-9-]+) *(/) *(.+) *$,', $param, $m)){
+					if (preg_match(',^ *([0-9-]+) *(/) *(.+) *$,', $param, $m)) {
 						$crit = phraser_critere_infixe($m[1], $m[3], $v, '/', '', '');
-					} elseif (preg_match(',^([!]?)(' . CHAMP_SQL_PLUS_FONC .
+					} elseif (preg_match(',^([!]?)('.CHAMP_SQL_PLUS_FONC.
 						')[[:space:]]*(\??)(!?)(<=?|>=?|==?|\b(?:IN|LIKE)\b)(.*)$,is', $param, $m)) {
 						$a2 = trim($m[8]);
-						if ($a2 AND ($a2[0]=="'" OR $a2[0]=='"') AND ($a2[0]==substr($a2, -1)))
+						if ($a2 AND ($a2[0] == "'" OR $a2[0] == '"') AND ($a2[0] == substr($a2, -1)))
 							$a2 = substr($a2, 1, -1);
 						$crit = phraser_critere_infixe($m[2], $a2, $v,
-							(($m[2]=='lang_select') ? $m[2] : $m[7]),
+							(($m[2] == 'lang_select') ? $m[2] : $m[7]),
 							$m[6], $m[5]);
 						$crit->exclus = $m[1];
-					} elseif (preg_match("/^([!]?)\s*(" .
-						CHAMP_SQL_PLUS_FONC .
+					} elseif (preg_match("/^([!]?)\s*(".
+						CHAMP_SQL_PLUS_FONC.
 						")\s*(\??)(.*)$/is", $param, $m)) {
 						// contient aussi les comparaisons implicites !
 						// Comme ci-dessus:
 						// le premier arg contient l'operateur
 						array_shift($v);
-						if ($m[6]){
+						if ($m[6]) {
 							$v[0][0] = new Texte;
 							$v[0][0]->texte = $m[6];
 						} else {
@@ -634,7 +634,7 @@ 
 }
 
 // http://doc.spip.org/@phraser_critere_infixe
-function phraser_critere_infixe($arg1, $arg2, $args, $op, $not, $cond){
+function phraser_critere_infixe($arg1, $arg2, $args, $op, $not, $cond) {
 	$args[0] = new Texte;
 	$args[0]->texte = $arg1;
 	$args[0] = array($args[0]);
@@ -649,11 +649,11 @@ 
 	return $crit;
 }
 
-function public_phraser_html_dist($texte, $id_parent, &$boucles, $descr, $ligne = 1){
+function public_phraser_html_dist($texte, $id_parent, &$boucles, $descr, $ligne = 1) {
 
 	$all_res = array();
 
-	while (($pos_boucle = strpos($texte, BALISE_BOUCLE))!==false){
+	while (($pos_boucle = strpos($texte, BALISE_BOUCLE)) !== false) {
 
 		$err_b = ''; // indiquera s'il y a eu une erreur
 		$result = new Boucle;
@@ -661,16 +661,16 @@ 
 		$result->descr = $descr;
 # attention: reperer la premiere des 2 balises: pre_boucle ou boucle
 
-		if (!preg_match("," . BALISE_PRE_BOUCLE . '[0-9_],', $texte, $r)
-			OR ($n = strpos($texte, $r[0]))===false
-			OR ($n>$pos_boucle)
-		){
+		if (!preg_match(",".BALISE_PRE_BOUCLE.'[0-9_],', $texte, $r)
+			OR ($n = strpos($texte, $r[0])) === false
+			OR ($n > $pos_boucle)
+		) {
 			$debut = substr($texte, 0, $pos_boucle);
 			$milieu = substr($texte, $pos_boucle);
 			$k = strpos($milieu, '(');
 			$id_boucle = trim(substr($milieu,
 				strlen(BALISE_BOUCLE),
-				$k-strlen(BALISE_BOUCLE)));
+				$k - strlen(BALISE_BOUCLE)));
 			$milieu = substr($milieu, $k);
 
 		} else {
@@ -679,18 +679,18 @@ 
 			$k = strpos($milieu, '>');
 			$id_boucle = substr($milieu,
 				strlen(BALISE_PRE_BOUCLE),
-				$k-strlen(BALISE_PRE_BOUCLE));
+				$k - strlen(BALISE_PRE_BOUCLE));
 
-			if (!preg_match("," . BALISE_BOUCLE . $id_boucle . "[[:space:]]*\(,", $milieu, $r)){
+			if (!preg_match(",".BALISE_BOUCLE.$id_boucle."[[:space:]]*\(,", $milieu, $r)) {
 				$err_b = array('zbug_erreur_boucle_syntaxe', array('id' => $id_boucle));
 				erreur_squelette($err_b, $result);
-				$texte = substr($texte, $n+1);
+				$texte = substr($texte, $n + 1);
 				continue;
 			} else {
 				$pos_boucle = $n;
 				$n = strpos($milieu, $r[0]);
-				$result->avant = substr($milieu, $k+1, $n-$k-1);
-				$milieu = substr($milieu, $n+strlen($id_boucle)+strlen(BALISE_BOUCLE));
+				$result->avant = substr($milieu, $k + 1, $n - $k - 1);
+				$milieu = substr($milieu, $n + strlen($id_boucle) + strlen(BALISE_BOUCLE));
 			}
 		}
 		$result->id_boucle = $id_boucle;
@@ -701,13 +701,13 @@ 
 		$type = $match[1];
 		$jointures = trim($match[2]);
 		$table_optionnelle = ($match[3]);
-		if ($jointures){
+		if ($jointures) {
 			// on affecte pas ici les jointures explicites, mais dans la compilation
 			// ou elles seront completees des jointures declarees
 			$result->jointures_explicites = $jointures;
 		}
 
-		if ($table_optionnelle){
+		if ($table_optionnelle) {
 			$result->table_optionnelle = $type;
 		}
 
@@ -725,20 +725,20 @@ 
 		//
 		// Recuperer la fin :
 		//
-		if ($milieu[0]==='/'){
+		if ($milieu[0] === '/') {
 			$suite = substr($milieu, 2);
 			$milieu = '';
 		} else {
 			$milieu = substr($milieu, 1);
-			$s = BALISE_FIN_BOUCLE . $id_boucle . ">";
+			$s = BALISE_FIN_BOUCLE.$id_boucle.">";
 			$p = strpos($milieu, $s);
-			if ($p===false){
+			if ($p === false) {
 				$err_b = array('zbug_erreur_boucle_fermant',
 					array('id' => $id_boucle));
 				erreur_squelette($err_b, $result);
 			}
 
-			$suite = substr($milieu, $p+strlen($s));
+			$suite = substr($milieu, $p + strlen($s));
 			$milieu = substr($milieu, 0, $p);
 		}
 
@@ -747,30 +747,30 @@ 
 		//
 		// 1. Recuperer la partie conditionnelle apres
 		//
-		$s = BALISE_POST_BOUCLE . $id_boucle . ">";
+		$s = BALISE_POST_BOUCLE.$id_boucle.">";
 		$p = strpos($suite, $s);
-		if ($p!==false){
+		if ($p !== false) {
 			$result->apres = substr($suite, 0, $p);
-			$suite = substr($suite, $p+strlen($s));
+			$suite = substr($suite, $p + strlen($s));
 		}
 
 		//
 		// 2. Recuperer la partie alternative
 		//
-		$s = BALISE_ALT_BOUCLE . $id_boucle . ">";
+		$s = BALISE_ALT_BOUCLE.$id_boucle.">";
 		$p = strpos($suite, $s);
-		if ($p!==false){
+		if ($p !== false) {
 			$result->altern = substr($suite, 0, $p);
-			$suite = substr($suite, $p+strlen($s));
+			$suite = substr($suite, $p + strlen($s));
 		}
-		$result->ligne = $ligne+substr_count($debut, "\n");
+		$result->ligne = $ligne + substr_count($debut, "\n");
 		$m = substr_count($milieu, "\n");
 		$b = substr_count($result->avant, "\n");
 		$a = substr_count($result->apres, "\n");
 
-		if ($p = strpos($type, ':')){
+		if ($p = strpos($type, ':')) {
 			$result->sql_serveur = substr($type, 0, $p);
-			$type = substr($type, $p+1);
+			$type = substr($type, $p + 1);
 		}
 		$soustype = strtolower($type);
 
@@ -783,7 +783,7 @@ 
 			$err_b = true;
 		else {
 			phraser_criteres($result->param, $result);
-			if (strncasecmp($soustype, TYPE_RECURSIF, strlen(TYPE_RECURSIF))==0){
+			if (strncasecmp($soustype, TYPE_RECURSIF, strlen(TYPE_RECURSIF)) == 0) {
 				$result->type_requete = TYPE_RECURSIF;
 				$args = $result->param;
 				array_unshift($args,
@@ -793,9 +793,9 @@ 
 		}
 
 		$result->avant = public_phraser_html_dist($result->avant, $id_parent, $boucles, $descr, $result->ligne);
-		$result->apres = public_phraser_html_dist($result->apres, $id_parent, $boucles, $descr, $result->ligne+$b+$m);
-		$result->altern = public_phraser_html_dist($result->altern, $id_parent, $boucles, $descr, $result->ligne+$a+$m+$b);
-		$result->milieu = public_phraser_html_dist($milieu, $id_boucle, $boucles, $descr, $result->ligne+$b);
+		$result->apres = public_phraser_html_dist($result->apres, $id_parent, $boucles, $descr, $result->ligne + $b + $m);
+		$result->altern = public_phraser_html_dist($result->altern, $id_parent, $boucles, $descr, $result->ligne + $a + $m + $b);
+		$result->milieu = public_phraser_html_dist($milieu, $id_boucle, $boucles, $descr, $result->ligne + $b);
 
 		// Prevenir le generateur de code que le squelette est faux
 		if ($err_b) $result->type_requete = false;
@@ -803,7 +803,7 @@ 
 		// Verifier qu'il n'y a pas double definition
 		// apres analyse des sous-parties (pas avant).
 
-		if (isset($boucles[$id_boucle])){
+		if (isset($boucles[$id_boucle])) {
 			$err_b_d = array('zbug_erreur_boucle_double',
 				array('id' => $id_boucle));
 			erreur_squelette($err_b_d, $result);
@@ -813,7 +813,7 @@ 
 			$boucles[$id_boucle] = $result;
 		$all_res = phraser_champs_etendus($debut, $ligne, $all_res);
 		$all_res[] = &$boucles[$id_boucle];
-		if (!empty($suite)){
+		if (!empty($suite)) {
 			$ligne += substr_count(substr($texte, 0, strpos($texte, $suite)), "\n");
 		}
 		$texte = $suite;

ecrire/public/normaliser.php

Spacing +17 added lines, -17 removed lines

@@ -30,9 +30,9 @@ 
 		$args = array_shift($p->param);
 	}
 
-	foreach($p->param as $couple) {
+	foreach ($p->param as $couple) {
 		$nom = trim($couple[0]);
-		if ($nom == '')  {array_shift($p->param); break;}
+		if ($nom == '') {array_shift($p->param); break; }
 		$r = phraser_logo_faux_filtres($nom);
 		if ($r === 0) {
 			$c = new Texte;
@@ -58,9 +58,9 @@ 
 				if ($m[6]) {
 				  $r = new Texte;
 				  $r->texte = $m[6];
-				  $champ[]= $r;
+				  $champ[] = $r;
 				}
-				$args[]= $champ;
+				$args[] = $champ;
 				array_shift($p->param);
 				spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
 
@@ -73,7 +73,7 @@ 
 
 function phraser_logo_faux_filtres($nom)
 {
-	switch($nom) {
+	switch ($nom) {
 	case 'top':
 	case 'left':
 	case 'right':
@@ -91,20 +91,20 @@ 
 function phraser_vieux_emb(&$p)
 {
 	if (!is_array($p->param))
-		$p->param=array();
+		$p->param = array();
 
 	// Produire le premier argument {emb}
 	$texte = new Texte;
-	$texte->texte='emb';
+	$texte->texte = 'emb';
 	$param = array('', array($texte));
 
 	// Transformer les filtres en arguments
-	for ($i=0; $i<count($p->param); $i++) {
+	for ($i = 0; $i < count($p->param); $i++) {
 		if ($p->param[$i][0]) {
 			if (!strstr($p->param[$i][0], '='))
-				break;# on a rencontre un vrai filtre, c'est fini
+				break; # on a rencontre un vrai filtre, c'est fini
 			$texte = new Texte;
-			$texte->texte=$p->param[$i][0];
+			$texte->texte = $p->param[$i][0];
 			$param[] = array($texte);
 		}
 		array_shift($p->param);
@@ -124,7 +124,7 @@ 
 		$p->param[0][1] = array($c);
 		$p->param[0][0] = '';
 		$p->fonctions = array();
-		spip_log('FORMULAIRE_RECHERCHE avec filtre ' . $c->texte, 'vieilles_defs');
+		spip_log('FORMULAIRE_RECHERCHE avec filtre '.$c->texte, 'vieilles_defs');
 	}
 }
 
@@ -150,8 +150,8 @@ 
 	}
 }
 
-function phraser_vieux_modele($p)  {normaliser_args_inclumodel($p);}
-function phraser_vieux_inclu($p) {normaliser_args_inclumodel($p);}
+function phraser_vieux_modele($p) {normaliser_args_inclumodel($p); }
+function phraser_vieux_inclu($p) {normaliser_args_inclumodel($p); }
 
 function normaliser_args_inclumodel($p)
 {
@@ -161,7 +161,7 @@ 
 	if ($args[0]) return; // filtre immediat
 	array_shift($p->param);
 	foreach ($p->param as $l) {
-		if (!array_shift($l))  {
+		if (!array_shift($l)) {
 			$args = array_merge($args, $l);
 			array_shift($p->param);
 		} else  break; // filtre
@@ -175,13 +175,13 @@ 
 	$l = $champ->param[0];
 	if (is_array($l) AND !$l[0]) {
 	  foreach ($l as $k => $p) {
-	    if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte,'=')) {
+	    if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte, '=')) {
 	      $p[0]->texte = trim($p[0]->texte);
 	      }
 	  }
 	  foreach ($l as $k => $p) {
 		if (!$p OR $p[0]->type != 'texte' OR 
-		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
+		    !preg_match('/^fond\s*=\s*(.*)$/', $p[0]->texte, $r))
 			continue;
 
 		if ($r[1])
@@ -189,7 +189,7 @@ 
 		else unset($p[0]);
 		$champ->texte = $p;
 		unset($champ->param[0][$k]);
-		if (count($champ->param[0]) ==1) 
+		if (count($champ->param[0]) == 1) 
 			array_shift($champ->param);
 		return;
 	  }

ecrire/public/interfaces.php

Spacing +38 added lines, -38 removed lines

@@ -168,7 +168,7 @@ 
 	/**
 	 * Identifiant de la boucle parente
 	 * @var string */
-	public $id_parent ='';
+	public $id_parent = '';
 
 	/**
 	 * Partie optionnelle avant
@@ -254,7 +254,7 @@ 
 	 * @var string|null */
 	public $doublons;
 
-	var $partie, $total_parties,$mode_partie='';
+	var $partie, $total_parties, $mode_partie = '';
 	var $externe = ''; # appel a partir d'une autre boucle (recursion)
 	// champs pour la construction de la requete SQL
 	var $select = array();
@@ -267,9 +267,9 @@ 
 	var $group = array();
 	var $order = array();
 	var $default_order = array();
-	var $date = 'date' ;
-	var $hash = "" ;
-	var $in = "" ;
+	var $date = 'date';
+	var $hash = "";
+	var $in = "";
 	var $sous_requete = false;
 	var $hierarchie = '';
 	var $statut = false; # definition/surcharge du statut des elements retournes
@@ -375,7 +375,7 @@ 
 	/**
 	 * Identifiant de la boucle parente si explicité
 	 * @var string|null */
-	public $nom_boucle= '';
+	public $nom_boucle = '';
 
 	/**
 	 * Partie optionnelle avant
@@ -586,7 +586,7 @@ 
 
 
 global $table_criteres_infixes;
-$table_criteres_infixes = array('<', '>', '<=', '>=', '==', '===', '!=', '!==', '<>',  '?');
+$table_criteres_infixes = array('<', '>', '<=', '>=', '==', '===', '!=', '!==', '<>', '?');
 
 global $exception_des_connect;
 $exception_des_connect[] = ''; // ne pas transmettre le connect='' par les inclure
@@ -605,13 +605,13 @@ 
  *
  * @return void
  */
-function declarer_interfaces(){
+function declarer_interfaces() {
 	global $exceptions_des_tables, $table_des_tables, $table_date, $table_titre, $table_statut;
 
-	$table_des_tables['articles']='articles';
-	$table_des_tables['auteurs']='auteurs';
-	$table_des_tables['rubriques']='rubriques';
-	$table_des_tables['hierarchie']='rubriques';
+	$table_des_tables['articles'] = 'articles';
+	$table_des_tables['auteurs'] = 'auteurs';
+	$table_des_tables['rubriques'] = 'rubriques';
+	$table_des_tables['hierarchie'] = 'rubriques';
 
 	// definition des statuts de publication
 	global $tables_statut;
@@ -634,35 +634,35 @@ 
 	define('_TRAITEMENT_RACCOURCIS', 'propre(%s, $connect, $Pile[0])');
 	define('_TRAITEMENT_TYPO_SANS_NUMERO', 'typo(supprimer_numero(%s), "TYPO", $connect, $Pile[0])');
 
-	$table_des_traitements['BIO'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['CHAPO'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['DATE'][]= 'normaliser_date(%s)';
-	$table_des_traitements['DATE_REDAC'][]= 'normaliser_date(%s)';
-	$table_des_traitements['DATE_MODIF'][]= 'normaliser_date(%s)';
-	$table_des_traitements['DATE_NOUVEAUTES'][]= 'normaliser_date(%s)';
-	$table_des_traitements['DESCRIPTIF'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['INTRODUCTION'][]= 'PtoBR('. _TRAITEMENT_RACCOURCIS .')';
-	$table_des_traitements['NOM_SITE_SPIP'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['NOM'][]= _TRAITEMENT_TYPO_SANS_NUMERO;
-	$table_des_traitements['AUTEUR'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['PS'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['SOURCE'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['SOUSTITRE'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['SURTITRE'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['TAGS'][]= '%s';
-	$table_des_traitements['TEXTE'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['TITRE'][]= _TRAITEMENT_TYPO_SANS_NUMERO;
-	$table_des_traitements['TYPE'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['DESCRIPTIF_SITE_SPIP'][]= _TRAITEMENT_RACCOURCIS;
-	$table_des_traitements['SLOGAN_SITE_SPIP'][]= _TRAITEMENT_TYPO;
-	$table_des_traitements['ENV'][]= 'entites_html(%s,true)';
+	$table_des_traitements['BIO'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['CHAPO'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['DATE'][] = 'normaliser_date(%s)';
+	$table_des_traitements['DATE_REDAC'][] = 'normaliser_date(%s)';
+	$table_des_traitements['DATE_MODIF'][] = 'normaliser_date(%s)';
+	$table_des_traitements['DATE_NOUVEAUTES'][] = 'normaliser_date(%s)';
+	$table_des_traitements['DESCRIPTIF'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['INTRODUCTION'][] = 'PtoBR('._TRAITEMENT_RACCOURCIS.')';
+	$table_des_traitements['NOM_SITE_SPIP'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['NOM'][] = _TRAITEMENT_TYPO_SANS_NUMERO;
+	$table_des_traitements['AUTEUR'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['PS'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['SOURCE'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['SOUSTITRE'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['SURTITRE'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['TAGS'][] = '%s';
+	$table_des_traitements['TEXTE'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['TITRE'][] = _TRAITEMENT_TYPO_SANS_NUMERO;
+	$table_des_traitements['TYPE'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['DESCRIPTIF_SITE_SPIP'][] = _TRAITEMENT_RACCOURCIS;
+	$table_des_traitements['SLOGAN_SITE_SPIP'][] = _TRAITEMENT_TYPO;
+	$table_des_traitements['ENV'][] = 'entites_html(%s,true)';
 
 	// valeur par defaut pour les balises non listees ci-dessus
-	$table_des_traitements['*'][]= false; // pas de traitement, mais permet au compilo de trouver la declaration suivante
+	$table_des_traitements['*'][] = false; // pas de traitement, mais permet au compilo de trouver la declaration suivante
 	// toujours securiser les DATA
-	$table_des_traitements['*']['DATA']= 'safehtml(%s)';
+	$table_des_traitements['*']['DATA'] = 'safehtml(%s)';
 	// expliciter pour VALEUR qui est un champ calcule et ne sera pas protege par le catch-all *
-	$table_des_traitements['VALEUR']['DATA']= 'safehtml(%s)';
+	$table_des_traitements['VALEUR']['DATA'] = 'safehtml(%s)';
 
 
 	// gerer l'affectation en 2 temps car si le pipe n'est pas encore declare, on ecrase les globales
@@ -677,7 +677,7 @@ 
 			'table_des_traitements'=>$table_des_traitements,
 			'table_statut'=>$table_statut,
 			));
-	if ($interfaces){
+	if ($interfaces) {
 			$table_des_tables = $interfaces['table_des_tables'];
 			$exceptions_des_tables = $interfaces['exceptions_des_tables'];
 			$table_date = $interfaces['table_date'];

ecrire/public/balises.php

Spacing +129 added lines, -129 removed lines

@@ -20,8 +20,8 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@interprete_argument_balise
-function interprete_argument_balise($n,$p) {
-	if (($p->param) && (!$p->param[0][0]) && (count($p->param[0])>$n))
+function interprete_argument_balise($n, $p) {
+	if (($p->param) && (!$p->param[0][0]) && (count($p->param[0]) > $n))
 		return calculer_liste($p->param[0][$n],
 			$p->descr,
 			$p->boucles,
@@ -96,7 +96,7 @@ 
 // quand #DATE est en dehors des boucles
 // http://www.spip.net/fr_article1971.html
 // http://doc.spip.org/@balise_DATE_dist
-function balise_DATE_dist ($p) {
+function balise_DATE_dist($p) {
 	$d = champ_sql('date', $p);
 #	if ($d === "@\$Pile[0]['date']")
 #		$d = "isset(\$Pile[0]['date']) ? $d : time()";
@@ -107,7 +107,7 @@ 
 // #DATE_REDAC
 // http://www.spip.net/fr_article1971.html
 // http://doc.spip.org/@balise_DATE_REDAC_dist
-function balise_DATE_REDAC_dist ($p) {
+function balise_DATE_REDAC_dist($p) {
 	$d = champ_sql('date_redac', $p);
 #	if ($d === "@\$Pile[0]['date_redac']")
 #		$d = "isset(\$Pile[0]['date_redac']) ? $d : time()";
@@ -119,7 +119,7 @@ 
 // #DATE_MODIF
 // http://www.spip.net/fr_article1971.html
 // http://doc.spip.org/@balise_DATE_MODIF_dist
-function balise_DATE_MODIF_dist ($p) {
+function balise_DATE_MODIF_dist($p) {
 	$p->code = champ_sql('date_modif', $p);
 	$p->interdire_scripts = false;
 	return $p;
@@ -140,7 +140,7 @@ 
 // http://doc.spip.org/@balise_DOSSIER_SQUELETTE_dist
 function balise_DOSSIER_SQUELETTE_dist($p) {
 	$code = substr(addslashes(dirname($p->descr['sourcefile'])), strlen(_DIR_RACINE));
-	$p->code = "_DIR_RACINE . '$code'" .
+	$p->code = "_DIR_RACINE . '$code'".
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -148,7 +148,7 @@ 
 // http://doc.spip.org/@balise_SQUELETTE_dist
 function balise_SQUELETTE_dist($p) {
 	$code = addslashes($p->descr['sourcefile']);
-	$p->code = "'$code'" .
+	$p->code = "'$code'".
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -183,12 +183,12 @@ 
 **/
 function balise_NOM_SITE_dist($p) {
 	if (!$p->etoile) {
-		$p->code = "supprimer_numero(calculer_url(" .
-		champ_sql('url_site',$p) ."," .
-		champ_sql('nom_site',$p) .
+		$p->code = "supprimer_numero(calculer_url(".
+		champ_sql('url_site', $p).",".
+		champ_sql('nom_site', $p).
 		", 'titre', \$connect, false))";
 	} else
-		$p->code = champ_sql('nom_site',$p);
+		$p->code = champ_sql('nom_site', $p);
 
 	$p->interdire_scripts = true;
 	return $p;
@@ -230,7 +230,7 @@ 
 	$b = $p->nom_boucle ? $p->nom_boucle : $p->descr['id_mere'];
 	if ($b === '' || !isset($p->boucles[$b])) {
 		$msg = array('zbug_champ_hors_boucle',
-				array('champ' => "#$b" . 'TOTAL_BOUCLE')
+				array('champ' => "#$b".'TOTAL_BOUCLE')
 			  );
 		erreur_squelette($msg, $p);
 	} else {
@@ -249,8 +249,8 @@ 
 
 // http://doc.spip.org/@balise_POPULARITE_ABSOLUE_dist
 function balise_POPULARITE_ABSOLUE_dist($p) {
-	$p->code = 'ceil(' .
-	champ_sql('popularite', $p) .
+	$p->code = 'ceil('.
+	champ_sql('popularite', $p).
 	')';
 	$p->interdire_scripts = false;
 	return $p;
@@ -273,10 +273,10 @@ 
 // http://doc.spip.org/@balise_EXPOSE_dist
 function balise_EXPOSE_dist($p) {
 	$on = "'on'";
-	$off= "''";
-	if (($v = interprete_argument_balise(1,$p))!==NULL){
+	$off = "''";
+	if (($v = interprete_argument_balise(1, $p)) !== NULL) {
 		$on = $v;
-		if (($v = interprete_argument_balise(2,$p))!==NULL)
+		if (($v = interprete_argument_balise(2, $p)) !== NULL)
 			$off = $v;
 
 	}
@@ -289,8 +289,8 @@ 
 // http://doc.spip.org/@balise_VALEUR_dist
 function balise_VALEUR_dist($p) {
 	$b = $p->nom_boucle ? $p->nom_boucle : $p->id_boucle;
-	$p->code = index_pile($p->id_boucle, 'valeur', $p->boucles, $b);;
-	if (($v = interprete_argument_balise(1,$p))!==NULL){
+	$p->code = index_pile($p->id_boucle, 'valeur', $p->boucles, $b); ;
+	if (($v = interprete_argument_balise(1, $p)) !== NULL) {
 		$p->code = 'table_valeur('.$p->code.', '.$v.')';
 	}
 	$p->interdire_scripts = true;
@@ -318,7 +318,7 @@ 
 		$parent = 0; // pour if (!$parent) dans calculer_expose
 	} elseif (isset($desc['field']['id_rubrique'])) {
 		$parent = index_pile($p->id_boucle, 'id_rubrique', $p->boucles, $b);
-	} elseif  (isset($desc['field']['id_groupe'])) {
+	} elseif (isset($desc['field']['id_groupe'])) {
 		$parent = index_pile($p->id_boucle, 'id_groupe', $p->boucles, $b);
 	} else $parent = "''";
 
@@ -337,13 +337,13 @@ 
 // http://doc.spip.org/@balise_DEBUT_SURLIGNE_dist
 function balise_DEBUT_SURLIGNE_dist($p) {
 	include_spip('inc/surligne');
-	$p->code = "'<!-- " . MARQUEUR_SURLIGNE . " -->'";
+	$p->code = "'<!-- ".MARQUEUR_SURLIGNE." -->'";
 	return $p;
 }
 // http://doc.spip.org/@balise_FIN_SURLIGNE_dist
 function balise_FIN_SURLIGNE_dist($p) {
 	include_spip('inc/surligne');
-	$p->code = "'<!-- " . MARQUEUR_FSURLIGNE . "-->'";
+	$p->code = "'<!-- ".MARQUEUR_FSURLIGNE."-->'";
 	return $p;
 }
 
@@ -367,7 +367,7 @@ 
 	}
 
 	// longueur en parametre, ou valeur par defaut
-	if (($v = interprete_argument_balise(1,$p))!==NULL) {
+	if (($v = interprete_argument_balise(1, $p)) !== NULL) {
 		$longueur = 'intval('.$v.')';
 	} else {
 		switch ($type) {
@@ -398,7 +398,7 @@ 
 // (celle du site ou celle qui a ete passee dans l'URL par le visiteur)
 // #LANG* n'affiche rien si aucune langue n'est trouvee dans le sql/le contexte
 // http://doc.spip.org/@balise_LANG_dist
-function balise_LANG_dist ($p) {
+function balise_LANG_dist($p) {
 	$_lang = champ_sql('lang', $p);
 	if (!$p->etoile)
 		$p->code = "spip_htmlentities($_lang ? $_lang : \$GLOBALS['spip_lang'])";
@@ -413,7 +413,7 @@ 
 // http://www.spip.net/fr_article902.html
 // http://www.spip.net/fr_article911.html
 // http://doc.spip.org/@balise_LESAUTEURS_dist
-function balise_LESAUTEURS_dist ($p) {
+function balise_LESAUTEURS_dist($p) {
 	// Cherche le champ 'lesauteurs' dans la pile
 	$_lesauteurs = champ_sql('lesauteurs', $p, false);
 
@@ -426,12 +426,12 @@ 
 		$p->code = "safehtml($_lesauteurs)";
 		// $p->interdire_scripts = true;
 	} else {
-		if(!$p->id_boucle){
+		if (!$p->id_boucle) {
 			$connect = '';
 			$objet = 'article';
 			$id_table_objet = 'id_article';
 		}
-		else{
+		else {
 			$b = $p->nom_boucle ? $p->nom_boucle : $p->id_boucle;
 			$connect = $p->boucles[$b]->sql_serveur;
 			$type_boucle = $p->boucles[$b]->type_requete;
@@ -442,9 +442,9 @@ 
 
 		$p->code = sprintf(CODE_RECUPERER_FOND, "'modeles/lesauteurs'",
 				   "array('objet'=>'".$objet.
-					   "','id_objet' => ".champ_sql($id_table_objet, $p) .
-					   ",'$id_table_objet' => ".champ_sql($id_table_objet, $p) .
-					   ($objet=='article'?"":",'id_article' => ".champ_sql('id_article', $p)).
+					   "','id_objet' => ".champ_sql($id_table_objet, $p).
+					   ",'$id_table_objet' => ".champ_sql($id_table_objet, $p).
+					   ($objet == 'article' ? "" : ",'id_article' => ".champ_sql('id_article', $p)).
 					   ")",
 				   "'trim'=>true, 'compil'=>array($c)",
 				   _q($connect));
@@ -504,7 +504,7 @@ 
 					$m = trim($m);
 					if ($m != "''") {
 						if (!preg_match(",\W,", $m)) {
-							$m = $boucle->id_table . ".$m";
+							$m = $boucle->id_table.".$m";
 						}
 						
 						$m .= " AS titre_rang";
@@ -534,7 +534,7 @@ 
 // #POPULARITE
 // http://www.spip.net/fr_article1846.html
 // http://doc.spip.org/@balise_POPULARITE_dist
-function balise_POPULARITE_dist ($p) {
+function balise_POPULARITE_dist($p) {
 	$_popularite = champ_sql('popularite', $p);
 	$p->code = "(ceil(min(100, 100 * $_popularite
 	/ max(1 , 0 + \$GLOBALS['meta']['popularite_max']))))";
@@ -555,7 +555,7 @@ 
 
 // http://www.spip.net/fr_article3367.html
 // http://doc.spip.org/@balise_PAGINATION_dist
-function balise_PAGINATION_dist($p, $liste='true') {
+function balise_PAGINATION_dist($p, $liste = 'true') {
 	$b = $p->nom_boucle ? $p->nom_boucle : $p->descr['id_mere'];
 
 	// s'il n'y a pas de nom de boucle, on ne peut pas paginer
@@ -583,16 +583,16 @@ 
 	// si false, le compilo va bloquer sur des syntaxes avec un filtre sans argument qui suit la balise
 	// si true, les arguments simples (sans truc=chose) vont degager
 	$_contexte = argumenter_inclure($p->param, true, $p, $p->boucles, $p->id_boucle, false, false);
-	if (count($_contexte)){
-		list($key,$val) = each($_contexte);
-		if (is_numeric($key)){
+	if (count($_contexte)) {
+		list($key, $val) = each($_contexte);
+		if (is_numeric($key)) {
 			array_shift($_contexte);
-			$__modele = interprete_argument_balise(1,$p);
+			$__modele = interprete_argument_balise(1, $p);
 		}
 	}
 
-	if (count($_contexte)){
-		$code_contexte = implode(',',$_contexte);
+	if (count($_contexte)) {
+		$code_contexte = implode(',', $_contexte);
 	}
 	else
 		$code_contexte = '';
@@ -601,8 +601,8 @@ 
 	$pas = $p->boucles[$b]->total_parties;
 	$f_pagination = chercher_filtre('pagination');
 	$type = $p->boucles[$b]->modificateur['debut_nom'];
-	$modif = ($type[0]!=="'") ? "'debut'.$type"
-	  : ("'debut" .substr($type,1));
+	$modif = ($type[0] !== "'") ? "'debut'.$type"
+	  : ("'debut".substr($type, 1));
 
 	$p->code = sprintf(CODE_PAGINATION, $f_pagination, $b, $type, $modif, $pas, $liste, ((isset($__modele) and $__modele) ? $__modele : "''"), _q($connect), $code_contexte);
 
@@ -617,7 +617,7 @@ 
 // http://doc.spip.org/@balise_ANCRE_PAGINATION_dist
 function balise_ANCRE_PAGINATION_dist($p) {
 	if ($f = charger_fonction('PAGINATION', 'balise', true))
-		return $f($p, $liste='false');
+		return $f($p, $liste = 'false');
 	else return NULL; // ou une erreur ?
 }
 
@@ -628,7 +628,7 @@ 
 	$b = $p->nom_boucle ? $p->nom_boucle : $p->descr['id_mere'];
 	if ($b === '' || !isset($p->boucles[$b])) {
 		$msg = array('zbug_champ_hors_boucle',
-				array('champ' => "#$b" . 'TOTAL_BOUCLE')
+				array('champ' => "#$b".'TOTAL_BOUCLE')
 			   );
 		erreur_squelette($msg, $p);
 	} else {
@@ -657,23 +657,23 @@ 
 //
 // http://doc.spip.org/@balise_CHEMIN_dist
 function balise_CHEMIN_dist($p) {
-	$arg = interprete_argument_balise(1,$p);
+	$arg = interprete_argument_balise(1, $p);
 	if (!$arg) {
-		$msg = array('zbug_balise_sans_argument',	array('balise' => ' CHEMIN'));
+		$msg = array('zbug_balise_sans_argument', array('balise' => ' CHEMIN'));
 		erreur_squelette($msg, $p);
 	} else
-	  $p->code = 'find_in_path(' . $arg .')';
+	  $p->code = 'find_in_path('.$arg.')';
 
 	#$p->interdire_scripts = true;
 	return $p;
 }
 
 function balise_CHEMIN_IMAGE_dist($p) {
-	$arg = interprete_argument_balise(1,$p);
+	$arg = interprete_argument_balise(1, $p);
 	if (!$arg) {
 		$msg = array('zbug_balise_sans_argument', array('balise' => ' CHEMIN_IMAGE'));
 		erreur_squelette($msg, $p);
-	} else $p->code = 'chemin_image(' . $arg .')';
+	} else $p->code = 'chemin_image('.$arg.')';
 
 	#$p->interdire_scripts = true;
 	return $p;
@@ -719,9 +719,9 @@ 
 function balise_ENV_dist($p, $src = NULL) {
 
 	// cle du tableau desiree
-	$_nom = interprete_argument_balise(1,$p);
+	$_nom = interprete_argument_balise(1, $p);
 	// valeur par defaut
-	$_sinon = interprete_argument_balise(2,$p);
+	$_sinon = interprete_argument_balise(2, $p);
 	
 	// $src est un tableau de donnees sources eventuellement transmis
 	// en absence, on utilise l'environnement du squelette $Pile[0]
@@ -766,14 +766,14 @@ 
  * @return Object
  */
 function balise_CONFIG_dist($p) {
-	if (!$arg = interprete_argument_balise(1,$p)) {
+	if (!$arg = interprete_argument_balise(1, $p)) {
 		$arg = "''";
 	}
-	$_sinon = interprete_argument_balise(2,$p);
-	$_unserialize = sinon(interprete_argument_balise(3,$p),"false");
+	$_sinon = interprete_argument_balise(2, $p);
+	$_unserialize = sinon(interprete_argument_balise(3, $p), "false");
 
-	$p->code = '(include_spip(\'inc/config\')?lire_config(' . $arg . ',' .
-		($_sinon && $_sinon != "''" ? $_sinon : 'null') . ',' . $_unserialize . "):'')";
+	$p->code = '(include_spip(\'inc/config\')?lire_config('.$arg.','.
+		($_sinon && $_sinon != "''" ? $_sinon : 'null').','.$_unserialize."):'')";
 
 	return $p;
 }
@@ -809,8 +809,8 @@ 
 // Ajoute x=y dans la session du visiteur
 // http://doc.spip.org/@balise_SESSION_SET_dist
 function balise_SESSION_SET_dist($p) {
-	$_nom = interprete_argument_balise(1,$p);
-	$_val = interprete_argument_balise(2,$p);
+	$_nom = interprete_argument_balise(1, $p);
+	$_val = interprete_argument_balise(2, $p);
 	if (!$_nom OR !$_val) {
 		$err_b_s_a = array('zbug_balise_sans_argument', array('balise' => 'SESSION_SET'));
 		erreur_squelette($err_b_s_a, $p);
@@ -836,12 +836,12 @@ 
 // #EVAL{'str_replace("r","z", "roger")'}  (attention les "'" sont interdits)
 // http://doc.spip.org/@balise_EVAL_dist
 function balise_EVAL_dist($p) {
-	$php = interprete_argument_balise(1,$p);
+	$php = interprete_argument_balise(1, $p);
 	if ($php) {
 		# optimisation sur les #EVAL{une expression sans #BALISE}
 		# attention au commentaire "// x signes" qui precede
 		if (preg_match(",^([[:space:]]*//[^\n]*\n)'([^']+)'$,ms",
-		$php,$r))
+		$php, $r))
 			$p->code = /* $r[1]. */'('.$r[2].')';
 		else
 			$p->code = "eval('return '.$php.';')";
@@ -860,7 +860,7 @@ 
 // (impossible via #NOTES qui est une balise calculee)
 // ne permet pas de passer une expression pour x qui ne peut etre qu'un texte statique !
 // http://doc.spip.org/@balise_CHAMP_SQL_dist
-function balise_CHAMP_SQL_dist($p){
+function balise_CHAMP_SQL_dist($p) {
 
 	if ($p->param
 	AND isset($p->param[0][1][0])
@@ -877,8 +877,8 @@ 
 // #VAL{x} renvoie 'x' (permet d'appliquer un filtre a une chaine)
 // Attention #VAL{1,2} renvoie '1', indiquer #VAL{'1,2'}
 // http://doc.spip.org/@balise_VAL_dist
-function balise_VAL_dist($p){
-	$p->code = interprete_argument_balise(1,$p);
+function balise_VAL_dist($p) {
+	$p->code = interprete_argument_balise(1, $p);
 	if (!strlen($p->code))
 		$p->code = "''";
 	$p->interdire_scripts = false;
@@ -894,7 +894,7 @@ 
 //
 // http://doc.spip.org/@balise_REM_dist
 function balise_REM_dist($p) {
-	$p->code="''";
+	$p->code = "''";
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -944,7 +944,7 @@ 
 	if ($p->param) {
 		$args = array();
 		foreach ($p->param as $i => $ignore)
-			$args[] = interprete_argument_balise($i+1,$p);
+			$args[] = interprete_argument_balise($i + 1, $p);
 		$p->code = "'<' . '"
 			.'?php header("X-Spip-Filtre: \'.'
 				.join('.\'|\'.', $args)
@@ -1061,7 +1061,7 @@ 
 // (Incompatible avec les balises dynamiques)
 // http://doc.spip.org/@balise_INCLUDE_dist
 function balise_INCLUDE_dist($p) {
-	if(function_exists('balise_INCLURE'))
+	if (function_exists('balise_INCLURE'))
 		return balise_INCLURE($p);
 	else
 		return balise_INCLURE_dist($p);
@@ -1103,21 +1103,21 @@ 
 
 		$_options = array();
 		if (isset($_contexte['ajax'])) {
-			$_options[] = preg_replace(",=>(.*)$,ims",'=> ($v=(\\1))?$v:true',$_contexte['ajax']);
+			$_options[] = preg_replace(",=>(.*)$,ims", '=> ($v=(\\1))?$v:true', $_contexte['ajax']);
 			unset($_contexte['ajax']);
 		}
 		if ($p->etoile) $_options[] = "'etoile'=>true";
-		$_options[] = "'compil'=>array(" . memoriser_contexte_compil($p) .")";
+		$_options[] = "'compil'=>array(".memoriser_contexte_compil($p).")";
 
-		$_l = 'array(' . join(",\n\t", $_contexte) .')';
+		$_l = 'array('.join(",\n\t", $_contexte).')';
 		if ($flag_env) $_l = "array_merge(\$Pile[0],$_l)";
 
-		$p->code = sprintf(CODE_RECUPERER_FOND, $f, $_l, join(',',$_options),"''");
+		$p->code = sprintf(CODE_RECUPERER_FOND, $f, $_l, join(',', $_options), "''");
 
 	} elseif (!isset($_contexte[1])) {
 			$msg = array('zbug_balise_sans_argument', array('balise' => ' INCLURE'));
 			erreur_squelette($msg, $p);
-	} else 		$p->code = 'charge_scripts(' . $_contexte[1] . ',false)';
+	} else 		$p->code = 'charge_scripts('.$_contexte[1].',false)';
 
 	$p->interdire_scripts = false; // la securite est assuree par recuperer_fond
 	return $p;
@@ -1141,14 +1141,14 @@ 
 		unset($_contexte[1]);
 
 		if (preg_match("/^\s*'[^']*'/s", $nom))
-			$nom = "'modeles/" . substr($nom,1);
+			$nom = "'modeles/".substr($nom, 1);
 		else $nom = "'modeles/' . $nom";
 
 		// Incoherence dans la syntaxe du contexte. A revoir.
 		// Reserver la cle primaire de la boucle courante si elle existe
 		if (isset($p->boucles[$p->id_boucle]->primary)) {
 			$primary = $p->boucles[$p->id_boucle]->primary;
-			if (!strpos($primary,',')) {
+			if (!strpos($primary, ',')) {
 				$id = champ_sql($primary, $p);
 				$_contexte[] = "'$primary'=>".$id;
 				$_contexte[] = "'id'=>".$id;
@@ -1161,12 +1161,12 @@ 
 
 		$_options = memoriser_contexte_compil($p);
 		$_options = "'compil'=>array($_options), 'trim'=>true";
-	  if (isset($_contexte['ajax'])){
-		  $_options .= ", ".preg_replace(",=>(.*)$,ims",'=> ($v=(\\1))?$v:true',$_contexte['ajax']);
+	  if (isset($_contexte['ajax'])) {
+		  $_options .= ", ".preg_replace(",=>(.*)$,ims", '=> ($v=(\\1))?$v:true', $_contexte['ajax']);
 			unset($_contexte['ajax']);
 	  }
 
-		$page = sprintf(CODE_RECUPERER_FOND, $nom, 'array(' . join(',', $_contexte) .')', $_options, _q($connect));
+		$page = sprintf(CODE_RECUPERER_FOND, $nom, 'array('.join(',', $_contexte).')', $_options, _q($connect));
 
 		$p->code = "\n\t(((\$recurs=(isset(\$Pile[0]['recurs'])?\$Pile[0]['recurs']:0))>=5)? '' :\n\t$page)\n";
 
@@ -1182,9 +1182,9 @@ 
 // #SET{nom,valeur}
 // la balise renvoie la valeur
 // http://doc.spip.org/@balise_SET_dist
-function balise_SET_dist($p){
-	$_nom = interprete_argument_balise(1,$p);
-	$_val = interprete_argument_balise(2,$p);
+function balise_SET_dist($p) {
+	$_nom = interprete_argument_balise(1, $p);
+	$_val = interprete_argument_balise(2, $p);
 
 	if (!$_nom OR !$_val) {
 		$err_b_s_a = array('zbug_balise_sans_argument', array('balise' => 'SET'));
@@ -1229,13 +1229,13 @@ 
  * 		Pile complétée par le code à générer
 **/
 function balise_DOUBLONS_dist($p) {
-	if ($type = interprete_argument_balise(1,$p)) {
-		if ($famille = interprete_argument_balise(2,$p))
-			$type .= '.' . $famille;
+	if ($type = interprete_argument_balise(1, $p)) {
+		if ($famille = interprete_argument_balise(2, $p))
+			$type .= '.'.$famille;
 		$p->code = '$doublons['.$type.']';
 		if (!$p->etoile)
 			$p->code = 'array_filter(array_map("intval",explode(",",'
-				. $p->code . ')))';
+				. $p->code.')))';
 	}
 	else
 		$p->code = '$doublons';
@@ -1254,13 +1254,13 @@ 
 //
 // http://doc.spip.org/@balise_PIPELINE_dist
 function balise_PIPELINE_dist($p) {
-	$_pipe = interprete_argument_balise(1,$p);
+	$_pipe = interprete_argument_balise(1, $p);
 	if (!$_pipe) {
 		$err_b_s_a = array('zbug_balise_sans_argument', array('balise' => 'PIPELINE'));
 		erreur_squelette($err_b_s_a, $p);
 	} else {
-		$_flux = interprete_argument_balise(2,$p);
-		$_flux = $_flux?$_flux:"''";
+		$_flux = interprete_argument_balise(2, $p);
+		$_flux = $_flux ? $_flux : "''";
 		$p->code = "pipeline( $_pipe , $_flux )";
 		$p->interdire_scripts = false;
 	}
@@ -1289,7 +1289,7 @@ 
 //
 // http://doc.spip.org/@balise_TOTAL_UNIQUE_dist
 function balise_TOTAL_UNIQUE_dist($p) {
-	$_famille = interprete_argument_balise(1,$p);
+	$_famille = interprete_argument_balise(1, $p);
 	$_famille = $_famille ? $_famille : "''";
 	$p->code = "unique('', $_famille, true)";
 	return $p;
@@ -1303,13 +1303,13 @@ 
 // http://doc.spip.org/@balise_ARRAY_dist
 function balise_ARRAY_dist($p) {
 	$_code = array();
-	$n=1;
+	$n = 1;
 	do {
-		$_key = interprete_argument_balise($n++,$p);
-		$_val = interprete_argument_balise($n++,$p);
+		$_key = interprete_argument_balise($n++, $p);
+		$_val = interprete_argument_balise($n++, $p);
 		if ($_key AND $_val) $_code[] = "$_key => $_val";
 	} while ($_key && $_val);
-	$p->code = 'array(' . join(', ',$_code).')';
+	$p->code = 'array('.join(', ', $_code).')';
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -1322,10 +1322,10 @@ 
  */
 function balise_LISTE_dist($p) {
 	$_code = array();
-	$n=1;
-	while ($_val = interprete_argument_balise($n++,$p))
+	$n = 1;
+	while ($_val = interprete_argument_balise($n++, $p))
 		$_code[] = $_val;
-	$p->code = 'array(' . join(', ',$_code).')';
+	$p->code = 'array('.join(', ', $_code).')';
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -1338,11 +1338,11 @@ 
 	$_code = array();
 	$p->descr['session'] = true; // faire un cache par session
 
-	$n=1;
-	while ($_v = interprete_argument_balise($n++,$p))
+	$n = 1;
+	while ($_v = interprete_argument_balise($n++, $p))
 		$_code[] = $_v;
 
-	$p->code = '((function_exists("autoriser")||include_spip("inc/autoriser"))&&autoriser(' . join(', ',$_code).')?" ":"")';
+	$p->code = '((function_exists("autoriser")||include_spip("inc/autoriser"))&&autoriser('.join(', ', $_code).')?" ":"")';
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -1351,9 +1351,9 @@ 
 // Afficher des informations sur les plugins dans le site public
 // http://doc.spip.org/@balise_PLUGIN_dist
 function balise_PLUGIN_dist($p) {
-	$plugin = interprete_argument_balise(1,$p);
+	$plugin = interprete_argument_balise(1, $p);
 	$plugin = isset($plugin) ? str_replace('\'', '"', $plugin) : '""';
-	$type_info = interprete_argument_balise(2,$p);
+	$type_info = interprete_argument_balise(2, $p);
 	$type_info = isset($type_info) ? str_replace('\'', '"', $type_info) : '"est_actif"';
 
 	$f = chercher_filtre('info_plugin');
@@ -1364,9 +1364,9 @@ 
 // Appelle la fonction inc_aider_dist
 // http://doc.spip.org/@balise_AIDER_dist
 function balise_AIDER_dist($p) {
-	$_motif = interprete_argument_balise(1,$p);
-	$s = "'" . addslashes($p->descr['sourcefile']) . "'";
-	$aider = charger_fonction('aider','inc');
+	$_motif = interprete_argument_balise(1, $p);
+	$s = "'".addslashes($p->descr['sourcefile'])."'";
+	$aider = charger_fonction('aider', 'inc');
 	$p->code = "((\$aider=charger_fonction('aider','inc'))?\$aider($_motif,$s, \$Pile[0]):'')";
 	return $p;
 }
@@ -1374,10 +1374,10 @@ 
 // Insertion du contexte des formulaires charger/verifier/traiter
 // avec les hidden de l'url d'action
 // http://doc.spip.org/@balise_ACTION_FORMULAIRE
-function balise_ACTION_FORMULAIRE($p){
-	if (!$_url = interprete_argument_balise(1,$p))
+function balise_ACTION_FORMULAIRE($p) {
+	if (!$_url = interprete_argument_balise(1, $p))
 		$_url = "@\$Pile[0]['action']";
-	if (!$_form = interprete_argument_balise(2,$p))
+	if (!$_form = interprete_argument_balise(2, $p))
 		$_form = "@\$Pile[0]['form']";
 
 	// envoyer le nom du formulaire que l'on traite
@@ -1410,18 +1410,18 @@ 
  * @param unknown_type $p
  * @return unknown
  */
-function balise_BOUTON_ACTION_dist($p){
+function balise_BOUTON_ACTION_dist($p) {
 
 	$args = array();
-	for ($k=1;$k<=6;$k++){
-		$_a = interprete_argument_balise($k,$p);
-		if (!$_a) $_a="''";
+	for ($k = 1; $k <= 6; $k++) {
+		$_a = interprete_argument_balise($k, $p);
+		if (!$_a) $_a = "''";
 	  $args[] = $_a;
 	}
 	// supprimer les args vides
-	while(end($args)=="''" AND count($args)>2)
+	while (end($args) == "''" AND count($args) > 2)
 		array_pop($args);
-	$args = implode(",",$args);
+	$args = implode(",", $args);
 
 	$bouton_action = chercher_filtre("bouton_action");
 	$p->code = "$bouton_action($args)";
@@ -1458,7 +1458,7 @@ 
  * @param unknown_type $liste
  * @return unknown
  */
-function balise_TRI_dist($p, $liste='true') {
+function balise_TRI_dist($p, $liste = 'true') {
 	$b = $p->nom_boucle ? $p->nom_boucle : $p->descr['id_mere'];
 
 	// s'il n'y a pas de nom de boucle, on ne peut pas trier
@@ -1483,21 +1483,21 @@ 
 		return $p;
 	}
 
-	$_champ = interprete_argument_balise(1,$p);
+	$_champ = interprete_argument_balise(1, $p);
 	// si pas de champ, renvoyer le critere de tri utilise
-	if (!$_champ){
+	if (!$_champ) {
 		$p->code = $boucle->modificateur['tri_champ'];
 		return $p;
 	}
 	// forcer la jointure si besoin, et si le champ est statique
-	if (preg_match(",^'([\w.]+)'$,i",$_champ,$m)){
+	if (preg_match(",^'([\w.]+)'$,i", $_champ, $m)) {
 		index_pile($b, $m[1], $p->boucles);
 	}
 
-	$_libelle = interprete_argument_balise(2,$p);
-	$_libelle = $_libelle?$_libelle:$_champ;
+	$_libelle = interprete_argument_balise(2, $p);
+	$_libelle = $_libelle ? $_libelle : $_champ;
 
-	$_class = interprete_argument_balise(3,$p);
+	$_class = interprete_argument_balise(3, $p);
 	// si champ = ">" c'est un lien vers le tri croissant : de gauche a droite ==> 1
 	// si champ = "<" c'est un lien vers le tri decroissant : (sens inverse) == -1
 	$_issens = "in_array($_champ,array('>','<'))";
@@ -1507,7 +1507,7 @@ 
 	$_url = "parametre_url(self(),$_variable,\$s?$_sens:$_champ)";
 	$_on = "\$s?(".$boucle->modificateur['tri_sens']."==$_sens".'):('.$boucle->modificateur['tri_champ']."==$_champ)";
 
-	$p->code = "lien_ou_expose($_url,$_libelle,$_on".($_class?",$_class":"").")";
+	$p->code = "lien_ou_expose($_url,$_libelle,$_on".($_class ? ",$_class" : "").")";
 	//$p->code = "''";
 	$p->interdire_scripts = false;
 	return $p;
@@ -1525,7 +1525,7 @@ 
  * @param <type> $p
  * @return <type>
  */
-function balise_SAUTER_dist($p){
+function balise_SAUTER_dist($p) {
 	$id_boucle = $p->id_boucle;
 	$boucle = $p->boucles[$id_boucle];
 
@@ -1534,7 +1534,7 @@ 
 		erreur_squelette($msg, $p);
 	}
 	else {
-		$_saut = interprete_argument_balise(1,$p);
+		$_saut = interprete_argument_balise(1, $p);
 		$_compteur = "\$Numrows['$id_boucle']['compteur_boucle']";
 		$_total = "\$Numrows['$id_boucle']['total']";
 
@@ -1552,12 +1552,12 @@ 
  * @return <type>
  */
 function balise_PUBLIE_dist($p) {
-	if (!$_type = interprete_argument_balise(1,$p)){
+	if (!$_type = interprete_argument_balise(1, $p)) {
 		$_type = _q($p->type_requete);
-		$_id = champ_sql($p->boucles[$p->id_boucle]->primary,$p);
+		$_id = champ_sql($p->boucles[$p->id_boucle]->primary, $p);
 	}
 	else
-		$_id = interprete_argument_balise(2,$p);
+		$_id = interprete_argument_balise(2, $p);
 
 	$connect = $p->boucles[$p->id_boucle]->sql_serveur;
 
@@ -1581,11 +1581,11 @@ 
  * @param object $p
  * @return object
  */
-function balise_PRODUIRE_dist($p){
-	$balise_inclure = charger_fonction('INCLURE','balise');
+function balise_PRODUIRE_dist($p) {
+	$balise_inclure = charger_fonction('INCLURE', 'balise');
 	$p = $balise_inclure($p);
 
-	$p->code = str_replace('recuperer_fond(','produire_fond_statique(',$p->code);
+	$p->code = str_replace('recuperer_fond(', 'produire_fond_statique(', $p->code);
 
 	return $p;
 }
@@ -1597,9 +1597,9 @@ 
  * @param  $p
  * @return
  */
-function balise_LARGEUR_ECRAN_dist($p){
-	$_class = interprete_argument_balise(1,$p);
-	if (!$_class) $_class='null';
+function balise_LARGEUR_ECRAN_dist($p) {
+	$_class = interprete_argument_balise(1, $p);
+	if (!$_class) $_class = 'null';
 	$p->code = "(is_string($_class)?vide(\$GLOBALS['largeur_ecran']=$_class):(isset(\$GLOBALS['largeur_ecran'])?\$GLOBALS['largeur_ecran']:''))";
 	return $p;
 }

ecrire/public/quete.php

Spacing +73 added lines, -75 removed lines

@@ -29,7 +29,7 @@ 
  * @return array|bool|null
  */
 function quete_virtuel($id_article, $connect) {
-	return sql_getfetsel('virtuel', 'spip_articles', array("id_article=".intval($id_article), "statut='publie'"), '','','','',$connect);
+	return sql_getfetsel('virtuel', 'spip_articles', array("id_article=".intval($id_article), "statut='publie'"), '', '', '', '', $connect);
 }
 
 /**
@@ -42,24 +42,24 @@ 
  * @param string $connect
  * @return array
  */
-function quete_parent_lang($table,$id,$connect=''){
+function quete_parent_lang($table, $id, $connect = '') {
 	static $cache_quete = array();
 
 	if (!isset($cache_quete[$connect][$table][$id])) {
-		if (!isset($cache_quete[$connect][$table]['_select'])){
-			$trouver_table = charger_fonction('trouver_table','base');
-			if (!$desc=$trouver_table($table,$connect) OR !isset($desc['field']['id_rubrique']))
+		if (!isset($cache_quete[$connect][$table]['_select'])) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+			if (!$desc = $trouver_table($table, $connect) OR !isset($desc['field']['id_rubrique']))
 				// pas de parent rubrique, on passe
 				$cache_quete[$connect][$table]['_select'] = false; 
 			else {
-				$select = ($table=='spip_rubriques'?'id_parent':'id_rubrique');
-				$select .= isset($desc['field']['lang'])?", lang":"";
+				$select = ($table == 'spip_rubriques' ? 'id_parent' : 'id_rubrique');
+				$select .= isset($desc['field']['lang']) ? ", lang" : "";
 				$cache_quete[$connect][$table]['_select'] = $select;
 				$cache_quete[$connect][$table]['_id'] = id_table_objet(objet_type($table));
 			}
 		}
 		if ($cache_quete[$connect][$table]['_select'])
-			$cache_quete[$connect][$table][$id] = sql_fetsel($cache_quete[$connect][$table]['_select'], $table,$cache_quete[$connect][$table]['_id']."=".intval($id),'','','','',$connect);
+			$cache_quete[$connect][$table][$id] = sql_fetsel($cache_quete[$connect][$table]['_select'], $table, $cache_quete[$connect][$table]['_id']."=".intval($id), '', '', '', '', $connect);
 	}
 	return isset($cache_quete[$connect][$table][$id]) ? $cache_quete[$connect][$table][$id] : null;
 }
@@ -76,10 +76,10 @@ 
  * @param string $connect
  * @return int
  */
-function quete_parent($id_rubrique, $connect='') {
+function quete_parent($id_rubrique, $connect = '') {
 	if (!$id_rubrique = intval($id_rubrique))
 		return 0;
-	$id_parent = quete_parent_lang('spip_rubriques',$id_rubrique,$connect);
+	$id_parent = quete_parent_lang('spip_rubriques', $id_rubrique, $connect);
 	return $id_parent['id_parent'];
 }
 
@@ -95,7 +95,7 @@ 
  * @return
  */
 function quete_rubrique($id_article, $serveur) {
-	$id_parent = quete_parent_lang('spip_articles',$id_article,$serveur);
+	$id_parent = quete_parent_lang('spip_articles', $id_article, $serveur);
 	return $id_parent['id_rubrique'];
 }
 
@@ -109,7 +109,7 @@ 
  * @param string $connect
  * @return int
  */
-function quete_profondeur($id, $connect='') {
+function quete_profondeur($id, $connect = '') {
 	$n = 0;
 	while ($id) {
 		$n++;
@@ -125,14 +125,14 @@ 
  * @param string $serveur
  * @return string
  */
-function quete_condition_postdates($champ_date, $serveur='', $ignore_previsu=false) {
+function quete_condition_postdates($champ_date, $serveur = '', $ignore_previsu = false) {
 	if (defined('_VAR_PREVIEW') AND _VAR_PREVIEW AND !$ignore_previsu)
 		return "1=1";
 	return
 	  (isset($GLOBALS['meta']['date_prochain_postdate'])
 	     AND $GLOBALS['meta']['date_prochain_postdate'] > time())
-			? "$champ_date<".sql_quote(date('Y-m-d H:i:s',$GLOBALS['meta']['date_prochain_postdate']),$serveur)
-	    : "1=1" ;
+			? "$champ_date<".sql_quote(date('Y-m-d H:i:s', $GLOBALS['meta']['date_prochain_postdate']), $serveur)
+	    : "1=1";
 }
 
 
@@ -149,66 +149,66 @@ 
  *  serveur de BDD
  * @return array
  */
-function quete_condition_statut($mstatut,$previsu,$publie, $serveur='', $ignore_previsu=false){
+function quete_condition_statut($mstatut, $previsu, $publie, $serveur = '', $ignore_previsu = false) {
 	static $cond = array();
 	$key = func_get_args();
-	$key = implode("-",$key);
+	$key = implode("-", $key);
 	if (isset($cond[$key])) return $cond[$key];
 
 	$liste = $publie;
 	if (defined('_VAR_PREVIEW') AND _VAR_PREVIEW AND !$ignore_previsu)
 		$liste = $previsu;
 	$not = false;
-	if (strncmp($liste,'!',1)==0){
+	if (strncmp($liste, '!', 1) == 0) {
 		$not = true;
-	  $liste = substr($liste,1);
+	  $liste = substr($liste, 1);
 	}
 	// '' => ne rien afficher, '!'=> ne rien filtrer
 	if (!strlen($liste))
-		return $cond[$key]=($not?"1=1":"'0=1'");
+		return $cond[$key] = ($not ? "1=1" : "'0=1'");
 
-	$liste = explode(',',$liste);
+	$liste = explode(',', $liste);
 	$where = array();
-	foreach($liste as $k=>$v) {
+	foreach ($liste as $k=>$v) {
 		// filtrage /auteur pour limiter les objets d'un statut (prepa en general)
 		// a ceux de l'auteur identifie
-		if (strpos($v,"/")!==false){
-			$v = explode("/",$v);
+		if (strpos($v, "/") !== false) {
+			$v = explode("/", $v);
 			$filtre = end($v);
 			$v = reset($v);
-			$v = preg_replace(",\W,","",$v);
-			if ($filtre=="auteur"
+			$v = preg_replace(",\W,", "", $v);
+			if ($filtre == "auteur"
 				AND isset($GLOBALS['visiteur_session']['id_auteur'])
 				AND intval($GLOBALS['visiteur_session']['id_auteur'])
-				AND (strpos($mstatut,".")!==false)
-			  AND $objet = explode(".",$mstatut)
+				AND (strpos($mstatut, ".") !== false)
+			  AND $objet = explode(".", $mstatut)
 				AND $id_table = reset($objet)
-			  AND $objet = objet_type($id_table)){
+			  AND $objet = objet_type($id_table)) {
 				$primary = id_table_objet($objet);
-				$where[] = "($mstatut<>".sql_quote($v)." OR $id_table.$primary IN (".sql_get_select("ssss.id_objet","spip_auteurs_liens AS ssss","ssss.objet=".sql_quote($objet)." AND ssss.id_auteur=".intval($GLOBALS['visiteur_session']['id_auteur']),'','','','',$serveur)."))";
+				$where[] = "($mstatut<>".sql_quote($v)." OR $id_table.$primary IN (".sql_get_select("ssss.id_objet", "spip_auteurs_liens AS ssss", "ssss.objet=".sql_quote($objet)." AND ssss.id_auteur=".intval($GLOBALS['visiteur_session']['id_auteur']), '', '', '', '', $serveur)."))";
 			}
 			// ignorer ce statut si on ne sait pas comment le filtrer
 			else
 				$v = "";
 		}
 		// securite
-		$liste[$k] = preg_replace(",\W,","",$v);
+		$liste[$k] = preg_replace(",\W,", "", $v);
 	}
 	$liste = array_filter($liste);
-  if (count($liste)==1){
-		$where[] = array('=', $mstatut, sql_quote(reset($liste),$serveur));
+  if (count($liste) == 1) {
+		$where[] = array('=', $mstatut, sql_quote(reset($liste), $serveur));
   }
   else {
-	  $where[] = sql_in($mstatut,$liste,$not,$serveur);
+	  $where[] = sql_in($mstatut, $liste, $not, $serveur);
   }
 
-	while (count($where)>1){
-		$and = array('AND',array_pop($where),array_pop($where));
+	while (count($where) > 1) {
+		$and = array('AND', array_pop($where), array_pop($where));
 		$where[] = $and;
 	}
 	$cond[$key] = reset($where);
 	if ($not)
-		$cond[$key] = array('NOT',$cond[$key]);
+		$cond[$key] = array('NOT', $cond[$key]);
 
 	return $cond[$key];
 }
@@ -222,8 +222,8 @@ 
  * @param string $serveur
  * @return array|bool|null
  */
-function quete_fichier($id_document, $serveur='') {
-	return sql_getfetsel('fichier', 'spip_documents', ("id_document=" . intval($id_document)),	'',array(), '', '', $serveur);
+function quete_fichier($id_document, $serveur = '') {
+	return sql_getfetsel('fichier', 'spip_documents', ("id_document=".intval($id_document)), '', array(), '', '', $serveur);
 }
 
 /**
@@ -233,8 +233,8 @@ 
  * @param string $serveur
  * @return array|bool
  */
-function quete_document($id_document, $serveur='') {
-	return sql_fetsel('*', 'spip_documents', ("id_document=" . intval($id_document)),	'',array(), '', '', $serveur);
+function quete_document($id_document, $serveur = '') {
+	return sql_fetsel('*', 'spip_documents', ("id_document=".intval($id_document)), '', array(), '', '', $serveur);
 }
 
 /**
@@ -247,8 +247,8 @@ 
  * @return array|bool|null
  */
 function quete_meta($nom, $serveur) {
-	return sql_getfetsel("valeur", "spip_meta", "nom=" . sql_quote($nom),
-			     '','','','',$serveur);
+	return sql_getfetsel("valeur", "spip_meta", "nom=".sql_quote($nom),
+			     '', '', '', '', $serveur);
 }
 
 /**
@@ -277,13 +277,12 @@ 
 				return "$on[2].$on[3]";
 			else {
 				$taille = @getimagesize($on[0]);
-				$off = ($onoff != 'ON') ? '' :
-					$chercher_logo($id, $type, 'off');
+				$off = ($onoff != 'ON') ? '' : $chercher_logo($id, $type, 'off');
 				// on retourne une url du type IMG/artonXX?timestamp
 				// qui permet de distinguer le changement de logo
 				// et placer un expire sur le dossier IMG/
-				return array ($on[0].($on[4]?"?$on[4]":""),
-					($off ? $off[0] . ($off[4]?"?$off[4]":"") : ''),
+				return array($on[0].($on[4] ? "?$on[4]" : ""),
+					($off ? $off[0].($off[4] ? "?$off[4]" : "") : ''),
 					(!$taille ? '' : (" ".$taille[3])));
 			}
 		}
@@ -308,18 +307,18 @@ 
  * @param string $connect
  * @return bool|string
  */
-function quete_logo_file($row, $connect=NULL) {
+function quete_logo_file($row, $connect = NULL) {
 	include_spip('inc/documents');
 	$logo = vignette_logo_document($row, $connect);
 	if (!$logo) $logo = image_du_document($row);
-	if (!$logo){
-		$f = charger_fonction('vignette','inc');
+	if (!$logo) {
+		$f = charger_fonction('vignette', 'inc');
 		$logo = $f($row['extension'], false);
 	}
 	// si c'est une vignette type doc, la renvoyer direct
-	if (strcmp($logo,_DIR_PLUGINS)==0
-		OR strcmp($logo,_DIR_PLUGINS_DIST)==0
-		OR strcmp($logo,_DIR_RACINE.'prive/')==0)
+	if (strcmp($logo, _DIR_PLUGINS) == 0
+		OR strcmp($logo, _DIR_PLUGINS_DIST) == 0
+		OR strcmp($logo, _DIR_RACINE.'prive/') == 0)
 		return $logo;
 	return get_spip_doc($logo);
 }
@@ -347,10 +346,10 @@ 
  *   serveur
  * @return string
  */
-function quete_logo_document($row, $lien, $align, $mode_logo, $x, $y, $connect=NULL) {
+function quete_logo_document($row, $lien, $align, $mode_logo, $x, $y, $connect = NULL) {
 	include_spip('inc/documents');
 	$logo = '';
-	if (!in_array($mode_logo,array('icone','apercu')))
+	if (!in_array($mode_logo, array('icone', 'apercu')))
 		$logo = vignette_logo_document($row, $connect);
 	// si on veut explicitement la vignette, ne rien renvoyer si il n'y en a pas
 	if ($mode_logo == 'vignette' AND !$logo)
@@ -369,7 +368,7 @@ 
  * @param string $connect
  * @return string
  */
-function vignette_logo_document($row, $connect='')
+function vignette_logo_document($row, $connect = '')
 {
 	if (!$row['id_vignette']) return '';
 	$fichier = quete_fichier($row['id_vignette'], $connect);
@@ -381,7 +380,7 @@ 
 	$f = get_spip_doc($fichier);
 	if ($f AND @file_exists($f)) return $f;
 	if ($row['mode'] !== 'vignette') return '';
-	return generer_url_entite($row['id_document'], 'document','','', $connect);
+	return generer_url_entite($row['id_document'], 'document', '', '', $connect);
 }
 
 /**
@@ -398,41 +397,40 @@ 
  * @param string $connect
  * @return bool|string
  */
-function calcul_exposer ($id, $prim, $reference, $parent, $type, $connect='') {
+function calcul_exposer($id, $prim, $reference, $parent, $type, $connect = '') {
 	static $exposer = array();
 
 	// Que faut-il exposer ? Tous les elements de $reference
 	// ainsi que leur hierarchie ; on ne fait donc ce calcul
 	// qu'une fois (par squelette) et on conserve le resultat
 	// en static.
-	if (!isset($exposer[$m=md5(serialize($reference))][$prim])) {
-		$principal = isset($reference[$type])?$reference[$type]:
-			// cas de la pagination indecte @xx qui positionne la page avec l'id xx
+	if (!isset($exposer[$m = md5(serialize($reference))][$prim])) {
+		$principal = isset($reference[$type]) ? $reference[$type] : // cas de la pagination indecte @xx qui positionne la page avec l'id xx
 			// et donne la reference dynamique @type=xx dans le contexte
-			(isset($reference["@$type"])?$reference["@$type"]:'');
+			(isset($reference["@$type"]) ? $reference["@$type"] : '');
 		// le parent fournit en argument est le parent de $id, pas celui de $principal
 		// il n'est donc pas utile
 		$parent = 0;
 		if (!$principal) { // regarder si un enfant est dans le contexte, auquel cas il expose peut etre le parent courant
-			$enfants = array('id_rubrique'=>array('id_article'),'id_groupe'=>array('id_mot'));
+			$enfants = array('id_rubrique'=>array('id_article'), 'id_groupe'=>array('id_mot'));
 			if (isset($enfants[$type]))
-				foreach($enfants[$type] as $t)
+				foreach ($enfants[$type] as $t)
 					if (isset($reference[$t])
 						// cas de la reference donnee dynamiquement par la pagination
 						OR isset($reference["@$t"])) {
 						$type = $t;
-						$principal = isset($reference[$type])?$reference[$type]:$reference["@$type"];
+						$principal = isset($reference[$type]) ? $reference[$type] : $reference["@$type"];
 						continue;
 					}
 		}
 		$exposer[$m][$type] = array();
 		if ($principal) {
-			$principaux = is_array($principal)?$principal:array($principal);
-			foreach($principaux as $principal){
+			$principaux = is_array($principal) ? $principal : array($principal);
+			foreach ($principaux as $principal) {
 				$exposer[$m][$type][$principal] = true;
-				if ($type == 'id_mot'){
+				if ($type == 'id_mot') {
 					if (!$parent) {
-						$parent = sql_getfetsel('id_groupe','spip_mots',"id_mot=" . intval($principal), '','','','',$connect);
+						$parent = sql_getfetsel('id_groupe', 'spip_mots', "id_mot=".intval($principal), '', '', '', '', $connect);
 					}
 					if ($parent)
 						$exposer[$m]['id_groupe'][$parent] = true;
@@ -442,7 +440,7 @@ 
 				  	if ($type == 'id_rubrique')
 				  		$parent = $principal;
 				  	if ($type == 'id_article') {
-						$parent = quete_rubrique($principal,$connect);
+						$parent = quete_rubrique($principal, $connect);
 				  	}
 				  }
 				  do { $exposer[$m]['id_rubrique'][$parent] = true; }
@@ -466,21 +464,21 @@ 
  * @param objetc $iter
  * @return int
  */
-function quete_debut_pagination($primary,$valeur,$pas,$iter){
+function quete_debut_pagination($primary, $valeur, $pas, $iter) {
 	// on ne devrait pas arriver ici si la cle primaire est inexistante
 	// ou composee, mais verifions
-	if (!$primary OR preg_match('/[,\s]/',$primary))
+	if (!$primary OR preg_match('/[,\s]/', $primary))
 		return 0;
 
 	$pos = 0;
-	while ($row = $iter->fetch() AND $row[$primary]!=$valeur){
+	while ($row = $iter->fetch() AND $row[$primary] != $valeur) {
 		$pos++;
 	}
 	// si on a pas trouve
-	if ($row[$primary]!=$valeur)
+	if ($row[$primary] != $valeur)
 		return 0;
 
 	// sinon, calculer le bon numero de page
-	return floor($pos/$pas)*$pas;
+	return floor($pos / $pas) * $pas;
 }
 ?>

ecrire/public/styliser.php

Spacing +23 added lines, -23 removed lines

@@ -31,12 +31,12 @@ 
  *
  * http://doc.spip.org/@public_styliser_dist
  */
-function public_styliser_dist($fond, $contexte, $lang='', $connect='') {
+function public_styliser_dist($fond, $contexte, $lang = '', $connect = '') {
 	static $styliser_par_z;
 
 	// s'assurer que le fond est licite
 	// car il peut etre construit a partir d'une variable d'environnement
-	if (strpos($fond,"../")!==false OR strncmp($fond,'/',1)==0)
+	if (strpos($fond, "../") !== false OR strncmp($fond, '/', 1) == 0)
 		$fond = "404";
   
 	// Choisir entre $fond-dist.html, $fond=7.html, etc?
@@ -49,7 +49,7 @@ 
 	// ne rien dire si on ne trouve pas, 
 	// c'est l'appelant qui sait comment gerer la situation
 	// ou les plugins qui feront mieux dans le pipeline
-	$squelette = trouver_fond($fond,"",true);
+	$squelette = trouver_fond($fond, "", true);
 	$ext = $squelette['extension'];
 
 	$flux = array(
@@ -66,7 +66,7 @@ 
 
 	if (test_espace_prive() OR defined('_ZPIP')) {
 		if (!$styliser_par_z)
-			$styliser_par_z = charger_fonction('styliser_par_z','public');
+			$styliser_par_z = charger_fonction('styliser_par_z', 'public');
 		$flux = $styliser_par_z($flux);
 	}
 
@@ -78,25 +78,25 @@ 
 	return array($squelette, $ext, $ext, "$squelette.$ext");
 }
 
-function styliser_par_objets($flux){
+function styliser_par_objets($flux) {
 	if (test_espace_prive()
 		AND !$squelette = $flux['data']
-	  AND strncmp($flux['args']['fond'],'prive/objets/',13)==0
-	  AND $echafauder = charger_fonction('echafauder','prive',true)) {
-		if (strncmp($flux['args']['fond'],'prive/objets/liste/',19)==0){
-			$table = table_objet(substr($flux['args']['fond'],19));
+	  AND strncmp($flux['args']['fond'], 'prive/objets/', 13) == 0
+	  AND $echafauder = charger_fonction('echafauder', 'prive', true)) {
+		if (strncmp($flux['args']['fond'], 'prive/objets/liste/', 19) == 0) {
+			$table = table_objet(substr($flux['args']['fond'], 19));
 			$table_sql = table_objet_sql($table);
 			$objets = lister_tables_objets_sql();
 			if (isset($objets[$table_sql]))
-				$flux['data'] = $echafauder($table,$table,$table_sql,"prive/objets/liste/objets",$flux['args']['ext']);
+				$flux['data'] = $echafauder($table, $table, $table_sql, "prive/objets/liste/objets", $flux['args']['ext']);
 		}
-		if (strncmp($flux['args']['fond'],'prive/objets/contenu/',21)==0){
-			$type = substr($flux['args']['fond'],21);
+		if (strncmp($flux['args']['fond'], 'prive/objets/contenu/', 21) == 0) {
+			$type = substr($flux['args']['fond'], 21);
 			$table = table_objet($type);
 			$table_sql = table_objet_sql($table);
 			$objets = lister_tables_objets_sql();
 			if (isset($objets[$table_sql]))
-				$flux['data'] = $echafauder($type,$table,$table_sql,"prive/objets/contenu/objet",$flux['args']['ext']);
+				$flux['data'] = $echafauder($type, $table, $table_sql, "prive/objets/contenu/objet", $flux['args']['ext']);
 		}
 	}
 	return $flux;
@@ -125,33 +125,33 @@ 
 		include_spip('public/quete');
 		$l = urls_liste_objets(false);
 		// placer la rubrique en tete des objets
-		$l = array_diff($l,array('rubrique'));
+		$l = array_diff($l, array('rubrique'));
 		array_unshift($l, 'rubrique');
-		foreach($l as $objet){
+		foreach ($l as $objet) {
 			$id = id_table_objet($objet);
 			if (!isset($liste_objets[$id]))
-				$liste_objets[$id] = objet_type($objet,false);
+				$liste_objets[$id] = objet_type($objet, false);
 		}
 	}
-	$c = array_intersect_key($contexte,$liste_objets);
+	$c = array_intersect_key($contexte, $liste_objets);
 	if (!count($c)) return false;
 
-	$c = array_map('intval',$c);
+	$c = array_map('intval', $c);
 	$s = serialize($c);
 	if (isset($quete[$s]))
 		return $quete[$s];
 
-	if (isset($c['id_rubrique']) AND $r = $c['id_rubrique']){
+	if (isset($c['id_rubrique']) AND $r = $c['id_rubrique']) {
 		unset($c['id_rubrique']);
 		$c = array('id_rubrique'=>$r) + $c;
 	}
 
-	foreach($c as $_id=>$id) {
+	foreach ($c as $_id=>$id) {
 		if ($id
-		  AND $row = quete_parent_lang(table_objet_sql($liste_objets[$_id]),$id)) {
+		  AND $row = quete_parent_lang(table_objet_sql($liste_objets[$_id]), $id)) {
 			$lang = isset($row['lang']) ? $row['lang'] : '';
-			if ($_id=='id_rubrique' OR (isset($row['id_rubrique']) AND $id=$row['id_rubrique']))
-				return $quete[$s] = array ($id, $lang);
+			if ($_id == 'id_rubrique' OR (isset($row['id_rubrique']) AND $id = $row['id_rubrique']))
+				return $quete[$s] = array($id, $lang);
 		}
 	}
 	return $quete[$s] = false;

ecrire/public/jointures.php

Spacing +50 added lines, -50 removed lines

@@ -29,8 +29,8 @@ 
  * @param string $champ
  * @return array
  */
-function decompose_champ_id_objet($champ){
-	if (($champ!=='id_objet') AND preg_match(',^id_([a-z_]+)$,', $champ, $regs)){
+function decompose_champ_id_objet($champ) {
+	if (($champ !== 'id_objet') AND preg_match(',^id_([a-z_]+)$,', $champ, $regs)) {
 		return array('id_objet', 'objet', objet_type($regs[1]));
 	}
 	return $champ;
@@ -45,14 +45,14 @@ 
  * @param array $desc
  * @return array
  */
-function trouver_champs_decomposes($champ, $desc){
+function trouver_champs_decomposes($champ, $desc) {
 	if (!is_array($desc) // on ne se risque pas en conjectures si on ne connait pas la table
 	    OR array_key_exists($champ, $desc['field'])
 	)
 		return array($champ);
-	if (is_array($decompose = decompose_champ_id_objet($champ))){
+	if (is_array($decompose = decompose_champ_id_objet($champ))) {
 		array_pop($decompose);
-		if (count(array_intersect($decompose, array_keys($desc['field'])))==count($decompose))
+		if (count(array_intersect($decompose, array_keys($desc['field']))) == count($decompose))
 			return $decompose;
 	}
 	return array($champ);
@@ -78,14 +78,14 @@ 
  *     flag pour savoir si le critere est conditionnel ou non
  * @return string
  */
-function calculer_jointure(&$boucle, $depart, $arrivee, $col = '', $cond = false, $max_liens=5){
+function calculer_jointure(&$boucle, $depart, $arrivee, $col = '', $cond = false, $max_liens = 5) {
 	// les jointures minimales sont optimales :
 	// on contraint le nombre d'etapes en l'augmentant
 	// jusqu'a ce qu'on trouve une jointure ou qu'on atteigne la limite maxi 
 	$max = 1;
 	$res = false;
-	$milieu_exclus = ($col?$col:array());
-	while ($max<=$max_liens AND !$res){
+	$milieu_exclus = ($col ? $col : array());
+	while ($max <= $max_liens AND !$res) {
 		$res = calculer_chaine_jointures($boucle, $depart, $arrivee, array(), $milieu_exclus, $max);
 		$max++;
 	}
@@ -124,18 +124,18 @@ 
  * @param bool $echap
  * @return string
  */
-function fabrique_jointures(&$boucle, $res, $cond = false, $desc = array(), $nom = '', $col = '', $echap = true){
+function fabrique_jointures(&$boucle, $res, $cond = false, $desc = array(), $nom = '', $col = '', $echap = true) {
 	static $num = array();
 	$id_table = "";
 	$cpt = &$num[$boucle->descr['nom']][$boucle->descr['gram']][$boucle->id_boucle];
-	foreach ($res as $cle => $r){
+	foreach ($res as $cle => $r) {
 		list($d, $a, $j) = $r;
 		if (!$id_table) $id_table = $d;
 		$n = ++$cpt;
-		if (is_array($j)){ // c'est un lien sur un champ du type id_objet,objet,'article'
+		if (is_array($j)) { // c'est un lien sur un champ du type id_objet,objet,'article'
 			list($j1, $j2, $obj, $type) = $j;
 			// trouver de quel cote est (id_objet,objet)
-			if ($j1=="id_$obj")
+			if ($j1 == "id_$obj")
 				$obj = "$id_table.$obj";
 			else
 				$obj = "L$n.$obj";
@@ -160,7 +160,7 @@ 
 	// de l'index principal et de l'index de jointure (non conditionnel! [6031])
 	// et operateur d'egalite (http://trac.rezo.net/trac/spip/ticket/477)
 
-	if ($pk = (isset($a[1]) && (count($boucle->from)==2) && !$cond)){
+	if ($pk = (isset($a[1]) && (count($boucle->from) == 2) && !$cond)) {
 		$pk = nogroupby_if($desc, $a[1], $col);
 	}
 
@@ -168,17 +168,17 @@ 
 	// si une seule jointure
 	// et si l'index de jointure est une primary key a l'arrivee !
 	if (!$pk
-	    AND (count($boucle->from)==2)
+	    AND (count($boucle->from) == 2)
 	        AND isset($a[1]['key']['PRIMARY KEY'])
-	            AND ($j==$a[1]['key']['PRIMARY KEY'])
+	            AND ($j == $a[1]['key']['PRIMARY KEY'])
 	)
 		$pk = true;
 
 	// la clause Group by est en conflit avec ORDER BY, a completer
 	$groups = liste_champs_jointures($nom, $desc, true);
-	if (!$pk) foreach ($groups as $id_prim){
+	if (!$pk) foreach ($groups as $id_prim) {
 		$id_field = $nom.'.'.$id_prim;
-		if (!in_array($id_field, $boucle->group)){
+		if (!in_array($id_field, $boucle->group)) {
 			$boucle->group[] = $id_field;
 		}
 	}
@@ -199,7 +199,7 @@ 
  * @param string|array $col
  * @return bool
  */
-function nogroupby_if($depart, $arrivee, $col){
+function nogroupby_if($depart, $arrivee, $col) {
 	$pk = $arrivee['key']['PRIMARY KEY'];
 	if (!$pk) return false;
 	$id_primary = $depart['key']['PRIMARY KEY'];
@@ -222,7 +222,7 @@ 
  * @param bool $primary
  * @return array
  */
-function liste_champs_jointures($nom, $desc, $primary = false){
+function liste_champs_jointures($nom, $desc, $primary = false) {
 
 	static $nojoin = array('idx', 'maj', 'date', 'statut');
 
@@ -262,7 +262,7 @@ 
  * @param array $join
  * @return array
  */
-function split_key($v, $join = array()){
+function split_key($v, $join = array()) {
 	foreach (preg_split('/,\s*/', $v) as $k) $join[$k] = $k;
 	return $join;
 }
@@ -285,7 +285,7 @@ 
  *  nombre maxi d'etapes
  * @return array
  */
-function calculer_chaine_jointures(&$boucle, $depart, $arrivee, $vu = array(), $milieu_exclus = array(), $max_liens = 5){
+function calculer_chaine_jointures(&$boucle, $depart, $arrivee, $vu = array(), $milieu_exclus = array(), $max_liens = 5) {
 	static $trouver_table;
 	if (!$trouver_table)
 		$trouver_table = charger_fonction('trouver_table', 'base');
@@ -294,12 +294,12 @@ 
 		$milieu_exclus = array($milieu_exclus);
 	// quand on a exclus id_objet comme cle de jointure, il faut aussi exclure objet
 	// faire une jointure sur objet tout seul n'a pas de sens
-	if (in_array('id_objet',$milieu_exclus) AND !in_array('objet',$milieu_exclus))
+	if (in_array('id_objet', $milieu_exclus) AND !in_array('objet', $milieu_exclus))
 		$milieu_exclus[] = 'objet';
 
 	list($dnom, $ddesc) = $depart;
 	list($anom, $adesc) = $arrivee;
-	if (!count($vu)){
+	if (!count($vu)) {
 		$vu[] = $dnom; // ne pas oublier la table de depart
 		$vu[] = $anom; // ne pas oublier la table d'arrivee
 	}
@@ -324,13 +324,13 @@ 
 		return array(array($dnom, array($adesc['table'], $adesc), array_shift($v)));
 
 	// regarder si l'on a (id_objet,objet) au depart et si on peut le mapper sur un id_xx
-	if (count(array_intersect(array('id_objet', 'objet'), $keys))==2){
+	if (count(array_intersect(array('id_objet', 'objet'), $keys)) == 2) {
 		// regarder si l'une des cles d'arrivee peut se decomposer en 
 		// id_objet,objet
 		// si oui on la prend
-		foreach ($akeys as $key){
+		foreach ($akeys as $key) {
 			$v = decompose_champ_id_objet($key);
-			if (is_array($v)){
+			if (is_array($v)) {
 				$objet = array_shift($v); // objet,'article'
 				array_unshift($v, $key); // id_article,objet,'article'
 				array_unshift($v, $objet); // id_objet,id_article,objet,'article'
@@ -342,9 +342,9 @@ 
 		// regarder si l'une des cles de depart peut se decomposer en 
 		// id_objet,objet a l'arrivee
 		// si oui on la prend
-		foreach ($keys as $key){
-			if (count($v = trouver_champs_decomposes($key, $adesc))>1){
-				if (count($v)==count(array_intersect($v, $akeys))){
+		foreach ($keys as $key) {
+			if (count($v = trouver_champs_decomposes($key, $adesc)) > 1) {
+				if (count($v) == count(array_intersect($v, $akeys))) {
 					$v = decompose_champ_id_objet($key); // id_objet,objet,'article'
 					array_unshift($v, $key); // id_article,id_objet,objet,'article'
 					return array(array($dnom, array($adesc['table'], $adesc), $v));
@@ -353,38 +353,38 @@ 
 		}
 	}
 	// si l'on voulait une jointure direct, c'est rate !
-	if ($max_liens<=1) return array();
+	if ($max_liens <= 1) return array();
 
 	// sinon essayer de passer par une autre table
 	$new = $vu;
-	foreach ($boucle->jointures as $v){
+	foreach ($boucle->jointures as $v) {
 		if ($v
 		    AND !in_array($v, $vu)
 		    AND $def = $trouver_table($v, $boucle->sql_serveur)
 			  AND !in_array($def['table_sql'], $vu)
-		){
+		) {
 			// ne pas tester les cles qui sont exclues a l'appel
 			// ie la cle de la jointure precedente
 			$test_cles = $milieu_exclus;
 			$new[] = $v;
 			$max_iter = 50; // securite
 			while (count($jointure_directe_possible = calculer_chaine_jointures($boucle, $depart, array($v, $def), $vu, $test_cles, 1))
-			       AND $max_iter--){
+			       AND $max_iter--) {
 				$jointure_directe_possible = reset($jointure_directe_possible);
 				$milieu = end($jointure_directe_possible);
 				$exclure_fin = $milieu_exclus;
-				if (is_string($milieu)){
+				if (is_string($milieu)) {
 					$exclure_fin[] = $milieu;
 					$test_cles[] = $milieu;
 				}
-				else{
+				else {
 					$exclure_fin = array_merge($exclure_fin, $milieu);
 					$test_cles = array_merge($test_cles, $milieu);
 				}
 				// essayer de rejoindre l'arrivee a partir de cette etape intermediaire
 				// sans repasser par la meme cle milieu, ni une cle deja vue !
-				$r = calculer_chaine_jointures($boucle, array($v, $def), $arrivee, $new, $exclure_fin, $max_liens-1);
-				if ($r){
+				$r = calculer_chaine_jointures($boucle, array($v, $def), $arrivee, $new, $exclure_fin, $max_liens - 1);
+				if ($r) {
 					array_unshift($r, $jointure_directe_possible);
 					return $r;
 				}
@@ -403,13 +403,13 @@ 
  * @param $keys
  * @return array
  */
-function trouver_cles_table($keys){
+function trouver_cles_table($keys) {
 	$res = array();
-	foreach ($keys as $v){
+	foreach ($keys as $v) {
 		if (!strpos($v, ","))
 			$res[$v] = 1;
 		else {
-			foreach (preg_split("/\s*,\s*/", $v) as $k){
+			foreach (preg_split("/\s*,\s*/", $v) as $k) {
 				$res[$k] = 1;
 			}
 		}
@@ -426,7 +426,7 @@ 
  * @param bool $checkarrivee
  * @return array|string
  */
-function trouver_champ_exterieur($cle, $joints, &$boucle, $checkarrivee = false){
+function trouver_champ_exterieur($cle, $joints, &$boucle, $checkarrivee = false) {
 	static $trouver_table = '';
 	if (!$trouver_table)
 		$trouver_table = charger_fonction('trouver_table', 'base');
@@ -438,28 +438,28 @@ 
 	if (!is_array($cle))
 		$cle = array($cle);
 
-	foreach ($joints as $k => $join){
-		if ($join && $table = $trouver_table($join, $boucle->sql_serveur)){
+	foreach ($joints as $k => $join) {
+		if ($join && $table = $trouver_table($join, $boucle->sql_serveur)) {
 			if (isset($table['field'])
 			    // verifier que toutes les cles cherchees sont la
-			    AND (count(array_intersect($cle, array_keys($table['field'])))==count($cle))
+			    AND (count(array_intersect($cle, array_keys($table['field']))) == count($cle))
 			        // si on sait ou on veut arriver, il faut que ca colle
-			        AND ($checkarrivee==false || $checkarrivee==$table['table'])
+			        AND ($checkarrivee == false || $checkarrivee == $table['table'])
 			)
 				return array($table['table'], $table);
 		}
 	}
 
 	// au premier coup, on essaye de decomposer, si possible
-	if (count($cle)==1
+	if (count($cle) == 1
 	    AND $c = reset($cle)
 	        AND is_array($decompose = decompose_champ_id_objet($c))
-	){
+	) {
 
 		$desc = $boucle->show;
 		// cas 1 : la cle id_xx est dans la table de depart
 		// -> on cherche uniquement id_objet,objet a l'arrivee
-		if (isset($desc['field'][$c])){
+		if (isset($desc['field'][$c])) {
 			$cle = array();
 			$cle[] = array_shift($decompose); // id_objet
 			$cle[] = array_shift($decompose); // objet
@@ -470,7 +470,7 @@ 
 			// id_objet,objet,zzz soit a l'arrivee
 		else {
 			$depart = liste_champs_jointures($desc['table'], $desc);
-			foreach ($depart as $d){
+			foreach ($depart as $d) {
 				$cle = array();
 				$cle[] = array_shift($decompose); // id_objet
 				$cle[] = array_shift($decompose); // objet
@@ -510,7 +510,7 @@ 
 		$jointures = $boucle->jointures;
 	}
 	$cle = trouver_champ_exterieur($champ, $jointures, $boucle);
-	if ($cle){
+	if ($cle) {
 		$desc = $boucle->show;
 		$cle = calculer_jointure($boucle, array($desc['id_table'], $desc), $cle, '', $cond);
 	}

ecrire/public/iterateur.php

Spacing +38 added lines, -38 removed lines

@@ -19,8 +19,8 @@ 
  * fourni dans le fichier iterateurs/xxx.php
  * 
  */
-class IterFactory{
-	public static function create($iterateur, $command, $info=null){
+class IterFactory {
+	public static function create($iterateur, $command, $info = null) {
 
 		// cas des SI {si expression} analises tres tot
 		// pour eviter le chargement de tout iterateur
@@ -38,11 +38,11 @@ 
 		// (il faudrait passer l'argument ->sql_serveur
 		// pour etre certain qu'on est sur un "php:")
 		if (class_exists($iterateur)) {
-			$a = isset($command['args']) ? $command['args'] : array() ;
+			$a = isset($command['args']) ? $command['args'] : array();
 
 			// permettre de passer un Iterateur directement {args #ITERATEUR} :
 			// si on recoit deja un iterateur en argument, on l'utilise
-			if (count($a)==1 and is_object($a[0]) and is_subclass_of($a[0], 'Iterator')) {
+			if (count($a) == 1 and is_object($a[0]) and is_subclass_of($a[0], 'Iterator')) {
 				$iter = $a[0];
 
 			// sinon, on cree un iterateur du type donne
@@ -51,11 +51,11 @@ 
 				// (pas glop)
 				try {
 					switch (count($a)) {
-						case 0:    $iter = new $iterateur();  break;
-						case 1:    $iter = new $iterateur($a[0]);  break;
-						case 2:    $iter = new $iterateur($a[0], $a[1]);  break;
-						case 3:    $iter = new $iterateur($a[0], $a[1], $a[2]);  break;
-						case 4:    $iter = new $iterateur($a[0], $a[1], $a[2], $a[3]);  break;
+						case 0:    $iter = new $iterateur(); break;
+						case 1:    $iter = new $iterateur($a[0]); break;
+						case 2:    $iter = new $iterateur($a[0], $a[1]); break;
+						case 3:    $iter = new $iterateur($a[0], $a[1], $a[2]); break;
+						case 4:    $iter = new $iterateur($a[0], $a[1], $a[2], $a[3]); break;
 					}
 				} catch (Exception $e) {
 					spip_log("Erreur de chargement de l'iterateur $iterateur");
@@ -68,8 +68,8 @@ 
 			// IterateurXXX
 			// definie dans le fichier iterateurs/xxx.php
 			$class = "Iterateur".$iterateur;
-			if (!class_exists($class)){
-				if (!include_spip("iterateur/" . strtolower($iterateur))
+			if (!class_exists($class)) {
+				if (!include_spip("iterateur/".strtolower($iterateur))
 					OR !class_exists($class)) {
 					die("Iterateur $iterateur non trouvé");
 					// si l'iterateur n'existe pas, on se rabat sur le generique
@@ -114,7 +114,7 @@ 
 	 * en tenant compte des filtres
 	 * @var int
 	 */
-	protected $fetched=0;
+	protected $fetched = 0;
 
 	/**
 	 * Y a t'il une erreur ?
@@ -135,7 +135,7 @@ 
 		return false;
 	}
 
-	public function __construct(Iterator $iter, $command, $info){
+	public function __construct(Iterator $iter, $command, $info) {
 		parent::__construct($iter);
 		parent::rewind(); // remettre a la premiere position (bug? connu de FilterIterator)
 		
@@ -168,7 +168,7 @@ 
 	// 
 	private function calculer_select() {
 		if ($select = &$this->command['select']) {
-			foreach($select as $s) {
+			foreach ($select as $s) {
 				// /!\ $s = '.nom'
 				if ($s[0] == '.') {
 					$s = substr($s, 1);
@@ -186,9 +186,9 @@ 
 		AND method_exists($this->iter, $nom)) {
 			try {
 				return $this->iter->$nom();
-			} catch(Exception $e) {
+			} catch (Exception $e) {
 				// #GETCHILDREN sur un fichier de DirectoryIterator ...
-				spip_log("Methode $nom en echec sur " . get_class($this->iter));
+				spip_log("Methode $nom en echec sur ".get_class($this->iter));
 				spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
 				return '';
 			}
@@ -220,13 +220,13 @@ 
 		// IN sur collection vide (ce dernier devrait pouvoir etre fait a la compil)
 		if ($where = &$this->command['where']) {
 			$menage = false;
-			foreach($where as $k => $v) { 
-				if (is_array($v)){
-					if ((count($v)>=2) && ($v[0]=='REGEXP') && ($v[2]=="'.*'")) $op= false;
-					elseif ((count($v)>=2) && ($v[0]=='LIKE') && ($v[2]=="'%'")) $op= false;
+			foreach ($where as $k => $v) { 
+				if (is_array($v)) {
+					if ((count($v) >= 2) && ($v[0] == 'REGEXP') && ($v[2] == "'.*'")) $op = false;
+					elseif ((count($v) >= 2) && ($v[0] == 'LIKE') && ($v[2] == "'%'")) $op = false;
 					else $op = $v[0] ? $v[0] : $v;
 				} else $op = $v;
-				if ((!$op) OR ($op==1) OR ($op=='0=0')) {
+				if ((!$op) OR ($op == 1) OR ($op == '0=0')) {
 					unset($where[$k]);
 					$menage = true;
 				}
@@ -235,17 +235,17 @@ 
 				// produites par sql_in quand plus de 255 valeurs passees a IN
 				if (preg_match_all(',\s+IN\s+(\(.*\)),', $op, $s_req)) {
 					$req = '';
-					foreach($s_req[1] as $key => $val) {
-						$req .= trim($val, '(,)') . ',';
+					foreach ($s_req[1] as $key => $val) {
+						$req .= trim($val, '(,)').',';
 					}
-					$req = '(' . rtrim($req, ',') . ')';
+					$req = '('.rtrim($req, ',').')';
 				}
 				if (preg_match(',^\(\(([\w/]+)(\s+NOT)?\s+IN\s+(\(.*\))\)(?:\s+(AND|OR)\s+\(([\w/]+)(\s+NOT)?\s+IN\s+(\(.*\))\))*\)$,', $op, $regs)) {
 					$this->ajouter_filtre($regs[1], 'IN', strlen($req) ? $req : $regs[3], $regs[2]);
 					unset($op);
 				}
 			}
-			foreach($where as $k => $v) {
+			foreach ($where as $k => $v) {
 				// 3 possibilites : count($v) =
 				// * 1 : {x y} ; on recoit $v[0] = y
 				// * 2 : {x !op y} ; on recoit $v[0] = 'NOT', $v[1] = array() // array du type {x op y}
@@ -266,7 +266,7 @@ 
 
 		// critere {2,7}
 		if (isset($this->command['limit']) AND $this->command['limit']) {
-			$limit = explode(',',$this->command['limit']);
+			$limit = explode(',', $this->command['limit']);
 			$this->offset = $limit[0];
 			$this->limit = $limit[1];
 		}
@@ -279,7 +279,7 @@ 
 
 
 
-	protected function ajouter_filtre($cle, $op, $valeur, $not=false) {
+	protected function ajouter_filtre($cle, $op, $valeur, $not = false) {
 		if (method_exists($this->iter, 'exception_des_criteres')) {
 			if (in_array($cle, $this->iter->exception_des_criteres())) {
 				return;
@@ -310,8 +310,8 @@ 
 		} else if ($op == 'IN') {
 			$filtre = 'in_array('.$a.', array'.$valeur.')';
 			$op = '';
-		} else if (!in_array($op, array('<','<=', '>', '>='))) {
-			spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
+		} else if (!in_array($op, array('<', '<=', '>', '>='))) {
+			spip_log('operateur non reconnu '.$op); // [todo] mettre une erreur de squelette
 			$op = '';
 		}
 	
@@ -327,7 +327,7 @@ 
 	}
 
 	
-	public function next(){
+	public function next() {
 		$this->pos++;
 		parent::next();
 	}
@@ -366,26 +366,26 @@ 
 	 * position courante de l'iterateur
 	 * @var int
 	 */
-	protected $pos=null;
+	protected $pos = null;
 
 	/**
 	 * nombre total resultats dans l'iterateur
 	 * @var int
 	 */
-	protected $total=null;
+	protected $total = null;
 
 	/**
 	 * nombre maximal de recherche pour $total
 	 * si l'iterateur n'implemente pas de fonction specifique
 	 */
-	 protected $max=100000;
+	 protected $max = 100000;
 
 
 	/**
 	 * Liste des champs a inserer dans les $row
 	 * retournes par ->fetch()
 	 */
-	 protected $select=array();
+	 protected $select = array();
 
 	 
 	/**
@@ -399,7 +399,7 @@ 
 	 * @return bool
 	 *   success or fail if not implemented
 	 */
-	public function seek($n=0, $continue=null) {
+	public function seek($n = 0, $continue = null) {
 		if ($this->func_filtre OR !method_exists($this->iter, 'seek') OR !$this->iter->seek($n)) {
 			$this->seek_loop($n);
 		}
@@ -429,16 +429,16 @@ 
 	 * @param  $max
 	 * @return int
 	 */
-	public function skip($saut, $max=null){
+	public function skip($saut, $max = null) {
 		// pas de saut en arriere autorise pour cette fonction
-		if (($saut=intval($saut))<=0) return $this->pos;
+		if (($saut = intval($saut)) <= 0) return $this->pos;
 		$seek = $this->pos + $saut;
 		// si le saut fait depasser le maxi, on libere la resource
 		// et on sort
 		if (is_null($max))
 			$max = $this->count();
 
-		if ($seek>=$max OR $seek>=$this->count()) {
+		if ($seek >= $max OR $seek >= $this->count()) {
 			// sortie plus rapide que de faire next() jusqu'a la fin !
 			$this->free();
 		  return $max;

ecrire/action/editer_objet.php

Spacing +36 added lines, -36 removed lines

@@ -22,10 +22,10 @@ 
  * @param array $set
  * @return array
  */
-function action_editer_objet_dist($id=null, $objet=null, $set=null) {
+function action_editer_objet_dist($id = null, $objet = null, $set = null) {
 
 	// appel direct depuis une url interdit
-	if (is_null($id) OR is_null($objet)){
+	if (is_null($id) OR is_null($objet)) {
 		include_spip('inc/minipres');
 		echo minipres(_T('info_acces_interdit'));
 		die();
@@ -39,13 +39,13 @@ 
 		$id = objet_inserer($objet, $id_parent);
 	}
 
-	if (!($id = intval($id))>0)
-		return array($id,_L('echec enregistrement en base'));
+	if (!($id = intval($id)) > 0)
+		return array($id, _L('echec enregistrement en base'));
 
 	// Enregistre l'envoi dans la BD
 	$err = objet_modifier($objet, $id, $set);
 
-	return array($id,$err);
+	return array($id, $err);
 }
 
 /**
@@ -57,16 +57,16 @@ 
  * @param array|null $set
  * @return mixed|string
  */
-function objet_modifier($objet, $id, $set=null) {
+function objet_modifier($objet, $id, $set = null) {
 	if (include_spip('action/editer_'.$objet)
 	  AND function_exists($modifier = $objet."_modifier"))
-		return $modifier($id,$set);
+		return $modifier($id, $set);
 
 	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 	if (!$desc OR !isset($desc['field'])) {
-		spip_log("Objet $objet inconnu dans objet_modifier",_LOG_ERREUR);
+		spip_log("Objet $objet inconnu dans objet_modifier", _LOG_ERREUR);
 		return _L("Erreur objet $objet inconnu");
 	}
 	include_spip('inc/modifier');
@@ -89,13 +89,13 @@ 
 		// white list
 		$white,
 		// black list
-		array($champ_date,'statut','id_parent','id_secteur'),
+		array($champ_date, 'statut', 'id_parent', 'id_secteur'),
 		// donnees eventuellement fournies
 		$set
 	);
 
 	// Si l'objet est publie, invalider les caches et demander sa reindexation
-	if (objet_test_si_publie($objet,$id)){
+	if (objet_test_si_publie($objet, $id)) {
 		$invalideur = "id='$objet/$id'";
 		$indexation = true;
 	}
@@ -111,7 +111,7 @@ 
 			'invalideur' => $invalideur,
 			'indexation' => $indexation,
 			 // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-			'date_modif' => (isset($desc['field']['date_modif'])?'date_modif':'')
+			'date_modif' => (isset($desc['field']['date_modif']) ? 'date_modif' : '')
 		),
 		$c))
 		return $err;
@@ -119,7 +119,7 @@ 
 	// Modification de statut, changement de rubrique ?
 	// FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
 	//         le tableau $set hors black liste, mais du coup on a possiblement des champs en trop. 
-	$c = collecter_requests(array($champ_date, 'statut', 'id_parent'),array(),$set);
+	$c = collecter_requests(array($champ_date, 'statut', 'id_parent'), array(), $set);
 	$err = objet_instituer($objet, $id, $c);
 
 	return $err;
@@ -132,24 +132,24 @@ 
  * @param array|null $set
  * @return bool|int
  */
-function objet_inserer($objet, $id_parent=null, $set=null) {
+function objet_inserer($objet, $id_parent = null, $set = null) {
 	if (include_spip('action/editer_'.$objet)
 	  AND function_exists($inserer = $objet."_inserer"))
 		return $inserer($id_parent);
 
 	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 	if (!$desc OR !isset($desc['field']))
 		return 0;
 
 	$lang_rub = "";
 	$champs = array();
-	if (isset($desc['field']['id_rubrique'])){
+	if (isset($desc['field']['id_rubrique'])) {
 		// Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet
 		// dans la premiere rubrique racine
 		if (!$id_rubrique = intval($id_parent)) {
-			$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
+			$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0", '', '0+titre,titre', "1");
 			$id_rubrique = $row['id_rubrique'];
 		}
 		else
@@ -179,8 +179,8 @@ 
 		$champs['langue_choisie'] = 'non';
 	}
 
-	if (isset($desc['field']['statut'])){
-		if (isset($desc['statut_textes_instituer'])){
+	if (isset($desc['field']['statut'])) {
+		if (isset($desc['statut_textes_instituer'])) {
 			$cles_statut = array_keys($desc['statut_textes_instituer']); 
 			$champs['statut'] = reset($cles_statut);
 		}
@@ -189,7 +189,7 @@ 
 	}
 
 
-	if ((isset($desc['date']) AND $d=$desc['date']) OR isset($desc['field'][$d='date']))
+	if ((isset($desc['date']) AND $d = $desc['date']) OR isset($desc['field'][$d = 'date']))
 		$champs[$d] = date('Y-m-d H:i:s');
 
 	if ($set)
@@ -207,7 +207,7 @@ 
 
 	$id = sql_insertq($table_sql, $champs);
 
-	if ($id){
+	if ($id) {
 		pipeline('post_insertion',
 			array(
 				'args' => array(
@@ -223,8 +223,8 @@ 
 		// si la table n'a pas deja un champ id_auteur
 		// et si le form n'a pas poste un id_auteur (meme vide, ce qui sert a annuler cette auto association)
 		if ($id > 0
-		  AND !isset($desc['field']['id_auteur'])){
-			$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur']))?
+		  AND !isset($desc['field']['id_auteur'])) {
+			$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur'])) ?
 				$GLOBALS['visiteur_session']['id_auteur']
 				:_request('id_auteur'));
 		  if ($id_auteur) {
@@ -249,13 +249,13 @@ 
  * @param bool $calcul_rub
  * @return mixed|string
  */
-function objet_instituer($objet, $id, $c, $calcul_rub=true) {
+function objet_instituer($objet, $id, $c, $calcul_rub = true) {
 	if (include_spip('action/editer_'.$objet)
 	  AND function_exists($instituer = $objet."_instituer"))
-		return $instituer($id,$c,$calcul_rub);
+		return $instituer($id, $c, $calcul_rub);
 
 	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 	if (!$desc OR !isset($desc['field']))
 		return _L("Impossible d'instituer $objet : non connu en base");
@@ -265,7 +265,7 @@ 
 	include_spip('inc/modifier');
 
 	$sel = array();
-	$sel[] = (isset($desc['field']['statut'])?"statut":"'' as statut");
+	$sel[] = (isset($desc['field']['statut']) ? "statut" : "'' as statut");
 
 	$champ_date = '';
 	if (isset($desc['date']) AND $desc['date'])
@@ -274,7 +274,7 @@ 
 		$champ_date = 'date';
 
 	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
-	$sel[] = (isset($desc['field']['id_rubrique'])?'id_rubrique':"0 as id_rubrique");
+	$sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : "0 as id_rubrique");
 
 	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet).'='.intval($id));
 
@@ -283,8 +283,8 @@ 
 	$date_ancienne = $date = $row['date'];
 	$champs = array();
 
-	$d = ($date AND isset($c[$champ_date]))?$c[$champ_date]:null;
-	$s = (isset($desc['field']['statut']) AND isset($c['statut']))?$c['statut']:$statut;
+	$d = ($date AND isset($c[$champ_date])) ? $c[$champ_date] : null;
+	$s = (isset($desc['field']['statut']) AND isset($c['statut'])) ? $c['statut'] : $statut;
 
 	// cf autorisations dans inc/instituer_objet
 	if ($s != $statut OR ($d AND $d != $date)) {
@@ -294,10 +294,10 @@ 
 				autoriser('instituer', $objet, $id, null, array('statut'=>$s))
 			)
 			$statut = $champs['statut'] = $s;
-		else if ($s!='publie' AND autoriser('modifier', $objet, $id))
+		else if ($s != 'publie' AND autoriser('modifier', $objet, $id))
 			$statut = $champs['statut'] = $s;
 		else
-			spip_log("editer_objet $id refus " . join(' ', $c));
+			spip_log("editer_objet $id refus ".join(' ', $c));
 
 		// En cas de publication, fixer la date a "maintenant"
 		// sauf si $c commande autre chose
@@ -308,7 +308,7 @@ 
 			 OR ($champs['statut'] == 'prop' AND !in_array($statut_ancien, array('publie', 'prop')))
 			 OR $d
 			) {
-				if ($d OR strtotime($d=$date)>time())
+				if ($d OR strtotime($d = $date) > time())
 					$champs[$champ_date] = $date = $d;
 				else
 					$champs[$champ_date] = $date = date('Y-m-d H:i:s');
@@ -402,9 +402,9 @@ 
  * @param bool $cond
  * @return 
  */
-function objet_editer_heritage($objet, $id, $id_rubrique, $statut, $champs, $cond=true) {
+function objet_editer_heritage($objet, $id, $id_rubrique, $statut, $champs, $cond = true) {
 	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 
 	// Si on deplace l'objet
@@ -418,7 +418,7 @@ 
 			$champs['id_secteur'] = $row_rub['id_secteur'];
 
 		if (isset($desc['field']['lang']) AND isset($desc['field']['langue_choisie']))
-			if (sql_fetsel('1', $table_sql, id_table_objet($objet)."=".intval($id)." AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+			if (sql_fetsel('1', $table_sql, id_table_objet($objet)."=".intval($id)." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue))) {
 				$champs['lang'] = $langue;
 			}
 	}