spip / SPIP

ecrire/oo/index.php

Spacing +1 added lines, -1 removed lines

@@ -3,6 +3,6 @@
 // ACCESSIBILITE
 // la page /oo offre une lecture en mode "texte seul"
 
-header("Location: ../?action=preferer&arg=display:4&redirect=" . urlencode(str_replace('/oo','', $_SERVER['REQUEST_URI'])));
+header("Location: ../?action=preferer&arg=display:4&redirect=".urlencode(str_replace('/oo', '', $_SERVER['REQUEST_URI'])));
 
 ?>

ecrire/prive.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	// On reexecute pour deboucher sur le include public.
 	// autrement on insiste
 		if (is_array($var_auth)) {
-			$var_auth = '../?' . $_SERVER['QUERY_STRING'];
+			$var_auth = '../?'.$_SERVER['QUERY_STRING'];
 			spip_setcookie('spip_session', $_COOKIE['spip_session'], time() + 3600 * 24 * 14);
 		}
 		include_spip('inc/headers');

ecrire/inc/lang_liste.php

Spacing +5 added lines, -5 removed lines

@@ -50,7 +50,7 @@ 
 	'cpf_dom' => "Kreyòl",
 	'cpf_hat' => "Kreyòl (Peyi Dayiti)",
 	'cs' => "čeština",
-	'cy' => "Cymraeg",	# welsh, gallois
+	'cy' => "Cymraeg", # welsh, gallois
 	'da' => "dansk",
 	'de' => "Deutsch",
 	'dz' => "Bhutani",
@@ -91,7 +91,7 @@ 
 	'hi' => "हिंदी",
 	'hr' => "hrvatski",
 	'hu' => "magyar",
-	'hy' => "Հայերեն",// Arménien
+	'hy' => "Հայերեն", // Arménien
 	'ia' => "Interlingua",
 	'id' => "Indonesia",
 	'ie' => "Interlingue",
@@ -105,7 +105,7 @@ 
 	'ka' => "ქართული",
 	'kk' => "қазақ тілі", // Kazakh
 	'kl' => "kalaallisut",
-	'km' => "ភាសាខ្មែរ",// Khmer
+	'km' => "ភាសាខ្មែរ", // Khmer
 	'kn' => "Kannada",
 	'ko' => "한국어",
 	'kok' => "कोंकणी", // Konkani
@@ -175,8 +175,8 @@ 
 	'sh_latn' => 'srpskohrvastski',
 	'sh_cyrl' => 'Српскохрватски',
 	'si' => "Sinhalese",
-	'sk' => "slovenčina",	// (Slovakia)
-	'sl' => "slovenščina",	// (Slovenia)
+	'sk' => "slovenčina", // (Slovakia)
+	'sl' => "slovenščina", // (Slovenia)
 	'sm' => "Samoan",
 	'sn' => "Shona",
 	'so' => "Somali",

ecrire/inc/invalideur.php

Spacing +24 added lines, -24 removed lines

@@ -24,21 +24,21 @@ 
 // Donne le nombre de fichiers dans un repertoire (plat, pour aller vite)
 // false si erreur
 // http://doc.spip.org/@nombre_de_fichiers_repertoire
