spip / SPIP

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",

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
  http://www.loc.gov/standards/iso639-2/langcodes.html
 */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 $GLOBALS['codes_langues'] = array(
 	'aa' => "Afar",

Indentation +196 added lines, -196 removed lines

@@ -22,202 +22,202 @@
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 $GLOBALS['codes_langues'] = array(
-	'aa' => "Afar",
-	'ab' => "Abkhazian",
-	'af' => "Afrikaans",
-	'am' => "Amharic",
-	'an' => "Aragonés",
-	'ar' => "عربي",
-	'as' => "Assamese",
-	'ast' => "asturianu",
-	'ay' => "Aymara",
-	'az' => "Azərbaycan dili",
-	'ba' => "Bashkir",
-	'be' => "Беларуская",
-	'ber_tam' => "Tamazigh",
-	'ber_tam_tfng' => "Tamazigh tifinagh",
-	'bg' => "български",
-	'bh' => "Bihari",
-	'bi' => "Bislama",
-	'bm' => "Bambara",
-	'bn' => "Bengali; Bangla",
-	'bo' => "Tibetan",
-	'br' => "brezhoneg",
-	'bs' => "bosanski",
-	'ca' => "català",
-	'co' => "corsu",
-	'cpf' => "Kréol réyoné",
-	'cpf_dom' => "Kreyòl",
-	'cpf_hat' => "Kreyòl (Peyi Dayiti)",
-	'cs' => "čeština",
-	'cy' => "Cymraeg",	# welsh, gallois
-	'da' => "dansk",
-	'de' => "Deutsch",
-	'dz' => "Bhutani",
-	'el' => "ελληνικά",
-	'en' => "English",
-	'en_hx' => "H4ck3R",
-	'en_sm' => "Smurf",
-	'eo' => "Esperanto",
-	'es' => "Español",
-	'es_co' => "Colombiano",
-	'es_mx_pop' => "Mexicano a lo güey",
-	'et' => "eesti",
-	'eu' => "euskara",
-	'fa' => "فارسى",
-	'ff' => "Fulah", // peul
-	'fi' => "suomi",
-	'fj' => "Fiji",
-	'fo' => "føroyskt",
-	'fon' => "fongbè",
-	'fr' => "français",
-	'fr_fem' => "français féminin",
-	'fr_sc' => "schtroumpf",
-	'fr_lpc' => "langue parlée complétée",
-	'fr_lsf' => "langue des signes française",
-	'fr_spl' => "français simplifié",
-	'fr_tu' => "français copain",
-	'fy' => "Frisian",
-	'ga' => "Irish",
-	'gd' => "Scots Gaelic",
-	'gl' => "galego",
-	'gn' => "Guarani",
-	'grc' => "Ἀρχαία Ἑλληνική", // grec ancien
-	'gu' => "Gujarati",
-	'ha' => "Hausa",
-	'hac' => "ک-هۆرامی", //"Kurdish-Horami"
-	'hbo' => "עברית־התנך", // hebreu classique ou biblique
-	'he' => "עברית",
-	'hi' => "हिंदी",
-	'hr' => "hrvatski",
-	'hu' => "magyar",
-	'hy' => "Հայերեն",// Arménien
-	'ia' => "Interlingua",
-	'id' => "Indonesia",
-	'ie' => "Interlingue",
-	'ik' => "Inupiak",
-	'is' => "íslenska",
-	'it' => "italiano",
-	'it_fem' => "italiana",
-	'iu' => "Inuktitut",
-	'ja' => "日本語",
-	'jv' => "Javanese",
-	'ka' => "ქართული",
-	'kk' => "қазақ тілі", // Kazakh
-	'kl' => "kalaallisut",
-	'km' => "ភាសាខ្មែរ",// Khmer
-	'kn' => "Kannada",
-	'ko' => "한국어",
-	'kok' => "कोंकणी", // Konkani
-	'ks' => "Kashmiri",
-	'ku' => "کوردی",
-	'ky' => "Kirghiz",
-	'la' => "lingua latina",
-	'lb' => "Lëtzebuergesch",
-	'ln' => "Lingala",
-	'lo' => "ພາສາລາວ", # lao
-	'lt' => "lietuvių",
-	'lu' => "luba-katanga",
-	'lv' => "latviešu",
-	'man' => "mandingue", # a traduire en mandingue
-	'mfv' => "manjak", # ISO-639-3
-	'mg' => "Malagasy",
-	'mi' => "Maori",
-	'mk' => "македонски јазик",
-	'ml' => "Malayalam",
-	'mn' => "Монгол хэл",
-	'mo' => "Moldavian",
-	'mos' => "Moré",
-	'mr' => "मराठी",
-	'ms' => "Bahasa Malaysia",
-	'mt' => "Maltese",
-	'my' => "Burmese",
-	'na' => "Nauru",
-	'nap' => "napulitano",
-	'ne' => "Nepali",
-	'nqo' => "N'ko", // www.manden.org
-	'nl' => "Nederlands",
-	'no' => "norsk",
-	'nb' => "norsk bokmål",
-	'nn' => "norsk nynorsk",
-	'oc' => "òc",
-	'oc_lnc' => "òc lengadocian",
-	'oc_ni' => "òc niçard",
-	'oc_ni_la' => "òc niçard (larg)",
-	'oc_ni_mis' => "òc nissart (mistralenc)",
-	'oc_prv' => "òc provençau",
-	'oc_gsc' => "òc gascon",
-	'oc_lms' => "òc lemosin",
-	'oc_auv' => "òc auvernhat",
-	'oc_va' => "òc vivaroaupenc",
-	'om' => "(Afan) Oromo",
-	'or' => "Oriya",
-	'pa' => "Punjabi",
-	'pbb' => 'Nasa Yuwe',
-	'pl' => "polski",
-	'prs' => "دری", // ISO-639-3 Dari (Afghanistan)
-	'ps' => "پښتو",
-	'pt' => "Português",
-	'pt_br' => "Português do Brasil",
-	'qu' => "Quechua",
-	'rm' => "Rhaeto-Romance",
-	'rn' => "Kirundi",
-	'ro' => "română",
-	'roa' => "ch'ti",
-	'ru' => "русский",
-	'rw' => "Kinyarwanda",
-	'sa' => "संस्कृत",
-	'sc' => "sardu",
-	'scn' => "sicilianu",
-	'sd' => "Sindhi",
-	'sg' => "Sangho",
-	'sh' => "srpskohrvastski",
-	'sh_latn' => 'srpskohrvastski',
-	'sh_cyrl' => 'Српскохрватски',
-	'si' => "Sinhalese",
-	'sk' => "slovenčina",	// (Slovakia)
-	'sl' => "slovenščina",	// (Slovenia)
-	'sm' => "Samoan",
-	'sn' => "Shona",
-	'so' => "Somali",
-	'sq' => "shqip",
-	'sr' => "српски",
-	'src' => 'sardu logudorésu', // sarde cf 'sc'
-	'sro' => 'sardu campidanésu',
-	'ss' => "Siswati",
-	'st' => "Sesotho",
-	'su' => "Sundanese",
-	'sv' => "svenska",
-	'sw' => "Kiswahili",
-	'ta' => "தமிழ்", // Tamil
-	'te' => "Telugu",
-	'tg' => "Tajik",
-	'th' => "ไทย",
-	'ti' => "Tigrinya",
-	'tk' => "Turkmen",
-	'tl' => "Tagalog",
-	'tn' => "Setswana",
-	'to' => "Tonga",
-	'tr' => "Türkçe",
-	'ts' => "Tsonga",
-	'tt' => "Татар",
-	'tw' => "Twi",
-	'ty' => "reo mā`ohi", // tahitien
-	'ug' => "Uighur",
-	'uk' => "українська",
-	'ur' => "ٱردو",
-	'uz' => "O'zbekcha",
-	'vi' => "Tiếng Việt",
-	'vo' => "Volapuk",
-	'wa' => "walon",
-	'wo' => "Wolof",
-	'xh' => "Xhosa",
-	'yi' => "Yiddish",
-	'yo' => "Yoruba",
-	'za' => "Zhuang",
-	'zh' => "中文", // chinois (ecriture simplifiee)
-	'zh_tw' => "台灣中文", // chinois taiwan (ecr. traditionnelle)
-	'zu' => "Zulu"
+    'aa' => "Afar",
+    'ab' => "Abkhazian",
+    'af' => "Afrikaans",
+    'am' => "Amharic",
+    'an' => "Aragonés",
+    'ar' => "عربي",
+    'as' => "Assamese",
+    'ast' => "asturianu",
+    'ay' => "Aymara",
+    'az' => "Azərbaycan dili",
+    'ba' => "Bashkir",
+    'be' => "Беларуская",
+    'ber_tam' => "Tamazigh",
+    'ber_tam_tfng' => "Tamazigh tifinagh",
+    'bg' => "български",
+    'bh' => "Bihari",
+    'bi' => "Bislama",
+    'bm' => "Bambara",
+    'bn' => "Bengali; Bangla",
+    'bo' => "Tibetan",
+    'br' => "brezhoneg",
+    'bs' => "bosanski",
+    'ca' => "català",
+    'co' => "corsu",
+    'cpf' => "Kréol réyoné",
+    'cpf_dom' => "Kreyòl",
+    'cpf_hat' => "Kreyòl (Peyi Dayiti)",
+    'cs' => "čeština",
+    'cy' => "Cymraeg",	# welsh, gallois
+    'da' => "dansk",
+    'de' => "Deutsch",
+    'dz' => "Bhutani",
+    'el' => "ελληνικά",
+    'en' => "English",
+    'en_hx' => "H4ck3R",
+    'en_sm' => "Smurf",
+    'eo' => "Esperanto",
+    'es' => "Español",
+    'es_co' => "Colombiano",
+    'es_mx_pop' => "Mexicano a lo güey",
+    'et' => "eesti",
+    'eu' => "euskara",
+    'fa' => "فارسى",
+    'ff' => "Fulah", // peul
+    'fi' => "suomi",
+    'fj' => "Fiji",
+    'fo' => "føroyskt",
+    'fon' => "fongbè",
+    'fr' => "français",
+    'fr_fem' => "français féminin",
+    'fr_sc' => "schtroumpf",
+    'fr_lpc' => "langue parlée complétée",
+    'fr_lsf' => "langue des signes française",
+    'fr_spl' => "français simplifié",
+    'fr_tu' => "français copain",
+    'fy' => "Frisian",
+    'ga' => "Irish",
+    'gd' => "Scots Gaelic",
+    'gl' => "galego",
+    'gn' => "Guarani",
+    'grc' => "Ἀρχαία Ἑλληνική", // grec ancien
+    'gu' => "Gujarati",
+    'ha' => "Hausa",
+    'hac' => "ک-هۆرامی", //"Kurdish-Horami"
+    'hbo' => "עברית־התנך", // hebreu classique ou biblique
+    'he' => "עברית",
+    'hi' => "हिंदी",
+    'hr' => "hrvatski",
+    'hu' => "magyar",
+    'hy' => "Հայերեն",// Arménien
+    'ia' => "Interlingua",
+    'id' => "Indonesia",
+    'ie' => "Interlingue",
+    'ik' => "Inupiak",
+    'is' => "íslenska",
+    'it' => "italiano",
+    'it_fem' => "italiana",
+    'iu' => "Inuktitut",
+    'ja' => "日本語",
+    'jv' => "Javanese",
+    'ka' => "ქართული",
+    'kk' => "қазақ тілі", // Kazakh
+    'kl' => "kalaallisut",
+    'km' => "ភាសាខ្មែរ",// Khmer
+    'kn' => "Kannada",
+    'ko' => "한국어",
+    'kok' => "कोंकणी", // Konkani
+    'ks' => "Kashmiri",
+    'ku' => "کوردی",
+    'ky' => "Kirghiz",
+    'la' => "lingua latina",
+    'lb' => "Lëtzebuergesch",
+    'ln' => "Lingala",
+    'lo' => "ພາສາລາວ", # lao
+    'lt' => "lietuvių",
+    'lu' => "luba-katanga",
+    'lv' => "latviešu",
+    'man' => "mandingue", # a traduire en mandingue
+    'mfv' => "manjak", # ISO-639-3
+    'mg' => "Malagasy",
+    'mi' => "Maori",
+    'mk' => "македонски јазик",
+    'ml' => "Malayalam",
+    'mn' => "Монгол хэл",
+    'mo' => "Moldavian",
+    'mos' => "Moré",
+    'mr' => "मराठी",
+    'ms' => "Bahasa Malaysia",
+    'mt' => "Maltese",
+    'my' => "Burmese",
+    'na' => "Nauru",
+    'nap' => "napulitano",
+    'ne' => "Nepali",
+    'nqo' => "N'ko", // www.manden.org
+    'nl' => "Nederlands",
+    'no' => "norsk",
+    'nb' => "norsk bokmål",
+    'nn' => "norsk nynorsk",
+    'oc' => "òc",
+    'oc_lnc' => "òc lengadocian",
+    'oc_ni' => "òc niçard",
+    'oc_ni_la' => "òc niçard (larg)",
+    'oc_ni_mis' => "òc nissart (mistralenc)",
+    'oc_prv' => "òc provençau",
+    'oc_gsc' => "òc gascon",
+    'oc_lms' => "òc lemosin",
+    'oc_auv' => "òc auvernhat",
+    'oc_va' => "òc vivaroaupenc",
+    'om' => "(Afan) Oromo",
+    'or' => "Oriya",
+    'pa' => "Punjabi",
+    'pbb' => 'Nasa Yuwe',
+    'pl' => "polski",
+    'prs' => "دری", // ISO-639-3 Dari (Afghanistan)
+    'ps' => "پښتو",
+    'pt' => "Português",
+    'pt_br' => "Português do Brasil",
+    'qu' => "Quechua",
+    'rm' => "Rhaeto-Romance",
+    'rn' => "Kirundi",
+    'ro' => "română",
+    'roa' => "ch'ti",
+    'ru' => "русский",
+    'rw' => "Kinyarwanda",
+    'sa' => "संस्कृत",
+    'sc' => "sardu",
+    'scn' => "sicilianu",
+    'sd' => "Sindhi",
+    'sg' => "Sangho",
+    'sh' => "srpskohrvastski",
+    'sh_latn' => 'srpskohrvastski',
+    'sh_cyrl' => 'Српскохрватски',
+    'si' => "Sinhalese",
+    'sk' => "slovenčina",	// (Slovakia)
+    'sl' => "slovenščina",	// (Slovenia)
+    'sm' => "Samoan",
+    'sn' => "Shona",
+    'so' => "Somali",
+    'sq' => "shqip",
+    'sr' => "српски",
+    'src' => 'sardu logudorésu', // sarde cf 'sc'
+    'sro' => 'sardu campidanésu',
+    'ss' => "Siswati",
+    'st' => "Sesotho",
+    'su' => "Sundanese",
+    'sv' => "svenska",
+    'sw' => "Kiswahili",
+    'ta' => "தமிழ்", // Tamil
+    'te' => "Telugu",
+    'tg' => "Tajik",
+    'th' => "ไทย",
+    'ti' => "Tigrinya",
+    'tk' => "Turkmen",
+    'tl' => "Tagalog",
+    'tn' => "Setswana",
+    'to' => "Tonga",
+    'tr' => "Türkçe",
+    'ts' => "Tsonga",
+    'tt' => "Татар",
+    'tw' => "Twi",
+    'ty' => "reo mā`ohi", // tahitien
+    'ug' => "Uighur",
+    'uk' => "українська",
+    'ur' => "ٱردو",
+    'uz' => "O'zbekcha",
+    'vi' => "Tiếng Việt",
+    'vo' => "Volapuk",
+    'wa' => "walon",
+    'wo' => "Wolof",
+    'xh' => "Xhosa",
+    'yi' => "Yiddish",
+    'yo' => "Yoruba",
+    'za' => "Zhuang",
+    'zh' => "中文", // chinois (ecriture simplifiee)
+    'zh_tw' => "台灣中文", // chinois taiwan (ecr. traditionnelle)
+    'zu' => "Zulu"
 
 );
 

ecrire/inc/presenter_enfants.php

Indentation +121 added lines, -121 removed lines

@@ -29,62 +29,62 @@ 
  * 	Un tableau des sous rubriques
  */
 function enfant_rub($collection,$debut=0,$limite=500){
-	$voir_logo = (isset($GLOBALS['meta']['image_process']) AND $GLOBALS['meta']['image_process'] != "non");
-	$logo = "";
+    $voir_logo = (isset($GLOBALS['meta']['image_process']) AND $GLOBALS['meta']['image_process'] != "non");
+    $logo = "";
 
-	if ($voir_logo) {
-		$chercher_logo = charger_fonction('chercher_logo', 'inc');
-		include_spip('inc/filtres_images_mini');
-	}
+    if ($voir_logo) {
+        $chercher_logo = charger_fonction('chercher_logo', 'inc');
+        include_spip('inc/filtres_images_mini');
+    }
 
-	$res = array();
+    $res = array();
 	
-	$result = sql_select("id_rubrique, id_parent, titre, descriptif, lang ", "spip_rubriques", "id_parent=$collection",'', '0+titre,titre',"$debut,$limite");
-	while($row=sql_fetch($result)){
-		$id_rubrique=$row['id_rubrique'];
-		$id_parent=$row['id_parent'];
-		$titre=generer_info_entite($id_rubrique,'rubrique','titre'); // pour etre sur de passer par tous les traitements
-		if ('' !== ($rang = recuperer_numero($row['titre'])))
-			$rang = "$rang. ";
-
-		if (autoriser('voir','rubrique',$id_rubrique)){
-
-			$les_sous_enfants = sous_enfant_rub($id_rubrique);
-
-			changer_typo($row['lang']);
-			$lang_dir = lang_dir($row['lang']);
-			$descriptif=propre($row['descriptif']);
-
-			if ($voir_logo) {
-				if ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on')) {
-					list($fid, $dir, $nom, $format) = $logo;
-					$logo = image_reduire("<img src='$fid' alt='' />", 48, 36);
-					if ($logo)
-						$logo =  inserer_attribut($logo,'class','logo');
-				}
-			}
-
-			$lib_bouton = (!acces_restreint_rubrique($id_rubrique) ? "" :
-			   http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))) .
-			  " <a dir='$lang_dir' href='" .
-			  generer_url_entite($id_rubrique,'rubrique') .
-			  "'>".
-				$rang . $titre .
-			  "</a>";
-
-			$titre = (is_string($logo) ? $logo : '') .
-				bouton_block_depliable($lib_bouton,$les_sous_enfants ?false:-1,"enfants$id_rubrique");
-
-			$res[] =
-			  debut_cadre_sous_rub(($id_parent ? "rubrique-24.png" : "secteur-24.png"), true, "", $titre) .
-			  (!$descriptif ? '' : "\n<div class='descriptif'>$descriptif</div>") .
-			  $les_sous_enfants .
-			  fin_cadre_sous_rub(true);
-		}
-	}
-
-	changer_typo($GLOBALS['spip_lang']); # remettre la typo de l'interface pour la suite
-	return $res;
+    $result = sql_select("id_rubrique, id_parent, titre, descriptif, lang ", "spip_rubriques", "id_parent=$collection",'', '0+titre,titre',"$debut,$limite");
+    while($row=sql_fetch($result)){
+        $id_rubrique=$row['id_rubrique'];
+        $id_parent=$row['id_parent'];
+        $titre=generer_info_entite($id_rubrique,'rubrique','titre'); // pour etre sur de passer par tous les traitements
+        if ('' !== ($rang = recuperer_numero($row['titre'])))
+            $rang = "$rang. ";
+
+        if (autoriser('voir','rubrique',$id_rubrique)){
+
+            $les_sous_enfants = sous_enfant_rub($id_rubrique);
+
+            changer_typo($row['lang']);
+            $lang_dir = lang_dir($row['lang']);
+            $descriptif=propre($row['descriptif']);
+
+            if ($voir_logo) {
+                if ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on')) {
+                    list($fid, $dir, $nom, $format) = $logo;
+                    $logo = image_reduire("<img src='$fid' alt='' />", 48, 36);
+                    if ($logo)
+                        $logo =  inserer_attribut($logo,'class','logo');
+                }
+            }
+
+            $lib_bouton = (!acces_restreint_rubrique($id_rubrique) ? "" :
+               http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))) .
+                " <a dir='$lang_dir' href='" .
+                generer_url_entite($id_rubrique,'rubrique') .
+                "'>".
+                $rang . $titre .
+                "</a>";
+
+            $titre = (is_string($logo) ? $logo : '') .
+                bouton_block_depliable($lib_bouton,$les_sous_enfants ?false:-1,"enfants$id_rubrique");
+
+            $res[] =
+                debut_cadre_sous_rub(($id_parent ? "rubrique-24.png" : "secteur-24.png"), true, "", $titre) .
+                (!$descriptif ? '' : "\n<div class='descriptif'>$descriptif</div>") .
+                $les_sous_enfants .
+                fin_cadre_sous_rub(true);
+        }
+    }
+
+    changer_typo($GLOBALS['spip_lang']); # remettre la typo de l'interface pour la suite
+    return $res;
 }
 
 /**
@@ -97,47 +97,47 @@ 
  * 	Le contenu du bloc dépliable 
  */
 function sous_enfant_rub($collection2){
-	$nb = sql_countsel('spip_rubriques',"id_parent=$collection2");
+    $nb = sql_countsel('spip_rubriques',"id_parent=$collection2");
 	
-	$retour = '';
-	$pagination = '';
-	$debut = 0;
-	$limite = 500;
+    $retour = '';
+    $pagination = '';
+    $debut = 0;
+    $limite = 500;
 	
-	/**
-	 * On ne va afficher que 500 résultats max
-	 * Si > 500 on affiche une pagination
-	 */
-	if($nb > $limite){
-		$debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
-		$pagination = chercher_filtre('pagination');
-		$pagination = '<p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
-		$limite = $debut + $limite;
-	}
+    /**
+     * On ne va afficher que 500 résultats max
+     * Si > 500 on affiche une pagination
+     */
+    if($nb > $limite){
+        $debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
+        $pagination = chercher_filtre('pagination');
+        $pagination = '<p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
+        $limite = $debut + $limite;
+    }
 	
-	$result =  sql_select("id_rubrique, id_parent, titre, lang", "spip_rubriques", "id_parent=$collection2",'', '0+titre,titre',"$debut,$limite");
+    $result =  sql_select("id_rubrique, id_parent, titre, lang", "spip_rubriques", "id_parent=$collection2",'', '0+titre,titre',"$debut,$limite");
 	
-	while($row=sql_fetch($result)){
-		$id_rubrique2=$row['id_rubrique'];
-		$id_parent2=$row['id_parent'];
-		$titre2=generer_info_entite($id_rubrique2,'rubrique','titre'); // pour etre sur de passer par tous les traitements
-		if ('' !== ($rang2 = recuperer_numero($row['titre'])))
-			$rang2 = "$rang2. ";
-
-		changer_typo($row['lang']);
-		$lang_dir = lang_dir($row['lang']);
-		if (autoriser('voir','rubrique',$id_rubrique2))
-			$retour.="\n<li class='item' dir='$lang_dir'><a href='" . generer_url_entite($id_rubrique2,'rubrique') . "'>".$rang2.$titre2."</a></li>\n";
-	}
+    while($row=sql_fetch($result)){
+        $id_rubrique2=$row['id_rubrique'];
+        $id_parent2=$row['id_parent'];
+        $titre2=generer_info_entite($id_rubrique2,'rubrique','titre'); // pour etre sur de passer par tous les traitements
+        if ('' !== ($rang2 = recuperer_numero($row['titre'])))
+            $rang2 = "$rang2. ";
+
+        changer_typo($row['lang']);
+        $lang_dir = lang_dir($row['lang']);
+        if (autoriser('voir','rubrique',$id_rubrique2))
+            $retour.="\n<li class='item' dir='$lang_dir'><a href='" . generer_url_entite($id_rubrique2,'rubrique') . "'>".$rang2.$titre2."</a></li>\n";
+    }
 	
-	$retour = $pagination.$retour.$pagination;
+    $retour = $pagination.$retour.$pagination;
 	 
-	if (!$retour) return '';
+    if (!$retour) return '';
 	
-	return debut_block_depliable($debut > 0 ? true : false,"enfants$collection2")
-	."\n<ul class='liste-items sous-sous-rub'>\n"
-	. $retour
-	. "</ul>\n".fin_block()."\n\n";
+    return debut_block_depliable($debut > 0 ? true : false,"enfants$collection2")
+    ."\n<ul class='liste-items sous-sous-rub'>\n"
+    . $retour
+    . "</ul>\n".fin_block()."\n\n";
 }
 
 /**
@@ -152,43 +152,43 @@ 
  * 	Le contenu textuel affiché, la liste des sous rubriques
  */
 function afficher_enfant_rub($id_rubrique=0) {
-	$pagination = '';
-	$debut = 0;
-	$limite = 500;
+    $pagination = '';
+    $debut = 0;
+    $limite = 500;
 
-	$nb = sql_countsel('spip_rubriques',"id_parent=$id_rubrique");
+    $nb = sql_countsel('spip_rubriques',"id_parent=$id_rubrique");
 	
-	if($nb > $limite){
-		$debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
-		$pagination = chercher_filtre('pagination');
-		$pagination = '<br class="nettoyeur"><p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
-	}
-
-	$les_enfants = enfant_rub($id_rubrique,$debut,$limite);
-
-	if (!$n = count($les_enfants)) return "";
-
-	if ($n==1) {
-		$les_enfants=reset($les_enfants);
-		$les_enfants2="";
-	}
-	else{
-		$n = ceil($n/2);
-		$les_enfants2 = implode('',array_slice($les_enfants,$n));
-		$les_enfants = implode('',array_slice($les_enfants,0,$n));
-	}
+    if($nb > $limite){
+        $debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
+        $pagination = chercher_filtre('pagination');
+        $pagination = '<br class="nettoyeur"><p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
+    }
+
+    $les_enfants = enfant_rub($id_rubrique,$debut,$limite);
+
+    if (!$n = count($les_enfants)) return "";
+
+    if ($n==1) {
+        $les_enfants=reset($les_enfants);
+        $les_enfants2="";
+    }
+    else{
+        $n = ceil($n/2);
+        $les_enfants2 = implode('',array_slice($les_enfants,$n));
+        $les_enfants = implode('',array_slice($les_enfants,0,$n));
+    }
 	
-	$res =
-	$pagination
-	. "<div class='gauche'>"
-	. $les_enfants
-	. "</div>"
-	. "<div class='droite'>"
-	. $les_enfants2
-	. "</div>"
-	. $pagination;
-
-	return $res;
+    $res =
+    $pagination
+    . "<div class='gauche'>"
+    . $les_enfants
+    . "</div>"
+    . "<div class='droite'>"
+    . $les_enfants2
+    . "</div>"
+    . $pagination;
+
+    return $res;
 }
 
 ?>

