spip / SPIP

ecrire/public/debusquer.php

Braces +108 added lines, -82 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('public/decompiler');
 include_spip('inc/filtres_mini');
@@ -86,17 +88,23 @@ 
 		set_request('var_mode', 'debug');
 		$GLOBALS['bouton_admin_debug'] = true;
 		// Permettre a la compil de continuer
-		if (is_object($lieu) AND (!isset($lieu->code) OR !$lieu->code))
-			$lieu->code = "''";
+		if (is_object($lieu) AND (!isset($lieu->code) OR !$lieu->code)) {
+					$lieu->code = "''";
+		}
 		// forcer l'appel au debusqueur en cas de boucles infernales
 		$urgence = (_DEBUG_MAX_SQUELETTE_ERREURS AND count($tableau_des_erreurs)>_DEBUG_MAX_SQUELETTE_ERREURS);
-		if (!$urgence) return;
+		if (!$urgence) {
+		    return;
+		}
+	}
+	if (empty($debug_objets['principal'])) {
+			$debug_objets['principal'] = $GLOBALS['fond'];
 	}
-	if (empty($debug_objets['principal']))
-		$debug_objets['principal'] = $GLOBALS['fond'];
 
 	include_spip('inc/autoriser');
-	if (!autoriser('debug')) return;
+	if (!autoriser('debug')) {
+	    return;
+	}
 	include_spip('inc/headers');
 	include_spip('inc/filtres');
 
@@ -122,24 +130,29 @@ 
 		. '<br />'
 		. debusquer_squelette($fonc, $mode, $self);
 
-	if (!_DIR_RESTREINT OR headers_sent()) return $res;
-	if ($tableau_des_erreurs) http_status(503);
+	if (!_DIR_RESTREINT OR headers_sent()) {
+	    return $res;
+	}
+	if ($tableau_des_erreurs) {
+	    http_status(503);
+	}
 
 	http_no_cache();
 	if (isset($_GET['var_profile'])){
 		$titre = parametre_url($GLOBALS['REQUEST_URI'], 'var_profile', '');
 		$titre = parametre_url($titre, 'var_mode', '');
-	}
-	else {
-		if (!$fonc) $fonc = $debug_objets['principal'];
+	} else {
+		if (!$fonc) {
+		    $fonc = $debug_objets['principal'];
+		}
 		$titre = !$mode ? $fonc : ($mode . ' ' . $debug_objets['sourcefile'][$fonc]);
 	}
 	if ($message===false){
 		lang_select();
 		return debusquer_entete($titre, $res);
+	} else {
+			echo debusquer_entete($titre, $res);
 	}
-	else
-		echo debusquer_entete($titre, $res);
 	exit;
 }
 
@@ -169,25 +182,29 @@ 
 	if (!empty($erreurs)){
 		$n = array(count($erreurs) . ' ' . _T('zbug_erreur_squelette'));
 		return debusquer_navigation($erreurs, $n);
-	}
-	elseif (!empty($GLOBALS['tableau_des_temps'])) {
+	} elseif (!empty($GLOBALS['tableau_des_temps'])) {
 		include_spip('public/tracer');
 		list($temps, $nav) = chrono_requete($GLOBALS['tableau_des_temps']);
 		return debusquer_navigation($temps, $nav, 'debug-profile');
+	} else {
+			return '';
+	}
 	}
