spip / SPIP

ecrire/public/decompiler.php

Braces +46 added lines, -21 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Decompilation de l'arbre de syntaxe abstraite d'un squelette SPIP
 
@@ -26,10 +28,12 @@ 
 	$type .= ($struct->type_requete ? $struct->type_requete :
 			    $struct->table_optionnelle);
 
-	if ($struct->jointures_explicites)
-	  $type .= " " . $struct->jointures_explicites;
-	if ($struct->table_optionnelle)
-	  $type .= "?";
+	if ($struct->jointures_explicites) {
+		  $type .= " " . $struct->jointures_explicites;
+	}
+	if ($struct->table_optionnelle) {
+		  $type .= "?";
+	}
 	// Revoir le cas de la boucle recursive
 
 	$crit = $struct->param;
@@ -88,8 +92,9 @@ 
 	$apres = decompiler_($struct->apres, $fmt, $prof);
 	$args = $filtres = '';
 	if ($p = $struct->param) {
-		if ($p[0][0]==='')
-		  $args = decompiler_liste(array(array_shift($p)), $fmt, $prof);
+		if ($p[0][0]==='') {
+				  $args = decompiler_liste(array(array_shift($p)), $fmt, $prof);
+		}
 		$filtres = decompiler_liste($p, $fmt, $prof);
 	}
 	$f = 'format_champ_' . $fmt;
@@ -97,7 +102,9 @@ 
 }
 
 function decompiler_liste($sources, $fmt='', $prof=0) {
-	if (!is_array($sources)) return '';
+	if (!is_array($sources)) {
+	    return '';
+	}
 	$f = 'format_liste_' . $fmt;
 	$res = '';
 	foreach($sources as $arg) {
@@ -110,11 +117,15 @@ 
 			if ((count($v) == 1) 
 			AND $v[0]->type=='texte'
 			AND (strlen($v[0]->apres) == 1)
-			AND $v[0]->apres == $v[0]->avant)
-			  $args[]= $v[0]->avant . $v[0]->texte . $v[0]->apres;
-			else $args[]= decompiler_($v, $fmt, 0-$prof);
+			AND $v[0]->apres == $v[0]->avant) {
+						  $args[]= $v[0]->avant . $v[0]->texte . $v[0]->apres;
+			} else {
+			    $args[]= decompiler_($v, $fmt, 0-$prof);
+			}
+		}
+		if (($r!=='') OR $args) {
+		    $res .= $f($r, $args, $prof);
 		}
-		if (($r!=='') OR $args) $res .= $f($r, $args, $prof);
 	}
 	return $res;
 }
@@ -125,26 +136,33 @@ 
 // - les champs sont implicitement etendus (crochets implicites mais interdits)
 function decompiler_criteres($boucle, $fmt='', $prof=0) {
 	$sources = $boucle->param;
-	if (!is_array($sources)) return '';
+	if (!is_array($sources)) {
+	    return '';
+	}
 	$res = '';
 	$f = 'format_critere_' . $fmt;
 	foreach($sources as $crit) {
-		if (!is_array($crit)) continue; // boucle recursive
+		if (!is_array($crit)) {
+		    continue;
+		}
+		// boucle recursive
 		array_shift($crit);
 		$args = array();
 		foreach($crit as $i => $v) {
 		  if ((count($v) == 1) 
 		      AND $v[0]->type=='texte'
-		      AND $v[0]->apres)
-		    $args[]= array(array('texte', ( $v[0]->apres . $v[0]->texte . $v[0]->apres)));
-		  else {
+		      AND $v[0]->apres) {
+		  		    $args[]= array(array('texte', ( $v[0]->apres . $v[0]->texte . $v[0]->apres)));
+		  } else {
 		    $res2 = array();
 		    foreach($v as $k => $p) {
 			if (isset($p->type)
 			AND function_exists($d = 'decompiler_' . $p->type)) {
 				$r = $d($p, $fmt, (0-$prof), @$v[$k+1]);
 				$res2[]= array($p->type, $r);
-			} else spip_log("critere $i / $k mal forme");
+			} else {
+			    spip_log("critere $i / $k mal forme");
+			}
 		    }
 		    $args[]= $res2;
 		  }
@@ -157,11 +175,16 @@ 
 
 function decompiler_($liste, $fmt='', $prof=0)
 {
-	if (!is_array($liste))  return '';
+	if (!is_array($liste)) {
+	    return '';
+	}
 	$prof2 = ($prof < 0) ? ($prof-1) : ($prof+1);
 	$contenu = array();
 	foreach($liste as $k => $p) {
-	    if (!isset($p->type)) continue; #??????
+	    if (!isset($p->type)) {
+	        continue;
+	    }
+	    #??????
 	    $d = 'decompiler_' . $p->type;
 	    $next = isset($liste[$k+1]) ? $liste[$k+1] : false;
 	  // Forcer le champ etendu si son source (pas les reecritures)
@@ -192,7 +215,9 @@ 
 
 function public_decompiler($liste, $fmt='', $prof=0, $quoi='') 
 {
-	if (!include_spip('public/format_' . $fmt)) return "'$fmt'?";
+	if (!include_spip('public/format_' . $fmt)) {
+	    return "'$fmt'?";
+	}
 	$f = 'decompiler_' . $quoi;
 	return $f($liste, $fmt, $prof);
 }

ecrire/public/evaluer_page.php

Braces +12 added lines, -10 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;
+}
 
 /**
  * Evaluer la page produite par un squelette
@@ -49,8 +51,9 @@ 
 		$notes($page['notes'],'restaurer_etat');
 	}
 	ob_start();
-	if (strpos($page['texte'],'?xml')!==false)
-		$page['texte'] = str_replace('<'.'?xml', "<\1?xml", $page['texte']);
+	if (strpos($page['texte'],'?xml')!==false) {
+			$page['texte'] = str_replace('<'.'?xml', "<\1?xml", $page['texte']);
+	}
 
 	try {
 		$res = eval('?' . '>' . $page['texte']);
@@ -70,12 +73,10 @@ 
 			$code = trim(highlight_string($code,true));
 			erreur_squelette("L".$erreur['line'].": ".$erreur['message']."<br />".$code,array($page['source'],'',$erreur['file'],'',$GLOBALS['spip_lang']));
 			$page['texte'] = "<!-- Erreur -->";
-		}
-		else {
+		} else {
 			$page['texte'] = ob_get_contents();
 		}
-	}
-	catch (Exception $e){
+	} catch (Exception $e){
 		$code = $page['texte'];
 		$GLOBALS['numero_ligne_php'] = 1;
 		if (!function_exists('numerote_ligne_php')){
@@ -93,8 +94,9 @@ 
 
 	$page['process_ins'] = 'html';
 
-	if (strpos($page['texte'],'?xml')!==false)
-		$page['texte'] = str_replace("<\1?xml", '<'.'?xml', $page['texte']);
-}
+	if (strpos($page['texte'],'?xml')!==false) {
+			$page['texte'] = str_replace("<\1?xml", '<'.'?xml', $page['texte']);
+	}
+	}
 
 page_base_href($page['texte']);

ecrire/public/references.php

Braces +103 added lines, -67 removed lines

@@ -13,7 +13,9 @@ 
 // fonctions de recherche et de reservation
 // dans l'arborescence des boucles
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Retrouver l'index de la boucle dans le cas ou une reference explicite est demandee
@@ -65,8 +67,9 @@ 
  * @return string
  */
 function index_pile($idb, $nom_champ, &$boucles, $explicite='', $defaut=null, $remonte_pile=true) {
-	if (!is_string($defaut))
-		$defaut = '@$Pile[0][\''. strtolower($nom_champ) . '\']';
+	if (!is_string($defaut)) {
+			$defaut = '@$Pile[0][\''. strtolower($nom_champ) . '\']';
+	}
 
 	$i = 0;
 	if (strlen($explicite)) {
@@ -91,8 +94,9 @@ 
 				$boucles[$idb]->select[] = $t;
 			}
 			$champ = '$Pile[$SP' . ($i ? "-$i" : "") . '][\'' . $c . '\']';
-			if (!$joker)
-				return index_compose($conditionnel,$champ);
+			if (!$joker) {
+							return index_compose($conditionnel,$champ);
+			}
 
 			$conditionnel[] = "isset($champ)?$champ";
 		}
@@ -102,9 +106,9 @@ 
 			// Sinon on remonte d'un cran
 			$idb = $boucles[$idb]->id_parent;
 			$i++;
+		} else {
+					$idb = null;
 		}
-		else
-			$idb = null;
 	}
 
 	#	spip_log("Pas vu $nom_champ");
@@ -123,9 +127,10 @@ 
  * @return string
  */
 function index_compose($conditionnel,$defaut){
-	while ($c = array_pop($conditionnel))
-		// si on passe defaut = '', ne pas générer d'erreur de compilation.
+	while ($c = array_pop($conditionnel)) {
+			// si on passe defaut = '', ne pas générer d'erreur de compilation.
 		$defaut = "($c:(".($defaut?$defaut:"''")."))";
+	}
 	return $defaut;
 }
 
@@ -135,7 +140,9 @@ 
 
 	$r = $boucles[$idb]->type_requete;
 
-	if ($r == 'boucle') return array();
+	if ($r == 'boucle') {
+	    return array();
+	}
 	if (!$r) {
 		$joker = false; // indiquer a l'appelant
 		# continuer pour chercher l'erreur suivante
@@ -144,13 +151,13 @@ 
 
 	$desc = $boucles[$idb]->show;
 	$excep = isset($exceptions_des_tables[$r]) ? $exceptions_des_tables[$r] : '';
-	if ($excep)
-		$excep = isset($excep[$nom_champ]) ? $excep[$nom_champ] : '';
+	if ($excep) {
+			$excep = isset($excep[$nom_champ]) ? $excep[$nom_champ] : '';
+	}
 	if ($excep) {
 		$joker = false; // indiquer a l'appelant
 	  return index_exception($boucles[$idb], $desc, $nom_champ, $excep);
-	}
-	else {
+	} else {
 		if (isset($desc['field'][$nom_champ])) {
 			$t = $boucles[$idb]->id_table;
 			$joker = false; // indiquer a l'appelant
@@ -160,18 +167,18 @@ 
 		elseif (/*$joker AND */isset($desc['field']['*'])) {
 			$joker = true; // indiquer a l'appelant
 			return array($nom_champ, $nom_champ);
-		}
-		else {
+		} else {
 			$joker = false; // indiquer a l'appelant
 		  if ($boucles[$idb]->jointures_explicites) {
 		    $t = trouver_champ_exterieur($nom_champ, 
 						 $boucles[$idb]->jointures,
 						 $boucles[$idb]);
-		    if ($t) 
-					return index_exception($boucles[$idb],
+		    if ($t) {
+		    					return index_exception($boucles[$idb],
 					       $desc,
 					       $nom_champ,
 					       array($t[1]['id_table'], $nom_champ));
+		    }
 		  }
 			return array('','');
 		}
@@ -186,8 +193,9 @@ 
 function index_exception(&$boucle, $desc, $nom_champ, $excep)
 {
 	static $trouver_table;
-	if (!$trouver_table)
-		$trouver_table = charger_fonction('trouver_table', 'base');
+	if (!$trouver_table) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+	}
 
 	if (is_array($excep)) {
 		// permettre aux plugins de gerer eux meme des jointures derogatoire ingerables
@@ -200,7 +208,9 @@ 
 			list($e, $x) = $excep;	#PHP4 affecte de gauche a droite
 			$excep = $x;		#PHP5 de droite a gauche !
 			$j = $trouver_table($e, $boucle->sql_serveur);
-			if (!$j) return array('','');
+			if (!$j) {
+			    return array('','');
+			}
 			$e = $j['table'];
 			if (!$t = array_search($e, $boucle->from)) {
 				$k = $j['key']['PRIMARY KEY'];
@@ -217,11 +227,14 @@ 
 				$t = array_search($e, $boucle->from);
 			}
 		}
+	} else {
+	    $t = $boucle->id_table;
 	}
-	else $t = $boucle->id_table;
 	// demander a SQL de gerer le synonyme
 	// ca permet que excep soit dynamique (Cedric, 2/3/06)
-	if ($excep != $nom_champ) $excep .= ' AS '. $nom_champ;
+	if ($excep != $nom_champ) {
+	    $excep .= ' AS '. $nom_champ;
+	}
 	return array("$t.$excep", $nom_champ);
 }
 
@@ -268,16 +281,18 @@ 
 	if ($f = charger_fonction($nom, 'balise', true)) {
 		$p->balise_calculee = true;
 		$res = $f($p);
-		if ($res !== NULL)
-			return $res;
+		if ($res !== NULL) {
+					return $res;
+		}
 	}
 
 	// Certaines des balises comportant un _ sont generiques
 	if ($f = strpos($nom, '_')
 	AND $f = charger_fonction(substr($nom,0,$f+1), 'balise', true)) {
 		$res = $f($p);
-		if ($res !== NULL)
-			return $res;
+		if ($res !== NULL) {
+					return $res;
+		}
 	}
 
 	$f = charger_fonction('DEFAUT', 'calculer_balise');
@@ -298,8 +313,9 @@ 
 	}
 
 	// ne pas passer le filtre securite sur les id_xxx
-	if (strpos($nom, 'ID_') === 0)
-		$p->interdire_scripts = false;
+	if (strpos($nom, 'ID_') === 0) {
+			$p->interdire_scripts = false;
+	}
 
 	// Compatibilite ascendante avec les couleurs html (#FEFEFE) :
 	// SI le champ SQL n'est pas trouve
@@ -349,7 +365,9 @@ 
 		array_shift($c);
 		// construire la liste d'arguments comme pour un filtre
 		$param = compose_filtres_args($p, $c, ',');
-	} else	$param = "";
+	} else {
+	    $param = "";
+	}
 	$collecte = collecter_balise_dynamique($l, $p, $nom);
 
 	$p->code = sprintf(CODE_EXECUTER_BALISE, $nom,
@@ -440,16 +458,17 @@ 
 function champs_traitements ($p) {
 	global $table_des_traitements;
 
-	if (isset($table_des_traitements[$p->nom_champ]))
-		$ps = $table_des_traitements[$p->nom_champ];
-	else {
+	if (isset($table_des_traitements[$p->nom_champ])) {
+			$ps = $table_des_traitements[$p->nom_champ];
+	} else {
 		// quand on utilise un traitement catch-all *
 		// celui-ci ne s'applique pas sur les balises calculees qui peuvent gerer
 		// leur propre securite
-		if (!$p->balise_calculee)
-			$ps = $table_des_traitements['*'];
-		else
-			$ps = false;
+		if (!$p->balise_calculee) {
+					$ps = $table_des_traitements['*'];
+		} else {
+					$ps = false;
+		}
 	}
 
 	if (is_array($ps)) {
@@ -460,19 +479,24 @@ 
 		$table_sql = isset($p->boucles[$idb]->show['table_sql'])?$p->boucles[$idb]->show['table_sql']:false;
 
 		// le traitement peut n'etre defini que pour une table en particulier "spip_articles"
-		if ($table_sql AND isset($ps[$table_sql]))
-			$ps = $ps[$table_sql];
+		if ($table_sql AND isset($ps[$table_sql])) {
+					$ps = $ps[$table_sql];
+		}
 		// ou pour une boucle en particulier "DATA","articles"
-		elseif ($type_requete AND isset($ps[$type_requete]))
-			$ps = $ps[$type_requete];
+		elseif ($type_requete AND isset($ps[$type_requete])) {
+					$ps = $ps[$type_requete];
+		}
 		// ou pour indiferrement quelle que soit la boucle
-		elseif(isset($ps[0]))
-			$ps = $ps[0];
-		else
-			$ps=false;
+		elseif(isset($ps[0])) {
+					$ps = $ps[0];
+		} else {
+					$ps=false;
+		}
 	}
 
-	if (!$ps) return $p->code;
+	if (!$ps) {
+	    return $p->code;
+	}
 
 	// Si une boucle DOCUMENTS{doublons} est presente dans le squelette,
 	// ou si in INCLURE contient {doublons}
@@ -487,8 +511,9 @@ 
 		(strpos($ps,'propre') !== false)
 		OR
 		(strpos($ps,'typo') !== false)
-	))
-		$ps = 'traiter_doublons_documents($doublons, '.$ps.')';
+	)) {
+			$ps = 'traiter_doublons_documents($doublons, '.$ps.')';
+	}
 
 	// La protection des champs par |safehtml est assuree par les extensions
 	// dans la declaration des traitements des champs sensibles