Braces +22 added lines, -13 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/autoriser');
 include_spip('inc/presentation');
@@ -44,8 +46,9 @@ 
 		$id_rubrique=$row['id_rubrique'];
 		$id_parent=$row['id_parent'];
 		$titre=generer_info_entite($id_rubrique,'rubrique','titre'); // pour etre sur de passer par tous les traitements
-		if ('' !== ($rang = recuperer_numero($row['titre'])))
-			$rang = "$rang. ";
+		if ('' !== ($rang = recuperer_numero($row['titre']))) {
+					$rang = "$rang. ";
+		}
 
 		if (autoriser('voir','rubrique',$id_rubrique)){
 
@@ -59,8 +62,9 @@ 
 				if ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on')) {
 					list($fid, $dir, $nom, $format) = $logo;
 					$logo = image_reduire("<img src='$fid' alt='' />", 48, 36);
-					if ($logo)
-						$logo =  inserer_attribut($logo,'class','logo');
+					if ($logo) {
+											$logo =  inserer_attribut($logo,'class','logo');
+					}
 				}
 			}
 
@@ -121,18 +125,22 @@ 
 		$id_rubrique2=$row['id_rubrique'];
 		$id_parent2=$row['id_parent'];
 		$titre2=generer_info_entite($id_rubrique2,'rubrique','titre'); // pour etre sur de passer par tous les traitements