-	else
-		return '';
-}
 
 function debusquer_contexte($env){
 
-	if (is_array($env_tab = @unserialize($env))) $env = $env_tab;
+	if (is_array($env_tab = @unserialize($env))) {
+	    $env = $env_tab;
+	}
 
-	if (!$env) return '';
+	if (!$env) {
+	    return '';
+	}
 	$res = "";
 	foreach ($env as $nom => $valeur){
-		if (is_array($valeur))
-			$valeur = '(' . count($valeur) . ' items) [' . join(',', $valeur) . ']';
+		if (is_array($valeur)) {
+					$valeur = '(' . count($valeur) . ' items) [' . join(',', $valeur) . ']';
+		}
 		$res .= "\n<tr><td><strong>" . nl2br(entites_html($nom))
 			. "</strong></td><td>:&nbsp;" . nl2br(entites_html($valeur))
 			. "</td></tr>\n";
@@ -201,7 +218,9 @@ 
 
 function debusquer_navigation($tableau, $caption = array(), $id = 'debug-nav'){
 
-	if (_request('exec')=='valider_xml') return '';
+	if (_request('exec')=='valider_xml') {
+	    return '';
+	}
 	$GLOBALS['bouton_admin_debug'] = true;
 	$res = '';
 	$href = quote_amp(parametre_url($GLOBALS['REQUEST_URI'], 'var_mode', 'debug'));
@@ -285,8 +304,7 @@ 
 	// il serait plus prudent certainement d'avoir une fonction d'analyse par moteur
 	if (preg_match(',err(no|code):?[[:space:]]*([0-9]+),i', $msg, $regs)){
 		$errno = $regs[2];
-	}
-	elseif (is_numeric($errno) and ($errno==1030 OR $errno<=1026)
+	} elseif (is_numeric($errno) and ($errno==1030 OR $errno<=1026)
 		AND preg_match(',[^[:alnum:]]([0-9]+)[^[:alnum:]],', $msg, $regs)
 	) {
 		$errno = $regs[1];
@@ -325,25 +343,27 @@ 
 	if (is_array($debug_objets['sequence'][$id])){
 		foreach ($debug_objets['sequence'][$id] as $v){
 
-			if (!preg_match('/^(.*)(<\?.*\?>)(.*)$/s', $v[0], $r))
-				$y = substr_count($v[0], "\n");
-			else {
-				if ($v[1][0]=='#')
-					// balise dynamique
+			if (!preg_match('/^(.*)(<\?.*\?>)(.*)$/s', $v[0], $r)) {
+							$y = substr_count($v[0], "\n");
+			} else {
+				if ($v[1][0]=='#') {
+									// balise dynamique
 					$incl = $debug_objets['resultat'][$v[2]];
-				else
-					// inclusion
+				} else {
+									// inclusion
 					$incl = $debug_objets['squelette'][trouve_squelette_inclus($v[0])];
+				}
 				$y = substr_count($incl, "\n")
 					+substr_count($r[1], "\n")
 					+substr_count($r[3], "\n");
 			}
 			if ($n<=($y+$debut)){
-				if ($v[1][0]=='?')
-					return trouve_boucle_debug($n, $nom, $debut, substr($v[1], 1));
-				elseif ($v[1][0]=='!') {
-					if ($incl = trouve_squelette_inclus($v[1]))
-						return trouve_boucle_debug($n, $incl, $debut);
+				if ($v[1][0]=='?') {
+									return trouve_boucle_debug($n, $nom, $debut, substr($v[1], 1));
+				} elseif ($v[1][0]=='!') {
+					if ($incl = trouve_squelette_inclus($v[1])) {
+											return trouve_boucle_debug($n, $incl, $debut);
+					}
 				}
 				return array($nom, $boucle, $v[2]-1+$n-$debut);
 			}
@@ -359,16 +379,19 @@ 
 	preg_match('/include\(.(.*).php3?.\);/', $script, $reg);
 	// si le script X.php n'est pas ecrire/public.php
 	// on suppose qu'il prend le squelette X.html (pas sur, mais y a pas mieux)
-	if ($reg[1]=='ecrire/public')
-		// si c'est bien ecrire/public on cherche le param 'fond'
+	if ($reg[1]=='ecrire/public') {
+			// si c'est bien ecrire/public on cherche le param 'fond'
 		if (!preg_match("/'fond' => '([^']*)'/", $script, $reg))
 			// a defaut on cherche le param 'page'
 			if (!preg_match("/'param' => '([^']*)'/", $script, $reg))
 				$reg[1] = "inconnu";
+	}
 	$incl = ',' . $reg[1] . '[.]\w$,';
 
 	foreach ($debug_objets['sourcefile'] as $k => $v){
-		if (preg_match($incl, $v)) return $k;
+		if (preg_match($incl, $v)) {
+		    return $k;
+		}
 	}
 	return "";
 }
@@ -377,13 +400,13 @@ 
 function reference_boucle_debug($n, $nom, $self){
 	list($skel, $boucle, $ligne) = trouve_boucle_debug($n, $nom);
 
-	if (!$boucle)
-		return !$ligne ? "" :
+	if (!$boucle) {
+			return !$ligne ? "" :
 			(" (" .
 				(($nom!=$skel) ? _T('squelette_inclus_ligne') :
 					_T('squelette_ligne')) .
 				" <a href='$self&amp;var_mode_objet=$skel&amp;var_mode_affiche=squelette&amp;var_mode_ligne=$ligne#L$ligne'>$ligne</a>)");
-	else {
+	} else {
 		$self .= "&amp;var_mode_objet=$skel$boucle&amp;var_mode_affiche=boucle";
 
 		return !$ligne ? " (boucle\n<a href='$self#$skel$boucle'>$boucle</a>)" :
@@ -397,7 +420,9 @@ 
 function ancre_texte($texte, $fautifs = array(), $nocpt = false){
 
 	$var_mode_ligne = _request('var_mode_ligne');
-	if ($var_mode_ligne) $fautifs[] = array($var_mode_ligne);
+	if ($var_mode_ligne) {
+	    $fautifs[] = array($var_mode_ligne);
+	}
 	$res = '';
 
 	$s = highlight_string($texte, true);
@@ -420,11 +445,14 @@ 
 	$i = 1;
 	$flignes = array();
 	$loc = array(0, 0);
-	foreach ($fautifs as $lc)
-		if (is_array($lc)){
+	foreach ($fautifs as $lc) {
+			if (is_array($lc)){
 			$l = array_shift($lc);
+	}
 			$flignes[$l] = $lc;
-		} else $flignes[$lc] = $loc;
+		} else {
+		    $flignes[$lc] = $loc;
+		}
 
 	$ancre = md5($texte);
 	foreach ($tableau as $ligne){
@@ -467,39 +495,38 @@ 
 			$res = "<div id='spip-boucles'>\n"
 				. debusquer_navigation_squelettes($self)
 				. "</div>";
+		} else {
+					$res = '';
 		}
-		else
-			$res = '';
 		if ($fonc){
 			$id = " id='$fonc'";
 			if (!empty($GLOBALS['debug_objets'][$mode][$fonc])){
 				list($legend, $texte, $res2) = debusquer_source($fonc, $mode);
 				$texte .= $res2;
-			}
-			elseif (!empty($debug_objets[$mode][$fonc . 'tout'])) {
+			} elseif (!empty($debug_objets[$mode][$fonc . 'tout'])) {
 				$legend = _T('zbug_' . $mode);
 				$texte = $debug_objets[$mode][$fonc . 'tout'];
 				$texte = ancre_texte($texte, array('', ''));
 			}
-		}
-		else
-			return strlen(trim($res))
+		} else {
+					return strlen(trim($res))
 			? "<img src='".chemin_image('compat-16.png')."' alt='afficher-masquer le debug' id='spip-debug-toggle' onclick=\"var x = document.getElementById('spip-debug'); (x.style.display == '' ? x.style.display = 'none' : x.style.display = '');\" /><div id='spip-debug'>$res</div>"
 			// cas de l'appel sur erreur: montre la page
 			: $GLOBALS['debug_objets']['resultat']['tout'];
-	}
-	else {
+		}
+	} else {
 		$valider = charger_fonction('valider', 'xml');
 		$val = $valider($debug_objets['validation'][$fonc . 'tout']);
 		// Si erreur, signaler leur nombre dans le formulaire admin
 		$debug_objets['validation'] = $val->err ? count($val->err) : '';
 		list($texte, $err) = emboite_texte($val, $fonc, $self);
-		if ($err===false)
-			$err = _T('impossible');
-		elseif ($err===true)
-			$err = _T('correcte');
-		else
-			$err = ": $err";
+		if ($err===false) {
+					$err = _T('impossible');
+		} elseif ($err===true) {
+					$err = _T('correcte');
+		} else {
+					$err = ": $err";
+		}
 		$legend = _T('validation') . ' ' . $err;
 		$res = $id = '';
 	}
@@ -520,10 +547,12 @@ 
 	$errs = $res->err;
 	$texte = $res->entete . ($errs ? '' : $res->page);
 
-	if (!$texte and !$errs)
-		return array(ancre_texte('', array('', '')), false);
-	if (!$errs)
-		return array(ancre_texte($texte, array('', '')), true);
+	if (!$texte and !$errs) {
+			return array(ancre_texte('', array('', '')), false);
+	}
+	if (!$errs) {
+			return array(ancre_texte($texte, array('', '')), true);
+	}
 
 	if (!isset($GLOBALS['debug_objets'])){
 
@@ -550,9 +579,9 @@ 
 			$i++;
 			list($msg, $ligne, $col) = $r;
 			#spip_log("$r = list($msg, $ligne, $col");
-			if (isset($encore2[$msg]))
-				$ref = ++$encore2[$msg];
-			else {
+			if (isset($encore2[$msg])) {
+							$ref = ++$encore2[$msg];
+			} else {
 				$encore2[$msg] = $ref = 1;
 			}
 			$err .= "<tr  style='background-color: "
@@ -578,8 +607,7 @@ 
 			. $err
 			. " </table><a id='fin_err'></a>";
 		return array(ancre_texte($texte, $fautifs), $err);
-	}
-	else {
+	} else {
 		list($msg, $fermant, $ouvrant) = $errs[0];
 		$rf = reference_boucle_debug($fermant, $fonc, $self);
 		$ro = reference_boucle_debug($ouvrant, $fonc, $self);
@@ -594,10 +622,11 @@ 
 function count_occ($regs){
 	$encore = array();
 	foreach ($regs as $r){
-		if (isset($encore[$r[0]]))
-			$encore[$r[0]]++;
-		else
-			$encore[$r[0]] = 1;
+		if (isset($encore[$r[0]])) {
+					$encore[$r[0]]++;
+		} else {
+					$encore[$r[0]] = 1;
+		}
 	}
 	return $encore;
 }
@@ -725,18 +754,15 @@ 
 			}
 		}
 
-	}
-	elseif ($affiche=='code'){
+	} elseif ($affiche=='code'){
 		$legend = $nom;
 		$res = ancre_texte("<" . "?php\n" . $quoi . "\n?" . ">");
-	}
-	elseif ($affiche=='boucle'){
+	} elseif ($affiche=='boucle'){
 		$legend = _T('zbug_boucle') . ' ' . $nom;
 		// Le compilateur prefixe le nom des boucles par l'extension du fichier source.
 		$gram = preg_match('/^([^_]+)_/', $objet, $r) ? $r[1] : '';
 		$res = ancre_texte(public_decompiler($quoi, $gram, 0, 'boucle'));
-	}
-	elseif ($affiche=='squelette'){
+	} elseif ($affiche=='squelette'){
 		$legend = $GLOBALS['debug_objets']['sourcefile'][$objet];
 		$res = ancre_texte($GLOBALS['debug_objets']['squelette'][$objet]);
 	}

ecrire/public/aiguiller.php

Braces +38 added lines, -25 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 function securiser_redirect_action($redirect) {
 	if ((tester_url_absolue($redirect) or preg_match(',^\w+:,',trim($redirect)))
@@ -18,11 +20,13 @@ 
 		// si l'url est une url du site, on la laisse passer sans rien faire
 		// c'est encore le plus simple
 		$base = $GLOBALS['meta']['adresse_site']."/";
-		if (strlen($base) AND strncmp($redirect,$base,strlen($base))==0)
-			return $redirect;
+		if (strlen($base) AND strncmp($redirect,$base,strlen($base))==0) {
+					return $redirect;
+		}
 		$base = url_de_base();
-		if (strlen($base) AND strncmp($redirect,$base,strlen($base))==0)
-			return $redirect;
+		if (strlen($base) AND strncmp($redirect,$base,strlen($base))==0) {
+					return $redirect;
+		}
 		return "";
 	}
 	return $redirect;
@@ -51,16 +55,16 @@ 
 			$url = parametre_url($url,'var_ajax',$v,'&');
 			$url = parametre_url($url,'var_ajax_env',$args,'&');
 			set_request('redirect',$url);
-		}
-		else if(_request('redirect')){
+		} else if(_request('redirect')){
 			set_request('redirect',securiser_redirect_action(_request('redirect')));
 		}
 		$var_f = charger_fonction($action, 'action');
 		$var_f();
 		if (!isset($GLOBALS['redirect'])) {
 			$GLOBALS['redirect'] = _request('redirect');
-			if ($_SERVER['REQUEST_METHOD'] == 'POST')
-				$GLOBALS['redirect'] = urldecode($GLOBALS['redirect']);
+			if ($_SERVER['REQUEST_METHOD'] == 'POST') {
+							$GLOBALS['redirect'] = urldecode($GLOBALS['redirect']);
+			}
 			$GLOBALS['redirect'] = securiser_redirect_action($GLOBALS['redirect']);
 		}
 		if ($url = $GLOBALS['redirect']) {
@@ -79,8 +83,10 @@ 
 			redirige_par_entete($url);
 		}
 		if (!headers_sent()
-			AND !ob_get_length())
-				http_status(204); // No Content
+			AND !ob_get_length()) {
+						http_status(204);
+		}
+		// No Content
 		return true;
 	}
 	return false;
@@ -124,8 +130,7 @@ 
 				$ancre = str_replace(array('<','"',"'"),array('&lt;','&quot;',''),$ancre);
 				$texte = "<a href='#$ancre' name='ajax_ancre' style='display:none;'>anchor</a>".$texte;
 			}
-		}
-		else {
+		} else {
 			include_spip('inc/headers');
 			http_status(403);
 			$texte = _L('signature ajax bloc incorrecte');
@@ -145,13 +150,19 @@ 
 	static $post = array();
 	static $done = false;
 
-	if ($get) return $post; 
-	if ($done) return false;
+	if ($get) {
+	    return $post;
+	}
+	if ($done) {
+	    return false;
+	}
 	$done = true;
 
 	if (!($form = _request('formulaire_action')
-	AND $args = _request('formulaire_action_args')))
-		return false; // le hit peut continuer normalement
+	AND $args = _request('formulaire_action_args'))) {
+			return false;
+	}
+	// le hit peut continuer normalement
 
 	include_spip('inc/filtres');
 	if (($args = decoder_contexte_ajax($args,$form))===false) {
@@ -185,8 +196,9 @@ 
 		$retour = "";
 		if ((count($post["erreurs_$form"])==0)){
 			$rev = "";
-			if ($traiter = charger_fonction("traiter","formulaires/$form/",true))
-				$rev = call_user_func_array($traiter,$args);
+			if ($traiter = charger_fonction("traiter","formulaires/$form/",true)) {
+							$rev = call_user_func_array($traiter,$args);
+			}
 
 			$rev = pipeline(
 				  'formulaire_traiter',
@@ -219,13 +231,14 @@ 
 				if (isset($rev['message_erreur'])) {
 					$post["erreurs_$form"]["message_erreur"] = $rev['message_erreur'];
 					
-				}
-				else {
+				} else {
 					// sinon faire ce qu'il faut :
-					if (isset($rev['message_ok']))
-						$post["message_ok_$form"] = $rev['message_ok'];
-					if (isset($rev['editable']))
-						$post["editable_$form"] = $rev['editable'];
+					if (isset($rev['message_ok'])) {
+											$post["message_ok_$form"] = $rev['message_ok'];
+					}
+					if (isset($rev['editable'])) {
+											$post["editable_$form"] = $rev['editable'];
+					}
 					// si une redirection est demandee, appeler redirigae_formulaire qui choisira
 					// le bon mode de redirection (302 et on ne revient pas ici, ou javascript et on continue)
 					if (isset($rev['redirect']) AND $rev['redirect']){

ecrire/public/sandbox.php

Braces +21 added lines, -14 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Composer le code d'execution d'un texte
@@ -58,14 +60,15 @@ 
 		$refs = $refl->getParameters();
 		if (isset($refs[0]) AND $refs[0]->name == 'Pile') {
 			$code = "$f(\$Pile,$code$arglist)";
-		}
-		else {
+		} else {
 			$code = "$f($code$arglist)";
 		}
 	}
 	// le filtre n'existe pas,
 	// on le notifie
-	else erreur_squelette(array('zbug_erreur_filtre', array('filtre'=>  texte_script($fonc))), $p);
+	else {
+	    erreur_squelette(array('zbug_erreur_filtre', array('filtre'=>  texte_script($fonc))), $p);
+	}
 
 	return $code;
 }
@@ -92,9 +95,11 @@ 
 function sandbox_composer_inclure_php($fichier, &$p, $_contexte){
 	$compil = texte_script(memoriser_contexte_compil($p));
 	// si inexistant, on essaiera a l'execution
-	if ($path = find_in_path($fichier))
-		$path = "\"$path\"";
-	else $path = "find_in_path(\"$fichier\")";
+	if ($path = find_in_path($fichier)) {
+			$path = "\"$path\"";
+	} else {
+	    $path = "find_in_path(\"$fichier\")";
+	}
 
 	return sprintf(CODE_INCLURE_SCRIPT, $path, $fichier, $compil, $_contexte);
 }
@@ -110,9 +115,9 @@ 
 	// Securite
 	if ($p->interdire_scripts
 	AND $p->etoile != '**') {
-		if (!preg_match("/^sinon[(](.*),'([^']*)'[)]$/", $code, $r))
-			$code = "interdire_scripts($code)";
-		else {
+		if (!preg_match("/^sinon[(](.*),'([^']*)'[)]$/", $code, $r)) {
+					$code = "interdire_scripts($code)";
+		} else {
 			$code = interdire_scripts($r[2]);
 			$code = "sinon(interdire_scripts($r[1]),'$code')";
 		}
@@ -138,17 +143,19 @@ 
 	array_shift($series_filtres);// corps
 
 	// proteger les <INCLUDE> et tous les morceaux de php licites
-	if ($skel['process_ins'] == 'php')
-		$corps = preg_replace_callback(',<[?](\s|php|=).*[?]>,UimsS','echapper_php_callback', $corps);
+	if ($skel['process_ins'] == 'php') {
+			$corps = preg_replace_callback(',<[?](\s|php|=).*[?]>,UimsS','echapper_php_callback', $corps);
+	}
 
 	// recuperer les couples de remplacement
 	$replace = echapper_php_callback();
 
 	foreach($series_filtres as $filtres){
-		if (count($filtres))
-			foreach ($filtres as $filtre) {
+		if (count($filtres)) {
+					foreach ($filtres as $filtre) {
 				if ($filtre AND $f = chercher_filtre($filtre))
 					$corps = $f($corps);
+		}
 			}
 	}
 

ecrire/public/boucles.php

Braces +8 added lines, -5 removed lines

@@ -15,7 +15,9 @@ 
 // Ce fichier definit les boucles standard de SPIP
 //
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 //
 // Boucle standard, sans condition rajoutee
@@ -60,10 +62,11 @@ 
 	$boucle->where[]= array("'IN'", "'$id_table'", '"($hierarchie)"');
 
 	$order = "FIELD($id_table, \$hierarchie)";
-	if (!isset($boucle->default_order[0]) OR $boucle->default_order[0] != " DESC")
-		$boucle->default_order[] = "\"$order\"";
-	else
-		$boucle->default_order[0] = "\"$order DESC\"";
+	if (!isset($boucle->default_order[0]) OR $boucle->default_order[0] != " DESC") {
+			$boucle->default_order[] = "\"$order\"";
+	} else {
+			$boucle->default_order[0] = "\"$order DESC\"";
+	}
 	return calculer_boucle($id_boucle, $boucles); 
 }
 

ecrire/public/criteres.php

Braces +313 added lines, -200 removed lines

@@ -16,7 +16,9 @@ 
  * @package SPIP\Compilateur\Criteres
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Une Regexp repérant une chaine produite par le compilateur,
@@ -78,8 +80,9 @@ 
 	$boucle = &$boucles[$idb];
 	$id = $boucle->primary;
 
-	if ($not OR !$id)
-		return (array('zbug_critere_inconnu', array('critere' => $not.$crit->op)));
+	if ($not OR !$id) {
+			return (array('zbug_critere_inconnu', array('critere' => $not.$crit->op)));
+	}
 	$arg = kwote(calculer_argument_precedent($idb, $id, $boucles));
 	$boucle->where[] = array("'!='", "'$boucle->id_table."."$id'", $arg);
 }
@@ -195,8 +198,12 @@ 
  *     AST complété de la gestion du critère
 **/
 function critere_lang_select_dist($idb, &$boucles, $crit){
-	if (!isset($crit->param[1][0]) OR !($param = $crit->param[1][0]->texte)) $param = 'oui';
-	if ($crit->not) $param = ($param=='oui') ? 'non' : 'oui';
+	if (!isset($crit->param[1][0]) OR !($param = $crit->param[1][0]->texte)) {
+	    $param = 'oui';
+	}
+	if ($crit->not) {
+	    $param = ($param=='oui') ? 'non' : 'oui';
+	}
 	$boucle = &$boucles[$idb];
 	$boucle->lang_select = $param;
 }
@@ -214,8 +221,10 @@ 
 		                        '"]) . ",'.
 		                        $deux.
 		                        '"';
-	} else calculer_critere_DEFAUT_dist($idb, $boucles, $crit);
-}
+	} else {
+	    calculer_critere_DEFAUT_dist($idb, $boucles, $crit);
+	}
+	}
 
 // {pagination}
 // {pagination 20}
@@ -263,9 +272,10 @@ 
 	if ($boucle->primary
 	    AND !preg_match('/[,\s]/', $boucle->primary)
 	        AND !in_array($t, $boucle->select)
-	)
-		$boucle->select[] = $t;
-}
+	) {
+			$boucle->select[] = $t;
+	}
+	}
 
 
 // {recherche} ou {recherche susan}