@@ -508,19 +533,22 @@ 
 function applique_filtres($p) {
 
 	// Traitements standards (cf. supra)
-	if ($p->etoile == '')
-		$code = champs_traitements($p);
-	else
-		$code = $p->code;
+	if ($p->etoile == '') {
+			$code = champs_traitements($p);
+	} else {
+			$code = $p->code;
+	}
 
 	// Appliquer les filtres perso
-	if ($p->param)
-		$code = compose_filtres($p, $code);
+	if ($p->param) {
+			$code = compose_filtres($p, $code);
+	}
 
 	// S'il y a un lien avec la session, ajouter un code qui levera
 	// un drapeau dans la structure d'invalidation $Cache
-	if (isset($p->descr['session']))
-		$code = "invalideur_session(\$Cache, $code)";
+	if (isset($p->descr['session'])) {
+			$code = "invalideur_session(\$Cache, $code)";
+	}
 
 	$code = sandbox_composer_interdire_scripts($code, $p);
 	return $code;
@@ -533,7 +561,10 @@ 
 	$image_miette = false;
 	foreach($p->param as $filtre) {
 		$fonc = array_shift($filtre);
-		if (!$fonc) continue; // normalement qu'au premier tour.
+		if (!$fonc) {
+		    continue;
+		}
+		// normalement qu'au premier tour.
 		$is_filtre_image = ((substr($fonc,0,6)=='image_') AND $fonc!='image_graver');
 		if ($image_miette AND !$is_filtre_image){
 	// il faut graver maintenant car apres le filtre en cours
@@ -547,21 +578,25 @@ 
 			$sep = ',';
 		} else {$sep = ':';
 			// |?{a,b} *doit* avoir exactement 2 arguments ; on les force
-			if (count($filtre) != 2)
-				$filtre = array(isset($filtre[0])?$filtre[0]:"", isset($filtre[1])?$filtre[1]:"");
+			if (count($filtre) != 2) {
+							$filtre = array(isset($filtre[0])?$filtre[0]:"", isset($filtre[1])?$filtre[1]:"");
+			}
 		}
 		$arglist = compose_filtres_args($p, $filtre, $sep);
 		$logique = filtre_logique($fonc, $code, substr($arglist,1));
-		if ($logique)
-			$code = $logique;
-		else {
+		if ($logique) {
+					$code = $logique;
+		} else {
 			$code = sandbox_composer_filtre($fonc,$code,$arglist,$p);
-			if ($is_filtre_image) $image_miette = true;
+			if ($is_filtre_image) {
+			    $image_miette = true;
+			}
 		}
 	}
 	// ramasser les images intermediaires inutiles et graver l'image finale
-	if ($image_miette)
-		$code = "filtrer('image_graver',$code)";
+	if ($image_miette) {
+			$code = "filtrer('image_graver',$code)";
+	}
 
 	return $code;
 }
@@ -650,8 +685,9 @@ 
 	}
 
 	// si on est hors d'une boucle de {recherche}, cette balise est vide
-	if (!$p->code)
-		$p->code = "''";
+	if (!$p->code) {
+			$p->code = "''";
+	}
 
 	$p->interdire_scripts = false;
 	return $p;

ecrire/public/styliser_par_z.php