-		if ('' !== ($rang2 = recuperer_numero($row['titre'])))
-			$rang2 = "$rang2. ";
+		if ('' !== ($rang2 = recuperer_numero($row['titre']))) {
+					$rang2 = "$rang2. ";
+		}
 
 		changer_typo($row['lang']);
 		$lang_dir = lang_dir($row['lang']);
-		if (autoriser('voir','rubrique',$id_rubrique2))
-			$retour.="\n<li class='item' dir='$lang_dir'><a href='" . generer_url_entite($id_rubrique2,'rubrique') . "'>".$rang2.$titre2."</a></li>\n";
+		if (autoriser('voir','rubrique',$id_rubrique2)) {
+					$retour.="\n<li class='item' dir='$lang_dir'><a href='" . generer_url_entite($id_rubrique2,'rubrique') . "'>".$rang2.$titre2."</a></li>\n";
+		}
 	}
 	
 	$retour = $pagination.$retour.$pagination;
 	 
-	if (!$retour) return '';
+	if (!$retour) {
+	    return '';
+	}
 	
 	return debut_block_depliable($debut > 0 ? true : false,"enfants$collection2")
 	."\n<ul class='liste-items sous-sous-rub'>\n"
@@ -166,13 +174,14 @@ 
 
 	$les_enfants = enfant_rub($id_rubrique,$debut,$limite);
 
-	if (!$n = count($les_enfants)) return "";
+	if (!$n = count($les_enfants)) {
+	    return "";
+	}
 
 	if ($n==1) {
 		$les_enfants=reset($les_enfants);
 		$les_enfants2="";
-	}
-	else{
+	} else{
 		$n = ceil($n/2);
 		$les_enfants2 = implode('',array_slice($les_enfants,$n));
 		$les_enfants = implode('',array_slice($les_enfants,0,$n));

Spacing +40 added lines, -41 removed lines

@@ -28,7 +28,7 @@ 
  * @return array $res
  * 	Un tableau des sous rubriques
  */
-function enfant_rub($collection,$debut=0,$limite=500){
+function enfant_rub($collection, $debut = 0, $limite = 500) {
 	$voir_logo = (isset($GLOBALS['meta']['image_process']) AND $GLOBALS['meta']['image_process'] != "non");
 	$logo = "";
 
@@ -39,46 +39,45 @@ 
 
 	$res = array();
 	
-	$result = sql_select("id_rubrique, id_parent, titre, descriptif, lang ", "spip_rubriques", "id_parent=$collection",'', '0+titre,titre',"$debut,$limite");
-	while($row=sql_fetch($result)){
-		$id_rubrique=$row['id_rubrique'];
-		$id_parent=$row['id_parent'];
-		$titre=generer_info_entite($id_rubrique,'rubrique','titre'); // pour etre sur de passer par tous les traitements
+	$result = sql_select("id_rubrique, id_parent, titre, descriptif, lang ", "spip_rubriques", "id_parent=$collection", '', '0+titre,titre', "$debut,$limite");
+	while ($row = sql_fetch($result)) {
+		$id_rubrique = $row['id_rubrique'];
+		$id_parent = $row['id_parent'];
+		$titre = generer_info_entite($id_rubrique, 'rubrique', 'titre'); // pour etre sur de passer par tous les traitements
 		if ('' !== ($rang = recuperer_numero($row['titre'])))
 			$rang = "$rang. ";
 
-		if (autoriser('voir','rubrique',$id_rubrique)){
+		if (autoriser('voir', 'rubrique', $id_rubrique)) {
 
 			$les_sous_enfants = sous_enfant_rub($id_rubrique);
 
 			changer_typo($row['lang']);
 			$lang_dir = lang_dir($row['lang']);
-			$descriptif=propre($row['descriptif']);
+			$descriptif = propre($row['descriptif']);
 
 			if ($voir_logo) {
 				if ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on')) {
 					list($fid, $dir, $nom, $format) = $logo;
 					$logo = image_reduire("<img src='$fid' alt='' />", 48, 36);
 					if ($logo)
-						$logo =  inserer_attribut($logo,'class','logo');
+						$logo = inserer_attribut($logo, 'class', 'logo');
 				}
 			}
 
-			$lib_bouton = (!acces_restreint_rubrique($id_rubrique) ? "" :
-			   http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))) .
-			  " <a dir='$lang_dir' href='" .
-			  generer_url_entite($id_rubrique,'rubrique') .
+			$lib_bouton = (!acces_restreint_rubrique($id_rubrique) ? "" : http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))).
+			  " <a dir='$lang_dir' href='".
+			  generer_url_entite($id_rubrique, 'rubrique').
 			  "'>".
-				$rang . $titre .
+				$rang.$titre.
 			  "</a>";
 
-			$titre = (is_string($logo) ? $logo : '') .
-				bouton_block_depliable($lib_bouton,$les_sous_enfants ?false:-1,"enfants$id_rubrique");
+			$titre = (is_string($logo) ? $logo : '').
+				bouton_block_depliable($lib_bouton, $les_sous_enfants ?false:-1, "enfants$id_rubrique");
 
 			$res[] =
-			  debut_cadre_sous_rub(($id_parent ? "rubrique-24.png" : "secteur-24.png"), true, "", $titre) .
-			  (!$descriptif ? '' : "\n<div class='descriptif'>$descriptif</div>") .
-			  $les_sous_enfants .
+			  debut_cadre_sous_rub(($id_parent ? "rubrique-24.png" : "secteur-24.png"), true, "", $titre).
+			  (!$descriptif ? '' : "\n<div class='descriptif'>$descriptif</div>").
+			  $les_sous_enfants.
 			  fin_cadre_sous_rub(true);
 		}
 	}
@@ -96,8 +95,8 @@ 
  * @return string
  * 	Le contenu du bloc dépliable 
  */
-function sous_enfant_rub($collection2){
-	$nb = sql_countsel('spip_rubriques',"id_parent=$collection2");
+function sous_enfant_rub($collection2) {
+	$nb = sql_countsel('spip_rubriques', "id_parent=$collection2");
 	
 	$retour = '';
 	$pagination = '';
@@ -108,33 +107,33 @@ 
 	 * On ne va afficher que 500 résultats max
 	 * Si > 500 on affiche une pagination
 	 */
-	if($nb > $limite){
+	if ($nb > $limite) {
 		$debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
 		$pagination = chercher_filtre('pagination');
 		$pagination = '<p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
 		$limite = $debut + $limite;
 	}
 	
-	$result =  sql_select("id_rubrique, id_parent, titre, lang", "spip_rubriques", "id_parent=$collection2",'', '0+titre,titre',"$debut,$limite");
+	$result = sql_select("id_rubrique, id_parent, titre, lang", "spip_rubriques", "id_parent=$collection2", '', '0+titre,titre', "$debut,$limite");
 	
-	while($row=sql_fetch($result)){
-		$id_rubrique2=$row['id_rubrique'];
-		$id_parent2=$row['id_parent'];
-		$titre2=generer_info_entite($id_rubrique2,'rubrique','titre'); // pour etre sur de passer par tous les traitements
+	while ($row = sql_fetch($result)) {
+		$id_rubrique2 = $row['id_rubrique'];
+		$id_parent2 = $row['id_parent'];
+		$titre2 = generer_info_entite($id_rubrique2, 'rubrique', 'titre'); // pour etre sur de passer par tous les traitements
 		if ('' !== ($rang2 = recuperer_numero($row['titre'])))
 			$rang2 = "$rang2. ";
 
 		changer_typo($row['lang']);
 		$lang_dir = lang_dir($row['lang']);
-		if (autoriser('voir','rubrique',$id_rubrique2))
-			$retour.="\n<li class='item' dir='$lang_dir'><a href='" . generer_url_entite($id_rubrique2,'rubrique') . "'>".$rang2.$titre2."</a></li>\n";
+		if (autoriser('voir', 'rubrique', $id_rubrique2))
+			$retour .= "\n<li class='item' dir='$lang_dir'><a href='".generer_url_entite($id_rubrique2, 'rubrique')."'>".$rang2.$titre2."</a></li>\n";
 	}
 	
 	$retour = $pagination.$retour.$pagination;
 	 
 	if (!$retour) return '';
 	
-	return debut_block_depliable($debut > 0 ? true : false,"enfants$collection2")
+	return debut_block_depliable($debut > 0 ? true : false, "enfants$collection2")
 	."\n<ul class='liste-items sous-sous-rub'>\n"
 	. $retour
 	. "</ul>\n".fin_block()."\n\n";
@@ -151,31 +150,31 @@ 
  * @return string $res
  * 	Le contenu textuel affiché, la liste des sous rubriques
  */
-function afficher_enfant_rub($id_rubrique=0) {
+function afficher_enfant_rub($id_rubrique = 0) {
 	$pagination = '';
 	$debut = 0;
 	$limite = 500;
 
-	$nb = sql_countsel('spip_rubriques',"id_parent=$id_rubrique");
+	$nb = sql_countsel('spip_rubriques', "id_parent=$id_rubrique");
 	
-	if($nb > $limite){
+	if ($nb > $limite) {
 		$debut = _request('debut_rubrique'.$collection2) ? _request('debut_rubrique'.$collection2) : $debut;
 		$pagination = chercher_filtre('pagination');
 		$pagination = '<br class="nettoyeur"><p class="pagination">'.$pagination($nb, '_rubrique'.$collection2, $debut, $limite, true, 'prive').'</p>';
 	}
 
-	$les_enfants = enfant_rub($id_rubrique,$debut,$limite);
+	$les_enfants = enfant_rub($id_rubrique, $debut, $limite);
 
 	if (!$n = count($les_enfants)) return "";
 
-	if ($n==1) {
-		$les_enfants=reset($les_enfants);
-		$les_enfants2="";
+	if ($n == 1) {
+		$les_enfants = reset($les_enfants);
+		$les_enfants2 = "";
 	}
-	else{
-		$n = ceil($n/2);
-		$les_enfants2 = implode('',array_slice($les_enfants,$n));
-		$les_enfants = implode('',array_slice($les_enfants,0,$n));
+	else {
+		$n = ceil($n / 2);
+		$les_enfants2 = implode('', array_slice($les_enfants, $n));
+		$les_enfants = implode('', array_slice($les_enfants, 0, $n));
 	}
 	
 	$res =

ecrire/inc/precharger_objet.php

Indentation +109 added lines, -109 removed lines

@@ -30,69 +30,69 @@ 
  * @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') {
-	global $connect_id_rubrique, $spip_lang;
+    global $connect_id_rubrique, $spip_lang;
 	
-	$table = table_objet_sql($type);
-	$_id_objet = id_table_objet($table);
-
-	// si l'objet existe deja, on retourne simplement ses valeurs
-	if (is_numeric($id_objet))
-		return sql_fetsel("*", $table, "$_id_objet=$id_objet");
-
-	// ici, on demande une creation.
-	// on prerempli certains elements : les champs si traduction,
-	// les id_rubrique et id_secteur si l'objet a ces champs
-	$desc = lister_tables_objets_sql($table);
-	# il faudrait calculer $champ_titre ici
-	$is_rubrique = isset($desc['field']['id_rubrique']);
-	$is_secteur  = isset($desc['field']['id_secteur']);
-
-	// si demande de traduction
-	// on recupere les valeurs de la traduction
-	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);
-	} else {
-		$row[$champ_titre] = '';
-		if ($is_rubrique) {
-			$row['id_rubrique'] = $id_rubrique;
-		}
-	}
-
-	// calcul de la rubrique
-	# note : comment faire pour des traductions sur l'objet rubriques ?
-	if ($is_rubrique) {
-		// appel du script a la racine, faut choisir 
-		// admin restreint ==> sa premiere rubrique
-		// autre ==> la derniere rubrique cree
-		if (!$row['id_rubrique']) {
-			if ($connect_id_rubrique)
-				$row['id_rubrique'] = $id_rubrique = current($connect_id_rubrique); 
-			else {
-				$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'] )){
-				// 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)){
-					$row['id_rubrique'] = $row_rub['id_rubrique'];
-				}
-			}
-		}
-	}
+    $table = table_objet_sql($type);
+    $_id_objet = id_table_objet($table);
+
+    // si l'objet existe deja, on retourne simplement ses valeurs
+    if (is_numeric($id_objet))
+        return sql_fetsel("*", $table, "$_id_objet=$id_objet");
+
+    // ici, on demande une creation.
+    // on prerempli certains elements : les champs si traduction,
+    // les id_rubrique et id_secteur si l'objet a ces champs
+    $desc = lister_tables_objets_sql($table);
+    # il faudrait calculer $champ_titre ici
+    $is_rubrique = isset($desc['field']['id_rubrique']);
+    $is_secteur  = isset($desc['field']['id_secteur']);
+
+    // si demande de traduction
+    // on recupere les valeurs de la traduction
+    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);
+    } else {
+        $row[$champ_titre] = '';
+        if ($is_rubrique) {
+            $row['id_rubrique'] = $id_rubrique;
+        }
+    }
+
+    // calcul de la rubrique
+    # note : comment faire pour des traductions sur l'objet rubriques ?
+    if ($is_rubrique) {
+        // appel du script a la racine, faut choisir 
+        // admin restreint ==> sa premiere rubrique
+        // autre ==> la derniere rubrique cree
+        if (!$row['id_rubrique']) {
+            if ($connect_id_rubrique)
+                $row['id_rubrique'] = $id_rubrique = current($connect_id_rubrique); 
+            else {
+                $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'] )){
+                // 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)){
+                    $row['id_rubrique'] = $row_rub['id_rubrique'];
+                }
+            }
+        }
+    }
 	
-	// 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['id_secteur'] = $row_rub;
-		}
-	}
-
-	return $row;
+    // 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['id_secteur'] = $row_rub;
+        }
+    }
+
+    return $row;
 }
 
 