@@ -280,10 +290,11 @@ 
 		return;
 	}
 
-	if (isset($crit->param[0]))
-		$quoi = calculer_liste($crit->param[0], array(), $boucles, $boucles[$idb]->id_parent);
-	else
-		$quoi = '(isset($Pile[0]["recherche"])?$Pile[0]["recherche"]:(isset($GLOBALS["recherche"])?$GLOBALS["recherche"]:""))';
+	if (isset($crit->param[0])) {
+			$quoi = calculer_liste($crit->param[0], array(), $boucles, $boucles[$idb]->id_parent);
+	} else {
+			$quoi = '(isset($Pile[0]["recherche"])?$Pile[0]["recherche"]:(isset($GLOBALS["recherche"])?$GLOBALS["recherche"]:""))';
+	}
 
 	$_modificateur = var_export($boucle->modificateur, true);
 	$boucle->hash .= '
@@ -300,8 +311,10 @@ 
 
 
 	$t = $boucle->id_table.'.'.$boucle->primary;
-	if (!in_array($t, $boucles[$idb]->select))
-		$boucle->select[] = $t; # pour postgres, neuneu ici
+	if (!in_array($t, $boucles[$idb]->select)) {
+			$boucle->select[] = $t;
+	}
+	# pour postgres, neuneu ici
 	// jointure uniquement sur le serveur principal
 	// (on ne peut joindre une table d'un serveur distant avec la table des resultats du serveur principal)
 	if (!$boucle->sql_serveur){
@@ -417,8 +430,9 @@ 
 			$champ = array_shift($decompose);
 			$boucle->where[] = array("'='", _q($cle.".".reset($decompose)), '"'.sql_quote(end($decompose)).'"');
 		}
+	} else {
+	    $cle = $boucle->id_table;
 	}
-	else $cle = $boucle->id_table;
 
 	$c = "sql_in('$cle".".$champ', calcul_branche_in($arg)"
 	     .($not ? ", 'NOT'" : '').")";
@@ -437,13 +451,16 @@ 
 	     $boucle->id_table.'.'.$boucle->primary
 	     ."', lister_objets_avec_logos('".$boucle->primary."'), '')";
 
-	if ($crit->cond) $c = "($arg ? $c : 1)";
+	if ($crit->cond) {
+	    $c = "($arg ? $c : 1)";
+	}
 
-	if ($not)
-		$boucle->where[] = array("'NOT'", $c);
-	else
-		$boucle->where[] = $c;
-}
+	if ($not) {
+			$boucle->where[] = array("'NOT'", $c);
+	} else {
+			$boucle->where[] = $c;
+	}
+	}
 
 // c'est la commande SQL "GROUP BY"
 // par exemple <boucle(articles){fusion lang}>
@@ -456,7 +473,9 @@ 
 			if (preg_match("/^(.*)\.(.*)$/", $t, $r)){
 				$t = table_objet_sql($r[1]);
 				$t = array_search($t, $boucles[$idb]->from);
-				if ($t) $t .= '.'.$r[2];
+				if ($t) {
+				    $t .= '.'.$r[2];
+				}
 			}
 		} else {
 			$t = '".'
@@ -466,11 +485,13 @@ 
 	}
 	if ($t){
 		$boucles[$idb]->group[] = $t;
-		if (!in_array($t, $boucles[$idb]->select))
-			$boucles[$idb]->select[] = $t;
-	} else
-		return (array('zbug_critere_inconnu', array('critere' => $crit->op.' ?')));
-}
+		if (!in_array($t, $boucles[$idb]->select)) {
+					$boucles[$idb]->select[] = $t;
+		}
+	} else {
+			return (array('zbug_critere_inconnu', array('critere' => $crit->op.' ?')));
+	}
+	}
 
 // c'est la commande SQL "COLLATE"
 // qui peut etre appliquee sur les order by, group by, where like ...
@@ -481,11 +502,13 @@ 
 		$boucle = $boucles[$idb];
 		$boucle->modificateur['collate'] = "($_coll ?' COLLATE '.$_coll:'')";
 		$n = count($boucle->order);
-		if ($n && (strpos($boucle->order[$n-1], 'COLLATE')===false))
-			$boucle->order[$n-1] .= " . ".$boucle->modificateur['collate'];
-	} else
-		return (array('zbug_critere_inconnu', array('critere' => $crit->op." ".count($boucles[$idb]->order))));
-}
+		if ($n && (strpos($boucle->order[$n-1], 'COLLATE')===false)) {
+					$boucle->order[$n-1] .= " . ".$boucle->modificateur['collate'];
+		}
+	} else {
+			return (array('zbug_critere_inconnu', array('critere' => $crit->op." ".count($boucles[$idb]->order))));
+	}
+	}
 
 // http://doc.spip.org/@calculer_critere_arg_dynamique
 function calculer_critere_arg_dynamique($idb, &$boucles, $crit, $suffix = ''){
@@ -498,7 +521,9 @@ 
 		$desc = implode(',', array_map('_q', array_keys($desc)));
 		$boucles[$idb]->in .= "\n\tstatic $var = array(".$desc.");";
 	}
-	if ($desc) $alt = "(in_array(\$x, $var)  ? $alt :(\$x$suffix))";
+	if ($desc) {
+	    $alt = "(in_array(\$x, $var)  ? $alt :(\$x$suffix))";
+	}
 	$arg = calculer_liste($crit, array(), $boucles, $boucle->id_parent);
 	return "((\$x = preg_replace(\"/\\W/\",'', $arg)) ? $alt : '')";
 }
@@ -514,7 +539,9 @@ 
 function critere_parinverse($idb, &$boucles, $crit, $sens = ''){
 	global $exceptions_des_jointures;
 	$boucle = &$boucles[$idb];
-	if ($crit->not) $sens = $sens ? "" : " . ' DESC'";
+	if ($crit->not) {
+	    $sens = $sens ? "" : " . ' DESC'";
+	}
 	$collecte = (isset($boucle->modificateur['collecte'])) ? " . ".$boucle->modificateur['collecte'] : "";
 
 	foreach ($crit->param as $tri){
@@ -528,8 +555,9 @@ 
 			// pour supporter 'hasard' comme tri dynamique
 			$par = "rand()";
 			$parha = $par." AS hasard";
-			if (!in_array($parha, $boucle->select))
-				$boucle->select[] = $parha;
+			if (!in_array($parha, $boucle->select)) {
+							$boucle->select[] = $parha;
+			}
 		} else {
 			$par = array_shift($tri);
 			$par = $par->texte;
@@ -566,8 +594,9 @@ 
 				if ($cle) { $cle .= '.'; }
 				$texte = '0+'. $cle . $champ;
 				$suite = calculer_liste($tri, array(), $boucles, $boucle->id_parent);
-				if ($suite!=="''")
-					$texte = "\" . ((\$x = $suite) ? ('$texte' . \$x) : '0')"." . \"";
+				if ($suite!=="''") {
+									$texte = "\" . ((\$x = $suite) ? ('$texte' . \$x) : '0')"." . \"";
+				}
 				$as = 'num'.($boucle->order ? count($boucle->order) : "");
 				$boucle->select[] = $texte." AS $as";
 				$order = "'$as'";
@@ -589,10 +618,10 @@ 
 					else if (isset($exceptions_des_jointures[$par])){
 						list($table, $champ) = $exceptions_des_jointures[$par];
 						$order = critere_par_joint($table, $champ, $boucle, $idb);
-						if (!$order)
-							return (array('zbug_critere_inconnu', array('critere' => $crit->op." $par")));
-					}
-					else if ($par=='date'
+						if (!$order) {
+													return (array('zbug_critere_inconnu', array('critere' => $crit->op." $par")));
+						}
+					} else if ($par=='date'
 					         AND $desc = $boucle->show
 					             AND $desc['date']
 					){
@@ -608,11 +637,14 @@ 
 						}
 						if (!$t){
 							return (array('zbug_critere_inconnu', array('critere' => $crit->op." $par")));
-						} else	 $order = "'".$t.'.'.$r[2]."'";
+						} else {
+						    $order = "'".$t.'.'.$r[2]."'";
+						}
 					} else {
 						$desc = $boucle->show;
-						if ($desc['field'][$par])
-							$par = $boucle->id_table.".".$par;
+						if ($desc['field'][$par]) {
+													$par = $boucle->id_table.".".$par;
+						}
 						// sinon tant pis, ca doit etre un champ synthetise (cf points)
 						$order = "'$par'";
 					}
@@ -624,16 +656,21 @@ 
 			if (strpos($t, '.') AND !in_array($t, $boucle->select)){
 				$boucle->select[] = $t;
 			}
-		} else $sens = '';
+		} else {
+		    $sens = '';
+		}
 
 		if ($fct){
-			if (preg_match("/^\s*'(.*)'\s*$/", $order, $r))
-				$order = "'$fct(".$r[1].")'";
-			else $order = "'$fct(' . $order . ')'";
+			if (preg_match("/^\s*'(.*)'\s*$/", $order, $r)) {
+							$order = "'$fct(".$r[1].")'";
+			} else {
+			    $order = "'$fct(' . $order . ')'";
+			}
 		}
 		$t = $order.$collecte.$sens;
-		if (preg_match("/^(.*)'\s*\.\s*'([^']*')$/", $t, $r))
-			$t = $r[1].$r[2];
+		if (preg_match("/^(.*)'\s*\.\s*'([^']*')$/", $t, $r)) {
+					$t = $r[1].$r[2];
+		}
 		$boucle->order[] = $t;
 	}
 }
@@ -641,7 +678,9 @@ 
 // http://doc.spip.org/@critere_par_joint
 function critere_par_joint($table, $champ, &$boucle, $idb){
 	$t = array_search($table, $boucle->from);
-	if (!$t) $t = trouver_jointure_champ($champ, $boucle);
+	if (!$t) {
+	    $t = trouver_jointure_champ($champ, $boucle);
+	}
 	return !$t ? '' : ("'".$t.'.'.$champ."'");
 }
 
@@ -653,9 +692,9 @@ 
 
 	$boucle = &$boucles[$idb];
 	// Classement par ordre inverse
-	if ($crit->not)
-		critere_parinverse($idb, $boucles, $crit);
-	else
+	if ($crit->not) {
+			critere_parinverse($idb, $boucles, $crit);
+	} else
 	{
 		$order = "' DESC'";
 		// Classement par ordre inverse fonction eventuelle de #ENV{...}
@@ -666,14 +705,16 @@ 
 
 		$n = count($boucle->order);
 		if (!$n){
-			if (isset($boucle->default_order[0]))
-				$boucle->default_order[0] .= ' . " DESC"';
-			else
-				$boucle->default_order[] = ' DESC';
+			if (isset($boucle->default_order[0])) {
+							$boucle->default_order[0] .= ' . " DESC"';
+			} else {
+							$boucle->default_order[] = ' DESC';
+			}
 		} else {
 			$t = $boucle->order[$n-1]." . $order";
-			if (preg_match("/^(.*)'\s*\.\s*'([^']*')$/", $t, $r))
-				$t = $r[1].$r[2];
+			if (preg_match("/^(.*)'\s*\.\s*'([^']*')$/", $t, $r)) {
+							$t = $r[1].$r[2];
+			}
 			$boucle->order[$n-1] = $t;
 		}
 	}