Braces +71 added lines, -55 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+    return;
+}
 
 /**
  * Fonction Page automatique a partir de contenu/xx
@@ -38,8 +40,7 @@ 
 			$page = 'exec';
 			$echafauder = charger_fonction('echafauder','prive',true);
 			define('_ZCORE_EXCLURE_PATH','');
-		}
-		else {
+		} else {
 			$prefix_path = "";
 			$prefix_length = 0;
 			$apl_constant = '_Z_AJAX_PARALLEL_LOAD';
@@ -78,8 +79,9 @@ 
 		}
 	  if ($prepend){
 		  $squelette = substr(find_in_path($prefix_path.$prepend."$fond.$ext"), 0, - strlen(".$ext"));
-	    if ($squelette)
-		    $flux['data'] = $squelette;
+	    if ($squelette) {
+	    		    $flux['data'] = $squelette;
+	    }
 	  }
 
 		// gerer les squelettes non trouves
@@ -95,11 +97,13 @@ 
 				// si on est sur un ?page=XX non trouve
 				// se brancher sur contenu/xx si il existe
 				// ou si c'est un objet spip, associe a une table, utiliser le fond homonyme
-				if (!isset($disponible[$fond]))
-					$disponible[$fond] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$fond,$ext,$echafauder);
+				if (!isset($disponible[$fond])) {
+									$disponible[$fond] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$fond,$ext,$echafauder);
+				}
 
-				if ($disponible[$fond])
-					$flux['data'] = substr(find_in_path($prefix_path."page.$ext"), 0, - strlen(".$ext"));
+				if ($disponible[$fond]) {
+									$flux['data'] = substr(find_in_path($prefix_path."page.$ext"), 0, - strlen(".$ext"));
+				}
 			}
 
 			// echafaudage :
@@ -107,22 +111,22 @@ 
 			// generer un fond automatique a la volee pour les webmestres
 			elseif (strncmp($fond, "$z_contenu/", strlen($z_contenu)+1)==0){
 				$type = substr($fond,strlen($z_contenu)+1);
-				if (($type=='page') AND isset($flux['args']['contexte'][$page]))
-					$type = $flux['args']['contexte'][$page];
-				if (!isset($disponible[$type]))
-					$disponible[$type] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$type,$ext,$echafauder);
+				if (($type=='page') AND isset($flux['args']['contexte'][$page])) {
+									$type = $flux['args']['contexte'][$page];
+				}
+				if (!isset($disponible[$type])) {
+									$disponible[$type] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$type,$ext,$echafauder);
+				}
 				if (is_string($disponible[$type])) {
 					$flux['data'] = $disponible[$type];
-				}
-				elseif ($echafauder
+				} elseif ($echafauder
 					AND include_spip('inc/autoriser')
 					AND isset($GLOBALS['visiteur_session']['statut']) // performance
 					AND autoriser('echafauder',$type)
 					AND $is = $disponible[$type]
 					AND is_array($is)) {
 					$flux['data'] = $echafauder($type,$is[0],$is[1],$is[2],$ext);
-				}
-				else{
+				} else{
 					$flux['data'] = ($disponible['404'] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,'404',$ext,$echafauder));
 				}
 			}
@@ -136,12 +140,15 @@ 
 					AND $dir !== $z_contenu
 					AND in_array($dir,$z_blocs)){
 					$type = substr($fond,strlen("$dir/"));
-					if (($type=='page') AND isset($flux['args']['contexte'][$page]))
-						$type = $flux['args']['contexte'][$page];
-					if ($type!=='page' AND !isset($disponible[$type]))
-						$disponible[$type] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$type,$ext,$echafauder);
-					if ($type=='page' OR $disponible[$type])
-						$flux['data'] = z_trouver_bloc($prefix_path.$prepend,$dir,'dist',$ext);
+					if (($type=='page') AND isset($flux['args']['contexte'][$page])) {
+											$type = $flux['args']['contexte'][$page];
+					}
+					if ($type!=='page' AND !isset($disponible[$type])) {
+											$disponible[$type] = z_contenu_disponible($prefix_path.$prepend,$z_contenu,$type,$ext,$echafauder);
+					}
+					if ($type=='page' OR $disponible[$type]) {
+											$flux['data'] = z_trouver_bloc($prefix_path.$prepend,$dir,'dist',$ext);
+					}
 				}
 			}
 			$squelette = $flux['data'];
@@ -158,10 +165,10 @@ 
 					AND file_exists(($f=$squelette."-".$flux['args']['contexte']['type-page']."-".$flux['args']['contexte']['composition']).".$ext"))
 					OR
 					file_exists(($f=$squelette."-".$flux['args']['contexte']['type-page']).".$ext")
-					))
-				$flux['data'] = $f;
-		}
-		elseif ($fond=='structure' 
+					)) {
+							$flux['data'] = $f;
+			}
+		} elseif ($fond=='structure' 
 			AND z_sanitize_var_zajax()
 			AND $f = find_in_path($prefix_path.$prepend.'ajax'.".$ext")) {
 			$flux['data'] = substr($f,0,-strlen(".$ext"));
@@ -187,8 +194,9 @@ 
  * @return array
  */
 function z_blocs($espace_prive=false) {
-	if ($espace_prive)
-		return (isset($GLOBALS['z_blocs_ecrire'])?$GLOBALS['z_blocs_ecrire']:array('contenu','navigation','extra','head','hierarchie','top'));
+	if ($espace_prive) {
+			return (isset($GLOBALS['z_blocs_ecrire'])?$GLOBALS['z_blocs_ecrire']:array('contenu','navigation','extra','head','hierarchie','top'));
+	}
 	return (isset($GLOBALS['z_blocs'])?$GLOBALS['z_blocs']:array('contenu'));
 }
 
@@ -204,15 +212,17 @@ 
  * @return mixed
  */
 function z_contenu_disponible($prefix_path,$z_contenu,$type,$ext,$echafauder=true){
-	if ($d = z_trouver_bloc($prefix_path,$z_contenu,$type,$ext))
-		return $d;
+	if ($d = z_trouver_bloc($prefix_path,$z_contenu,$type,$ext)) {
+			return $d;
+	}
 	return $echafauder?z_echafaudable($type):false;
 }
 
 function z_fond_valide($squelette){
 	if (!_ZCORE_EXCLURE_PATH
-		OR !preg_match(',('._ZCORE_EXCLURE_PATH.')/,',$squelette))
-		return true;
+		OR !preg_match(',('._ZCORE_EXCLURE_PATH.')/,',$squelette)) {
+			return true;
+	}
   return false;
 }
 