@@ -109,58 +109,58 @@ 
  * @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') {
-	$table = table_objet_sql($type);
-	$_id_objet = id_table_objet($table);
-
-	// Recuperer les donnees de l'objet original
-	$row = sql_fetsel("*", $table, "$_id_objet=$lier_trad");
-	if ($row) {
-		$row[$champ_titre] = filtrer_entites(_T('info_nouvelle_traduction')).' '.$row[$champ_titre];
-	} else {
-		$row = array();
-	}
-
-	// on met l'objet dans une rubrique si l'objet le peut
-	$desc = lister_tables_objets_sql($table);
-	$is_rubrique = isset($desc['field']['id_rubrique']);
+    $table = table_objet_sql($type);
+    $_id_objet = id_table_objet($table);
+
+    // Recuperer les donnees de l'objet original
+    $row = sql_fetsel("*", $table, "$_id_objet=$lier_trad");
+    if ($row) {
+        $row[$champ_titre] = filtrer_entites(_T('info_nouvelle_traduction')).' '.$row[$champ_titre];
+    } else {
+        $row = array();
+    }
+
+    // on met l'objet dans une rubrique si l'objet le peut
+    $desc = lister_tables_objets_sql($table);
+    $is_rubrique = isset($desc['field']['id_rubrique']);
 	
-	if ($is_rubrique) {
-		if ($id_rubrique) {
-			$row['id_rubrique'] = $id_rubrique;
-			return $row;
-		}
-		$id_rubrique = $row['id_rubrique'];
+    if ($is_rubrique) {
+        if ($id_rubrique) {
+            $row['id_rubrique'] = $id_rubrique;
+            return $row;
+        }
+        $id_rubrique = $row['id_rubrique'];
 	
 
-		// Regler la langue, si possible, sur celle du redacteur
-		// Cela implique souvent de choisir une rubrique ou un secteur
-		if (in_array($GLOBALS['spip_lang'],
-		explode(',', $GLOBALS['meta']['langues_multilingue']))) {
-
-			// 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']))) {
-				$row['id_rubrique'] = $row['id_rubrique']; # explicite :-)
-
-			// Sinon, chercher la rubrique la plus adaptee pour
-			// accueillir l'objet dans la langue du traducteur
-			} elseif ($is_rubrique and $GLOBALS['meta']['multi_rubriques'] == 'oui') {
-				if ($GLOBALS['meta']['multi_secteurs'] == 'oui') {
-					$id_parent = 0;
-				} else {
-					// on cherche une rubrique soeur dans la bonne langue
-					$row_rub = sql_fetsel("id_parent", "spip_rubriques", "id_rubrique=$id_rubrique");
-					$id_parent = $row_rub['id_parent'];
-				}
+        // Regler la langue, si possible, sur celle du redacteur
+        // Cela implique souvent de choisir une rubrique ou un secteur
+        if (in_array($GLOBALS['spip_lang'],
+        explode(',', $GLOBALS['meta']['langues_multilingue']))) {
+
+            // 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']))) {
+                $row['id_rubrique'] = $row['id_rubrique']; # explicite :-)
+
+            // Sinon, chercher la rubrique la plus adaptee pour
+            // accueillir l'objet dans la langue du traducteur
+            } elseif ($is_rubrique and $GLOBALS['meta']['multi_rubriques'] == 'oui') {
+                if ($GLOBALS['meta']['multi_secteurs'] == 'oui') {
+                    $id_parent = 0;
+                } else {
+                    // on cherche une rubrique soeur dans la bonne langue
+                    $row_rub = sql_fetsel("id_parent", "spip_rubriques", "id_rubrique=$id_rubrique");
+                    $id_parent = $row_rub['id_parent'];
+                }
 				
-				$row_rub = sql_fetsel("id_rubrique", "spip_rubriques", "lang='".$GLOBALS['spip_lang']."' AND id_parent=$id_parent");
-				if ($row_rub)
-					$row['id_rubrique'] = $row_rub['id_rubrique'];	
-			}
-		}
-	}
-	return $row;
+                $row_rub = sql_fetsel("id_rubrique", "spip_rubriques", "lang='".$GLOBALS['spip_lang']."' AND id_parent=$id_parent");
+                if ($row_rub)
+                    $row['id_rubrique'] = $row_rub['id_rubrique'];	
+            }
+        }
+    }
+    return $row;
 }
 
 

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

Braces +17 added lines, -12 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/autoriser'); // necessaire si appel de l'espace public
 
@@ -36,8 +38,9 @@ 
 	$_id_objet = id_table_objet($table);
 
 	// si l'objet existe deja, on retourne simplement ses valeurs
-	if (is_numeric($id_objet))
-		return sql_fetsel("*", $table, "$_id_objet=$id_objet");
+	if (is_numeric($id_objet)) {
+			return sql_fetsel("*", $table, "$_id_objet=$id_objet");
+	}
 
 	// ici, on demande une creation.
 	// on prerempli certains elements : les champs si traduction,
@@ -50,10 +53,11 @@ 
 	// si demande de traduction
 	// on recupere les valeurs de la traduction
 	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);
+		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);
+		}
 	} else {
 		$row[$champ_titre] = '';
 		if ($is_rubrique) {
@@ -68,9 +72,9 @@ 
 		// admin restreint ==> sa premiere rubrique
 		// autre ==> la derniere rubrique cree
 		if (!$row['id_rubrique']) {
-			if ($connect_id_rubrique)
-				$row['id_rubrique'] = $id_rubrique = current($connect_id_rubrique); 
-			else {
+			if ($connect_id_rubrique) {
+							$row['id_rubrique'] = $id_rubrique = current($connect_id_rubrique);
+			} else {
 				$row_rub = sql_fetsel("id_rubrique", "spip_rubriques", "", "", "id_rubrique DESC", 1);
 				$row['id_rubrique'] = $id_rubrique = $row_rub['id_rubrique'];
 			}
@@ -155,8 +159,9 @@ 
 				}
 				
 				$row_rub = sql_fetsel("id_rubrique", "spip_rubriques", "lang='".$GLOBALS['spip_lang']."' AND id_parent=$id_parent");
-				if ($row_rub)
-					$row['id_rubrique'] = $row_rub['id_rubrique'];	
+				if ($row_rub) {
+									$row['id_rubrique'] = $row_rub['id_rubrique'];
+				}
 			}
 		}
 	}

ecrire/inc/commencer_page.php

Indentation +49 added lines, -49 removed lines