@@ -683,8 +724,9 @@ 
 function critere_agenda_dist($idb, &$boucles, $crit){
 	$params = $crit->param;
 
-	if (count($params)<1)
-		return (array('zbug_critere_inconnu', array('critere' => $crit->op." ?")));
+	if (count($params)<1) {
+			return (array('zbug_critere_inconnu', array('critere' => $crit->op." ?")));
+	}
 
 	$parent = $boucles[$idb]->id_parent;
 
@@ -731,26 +773,27 @@ 
 	$date = $boucle->id_table.".$date";
 
 	$quote_end = ",'".$boucle->sql_serveur."','text'";
-	if ($type=='jour')
-		$boucle->where[] = array("'='", "'DATE_FORMAT($date, \'%Y%m%d\')'",
+	if ($type=='jour') {
+			$boucle->where[] = array("'='", "'DATE_FORMAT($date, \'%Y%m%d\')'",
 		                         ("sql_quote($annee . $mois . $jour$quote_end)"));
-	elseif ($type=='mois')
-		$boucle->where[] = array("'='", "'DATE_FORMAT($date, \'%Y%m\')'",
+	} elseif ($type=='mois') {
+			$boucle->where[] = array("'='", "'DATE_FORMAT($date, \'%Y%m\')'",
 		                         ("sql_quote($annee . $mois$quote_end)"));
-	elseif ($type=='semaine')
-		$boucle->where[] = array("'AND'",
+	} elseif ($type=='semaine') {
+			$boucle->where[] = array("'AND'",
 		                         array("'>='",
 		                               "'DATE_FORMAT($date, \'%Y%m%d\')'",
 		                               ("date_debut_semaine($annee, $mois, $jour)")),
 		                         array("'<='",
 		                               "'DATE_FORMAT($date, \'%Y%m%d\')'",
 		                               ("date_fin_semaine($annee, $mois, $jour)")));
-	elseif (count($crit->param)>2)
-		$boucle->where[] = array("'AND'",
+	} elseif (count($crit->param)>2) {
+			$boucle->where[] = array("'AND'",
 		                         array("'>='",
 		                               "'DATE_FORMAT($date, \'%Y%m%d\')'",
 		                               ("sql_quote($annee . $mois . $jour$quote_end)")),
 		                         array("'<='", "'DATE_FORMAT($date, \'%Y%m%d\')'", ("sql_quote($annee2 . $mois2 . $jour2$quote_end)")));
+	}
 	// sinon on prend tout
 }
 
@@ -766,8 +809,7 @@ 
 
 	if (($op==',') && (is_numeric($a11) && (is_numeric($a21)))){
 		$boucle->limit = $a11.','.$a21;
-	}
-	else {
+	} else {
 		$boucle->total_parties = ($a21!='n') ? $a21 : $a22;
 		$partie = ($a11!='n') ? $a11 : $a12;
 		$mode = (($op=='/') ? '/' :
@@ -778,9 +820,9 @@ 
 				(is_numeric($a11)?"'$a11'":$a11)
 				.".','."
 				.(is_numeric($a21)?"'$a21'":$a21);
+		} else {
+					calculer_parties($boucles, $idb, $partie, $mode);
 		}
-		else
-			calculer_parties($boucles, $idb, $partie, $mode);
 	}
 }
 
@@ -803,7 +845,9 @@ 
 		$debut = !$pmoins1 ? 0 : "ceil(($nombre_boucle * $pmoins1)/$totpos);";
 	} else {
 		// cas {n-1,x}
-		if ($op1=='-') $debut = "$nombre_boucle - $debut;";
+		if ($op1=='-') {
+		    $debut = "$nombre_boucle - $debut;";
+		}
 
 		// cas {x,n-1}
 		if ($op2=='-'){
@@ -853,12 +897,14 @@ 
 	} else {
 		preg_match(',^ *(([0-9]+)|n) *(- *([0-9]+)? *)?$,', $param[0]->texte, $m);
 		$a1 = $m[1];
-		if (!@$m[3])
-			return array($a1, 0);
-		elseif ($m[4])
-			return array($a1, $m[4]);
-		else return array($a1,
+		if (!@$m[3]) {
+					return array($a1, 0);
+		} elseif ($m[4]) {
+					return array($a1, $m[4]);
+		} else {
+		    return array($a1,
 		                  calculer_liste(array($param[1]), array(), $boucles[$idb]->id_parent, $boucles));
+		}
 	}
 }
 
@@ -893,7 +939,9 @@ 
 
 	$defaut = charger_fonction('DEFAUT', 'calculer_critere');
 	// s'il y avait une erreur de syntaxe, propager cette info
-	if (!is_array($boucle->criteres)) return array();
+	if (!is_array($boucle->criteres)) {
+	    return array();
+	}
 
 	foreach ($boucle->criteres as $crit){
 		$critere = $crit->op;
@@ -938,11 +986,12 @@ 
  * @return string
  */
 function kwote($lisp, $serveur='', $type=''){
-	if (preg_match(_CODE_QUOTE, $lisp, $r))
-		return $r[1]."\"".sql_quote(str_replace(array("\\'", "\\\\"), array("'", "\\"), $r[2]),$serveur,$type)."\"";
-	else
-		return "sql_quote($lisp)";
-}
+	if (preg_match(_CODE_QUOTE, $lisp, $r)) {
+			return $r[1]."\"".sql_quote(str_replace(array("\\'", "\\\\"), array("'", "\\"), $r[2]),$serveur,$type)."\"";
+	} else {
+			return "sql_quote($lisp)";
+	}
+	}
 
 // Si on a une liste de valeurs dans #ENV{x}, utiliser la double etoile
 // pour faire par exemple {id_article IN #ENV**{liste_articles}}
@@ -961,21 +1010,26 @@ 
 	if ($crit->cond){
 		$pred = calculer_argument_precedent($idb, $col, $boucles);
 		$where = array("'?'", $pred, $where, "''");
-		if ($where_complement) // condition annexe du type "AND (objet='article')"
+		if ($where_complement) {
+		    // condition annexe du type "AND (objet='article')"
 			$where_complement = array("'?'", $pred, $where_complement, "''");
+		}
 	}
-	if ($crit->exclus)
-		if (!preg_match(",^L[0-9]+[.],", $arg))
+	if ($crit->exclus) {
+			if (!preg_match(",^L[0-9]+[.],", $arg))
 			$where = array("'NOT'", $where);
-		else
-			// un not sur un critere de jointure se traduit comme un NOT IN avec une sous requete
+	} else {
+					// un not sur un critere de jointure se traduit comme un NOT IN avec une sous requete
 			// c'est une sous requete identique a la requete principale sous la forme (SELF,$select,$where) avec $select et $where qui surchargent
 			$where = array("'NOT'", array("'IN'", "'".$boucles[$idb]->id_table.".".$boucles[$idb]->primary."'", array("'SELF'", "'".$boucles[$idb]->id_table.".".$boucles[$idb]->primary."'", $where)));
+		}
 
 	$boucles[$idb]->where[] = $where;
-	if ($where_complement) // condition annexe du type "AND (objet='article')"
+	if ($where_complement) {
+	    // condition annexe du type "AND (objet='article')"
 		$boucles[$idb]->where[] = $where_complement;
-}
+	}
+	}
 
 // http://doc.spip.org/@critere_IN_cas
 function critere_IN_cas($idb, &$boucles, $crit2, $arg, $op, $val, $col){
@@ -988,10 +1042,11 @@ 
 	foreach ($val as $k => $v){
 		if (preg_match(",^(\n//.*\n)?'(.*)'$,", $v, $r)){
 			// optimiser le traitement des constantes
-			if (is_numeric($r[2]))
-				$x .= "\n\t$var"."[]= $r[2];";
-			else
-				$x .= "\n\t$var"."[]= ".sql_quote($r[2]).";";
+			if (is_numeric($r[2])) {
+							$x .= "\n\t$var"."[]= $r[2];";
+			} else {
+							$x .= "\n\t$var"."[]= ".sql_quote($r[2]).";";
+			}
 		} else {
 			// Pour permettre de passer des tableaux de valeurs
 			// on repere l'utilisation brute de #ENV**{X},
@@ -1024,16 +1079,19 @@ 
  */
 function critere_where_dist($idb, &$boucles, $crit){
 	$boucle = &$boucles[$idb];
-	if (isset($crit->param[0]))
-		$_where = calculer_liste($crit->param[0], array(), $boucles, $boucle->id_parent);
-	else
-		$_where = '@$Pile[0]["where"]';
+	if (isset($crit->param[0])) {
+			$_where = calculer_liste($crit->param[0], array(), $boucles, $boucle->id_parent);
+	} else {
+			$_where = '@$Pile[0]["where"]';
+	}
 
-	if ($crit->cond)
-		$_where = "(($_where) ? ($_where) : '')";
+	if ($crit->cond) {
+			$_where = "(($_where) ? ($_where) : '')";
+	}
 
-	if ($crit->not)
-		$_where = "array('NOT',$_where)";
+	if ($crit->not) {
+			$_where = "array('NOT',$_where)";
+	}
 
 	$boucle->where[] = $_where;
 }
@@ -1122,8 +1180,9 @@ 
 // http://doc.spip.org/@calculer_critere_DEFAUT
 function calculer_critere_DEFAUT_dist($idb, &$boucles, $crit){
 	// double cas particulier {0,1} et {1/2} repere a l'analyse lexicale
-	if (($crit->op==",") OR ($crit->op=='/'))
-		return calculer_critere_parties($idb, $boucles, $crit);
+	if (($crit->op==",") OR ($crit->op=='/')) {
+			return calculer_critere_parties($idb, $boucles, $crit);
+	}
 
 	$r = calculer_critere_infixe($idb, $boucles, $crit);
 	if (!$r){
@@ -1131,8 +1190,10 @@ 
 	#	if (!$crit->cond) {
 			return (array('zbug_critere_inconnu', array('critere' => $crit->op)));
 	#	}
-	} else calculer_critere_DEFAUT_args($idb, $boucles, $crit, $r);
-}
+	} else {
+	    calculer_critere_DEFAUT_args($idb, $boucles, $crit, $r);
+	}
+	}
 
 function calculer_critere_DEFAUT_args($idb, &$boucles, $crit, $args){
 	list($arg, $op, $val, $col, $where_complement) = $args;
@@ -1141,14 +1202,17 @@ 
 
 	// inserer la negation (cf !...)
 
-	if ($crit->not) $where = array("'NOT'", $where);
-	if ($crit->exclus)
-		if (!preg_match(",^L[0-9]+[.],", $arg))
+	if ($crit->not) {
+	    $where = array("'NOT'", $where);
+	}
+	if ($crit->exclus) {
+			if (!preg_match(",^L[0-9]+[.],", $arg))
 			$where = array("'NOT'", $where);
-		else
-			// un not sur un critere de jointure se traduit comme un NOT IN avec une sous requete
+	} else {
+					// un not sur un critere de jointure se traduit comme un NOT IN avec une sous requete
 			// c'est une sous requete identique a la requete principale sous la forme (SELF,$select,$where) avec $select et $where qui surchargent
 			$where = array("'NOT'", array("'IN'", "'".$boucles[$idb]->id_table.".".$boucles[$idb]->primary."'", array("'SELF'", "'".$boucles[$idb]->id_table.".".$boucles[$idb]->primary."'", $where)));
+		}
 
 	// inserer la condition (cf {lang?})
 	// traiter a part la date, elle est mise d'office par SPIP,
@@ -1160,19 +1224,24 @@ 
 			}
 		}
 
-		if ($op=='=' AND !$crit->not)
-			$where = array("'?'", "(is_array($pred))",
+		if ($op=='=' AND !$crit->not) {
+					$where = array("'?'", "(is_array($pred))",
 			               critere_IN_cas($idb, $boucles, 'COND', $arg, $op, array($pred), $col),
 			               $where);
+		}
 		$where = array("'?'", "!(is_array($pred)?count($pred):strlen($pred))", "''", $where);
-		if ($where_complement) // condition annexe du type "AND (objet='article')"
+		if ($where_complement) {
+		    // condition annexe du type "AND (objet='article')"
 			$where_complement = array("'?'", "!(is_array($pred)?count($pred):strlen($pred))", "''", $where_complement);
+		}
 	}
 
 	$boucles[$idb]->where[] = $where;
-	if ($where_complement) // condition annexe du type "AND (objet='article')"
+	if ($where_complement) {
+	    // condition annexe du type "AND (objet='article')"
 		$boucles[$idb]->where[] = $where_complement;
-}
+	}
+	}
 
 // http://doc.spip.org/@calculer_critere_infixe
 function calculer_critere_infixe($idb, &$boucles, $crit){
@@ -1193,13 +1262,15 @@ 
 	$where_complement = false;
 
 	// Cas particulier : id_enfant => utiliser la colonne id_objet
-	if ($col=='id_enfant')
-		$col = $boucle->primary;
+	if ($col=='id_enfant') {
+			$col = $boucle->primary;
+	}
 
 	// Cas particulier : id_parent => verifier les exceptions de tables
 	if (in_array($col,array('id_parent','id_secteur'))
-	  AND isset($exceptions_des_tables[$table][$col]))
-		$col = $exceptions_des_tables[$table][$col];
+	  AND isset($exceptions_des_tables[$table][$col])) {
+			$col = $exceptions_des_tables[$table][$col];
+	}
 
 	// et possibilite de gerer un critere secteur sur des tables de plugins (ie forums)
 	else if (($col=='id_secteur') AND ($critere_secteur = charger_fonction("critere_secteur_$type", "public", true))){
@@ -1221,28 +1292,30 @@ 
 	else if ($c = calculer_critere_infixe_date($idb, $boucles, $col)){
 		list($col,$col_vraie) = $c;
 		$table = '';
-	}
-	else if (preg_match('/^(.*)\.(.*)$/', $col, $r)){
+	} else if (preg_match('/^(.*)\.(.*)$/', $col, $r)){
 		list(, $table, $col) = $r;
 		$col_alias = $col;
 
 		$trouver_table = charger_fonction('trouver_table','base');
 		if ($desc = $trouver_table($table, $boucle->sql_serveur)
 		  AND isset($desc['field'][$col])
-		  AND $cle = array_search($desc['table'],$boucle->from))
-			$table = $cle;
-		else {
+		  AND $cle = array_search($desc['table'],$boucle->from)) {
+					$table = $cle;
+		} else {
 			$table = trouver_jointure_champ($col, $boucle, array($table), ($crit->cond OR $op!='='));
 		}
 		#$table = calculer_critere_externe_init($boucle, array($table), $col, $desc, ($crit->cond OR $op!='='), true);
-		if (!$table) return '';
-	}
-	elseif (@!array_key_exists($col, $desc['field'])
+		if (!$table) {
+		    return '';
+		}
+	} elseif (@!array_key_exists($col, $desc['field'])
 	         // Champ joker * des iterateurs DATA qui accepte tout
 	         AND @!array_key_exists('*', $desc['field'])
 	) {
 		$r = calculer_critere_infixe_externe($boucle, $crit, $op, $desc, $col, $col_alias, $table);
-		if (!$r) return '';
+		if (!$r) {
+		    return '';
+		}
 		list($col, $col_alias, $table, $where_complement, $desc) = $r;
 	}
 
@@ -1256,8 +1329,9 @@ 
 
 		// defaire le quote des int et les passer dans sql_quote avec le bon type de champ si on le connait, int sinon
 		// prendre en compte le debug ou la valeur arrive avec un commentaire PHP en debut
-		if (preg_match(",^\\A(\s*//.*?$\s*)?\"'(-?\d+)'\"\\z,ms", $val[0], $r))
-			$val[0] = $r[1].'"'.sql_quote($r[2],$boucle->sql_serveur,(isset($desc['field'][$col_vraie])?$desc['field'][$col_vraie]:'int NOT NULL')).'"';
+		if (preg_match(",^\\A(\s*//.*?$\s*)?\"'(-?\d+)'\"\\z,ms", $val[0], $r)) {
+					$val[0] = $r[1].'"'.sql_quote($r[2],$boucle->sql_serveur,(isset($desc['field'][$col_vraie])?$desc['field'][$col_vraie]:'int NOT NULL')).'"';
+		}
 
 		// sinon expliciter les
 		// sql_quote(truc) en sql_quote(truc,'',type)
@@ -1279,8 +1353,9 @@ 
 	// Ne pas confondre champs de la table principale et des jointures
 	if ($table===$boucle->id_table){
 		$boucles[$idb]->modificateur['criteres'][$col_vraie] = true;
-		if ($col_alias!=$col_vraie)
-			$boucles[$idb]->modificateur['criteres'][$col_alias] = true;
+		if ($col_alias!=$col_vraie) {
+					$boucles[$idb]->modificateur['criteres'][$col_alias] = true;
+		}
 	}
 
 	// ajout pour le cas special d'une condition sur le champ statut:
@@ -1288,17 +1363,25 @@ 
 	// de mettre ses propres criteres de statut
 	// http://www.spip.net/@statut (a documenter)
 	// garde pour compatibilite avec code des plugins anterieurs, mais redondant avec la ligne precedente
-	if ($col=='statut') $boucles[$idb]->statut = true;
+	if ($col=='statut') {
+	    $boucles[$idb]->statut = true;
+	}
 
 	// inserer le nom de la table SQL devant le nom du champ
 	if ($table){
-		if ($col[0]=="`")
-			$arg = "$table.".substr($col, 1, -1);
-		else $arg = "$table.$col";
-	} else $arg = $col;
+		if ($col[0]=="`") {
+					$arg = "$table.".substr($col, 1, -1);
+		} else {
+		    $arg = "$table.$col";
+		}
+	} else {
+	    $arg = $col;
+	}
 
 	// inserer la fonction SQL