@@ -252,34 +262,37 @@ 
 function z_echafaudable($type){
 	static $pages = null;
 	static $echafaudable = array();
-	if (isset($echafaudable[$type]))
-		return $echafaudable[$type];
-	if (preg_match(',[^\w],',$type))
-		return $echafaudable[$type] = false;
+	if (isset($echafaudable[$type])) {
+			return $echafaudable[$type];
+	}
+	if (preg_match(',[^\w],',$type)) {
+			return $echafaudable[$type] = false;
+	}
 
 	if (test_espace_prive()){
-		if (!function_exists('trouver_objet_exec'))
-			include_spip('inc/pipelines_ecrire');
+		if (!function_exists('trouver_objet_exec')) {
+					include_spip('inc/pipelines_ecrire');
+		}
 		if ($e=trouver_objet_exec($type)){
 			return $echafaudable[$type] = array($e['table'],$e['table_objet_sql'],$e);
-		}
-		else {
+		} else {
 			// peut etre c'est un exec=types qui liste tous les objets "type"
 			if (($t=objet_type($type,false))!==$type
 			  AND $e=trouver_objet_exec($t)){
 				return $echafaudable[$type] = array($e['table'],$e['table_objet_sql'],$t);
 			}
 		}
-	}
-	else {
+	} else {
 		if (is_null($pages)) {
 			$pages = array();
 			$liste = lister_tables_objets_sql();
-			foreach($liste as $t=>$d)
-				if ($d['page']) $pages[$d['page']] = array($d['table_objet'],$t);
+			foreach($liste as $t=>$d) {
+							if ($d['page']) $pages[$d['page']] = array($d['table_objet'],$t);
+			}
+		}
+		if (!isset($pages[$type])) {
+					return $echafaudable[$type] = false;
 		}
-		if (!isset($pages[$type]))
-			return $echafaudable[$type] = false;
 		if (count($pages[$type])==2){
 			$trouver_table = charger_fonction('trouver_table','base');
 			$pages[$type][] = $trouver_table(reset($pages[$type]));
@@ -310,15 +323,16 @@ 
 		$type = $desc_exec['type'];
 		$primary = $desc_exec['id_table_objet'];
 
-		if ($desc_exec['edition']===false)
-			$fond = "objet";
-		else {
+		if ($desc_exec['edition']===false) {
+					$fond = "objet";
+		} else {
 			$trouver_table = charger_fonction('trouver_table','base');
 			$desc = $trouver_table($table_sql);
-			if (isset($desc['field']['id_rubrique']))
-				$fond = 'objet_edit';
-			else
-				$fond = 'objet_edit.sans_rubrique';
+			if (isset($desc['field']['id_rubrique'])) {
+							$fond = 'objet_edit';
+			} else {
+							$fond = 'objet_edit.sans_rubrique';
+			}
 		}
 		$dir = z_blocs(test_espace_prive());
 		$dir = reset($dir);
@@ -350,7 +364,9 @@ 
  */
 function z_sanitize_var_zajax(){
 	$z_ajax = _request('var_zajax');
-	if (!$z_ajax) return false;
+	if (!$z_ajax) {
+	    return false;
+	}
 	if (!$z_blocs = z_blocs(test_espace_prive())
 	  OR !in_array($z_ajax,$z_blocs)) {
 		set_request('var_zajax'); // enlever cette demande incongrue

ecrire/public/format_html.php

Braces +32 added lines, -16 removed lines

@@ -10,22 +10,28 @@ 
  *  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 format_boucle_html ($avant, $nom, $type, $crit, $corps, $apres, $altern, $prof)
 {
 	$avant = $avant ? "<B$nom>$avant" : "";
 	$apres = $apres ? "$apres</B$nom>" : "";
 	$altern = $altern ? "$altern<//B$nom>" : "";
-	if (!$corps) $corps = " />"; else $corps = ">$corps</BOUCLE$nom>";
+	if (!$corps) {
+	    $corps = " />";
+	} else {
+	    $corps = ">$corps</BOUCLE$nom>";
+	}
 	return "$avant<BOUCLE$nom($type)$crit$corps$apres$altern";
 }
 
 function format_inclure_html ($file, $args, $prof)
 {
-	if (strpos($file, '#')===false)
-	 	$t = $file ? ("(" . $file . ")") : "" ;
-	else {
+	if (strpos($file, '#')===false) {
+		 	$t = $file ? ("(" . $file . ")") : "" ;
+	} else {
 		$t = "{fond=" . $file . '}';
 	}
 	$args = !$args ? '' : ("{" . join(", ",$args) . "}");
@@ -35,14 +41,17 @@ 
 function format_polyglotte_html ($args, $prof)
 {
 	$contenu = array(); 
-	foreach($args as $l=>$t)
-		$contenu[]= ($l ? "[$l]" : '') . $t;
+	foreach($args as $l=>$t) {
+			$contenu[]= ($l ? "[$l]" : '') . $t;
+	}
 	return ("<multi>" . join(" ", $contenu) . "</multi>");
 }
 
 function format_idiome_html ($nom, $module, $args, $filtres, $prof)
 {
-	foreach ($args as $k => $v) $args[$k] = "$k=$v";
+	foreach ($args as $k => $v) {
+	    $args[$k] = "$k=$v";
+	}
 	$args = (!$args ? '' : ('{' . join(',', $args) . '}'));
 	return ("<:" . ($module ? "$module:" : "") . $nom . $args . $filtres . ":>");
 }
@@ -72,8 +81,9 @@ 
 			list($type, $valeur) = $operande;
 			if ($type == 'champ' AND $valeur[0]=='[') {
 			  $valeur = substr($valeur,1,-1);
-			  if (preg_match(',^[(](#[^|]*)[)]$,sS', $valeur))
-				$valeur = substr($valeur,1,-1);
+			  if (preg_match(',^[(](#[^|]*)[)]$,sS', $valeur)) {
+			  				$valeur = substr($valeur,1,-1);
+			  }
 			}
 			$crit_s .= $valeur;
 		}
@@ -94,21 +104,27 @@ 
 	for($i=0; $i < count($args)-1; $i++) {
 		list($texte, $type) = $args[$i];
 		list($texte2, $type2) = $args[$i+1];
-		if (!$texte OR !$texte2) continue; 
+		if (!$texte OR !$texte2) {
+		    continue;
+		}
 		$c1 = substr($texte,-1);
 		if ($type2 !== 'texte') {
 		  // si un texte se termine par ( et est suivi d'un champ
 		  // ou assimiles, forcer la notation pleine
-			if ($c1 == '(' AND substr($texte2,0,1) == '#')
-				$args[$i+1][0] = '[(' . $texte2 . ')]';
+			if ($c1 == '(' AND substr($texte2,0,1) == '#') {
+							$args[$i+1][0] = '[(' . $texte2 . ')]';
+			}
 		} else {
-			if ($type == 'texte') continue;
+			if ($type == 'texte') {
+			    continue;
+			}
 			// si un champ ou assimiles est suivi d'un texte
 			// et si celui-ci commence par un caractere de champ
 			// forcer la notation pleine
 			if (($c1 == '}' AND substr(ltrim($texte2),0,1) == '|')
-			OR (preg_match('/[\w\d_*]/', $c1) AND preg_match('/^[\w\d_*{|]/', $texte2)))
-				$args[$i][0] = '[(' . $texte . ')]';
+			OR (preg_match('/[\w\d_*]/', $c1) AND preg_match('/^[\w\d_*{|]/', $texte2))) {
+							$args[$i][0] = '[(' . $texte . ')]';
+			}
 		}
 	}
 	return join("", array_map('array_shift', $args));

ecrire/public/compiler.php

Braces +198 added lines, -111 removed lines

@@ -17,7 +17,9 @@ 
  * @package SPIP\Compilateur\Compilation
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /** Repérer un code ne calculant rien, meme avec commentaire */
 define('CODE_MONOTONE', ",^(\n//[^\n]*\n)?\(?'([^'])*'\)?$,");
@@ -60,11 +62,15 @@ 
 function argumenter_inclure($params, $rejet_filtres, $p, &$boucles, $id_boucle, $echap=true, $lang = '', $fond1=false){
 	$l = array();
 	$erreur_p_i_i = '';
-	if (!is_array($params)) return $l;
+	if (!is_array($params)) {
+	    return $l;
+	}
 	foreach($params as $k => $couple) {
 	// la liste d'arguments d'inclusion peut se terminer par un filtre
 		$filtre = array_shift($couple);
-		if ($filtre) break;
+		if ($filtre) {
+		    break;
+		}
 		foreach($couple as $n => $val) {
 			$var = $val[0];
 			if ($var->type != 'texte') {
@@ -73,20 +79,25 @@ 
 					 array('param' => $var->nom_champ));
 				erreur_squelette($erreur_p_i_i, $p);
 				  break;
+			  } else {
+			      $l[1] = calculer_liste($val, $p->descr, $boucles, $id_boucle);
 			  }
-			  else $l[1] = calculer_liste($val, $p->descr, $boucles, $id_boucle);
 			} else {
 				preg_match(",^([^=]*)(=?)(.*)$,m", $var->texte,$m);
 				$var = $m[1];
 				$auto = false;;
 				if ($m[2]) {
 				  $v = $m[3];
-				  if (preg_match(',^[\'"](.*)[\'"]$,', $v, $m)) $v = $m[1];
+				  if (preg_match(',^[\'"](.*)[\'"]$,', $v, $m)) {
+				      $v = $m[1];
+				  }
 				  $val[0] = new Texte;
 				  $val[0]->texte = $v;
 				} elseif ($k OR $n OR $fond1) {
 				  $auto = true;
-				} else $var = 1;
+				} else {
+				    $var = 1;
+				}
 
 				if ($var == 'lang') {
 				  $lang = !$auto 
@@ -96,21 +107,29 @@ 
 				  $val = $auto
 				    ? index_pile($id_boucle, $var, $boucles)
 				    : calculer_liste($val, $p->descr, $boucles, $id_boucle);
-				  if ($var !== 1)
-				    $val = ($echap?"\'$var\' => ' . argumenter_squelette(":"'$var' => ")
+				  if ($var !== 1) {
+				  				    $val = ($echap?"\'$var\' => ' . argumenter_squelette(":"'$var' => ")
 				    . $val . ($echap? ") . '":" ");
-				  else $val = $echap ? "'.$val.'" : $val;
+				  } else {
+				      $val = $echap ? "'.$val.'" : $val;
+				  }
 				  $l[$var] = $val;
 				}
 			}
 		}
 	}
-	if ($erreur_p_i_i) return false;
+	if ($erreur_p_i_i) {
+	    return false;
+	}
 	// Cas particulier de la langue : si {lang=xx} est definie, on
 	// la passe, sinon on passe la langue courante au moment du calcul
 	// sauf si on n'en veut pas 
-	if ($lang === false) return $l;
-	if (!$lang) $lang = '$GLOBALS["spip_lang"]';
+	if ($lang === false) {
+	    return $l;
+	}
+	if (!$lang) {
+	    $lang = '$GLOBALS["spip_lang"]';
+	}
 	$l['lang'] = ($echap?"\'lang\' => ' . argumenter_squelette(":"'lang' => ")  . $lang . ($echap?") . '":" ");
 
 	return $l;
@@ -144,9 +163,11 @@ 
 
 	} else {
 		$code = calculer_liste($p->texte, $p->descr, $boucles, $id_boucle);
-		if ($code AND preg_match("/^'([^']*)'/s", $code, $r))
-			$fichier = $r[1];
-		else $fichier = '';
+		if ($code AND preg_match("/^'([^']*)'/s", $code, $r)) {
+					$fichier = $r[1];
+		} else {
+		    $fichier = '';
+		}
 	}
 	if (!$code OR $code === '""') {
 		$erreur_p_i_i = array('zbug_parametres_inclus_incorrects',
@@ -173,9 +194,10 @@ 
 		}
 
 		$_contexte = join(",\n\t", $_contexte);
+	} else {
+			return false;
 	}
-	else
-		return false; // j'aurais voulu toucher le fond ...
+	// j'aurais voulu toucher le fond ...
 		
 	$contexte = 'array(' . $_contexte  .')';
 
@@ -188,8 +210,9 @@ 
 		$code = sandbox_composer_inclure_php($fichier, $p, $contexte);
 	} else 	{
 		$_options[] = "\"compil\"=>array($compil)";
-		if ($ajax)
-			$_options[] = $ajax;
+		if ($ajax) {
+					$_options[] = $ajax;
+		}
 		$code = " ' . argumenter_squelette($code) . '"; 
 		$code = "echo " . sprintf(CODE_RECUPERER_FOND, $code, $contexte, implode(',',$_options), "_request(\"connect\")") . ';';
 	}
@@ -270,8 +293,7 @@ 
 					}
 					// trouver l'alias de la table d'arrivee qui porte le statut
 					$id = array_search($id, $boucle->from);
-				}
-				else {
+				} else {
 					$id = $id_table;
 					$statut = preg_replace(',\W,','',$s['champ']); // securite
 				}
@@ -325,9 +347,9 @@ 
 
 	// en mode debug memoriser les premiers passages dans la boucle,
 	// mais pas tous, sinon ca pete.
-	if  (_request('var_mode_affiche') != 'resultat') 
-		$trace = '';
-	else {
+	if  (_request('var_mode_affiche') != 'resultat') {
+			$trace = '';
+	} else {
 		$trace = $boucles[$id_boucle]->descr['nom'] . $id_boucle;
 		$trace = "if (count(@\$GLOBALS['debug_objets']['resultat']['$trace'])<3)
 	    \$GLOBALS['debug_objets']['resultat']['$trace'][] = \$t0;";
@@ -427,10 +449,11 @@ 
 
 	// faudrait expanser le foreach a la compil, car y en a souvent qu'un 
 	// et puis faire un [] plutot qu'un "','."
-	if ($boucle->doublons)
-		$corps .= "\n\t\t\tforeach(" . $boucle->doublons . ' as $k) $doublons[$k] .= "," . ' .
+	if ($boucle->doublons) {
+			$corps .= "\n\t\t\tforeach(" . $boucle->doublons . ' as $k) $doublons[$k] .= "," . ' .
 		index_pile($id_boucle, $primary, $boucles)
 		. "; // doublons\n";
+	}
 
 	// La boucle doit-elle selectionner la langue ?
 	// - par defaut, les boucles suivantes le font
@@ -456,8 +479,7 @@ 
 				'articles', 'rubriques', 'hierarchie', 'breves'
 				)) ? ', '.index_pile($id_boucle, 'titre', $boucles) : '')
 			. ');';
-	}
-	else {
+	} else {
 		$init_lang = '';
 		$fin_lang = '';
 		$fin_lang_select_public = '';
@@ -474,8 +496,9 @@ 
 	}
 
 	// gestion optimale des separateurs et des boucles constantes
-	if (count($boucle->separateur))
-	  $code_sep = ("'" . str_replace("'","\'",join('',$boucle->separateur)) . "'");
+	if (count($boucle->separateur)) {
+		  $code_sep = ("'" . str_replace("'","\'",join('',$boucle->separateur)) . "'");
+	}
 
 	$corps .= 
 		((!$boucle->separateur) ? 
@@ -493,17 +516,19 @@ 
 	// souhaite invalider ces elements
 	if (!$constant AND $primary) {
 		include_spip('inc/invalideur');
-		if (function_exists($i = 'calcul_invalideurs'))
-			$corps = $i($corps, $primary, $boucles, $id_boucle);
+		if (function_exists($i = 'calcul_invalideurs')) {
+					$corps = $i($corps, $primary, $boucles, $id_boucle);
+		}
 	}
 
 	// gerer le compteur de boucle 
 	// avec ou sans son utilisation par les criteres {1/3} {1,4} {n-2,1}...
 
-	if ($boucle->partie OR $boucle->cptrows)
-		$corps = "\n\t\t\$Numrows['$id_boucle']['compteur_boucle']++;"
+	if ($boucle->partie OR $boucle->cptrows) {
+			$corps = "\n\t\t\$Numrows['$id_boucle']['compteur_boucle']++;"
 		. $boucle->partie 
 		. $corps;
+	}
 
 	// depiler la lang de la boucle si besoin
 	$corps .= $fin_lang_select_public;
@@ -512,28 +537,35 @@ 
 
 	if (preg_match(CODE_MONOTONE,str_replace("\\'",'',$corps), $r)) {
 		if (!isset($r[2]) OR (!$r[2])) {
-			if (!$boucle->numrows)
-				return "\n\t\$t0 = '';";
-			else
-				$corps = "";
+			if (!$boucle->numrows) {
+							return "\n\t\$t0 = '';";
+			} else {
+							$corps = "";
+			}
 		} else {
 			$boucle->numrows = true;
 			$corps = "\n\t\$t0 = str_repeat($corps, \$Numrows['$id_boucle']['total']);";
 		}
-	} else $corps = "while (\$Pile[\$SP]=\$iter->fetch()) {\n$corps\n	}"; 
+	} else {
+	    $corps = "while (\$Pile[\$SP]=\$iter->fetch()) {\n$corps\n	}";
+	}
 
 	$count = '';
 	if (!$boucle->select) {
-		if (!$boucle->numrows OR $boucle->limit OR $boucle->mode_partie OR $boucle->group)
-			$count = '1';
-		else $count = 'count(*)';
+		if (!$boucle->numrows OR $boucle->limit OR $boucle->mode_partie OR $boucle->group) {
+					$count = '1';
+		} else {
+		    $count = 'count(*)';
+		}
 		$boucles[$id_boucle]->select[]= $count; 
 	}
 