@@ -18,88 +18,88 @@
 
 // http://doc.spip.org/@inc_commencer_page_dist
 function inc_commencer_page_dist($titre = "", $rubrique = "accueil", $sous_rubrique = "accueil", $id_rubrique = "",$menu=true,$minipres=false, $alertes = true) {
-	global $connect_id_auteur;
+    global $connect_id_auteur;
 
-	include_spip('inc/headers');
+    include_spip('inc/headers');
 
-	http_no_cache();
+    http_no_cache();
 
-	return init_entete($titre, $id_rubrique, $minipres)
-	. init_body($rubrique, $sous_rubrique, $id_rubrique,$menu)
-	. "<div id='page'>"
-	. auteurs_recemment_connectes($connect_id_auteur)
-	. ($alertes?alertes_auteur($connect_id_auteur):'')
-	. '<div class="largeur">';
+    return init_entete($titre, $id_rubrique, $minipres)
+    . init_body($rubrique, $sous_rubrique, $id_rubrique,$menu)
+    . "<div id='page'>"
+    . auteurs_recemment_connectes($connect_id_auteur)
+    . ($alertes?alertes_auteur($connect_id_auteur):'')
+    . '<div class="largeur">';
 }
 
 // envoi du doctype et du <head><title>...</head>
 // http://doc.spip.org/@init_entete
 function init_entete($titre='', $dummy=0, $minipres=false) {
-	include_spip('inc/texte');
-	if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']["nom_site"])))
-		$nom_site_spip=  _T('info_mon_site_spip');
-
-	$titre = "["
-		. $nom_site_spip
-		. "]" 
-	  . ($titre ? " ".textebrut(typo($titre)):"");
-
-	return _DOCTYPE_ECRIRE
-	. html_lang_attributes()
-	. "<head>\n"
-	. init_head($titre, $dummy, $minipres)
-	. "</head>\n";
+    include_spip('inc/texte');
+    if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']["nom_site"])))
+        $nom_site_spip=  _T('info_mon_site_spip');
+
+    $titre = "["
+        . $nom_site_spip
+        . "]" 
+        . ($titre ? " ".textebrut(typo($titre)):"");
+
+    return _DOCTYPE_ECRIRE
+    . html_lang_attributes()
+    . "<head>\n"
+    . init_head($titre, $dummy, $minipres)
+    . "</head>\n";
 }
 
 function init_head($titre='', $dummy=0, $minipres=false) {
-	return recuperer_fond("prive/squelettes/head/dist",array('titre'=>$titre,'minipres'=>$minipres?' ':''));
+    return recuperer_fond("prive/squelettes/head/dist",array('titre'=>$titre,'minipres'=>$minipres?' ':''));
 }
 
 // fonction envoyant la double serie d'icones de redac
 // http://doc.spip.org/@init_body
 function init_body($rubrique='accueil', $sous_rubrique='accueil', $id_rubrique='',$menu=true) {
-	global $connect_id_auteur, $auth_can_disconnect;
+    global $connect_id_auteur, $auth_can_disconnect;
 
-	$res = pipeline('body_prive',"<body class='"
-			. init_body_class()." "._request('exec')."'"
-			. ($GLOBALS['spip_lang_rtl'] ? " dir='rtl'" : "")
-			.'>');
+    $res = pipeline('body_prive',"<body class='"
+            . init_body_class()." "._request('exec')."'"
+            . ($GLOBALS['spip_lang_rtl'] ? " dir='rtl'" : "")
+            .'>');
 
-	if (!$menu) return $res;
+    if (!$menu) return $res;
 
 
-	$bandeau = charger_fonction('bandeau', 'inc');
+    $bandeau = charger_fonction('bandeau', 'inc');
 
-	return $res
-	 . $bandeau();
+    return $res
+        . $bandeau();
 }
 
 function init_body_class() {
-	$GLOBALS['spip_display'] = isset($GLOBALS['visiteur_session']['prefs']['display'])
-		? $GLOBALS['visiteur_session']['prefs']['display']
-		: 2;
-	$spip_display_navigation = isset($GLOBALS['visiteur_session']['prefs']['display_navigation'])
-		? $GLOBALS['visiteur_session']['prefs']['display_navigation']
-		: 'navigation_avec_icones';
-	$spip_display_outils = isset($GLOBALS['visiteur_session']['prefs']['display_outils'])
-		? ($GLOBALS['visiteur_session']['prefs']['display_outils']?'navigation_avec_outils':'navigation_sans_outils')
-		: 'navigation_avec_outils';
-	$GLOBALS['spip_ecran'] = isset($_COOKIE['spip_ecran']) ? $_COOKIE['spip_ecran'] : "etroit";
-
-	$display_class=array(0=>'icones_img_texte'/*init*/,1=>'icones_texte',2=>'icones_img_texte',3=>'icones_img');
-	return $GLOBALS['spip_ecran'] . " $spip_display_navigation $spip_display_outils ".$display_class[$GLOBALS['spip_display']];
+    $GLOBALS['spip_display'] = isset($GLOBALS['visiteur_session']['prefs']['display'])
+        ? $GLOBALS['visiteur_session']['prefs']['display']
+        : 2;
+    $spip_display_navigation = isset($GLOBALS['visiteur_session']['prefs']['display_navigation'])
+        ? $GLOBALS['visiteur_session']['prefs']['display_navigation']
+        : 'navigation_avec_icones';
+    $spip_display_outils = isset($GLOBALS['visiteur_session']['prefs']['display_outils'])
+        ? ($GLOBALS['visiteur_session']['prefs']['display_outils']?'navigation_avec_outils':'navigation_sans_outils')
+        : 'navigation_avec_outils';
+    $GLOBALS['spip_ecran'] = isset($_COOKIE['spip_ecran']) ? $_COOKIE['spip_ecran'] : "etroit";
+
+    $display_class=array(0=>'icones_img_texte'/*init*/,1=>'icones_texte',2=>'icones_img_texte',3=>'icones_img');
+    return $GLOBALS['spip_ecran'] . " $spip_display_navigation $spip_display_outils ".$display_class[$GLOBALS['spip_display']];
 }
 
 
 // http://doc.spip.org/@lien_change_var
 function lien_change_var($lien, $set, $couleur, $coords, $titre, $mouseOver="") {
-	$lien = parametre_url($lien, $set, $couleur);
-	return "\n<area shape='rect' href='$lien' coords='$coords' title=\"$titre\" alt=\"$titre\" $mouseOver />";
+    $lien = parametre_url($lien, $set, $couleur);
+    return "\n<area shape='rect' href='$lien' coords='$coords' title=\"$titre\" alt=\"$titre\" $mouseOver />";
 }
 
 // http://doc.spip.org/@auteurs_recemment_connectes
 function auteurs_recemment_connectes($id_auteur){
-	return recuperer_fond('prive/objets/liste/auteurs_enligne');
+    return recuperer_fond('prive/objets/liste/auteurs_enligne');
 }
 
 ?>

Braces +9 added lines, -4 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;
+}
 
 //
 // Presentation de l'interface privee, debut du HTML
@@ -36,8 +38,9 @@ 
 // http://doc.spip.org/@init_entete
 function init_entete($titre='', $dummy=0, $minipres=false) {
 	include_spip('inc/texte');
-	if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']["nom_site"])))
-		$nom_site_spip=  _T('info_mon_site_spip');
+	if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']["nom_site"]))) {
+			$nom_site_spip=  _T('info_mon_site_spip');
+	}
 
 	$titre = "["
 		. $nom_site_spip
@@ -65,7 +68,9 @@ 
 			. ($GLOBALS['spip_lang_rtl'] ? " dir='rtl'" : "")
 			.'>');
 
-	if (!$menu) return $res;
+	if (!$menu) {
+	    return $res;
+	}
 
 
 	$bandeau = charger_fonction('bandeau', 'inc');

Spacing +15 added lines, -15 removed lines

@@ -17,7 +17,7 @@ 
 //
 
 // http://doc.spip.org/@inc_commencer_page_dist
-function inc_commencer_page_dist($titre = "", $rubrique = "accueil", $sous_rubrique = "accueil", $id_rubrique = "",$menu=true,$minipres=false, $alertes = true) {
+function inc_commencer_page_dist($titre = "", $rubrique = "accueil", $sous_rubrique = "accueil", $id_rubrique = "", $menu = true, $minipres = false, $alertes = true) {
 	global $connect_id_auteur;
 
 	include_spip('inc/headers');
@@ -25,24 +25,24 @@ 
 	http_no_cache();
 
 	return init_entete($titre, $id_rubrique, $minipres)
-	. init_body($rubrique, $sous_rubrique, $id_rubrique,$menu)
+	. init_body($rubrique, $sous_rubrique, $id_rubrique, $menu)
 	. "<div id='page'>"
 	. auteurs_recemment_connectes($connect_id_auteur)
-	. ($alertes?alertes_auteur($connect_id_auteur):'')
+	. ($alertes ?alertes_auteur($connect_id_auteur) : '')
 	. '<div class="largeur">';
 }
 
 // envoi du doctype et du <head><title>...</head>
 // http://doc.spip.org/@init_entete
-function init_entete($titre='', $dummy=0, $minipres=false) {
+function init_entete($titre = '', $dummy = 0, $minipres = false) {
 	include_spip('inc/texte');
 	if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']["nom_site"])))
-		$nom_site_spip=  _T('info_mon_site_spip');
+		$nom_site_spip = _T('info_mon_site_spip');
 
 	$titre = "["
 		. $nom_site_spip
 		. "]" 
-	  . ($titre ? " ".textebrut(typo($titre)):"");
+	  . ($titre ? " ".textebrut(typo($titre)) : "");
 
 	return _DOCTYPE_ECRIRE
 	. html_lang_attributes()
@@ -51,16 +51,16 @@ 
 	. "</head>\n";
 }
 
-function init_head($titre='', $dummy=0, $minipres=false) {
-	return recuperer_fond("prive/squelettes/head/dist",array('titre'=>$titre,'minipres'=>$minipres?' ':''));
+function init_head($titre = '', $dummy = 0, $minipres = false) {
+	return recuperer_fond("prive/squelettes/head/dist", array('titre'=>$titre, 'minipres'=>$minipres ? ' ' : ''));
 }
 
 // fonction envoyant la double serie d'icones de redac
 // http://doc.spip.org/@init_body
-function init_body($rubrique='accueil', $sous_rubrique='accueil', $id_rubrique='',$menu=true) {
+function init_body($rubrique = 'accueil', $sous_rubrique = 'accueil', $id_rubrique = '', $menu = true) {
 	global $connect_id_auteur, $auth_can_disconnect;
 
-	$res = pipeline('body_prive',"<body class='"
+	$res = pipeline('body_prive', "<body class='"
 			. init_body_class()." "._request('exec')."'"
 			. ($GLOBALS['spip_lang_rtl'] ? " dir='rtl'" : "")
 			.'>');
@@ -82,23 +82,23 @@ 
 		? $GLOBALS['visiteur_session']['prefs']['display_navigation']
 		: 'navigation_avec_icones';
 	$spip_display_outils = isset($GLOBALS['visiteur_session']['prefs']['display_outils'])
-		? ($GLOBALS['visiteur_session']['prefs']['display_outils']?'navigation_avec_outils':'navigation_sans_outils')
+		? ($GLOBALS['visiteur_session']['prefs']['display_outils'] ? 'navigation_avec_outils' : 'navigation_sans_outils')
 		: 'navigation_avec_outils';
 	$GLOBALS['spip_ecran'] = isset($_COOKIE['spip_ecran']) ? $_COOKIE['spip_ecran'] : "etroit";
 
-	$display_class=array(0=>'icones_img_texte'/*init*/,1=>'icones_texte',2=>'icones_img_texte',3=>'icones_img');
-	return $GLOBALS['spip_ecran'] . " $spip_display_navigation $spip_display_outils ".$display_class[$GLOBALS['spip_display']];
+	$display_class = array(0=>'icones_img_texte'/*init*/, 1=>'icones_texte', 2=>'icones_img_texte', 3=>'icones_img');
+	return $GLOBALS['spip_ecran']." $spip_display_navigation $spip_display_outils ".$display_class[$GLOBALS['spip_display']];
 }
 
 
 // http://doc.spip.org/@lien_change_var
-function lien_change_var($lien, $set, $couleur, $coords, $titre, $mouseOver="") {
+function lien_change_var($lien, $set, $couleur, $coords, $titre, $mouseOver = "") {
 	$lien = parametre_url($lien, $set, $couleur);
 	return "\n<area shape='rect' href='$lien' coords='$coords' title=\"$titre\" alt=\"$titre\" $mouseOver />";
 }
 
 // http://doc.spip.org/@auteurs_recemment_connectes
-function auteurs_recemment_connectes($id_auteur){
+function auteurs_recemment_connectes($id_auteur) {
 	return recuperer_fond('prive/objets/liste/auteurs_enligne');
 }
 

ecrire/inc/cvt_configurer.php

Indentation +136 added lines, -136 removed lines

@@ -21,28 +21,28 @@ 
  * @return array
  */
 function cvtconf_formulaire_charger($flux){
-	if (
-		$form = $flux['args']['form']
-		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))) {
-			return false;
-		}
+    if (
+        $form = $flux['args']['form']
+        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))) {
+            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)) {
-			$flux['data'] = cvtconf_formulaires_configurer_recense($form);
-			$flux['data']['editable'] = true;
-			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' />";
-			}
-		}
-	}
-	return $flux;
+        // S'il n'y a pas de fonction charger(), on génère un contexte automatiquement
+        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 (!_AJAX) var_dump($flux['data']);
+                // reinjecter pour la trace au traitement
+                $flux['data']['_hidden'] = "<input type='hidden' name='var_mode' value='configurer' />";
+            }
+        }
+    }
+    return $flux;
 }
 
 /**
@@ -52,14 +52,14 @@ 
  * @return array
  */
 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()
-		) {
-		$trace = cvtconf_formulaires_configurer_enregistre($form,$flux['args']['args']);
-		$flux['data'] = array('message_ok'=>_T('config_info_enregistree').$trace,'editable'=>true);
-	}
-	return $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()
+        ) {
+        $trace = cvtconf_formulaires_configurer_enregistre($form,$flux['args']['args']);
+        $flux['data'] = array('message_ok'=>_T('config_info_enregistree').$trace,'editable'=>true);
+    }
+    return $flux;
 }
 
 /**
@@ -75,29 +75,29 @@ 
  * @return string
  */
 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);
-		$valeurs = pipeline(
-			'formulaire_charger',
-			array(
-				'args'=>array('form'=>$form,'args'=>$args,'je_suis_poste'=>false),
-				'data'=>$valeurs)
-		);
-		// ne pas stocker editable !
-		unset($valeurs['editable']);
-
-		// recuperer les valeurs postees
-		$store = array();
-		foreach($valeurs as $k=>$v){
-			if (substr($k,0,1)!=='_')
-				$store[$k] = _request($k);
-		}
-
-		return cvtconf_configurer_stocker($form,$valeurs,$store);
+        $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);
+        $valeurs = pipeline(
+            'formulaire_charger',
+            array(
+                'args'=>array('form'=>$form,'args'=>$args,'je_suis_poste'=>false),
+                'data'=>$valeurs)
+        );
+        // ne pas stocker editable !
+        unset($valeurs['editable']);
+
+        // recuperer les valeurs postees
+        $store = array();
+        foreach($valeurs as $k=>$v){
+            if (substr($k,0,1)!=='_')
+                $store[$k] = _request($k);
+        }
+
+        return cvtconf_configurer_stocker($form,$valeurs,$store);
 }
 
 /**
@@ -113,25 +113,25 @@ 
  * @return array
  */
 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);