-	if ($fct) $arg = "$fct($arg$args_sql)";
+	if ($fct) {
+	    $arg = "$fct($arg$args_sql)";
+	}
 
 	return array($arg, $op, $val, $col_alias, $where_complement);
 }
@@ -1324,22 +1407,23 @@ 
 		$index = isset($t[$col])
 			? $t[$col] : (isset($t['']) ? $t[''] : array());
 
-		if (count($index)==3)
-			list($t, $col, $calculer_critere_externe) = $index;
-		elseif (count($index)==2) {
+		if (count($index)==3) {
+					list($t, $col, $calculer_critere_externe) = $index;
+		} elseif (count($index)==2) {
 			list($t, $col) = $t[$col];
-		}
-		elseif (count($index)==1) {
+		} elseif (count($index)==1) {
 			list($calculer_critere_externe) = $index;
 			$t = $table;
+		} else {
+					$t = '';
 		}
-		else
-			$t = ''; // jointure non declaree. La trouver.
+		// jointure non declaree. La trouver.
+	} elseif (isset($exceptions_des_jointures[$col])) {
+			list($t, $col) = $exceptions_des_jointures[$col];
+	} else {
+			$t = '';
 	}
-	elseif (isset($exceptions_des_jointures[$col]))
-		list($t, $col) = $exceptions_des_jointures[$col];
-	else
-		$t = ''; // jointure non declaree. La trouver.
+	// jointure non declaree. La trouver.
 
 	// ici on construit le from pour fournir $col en piochant dans les jointures
 
@@ -1358,7 +1442,9 @@ 
 		$table = $calculer_critere_externe($boucle, $boucle->jointures, $col, $desc, ($crit->cond OR $op!='='), $t);
 	}
 
-	if (!$table) return '';
+	if (!$table) {
+	    return '';
+	}
 
 	// il ne reste plus qu'a trouver le champ dans les from
 	list($nom, $desc) = trouver_champ_exterieur($col, $boucle->from, $boucle);