-	if ($flag_cpt)
-		$nums = "\n\t// COMPTEUR\n\t"
+	if ($flag_cpt) {
+			$nums = "\n\t// COMPTEUR\n\t"
 		. "\$Numrows['$id_boucle']['compteur_boucle'] = 0;\n\t";
-	else $nums = '';
+	} else {
+	    $nums = '';
+	}
 
 	if ($boucle->numrows OR $boucle->mode_partie) {
 		$nums .= "\$Numrows['$id_boucle']['total'] = @intval(\$iter->count());"
@@ -600,10 +632,11 @@ 
 	$init[] = calculer_dec('having', calculer_dump_array($boucle->having));
 	$s = $d = "";
 	foreach ($init as $i){
-		if (reset($i))
-			$s .= "\n\t\t".end($i);
-		else
-			$d .= "\n\t".end($i);
+		if (reset($i)) {
+					$s .= "\n\t\t".end($i);
+		} else {
+					$d .= "\n\t".end($i);
+		}
 	}
 
 	return ($boucle->hierarchie ? "\n\t$boucle->hierarchie" : '')
@@ -654,7 +687,9 @@ 
 **/
 function reconstruire_contexte_compil($context_compil)
 {
-	if (!is_array($context_compil)) return $context_compil;
+	if (!is_array($context_compil)) {
+	    return $context_compil;
+	}
 	$p = new Contexte;
 	$p->descr = array('sourcefile' => $context_compil[0],
 				  'nom' => $context_compil[1]);
@@ -680,8 +715,9 @@ 
 			AND $test = preg_replace(",'[^']*',UimsS","",$test) // supprimer les chaines qui peuvent contenir des fonctions SQL qui ne genent pas
 			AND preg_match(",\w+\s*\(,UimsS",$test,$regs) // tester la presence de fonctions restantes
 		)*/
-	)
-		$static = "";
+	) {
+			$static = "";
+	}
 
 	return array($static,'$command[\''.$nom.'\'] = ' . $val . ';');
 }