-		$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'] . '::';
+        // stocker en base
+        // par defaut, dans un casier serialize dans spip_meta (idem CFG)
+        $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'] . '::';
 			
-		// 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']:'');
-		}
+        // 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']:'');
+        }
 	
-		return array($table,$casier,$prefixe,$stockage);
+        return array($table,$casier,$prefixe,$stockage);
 }
 
 /**
@@ -142,39 +142,39 @@ 
  * @return array
  */
 function cvtconf_formulaires_configurer_recense($form){
-	$valeurs = array('editable'=>' ');
-
-	// sinon cas analyse du squelette
-	if ($f = find_in_path($form.'.' . _EXTENSION_SQUELETTES, 'formulaires/')
-		AND lire_fichier($f, $contenu)) {
-
-		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);
-
-			$balises = array_merge(extraire_balises($contenu,'input'),
-				extraire_balises($contenu,'textarea'),
-				extraire_balises($contenu,'select'));
-
-			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') {
-						$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');
-					}
-			}
-		}
-	}
-
-
-	cvtconf_configurer_lire_meta($form,$valeurs);
-	return $valeurs;
+    $valeurs = array('editable'=>' ');
+
+    // sinon cas analyse du squelette
+    if ($f = find_in_path($form.'.' . _EXTENSION_SQUELETTES, 'formulaires/')
+        AND lire_fichier($f, $contenu)) {
+
+        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);
+
+            $balises = array_merge(extraire_balises($contenu,'input'),
+                extraire_balises($contenu,'textarea'),
+                extraire_balises($contenu,'select'));
+
+            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') {
+                        $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');
+                    }
+            }
+        }
+    }
+
+
+    cvtconf_configurer_lire_meta($form,$valeurs);
+    return $valeurs;
 }
 
 /**
@@ -185,24 +185,24 @@ 
  * @return string
  */
 function cvtconf_configurer_stocker($form,$valeurs,$store) {
-	$trace = '';
-	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.'_':'');
-	$table = ($table) ? "/$table/" : ""; 
-	$casier = ($casier) ? rtrim($casier,'/').'/' : ""; // slash final, sinon rien
+    $trace = '';
+    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.'_':'');
+    $table = ($table) ? "/$table/" : ""; 
+    $casier = ($casier) ? rtrim($casier,'/').'/' : ""; // slash final, sinon rien
 	
-	foreach($store as $k=>$v){
-		ecrire_config("$stockage$table$prefixe$casier$k", $v);
-		if (_request('var_mode')=='configurer' AND autoriser('webmestre')){
-			$trace .= "<br />table $table : ".$prefixe.$k." = $v;";
-		}
-	}
-	return $trace;
+    foreach($store as $k=>$v){
+        ecrire_config("$stockage$table$prefixe$casier$k", $v);
+        if (_request('var_mode')=='configurer' AND autoriser('webmestre')){
+            $trace .= "<br />table $table : ".$prefixe.$k." = $v;";
+        }
+    }
+    return $trace;
 }
 
 /**
@@ -211,23 +211,23 @@ 
  * @param array $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.'_':'');
-	if ($casier) {
-		$meta = lire_config("$stockage$table$prefixe$casier");
-		$prefixe = '';
-	}
-	else {
-		$table = rtrim($table, '/');
-		$meta = lire_config("$stockage$table");
-	}
-
-	foreach($valeurs as $k=>$v){
-		if (substr($k,0,1)!=='_')
-		$valeurs[$k] = (isset($meta[$prefixe.$k])?$meta[$prefixe.$k]:'');
-	}
+    list($table,$casier,$prefixe,$stockage) = cvtconf_definir_configurer_conteneur($form,$valeurs);
+
+    $table = ($table) ? "/$table/" : ""; 
+    $prefixe = ($prefixe?$prefixe.'_':'');
+    if ($casier) {
+        $meta = lire_config("$stockage$table$prefixe$casier");
+        $prefixe = '';
+    }
+    else {
+        $table = rtrim($table, '/');
+        $meta = lire_config("$stockage$table");
+    }
+
+    foreach($valeurs as $k=>$v){
+        if (substr($k,0,1)!=='_')
+        $valeurs[$k] = (isset($meta[$prefixe.$k])?$meta[$prefixe.$k]:'');
+    }
 }
 
 

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] : '');
 	}
 }
 

Braces +34 added lines, -18 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/config');
 
@@ -36,7 +38,9 @@ 
 			$flux['data'] = cvtconf_formulaires_configurer_recense($form);
 			$flux['data']['editable'] = true;
 			if (_request('var_mode')=='configurer' AND autoriser('webmestre')){
-				if (!_AJAX) var_dump($flux['data']);
+				if (!_AJAX) {
+				    var_dump($flux['data']);
+				}
 				// reinjecter pour la trace au traitement
 				$flux['data']['_hidden'] = "<input type='hidden' name='var_mode' value='configurer' />";
 			}
@@ -79,8 +83,9 @@ 
 		// 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(
@@ -93,8 +98,9 @@ 
 		// recuperer les valeurs postees
 		$store = array();
 		foreach($valeurs as $k=>$v){
-			if (substr($k,0,1)!=='_')
-				$store[$k] = _request($k);
+			if (substr($k,0,1)!=='_') {
+							$store[$k] = _request($k);
+			}
 		}
 
 		return cvtconf_configurer_stocker($form,$valeurs,$store);
@@ -120,9 +126,15 @@ 
 		$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_casier'])) {
+		    $casier   = $valeurs['_meta_casier'];
+		}
+		if (isset($valeurs['_meta_prefixe'])) {
+		    $prefixe  = $valeurs['_meta_prefixe'];
+		}
+		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
@@ -151,7 +163,9 @@ 
 		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'),
@@ -165,8 +179,9 @@ 
 						$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');
+						}
 					}
 			}
 		}
@@ -189,8 +204,9 @@ 
 	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);
+	if (!isset($GLOBALS[$table])) {
+			lire_metas($table);
+	}
 
 	$prefixe = ($prefixe?$prefixe.'_':'');
 	$table = ($table) ? "/$table/" : ""; 
@@ -218,15 +234,15 @@ 
 	if ($casier) {
 		$meta = lire_config("$stockage$table$prefixe$casier");
 		$prefixe = '';
-	}
-	else {
+	} else {
 		$table = rtrim($table, '/');
 		$meta = lire_config("$stockage$table");
 	}
 
 	foreach($valeurs as $k=>$v){
-		if (substr($k,0,1)!=='_')
-		$valeurs[$k] = (isset($meta[$prefixe.$k])?$meta[$prefixe.$k]:'');
+		if (substr($k,0,1)!=='_') {
+				$valeurs[$k] = (isset($meta[$prefixe.$k])?$meta[$prefixe.$k]:'');
+		}
 	}
 }
 

ecrire/inc/notifications.php

Indentation +99 added lines, -99 removed lines

@@ -26,16 +26,16 @@ 
  */
 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)));
-
-	if ($notification = charger_fonction($quoi,'notifications',true)) {
-		spip_log("$notification($quoi,$id"
-			.($options?",".serialize($options):"")
-			.")",'notifications');
-		$notification($quoi, $id, $options);
-	}
+    // 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)));
+
+    if ($notification = charger_fonction($quoi,'notifications',true)) {
+        spip_log("$notification($quoi,$id"
+            .($options?",".serialize($options):"")
+            .")",'notifications');
+        $notification($quoi, $id, $options);
+    }
 }
 
 /**
@@ -48,15 +48,15 @@ 
  * @param array $exclure
  */
 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)){
-		// nettoyer les exclusions d'abord
-		notifications_nettoyer_emails($exclure);
-		// faire un diff
-		$emails = array_diff($emails,$exclure);
-	}
+    // 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)){
+        // nettoyer les exclusions d'abord
+        notifications_nettoyer_emails($exclure);
+        // faire un diff
+        $emails = array_diff($emails,$exclure);
+    }
 }
 
 /**
@@ -70,71 +70,71 @@ 
  * @param string $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);
-
-	notifications_nettoyer_emails($emails);
-
-	// tester si le mail est deja en html
-	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){
-
-		if(!strlen($sujet)){
-			// dans ce cas on ruse un peu : extraire le sujet du title
-			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))
-					$ttrim = $m[1];
-
-				// et on extrait la premiere ligne de vrai texte...
-				// nettoyer le html et les retours chariots
-				$ttrim = textebrut($ttrim);
-				$ttrim = str_replace("\r\n", "\r", $ttrim);
-				$ttrim = str_replace("\r", "\n", $ttrim);
-				// decouper
-				$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)
-			$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)){
-		// 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));
-		// extraire la premiere ligne
-		$sujet = array_shift($texte);
-		$texte = trim(implode("\n",$texte));
-	}
-
-	$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));
-		$email = $envoi['email'];
-
-		job_queue_add('envoyer_mail', ">$email : ".$envoi['sujet'], array($email, $envoi['sujet'], $envoi['texte'], $from, $headers), 'inc/');
-	}
+    // 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);
+
+    notifications_nettoyer_emails($emails);
+
+    // tester si le mail est deja en html
+    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){
+
+        if(!strlen($sujet)){
+            // dans ce cas on ruse un peu : extraire le sujet du title
+            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))
+                    $ttrim = $m[1];
+
+                // et on extrait la premiere ligne de vrai texte...
+                // nettoyer le html et les retours chariots
+                $ttrim = textebrut($ttrim);
+                $ttrim = str_replace("\r\n", "\r", $ttrim);
+                $ttrim = str_replace("\r", "\n", $ttrim);
+                // decouper
+                $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)
+            $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)){
+        // 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));
+        // extraire la premiere ligne
+        $sujet = array_shift($texte);
+        $texte = trim(implode("\n",$texte));
+    }
+
+    $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));
+        $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
-	$id_type = id_table_objet($type_objet);
-	return recuperer_fond($modele,array($id_type=>$id_objet,"id"=>$id_objet));
+    $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));
 }
 
 /**
@@ -166,29 +166,29 @@ 
  * @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
 // http://doc.spip.org/@notifier_publication_article
 function notifier_publication_article($id_article) {
-	if ($GLOBALS['meta']["suivi_edito"] == "oui") {
-		$adresse_suivi = $GLOBALS['meta']["adresse_suivi"];
-		$texte = email_notification_article($id_article, "notifications/article_publie");
-		notifications_envoyer_mails($adresse_suivi, $texte);
-	}
+    if ($GLOBALS['meta']["suivi_edito"] == "oui") {
+        $adresse_suivi = $GLOBALS['meta']["adresse_suivi"];
+        $texte = email_notification_article($id_article, "notifications/article_publie");
+        notifications_envoyer_mails($adresse_suivi, $texte);
+    }
 }
 
 // Compatibilite, ne plus utiliser
 // http://doc.spip.org/@notifier_proposition_article
 function notifier_proposition_article($id_article) {
-	if ($GLOBALS['meta']["suivi_edito"] == "oui") {
-		$adresse_suivi = $GLOBALS['meta']["adresse_suivi"];
-		$texte = email_notification_article($id_article, "notifications/article_propose");
-		notifications_envoyer_mails($adresse_suivi, $texte);
-	}
+    if ($GLOBALS['meta']["suivi_edito"] == "oui") {
+        $adresse_suivi = $GLOBALS['meta']["adresse_suivi"];
+        $texte = email_notification_article($id_article, "notifications/article_propose");
+        notifications_envoyer_mails($adresse_suivi, $texte);
+    }
 }
 
 ?>

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

Braces +18 added lines, -12 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;
+}
 
 
 /**
@@ -71,12 +73,14 @@ 
  */
 function notifications_envoyer_mails($emails, $texte, $sujet="", $from = "", $headers = ""){
 	// rien a faire si pas de texte !
-	if (!strlen($texte))
-		return;
+	if (!strlen($texte)) {
+			return;
+	}
 
 	// si on ne specifie qu'un email, le mettre dans un tableau
-	if (!is_array($emails))
-		$emails = explode(',',$emails);
+	if (!is_array($emails)) {
+			$emails = explode(',',$emails);
+	}
 
 	notifications_nettoyer_emails($emails);
 
@@ -89,12 +93,13 @@ 
 
 		if(!strlen($sujet)){
 			// dans ce cas on ruse un peu : extraire le sujet du title
-			if (preg_match(",<title>(.*)</title>,Uims",$texte,$m))
-				$sujet = $m[1];
-			else {
+			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))
-					$ttrim = $m[1];
+				if (preg_match(",<body[^>]*>(.*)</body>,Uims",$texte,$m)) {
+									$ttrim = $m[1];
+				}
 
 				// et on extrait la premiere ligne de vrai texte...
 				// nettoyer le html et les retours chariots
@@ -109,8 +114,9 @@ 
 		}
 
 		// si besoin on ajoute le content-type dans les headers