@@ -1425,14 +1511,19 @@ 
 				$joindre = false;
 				foreach ($cols as $col){
 					$c = '/\b'.$t.".$col".'\b/';
-					if (trouver_champ($c, $boucle->where)) $joindre = true;
-					else {
+					if (trouver_champ($c, $boucle->where)) {
+					    $joindre = true;
+					} else {
 						// mais ca peut etre dans le FIELD pour le Having
 						$c = "/FIELD.$t".".$col,/";
-						if (trouver_champ($c, $boucle->select)) $joindre = true;
+						if (trouver_champ($c, $boucle->select)) {
+						    $joindre = true;
+						}
 					}
 				}
-				if (!$joindre) return $t;
+				if (!$joindre) {
+				    return $t;
+				}
 			}
 			if ($res = calculer_jointure($boucle, array($boucle->id_table, $desc), $arrivee, $cols, $cond, 1)) {
 				return $res;
@@ -1460,7 +1551,9 @@ 
 	$intermediaire = trouver_champ_exterieur($primary_arrivee, $joints, $boucle, $checkarrivee."_liens");
 	$arrivee = trouver_champ_exterieur($col, $joints, $boucle, $checkarrivee);
 
-	if (!$intermediaire OR !$arrivee) return '';
+	if (!$intermediaire OR !$arrivee) {
+	    return '';
+	}
 
 	$res = fabrique_jointures($boucle,
 	                          array(
@@ -1474,11 +1567,13 @@ 
 
 // http://doc.spip.org/@trouver_champ
 function trouver_champ($champ, $where){
-	if (!is_array($where))
-		return preg_match($champ, $where);
-	else {
+	if (!is_array($where)) {
+			return preg_match($champ, $where);
+	} else {
 		foreach ($where as $clause){
-			if (trouver_champ($champ, $clause)) return true;
+			if (trouver_champ($champ, $clause)) {
+			    return true;
+			}
 		}
 		return false;
 	}
@@ -1493,11 +1588,13 @@ 
 	if (count($crit->param)==0){
 		$op = '=';
 		$col = $val = $crit->op;
-		if (preg_match('/^(.*)\.(.*)$/', $col, $r)) $val = $r[2];
+		if (preg_match('/^(.*)\.(.*)$/', $col, $r)) {
+		    $val = $r[2];
+		}
 		// Cas special {lang} : aller chercher $GLOBALS['spip_lang']
-		if ($val=='lang')
-			$val = array(kwote('$GLOBALS[\'spip_lang\']'));
-		else {
+		if ($val=='lang') {
+					$val = array(kwote('$GLOBALS[\'spip_lang\']'));
+		} else {
 			$defaut = null;
 			if ($val=='id_parent') {
 				// Si id_parent, comparer l'id_parent avec l'id_objet
@@ -1506,13 +1603,11 @@ 
 				$val = $boucles[$idb]->primary;
 				// mais si pas de boucle superieure, prendre id_parent dans l'env
 				$defaut = "\$Pile[0]['id_parent']";
-			}
-			elseif ($val=='id_enfant'){
+			} elseif ($val=='id_enfant'){
 				// Si id_enfant, comparer l'id_objet avec l'id_parent
 				// de la boucle superieure
 				$val = 'id_parent';
-			}
-			elseif ($crit->cond AND ($col=="date" OR $col=="date_redac")){
+			} elseif ($crit->cond AND ($col=="date" OR $col=="date_redac")){
 				// un critere conditionnel sur date est traite a part
 				// car la date est mise d'office par SPIP,
 				$defaut = "(\$Pile[0]['{$col}_default']?'':\$Pile[0]['".$col."'])";
@@ -1526,7 +1621,9 @@ 
 		// le phraseur impose que le premier param soit du texte
 		$params = $crit->param;
 		$op = $crit->op;
-		if ($op=='==') $op = 'REGEXP';
+		if ($op=='==') {
+		    $op = 'REGEXP';
+		}
 		$col = array_shift($params);
 		$col = $col[0]->texte;
 
@@ -1546,11 +1643,16 @@ 
 		                         AND $params[0][1]->type=='champ'
 		){
 			$val[] = "$p\\$p#".$params[0][1]->nom_champ."\\$p$p";
-		} else
-			foreach ((($op!='IN') ? $params : calculer_vieux_in($params)) as $p){
+		} else {
+					foreach ((($op!='IN') ? $params : calculer_vieux_in($params)) as $p){
 				$a = calculer_liste($p, $desc, $boucles, $parent);
-				if (strcasecmp($op,'IN')==0) $val[] = $a;
-				else $val[] = kwote($a, $boucles[$idb]->sql_serveur, 'char'); // toujours quoter en char ici
+		}
+				if (strcasecmp($op,'IN')==0) {
+				    $val[] = $a;
+				} else {
+				    $val[] = kwote($a, $boucles[$idb]->sql_serveur, 'char');
+				}
+				// toujours quoter en char ici
 			}
 	}
 
@@ -1583,8 +1685,9 @@ 
 	$n = isset($last->texte) ? strlen($last->texte) : 0;
 
 	if (!((isset($deb->texte[0])     AND $deb->texte[0]=='(')
-	   && (isset($last->texte[$n-1]) AND $last->texte[$n-1]==')')))
-		return $params;
+	   && (isset($last->texte[$n-1]) AND $last->texte[$n-1]==')'))) {
+			return $params;
+	}
 	$params[0][0]->texte = substr($deb->texte, 1);
 	// attention, on peut avoir k=0,j=0 ==> recalculer
 	$last = $params[$k][$j];
@@ -1592,9 +1695,9 @@ 
 	$params[$k][$j]->texte = substr($last->texte, 0, $n-1);
 	$newp = array();
 	foreach ($params as $v){
-		if ($v[0]->type!='texte')
-			$newp[] = $v;
-		else {
+		if ($v[0]->type!='texte') {
+					$newp[] = $v;
+		} else {
 			foreach (explode(',', $v[0]->texte) as $x){
 				$t = new Texte;
 				$t->texte = $x;
@@ -1607,27 +1710,35 @@ 
 
 // http://doc.spip.org/@calculer_critere_infixe_date
 function calculer_critere_infixe_date($idb, &$boucles, $col){
-	if (!preg_match(",^((age|jour|mois|annee)_relatif|date|mois|annee|jour|heure|age)(_[a-z]+)?$,", $col, $regs)) return '';
+	if (!preg_match(",^((age|jour|mois|annee)_relatif|date|mois|annee|jour|heure|age)(_[a-z]+)?$,", $col, $regs)) {
+	    return '';
+	}
 	$boucle = $boucles[$idb];
 	$table = $boucle->show;
 	// si c'est une colonne de la table, ne rien faire
-	if(isset($table['field'][$col])) return '';
+	if(isset($table['field'][$col])) {
+	    return '';
+	}
 
-	if (!$table['date'] && !isset($GLOBALS['table_date'][$table['id_table']])) return '';
+	if (!$table['date'] && !isset($GLOBALS['table_date'][$table['id_table']])) {
+	    return '';
+	}
 	$pred = $date_orig = isset($GLOBALS['table_date'][$table['id_table']])? $GLOBALS['table_date'][$table['id_table']] : $table['date'];
 	$col = $regs[1];
 	if (isset($regs[3]) AND $suite = $regs[3]){
 		# Recherche de l'existence du champ date_xxxx,
 		# si oui choisir ce champ, sinon choisir xxxx
 
-		if (isset($table['field']["date$suite"]))
-			$date_orig = 'date'.$suite;
-		else
-			$date_orig = substr($suite, 1);
+		if (isset($table['field']["date$suite"])) {
+					$date_orig = 'date'.$suite;
+		} else {
+					$date_orig = substr($suite, 1);
+		}
 		$pred = $date_orig;
-	}
-	else
-		if (isset($regs[2]) AND $rel = $regs[2]) $pred = 'date';
+	} else
+		if (isset($regs[2]) AND $rel = $regs[2]) {
+		    $pred = 'date';
+		}
 
 	$date_compare = "\"' . normaliser_date(".
 	                calculer_argument_precedent($idb, $pred, $boucles).
@@ -1684,9 +1795,9 @@ 
 	if (preg_match(",'\" *\.(.*)\. *\"',", $date_compare, $r)){
 		$init = "'\" . (\$x = $r[1]) . \"'";
 		$date_compare = '\'$x\'';
+	} else {
+			$init = $date_compare;
 	}
-	else
-		$init = $date_compare;
 
 	return
 		"LEAST((UNIX_TIMESTAMP(".
@@ -1722,9 +1833,10 @@ 
 	$boucle = &$boucles[$idb];
 
 	$args = array();
-	foreach ($crit->param as &$param)
-		array_push($args,
+	foreach ($crit->param as &$param) {
+			array_push($args,
 		           calculer_liste($param, array(), $boucles, $boucles[$idb]->id_parent));
+	}
 
 	$boucle->hash .= '
 	$command[\'sourcemode\'] = '.array_shift($args).";\n";
@@ -1896,8 +2008,9 @@ 
 
 	$in = "IN";
 	$where = array("'IN'", "'$boucle->id_table."."$primary'", "'('.sql_get_select('$id_parent', '$table_sql').')'");
-	if ($not)
-		$where = array("'NOT'", $where);
+	if ($not) {
+			$where = array("'NOT'", $where);
+	}
 
 	$boucle->where[] = $where;
 }

ecrire/public/phraser_html.php

Braces +152 added lines, -88 removed lines

@@ -11,7 +11,9 @@ 
  * \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 # Ce fichier transforme un squelette en un tableau d'objets de classe Boucle
 # il est charge par un include calcule 
@@ -44,7 +46,9 @@ 
 	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);
+		if ($p) {
+		    $result = phraser_idiomes($debut, $ligne, $result);
+		}
 		$ligne += substr_count($debut, "\n");
 		$champ = new Inclure;
 		$champ->ligne = $ligne;
@@ -60,8 +64,9 @@ 
 		// 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 (!function_exists('normaliser_inclure'))
-				include_spip('public/normaliser');
+			if (!function_exists('normaliser_inclure')) {
+							include_spip('public/normaliser');
+			}
 			normaliser_inclure($champ);
 		}
 		$texte = substr($champ->apres, strpos($champ->apres, '>')+1);
@@ -76,9 +81,10 @@ 
 // http://doc.spip.org/@phraser_polyglotte
 function phraser_polyglotte($texte, $ligne, $result){
 
-	if (preg_match_all(BALISE_POLYGLOTTE, $texte, $m, PREG_SET_ORDER))
-		foreach ($m as $match){
+	if (preg_match_all(BALISE_POLYGLOTTE, $texte, $m, PREG_SET_ORDER)) {
+			foreach ($m as $match){
 			$p = strpos($texte, $match[0]);
+	}
 			$debut = substr($texte, 0, $p);
 			if ($p){
 				$champ = new Texte;
@@ -96,8 +102,9 @@ 
 			$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;
+				if ($trad OR $lang) {
+									$champ->traductions[$lang] = $trad;
+				}
 				$lang = $regs[2];
 				$bloc = $regs[3];
 			}
@@ -125,10 +132,15 @@ 
 		$p = strpos($texte, $match[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);
+		if ($debut) {
+		    $result = phraser_champs($debut, $ligne, $result);
+		}
 		$texte = substr($texte, $p+strlen($match[0]));
 		$ligne += substr_count($debut, "\n");
-		if ($ko) continue; // faux idiome
+		if ($ko) {
+		    continue;
+		}
+		// faux idiome
 		$champ = new Idiome;
 		$champ->ligne = $ligne;
 		$ligne += substr_count($match[0], "\n");
@@ -146,7 +158,9 @@ 
 		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;
 }
@@ -158,7 +172,9 @@ 
 		$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);
+			if ($p) {
+			    $result = phraser_polyglotte($debut, $ligne, $result);
+			}
 			$ligne += substr_count($debut, "\n");
 			$champ = new Champ;
 			$champ->ligne = $ligne;
@@ -180,7 +196,9 @@ 
 						$champ->apres[0]->texte = substr($suite,0,$n);
 					}
 				*/
-			} else $texte = $suite;
+			} else {
+			    $texte = $suite;
+			}
 			phraser_vieux($champ);
 			$result[] = $champ;
 		} else {
@@ -189,7 +207,9 @@ 
 			$texte = (substr($texte, $p+1));
 		}
 	}
-	if ($texte!=="") $result = phraser_polyglotte($texte, $ligne, $result);
+	if ($texte!=="") {
+	    $result = phraser_polyglotte($texte, $ligne, $result);
+	}
 
 	return $result;
 }
@@ -201,10 +221,13 @@ 
 
 // http://doc.spip.org/@phraser_champs_etendus
 function phraser_champs_etendus($texte, $ligne, $result){
-	if ($texte==="") return $result;
+	if ($texte==="") {
+	    return $result;
+	}
 	$sep = '##';
-	while (strpos($texte, $sep)!==false)
-		$sep .= '#';
+	while (strpos($texte, $sep)!==false) {
+			$sep .= '#';
+	}
 
 	return array_merge($result, phraser_champs_interieurs($texte, $ligne, $sep, array()));
 }
@@ -234,7 +257,9 @@ 
 	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 /
@@ -244,9 +269,14 @@ 
 			$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;
+		} else {
+		    $texte = $suite;
+		}
+		if ($err_f) {
+		    $pointeur_champ->param = false;
+		} elseif ($fonc!=='') {
+		    $pointeur_champ->param[] = $res;
+		}
 		// pour les balises avec faux filtres qui boudent ce dur larbeur
 		$pointeur_champ->fonctions[] = array($fonc, '');
 
@@ -255,11 +285,11 @@ 
 	$args = ltrim(substr($suite, 1)); // virer le '(' initial
 	$collecte = array();
 	while ($args && $args[0]!='}'){
-		if ($args[0]=='"')
-			preg_match('/^(")([^"]*)(")(.*)$/ms', $args, $regs);
-		else if ($args[0]=="'")
-			preg_match("/^(')([^']*)(')(.*)$/ms", $args, $regs);
-		else {
+		if ($args[0]=='"') {
+					preg_match('/^(")([^"]*)(")(.*)$/ms', $args, $regs);
+		} else if ($args[0]=="'") {
+					preg_match("/^(')([^']*)(')(.*)$/ms", $args, $regs);
+		} else {
 			preg_match("/^([[:space:]]*)([^,([{}]*([(\[{][^])}]*[])}])?[^,}]*)([,}].*)$/ms", $args, $regs);
 			if (!strlen($regs[2])){
 				$err_f = array('zbug_erreur_filtre', array('filtre' => $args));
@@ -318,10 +348,14 @@ 
 					$next = isset($args[0]) ? $args[0] : '';
 				}
 				// Si erreur de syntaxe dans un sous-argument, propager.
-				if ($champ->param===false)
-					$err_f = true;
-				else phraser_vieux($champ);
-				if ($par==')') $args = substr($args, 1);
+				if ($champ->param===false) {
+									$err_f = true;
+				} else {
+				    phraser_vieux($champ);
+				}
+				if ($par==')') {
+				    $args = substr($args, 1);
+				}
 				$collecte[] = $champ;
 				$result[] = $champ;
 			}
@@ -342,10 +376,11 @@ 
 	$source = substr($suite, 0, strlen($suite)-strlen($texte));
 	// propager les erreurs, et ignorer les param vides
 	if ($pointeur_champ->param!==false){
-		if ($err_f)
-			$pointeur_champ->param = false;
-		elseif ($fonc!=='' || count($res)>1)
-			$pointeur_champ->param[] = $res;
+		if ($err_f) {
+					$pointeur_champ->param = false;
+		} elseif ($fonc!=='' || count($res)>1) {
+					$pointeur_champ->param[] = $res;
+		}
 	}
 	// pour les balises avec faux filtres qui boudent ce dur larbeur
 	$pointeur_champ->fonctions[] = array($fonc, $source);
@@ -358,12 +393,14 @@ 
 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))
-			break;
+		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]));
-		if ($p)
-			$res = phraser_inclure($debut, $ligne, $res);
+		if ($p) {
+					$res = phraser_inclure($debut, $ligne, $res);
+		}
 		$ligne += substr_count($debut, "\n");
 		$res[] = $nested[$m[1]];
 	}
@@ -417,45 +454,53 @@ 
 		while ($j<$i){
 			$z = $result[$j];
 			// j'aurais besoin de connaitre le nombre de lignes...
-			if (is_object($z))
-				$x .= "%$sep$j@";
-			else
-				$x .= $z;
+			if (is_object($z)) {
+							$x .= "%$sep$j@";
+			} else {
+							$x .= $z;
+			}
 			$j++;
 		}
-		if (preg_match(CHAMP_ETENDU, $x))
-			$texte = $x;
-		else
-			return phraser_champs_exterieurs($x, $ligne, $sep, $result);
+		if (preg_match(CHAMP_ETENDU, $x)) {
+					$texte = $x;
+		} else {
+					return phraser_champs_exterieurs($x, $ligne, $sep, $result);
+		}
 	}
 }
 
 function phraser_vieux(&$champ){
 	$nom = $champ->nom_champ;
 	if ($nom=='EMBED_DOCUMENT'){
-		if (!function_exists('phraser_vieux_emb'))
-			include_spip('public/normaliser');
+		if (!function_exists('phraser_vieux_emb')) {
+					include_spip('public/normaliser');
+		}
 		phraser_vieux_emb($champ);
 	} elseif ($nom=='EXPOSER') {
-		if (!function_exists('phraser_vieux_exposer'))
-			include_spip('public/normaliser');
+		if (!function_exists('phraser_vieux_exposer')) {
+					include_spip('public/normaliser');
+		}
 		phraser_vieux_exposer($champ);
 	} elseif ($champ->param) {
 		if ($nom=='FORMULAIRE_RECHERCHE'){
-			if (!function_exists('phraser_vieux_recherche'))
-				include_spip('public/normaliser');
+			if (!function_exists('phraser_vieux_recherche')) {
+							include_spip('public/normaliser');
+			}
 			phraser_vieux_recherche($champ);
 		} elseif (preg_match(",^LOGO_[A-Z]+,", $nom)) {
-			if (!function_exists('phraser_vieux_logos'))
-				include_spip('public/normaliser');
+			if (!function_exists('phraser_vieux_logos')) {
+							include_spip('public/normaliser');
+			}
 			phraser_vieux_logos($champ);
 		} elseif ($nom=='MODELE') {
-			if (!function_exists('phraser_vieux_modele'))
-				include_spip('public/normaliser');
+			if (!function_exists('phraser_vieux_modele')) {
+							include_spip('public/normaliser');
+			}
 			phraser_vieux_modele($champ);
 		} elseif ($nom=='INCLURE' OR $nom=='INCLUDE') {
-			if (!function_exists('phraser_vieux_inclu'))
-				include_spip('public/normaliser');
+			if (!function_exists('phraser_vieux_inclu')) {
+							include_spip('public/normaliser');
+			}
 			phraser_vieux_inclu($champ);
 		}
 	}
@@ -517,13 +562,16 @@ 
 					if (preg_match(',^(["\'])(.*)\1$,', $m[4])){
 						$c = null;
 						eval ('$c = ' . $m[4] . ';');
-						if (isset($c))
-							$m[4] = $c;
+						if (isset($c)) {
+													$m[4] = $c;
+						}
 					}
 					$texte = new Texte;
 					$texte->texte = $m[4];
 					$v[1][0] = $texte;
-				} else array_shift($v[1]);
+				} else {
+				    array_shift($v[1]);
+				}
 			}
 			array_shift($v); // $v[O] est vide
 			$crit = new Critere;