@@ -689,15 +725,19 @@ 
 // http://doc.spip.org/@calculer_dump_array
 function calculer_dump_array($a)
 {
-  if (!is_array($a)) return $a ;
+  if (!is_array($a)) {
+      return $a ;
+  }
   $res = "";
-  if ($a AND $a[0] == "'?'") 
-    return ("(" . calculer_dump_array($a[1]) .
+  if ($a AND $a[0] == "'?'") {
+      return ("(" . calculer_dump_array($a[1]) .
 	    " ? " . calculer_dump_array($a[2]) .
 	    " : " . calculer_dump_array($a[3]) .
 	    ")");
-  else {
-    foreach($a as $v) $res .= ", " . calculer_dump_array($v);
+  } else {
+    foreach($a as $v) {
+        $res .= ", " . calculer_dump_array($v);
+    }
     return "\n\t\t\tarray(" . substr($res,2) . ')';
   }
 }
@@ -706,8 +746,9 @@ 
 function calculer_dump_join($a)
 {
   $res = "";
-  foreach($a as $k => $v) 
-		$res .= ", '$k' => array(".implode(',',$v).")";
+  foreach($a as $k => $v) {
+  		$res .= ", '$k' => array(".implode(',',$v).")";
+  }
   return 'array(' . substr($res,2) . ')';
 }
 
@@ -715,7 +756,9 @@ 
 function calculer_from(&$boucle)
 {
   $res = "";
-  foreach($boucle->from as $k => $v) $res .= ",'$k' => '$v'";
+  foreach($boucle->from as $k => $v) {
+      $res .= ",'$k' => '$v'";
+  }
   return 'array(' . substr($res,1) . ')';
 }
 
@@ -723,7 +766,9 @@ 
 function calculer_from_type(&$boucle)
 {
   $res = "";
-  foreach($boucle->from_type as $k => $v) $res .= ",'$k' => '$v'";
+  foreach($boucle->from_type as $k => $v) {
+      $res .= ",'$k' => '$v'";
+  }
   return 'array(' . substr($res,1) . ')';
 }
 
@@ -731,8 +776,9 @@ 
 function calculer_order(&$boucle)
 {
 	if (!$order = $boucle->order
-	AND !$order = $boucle->default_order)
-		$order = array();
+	AND !$order = $boucle->default_order) {
+			$order = array();
+	}
 
 	/*if (isset($boucle->modificateur['collate'])){
 		$col = "." . $boucle->modificateur['collate'];
@@ -750,23 +796,31 @@ 
 
 // http://doc.spip.org/@calculer_liste
 function calculer_liste($tableau, $descr, &$boucles, $id_boucle='') {
-	if (!$tableau) return "''";
-	if (!isset($descr['niv'])) $descr['niv'] = 0;
+	if (!$tableau) {
+	    return "''";
+	}
+	if (!isset($descr['niv'])) {
+	    $descr['niv'] = 0;
+	}
 	$codes = compile_cas($tableau, $descr, $boucles, $id_boucle);
-	if ($codes === false) return false;
+	if ($codes === false) {
+	    return false;
+	}
 	$n = count($codes);
-	if (!$n) return "''";
+	if (!$n) {
+	    return "''";
+	}
 	$tab = str_repeat("\t", $descr['niv']);
 	if (_request('var_mode_affiche') != 'validation') {
-		if ($n==1) 
-			return $codes[0];
-		else {
+		if ($n==1) {
+					return $codes[0];
+		} else {
 			$res = '';
 			foreach($codes as $code) {
 				if (!preg_match("/^'[^']*'$/", $code)
-				OR substr($res,-1,1)!=="'")
-				  $res .=  " .\n$tab$code";
-				else {
+				OR substr($res,-1,1)!=="'") {
+								  $res .=  " .\n$tab$code";
+				} else {
 				  $res = substr($res,0,-1) . substr($code,1);
 				}
 			}
@@ -787,8 +841,9 @@ 
 
         $codes = array();
 	// cas de la boucle recursive
-	if (is_array($id_boucle)) 
-	  $id_boucle = $id_boucle[0];
+	if (is_array($id_boucle)) {
+		  $id_boucle = $id_boucle[0];
+	}
 	$type = !$id_boucle ? '' : $boucles[$id_boucle]->type_requete;
 	$tab = str_repeat("\t", ++$descr['niv']);
 	$mode = _request('var_mode_affiche');
@@ -863,11 +918,16 @@ 
 				$commentaire= "?$nom";
 				if (!$boucles[$nom]->milieu
 				AND $boucles[$nom]->type_requete <> TYPE_RECURSIF) {
-					if ($altern != "''") $code .= "\n. $altern";
-					if ($avant<>"''" OR $apres<>"''")
-					  spip_log("boucle $nom toujours vide, code superflu dans $id");
+					if ($altern != "''") {
+					    $code .= "\n. $altern";
+					}
+					if ($avant<>"''" OR $apres<>"''") {
+										  spip_log("boucle $nom toujours vide, code superflu dans $id");
+					}
 					$avant = $apres = $altern = "''";
-				} else if ($altern != "''") $altern = "($altern)";
+				} else if ($altern != "''") {
+				    $altern = "($altern)";
+				}
 			}
 			break;
 
@@ -930,8 +990,9 @@ 
 #			AND (strpos($code,'interdire_scripts') !== 0)
 			AND !preg_match(_REGEXP_COND_VIDE_NONVIDE, $code)
 			AND !preg_match(_REGEXP_COND_NONVIDE_VIDE, $code)
-			AND !preg_match(_REGEXP_CONCAT_NON_VIDE, $code)) 
-				$code = "strval($code)";
+			AND !preg_match(_REGEXP_CONCAT_NON_VIDE, $code)) {
+							$code = "strval($code)";
+			}
 			break;
 
 		default: 
@@ -962,9 +1023,15 @@ 
 // http://doc.spip.org/@compile_retour
 function compile_retour($code, $avant, $apres, $altern, $tab, $n)
 {
-	if ($avant == "''") $avant = '';
-	if ($apres == "''") $apres = '';
-	if (!$avant AND !$apres AND ($altern==="''")) return $code;
+	if ($avant == "''") {
+	    $avant = '';
+	}
+	if ($apres == "''") {
+	    $apres = '';
+	}
+	if (!$avant AND !$apres AND ($altern==="''")) {
+	    return $code;
+	}
 
 	if (preg_match(_REGEXP_CONCAT_NON_VIDE, $code)) {
 		$t = $code;
@@ -984,18 +1051,21 @@ 
 		$t .
 		(!$apres ? "" : " . $apres");
 
-	if ($res !== $t) $res = "($res)";
+	if ($res !== $t) {
+	    $res = "($res)";
+	}
 	return !$cond ? $res : "($cond ?\n\t$tab$res :\n\t$tab$altern)";
 }
 
 
 function compile_inclure_doublons($lexemes)
 {
-	foreach($lexemes as $v)
-	  if($v->type === 'include' AND $v->param) 
+	foreach($lexemes as $v) {
+		  if($v->type === 'include' AND $v->param) 
 	    foreach($v->param as $r) 
 	      if (trim($r[0]) === 'doublons') 
 		return true;
+	}
 	return false;
 }
 
@@ -1023,7 +1093,9 @@ 
 
 	// rendre inertes les echappements de #[](){}<>
 	$i = 0;
-	while(false !== strpos($squelette, $inerte = '-INERTE'.$i)) $i++;
+	while(false !== strpos($squelette, $inerte = '-INERTE'.$i)) {
+	    $i++;
+	}
 	$squelette = preg_replace_callback(',\\\\([#[()\]{}<>]),',
 		create_function('$a', "return '$inerte-'.ord(\$a[1]).'-';"), $squelette, -1, $esc);
 
@@ -1041,10 +1113,11 @@ 
 	$boucles = compiler_squelette($squelette, $boucles, $nom, $descr, $sourcefile, $connect);
 
 	// restituer les echappements
-	if ($esc)
-		foreach($boucles as $i=>$boucle) {
+	if ($esc) {
+			foreach($boucles as $i=>$boucle) {
 			$boucles[$i]->return = preg_replace_callback(",$inerte-(\d+)-,", create_function('$a', 'return chr($a[1]);'),
 				$boucle->return);
+	}
 			$boucles[$i]->descr['squelette'] = preg_replace_callback(",$inerte-(\d+)-,", create_function('$a', 'return "\\\\".chr($a[1]);'),
 				$boucle->descr['squelette']);
 		}
@@ -1053,15 +1126,17 @@ 
 	if ($debug) {
 		include_spip('public/decompiler');
 		foreach($boucles as $id => $boucle) {
-			if ($id)
-			  $decomp = "\n/* BOUCLE " .
+			if ($id) {
+						  $decomp = "\n/* BOUCLE " .
 			    $boucle->type_requete .
 			    " " .
 			    str_replace('*/', '* /', public_decompiler($boucle, $gram, 0, 'criteres')) .
 			    " */\n";
-			else $decomp = ("\n/*\n" . 
+			} else {
+			    $decomp = ("\n/*\n" . 
 				 str_replace('*/', '* /', public_decompiler($squelette, $gram)) 
 				 . "\n*/");
+			}
 			$boucles[$id]->return = $decomp .$boucle->return; 
 			$GLOBALS['debug_objets']['code'][$nom.$id] = $boucle->return;
 		}
@@ -1081,8 +1156,9 @@ 
 		$GLOBALS['debug_objets']['squelette'][$nom] = $descr['squelette'];
 		$GLOBALS['debug_objets']['sourcefile'][$nom] = $sourcefile;
 
-		if (!isset($GLOBALS['debug_objets']['principal']))
-			$GLOBALS['debug_objets']['principal'] = $nom;
+		if (!isset($GLOBALS['debug_objets']['principal'])) {
+					$GLOBALS['debug_objets']['principal'] = $nom;
+		}
 	}
 	foreach ($boucles as $id => $boucle) {
 		$GLOBALS['debug_objets']['boucle'][$nom.$id] = $boucle;
@@ -1093,21 +1169,26 @@ 
 	// et reperer si les doublons sont demandes
 	// pour un inclure ou une boucle document
 	// c'est utile a la fonction champs_traitements
-	if (!$trouver_table)
-		$trouver_table = charger_fonction('trouver_table', 'base');
+	if (!$trouver_table) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+	}
 
 	foreach($boucles as $id => $boucle) {
-		if (!($type = $boucle->type_requete)) continue;
+		if (!($type = $boucle->type_requete)) {
+		    continue;
+		}
 		if (!$descr['documents'] AND (
 			(($type == 'documents') AND $boucle->doublons) OR
 				compile_inclure_doublons($boucle->avant) OR
 				compile_inclure_doublons($boucle->apres) OR
 				compile_inclure_doublons($boucle->milieu) OR
-				compile_inclure_doublons($boucle->altern)))
-			$descr['documents'] = true;  
+				compile_inclure_doublons($boucle->altern))) {
+					$descr['documents'] = true;
+		}
 		if ($type != TYPE_RECURSIF) {
-			if (!$boucles[$id]->sql_serveur AND $connect)
-				$boucles[$id]->sql_serveur = $connect;
+			if (!$boucles[$id]->sql_serveur AND $connect) {
+							$boucles[$id]->sql_serveur = $connect;
+			}
 
 			// chercher dans les iterateurs du repertoire iterateur/
 			if ($g = charger_fonction(
@@ -1141,18 +1222,20 @@ 
 					$boucles[$id]->descr = &$descr;
 					if ((!$boucles[$id]->jointures)
 					    AND is_array($show['tables_jointures'])
-					        AND count($x = $show['tables_jointures']))
-						$boucles[$id]->jointures = $x;
+					        AND count($x = $show['tables_jointures'])) {
+											$boucles[$id]->jointures = $x;
+					}
 					if ($boucles[$id]->jointures_explicites){
 						$jointures = preg_split("/\s+/",$boucles[$id]->jointures_explicites);
-						while ($j=array_pop($jointures))
-							array_unshift($boucles[$id]->jointures,$j);
+						while ($j=array_pop($jointures)) {
+													array_unshift($boucles[$id]->jointures,$j);
+						}
 					}
 				} else {
 					// Pas une erreur si la table est optionnelle
-					if ($boucles[$id]->table_optionnelle)
-						$boucles[$id]->type_requete = '';
-					else  {
+					if ($boucles[$id]->table_optionnelle) {
+											$boucles[$id]->type_requete = '';
+					} else  {
 						$boucles[$id]->type_requete = false;
 						$boucle = $boucles[$id];
 						$x = (!$boucle->sql_serveur ? '' :
@@ -1255,7 +1338,9 @@ 
 				_q($boucle->sql_serveur) .
 				";" .
 				$f($id, $boucles);
-		} else $req = ("\n\treturn '';");
+		} else {
+		    $req = ("\n\treturn '';");
+		}
 
 		$boucles[$id]->return = 
 			"\n\nfunction BOUCLE" . strtr($id,"-","_") . $nom .
@@ -1266,7 +1351,9 @@ 
 
 	// Au final, si le corps ou un critere au moins s'est mal compile
 	// retourner False, sinon inserer leur decompilation
-	if (is_bool($corps)) return false;
+	if (is_bool($corps)) {
+	    return false;
+	}
 
 	$principal = "\nfunction " . $nom . '($Cache, $Pile, $doublons=array(), $Numrows=array(), $SP=0) {
 '

ecrire/public/composer.php

Braces +157 added lines, -94 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/texte');
 include_spip('inc/documents');
@@ -36,10 +38,13 @@ 
 	$nom = calculer_nom_fonction_squel($squelette, $mime_type, $connect);
 
 	//  si deja en memoire (INCLURE  a repetition) c'est bon.
-	if (function_exists($nom)) return $nom;
+	if (function_exists($nom)) {
+	    return $nom;
+	}
 
-	if (defined('_VAR_MODE') AND _VAR_MODE == 'debug')
-		$GLOBALS['debug_objets']['courant'] = $nom;
+	if (defined('_VAR_MODE') AND _VAR_MODE == 'debug') {
+			$GLOBALS['debug_objets']['courant'] = $nom;
+	}
 
 	$phpfile = sous_repertoire(_DIR_SKELS,'',false,true) . $nom . '.php';
 
@@ -60,7 +65,9 @@ 
 
 	// tester si le eval ci-dessus a mis le squelette en memoire
 
-	if (function_exists($nom)) return $nom;
+	if (function_exists($nom)) {
+	    return $nom;
+	}
 
 	// charger le source, si possible, et compiler 
 	if (lire_fichier ($source, $skel)) {
@@ -69,7 +76,9 @@ 
 	}
 
 	// Ne plus rien faire si le compilateur n'a pas pu operer.
-	if (!$skel_code) return false;
+	if (!$skel_code) {
+	    return false;
+	}
 
 	foreach($skel_code as $id => $boucle) {
 		$f = $boucle->return;
@@ -88,9 +97,9 @@ 
 
 	if ($nom) {
 		// Si le code est bon, concatener et mettre en cache
-		if (function_exists($nom))
-			$code = squelette_traduit($skel, $source, $phpfile, $skel_code);
-		else {
+		if (function_exists($nom)) {
+					$code = squelette_traduit($skel, $source, $phpfile, $skel_code);
+		} else {
 		// code semantiquement faux: bug du compilateur
 		// $boucle est en fait ici la fct principale du squelette
 			$msg = _T('zbug_erreur_compilation');
@@ -107,8 +116,9 @@ 
 		// si c'est ce que demande le debusqueur, lui passer la main
 		if ($GLOBALS['debug_objets']['sourcefile']
 		AND (_request('var_mode_objet') == $nom)
-		AND (_request('var_mode_affiche') == 'code')  )
-			erreur_squelette();
+		AND (_request('var_mode_affiche') == 'code')  ) {
+					erreur_squelette();
+		}
 	}
 	return $nom ? $nom : false;
 }
@@ -118,18 +128,20 @@ 
 
 	// Le dernier index est '' (fonction principale)
 	$noms = substr(join (', ', array_keys($boucles)), 0, -2);
-	if (CODE_COMMENTE)
-	$code = "
+	if (CODE_COMMENTE) {
+		$code = "
 /*
  * Squelette : $sourcefile
  * Date :      ".gmdate("D, d M Y H:i:s", @filemtime($sourcefile))." GMT
  * Compile :   ".gmdate("D, d M Y H:i:s", time())." GMT
  * " . (!$boucles ?  "Pas de boucle" :	("Boucles :   " . $noms)) ."
  */ " ;
+	}
 
 	$code = '<'. "?php\n" . $code . join('', $boucles)  . "\n?" .'>';
-	if (!defined('_VAR_NOCACHE') OR !_VAR_NOCACHE)
-		ecrire_fichier($phpfile, $code);
+	if (!defined('_VAR_NOCACHE') OR !_VAR_NOCACHE) {
+			ecrire_fichier($phpfile, $code);
+	}
 	return $code;
 }
 
@@ -140,10 +152,13 @@ 
 	// ne verifier la date de mes_fonctions et mes_options qu'une seule fois
 	// par hit
 	if (is_null($date_change)){
-		if (@file_exists($fonc = 'mes_fonctions.php'))
-			$date_change = @filemtime($fonc); # compatibilite
-		if (defined('_FILE_OPTIONS'))
-			$date_change = max($date_change,@filemtime(_FILE_OPTIONS));
+		if (@file_exists($fonc = 'mes_fonctions.php')) {
+					$date_change = @filemtime($fonc);
+		}
+		# compatibilite
+		if (defined('_FILE_OPTIONS')) {
+					$date_change = max($date_change,@filemtime(_FILE_OPTIONS));
+		}
 	}
 	return (
 		(defined('_VAR_MODE') AND in_array(_VAR_MODE, array('recalcul','preview','debug')))
@@ -179,10 +194,11 @@ 
 			# $j = Content-Type, et pas content-TYPE.
 			$j = join('-', array_map('ucwords', explode('-', strtolower($r[2]))));
 
-			if ($j=='X-Spip-Filtre' AND isset($headers[$j]))
-				$headers[$j].="|".$r[3];
-			else
-				$headers[$j] = $r[3];
+			if ($j=='X-Spip-Filtre' AND isset($headers[$j])) {
+							$headers[$j].="|".$r[3];
+			} else {
+							$headers[$j] = $r[3];
+			}
 		}
 	}
 	// S'agit-il d'un resultat constant ou contenant du code php
@@ -237,8 +253,9 @@ 
 // http://doc.spip.org/@filtre_introduction_dist
 function filtre_introduction_dist($descriptif, $texte, $longueur, $connect) {
 	// Si un descriptif est envoye, on l'utilise directement
-	if (strlen($descriptif))
-		return propre($descriptif,$connect);
+	if (strlen($descriptif)) {
+			return propre($descriptif,$connect);
+	}
 
 	// De preference ce qui est marque <intro>...</intro>
 	$intro = '';
@@ -246,8 +263,9 @@ 
 	while ($fin = strpos($texte, "</intro>")) {
 		$zone = substr($texte, 0, $fin);
 		$texte = substr($texte, $fin + strlen("</intro>"));
-		if ($deb = strpos($zone, "<intro>") OR substr($zone, 0, 7) == "<intro>")
-			$zone = substr($zone, $deb + 7);
+		if ($deb = strpos($zone, "<intro>") OR substr($zone, 0, 7) == "<intro>") {
+					$zone = substr($zone, $deb + 7);
+		}
 		$intro .= $zone;
 	}
 
@@ -262,34 +280,39 @@ 
 	// Cependant pour des questions de perfs on coupe quand meme, en prenant
 	// large et en se mefiant des tableaux #1323
 
-	if (strlen($intro))
-		$texte = $intro;
-
-	else
+	if (strlen($intro)) {
+			$texte = $intro;
+	} else
 	if (strpos("\n".$texte, "\n|")===false
 	  AND strlen($texte) > 2.5*$longueur){
-		if (strpos($texte,"<multi")!==false)
-			$texte = extraire_multi($texte);
+		if (strpos($texte,"<multi")!==false) {
+					$texte = extraire_multi($texte);
+		}
 		$texte = couper($texte, 2*$longueur);
 	}
 
 	// ne pas tenir compte des notes
-	if ($notes = charger_fonction('notes', 'inc', true))
-		$notes('','empiler');
+	if ($notes = charger_fonction('notes', 'inc', true)) {
+			$notes('','empiler');
+	}
 	// Supprimer les modèles avant le propre afin d'éviter qu'ils n'ajoutent du texte indésirable
 	// dans l'introduction.
 	$texte = supprime_img($texte, '');
 	$texte = propre($texte,$connect);
-	if ($notes)
-		$notes('','depiler');
+	if ($notes) {
+			$notes('','depiler');
+	}
 
-	if (!defined('_INTRODUCTION_SUITE')) define('_INTRODUCTION_SUITE', '&nbsp;(...)');
+	if (!defined('_INTRODUCTION_SUITE')) {
+	    define('_INTRODUCTION_SUITE', '&nbsp;(...)');
+	}
 	$texte = couper($texte, $longueur, _INTRODUCTION_SUITE);
 
 	// et reparagrapher si necessaire (coherence avec le cas descriptif)
-	if ($GLOBALS['toujours_paragrapher'])
-		// Fermer les paragraphes
+	if ($GLOBALS['toujours_paragrapher']) {
+			// Fermer les paragraphes
 		$texte = paragrapher($texte, $GLOBALS['toujours_paragrapher']);
+	}
 
 	return $texte;
 }
@@ -323,12 +346,13 @@ 
 // http://doc.spip.org/@argumenter_squelette
 function argumenter_squelette($v) {
 
-	if (!is_array($v))
-		return "'" . texte_script($v) . "'";
-	else {
+	if (!is_array($v)) {
+			return "'" . texte_script($v) . "'";
+	} else {
 		$out = array();
-		foreach($v as $k=>$val) 
-			$out [] = argumenter_squelette($k) . '=>' . argumenter_squelette($val);
+		foreach($v as $k=>$val) {
+					$out [] = argumenter_squelette($k) . '=>' . argumenter_squelette($val);
+		}
 	  return 'array(' . join(", ", $out) . ')';
 	}
 }
@@ -347,8 +371,7 @@ 
 			array_unshift($args,$nom);
 			// et passer sur la fonction generique pour la suite
 			$nomfonction = $nomfonction_generique;
-		}
-		else {
+		} else {
 			$msg = array('zbug_balise_inexistante',array('from'=>'CVT','balise'=>$nom));
 			erreur_squelette($msg, $context_compil);
 			return '';
@@ -359,7 +382,9 @@ 
 
 	$r = !function_exists($f) ? $args : $f($args, $context_compil); 
 
-	if (!is_array($r)) return $r;
+	if (!is_array($r)) {
+	    return $r;
+	}
 
 	// verifier que la fonction dyn est la, 
 	// sinon se replier sur la generique si elle existe
@@ -376,8 +401,9 @@ 
 		}
 	}
 
-	if (!_DIR_RESTREINT) 
-		$file = _DIR_RESTREINT_ABS . $file;
+	if (!_DIR_RESTREINT) {
+			$file = _DIR_RESTREINT_ABS . $file;
+	}
 	return synthetiser_balise_dynamique($nomfonction, $r, $file, $context_compil);
 }
 