-function nombre_de_fichiers_repertoire($dir,$nb_estim_taille = 20) {
+function nombre_de_fichiers_repertoire($dir, $nb_estim_taille = 20) {
 	$taille = 0; // mesurer la taille de N fichiers au hasard dans le repertoire
 	$nb = $nb_estim_taille;
 	if (!$h = @opendir($dir)) return false;
 	$total = 0;
 	while (($fichier = @readdir($h)) !== false)
-		if ($fichier[0]!='.' AND !is_dir("$dir/$fichier")){
+		if ($fichier[0] != '.' AND !is_dir("$dir/$fichier")) {
 			$total++;
-			if ($nb AND rand(1,10)==1){
+			if ($nb AND rand(1, 10) == 1) {
 				$taille += filesize("$dir/$fichier");
 				$nb--;
 			}
 		}
 	closedir($h);
-	return array($total,$taille?$taille/($nb_estim_taille-$nb):_TAILLE_MOYENNE_FICHIER_CACHE);
+	return array($total, $taille ? $taille / ($nb_estim_taille - $nb) : _TAILLE_MOYENNE_FICHIER_CACHE);
 }
 
 // Indique la taille du repertoire cache ; pour de gros volumes,
@@ -47,10 +47,10 @@ 
 function taille_du_cache() {
 	$total = 0;
 	$taille = 0;
-	for ($i=0;$i<16;$i++) {
+	for ($i = 0; $i < 16; $i++) {
 		$l = dechex($i);
 		$dir = sous_repertoire(_DIR_CACHE, $l);
-		list($n,$s) = nombre_de_fichiers_repertoire($dir);
+		list($n, $s) = nombre_de_fichiers_repertoire($dir);
 		$total += $n;
 		$taille += $s;
 	}
@@ -62,7 +62,7 @@ 
 // la condition est generalement "id='objet/id_objet'"
 // ici on se contente de noter la date de mise a jour dans les metas
 // http://doc.spip.org/@suivre_invalideur
-function suivre_invalideur($cond, $modif=true) {
+function suivre_invalideur($cond, $modif = true) {
 	if (!$modif)
 		return;
 
@@ -108,7 +108,7 @@ 
  * @return int
  *     Nombre de fichiers supprimés
 **/
-function purger_repertoire($dir, $options=array()) {
+function purger_repertoire($dir, $options = array()) {
 	$handle = @opendir($dir);
 	if (!$handle) return;
 
@@ -119,14 +119,14 @@ 
 		if ($fichier[0] == '.') continue;
 		$chemin = "$dir/$fichier";
 		if (is_file($chemin)) {
-			if (  (!isset($options['atime']) OR (@fileatime($chemin) < $options['atime']))
+			if ((!isset($options['atime']) OR (@fileatime($chemin) < $options['atime']))
 				AND (!isset($options['mtime']) OR (@filemtime($chemin) < $options['mtime']))
 			  ) {
 				supprimer_fichier($chemin);
-				$total ++;
+				$total++;
 			}
 		}
-		else if (is_dir($chemin)){
+		else if (is_dir($chemin)) {
 			$opts = $options;
 			if (isset($otpions['limit']))
 				$otps['limit'] = $otpions['limit'] - $total;
@@ -135,7 +135,7 @@ 
 				spip_unlink($chemin);
 		}
 
-		if (isset($options['limit']) AND $total>=$options['limit'])
+		if (isset($options['limit']) AND $total >= $options['limit'])
 			break;
 	}
 	closedir($handle);
@@ -155,20 +155,20 @@ 
 	global $quota_cache;
 	$encore = false;
 
-	$tour_quota_cache = intval(1+$GLOBALS['meta']['tour_quota_cache'])%16;
+	$tour_quota_cache = intval(1 + $GLOBALS['meta']['tour_quota_cache']) % 16;
 	ecrire_meta('tour_quota_cache', $tour_quota_cache);
 
 	$l = dechex($tour_quota_cache);
 	$dir = sous_repertoire(_DIR_CACHE, $l);
-	list($nombre,$taille) = nombre_de_fichiers_repertoire($dir);
+	list($nombre, $taille) = nombre_de_fichiers_repertoire($dir);
 	$total_cache = $taille * $nombre;
 	spip_log("Taille du CACHE estimee ($l): "
-		.(intval(16*$total_cache/(1024*1024/10))/10)." Mo","invalideur");
+		.(intval(16 * $total_cache / (1024 * 1024 / 10)) / 10)." Mo", "invalideur");
 
 	// Nombre max de fichiers a supprimer
 	if ($quota_cache > 0
 	AND $taille > 0) {
-		$trop = $total_cache - ($quota_cache/16)*1024*1024;
+		$trop = $total_cache - ($quota_cache / 16) * 1024 * 1024;
 		$trop = 3 * intval($trop / $taille);
 		if ($trop > 0) {
 			$n = purger_repertoire($dir,
@@ -178,17 +178,17 @@ 
 					'subdir' => true // supprimer les vieux sous repertoire de session (avant [15851])
 				)
 			);
-			spip_log("$dir : $n/$trop caches supprimes [taille moyenne $taille]","invalideur");
-			$total_cache = intval(max(0,(16*$total_cache) - $n*$taille)/(1024*1024)*10)/10;
-			spip_log("cache restant estime : $total_cache Mo, ratio ".$total_cache/$quota_cache,"invalideur");
+			spip_log("$dir : $n/$trop caches supprimes [taille moyenne $taille]", "invalideur");
+			$total_cache = intval(max(0, (16 * $total_cache) - $n * $taille) / (1024 * 1024) * 10) / 10;
+			spip_log("cache restant estime : $total_cache Mo, ratio ".$total_cache / $quota_cache, "invalideur");
 
 			// redemander la main pour eviter que le cache ne gonfle trop
 			// mais pas si on ne peut pas purger car les fichiers sont trops recents
 			if (
-			  $total_cache/$quota_cache>1.5
-			  AND $n*50>$trop) {
+			  $total_cache / $quota_cache > 1.5
+			  AND $n * 50 > $trop) {
 				$encore = true;
-				spip_log("Il faut encore purger","invalideur");
+				spip_log("Il faut encore purger", "invalideur");
 			}
 		}
 	}
@@ -208,7 +208,7 @@ 
 	"|^([0-9a-f]/)?([0-9]+/)?[^.][\-_\%0-9a-z]+--[0-9a-f]+(\.gz)?$|i",
 	$cache)) {
 		// supprimer le fichier (de facon propre)
-		supprimer_fichier(_DIR_CACHE . $cache);
+		supprimer_fichier(_DIR_CACHE.$cache);
 	} else
 		spip_log("Nom de fichier cache incorrect : $cache");
 }
@@ -249,7 +249,7 @@ 
 
 // Calcul des pages : noter dans la base les liens d'invalidation
 // http://doc.spip.org/@maj_invalideurs
-function maj_invalideurs ($fichier, &$page) { }
+function maj_invalideurs($fichier, &$page) { }
 
 // les invalideurs sont de la forme "objet/id_objet"
 // http://doc.spip.org/@insere_invalideur

ecrire/inc/xml.php

Spacing +71 added lines, -71 removed lines

@@ -13,31 +13,31 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@spip_xml_load
-function spip_xml_load($fichier, $strict=true, $clean=true, $taille_max = 1048576, $datas='', $profondeur = -1){
+function spip_xml_load($fichier, $strict = true, $clean = true, $taille_max = 1048576, $datas = '', $profondeur = -1) {
 	$contenu = "";
-	if (preg_match(",^(http|ftp)://,",$fichier)){
+	if (preg_match(",^(http|ftp)://,", $fichier)) {
 		include_spip('inc/distant');
-		$contenu = recuperer_page($fichier,false,false,$taille_max, $datas);
+		$contenu = recuperer_page($fichier, false, false, $taille_max, $datas);
 	}
-	else lire_fichier ($fichier, $contenu);
+	else lire_fichier($fichier, $contenu);
 	$arbre = array();
 	if ($contenu)
 		$arbre = spip_xml_parse($contenu, $strict, $clean, $profondeur);
 		
-	return count($arbre)?$arbre:false;
+	return count($arbre) ? $arbre : false;
 }
 
 if (!defined('_SPIP_XML_TAG_SPLIT')) define('_SPIP_XML_TAG_SPLIT', "{<([^:>][^>]*?)>}sS");
 // http://doc.spip.org/@spip_xml_parse
-function spip_xml_parse(&$texte, $strict=true, $clean=true, $profondeur = -1){
+function spip_xml_parse(&$texte, $strict = true, $clean = true, $profondeur = -1) {
 	$out = array();
   // enlever les commentaires
   $charset = 'AUTO';
-  if ($clean===true){
-  	if (preg_match(",<\?xml\s(.*?)encoding=['\"]?(.*?)['\"]?(\s(.*))?\?>,im",$texte,$regs))
+  if ($clean === true) {
+  	if (preg_match(",<\?xml\s(.*?)encoding=['\"]?(.*?)['\"]?(\s(.*))?\?>,im", $texte, $regs))
   		$charset = $regs[2];
-	  $texte = preg_replace(',<!--(.*?)-->,is','',$texte);
-	  $texte = preg_replace(',<\?(.*?)\?>,is','',$texte);
+	  $texte = preg_replace(',<!--(.*?)-->,is', '', $texte);
+	  $texte = preg_replace(',<\?(.*?)\?>,is', '', $texte);
 		include_spip('inc/charsets');
 		$clean = $charset;
 		//$texte = importer_charset($texte,$charset);
@@ -46,128 +46,128 @@ 
   $txt = $texte;
 
 	// tant qu'il y a des tags
-	$chars = preg_split(_SPIP_XML_TAG_SPLIT,$txt,2,PREG_SPLIT_DELIM_CAPTURE);
-	while(count($chars)>=2){
+	$chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
+	while (count($chars) >= 2) {
 		// tag ouvrant
 		//$chars = preg_split("{<([^>]*?)>}s",$txt,2,PREG_SPLIT_DELIM_CAPTURE);
 	
 		// $before doit etre vide ou des espaces uniquements!
 		$before = trim($chars[0]);
 
-		if (strlen($before)>0)
-			return importer_charset($texte,$charset);//$texte; // before non vide, donc on est dans du texte
+		if (strlen($before) > 0)
+			return importer_charset($texte, $charset); //$texte; // before non vide, donc on est dans du texte
 	
 		$tag = rtrim($chars[1]);
 		$txt = $chars[2];
 		
-		if (strncmp($tag,'![CDATA[',8)==0) return importer_charset($texte,$charset);//$texte;
-		if(substr($tag,-1)=='/'){ // self closing tag
-			$tag = rtrim(substr($tag,0,strlen($tag)-1));
-			$out[$tag][]="";
+		if (strncmp($tag, '![CDATA[', 8) == 0) return importer_charset($texte, $charset); //$texte;
+		if (substr($tag, -1) == '/') { // self closing tag
+			$tag = rtrim(substr($tag, 0, strlen($tag) - 1));
+			$out[$tag][] = "";
 		}
-		else{
-			$closing_tag = preg_split(",\s|\t|\n|\r,",trim($tag));
-			$closing_tag=reset($closing_tag);
+		else {
+			$closing_tag = preg_split(",\s|\t|\n|\r,", trim($tag));
+			$closing_tag = reset($closing_tag);
 			// tag fermant
 			$ncclos = strlen("</$closing_tag>");
-			$p = strpos($txt,"</$closing_tag>");
-			if ($p!==FALSE  AND (strpos($txt,"<")<$p)){
-				$nclose =0; $nopen = 0;
+			$p = strpos($txt, "</$closing_tag>");
+			if ($p !== FALSE AND (strpos($txt, "<") < $p)) {
+				$nclose = 0; $nopen = 0;
 				$d = 0;
 				while (
-					$p!==FALSE
-					AND ($morceau = substr($txt,$d,$p-$d))
-					AND (($nopen+=preg_match_all("{<".preg_quote($closing_tag)."(\s*>|\s[^>]*[^/>]>)}is",$morceau,$matches,PREG_SET_ORDER))>$nclose)
-					){
+					$p !== FALSE
+					AND ($morceau = substr($txt, $d, $p - $d))
+					AND (($nopen += preg_match_all("{<".preg_quote($closing_tag)."(\s*>|\s[^>]*[^/>]>)}is", $morceau, $matches, PREG_SET_ORDER)) > $nclose)
+					) {
 					$nclose++;
-					$d=$p+$ncclos;
-					$p = strpos($txt,"</$closing_tag>",$d);
+					$d = $p + $ncclos;
+					$p = strpos($txt, "</$closing_tag>", $d);
 				}
 			}
-			if ($p===FALSE){
-				if ($strict){
-					$out[$tag][]="erreur : tag fermant $tag manquant::$txt"; 
+			if ($p === FALSE) {
+				if ($strict) {
+					$out[$tag][] = "erreur : tag fermant $tag manquant::$txt"; 
 					return $out;
 				}
-				else return importer_charset($texte,$charset);//$texte // un tag qui constitue du texte a reporter dans $before
+				else return importer_charset($texte, $charset); //$texte // un tag qui constitue du texte a reporter dans $before
 			}
-			$content = substr($txt,0,$p);
-			$txt = substr($txt,$p+$ncclos);
-			if ($profondeur==0 OR strpos($content,"<")===FALSE) // eviter une recursion si pas utile
-				$out[$tag][] = importer_charset($content,$charset);//$content;
+			$content = substr($txt, 0, $p);
+			$txt = substr($txt, $p + $ncclos);
+			if ($profondeur == 0 OR strpos($content, "<") === FALSE) // eviter une recursion si pas utile
+				$out[$tag][] = importer_charset($content, $charset); //$content;
 			else
-				$out[$tag][]=spip_xml_parse($content, $strict, $clean, $profondeur-1);
+				$out[$tag][] = spip_xml_parse($content, $strict, $clean, $profondeur - 1);
 		}
-		$chars = preg_split(_SPIP_XML_TAG_SPLIT,$txt,2,PREG_SPLIT_DELIM_CAPTURE);
+		$chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
 	}
-	if (count($out)&&(strlen(trim($txt))==0))
+	if (count($out) && (strlen(trim($txt)) == 0))
 		return $out;
 	else
-		return importer_charset($texte,$charset);//$texte;
+		return importer_charset($texte, $charset); //$texte;
 }
 
 // http://doc.spip.org/@spip_xml_aplatit
-function spip_xml_aplatit($arbre,$separateur = " "){
+function spip_xml_aplatit($arbre, $separateur = " ") {
 	$s = "";
 	if (is_array($arbre))
-		foreach($arbre as $tag=>$feuille){
-			if (is_array($feuille)){
-				if ($tag!==intval($tag)){
+		foreach ($arbre as $tag=>$feuille) {
+			if (is_array($feuille)) {
+				if ($tag !== intval($tag)) {
 					$f = spip_xml_aplatit($feuille, $separateur);
 					if (strlen($f)) {
-						$tagf = explode(" ",$tag);
+						$tagf = explode(" ", $tag);
 						$tagf = $tagf[0];
-						$s.="<$tag>$f</$tagf>";
+						$s .= "<$tag>$f</$tagf>";
 					}
-					else $s.="<$tag />";
+					else $s .= "<$tag />";
 				}
 				else
-					$s.=spip_xml_aplatit($feuille);
+					$s .= spip_xml_aplatit($feuille);
 				$s .= $separateur;
 			}
 			else
-				$s.="$feuille$separateur";
+				$s .= "$feuille$separateur";
 		}
 	return strlen($separateur) ? substr($s, 0, -strlen($separateur)) : $s;
 }
 
 // http://doc.spip.org/@spip_xml_tagname
-function spip_xml_tagname($tag){
-	if (preg_match(',^([a-z][\w:]*),i',$tag,$reg))
+function spip_xml_tagname($tag) {
+	if (preg_match(',^([a-z][\w:]*),i', $tag, $reg))
 		return $reg[1];
 	return "";
 }
 // http://doc.spip.org/@spip_xml_decompose_tag
-function spip_xml_decompose_tag($tag){
+function spip_xml_decompose_tag($tag) {
 	$tagname = spip_xml_tagname($tag);
 	$liste = array();
-	$p=strpos($tag,' ');
-	$tag = substr($tag,$p);
-	$p=strpos($tag,'=');
-	while($p!==false){
-		$attr = trim(substr($tag,0,$p));
-		$tag = ltrim(substr($tag,$p+1));
+	$p = strpos($tag, ' ');
+	$tag = substr($tag, $p);
+	$p = strpos($tag, '=');
+	while ($p !== false) {
+		$attr = trim(substr($tag, 0, $p));
+		$tag = ltrim(substr($tag, $p + 1));
 		$quote = $tag{0};
-		$p=strpos($tag,$quote,1);
-		$cont = substr($tag,1,$p-1);
+		$p = strpos($tag, $quote, 1);
+		$cont = substr($tag, 1, $p - 1);
 		$liste[$attr] = $cont;
-		$tag = substr($tag,$p+1);
-		$p=strpos($tag,'=');
+		$tag = substr($tag, $p + 1);
+		$p = strpos($tag, '=');
 	}
-	return array($tagname,$liste);
+	return array($tagname, $liste);
 }
 
 // http://doc.spip.org/@spip_xml_match_nodes
-function spip_xml_match_nodes($regexp,&$arbre,&$matches,$init=true){
+function spip_xml_match_nodes($regexp, &$arbre, &$matches, $init = true) {
 	if ($init)
 		$matches = array();
-	if(is_array($arbre) && count($arbre))
-		foreach(array_keys($arbre) as $tag){
-			if (preg_match($regexp,$tag))
+	if (is_array($arbre) && count($arbre))
+		foreach (array_keys($arbre) as $tag) {
+			if (preg_match($regexp, $tag))
 				$matches[$tag] = &$arbre[$tag];
 			if (is_array($arbre[$tag]))
-				foreach(array_keys($arbre[$tag]) as $occurences)
-					spip_xml_match_nodes($regexp,$arbre[$tag][$occurences],$matches,false);
+				foreach (array_keys($arbre[$tag]) as $occurences)
+					spip_xml_match_nodes($regexp, $arbre[$tag][$occurences], $matches, false);
 		}
 	return (count($matches));
 }

ecrire/inc/texte_mini.php

Spacing +64 added lines, -64 removed lines

@@ -25,7 +25,7 @@ 
 	// celle du texte) et public (spip_lang est la langue du texte)
 	$dir = _DIR_RESTREINT ? lang_dir() : lang_dir($GLOBALS['spip_lang']);
 
-	$p = 'puce' . (test_espace_prive() ? '_prive' : '');
+	$p = 'puce'.(test_espace_prive() ? '_prive' : '');
 	if ($dir == 'rtl') $p .= '_rtl';
 
 	if (!isset($GLOBALS[$p])) {
@@ -48,7 +48,7 @@ 
 	.'d[ltd]|script|noscript|map|button|fieldset|style');
 
 if (!defined('_BALISES_BLOCS_REGEXP'))
-	define('_BALISES_BLOCS_REGEXP',',</?('._BALISES_BLOCS.')[>[:space:]],iS');
+	define('_BALISES_BLOCS_REGEXP', ',</?('._BALISES_BLOCS.')[>[:space:]],iS');
 
 //
 // Echapper les elements perilleux en les passant en base64
@@ -58,17 +58,17 @@ 
 // une $source differente ; le script detecte automagiquement si ce qu'on
 // echappe est un div ou un span
 // http://doc.spip.org/@code_echappement
-function code_echappement($rempl, $source='', $no_transform=false, $mode=NULL) {
+function code_echappement($rempl, $source = '', $no_transform = false, $mode = NULL) {
 	if (!strlen($rempl)) return '';
 
 	// Tester si on echappe en span ou en div
-	if (is_null($mode) OR !in_array($mode,array('div','span')))
+	if (is_null($mode) OR !in_array($mode, array('div', 'span')))
 		$mode = preg_match(',</?('._BALISES_BLOCS.')[>[:space:]],iS', $rempl) ? 'div' : 'span';
 
 	// Decouper en morceaux, base64 a des probleme selon la taille de la pile
 	$taille = 30000;
 	$return = "";
-	for($i = 0; $i < strlen($rempl); $i += $taille) {
+	for ($i = 0; $i < strlen($rempl); $i += $taille) {
 		// Convertir en base64 et cacher dans un attribut
 		// utiliser les " pour eviter le re-encodage de ' et &#8217
 		$base64 = base64_encode(substr($rempl, $i, $taille));
@@ -89,11 +89,11 @@ 
 // Echapper les <code>...</ code>
 // http://doc.spip.org/@traiter_echap_code_dist
 function traiter_echap_code_dist($regs) {
-	list(,,$att,$corps) = $regs;
+	list(,, $att, $corps) = $regs;
 	$echap = spip_htmlspecialchars($corps); // il ne faut pas passer dans entites_html, ne pas transformer les &#xxx; du code !
 
 	// ne pas mettre le <div...> s'il n'y a qu'une ligne
-	if (is_int(strpos($echap,"\n"))) {
+	if (is_int(strpos($echap, "\n"))) {
 		// supprimer les sauts de ligne debut/fin
 		// (mais pas les espaces => ascii art).
 		$echap = preg_replace("/^[\n\r]+|[\n\r]+$/s", "", $echap);
@@ -115,11 +115,11 @@ 
 function traiter_echap_cadre_dist($regs) {
 	$echap = trim(entites_html($regs[3]));
 	// compter les lignes un peu plus finement qu'avec les \n
-	$lignes = explode("\n",trim($echap));
+	$lignes = explode("\n", trim($echap));
 	$n = 0;
-	foreach($lignes as $l)
-		$n+=floor(strlen($l)/60)+1;
-	$n = max($n,2);
+	foreach ($lignes as $l)
+		$n += floor(strlen($l) / 60) + 1;
+	$n = max($n, 2);
 	$echap = "\n<textarea readonly='readonly' cols='40' rows='$n' class='spip_cadre' dir='ltr'>$echap</textarea>";
 	return $echap;
 }
@@ -132,7 +132,7 @@ 
 function traiter_echap_script_dist($regs) {
 	// rendre joli (et inactif) si c'est un script language=php
 	if (preg_match(',<script\b[^>]+php,ims', $regs[0]))
-		return highlight_string($regs[0],true);
+		return highlight_string($regs[0], true);
 
 	// Cas normal : le script passe tel quel
 	return $regs[0];
@@ -157,8 +157,8 @@ 
 // - pour $source voir commentaire infra (echappe_retour)
 // - pour $no_transform voir le filtre post_autobr dans inc/filtres
 // http://doc.spip.org/@echappe_html
-function echappe_html($letexte, $source='', $no_transform=false,
-$preg='') {
+function echappe_html($letexte, $source = '', $no_transform = false,
+$preg = '') {
 	if (!is_string($letexte) or !strlen($letexte))
 		return $letexte;
 
@@ -172,7 +172,7 @@ 
 		}
 	}
 
-	if (($preg OR strpos($letexte,"<")!==false)
+	if (($preg OR strpos($letexte, "<") !== false)
 	  AND preg_match_all($preg ? $preg : _PROTEGE_BLOCS, $letexte, $matches, PREG_SET_ORDER)) {
 		foreach ($matches as $regs) {
 			// echappements tels quels ?
@@ -186,8 +186,8 @@ 
 			else if (function_exists($f = $f.'_dist'))
 				$echap = $f($regs);
 
-			$p = strpos($letexte,$regs[0]);
-			$letexte = substr_replace($letexte,code_echappement($echap, $source, $no_transform),$p,strlen($regs[0]));
+			$p = strpos($letexte, $regs[0]);
+			$letexte = substr_replace($letexte, code_echappement($echap, $source, $no_transform), $p, strlen($regs[0]));
 		}
 	}
 
@@ -198,7 +198,7 @@ 
 	// code mort sauf si on a personalise _PROTEGE_BLOCS sans y mettre <math>
 	// eviter la rupture de compat en branche 3.0
 	// a supprimer en branche 3.1
-	if (strpos($preg ? $preg : _PROTEGE_BLOCS,'code')!==false){
+	if (strpos($preg ? $preg : _PROTEGE_BLOCS, 'code') !== false) {
 		if (strpos($letexte, "<math>") !== false) {
 			include_spip('inc/math');
 			$letexte = traiter_math($letexte, $source);
@@ -209,12 +209,12 @@ 
 	// seulement si on a echappe les <script>
 	// (derogatoire car on ne peut pas faire passer < ? ... ? >
 	// dans une callback autonommee
-	if (strpos($preg ? $preg : _PROTEGE_BLOCS,'script')!==false){
-		if (strpos($letexte,"<"."?")!==false AND preg_match_all(',<[?].*($|[?]>),UisS',
+	if (strpos($preg ? $preg : _PROTEGE_BLOCS, 'script') !== false) {
+		if (strpos($letexte, "<"."?") !== false AND preg_match_all(',<[?].*($|[?]>),UisS',
 		$letexte, $matches, PREG_SET_ORDER))
 		foreach ($matches as $regs) {
 			$letexte = str_replace($regs[0],
-				code_echappement(highlight_string($regs[0],true), $source),
+				code_echappement(highlight_string($regs[0], true), $source),
 				$letexte);
 		}
 	}
@@ -227,11 +227,11 @@ 
 // Rq: $source sert a faire des echappements "a soi" qui ne sont pas nettoyes
 // par propre() : exemple dans multi et dans typo()
 // http://doc.spip.org/@echappe_retour
-function echappe_retour($letexte, $source='', $filtre = "") {
-	if (strpos($letexte,"base64$source")) {
+function echappe_retour($letexte, $source = '', $filtre = "") {
+	if (strpos($letexte, "base64$source")) {
 		# spip_log(spip_htmlspecialchars($letexte));  ## pour les curieux
 		$max_prof = 5;
-		while (strpos($letexte,"<")!==false
+		while (strpos($letexte, "<") !== false
 			AND
 		  preg_match_all(',<(span|div)\sclass=[\'"]base64'.$source.'[\'"]\s(.*)>\s*</\1>,UmsS',
 		$letexte, $regs, PREG_SET_ORDER)
@@ -240,13 +240,13 @@ 
 				$rempl = base64_decode(extraire_attribut($reg[0], 'title'));
 				// recherche d'attributs supplementaires
 				$at = array();
-				foreach(array('lang', 'dir') as $attr) {
+				foreach (array('lang', 'dir') as $attr) {
 					if ($a = extraire_attribut($reg[0], $attr))
 						$at[$attr] = $a;
 				}
 				if ($at) {
 					$rempl = '<'.$reg[1].'>'.$rempl.'</'.$reg[1].'>';
-					foreach($at as $attr => $a)
+					foreach ($at as $attr => $a)
 						$rempl = inserer_attribut($rempl, $attr, $a);
 				}
 				if ($filtre) $rempl = $filtre($rempl);
@@ -260,7 +260,7 @@ 
 // Reinserer le javascript de confiance (venant des modeles)
 
 // http://doc.spip.org/@echappe_retour_modeles
-function echappe_retour_modeles($letexte, $interdire_scripts=false)
+function echappe_retour_modeles($letexte, $interdire_scripts = false)
 {
 	$letexte = echappe_retour($letexte);
 
@@ -273,17 +273,17 @@ 
 
 
 // http://doc.spip.org/@couper
-function couper($texte, $taille=50, $suite = '&nbsp;(...)') {
-	if (!($length=strlen($texte)) OR $taille <= 0) return '';
-	$offset = 400 + 2*$taille;
-	while ($offset<$length
-		AND strlen(preg_replace(",<[^>]+>,Uims","",substr($texte,0,$offset)))<$taille)
-		$offset = 2*$offset;
-	if (	$offset<$length
-			&& ($p_tag_ouvrant = strpos($texte,'<',$offset))!==NULL){
-		$p_tag_fermant = strpos($texte,'>',$offset);
-		if ($p_tag_fermant && ($p_tag_fermant<$p_tag_ouvrant))
-			$offset = $p_tag_fermant+1; // prolonger la coupe jusqu'au tag fermant suivant eventuel
+function couper($texte, $taille = 50, $suite = '&nbsp;(...)') {
+	if (!($length = strlen($texte)) OR $taille <= 0) return '';
+	$offset = 400 + 2 * $taille;
+	while ($offset < $length
+		AND strlen(preg_replace(",<[^>]+>,Uims", "", substr($texte, 0, $offset))) < $taille)
+		$offset = 2 * $offset;
+	if ($offset < $length
+			&& ($p_tag_ouvrant = strpos($texte, '<', $offset)) !== NULL) {
+		$p_tag_fermant = strpos($texte, '>', $offset);
+		if ($p_tag_fermant && ($p_tag_fermant < $p_tag_ouvrant))
+			$offset = $p_tag_fermant + 1; // prolonger la coupe jusqu'au tag fermant suivant eventuel
 	}
 	$texte = substr($texte, 0, $offset); /* eviter de travailler sur 10ko pour extraire 150 caracteres */
 
@@ -300,8 +300,8 @@ 
 
 	// supprimer les tags
 	$texte = supprimer_tags($texte);
-	$texte = trim(str_replace("\n"," ", $texte));
-	$texte .= "\n";	// marquer la fin
+	$texte = trim(str_replace("\n", " ", $texte));
+	$texte .= "\n"; // marquer la fin
 
 	// travailler en accents charset
 	$texte = unicode2charset(html2unicode($texte, /* secure */ true));
@@ -311,22 +311,22 @@ 
 
 	// corriger la longueur de coupe
 	// en fonction de la presence de caracteres utf
-	if ($GLOBALS['meta']['charset']=='utf-8'){
+	if ($GLOBALS['meta']['charset'] == 'utf-8') {
 		$long = charset2unicode($texte);
-		$long = spip_substr($long, 0, max($taille,1));
+		$long = spip_substr($long, 0, max($taille, 1));
 		$nbcharutf = preg_match_all('/(&#[0-9]{3,5};)/S', $long, $matches);
 		$taille += $nbcharutf;
 	}
 
 
 	// couper au mot precedent
-	$long = spip_substr($texte, 0, max($taille-4,1));
+	$long = spip_substr($texte, 0, max($taille - 4, 1));
 	$u = $GLOBALS['meta']['pcre_u'];
 	$court = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
 	$points = $suite;
 
 	// trop court ? ne pas faire de (...)
-	if (spip_strlen($court) < max(0.75 * $taille,2)) {
+	if (spip_strlen($court) < max(0.75 * $taille, 2)) {
 		$points = '';
 		$long = spip_substr($texte, 0, $taille);
 		$texte = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
@@ -351,37 +351,37 @@ 
 
 // http://doc.spip.org/@protege_js_modeles
 function protege_js_modeles($t) {
-	if (isset($GLOBALS['visiteur_session'])){
-		if (preg_match_all(',<script.*?($|</script.),isS', $t, $r, PREG_SET_ORDER)){
-			if (!defined('_PROTEGE_JS_MODELES')){
+	if (isset($GLOBALS['visiteur_session'])) {
+		if (preg_match_all(',<script.*?($|</script.),isS', $t, $r, PREG_SET_ORDER)) {
+			if (!defined('_PROTEGE_JS_MODELES')) {
 				include_spip('inc/acces');
-				define('_PROTEGE_JS_MODELES',creer_uniqid());
+				define('_PROTEGE_JS_MODELES', creer_uniqid());
 			}
 			foreach ($r as $regs)
-				$t = str_replace($regs[0],code_echappement($regs[0],'javascript'._PROTEGE_JS_MODELES),$t);
+				$t = str_replace($regs[0], code_echappement($regs[0], 'javascript'._PROTEGE_JS_MODELES), $t);
 		}
-		if (preg_match_all(',<\?php.*?($|\?'.'>),isS', $t, $r, PREG_SET_ORDER)){
-			if (!defined('_PROTEGE_PHP_MODELES')){
+		if (preg_match_all(',<\?php.*?($|\?'.'>),isS', $t, $r, PREG_SET_ORDER)) {
+			if (!defined('_PROTEGE_PHP_MODELES')) {
 				include_spip('inc/acces');
-				define('_PROTEGE_PHP_MODELES',creer_uniqid());
+				define('_PROTEGE_PHP_MODELES', creer_uniqid());
 			}
 			foreach ($r as $regs)
-				$t = str_replace($regs[0],code_echappement($regs[0],'php'._PROTEGE_PHP_MODELES),$t);
+				$t = str_replace($regs[0], code_echappement($regs[0], 'php'._PROTEGE_PHP_MODELES), $t);
 		}
 	}
 	return $t;
 }
 
 
-function echapper_faux_tags($letexte){
-	if (strpos($letexte,'<')===false)
+function echapper_faux_tags($letexte) {
+	if (strpos($letexte, '<') === false)
 		return $letexte;
-  $textMatches = preg_split (',(</?[a-z!][^<>]*>),', $letexte, null, PREG_SPLIT_DELIM_CAPTURE);
+  $textMatches = preg_split(',(</?[a-z!][^<>]*>),', $letexte, null, PREG_SPLIT_DELIM_CAPTURE);
 
   $letexte = "";
   while (count($textMatches)) {
   	// un texte a echapper
-  	$letexte .= str_replace("<",'&lt;',array_shift($textMatches));
+  	$letexte .= str_replace("<", '&lt;', array_shift($textMatches));
   	// un tag html qui a servit a faite le split
  		$letexte .= array_shift($textMatches);
   }
@@ -396,15 +396,15 @@ 
  * @param string $texte
  * @return string
  */
-function echapper_html_suspect($texte){
-	if (strpos($texte,'<')===false OR strpos($texte,'=')===false)
+function echapper_html_suspect($texte) {
+	if (strpos($texte, '<') === false OR strpos($texte, '=') === false)
 		return $texte;
 
 	// on teste sur strlen car safehtml supprime le contenu dangereux
 	// mais il peut aussi changer des ' en " sur les attributs html,
 	// donc un test d'egalite est trop strict
-	if (strlen(safehtml($texte))!==strlen($texte)){
-		$texte = str_replace("<","&lt;",$texte);
+	if (strlen(safehtml($texte)) !== strlen($texte)) {
+		$texte = str_replace("<", "&lt;", $texte);
 	}
 
 	return $texte;
@@ -433,7 +433,7 @@ 
 	if (!$t OR !is_string($t))
 		return $t;
 	# attention safehtml nettoie deux ou trois caracteres de plus. A voir
-	if (strpos($t,'<')===false)
+	if (strpos($t, '<') === false)
 		return str_replace("\x00", '', $t);
 
 	$t = interdire_scripts($t); // jolifier le php
@@ -453,8 +453,8 @@ 
 // Sert aussi a nettoyer un texte qu'on veut mettre dans un <a> etc.
 // TODO: gerer les modeles ?
 // http://doc.spip.org/@supprime_img
-function supprime_img($letexte, $message=NULL) {
-	if ($message===NULL) $message = '(' . _T('img_indisponible') . ')';
+function supprime_img($letexte, $message = NULL) {
+	if ($message === NULL) $message = '('._T('img_indisponible').')';
 	return preg_replace(',<(img|doc|emb)([0-9]+)(\|([^>]*))?'.'\s*/?'.'>,i',
 		$message, $letexte);
 }

ecrire/inc/precharger_objet.php

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
  *                             dans la declaration de l'objet
  * @return array couples cles / valeurs des champs du formulaire à charger.
 **/
-function precharger_objet($type, $id_objet, $id_rubrique=0, $lier_trad=0, $champ_titre = 'titre') {
+function precharger_objet($type, $id_objet, $id_rubrique = 0, $lier_trad = 0, $champ_titre = 'titre') {
 	global $connect_id_rubrique, $spip_lang;
 	
 	$table = table_objet_sql($type);
@@ -49,8 +49,8 @@ 
 
 	// si demande de traduction
 	// on recupere les valeurs de la traduction
-	if ($lier_trad){
-		if ($select = charger_fonction("precharger_traduction_" . $type,'inc',true))
+	if ($lier_trad) {
+		if ($select = charger_fonction("precharger_traduction_".$type, 'inc', true))
 			$row = $select($id_objet, $id_rubrique, $lier_trad);
 		else
 			$row = precharger_traduction_objet($type, $id_objet, $id_rubrique, $lier_trad, $champ_titre);
@@ -74,10 +74,10 @@ 
 				$row_rub = sql_fetsel("id_rubrique", "spip_rubriques", "", "", "id_rubrique DESC", 1);
 				$row['id_rubrique'] = $id_rubrique = $row_rub['id_rubrique'];
 			}
-			if (!autoriser('creerarticledans','rubrique',$row['id_rubrique'] )){
+			if (!autoriser('creerarticledans', 'rubrique', $row['id_rubrique'])) {
 				// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
 				$res = sql_select("id_rubrique", "spip_rubriques", "id_parent=0");
-				while (!autoriser('creerarticledans','rubrique',$row['id_rubrique'] ) && $row_rub = sql_fetch($res)){
+				while (!autoriser('creerarticledans', 'rubrique', $row['id_rubrique']) && $row_rub = sql_fetch($res)) {
 					$row['id_rubrique'] = $row_rub['id_rubrique'];
 				}
 			}
@@ -87,7 +87,7 @@ 
 	// recuperer le secteur, pour affecter les bons champs extras
 	if ($id_rubrique and $is_secteur) {
 		if (!$row['id_secteur']) {
-			$row_rub = sql_getfetsel("id_secteur", "spip_rubriques", "id_rubrique=" . sql_quote($id_rubrique));
+			$row_rub = sql_getfetsel("id_secteur", "spip_rubriques", "id_rubrique=".sql_quote($id_rubrique));
 			$row['id_secteur'] = $row_rub;
 		}
 	}
@@ -108,7 +108,7 @@ 
  * 
  * @return array couples cles / valeurs des champs du formulaire à charger
 **/
-function precharger_traduction_objet($type, $id_objet, $id_rubrique=0, $lier_trad=0, $champ_titre = 'titre') {
+function precharger_traduction_objet($type, $id_objet, $id_rubrique = 0, $lier_trad = 0, $champ_titre = 'titre') {
 	$table = table_objet_sql($type);
 	$_id_objet = id_table_objet($table);
 
@@ -140,7 +140,7 @@ 
 			// Si le menu de langues est autorise sur l'objet,
 			// on peut changer la langue quelle que soit la rubrique
 			// donc on reste dans la meme rubrique
-			if (in_array($table, explode(',',$GLOBALS['meta']['multi_objets']))) {
+			if (in_array($table, explode(',', $GLOBALS['meta']['multi_objets']))) {
 				$row['id_rubrique'] = $row['id_rubrique']; # explicite :-)
 
 			// Sinon, chercher la rubrique la plus adaptee pour

ecrire/inc/cvt_configurer.php

Spacing +48 added lines, -48 removed lines

@@ -20,22 +20,22 @@ 
  * @param array $flux
  * @return array
  */
-function cvtconf_formulaire_charger($flux){
+function cvtconf_formulaire_charger($flux) {
 	if (
 		$form = $flux['args']['form']
-		and strncmp($form,'configurer_',11)==0 // un #FORMULAIRE_CONFIGURER_XXX
+		and strncmp($form, 'configurer_', 11) == 0 // un #FORMULAIRE_CONFIGURER_XXX
 	) {
 		// Pour tous les formulaires CONFIGURER, ayant une fonction charger ou pas, on teste si autorisé
 		include_spip('inc/autoriser');
-		if (!autoriser('configurer', '_'.substr($form,11))) {
+		if (!autoriser('configurer', '_'.substr($form, 11))) {
 			return false;
 		}
 		
 		// S'il n'y a pas de fonction charger(), on génère un contexte automatiquement
-		if (!charger_fonction("charger","formulaires/$form/",true)) {
+		if (!charger_fonction("charger", "formulaires/$form/", true)) {
 			$flux['data'] = cvtconf_formulaires_configurer_recense($form);
 			$flux['data']['editable'] = true;
-			if (_request('var_mode')=='configurer' AND autoriser('webmestre')){
+			if (_request('var_mode') == 'configurer' AND autoriser('webmestre')) {
 				if (!_AJAX) var_dump($flux['data']);
 				// reinjecter pour la trace au traitement
 				$flux['data']['_hidden'] = "<input type='hidden' name='var_mode' value='configurer' />";
@@ -51,13 +51,13 @@ 
  * @param array $flux
  * @return array
  */
-function cvtconf_formulaire_traiter($flux){
+function cvtconf_formulaire_traiter($flux) {
 	if ($form = $flux['args']['form']
-	  AND strncmp($form,'configurer_',11)==0 // un #FORMULAIRE_CONFIGURER_XXX
-		AND !charger_fonction("traiter","formulaires/$form/",true) // sans fonction traiter()
+	  AND strncmp($form, 'configurer_', 11) == 0 // un #FORMULAIRE_CONFIGURER_XXX
+		AND !charger_fonction("traiter", "formulaires/$form/", true) // sans fonction traiter()
 		) {
-		$trace = cvtconf_formulaires_configurer_enregistre($form,$flux['args']['args']);
-		$flux['data'] = array('message_ok'=>_T('config_info_enregistree').$trace,'editable'=>true);
+		$trace = cvtconf_formulaires_configurer_enregistre($form, $flux['args']['args']);
+		$flux['data'] = array('message_ok'=>_T('config_info_enregistree').$trace, 'editable'=>true);
 	}
 	return $flux;
 }
@@ -74,17 +74,17 @@ 
  *   arguments de l'appel de la fonction traiter ($args = func_get_args();)
  * @return string
  */
-function cvtconf_formulaires_configurer_enregistre($form,$args){
+function cvtconf_formulaires_configurer_enregistre($form, $args) {
 		$valeurs = array();
 		// charger les valeurs
 		// ce qui permet de prendre en charge une fonction charger() existante
 		// qui prend alors la main sur l'auto detection
-		if ($charger_valeurs = charger_fonction("charger","formulaires/$form/",true))
-			$valeurs = call_user_func_array($charger_valeurs,$args);
+		if ($charger_valeurs = charger_fonction("charger", "formulaires/$form/", true))
+			$valeurs = call_user_func_array($charger_valeurs, $args);
 		$valeurs = pipeline(
 			'formulaire_charger',
 			array(
-				'args'=>array('form'=>$form,'args'=>$args,'je_suis_poste'=>false),
+				'args'=>array('form'=>$form, 'args'=>$args, 'je_suis_poste'=>false),
 				'data'=>$valeurs)
 		);
 		// ne pas stocker editable !
@@ -92,12 +92,12 @@ 
 
 		// recuperer les valeurs postees
 		$store = array();
-		foreach($valeurs as $k=>$v){
-			if (substr($k,0,1)!=='_')
+		foreach ($valeurs as $k=>$v) {
+			if (substr($k, 0, 1) !== '_')
 				$store[$k] = _request($k);
 		}
 
-		return cvtconf_configurer_stocker($form,$valeurs,$store);
+		return cvtconf_configurer_stocker($form, $valeurs, $store);
 }
 
 /**
@@ -112,26 +112,26 @@ 
  * @param array $valeurs
  * @return array
  */
-function cvtconf_definir_configurer_conteneur($form,$valeurs) {
+function cvtconf_definir_configurer_conteneur($form, $valeurs) {
 		// stocker en base
 		// par defaut, dans un casier serialize dans spip_meta (idem CFG)
-		$casier = substr($form,11);
+		$casier = substr($form, 11);
 		$table = 'meta';
 		$prefixe = '';
 		$stockage = '';
 
 		if (isset($valeurs['_meta_casier']))   $casier   = $valeurs['_meta_casier'];
 		if (isset($valeurs['_meta_prefixe']))  $prefixe  = $valeurs['_meta_prefixe'];
-		if (isset($valeurs['_meta_stockage'])) $stockage = $valeurs['_meta_stockage'] . '::';
+		if (isset($valeurs['_meta_stockage'])) $stockage = $valeurs['_meta_stockage'].'::';
 			
 		// si on indique juste une table, il faut vider les autres proprietes
 		// car par defaut on utilise ni casier ni prefixe dans ce cas
 		if (isset($valeurs['_meta_table'])) {
 			$table = $valeurs['_meta_table'];
-			$casier = (isset($valeurs['_meta_casier'])?$valeurs['_meta_casier']:'');
+			$casier = (isset($valeurs['_meta_casier']) ? $valeurs['_meta_casier'] : '');
 		}
 	
-		return array($table,$casier,$prefixe,$stockage);
+		return array($table, $casier, $prefixe, $stockage);
 }
 
 /**
@@ -141,39 +141,39 @@ 
  * @param string $form
  * @return array
  */
-function cvtconf_formulaires_configurer_recense($form){
+function cvtconf_formulaires_configurer_recense($form) {
 	$valeurs = array('editable'=>' ');
 
 	// sinon cas analyse du squelette
-	if ($f = find_in_path($form.'.' . _EXTENSION_SQUELETTES, 'formulaires/')
+	if ($f = find_in_path($form.'.'._EXTENSION_SQUELETTES, 'formulaires/')
 		AND lire_fichier($f, $contenu)) {
 
-		for ($i=0;$i<2;$i++) {
+		for ($i = 0; $i < 2; $i++) {
 			// a la seconde iteration, evaluer le fond avec les valeurs deja trouvees
 			// permet de trouver aussi les name="#GET{truc}"
-			if ($i==1) $contenu = recuperer_fond("formulaires/$form",$valeurs);
+			if ($i == 1) $contenu = recuperer_fond("formulaires/$form", $valeurs);
 
-			$balises = array_merge(extraire_balises($contenu,'input'),
-				extraire_balises($contenu,'textarea'),
-				extraire_balises($contenu,'select'));
+			$balises = array_merge(extraire_balises($contenu, 'input'),
+				extraire_balises($contenu, 'textarea'),
+				extraire_balises($contenu, 'select'));
 
-			foreach($balises as $b) {
+			foreach ($balises as $b) {
 				if ($n = extraire_attribut($b, 'name')
-					AND preg_match(",^([\w\-]+)(\[\w*\])?$,",$n,$r)
-					AND !in_array($n,array('formulaire_action','formulaire_action_args'))
-					AND extraire_attribut($b,'type')!=='submit') {
+					AND preg_match(",^([\w\-]+)(\[\w*\])?$,", $n, $r)
+					AND !in_array($n, array('formulaire_action', 'formulaire_action_args'))
+					AND extraire_attribut($b, 'type') !== 'submit') {
 						$valeurs[$r[1]] = '';
 						// recuperer les valeurs _meta_xx qui peuvent etre fournies
 						// en input hidden dans le squelette
-						if (strncmp($r[1],'_meta_',6)==0)
-							$valeurs[$r[1]] = extraire_attribut($b,'value');
+						if (strncmp($r[1], '_meta_', 6) == 0)
+							$valeurs[$r[1]] = extraire_attribut($b, 'value');
 					}
 			}
 		}
 	}
 
 
-	cvtconf_configurer_lire_meta($form,$valeurs);
+	cvtconf_configurer_lire_meta($form, $valeurs);
 	return $valeurs;
 }
 
@@ -184,21 +184,21 @@ 
  * @param array $store
  * @return string
  */
-function cvtconf_configurer_stocker($form,$valeurs,$store) {
+function cvtconf_configurer_stocker($form, $valeurs, $store) {
 	$trace = '';
-	list($table,$casier,$prefixe,$stockage) = cvtconf_definir_configurer_conteneur($form,$valeurs);
+	list($table, $casier, $prefixe, $stockage) = cvtconf_definir_configurer_conteneur($form, $valeurs);
 	// stocker en base
 	// par defaut, dans un casier serialize dans spip_meta (idem CFG)
 	if (!isset($GLOBALS[$table]))
 		lire_metas($table);
 
-	$prefixe = ($prefixe?$prefixe.'_':'');
+	$prefixe = ($prefixe ? $prefixe.'_' : '');
 	$table = ($table) ? "/$table/" : ""; 
-	$casier = ($casier) ? rtrim($casier,'/').'/' : ""; // slash final, sinon rien
+	$casier = ($casier) ? rtrim($casier, '/').'/' : ""; // slash final, sinon rien
 	
-	foreach($store as $k=>$v){
+	foreach ($store as $k=>$v) {
 		ecrire_config("$stockage$table$prefixe$casier$k", $v);
-		if (_request('var_mode')=='configurer' AND autoriser('webmestre')){
+		if (_request('var_mode') == 'configurer' AND autoriser('webmestre')) {
 			$trace .= "<br />table $table : ".$prefixe.$k." = $v;";
 		}
 	}
@@ -210,11 +210,11 @@ 
  * @param string $form
  * @param array $valeurs
  */
-function cvtconf_configurer_lire_meta($form,&$valeurs) {
-	list($table,$casier,$prefixe,$stockage) = cvtconf_definir_configurer_conteneur($form,$valeurs);
+function cvtconf_configurer_lire_meta($form, &$valeurs) {
+	list($table, $casier, $prefixe, $stockage) = cvtconf_definir_configurer_conteneur($form, $valeurs);
 
 	$table = ($table) ? "/$table/" : ""; 
-	$prefixe = ($prefixe?$prefixe.'_':'');
+	$prefixe = ($prefixe ? $prefixe.'_' : '');
 	if ($casier) {
 		$meta = lire_config("$stockage$table$prefixe$casier");
 		$prefixe = '';
@@ -224,9 +224,9 @@ 
 		$meta = lire_config("$stockage$table");
 	}
 
-	foreach($valeurs as $k=>$v){
-		if (substr($k,0,1)!=='_')
-		$valeurs[$k] = (isset($meta[$prefixe.$k])?$meta[$prefixe.$k]:'');
+	foreach ($valeurs as $k=>$v) {
+		if (substr($k, 0, 1) !== '_')
+		$valeurs[$k] = (isset($meta[$prefixe.$k]) ? $meta[$prefixe.$k] : '');
 	}
 }
 

ecrire/inc/notifications.php

Spacing +30 added lines, -30 removed lines

@@ -24,16 +24,16 @@ 
  * @param array $options
  *   options de notification, interpretees en fonction de la notification
  */
-function inc_notifications_dist($quoi, $id=0, $options=array()) {
+function inc_notifications_dist($quoi, $id = 0, $options = array()) {
 
 	// charger les fichiers qui veulent ajouter des definitions
 	// ou faire des trucs aussi dans le pipeline, ca fait deux api pour le prix d'une ...
-	pipeline('notifications',array('args'=>array('quoi'=>$quoi,'id'=>$id,'options'=>$options)));
+	pipeline('notifications', array('args'=>array('quoi'=>$quoi, 'id'=>$id, 'options'=>$options)));
 
-	if ($notification = charger_fonction($quoi,'notifications',true)) {
+	if ($notification = charger_fonction($quoi, 'notifications', true)) {
 		spip_log("$notification($quoi,$id"
-			.($options?",".serialize($options):"")
-			.")",'notifications');
+			.($options ? ",".serialize($options) : "")
+			.")", 'notifications');
 		$notification($quoi, $id, $options);
 	}
 }
@@ -47,15 +47,15 @@ 
  * @param array $emails
  * @param array $exclure
  */
-function notifications_nettoyer_emails(&$emails, $exclure = array()){
+function notifications_nettoyer_emails(&$emails, $exclure = array()) {
 	// filtrer et unifier
 	include_spip('inc/filtres');
-	$emails = array_unique(array_filter(array_map('email_valide',array_map('trim', $emails))));
-	if ($exclure AND count($exclure)){
+	$emails = array_unique(array_filter(array_map('email_valide', array_map('trim', $emails))));
+	if ($exclure AND count($exclure)) {
 		// nettoyer les exclusions d'abord
 		notifications_nettoyer_emails($exclure);
 		// faire un diff
-		$emails = array_diff($emails,$exclure);
+		$emails = array_diff($emails, $exclure);
 	}
 }
 
@@ -69,31 +69,31 @@ 
  * @param string $from
  * @param string $headers
  */
-function notifications_envoyer_mails($emails, $texte, $sujet="", $from = "", $headers = ""){
+function notifications_envoyer_mails($emails, $texte, $sujet = "", $from = "", $headers = "") {
 	// rien a faire si pas de texte !
 	if (!strlen($texte))
 		return;
 
 	// si on ne specifie qu'un email, le mettre dans un tableau
 	if (!is_array($emails))
-		$emails = explode(',',$emails);
+		$emails = explode(',', $emails);
 
 	notifications_nettoyer_emails($emails);
 
 	// tester si le mail est deja en html
-	if (strpos($texte,"<")!==false // eviter les tests suivants si possible
+	if (strpos($texte, "<") !== false // eviter les tests suivants si possible
 		AND $ttrim = trim($texte)
-		AND substr($ttrim,0,1)=="<"
-	  AND substr($ttrim,-1,1)==">"
-	  AND stripos($ttrim,"</html>")!==false){
+		AND substr($ttrim, 0, 1) == "<"
+	  AND substr($ttrim, -1, 1) == ">"
+	  AND stripos($ttrim, "</html>") !== false) {
 
-		if(!strlen($sujet)){
+		if (!strlen($sujet)) {
 			// dans ce cas on ruse un peu : extraire le sujet du title
-			if (preg_match(",<title>(.*)</title>,Uims",$texte,$m))
+			if (preg_match(",<title>(.*)</title>,Uims", $texte, $m))
 				$sujet = $m[1];
 			else {
 				// fallback, on prend le body si on le trouve
-				if (preg_match(",<body[^>]*>(.*)</body>,Uims",$texte,$m))
+				if (preg_match(",<body[^>]*>(.*)</body>,Uims", $texte, $m))
 					$ttrim = $m[1];
 
 				// et on extrait la premiere ligne de vrai texte...
@@ -102,35 +102,35 @@ 
 				$ttrim = str_replace("\r\n", "\r", $ttrim);
 				$ttrim = str_replace("\r", "\n", $ttrim);
 				// decouper
-				$ttrim = explode("\n",trim($ttrim));
+				$ttrim = explode("\n", trim($ttrim));
 				// extraire la premiere ligne de texte brut
 				$sujet = array_shift($ttrim);
 			}
 		}
 
 		// si besoin on ajoute le content-type dans les headers
-		if (stripos($headers,"Content-Type")===false)
+		if (stripos($headers, "Content-Type") === false)
 			$headers .= "Content-Type: text/html\n";
 	}
 
 	// si le sujet est vide, extraire la premiere ligne du corps
 	// du mail qui est donc du texte
-	if (!strlen($sujet)){
+	if (!strlen($sujet)) {
 		// nettoyer un peu les retours chariots
 		$texte = str_replace("\r\n", "\r", $texte);
 		$texte = str_replace("\r", "\n", $texte);
 		// decouper
-		$texte = explode("\n",trim($texte));
+		$texte = explode("\n", trim($texte));
 		// extraire la premiere ligne
 		$sujet = array_shift($texte);
-		$texte = trim(implode("\n",$texte));
+		$texte = trim(implode("\n", $texte));
 	}
 
-	$envoyer_mail = charger_fonction('envoyer_mail','inc');
-	foreach($emails as $email){
+	$envoyer_mail = charger_fonction('envoyer_mail', 'inc');
+	foreach ($emails as $email) {
 		// passer dans un pipeline qui permet un ajout eventuel
 		// (url de suivi des notifications par exemple)
-		$envoi = pipeline('notifications_envoyer_mails',array('email'=>$email,'sujet'=>$sujet,'texte'=>$texte));
+		$envoi = pipeline('notifications_envoyer_mails', array('email'=>$email, 'sujet'=>$sujet, 'texte'=>$texte));
 		$email = $envoi['email'];
 
 		job_queue_add('envoyer_mail', ">$email : ".$envoi['sujet'], array($email, $envoi['sujet'], $envoi['texte'], $from, $headers), 'inc/');
@@ -150,9 +150,9 @@ 
  * @return string
  */
 function email_notification_objet($id_objet, $type_objet, $modele) {
-	$envoyer_mail = charger_fonction('envoyer_mail','inc'); // pour nettoyer_titre_email
+	$envoyer_mail = charger_fonction('envoyer_mail', 'inc'); // pour nettoyer_titre_email
 	$id_type = id_table_objet($type_objet);
-	return recuperer_fond($modele,array($id_type=>$id_objet,"id"=>$id_objet));
+	return recuperer_fond($modele, array($id_type=>$id_objet, "id"=>$id_objet));
 }
 
 /**
@@ -166,9 +166,9 @@ 
  * @return string
  */
 function email_notification_article($id_article, $modele) {
-	$envoyer_mail = charger_fonction('envoyer_mail','inc'); // pour nettoyer_titre_email
+	$envoyer_mail = charger_fonction('envoyer_mail', 'inc'); // pour nettoyer_titre_email
 
-	return recuperer_fond($modele,array('id_article'=>$id_article));
+	return recuperer_fond($modele, array('id_article'=>$id_article));
 }
 
 // Compatibilite, ne plus utiliser