@@ -552,12 +600,13 @@ 
 			} else {
 				// traiter qq lexemes particuliers pour faciliter la suite
 				// les separateurs
-				if ($var->apres)
-					$result->separateur[] = $param;
-				elseif (($param=='tout') OR ($param=='tous'))
-					$result->modificateur['tout'] = true;
-				elseif ($param=='plat')
-					$result->modificateur['plat'] = true;
+				if ($var->apres) {
+									$result->separateur[] = $param;
+				} elseif (($param=='tout') OR ($param=='tous')) {
+									$result->modificateur['tout'] = true;
+				} elseif ($param=='plat') {
+									$result->modificateur['plat'] = true;
+				}
 
 				// Boucle hierarchie, analyser le critere id_rubrique
 				// et les autres critères {id_x} pour forcer {tout} sur
@@ -578,17 +627,21 @@ 
 						// 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')
-						// meme chose (a cause de #nom_de_boucle:URL_*)
+						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)){
 						$crit = phraser_critere_infixe($m[1], $m[3], $v, '/', '', '');
 					} 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)))
-							$a2 = substr($a2, 1, -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[6], $m[5]);
@@ -605,7 +658,9 @@ 
 							$v[0][0]->texte = $m[6];
 						} else {
 							array_shift($v[0]);
-							if (!$v[0]) array_shift($v);
+							if (!$v[0]) {
+							    array_shift($v);
+							}
 						}
 						$crit = new Critere;
 						$crit->op = $m[2];
@@ -617,10 +672,11 @@ 
 							array('critere' => $param));
 						erreur_squelette($err_ci, $result);
 					}
-					if ((!preg_match(',^!?doublons *,', $param)) || $crit->not)
-						$args[] = $crit;
-					else
-						$doublons[] = $crit;
+					if ((!preg_match(',^!?doublons *,', $param)) || $crit->not) {
+											$args[] = $crit;
+					} else {
+											$doublons[] = $crit;
+					}
 				}
 			}
 		}
@@ -628,10 +684,14 @@ 
 	// les doublons non nies doivent etre le dernier critere
 	// pour que la variable $doublon_index ait la bonne valeur
 	// cf critere_doublon
-	if ($doublons) $args = array_merge($args, $doublons);
+	if ($doublons) {
+	    $args = array_merge($args, $doublons);
+	}
 	// Si erreur, laisser la chaine dans ce champ pour le HTTP 503
-	if (!$err_ci) $result->criteres = $args;
-}
+	if (!$err_ci) {
+	    $result->criteres = $args;
+	}
+	}
 
 // http://doc.spip.org/@phraser_critere_infixe
 function phraser_critere_infixe($arg1, $arg2, $args, $op, $not, $cond){
@@ -774,14 +834,15 @@ 
 		}
 		$soustype = strtolower($type);
 
-		if (!isset($GLOBALS["table_des_tables"][$soustype]))
-			$soustype = $type;
+		if (!isset($GLOBALS["table_des_tables"][$soustype])) {
+					$soustype = $type;
+		}
 
 		$result->type_requete = $soustype;
 		// Lancer la 2e passe sur les criteres si la 1ere etait bonne
-		if (!is_array($result->param))
-			$err_b = true;
-		else {
+		if (!is_array($result->param)) {
+					$err_b = true;
+		} else {
 			phraser_criteres($result->param, $result);
 			if (strncasecmp($soustype, TYPE_RECURSIF, strlen(TYPE_RECURSIF))==0){
 				$result->type_requete = TYPE_RECURSIF;
@@ -798,7 +859,9 @@ 
 		$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;
+		if ($err_b) {
+		    $result->type_requete = false;
+		}
 
 		// Verifier qu'il n'y a pas double definition
 		// apres analyse des sous-parties (pas avant).
@@ -809,8 +872,9 @@ 
 			erreur_squelette($err_b_d, $result);
 			// Prevenir le generateur de code que le squelette est faux
 			$boucles[$id_boucle]->type_requete = false;
-		} else
-			$boucles[$id_boucle] = $result;
+		} else {
+					$boucles[$id_boucle] = $result;
+		}
 		$all_res = phraser_champs_etendus($debut, $ligne, $all_res);
 		$all_res[] = &$boucles[$id_boucle];
 		if (!empty($suite)){

ecrire/public/normaliser.php

Braces +35 added lines, -18 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Les fonctions de ce fichier sont appelees en certains points 
 // de l'analyseur syntaxique afin de normaliser de vieilles syntaxes,
@@ -24,9 +26,9 @@ 
 
 function phraser_vieux_logos(&$p)
 {
-	if ($p->param[0][0])
-		$args = array('');
-	else {
+	if ($p->param[0][0]) {
+			$args = array('');
+	} else {
 		$args = array_shift($p->param);
 	}
 
@@ -90,8 +92,9 @@ 
 
 function phraser_vieux_emb(&$p)
 {
-	if (!is_array($p->param))
-		$p->param=array();
+	if (!is_array($p->param)) {
+			$p->param=array();
+	}
 
 	// Produire le premier argument {emb}
 	$texte = new Texte;
@@ -101,8 +104,10 @@ 
 	// Transformer les filtres en arguments
 	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
+			if (!strstr($p->param[$i][0], '=')) {
+							break;
+			}
+			# on a rencontre un vrai filtre, c'est fini
 			$texte = new Texte;
 			$texte->texte=$p->param[$i][0];
 			$param[] = array($texte);
@@ -156,15 +161,23 @@ 
 function normaliser_args_inclumodel($p)
 {
 	$params = $p->param;
-	if (!$params) return;
+	if (!$params) {
+	    return;
+	}
 	$args = $params[0];
-	if ($args[0]) return; // filtre immediat
+	if ($args[0]) {
+	    return;
+	}
+	// filtre immediat
 	array_shift($p->param);
 	foreach ($p->param as $l) {
 		if (!array_shift($l))  {
 			$args = array_merge($args, $l);
 			array_shift($p->param);
-		} else  break; // filtre
+		} else {
+		    break;
+		}
+		// filtre
 	}
 	array_unshift($p->param, $args);
 }
@@ -181,16 +194,20 @@ 
 	  }
 	  foreach ($l as $k => $p) {
 		if (!$p OR $p[0]->type != 'texte' OR 
-		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
-			continue;
+		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r)) {
+					continue;
+		}
 
-		if ($r[1])
-			$p[0]->texte = $r[1];
-		else unset($p[0]);
+		if ($r[1]) {
+					$p[0]->texte = $r[1];
+		} else {
+		    unset($p[0]);
+		}
 		$champ->texte = $p;
 		unset($champ->param[0][$k]);
-		if (count($champ->param[0]) ==1) 
-			array_shift($champ->param);
+		if (count($champ->param[0]) ==1) {
+					array_shift($champ->param);
+		}
 		return;
 	  }
 	}

ecrire/public/interfaces.php

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
  * @package SPIP\Compilateur\AST
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 
 /**

ecrire/public/balises.php

Braces +143 added lines, -88 removed lines

@@ -17,18 +17,21 @@ 
 // elle recoit en entree un objet de classe CHAMP, le modifie et le retourne.
 // Cette classe est definie dans public/interfaces
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+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))
-		return calculer_liste($p->param[0][$n],
+	if (($p->param) && (!$p->param[0][0]) && (count($p->param[0])>$n)) {
+			return calculer_liste($p->param[0][$n],
 			$p->descr,
 			$p->boucles,
 			$p->id_boucle);
-	else
-		return NULL;
-}
+	} else {
+			return NULL;
+	}
+	}
 //
 // Definition des balises
 //
@@ -187,8 +190,9 @@ 
 		champ_sql('url_site',$p) ."," .
 		champ_sql('nom_site',$p) .
 		", 'titre', \$connect, false))";
-	} else
-		$p->code = champ_sql('nom_site',$p);
+	} else {
+			$p->code = champ_sql('nom_site',$p);
+	}
 
 	$p->interdire_scripts = true;
 	return $p;
@@ -276,8 +280,9 @@ 
 	$off= "''";
 	if (($v = interprete_argument_balise(1,$p))!==NULL){
 		$on = $v;
-		if (($v = interprete_argument_balise(2,$p))!==NULL)
-			$off = $v;
+		if (($v = interprete_argument_balise(2,$p))!==NULL) {
+					$off = $v;
+		}
 
 	}
 	return calculer_balise_expose($p, $on, $off);
@@ -320,7 +325,9 @@ 
 		$parent = index_pile($p->id_boucle, 'id_rubrique', $p->boucles, $b);
 	} elseif  (isset($desc['field']['id_groupe'])) {
 		$parent = index_pile($p->id_boucle, 'id_groupe', $p->boucles, $b);
-	} else $parent = "''";
+	} else {
+	    $parent = "''";
+	}
 
 	$p->code = "(calcul_exposer($c, '$type', \$Pile[0], $parent, '$key', $connect) ? $on : $off)";
 
@@ -400,10 +407,11 @@ 
 // http://doc.spip.org/@balise_LANG_dist
 function balise_LANG_dist ($p) {
 	$_lang = champ_sql('lang', $p);
-	if (!$p->etoile)
-		$p->code = "spip_htmlentities($_lang ? $_lang : \$GLOBALS['spip_lang'])";
-	else
-		$p->code = "spip_htmlentities($_lang)";
+	if (!$p->etoile) {
+			$p->code = "spip_htmlentities($_lang ? $_lang : \$GLOBALS['spip_lang'])";
+	} else {
+			$p->code = "spip_htmlentities($_lang)";
+	}
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -430,8 +438,7 @@ 
 			$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;
@@ -593,9 +600,9 @@ 
 
 	if (count($_contexte)){
 		$code_contexte = implode(',',$_contexte);
+	} else {
+			$code_contexte = '';
 	}
-	else
-		$code_contexte = '';
 
 	$connect = $p->boucles[$b]->sql_serveur;
 	$pas = $p->boucles[$b]->total_parties;
@@ -616,9 +623,12 @@ 
 // qu'on mettra les liens en-dessous de la liste paginee)
 // 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');
-	else return NULL; // ou une erreur ?
+	if ($f = charger_fonction('PAGINATION', 'balise', true)) {
+			return $f($p, $liste='false');
+	} else {
+	    return NULL;
+	}
+	// ou une erreur ?
 }
 
 // equivalent a #TOTAL_BOUCLE sauf pour les boucles paginees, ou elle
@@ -661,8 +671,9 @@ 
 	if (!$arg) {
 		$msg = array('zbug_balise_sans_argument',	array('balise' => ' CHEMIN'));
 		erreur_squelette($msg, $p);
-	} else
-	  $p->code = 'find_in_path(' . $arg .')';
+	} else {
+		  $p->code = 'find_in_path(' . $arg .')';
+	}
 
 	#$p->interdire_scripts = true;
 	return $p;
@@ -673,7 +684,9 @@ 
 	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;
@@ -814,7 +827,9 @@ 
 	if (!$_nom OR !$_val) {
 		$err_b_s_a = array('zbug_balise_sans_argument', array('balise' => 'SESSION_SET'));
 		erreur_squelette($err_b_s_a, $p);
-	} else 	$p->code = '(include_spip("inc/session") AND session_set('.$_nom.','.$_val.'))';
+	} else {
+	    $p->code = '(include_spip("inc/session") AND session_set('.$_nom.','.$_val.'))';
+	}
 
 	$p->interdire_scripts = false;
 
@@ -841,10 +856,11 @@ 
 		# optimisation sur les #EVAL{une expression sans #BALISE}
 		# attention au commentaire "// x signes" qui precede
 		if (preg_match(",^([[:space:]]*//[^\n]*\n)'([^']+)'$,ms",