@@ -394,8 +420,9 @@ 
 
 	if ($d = @opendir(_DIR_LOGOS)) {
 		while($f = readdir($d)) {
-			if (preg_match($type, $f, $r))
-				$logos[] = $r[1];
+			if (preg_match($type, $f, $r)) {
+							$logos[] = $r[1];
+			}
 		}
 	}
 	@closedir($d);
@@ -420,19 +447,22 @@ 
 function lang_select_public($lang, $lang_select, $titre=null) {
 	// Cas 1. forcer_lang = true et pas de critere {lang_select}
 	if (isset($GLOBALS['forcer_lang']) AND $GLOBALS['forcer_lang']
-	AND $lang_select !== 'oui')
-		$lang = $GLOBALS['spip_lang'];
+	AND $lang_select !== 'oui') {
+			$lang = $GLOBALS['spip_lang'];
+	}
 
 	// Cas 2. l'objet n'a pas de langue definie (ou definie a '')
-	elseif (!strlen($lang))
-		$lang = $GLOBALS['spip_lang'];
+	elseif (!strlen($lang)) {
+			$lang = $GLOBALS['spip_lang'];
+	}
 
 	// Cas 3. l'objet est multilingue !
 	elseif ($lang_select !== 'oui'
 	  AND strlen($titre) > 10
 	  AND strpos($titre, '<multi>') !== false
-	  AND strpos(echappe_html($titre), '<multi>') !== false)
-		$lang = $GLOBALS['spip_lang'];
+	  AND strpos(echappe_html($titre), '<multi>') !== false) {
+			$lang = $GLOBALS['spip_lang'];
+	}
 
 	// faire un lang_select() eventuellement sur la langue inchangee
 	lang_select($lang);
@@ -448,33 +478,43 @@ 
 	foreach ($envd as $table => $liste) {
 		$n = '';
 		foreach(explode(',',$liste) as $val) {
-			if ($a = intval($val) AND $val === strval($a))
-				$n.= ','.$val;
+			if ($a = intval($val) AND $val === strval($a)) {
+							$n.= ','.$val;
+			}
+		}
+		if (strlen($n)) {
+					$envd[$table] = $n;
+		} else {
+					unset($envd[$table]);
 		}
-		if (strlen($n))
-			$envd[$table] = $n;
-		else
-			unset($envd[$table]);
 	}
 	return $envd;
 }
 
 // http://doc.spip.org/@match_self
 function match_self($w){
-	if (is_string($w)) return false;
+	if (is_string($w)) {
+	    return false;
+	}
 	if (is_array($w)) {
-		if (in_array(reset($w),array("SELF","SUBSELECT"))) return $w;
-		foreach(array_filter($w,'is_array') as $sw)
-			if ($m=match_self($sw)) return $m;
+		if (in_array(reset($w),array("SELF","SUBSELECT"))) {
+		    return $w;
+		}
+		foreach(array_filter($w,'is_array') as $sw) {
+					if ($m=match_self($sw)) return $m;
+		}
 	}
 	return false;
 }
 // http://doc.spip.org/@remplace_sous_requete
 function remplace_sous_requete($w,$sousrequete){
 	if (is_array($w)) {
-		if (in_array(reset($w),array("SELF","SUBSELECT"))) return $sousrequete;
-		foreach($w as $k=>$sw)
-			$w[$k] = remplace_sous_requete($sw,$sousrequete);
+		if (in_array(reset($w),array("SELF","SUBSELECT"))) {
+		    return $sousrequete;
+		}
+		foreach($w as $k=>$sw) {
+					$w[$k] = remplace_sous_requete($sw,$sousrequete);
+		}
 	}
 	return $w;
 }
@@ -483,8 +523,11 @@ 
 	$where_simples = array();
 	$where_sous = array();
 	foreach($where as $k=>$w){
-		if (match_self($w)) $where_sous[$k] = $w;
-		else $where_simples[$k] = $w;
+		if (match_self($w)) {
+		    $where_sous[$k] = $w;
+		} else {
+		    $where_simples[$k] = $w;
+		}
 	}
 	return array($where_simples,$where_sous);
 }
@@ -523,10 +566,16 @@ 
 	$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;
-			else $op = $v[0] ? $v[0] : $v;
-		} else $op = $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')) {
 			unset($where[$k]);
 			$menage = true;
@@ -613,13 +662,18 @@ 
 		$cle = $cledef;
 		// le format de join est :
 		// array(table depart, cle depart [,cle arrivee[,condition optionnelle and ...]])
-		if (count($join[$cle])==2) $join[$cle][] = $join[$cle][1];
-		if (count($join[$cle])==3) $join[$cle][] = '';
+		if (count($join[$cle])==2) {
+		    $join[$cle][] = $join[$cle][1];
+		}
+		if (count($join[$cle])==3) {
+		    $join[$cle][] = '';
+		}
 		list($t,$c,$carr,$and) = $join[$cle];
 		// si le nom de la jointure n'a pas ete specifiee, on prend Lx avec x sont rang dans la liste
 		// pour compat avec ancienne convention
-		if (is_numeric($cle))
-			$cle = "L$k";
+		if (is_numeric($cle)) {
+					$cle = "L$k";
+		}
 		if (!$menage
 		OR isset($afrom[$cle])
 		OR calculer_jointnul($cle, $select)
@@ -711,9 +765,9 @@ 
 		    $as = '/\b'.preg_quote($nfrom[6]).'\s+(AS\s+\w+)\b/';
 		    if (preg_match($as,implode(',',$select),$m)){
 			    $alias = "";
+		    } else {
+		    					$alias = ", ".$nfrom[4]." AS $oldcle";
 		    }
-		    else
-					$alias = ", ".$nfrom[4]." AS $oldcle";
 	    }
 	    $select = remplacer_jointnul($t . $alias, $select, $e);
 	    $join = remplacer_jointnul($t, $join, $e);
@@ -738,17 +792,19 @@ 
  * @return string
  */
 function calculer_where_to_string($v, $join = 'AND'){
-	if (empty($v))
-		return '';
+	if (empty($v)) {
+			return '';
+	}
 
 	if (!is_array($v)) {
 		return $v;
 	} else {
 		$exp = "";
-		if (strtoupper($join) === 'AND')
-			return $exp . join(" $join ", array_map('calculer_where_to_string', $v));
-		else
-			return $exp . join($join, $v);
+		if (strtoupper($join) === 'AND') {
+					return $exp . join(" $join ", array_map('calculer_where_to_string', $v));
+		} else {
+					return $exp . join($join, $v);
+		}
 	}
 }
 