-		if (stripos($headers,"Content-Type")===false)
-			$headers .= "Content-Type: text/html\n";
+		if (stripos($headers,"Content-Type")===false) {
+					$headers .= "Content-Type: text/html\n";
+		}
 	}
 
 	// si le sujet est vide, extraire la premiere ligne du corps

ecrire/inc/csv.php

Indentation +66 added lines, -66 removed lines

@@ -14,76 +14,76 @@
 
 function analyse_csv($t)
 {
-	$virg = substr_count($t, ',');
-	$pvirg = substr_count($t, ';');
-	$tab = substr_count($t, "\t");
-	if ($virg > $pvirg)
-		{ $sep = ','; $hs = ',';}
-	else	{ $sep = ';'; $hs = '&#59;'; $virg = $pvirg;}
-	if ($tab > $virg) {$sep = "\t"; $hs = "\t";}
+    $virg = substr_count($t, ',');
+    $pvirg = substr_count($t, ';');
+    $tab = substr_count($t, "\t");
+    if ($virg > $pvirg)
+        { $sep = ','; $hs = ',';}
+    else	{ $sep = ';'; $hs = '&#59;'; $virg = $pvirg;}
+    if ($tab > $virg) {$sep = "\t"; $hs = "\t";}
 
-	$t = preg_replace('/\r?\n/', "\n",
-				      preg_replace('/[\r\n]+/', "\n", $t));
-	// un separateur suivi de 3 guillemets attention !
-	// attention au ; suceptible d'etre confondu avec un separateur
-	// on substitue un # et on remplacera a la fin
-	$t = preg_replace("/([\n$sep])\"\"\"/",'\\1"&#34#',$t);
-	$t = str_replace('""','&#34#',$t);
-	preg_match_all('/"[^"]*"/', $t, $r);
-	foreach($r[0] as $cell)
-		$t = str_replace($cell,
-			str_replace($sep, $hs,
-				str_replace("\n", "<br />",
-					    substr($cell,1,-1))),
-			$t);
-	list($entete, $corps) = explode("\n",$t,2);
-	$caption = '';
-	// sauter la ligne de tete formee seulement de separateurs
-	if (substr_count($entete, $sep) == strlen($entete)) {
-		list($entete, $corps) = explode("\n",$corps,2);
-	}
-	// si une seule colonne, en faire le titre
-	if (preg_match("/^([^$sep]+)$sep+\$/", $entete, $l)) {
-			$caption = "\n||" .  $l[1] . "|";
-			list($entete, $corps) = explode("\n",$corps,2);
-	}
-	// si premiere colonne vide, le raccourci doit quand meme produire <th...
-	if ($entete[0] == $sep) $entete = ' ' . $entete;
+    $t = preg_replace('/\r?\n/', "\n",
+                        preg_replace('/[\r\n]+/', "\n", $t));
+    // un separateur suivi de 3 guillemets attention !
+    // attention au ; suceptible d'etre confondu avec un separateur
+    // on substitue un # et on remplacera a la fin
+    $t = preg_replace("/([\n$sep])\"\"\"/",'\\1"&#34#',$t);
+    $t = str_replace('""','&#34#',$t);
+    preg_match_all('/"[^"]*"/', $t, $r);
+    foreach($r[0] as $cell)
+        $t = str_replace($cell,
+            str_replace($sep, $hs,
+                str_replace("\n", "<br />",
+                        substr($cell,1,-1))),
+            $t);
+    list($entete, $corps) = explode("\n",$t,2);
+    $caption = '';
+    // sauter la ligne de tete formee seulement de separateurs
+    if (substr_count($entete, $sep) == strlen($entete)) {
+        list($entete, $corps) = explode("\n",$corps,2);
+    }
+    // si une seule colonne, en faire le titre
+    if (preg_match("/^([^$sep]+)$sep+\$/", $entete, $l)) {
+            $caption = "\n||" .  $l[1] . "|";
+            list($entete, $corps) = explode("\n",$corps,2);
+    }
+    // si premiere colonne vide, le raccourci doit quand meme produire <th...
+    if ($entete[0] == $sep) $entete = ' ' . $entete;
 
-	$lignes = explode("\n", $corps);
+    $lignes = explode("\n", $corps);
 
-	// retrait des lignes vides finales
-	while(count($lignes) > 0
-	AND preg_match("/^$sep*$/", $lignes[count($lignes)-1]))
-	  unset($lignes[count($lignes)-1]);
-	//  calcul du  nombre de colonne a chaque ligne
-	$nbcols = array();
-	$max = $mil = substr_count($entete, $sep);
-	foreach($lignes as $k=>$v) {
-	  if ($max <> ($nbcols[$k]= substr_count($v, $sep))) {
-	    if ($max > $nbcols[$k])
-	      $mil = $nbcols[$k];
-	    else { $mil = $max; $max = $nbcols[$k];}
-	  }
-	}
-	// Si pas le meme nombre, cadrer au nombre max
-	if ($mil <> $max)
-	  foreach($nbcols as $k=>$v) {
-	    if ($v < $max) $lignes[$k].= str_repeat($sep, $max-$v);
-	    }
-	// et retirer les colonnes integralement vides
-	while(true) {
-	  $nbcols =  ($entete[strlen($entete)-1]===$sep);
-	  foreach($lignes as $v) $nbcols &= ($v[strlen($v)-1]===$sep);
-	  if (!$nbcols) break;
-	  $entete = substr($entete,0,-1);
-	  foreach($lignes as $k=>$v) $lignes[$k] = substr($v,0,-1);
-	}
+    // retrait des lignes vides finales
+    while(count($lignes) > 0
+    AND preg_match("/^$sep*$/", $lignes[count($lignes)-1]))
+        unset($lignes[count($lignes)-1]);
+    //  calcul du  nombre de colonne a chaque ligne
+    $nbcols = array();
+    $max = $mil = substr_count($entete, $sep);
+    foreach($lignes as $k=>$v) {
+        if ($max <> ($nbcols[$k]= substr_count($v, $sep))) {
+        if ($max > $nbcols[$k])
+            $mil = $nbcols[$k];
+        else { $mil = $max; $max = $nbcols[$k];}
+        }
+    }
+    // Si pas le meme nombre, cadrer au nombre max
+    if ($mil <> $max)
+        foreach($nbcols as $k=>$v) {
+        if ($v < $max) $lignes[$k].= str_repeat($sep, $max-$v);
+        }
+    // et retirer les colonnes integralement vides
+    while(true) {
+        $nbcols =  ($entete[strlen($entete)-1]===$sep);
+        foreach($lignes as $v) $nbcols &= ($v[strlen($v)-1]===$sep);
+        if (!$nbcols) break;
+        $entete = substr($entete,0,-1);
+        foreach($lignes as $k=>$v) $lignes[$k] = substr($v,0,-1);
+    }
 
-	foreach($lignes as &$l) {
-		$l = explode($sep, $l);
-	}
-	return array(explode($sep, $entete), $lignes);
+    foreach($lignes as &$l) {
+        $l = explode($sep, $l);
+    }
+    return array(explode($sep, $entete), $lignes);
 }
 
 

Spacing +26 added lines, -26 removed lines

@@ -18,69 +18,69 @@
 	$pvirg = substr_count($t, ';');
 	$tab = substr_count($t, "\t");
 	if ($virg > $pvirg)
-		{ $sep = ','; $hs = ',';}
-	else	{ $sep = ';'; $hs = '&#59;'; $virg = $pvirg;}
-	if ($tab > $virg) {$sep = "\t"; $hs = "\t";}
+		{ $sep = ','; $hs = ','; }
+	else { $sep = ';'; $hs = '&#59;'; $virg = $pvirg; }
+	if ($tab > $virg) {$sep = "\t"; $hs = "\t"; }
 
 	$t = preg_replace('/\r?\n/', "\n",
 				      preg_replace('/[\r\n]+/', "\n", $t));
 	// un separateur suivi de 3 guillemets attention !
 	// attention au ; suceptible d'etre confondu avec un separateur
 	// on substitue un # et on remplacera a la fin
-	$t = preg_replace("/([\n$sep])\"\"\"/",'\\1"&#34#',$t);
-	$t = str_replace('""','&#34#',$t);
+	$t = preg_replace("/([\n$sep])\"\"\"/", '\\1"&#34#', $t);
+	$t = str_replace('""', '&#34#', $t);
 	preg_match_all('/"[^"]*"/', $t, $r);
-	foreach($r[0] as $cell)
+	foreach ($r[0] as $cell)
 		$t = str_replace($cell,
 			str_replace($sep, $hs,
 				str_replace("\n", "<br />",
-					    substr($cell,1,-1))),
+					    substr($cell, 1, -1))),
 			$t);
-	list($entete, $corps) = explode("\n",$t,2);
+	list($entete, $corps) = explode("\n", $t, 2);
 	$caption = '';
 	// sauter la ligne de tete formee seulement de separateurs
 	if (substr_count($entete, $sep) == strlen($entete)) {
-		list($entete, $corps) = explode("\n",$corps,2);
+		list($entete, $corps) = explode("\n", $corps, 2);
 	}
 	// si une seule colonne, en faire le titre
 	if (preg_match("/^([^$sep]+)$sep+\$/", $entete, $l)) {
-			$caption = "\n||" .  $l[1] . "|";
-			list($entete, $corps) = explode("\n",$corps,2);
+			$caption = "\n||".$l[1]."|";
+			list($entete, $corps) = explode("\n", $corps, 2);
 	}
 	// si premiere colonne vide, le raccourci doit quand meme produire <th...
-	if ($entete[0] == $sep) $entete = ' ' . $entete;
+	if ($entete[0] == $sep) $entete = ' '.$entete;
 
 	$lignes = explode("\n", $corps);
 
 	// retrait des lignes vides finales
-	while(count($lignes) > 0
-	AND preg_match("/^$sep*$/", $lignes[count($lignes)-1]))
-	  unset($lignes[count($lignes)-1]);
+	while (count($lignes) > 0
+	AND preg_match("/^$sep*$/", $lignes[count($lignes) - 1]))
+	  unset($lignes[count($lignes) - 1]);
 	//  calcul du  nombre de colonne a chaque ligne
 	$nbcols = array();
 	$max = $mil = substr_count($entete, $sep);