-		$php,$r))
-			$p->code = /* $r[1]. */'('.$r[2].')';
-		else
-			$p->code = "eval('return '.$php.';')";
+		$php,$r)) {
+					$p->code = /* $r[1]. */'('.$r[2].')';
+		} else {
+					$p->code = "eval('return '.$php.';')";
+		}
 	} else {
 		$msg = array('zbug_balise_sans_argument', array('balise' => ' EVAL'));
 		erreur_squelette($msg, $p);
@@ -864,9 +880,9 @@ 
 
 	if ($p->param
 	AND isset($p->param[0][1][0])
-	AND $champ = ($p->param[0][1][0]->texte))
-		$p->code = champ_sql($champ, $p);
-	else {
+	AND $champ = ($p->param[0][1][0]->texte)) {
+			$p->code = champ_sql($champ, $p);
+	} else {
 		$err_b_s_a = array('zbug_balise_sans_argument', array('balise' => ' CHAMP_SQL'));
 		erreur_squelette($err_b_s_a, $p);
 	}
@@ -879,8 +895,9 @@ 
 // http://doc.spip.org/@balise_VAL_dist
 function balise_VAL_dist($p){
 	$p->code = interprete_argument_balise(1,$p);
-	if (!strlen($p->code))
-		$p->code = "''";
+	if (!strlen($p->code)) {
+			$p->code = "''";
+	}
 	$p->interdire_scripts = false;
 	return $p;
 }
@@ -943,8 +960,9 @@ 
 function balise_FILTRE_dist($p) {
 	if ($p->param) {
 		$args = array();
-		foreach ($p->param as $i => $ignore)
-			$args[] = interprete_argument_balise($i+1,$p);
+		foreach ($p->param as $i => $ignore) {
+					$args[] = interprete_argument_balise($i+1,$p);
+		}
 		$p->code = "'<' . '"
 			.'?php header("X-Spip-Filtre: \'.'
 				.join('.\'|\'.', $args)
@@ -979,12 +997,13 @@ 
 
 		// Remplir le header Cache-Control
 		// cas #CACHE{0}
-		if ($duree == 0)
-			$code .= '.\'<'
+		if ($duree == 0) {
+					$code .= '.\'<'
 			.'?php header("Cache-Control: no-cache, must-revalidate"); ?'
 			.'><'
 			.'?php header("Pragma: no-cache"); ?'
 			.'>\'';
+		}
 
 		// recuperer les parametres suivants
 		$i = 1;
@@ -1001,10 +1020,13 @@ 
 			}
 
 			if ($pa == 'statique'
-			AND $duree > 0)
-				$code .= '.\'<'.'?php header("X-Spip-Statique: oui"); ?'.'>\'';
+			AND $duree > 0) {
+							$code .= '.\'<'.'?php header("X-Spip-Statique: oui"); ?'.'>\'';
+			}
 		}
-	} else $code = "''";
+	} else {
+	    $code = "''";
+	}
 	$p->code = $code;
 	$p->interdire_scripts = false;
 	return $p;
@@ -1061,11 +1083,12 @@ 
 // (Incompatible avec les balises dynamiques)
 // http://doc.spip.org/@balise_INCLUDE_dist
 function balise_INCLUDE_dist($p) {
-	if(function_exists('balise_INCLURE'))
-		return balise_INCLURE($p);
-	else
-		return balise_INCLURE_dist($p);
-}
+	if(function_exists('balise_INCLURE')) {
+			return balise_INCLURE($p);
+	} else {
+			return balise_INCLURE_dist($p);
+	}
+	}
 // http://doc.spip.org/@balise_INCLURE_dist
 function balise_INCLURE_dist($p) {
 	$id_boucle = $p->id_boucle;
@@ -1077,7 +1100,9 @@ 
 
 	// erreur de syntaxe = fond absent
 	// (2 messages d'erreur SPIP pour le prix d'un, mais pas d'erreur PHP
-	if (!$_contexte) $contexte = array();
+	if (!$_contexte) {
+	    $contexte = array();
+	}
 
 	if (isset($_contexte['fond'])) {
 
@@ -1086,7 +1111,9 @@ 
 		if (preg_match('/^.fond.\s*=>(.*)$/s', $f, $r)) {
 			$f = $r[1];
 			unset($_contexte['fond']);
-		} else spip_log("compilation de #INCLURE a revoir");
+		} else {
+		    spip_log("compilation de #INCLURE a revoir");
+		}
 
 		// #INCLURE{doublons}
 		if (isset($_contexte['doublons'])) {
@@ -1099,25 +1126,33 @@ 
 		) {
 			$flag_env = true;
 			unset($_contexte['env']);
-		} else $flag_env = false;
+		} else {
+		    $flag_env = false;
+		}
 
 		$_options = array();
 		if (isset($_contexte['ajax'])) {
 			$_options[] = preg_replace(",=>(.*)$,ims",'=> ($v=(\\1))?$v:true',$_contexte['ajax']);
 			unset($_contexte['ajax']);
 		}
-		if ($p->etoile) $_options[] = "'etoile'=>true";
+		if ($p->etoile) {
+		    $_options[] = "'etoile'=>true";
+		}
 		$_options[] = "'compil'=>array(" . memoriser_contexte_compil($p) .")";
 
 		$_l = 'array(' . join(",\n\t", $_contexte) .')';
-		if ($flag_env) $_l = "array_merge(\$Pile[0],$_l)";
+		if ($flag_env) {
+		    $_l = "array_merge(\$Pile[0],$_l)";
+		}
 
 		$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;
@@ -1131,7 +1166,9 @@ 
 
 	// erreur de syntaxe = fond absent
 	// (2 messages d'erreur SPIP pour le prix d'un, mais pas d'erreur PHP
-	if (!$_contexte) $contexte = array();
+	if (!$_contexte) {
+	    $contexte = array();
+	}
 
 	if (!isset($_contexte[1])) {
 		$msg = array('zbug_balise_sans_argument', array('balise' => ' MODELE'));
@@ -1140,9 +1177,11 @@ 
 		$nom = $_contexte[1];
 		unset($_contexte[1]);
 
-		if (preg_match("/^\s*'[^']*'/s", $nom))
-			$nom = "'modeles/" . substr($nom,1);
-		else $nom = "'modeles/' . $nom";
+		if (preg_match("/^\s*'[^']*'/s", $nom)) {
+					$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
@@ -1156,8 +1195,9 @@ 
 		}
 		$_contexte[] = "'recurs'=>(++\$recurs)";
 		$connect = '';
-		if (isset($p->boucles[$p->id_boucle]))
-			$connect = $p->boucles[$p->id_boucle]->sql_serveur;
+		if (isset($p->boucles[$p->id_boucle])) {
+					$connect = $p->boucles[$p->id_boucle]->sql_serveur;
+		}
 
 		$_options = memoriser_contexte_compil($p);
 		$_options = "'compil'=>array($_options), 'trim'=>true";
@@ -1192,7 +1232,9 @@ 
 	}
 	// affectation $_zzz inutile, mais permet de contourner un bug OpCode cache sous PHP 5.5.4
 	// cf https://bugs.php.net/bug.php?id=65845
-	else  $p->code = "vide(\$Pile['vars'][\$_zzz=(string)$_nom] = $_val)";
+	else {
+	    $p->code = "vide(\$Pile['vars'][\$_zzz=(string)$_nom] = $_val)";
+	}
 
 	$p->interdire_scripts = false; // la balise ne renvoie rien
 	return $p;
@@ -1206,11 +1248,12 @@ 
 // http://doc.spip.org/@balise_GET_dist
 function balise_GET_dist($p) {
 	$p->interdire_scripts = false; // le contenu vient de #SET, donc il est de confiance
-	if (function_exists('balise_ENV'))
-		return balise_ENV($p, '$Pile["vars"]');
-	else
-		return balise_ENV_dist($p, '$Pile["vars"]');
-}
+	if (function_exists('balise_ENV')) {
+			return balise_ENV($p, '$Pile["vars"]');
+	} else {
+			return balise_ENV_dist($p, '$Pile["vars"]');
+	}
+	}
 
 
 /**
@@ -1230,15 +1273,17 @@ 
 **/
 function balise_DOUBLONS_dist($p) {
 	if ($type = interprete_argument_balise(1,$p)) {
-		if ($famille = interprete_argument_balise(2,$p))
-			$type .= '.' . $famille;
+		if ($famille = interprete_argument_balise(2,$p)) {
+					$type .= '.' . $famille;
+		}
 		$p->code = '$doublons['.$type.']';
-		if (!$p->etoile)
-			$p->code = 'array_filter(array_map("intval",explode(",",'
+		if (!$p->etoile) {
+					$p->code = 'array_filter(array_map("intval",explode(",",'
 				. $p->code . ')))';
+		}
+	} else {
+			$p->code = '$doublons';
 	}
-	else
-		$p->code = '$doublons';
 
 	$p->interdire_scripts = false;
 
@@ -1307,7 +1352,9 @@ 
 	do {
 		$_key = interprete_argument_balise($n++,$p);
 		$_val = interprete_argument_balise($n++,$p);
-		if ($_key AND $_val) $_code[] = "$_key => $_val";
+		if ($_key AND $_val) {
+		    $_code[] = "$_key => $_val";
+		}
 	} while ($_key && $_val);
 	$p->code = 'array(' . join(', ',$_code).')';
 	$p->interdire_scripts = false;
@@ -1323,8 +1370,9 @@ 
 function balise_LISTE_dist($p) {
 	$_code = array();
 	$n=1;
-	while ($_val = interprete_argument_balise($n++,$p))
-		$_code[] = $_val;
+	while ($_val = interprete_argument_balise($n++,$p)) {
+			$_code[] = $_val;
+	}
 	$p->code = 'array(' . join(', ',$_code).')';
 	$p->interdire_scripts = false;
 	return $p;
@@ -1339,8 +1387,9 @@ 
 	$p->descr['session'] = true; // faire un cache par session
 
 	$n=1;
-	while ($_v = interprete_argument_balise($n++,$p))
-		$_code[] = $_v;
+	while ($_v = interprete_argument_balise($n++,$p)) {
+			$_code[] = $_v;
+	}
 
 	$p->code = '((function_exists("autoriser")||include_spip("inc/autoriser"))&&autoriser(' . join(', ',$_code).')?" ":"")';
 	$p->interdire_scripts = false;
@@ -1375,10 +1424,12 @@ 
 // 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))
-		$_url = "@\$Pile[0]['action']";
-	if (!$_form = interprete_argument_balise(2,$p))
-		$_form = "@\$Pile[0]['form']";
+	if (!$_url = interprete_argument_balise(1,$p)) {
+			$_url = "@\$Pile[0]['action']";
+	}
+	if (!$_form = interprete_argument_balise(2,$p)) {
+			$_form = "@\$Pile[0]['form']";
+	}
 
 	// envoyer le nom du formulaire que l'on traite
 	// transmettre les eventuels args de la balise formulaire
@@ -1415,12 +1466,15 @@ 
 	$args = array();
 	for ($k=1;$k<=6;$k++){
 		$_a = interprete_argument_balise($k,$p);
-		if (!$_a) $_a="''";
+		if (!$_a) {
+		    $_a="''";
+		}
 	  $args[] = $_a;
 	}
 	// supprimer les args vides
-	while(end($args)=="''" AND count($args)>2)
-		array_pop($args);
+	while(end($args)=="''" AND count($args)>2) {
+			array_pop($args);
+	}
 	$args = implode(",",$args);
 
 	$bouton_action = chercher_filtre("bouton_action");
@@ -1532,8 +1586,7 @@ 
 	if (!$boucle) {
 		$msg = array('zbug_champ_hors_boucle', array('champ' => '#SAUTER'));
 		erreur_squelette($msg, $p);
-	}
-	else {
+	} else {
 		$_saut = interprete_argument_balise(1,$p);
 		$_compteur = "\$Numrows['$id_boucle']['compteur_boucle']";
 		$_total = "\$Numrows['$id_boucle']['total']";
@@ -1555,9 +1608,9 @@ 
 	if (!$_type = interprete_argument_balise(1,$p)){
 		$_type = _q($p->type_requete);
 		$_id = champ_sql($p->boucles[$p->id_boucle]->primary,$p);
+	} else {
+			$_id = interprete_argument_balise(2,$p);
 	}
-	else
-		$_id = interprete_argument_balise(2,$p);
 
 	$connect = $p->boucles[$p->id_boucle]->sql_serveur;
 
@@ -1599,7 +1652,9 @@ 
  */
 function balise_LARGEUR_ECRAN_dist($p){
 	$_class = interprete_argument_balise(1,$p);
-	if (!$_class) $_class='null';
+	if (!$_class) {
+	    $_class='null';
+	}
 	$p->code = "(is_string($_class)?vide(\$GLOBALS['largeur_ecran']=$_class):(isset(\$GLOBALS['largeur_ecran'])?\$GLOBALS['largeur_ecran']:''))";
 	return $p;
 }