@@ -759,11 +815,15 @@ 
 function calculer_jointnul($cle, $exp, $equiv='')
 {
 	if (!is_array($exp)) {
-		if ($equiv) $exp = preg_replace($equiv, '', $exp);
+		if ($equiv) {
+		    $exp = preg_replace($equiv, '', $exp);
+		}
 		return preg_match("/\\b$cle\\./", $exp);
 	} else {
 		foreach($exp as $v) {
-			if (calculer_jointnul($cle, $v, $equiv)) return true;
+			if (calculer_jointnul($cle, $v, $equiv)) {
+			    return true;
+			}
 		}
 		return false;
 	}
@@ -776,7 +836,9 @@ 
 	  foreach($from as $k=>$v){
 	  	$from_synth[$k]=$from[$k];
 	  	if (isset($afrom[$k])) {
-	  		foreach($afrom[$k] as $kk=>$vv) $afrom[$k][$kk] = implode(' ',$afrom[$k][$kk]);
+	  		foreach($afrom[$k] as $kk=>$vv) {
+	  		    $afrom[$k][$kk] = implode(' ',$afrom[$k][$kk]);
+	  		}
 	  		$from_synth["$k@"]= implode(' ',$afrom[$k]);
 	  		unset($afrom[$k]);
 	  	}
@@ -802,8 +864,9 @@ 
 function calculer_nom_fonction_squel($skel, $mime_type='html', $connect='')
 {
 	// ne pas doublonner les squelette selon qu'ils sont calcules depuis ecrire/ ou depuis la racine
-	if ($l=strlen(_DIR_RACINE) AND strncmp($skel,_DIR_RACINE,$l)==0)
-		$skel = substr($skel,strlen(_DIR_RACINE));
+	if ($l=strlen(_DIR_RACINE) AND strncmp($skel,_DIR_RACINE,$l)==0) {
+			$skel = substr($skel,strlen(_DIR_RACINE));
+	}
 	return $mime_type
 	. (!$connect ?  '' : preg_replace('/\W/',"_", $connect)) . '_'
 	. md5($GLOBALS['spip_version_code'] . ' * ' . $skel . (isset($GLOBALS['marqueur_skel'])?'*'.$GLOBALS['marqueur_skel']:''));

ecrire/public/tracer.php

Braces +27 added lines, -13 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // http://doc.spip.org/@trace_query_start
 function trace_query_start()
@@ -27,11 +29,13 @@ 
 
 // http://doc.spip.org/@trace_query_end
 function trace_query_end($query, $start, $result, $erreur, $serveur=''){
-	if ($start)
-		trace_query_chrono($start, microtime(), $query, $result, $serveur);
+	if ($start) {
+			trace_query_chrono($start, microtime(), $query, $result, $serveur);
+	}
 	// tracer les erreurs, sauf pour select, c'est fait dans abstract_sql
-	if ($erreur AND !preg_match('/^select\b/i', $query))
-		erreur_squelette(array(sql_errno($serveur), $erreur, $query));
+	if ($erreur AND !preg_match('/^select\b/i', $query)) {
+			erreur_squelette(array(sql_errno($serveur), $erreur, $query));
+	}
 	return $result;
 }
 
@@ -46,12 +50,17 @@ 
 	$x = _request('var_mode_objet');
 	if (isset($GLOBALS['debug']['aucasou'])) {
 		list(, $boucle, $serveur, $contexte) = $GLOBALS['debug']['aucasou'];
-		if ($x AND !preg_match("/$boucle\$/", $x))
-			return;
-		if ($serveur) $boucle .= " ($serveur)";
+		if ($x AND !preg_match("/$boucle\$/", $x)) {
+					return;
+		}
+		if ($serveur) {
+		    $boucle .= " ($serveur)";
+		}
 		$boucle = "<b>$boucle</b>";
 	} else {
-		if ($x) return;
+		if ($x) {
+		    return;
+		}
 		$boucle = $contexte = '';
 	}
 
@@ -80,16 +89,21 @@ 
 			$k = ($contexte[0] . " $boucle");
 			include_spip('public/compiler');
 			$env = reconstruire_contexte_compil($contexte);
-		} else $k = $env = $boucle;
+		} else {
+		    $k = $env = $boucle;
+		}
 
 		$total += $dt;
 		$t[$key] = $dt;
 		$q[$key] = $nb;
 		$d[$k]+= $dt;
-		if ($k) @++$n[$k];
+		if ($k) {
+		    @++$n[$k];
+		}
 
-		if (!is_array($explain))
-			$explain = array();
+		if (!is_array($explain)) {
+					$explain = array();
+		}
 		foreach($explain as $j => $v) {
 			$explain[$j] = "<tr><th>$j</th><td>"
 			  . str_replace(';','<br />',$v)

ecrire/public/parametrer.php

Braces +63 added lines, -38 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/lang');
 
@@ -25,8 +27,9 @@ 
 	// chargement optimise precompile
 	include_once(_CACHE_PLUGINS_FCT);
 }
-if (test_espace_prive ())
+if (test_espace_prive ()) {
 	include_spip('inc/filtres_ecrire');
+}
 
 # Determine le squelette associe a une requete 
 # et l'applique sur le contexte, le nom du cache et le serveur
@@ -45,20 +48,26 @@ 
 function public_parametrer_dist($fond, $contexte='', $cache='', $connect='')  {
 	static $composer,$styliser,$notes=null;
 	$page = tester_redirection($fond, $contexte, $connect);
-	if ($page) return $page;
+	if ($page) {
+	    return $page;
+	}
 
-	if (isset($contexte['lang']))
-		$lang = $contexte['lang'];
-	elseif (!isset($lang))
-		$lang = $GLOBALS['meta']['langue_site'];
+	if (isset($contexte['lang'])) {
+			$lang = $contexte['lang'];
+	} elseif (!isset($lang)) {
+			$lang = $GLOBALS['meta']['langue_site'];
+	}
 
 	$select = ((!isset($GLOBALS['forcer_lang']) OR !$GLOBALS['forcer_lang']) AND $lang <> $GLOBALS['spip_lang']);
-	if ($select) $select = lang_select($lang);
+	if ($select) {
+	    $select = lang_select($lang);
+	}
 
 	$debug = (defined('_VAR_MODE') && _VAR_MODE == 'debug');
 
-	if (!$styliser)
-		$styliser = charger_fonction('styliser', 'public');
+	if (!$styliser) {
+			$styliser = charger_fonction('styliser', 'public');
+	}
 	list($skel,$mime_type, $gram, $sourcefile) =
 		$styliser($fond, $contexte, $GLOBALS['spip_lang'], $connect);
 
@@ -74,27 +83,29 @@ 
 		// charger le squelette en specifiant les langages cibles et source
 		// au cas il faudrait le compiler (source posterieure au resultat)
 
-		if (!$composer)
-			$composer = charger_fonction('composer', 'public');
+		if (!$composer) {
+					$composer = charger_fonction('composer', 'public');
+		}
 		$fonc = $composer($skel, $mime_type, $gram, $sourcefile, $connect);
+	} else {
+			$fonc = '';
 	}
-	else
-		$fonc = '';
 
 	if (!$fonc) { // squelette inconnu (==='') ou faux (===false)
 		$page = $fonc;
-	}
-	else {
+	} else {
 		// Preparer l'appel de la fonction principale du squelette 
 
 		spip_timer($a = 'calcul page '.rand(0,1000));
 
 		// On cree un marqueur de notes unique lie a cette composition
 		// et on enregistre l'etat courant des globales de notes...
-		if (is_null($notes))
-			$notes = charger_fonction('notes', 'inc', true);
-		if ($notes)
-			$notes('','empiler');
+		if (is_null($notes)) {
+					$notes = charger_fonction('notes', 'inc', true);
+		}
+		if ($notes) {
+					$notes('','empiler');
+		}
 
 		// Rajouter d'office ces deux parametres
 		// (mais vaudrait mieux que le compilateur sache le simuler
@@ -102,12 +113,16 @@ 
 		if (!isset($contexte['date'])) {
 			$contexte['date'] = date("Y-m-d H:i:s");
 			$contexte['date_default'] = true;
-		} else $contexte['date'] = normaliser_date($contexte['date'], true);
+		} else {
+		    $contexte['date'] = normaliser_date($contexte['date'], true);
+		}
 
 		if (!isset($contexte['date_redac'])) {
 			$contexte['date_redac'] = date("Y-m-d H:i:s");
 			$contexte['date_redac_default'] = true;
-		} else $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
+		} else {
+		    $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
+		}
 
 	// Passer le nom du cache pour produire sa destruction automatique
 		$page = $fonc(array('cache' => $cache), array($contexte));
@@ -117,25 +132,31 @@ 
 	// etre dans son resultat, autrement elles ne seraient pas prises en
 	// compte a chaque calcul d'un texte contenant un modele, mais seulement
 	// quand le modele serait calcule, et on aurait des resultats incoherents)
-		if ($notes)
-			$notes('','depiler');
+		if ($notes) {
+					$notes('','depiler');
+		}
 
 		// reinjecter en dynamique la pile des notes
 		// si il y a des inclure dynamiques
 		// si la pile n'est pas vide
 		// la generalisation de cette injection permettrait de corriger le point juste au dessus
 		// en faisant remonter les notes a l'incluant (A tester et valider avant application)
-		if ($notes)
-			$page['notes'] = $notes('','sauver_etat');
+		if ($notes) {
+					$page['notes'] = $notes('','sauver_etat');
+		}
 
 		// spip_log: un joli contexte
 		$infos = array();
 		foreach (array_filter($contexte) as $var => $val) {
-			if (is_array($val)) $val = serialize($val);
-			if (strlen("$val") > 30)
-				$val = substr("$val", 0,27).'..';
-			if (strstr($val,' '))
-				$val = "'$val'";
+			if (is_array($val)) {
+			    $val = serialize($val);
+			}
+			if (strlen("$val") > 30) {
+							$val = substr("$val", 0,27).'..';
+			}
+			if (strstr($val,' ')) {
+							$val = "'$val'";
+			}
 			$infos[] = $var.'='.$val;
 		}
 		$profile = spip_timer($a);
@@ -167,10 +188,11 @@ 
 			// si aucun #CACHE{} specifie
 			// le contexte implicite qui conditionne le cache assure qu'on retombe pas sur le meme
 			// entre public et prive
-			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0)
-				$page['entetes']['X-Spip-Cache'] = 0;
-			else
-				$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0) {
+							$page['entetes']['X-Spip-Cache'] = 0;
+			} else {
+							$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+			}
 		}
 
 		$page['contexte'] = $contexte;
@@ -192,7 +214,9 @@ 
 		}
 	}
 
-	if ($select) lang_select();
+	if ($select) {
+	    lang_select();
+	}
 
 	return $page;
 }
@@ -224,8 +248,9 @@ 
 				// etre utilisee dans un contexte d'url qui change
 				// y compris url arbo
 				$status = 302;
-				if (defined('_STATUS_REDIRECTION_VIRTUEL'))
-					$status=_STATUS_REDIRECTION_VIRTUEL;
+				if (defined('_STATUS_REDIRECTION_VIRTUEL')) {
+									$status=_STATUS_REDIRECTION_VIRTUEL;
+				}
 				if (!preg_match(',^\w+:,', $url)) {
 					include_spip('inc/filtres_mini');
 					$url = url_absolue($url);