-	foreach($lignes as $k=>$v) {
-	  if ($max <> ($nbcols[$k]= substr_count($v, $sep))) {
+	foreach ($lignes as $k=>$v) {
+	  if ($max <> ($nbcols[$k] = substr_count($v, $sep))) {
 	    if ($max > $nbcols[$k])
 	      $mil = $nbcols[$k];
-	    else { $mil = $max; $max = $nbcols[$k];}
+	    else { $mil = $max; $max = $nbcols[$k]; }
 	  }
 	}
 	// Si pas le meme nombre, cadrer au nombre max
 	if ($mil <> $max)
-	  foreach($nbcols as $k=>$v) {
-	    if ($v < $max) $lignes[$k].= str_repeat($sep, $max-$v);
+	  foreach ($nbcols as $k=>$v) {
+	    if ($v < $max) $lignes[$k] .= str_repeat($sep, $max - $v);
 	    }
 	// et retirer les colonnes integralement vides
-	while(true) {
-	  $nbcols =  ($entete[strlen($entete)-1]===$sep);
-	  foreach($lignes as $v) $nbcols &= ($v[strlen($v)-1]===$sep);
+	while (true) {
+	  $nbcols = ($entete[strlen($entete) - 1] === $sep);
+	  foreach ($lignes as $v) $nbcols &= ($v[strlen($v) - 1] === $sep);
 	  if (!$nbcols) break;
-	  $entete = substr($entete,0,-1);
-	  foreach($lignes as $k=>$v) $lignes[$k] = substr($v,0,-1);
+	  $entete = substr($entete, 0, -1);
+	  foreach ($lignes as $k=>$v) $lignes[$k] = substr($v, 0, -1);
 	}
 
-	foreach($lignes as &$l) {
+	foreach ($lignes as &$l) {
 		$l = explode($sep, $l);
 	}
 	return array(explode($sep, $entete), $lignes);

Braces +28 added lines, -16 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 function analyse_csv($t)
 {
@@ -18,8 +20,7 @@ 
 	$pvirg = substr_count($t, ';');
 	$tab = substr_count($t, "\t");
 	if ($virg > $pvirg)
-		{ $sep = ','; $hs = ',';}
-	else	{ $sep = ';'; $hs = '&#59;'; $virg = $pvirg;}
+		{ $sep = ','; $hs = ',';} else	{ $sep = ';'; $hs = '&#59;'; $virg = $pvirg;}
 	if ($tab > $virg) {$sep = "\t"; $hs = "\t";}
 
 	$t = preg_replace('/\r?\n/', "\n",
@@ -30,12 +31,13 @@ 
 	$t = preg_replace("/([\n$sep])\"\"\"/",'\\1"&#34#',$t);
 	$t = str_replace('""','&#34#',$t);
 	preg_match_all('/"[^"]*"/', $t, $r);
-	foreach($r[0] as $cell)
-		$t = str_replace($cell,
+	foreach($r[0] as $cell) {
+			$t = str_replace($cell,
 			str_replace($sep, $hs,
 				str_replace("\n", "<br />",
 					    substr($cell,1,-1))),
 			$t);
+	}
 	list($entete, $corps) = explode("\n",$t,2);
 	$caption = '';
 	// sauter la ligne de tete formee seulement de separateurs
@@ -48,36 +50,46 @@ 
 			list($entete, $corps) = explode("\n",$corps,2);
 	}
 	// si premiere colonne vide, le raccourci doit quand meme produire <th...
-	if ($entete[0] == $sep) $entete = ' ' . $entete;
+	if ($entete[0] == $sep) {
+	    $entete = ' ' . $entete;
+	}
 
 	$lignes = explode("\n", $corps);
 
 	// retrait des lignes vides finales
 	while(count($lignes) > 0
-	AND preg_match("/^$sep*$/", $lignes[count($lignes)-1]))
-	  unset($lignes[count($lignes)-1]);
+	AND preg_match("/^$sep*$/", $lignes[count($lignes)-1])) {
+		  unset($lignes[count($lignes)-1]);
+	}
 	//  calcul du  nombre de colonne a chaque ligne
 	$nbcols = array();
 	$max = $mil = substr_count($entete, $sep);
 	foreach($lignes as $k=>$v) {
 	  if ($max <> ($nbcols[$k]= substr_count($v, $sep))) {
-	    if ($max > $nbcols[$k])
-	      $mil = $nbcols[$k];
-	    else { $mil = $max; $max = $nbcols[$k];}
+	    if ($max > $nbcols[$k]) {
+	    	      $mil = $nbcols[$k];
+	    } else { $mil = $max; $max = $nbcols[$k];}
 	  }
 	}
 	// Si pas le meme nombre, cadrer au nombre max
-	if ($mil <> $max)
-	  foreach($nbcols as $k=>$v) {
+	if ($mil <> $max) {
+		  foreach($nbcols as $k=>$v) {
 	    if ($v < $max) $lignes[$k].= str_repeat($sep, $max-$v);
+	}
 	    }
 	// et retirer les colonnes integralement vides
 	while(true) {
 	  $nbcols =  ($entete[strlen($entete)-1]===$sep);
-	  foreach($lignes as $v) $nbcols &= ($v[strlen($v)-1]===$sep);
-	  if (!$nbcols) break;
+	  foreach($lignes as $v) {
+	      $nbcols &= ($v[strlen($v)-1]===$sep);
+	  }
+	  if (!$nbcols) {
+	      break;
+	  }
 	  $entete = substr($entete,0,-1);
-	  foreach($lignes as $k=>$v) $lignes[$k] = substr($v,0,-1);
+	  foreach($lignes as $k=>$v) {
+	      $lignes[$k] = substr($v,0,-1);
+	  }
 	}
 
 	foreach($lignes as &$l) {

ecrire/inc/lien_court.php

Indentation +6 added lines, -6 removed lines

@@ -20,14 +20,14 @@
  * http://zoumzamzouilam/truc/chose/machin..."
  */
 function inc_lien_court($url) {
-	$long_url = defined('_MAX_LONG_URL') ? _MAX_LONG_URL : 40;
-	$coupe_url = defined('_MAX_COUPE_URL') ? _MAX_COUPE_URL : 35;
+    $long_url = defined('_MAX_LONG_URL') ? _MAX_LONG_URL : 40;
+    $coupe_url = defined('_MAX_COUPE_URL') ? _MAX_COUPE_URL : 35;
 
-	if (strlen($url)>$long_url) {
-		$url = substr($url,0,$coupe_url).'...';
-	}
+    if (strlen($url)>$long_url) {
+        $url = substr($url,0,$coupe_url).'...';
+    }
 
-	return $url;
+    return $url;
 }
 
 ?>
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -23,8 +23,8 @@
 	$long_url = defined('_MAX_LONG_URL') ? _MAX_LONG_URL : 40;
 	$coupe_url = defined('_MAX_COUPE_URL') ? _MAX_COUPE_URL : 35;
 
-	if (strlen($url)>$long_url) {
-		$url = substr($url,0,$coupe_url).'...';
+	if (strlen($url) > $long_url) {
+		$url = substr($url, 0, $coupe_url).'...';
 	}
 
 	return $url;

Braces +3 added lines, -1 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;
+}
 
 /*
  * Cette fonction prend une URL et la raccourcit si elle est trop longue

ecrire/inc/informer.php

Braces +12 added lines, -7 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 # Les information d'une rubrique selectionnee dans le mini navigateur
 
@@ -26,19 +28,22 @@ 
 		} else {
 			$titre = _T('info_racine_site');
 		}
-	} else
-		$titre = '';
+	} else {
+			$titre = '';
+	}
 
 	$res = '';
-	if ($type == "rubrique" AND $GLOBALS['spip_display'] != 1 AND isset($GLOBALS['meta']['image_process']))
-	  if ($GLOBALS['meta']['image_process'] != "non") {
+	if ($type == "rubrique" AND $GLOBALS['spip_display'] != 1 AND isset($GLOBALS['meta']['image_process'])) {
+		  if ($GLOBALS['meta']['image_process'] != "non") {
 		$chercher_logo = charger_fonction('chercher_logo', 'inc');
+	}
 		if ($res = $chercher_logo($id, 'id_rubrique', 'on'))  {
 			list($fid, $dir, $nom, $format) = $res;
 			include_spip('inc/filtres_images_mini');
 			$res = image_reduire("<img src='$fid' alt='' />", 100, 48);
-			if ($res)
-				$res =  "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
+			if ($res) {
+							$res =  "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
+			}
 		}
 	}
 

Indentation +46 added lines, -46 removed lines

@@ -17,57 +17,57 @@
 // http://doc.spip.org/@inc_informer_dist
 function inc_informer_dist($id, $col, $exclus, $rac, $type, $do='aff')
 {
-	include_spip('inc/texte');
-	if ($type == "rubrique") {
-		$row = sql_fetsel("titre, descriptif", "spip_rubriques", "id_rubrique = " . intval($id));
-		if ($row) {
-			$titre = typo($row["titre"]);
-			$descriptif = propre($row["descriptif"]);
-		} else {
-			$titre = _T('info_racine_site');
-		}
-	} else
-		$titre = '';
+    include_spip('inc/texte');
+    if ($type == "rubrique") {
+        $row = sql_fetsel("titre, descriptif", "spip_rubriques", "id_rubrique = " . intval($id));
+        if ($row) {
+            $titre = typo($row["titre"]);
+            $descriptif = propre($row["descriptif"]);
+        } else {
+            $titre = _T('info_racine_site');
+        }
+    } else
+        $titre = '';
 
-	$res = '';
-	if ($type == "rubrique" AND $GLOBALS['spip_display'] != 1 AND isset($GLOBALS['meta']['image_process']))
-	  if ($GLOBALS['meta']['image_process'] != "non") {
-		$chercher_logo = charger_fonction('chercher_logo', 'inc');
-		if ($res = $chercher_logo($id, 'id_rubrique', 'on'))  {
-			list($fid, $dir, $nom, $format) = $res;
-			include_spip('inc/filtres_images_mini');
-			$res = image_reduire("<img src='$fid' alt='' />", 100, 48);
-			if ($res)
-				$res =  "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
-		}
-	}
+    $res = '';
+    if ($type == "rubrique" AND $GLOBALS['spip_display'] != 1 AND isset($GLOBALS['meta']['image_process']))
+        if ($GLOBALS['meta']['image_process'] != "non") {
+        $chercher_logo = charger_fonction('chercher_logo', 'inc');
+        if ($res = $chercher_logo($id, 'id_rubrique', 'on'))  {
+            list($fid, $dir, $nom, $format) = $res;
+            include_spip('inc/filtres_images_mini');
+            $res = image_reduire("<img src='$fid' alt='' />", 100, 48);
+            if ($res)
+                $res =  "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
+        }
+    }
 
-	$rac = spip_htmlentities($rac, ENT_QUOTES);
-	$do = spip_htmlentities($do, ENT_QUOTES);
-	$id = intval($id);
+    $rac = spip_htmlentities($rac, ENT_QUOTES);
+    $do = spip_htmlentities($do, ENT_QUOTES);
+    $id = intval($id);
 
 # ce lien provoque la selection (directe) de la rubrique cliquee
 # et l'affichage de son titre dans le bandeau
-	$titre = strtr(str_replace("'", "&#8217;",
-			str_replace('"', "&#34;", textebrut($titre))),
-		       "\n\r", "  ");
+    $titre = strtr(str_replace("'", "&#8217;",
+            str_replace('"', "&#34;", textebrut($titre))),
+                "\n\r", "  ");
 
-	$js_func = $do . '_selection_titre';
-	return "<div style='display: none;'>"
-	. "<input type='text' id='".$rac."_sel' value='$id' />"
-	. "<input type='text' id='".$rac."_sel2' value=\""
-	. entites_html($titre)
-	. "\" />"
-	. "</div>"
-	. "<div class='informer' style='padding: 5px; border-top: 0px;'>"
-	. (!$res ? '' : $res)
-	. "<p><b>".safehtml($titre)."</b></p>"
-	. (!$descriptif ? '' : "<div>".safehtml($descriptif)."</div>")
-	. "<div style='text-align: ".$GLOBALS['spip_lang_right'].";'>"
-	. "<input type='submit' class='fondo' value='"
-	. _T('bouton_choisir')
-	. "'\nonclick=\"$js_func('$titre',$id,'selection_rubrique','id_parent'); return false;\" />"
-	.  "</div>"
-	.  "</div>";
+    $js_func = $do . '_selection_titre';
+    return "<div style='display: none;'>"
+    . "<input type='text' id='".$rac."_sel' value='$id' />"
+    . "<input type='text' id='".$rac."_sel2' value=\""
+    . entites_html($titre)
+    . "\" />"
+    . "</div>"
+    . "<div class='informer' style='padding: 5px; border-top: 0px;'>"
+    . (!$res ? '' : $res)
+    . "<p><b>".safehtml($titre)."</b></p>"
+    . (!$descriptif ? '' : "<div>".safehtml($descriptif)."</div>")
+    . "<div style='text-align: ".$GLOBALS['spip_lang_right'].";'>"
+    . "<input type='submit' class='fondo' value='"
+    . _T('bouton_choisir')
+    . "'\nonclick=\"$js_func('$titre',$id,'selection_rubrique','id_parent'); return false;\" />"
+    .  "</div>"
+    .  "</div>";
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -15,11 +15,11 @@ 
 # Les information d'une rubrique selectionnee dans le mini navigateur
 
 // http://doc.spip.org/@inc_informer_dist
-function inc_informer_dist($id, $col, $exclus, $rac, $type, $do='aff')
+function inc_informer_dist($id, $col, $exclus, $rac, $type, $do = 'aff')
 {
 	include_spip('inc/texte');
 	if ($type == "rubrique") {
-		$row = sql_fetsel("titre, descriptif", "spip_rubriques", "id_rubrique = " . intval($id));
+		$row = sql_fetsel("titre, descriptif", "spip_rubriques", "id_rubrique = ".intval($id));
 		if ($row) {
 			$titre = typo($row["titre"]);
 			$descriptif = propre($row["descriptif"]);
@@ -33,12 +33,12 @@ 
 	if ($type == "rubrique" AND $GLOBALS['spip_display'] != 1 AND isset($GLOBALS['meta']['image_process']))
 	  if ($GLOBALS['meta']['image_process'] != "non") {
 		$chercher_logo = charger_fonction('chercher_logo', 'inc');
-		if ($res = $chercher_logo($id, 'id_rubrique', 'on'))  {
+		if ($res = $chercher_logo($id, 'id_rubrique', 'on')) {
 			list($fid, $dir, $nom, $format) = $res;
 			include_spip('inc/filtres_images_mini');
 			$res = image_reduire("<img src='$fid' alt='' />", 100, 48);
 			if ($res)
-				$res =  "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
+				$res = "<div style='float: ".$GLOBALS['spip_lang_right']."; margin-".$GLOBALS['spip_lang_right'].": -5px; margin-top: -5px;'>$res</div>";
 		}
 	}
 
@@ -52,7 +52,7 @@ 
 			str_replace('"', "&#34;", textebrut($titre))),
 		       "\n\r", "  ");
 
-	$js_func = $do . '_selection_titre';
+	$js_func = $do.'_selection_titre';
 	return "<div style='display: none;'>"
 	. "<input type='text' id='".$rac."_sel' value='$id' />"
 	. "<input type='text' id='".$rac."_sel2' value=\""