spip / SPIP

ecrire/exec/fond_monobloc.php

Spacing +19 added lines, -19 removed lines

@@ -20,13 +20,13 @@ 
  * deprecie, ne plus utiliser
  *
  */
-function exec_fond_monobloc_dist(){
+function exec_fond_monobloc_dist() {
 
 	// pas d'autorisation
 	// c'est au fond de les gerer avec #AUTORISER, et de renvoyer un fond vide le cas echeant
 	// qui declenchera un minipres acces interdit
 	$exec = _request('exec');
-	$fond = trim(recuperer_fond("prive/exec/$exec",$_REQUEST));
+	$fond = trim(recuperer_fond("prive/exec/$exec", $_REQUEST));
 	if (!$fond) {
 		include_spip('inc/minipres');
 		echo minipres();
@@ -38,33 +38,33 @@ 
 	$extra = "";
 
 	// recuperer le titre dans le premier hn de la page
-	if (preg_match(",<h[1-6][^>]*>(.+)</h[1-6]>,Uims",$fond,$match)){
+	if (preg_match(",<h[1-6][^>]*>(.+)</h[1-6]>,Uims", $fond, $match)) {
 		$titre = $match[1];
 	}
 
 	// recuperer la hierarchie (au-dessus du contenu)
-	if (preg_match(",<!--#hierarchie-->.+<!--/#hierarchie-->,Uims",$fond,$match)){
+	if (preg_match(",<!--#hierarchie-->.+<!--/#hierarchie-->,Uims", $fond, $match)) {
 		$hierarchie = $match[0];
-		$fond = str_replace($hierarchie,"",$fond);
+		$fond = str_replace($hierarchie, "", $fond);
 	}
 
 	// recuperer la navigation (colonne de gauche)
-	if (preg_match(",<!--#navigation-->.+<!--/#navigation-->,Uims",$fond,$match)){
+	if (preg_match(",<!--#navigation-->.+<!--/#navigation-->,Uims", $fond, $match)) {
 		$navigation = $match[0];
-		$fond = str_replace($navigation,"",$fond);
+		$fond = str_replace($navigation, "", $fond);
 	}
 
 	// recuperer les extras (colonne de droite)
-	if (preg_match(",<!--#extra-->.+<!--/#extra-->,Uims",$fond,$match)){
+	if (preg_match(",<!--#extra-->.+<!--/#extra-->,Uims", $fond, $match)) {
 		$extra = $match[0];
-		$fond = str_replace($extra,"",$fond);
+		$fond = str_replace($extra, "", $fond);
 	}
 
 	include_spip('inc/presentation'); // alleger les inclusions avec un inc/presentation_mini
-	$commencer_page = charger_fonction('commencer_page','inc');
+	$commencer_page = charger_fonction('commencer_page', 'inc');
 	echo $commencer_page($titre);
 
-	if ($hierarchie){
+	if ($hierarchie) {
 		echo debut_grand_cadre(true);
 		echo pipeline(
 			'affiche_hierarchie',
@@ -78,27 +78,27 @@ 
 		echo fin_grand_cadre(true);
 	}
 
-	echo debut_gauche("exec_$exec",true);
+	echo debut_gauche("exec_$exec", true);
 
 	$contexte = array('exec'=>$exec);
-	if ($objet_exec = trouver_objet_exec($exec)){
+	if ($objet_exec = trouver_objet_exec($exec)) {
 		$id = $objet_exec['id_table_objet'];
 		if (_request($id))
 			$contexte[$id] = _request($id);
 	}
 
 	echo $navigation;
-	echo pipeline('affiche_gauche',array('args'=>$contexte,'data'=>''));
+	echo pipeline('affiche_gauche', array('args'=>$contexte, 'data'=>''));
 
-	echo creer_colonne_droite("exec_$exec",true);
+	echo creer_colonne_droite("exec_$exec", true);
 	echo $extra;
-	echo pipeline('affiche_droite',array('args'=>$contexte,'data'=>''));
+	echo pipeline('affiche_droite', array('args'=>$contexte, 'data'=>''));
 
-	echo debut_droite("exec_$exec",true);
+	echo debut_droite("exec_$exec", true);
 	echo $fond;
-	echo pipeline('affiche_milieu',array('args'=>$contexte,'data'=>''));
+	echo pipeline('affiche_milieu', array('args'=>$contexte, 'data'=>''));
 
-	echo fin_gauche(),fin_page();
+	echo fin_gauche(), fin_page();
 	}
 }
 

ecrire/exec/rechercher.php

Spacing +21 added lines, -21 removed lines

@@ -15,7 +15,7 @@ 
 include_spip('inc/texte');
 
 // http://doc.spip.org/@exec_rechercher_dist
-function exec_rechercher_dist(){
+function exec_rechercher_dist() {
 	$id = intval(_request('id'));
 	$exclus = intval(_request('exclus'));
 	$rac = spip_htmlentities(_request('rac'));
@@ -28,41 +28,41 @@ 
 }
 
 // http://doc.spip.org/@exec_rechercher_args
-function exec_rechercher_args($id, $type, $exclus, $rac, $do){
+function exec_rechercher_args($id, $type, $exclus, $rac, $do) {
 	if (!$do) $do = 'aff';
 
 	$where = preg_split(",\s+,", $type);
-	if ($where){
-		foreach ($where as $k => $v){
-			$where[$k] = "'%" . substr(str_replace("%", "\%", sql_quote($v,'','string')), 1, -1) . "%'";
+	if ($where) {
+		foreach ($where as $k => $v) {
+			$where[$k] = "'%".substr(str_replace("%", "\%", sql_quote($v, '', 'string')), 1, -1)."%'";
 		}
-		$where_titre = ("(titre LIKE " . join(" AND titre LIKE ", $where) . ")");
-		$where_desc = ("(descriptif LIKE " . join(" AND descriptif LIKE ", $where) . ")");
-		$where_id = ("(id_rubrique = " . intval($type) . ")");
+		$where_titre = ("(titre LIKE ".join(" AND titre LIKE ", $where).")");
+		$where_desc = ("(descriptif LIKE ".join(" AND descriptif LIKE ", $where).")");
+		$where_id = ("(id_rubrique = ".intval($type).")");
 	} else {
 		$where_titre = " 1=2";
 		$where_desc = " 1=2";
 		$where_id = " 1=2";
 	}
 
-	if ($exclus){
+	if ($exclus) {
 		include_spip('inc/rubriques');
-		$where_exclus = " AND " . sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
+		$where_exclus = " AND ".sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
 	} else  $where_exclus = '';
 
 	$res = sql_select("id_rubrique, id_parent, titre", "spip_rubriques", "$where_id$where_exclus");
 
 	$points = $rub = array();
 
-	while ($row = sql_fetch($res)){
+	while ($row = sql_fetch($res)) {
 		$id_rubrique = $row["id_rubrique"];
 		$rub[$id_rubrique]["titre"] = typo($row["titre"]);
 		$rub[$id_rubrique]["id_parent"] = $row["id_parent"];
-		$points[$id_rubrique] = $points[$id_rubrique]+3;
+		$points[$id_rubrique] = $points[$id_rubrique] + 3;
 	}
 	$res = sql_select("id_rubrique, id_parent, titre", "spip_rubriques", "$where_titre$where_exclus");
 
-	while ($row = sql_fetch($res)){
+	while ($row = sql_fetch($res)) {
 		$id_rubrique = $row["id_rubrique"];
 		$rub[$id_rubrique]["titre"] = typo($row["titre"]);
 		$rub[$id_rubrique]["id_parent"] = $row["id_parent"];
@@ -72,7 +72,7 @@ 
 	}
 	$res = sql_select("id_rubrique, id_parent, titre", "spip_rubriques", "$where_desc$where_exclus");
 
-	while ($row = sql_fetch($res)){
+	while ($row = sql_fetch($res)) {
 		$id_rubrique = $row["id_rubrique"];
 		$rub[$id_rubrique]["titre"] = typo($row["titre"]);
 		$rub[$id_rubrique]["id_parent"] = $row["id_parent"];
@@ -81,10 +81,10 @@ 
 		else $points[$id_rubrique] = 0;
 	}
 
-	if ($points){
+	if ($points) {
 		arsort($points);
-		$style = " style='background-image: url(" . chemin_image('secteur-12.png') . ")'";
-		foreach ($rub as $k => $v){
+		$style = " style='background-image: url(".chemin_image('secteur-12.png').")'";
+		foreach ($rub as $k => $v) {
 			$rub[$k]['atts'] = ($v["id_parent"] ? $style : '')
 				. " class='petite-rubrique'";
 		}
@@ -99,21 +99,21 @@ 
 // ==> attention a composer le message d'erreur avec au moins 2 balises
 
 // http://doc.spip.org/@proposer_item
-function proposer_item($ids, $titles, $rac, $type, $do){
+function proposer_item($ids, $titles, $rac, $type, $do) {
 
 	if (!$ids)
 		return "<br /><br /><div style='padding: 5px; color: red;'><b>"
 		. spip_htmlentities($type)
-		. "</b> :  " . _T('avis_aucun_resultat') . "</div>";
+		. "</b> :  "._T('avis_aucun_resultat')."</div>";
 
 	$ret = '';
 	$info = generer_url_ecrire('informer', "type=rubrique&rac=$rac&id=");
 
-	$onClick = "aff_selection(this.firstChild.title,'$rac" . "_selection','$info', event)";
+	$onClick = "aff_selection(this.firstChild.title,'$rac"."_selection','$info', event)";
 
 	$ondbClick = "$do(this.firstChild.firstChild.nodeValue,this.firstChild.title,'selection_rubrique', 'id_parent');";
 
-	foreach ($ids as $id => $bof){
+	foreach ($ids as $id => $bof) {
 
 		$titre = strtr(str_replace("'", "&#8217;", str_replace('"', "&#34;", textebrut($titles[$id]["titre"]))), "\n\r", "  ");
 

ecrire/exec/fond.php

Spacing +3 added lines, -3 removed lines

@@ -14,7 +14,7 @@ 
 
 
 $fond = _request('exec');
-$GLOBALS['delais'] = 0;// pas de cache !
+$GLOBALS['delais'] = 0; // pas de cache !
 // Securite
 if (strstr($fond, '/')) {
 	if (!include_spip('inc/autoriser')
@@ -30,7 +30,7 @@ 
 // quelques inclusions et ini prealables
 include_spip('inc/commencer_page');
 
-function shutdown_error(){
+function shutdown_error() {
 
 	// si on arrive ici avec un tampon non ferme : erreur fatale
 /*	if (ob_get_level()){
@@ -61,7 +61,7 @@ 
  * La fonction ne fait rien, c'est l'inclusion du fichier qui declenche le traitement
  *
  */
-function exec_fond_dist(){
+function exec_fond_dist() {
 
 }
 

ecrire/exec/demande_mise_a_jour.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
 	// si jamais la liste des plugins actifs change, il faut faire un refresh du hit
 	// pour etre sur que les bons fichiers seront charges lors de l'install
 	include_spip('inc/plugin');
-	if (actualise_plugins_actifs()){
+	if (actualise_plugins_actifs()) {
 		include_spip('inc/headers');
 		redirige_par_entete(self());
 	}
@@ -28,13 +28,13 @@ 
 	include_spip('inc/presentation');
 	include_spip('inc/filtres_boites');
 	$commencer_page = charger_fonction('commencer_page', 'inc');
-	echo $commencer_page('','','','',true,false,false);
+	echo $commencer_page('', '', '', '', true, false, false);
 
 	echo debut_grand_cadre(true);
-	echo boite_ouvrir(_T('info_message_technique'),'notice');
+	echo boite_ouvrir(_T('info_message_technique'), 'notice');
 	echo "<p>"._T('info_procedure_maj_version')."</p>",
 	     "<p>"._T('info_administrateur_site_01')."</p>";
-	echo bouton_action(_T('bouton_mettre_a_jour_base'),generer_url_ecrire("upgrade","reinstall=non"));
+	echo bouton_action(_T('bouton_mettre_a_jour_base'), generer_url_ecrire("upgrade", "reinstall=non"));
 	echo boite_fermer();
 	// masquer les erreurs sql sur cette page car proviennent de la base pas a jour !
 	echo '<style type="text/css">#debug-nav {display: none;}</style>';

ecrire/exec/selectionner.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
 		$selectionner = charger_fonction('selectionner', 'inc');
 
-		$r = $selectionner($id, "choix_parent", $exclus, $rac, $type!='breve', $do);
+		$r = $selectionner($id, "choix_parent", $exclus, $rac, $type != 'breve', $do);
 	} else $r = '';
 	ajax_retour($r);
 }

ecrire/exec/admin_plugin.php

Spacing +23 added lines, -23 removed lines

@@ -20,7 +20,7 @@ 
 include_spip('inc/securiser_action');
 
 // http://doc.spip.org/@exec_admin_plugin_dist
-function exec_admin_plugin_dist($retour='') {
+function exec_admin_plugin_dist($retour = '') {
 
 	if (!autoriser('configurer', '_plugins')) {
 		include_spip('inc/minipres');
@@ -31,9 +31,9 @@ 
 	// si jamais la liste des plugins actifs change, il faut faire un refresh du hit
 	// pour etre sur que les bons fichiers seront charges lors de l'install
 		$new = actualise_plugins_actifs();
-		if ($new AND _request('actualise')<2) {
+		if ($new AND _request('actualise') < 2) {
 			include_spip('inc/headers');
-			redirige_par_entete(parametre_url(self(),'actualise',_request('actualise')+1,'&'));
+			redirige_par_entete(parametre_url(self(), 'actualise', _request('actualise') + 1, '&'));
 		}
 		else {
 			admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
@@ -52,8 +52,8 @@ 
 	$commencer_page = charger_fonction('commencer_page', 'inc');
 	echo $commencer_page(_T('icone_admin_plugin'), "configuration", "plugin");
 
-	echo debut_gauche('plugin',true);
-	echo recuperer_fond('prive/squelettes/navigation/configurer',array('exec'=>'admin_plugin'));
+	echo debut_gauche('plugin', true);
+	echo recuperer_fond('prive/squelettes/navigation/configurer', array('exec'=>'admin_plugin'));
 
 	echo pipeline('affiche_gauche',
 		array(
@@ -63,12 +63,12 @@ 
 	);
 
 	echo debut_droite('plugin', true);
-	echo gros_titre(_T('icone_admin_plugin'),'',false);
+	echo gros_titre(_T('icone_admin_plugin'), '', false);
 
 	// Barre d'onglets de premier niveau
 	echo barre_onglets("plugins", "plugins_actifs");
 	// Barre d'onglets de second niveau
-	$onglet2 = $quoi=='actifs' ? 'plugins_actifs' : 'admin_plugin';
+	$onglet2 = $quoi == 'actifs' ? 'plugins_actifs' : 'admin_plugin';
 	echo debut_onglet('onglets_simple second');
 	echo onglet(_T('plugins_tous_liste'), generer_url_ecrire("admin_plugin", "voir=tous"), 'admin_plugin', $onglet2);
 	echo onglet(_T('plugins_actifs_liste'), generer_url_ecrire("admin_plugin"), 'plugins_actifs', $onglet2);
@@ -77,7 +77,7 @@ 
 	// message d'erreur au retour d'une operation
 	if ($erreur)
 		echo "<div class='error'>$erreur</div>";
-	if ($erreur_activation){
+	if ($erreur_activation) {
 		echo "<div class='error'>$erreur_activation</div>";
 	}
 
@@ -88,7 +88,7 @@ 
 	// Les affichages se basent sur le repertoire, pas sur le nom
 	$actifs = liste_chemin_plugin($actifs, '');
 	if (defined('_DIR_PLUGINS_SUPPL'))
-		$lcpas = liste_chemin_plugin($lcpa,_DIR_PLUGINS_SUPPL);
+		$lcpas = liste_chemin_plugin($lcpa, _DIR_PLUGINS_SUPPL);
 	$lcpa = liste_chemin_plugin($lcpa);
 	
 	// on installe les plugins maintenant,
@@ -96,15 +96,15 @@ 
 	plugin_installes_meta();
 
 	echo "<div class='liste-plugins formulaire_spip'>";
-	echo debut_cadre_trait_couleur('plugin-24.png',true,'',_T('plugins_liste'), 'plugins');
+	echo debut_cadre_trait_couleur('plugin-24.png', true, '', _T('plugins_liste'), 'plugins');
 
-	if ($quoi!=='actifs'){
+	if ($quoi !== 'actifs') {
 		$lpf = liste_plugin_files();
 		if ($lpf)
 			echo "<p>"._T('texte_presente_plugin')."</p>";
 		else {
 			if (!@is_dir(_DIR_PLUGINS))
-				echo  "<p>"._T('plugin_info_automatique_ftp',array('rep'=>joli_repertoire(_DIR_PLUGINS)))
+				echo  "<p>"._T('plugin_info_automatique_ftp', array('rep'=>joli_repertoire(_DIR_PLUGINS)))
 							. " &mdash; "._T('plugin_info_automatique_creer')."</p>";
 		}
 		$lcpaffiche = $lpf;
@@ -119,7 +119,7 @@ 
 			$lcpaffichesup = $lcpas;
 	}
 
-	if ($quoi=='actifs' OR $lpf){
+	if ($quoi == 'actifs' OR $lpf) {
 		$nb = count($lcpa);
 		if (defined('_DIR_PLUGINS_SUPPL'))
 			$nb += count($lcpas);
@@ -128,13 +128,13 @@ 
 
 	if (empty($format))
 	  $format = 'liste';
-	elseif (!in_array($format,array('liste','repertoires')))
+	elseif (!in_array($format, array('liste', 'repertoires')))
 		$format = 'repertoires';
 
-	$afficher = charger_fonction("afficher_$format",'plugins');
-	$corps = $afficher(self(),$lcpaffiche, $lcpa, $actifs);
+	$afficher = charger_fonction("afficher_$format", 'plugins');
+	$corps = $afficher(self(), $lcpaffiche, $lcpa, $actifs);
 	if (defined('_DIR_PLUGINS_SUPPL'))
-		$corps .= $afficher(self(),$lcpaffichesup, $lcpas, $actifs, _DIR_PLUGINS_SUPPL);
+		$corps .= $afficher(self(), $lcpaffichesup, $lcpas, $actifs, _DIR_PLUGINS_SUPPL);
 
 	if ($corps)
 	  $corps .= "\n<div class='boutons' style='display:none;'>"
@@ -142,11 +142,11 @@ 
 	    ."' />"
 	    . "</div>";
 
-	echo redirige_action_post('activer_plugins','activer','admin_plugin','', $corps);
+	echo redirige_action_post('activer_plugins', 'activer', 'admin_plugin', '', $corps);
 
 	echo fin_cadre_trait_couleur(true);
 
-	if ($quoi=='actifs')
+	if ($quoi == 'actifs')
 		echo affiche_les_plugins_verrouilles($actifs);
 	echo "</div>";
 	
@@ -190,15 +190,15 @@ 
 {
 	if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) return '';
 
-	$afficher = charger_fonction("afficher_liste",'plugins');
+	$afficher = charger_fonction("afficher_liste", 'plugins');
 	$liste = $afficher(self(), $liste, array(), $actifs, _DIR_PLUGINS_DIST);
 
 	return 
 		"<div id='plugins_dist'>"
-		. debut_cadre_trait_couleur('',true,'',_T('plugins_liste_dist'), 'liste_plugins_dist')
+		. debut_cadre_trait_couleur('', true, '', _T('plugins_liste_dist'), 'liste_plugins_dist')
 		. "<p>"
 		. _T('plugin_info_plugins_dist_1', array('plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)))
-		. '<br />'. _T('plugin_info_plugins_dist_2')
+		. '<br />'._T('plugin_info_plugins_dist_2')
 		. "</p>"
 		. $liste
 		. fin_cadre_trait_couleur(true)
@@ -210,7 +210,7 @@ 
  *
  * @return <type>
  */
-function afficher_librairies(){
+function afficher_librairies() {
 
 	if (!$libs = liste_librairies()) return '';
 	ksort($libs);

ecrire/xml/indenter.php

Spacing +9 added lines, -9 removed lines

@@ -16,22 +16,22 @@ 
 
 // http://doc.spip.org/@debutElement
 function debutElement($phraseur, $name, $attrs)
-{ xml_debutElement($this, $name, $attrs);}
+{ xml_debutElement($this, $name, $attrs); }
 
 // http://doc.spip.org/@finElement
 function finElement($phraseur, $name)
-{ xml_finElement($this, $name);}
+{ xml_finElement($this, $name); }
 
 // http://doc.spip.org/@textElement
 function textElement($phraseur, $data)
-{ xml_textElement($this, $data);}
+{ xml_textElement($this, $data); }
 
 function piElement($phraseur, $target, $data)
-{ xml_PiElement($this, $target, $data);}
+{ xml_PiElement($this, $target, $data); }
 
 // http://doc.spip.org/@defautElement
 function defaultElement($phraseur, $data)
-{  xml_defaultElement($this, $data);}
+{  xml_defaultElement($this, $data); }
 
 // http://doc.spip.org/@phraserTout
 function phraserTout($phraseur, $data)
@@ -52,14 +52,14 @@ 
 }
 
 // http://doc.spip.org/@xml_indenter_dist
-function xml_indenter_dist($page, $apply=false)
+function xml_indenter_dist($page, $apply = false)
 {
 	$sax = charger_fonction('sax', 'xml');
 	$f = new IndenteurXML();
 	$sax($page, $apply, $f);
-	if (!$f->err) return $f->entete . $f->res;
-	spip_log("indentation impossible " . count($f->err) . " erreurs de validation");
-	return $f->entete . $f->page;
+	if (!$f->err) return $f->entete.$f->res;
+	spip_log("indentation impossible ".count($f->err)." erreurs de validation");
+	return $f->entete.$f->page;
 }
 
 ?>

ecrire/xml/sax.php

Spacing +37 added lines, -38 removed lines

@@ -20,14 +20,14 @@ 
  * @param string $texte
  * @return string
  */
-function xml_entites_html($texte){
+function xml_entites_html($texte) {
 	if (!is_string($texte) OR !$texte
-	OR strpbrk($texte, "&\"'<>")==false
+	OR strpbrk($texte, "&\"'<>") == false
 	) return $texte;
 
 	if (!function_exists('spip_htmlspecialchars'))
 		include_spip("inc/filtres_mini");
-	$texte = spip_htmlspecialchars($texte,ENT_QUOTES);
+	$texte = spip_htmlspecialchars($texte, ENT_QUOTES);
 	return $texte;
 }
 
@@ -40,40 +40,40 @@ 
 	// espace initial signifie: deja integree au resultat
 	if ($t[0] != ' ')
 	  {
-	    $phraseur->res .= '<' . $t . '>';
-	    $phraseur->ouvrant[$depth] = ' ' . $t;
+	    $phraseur->res .= '<'.$t.'>';
+	    $phraseur->ouvrant[$depth] = ' '.$t;
 	  }
 	$t = $phraseur->contenu[$depth];
 	// n'indenter que s'il y a un separateur avant
-	$phraseur->res .= preg_replace("/[\n\t ]+$/",  "\n$depth", $t);
+	$phraseur->res .= preg_replace("/[\n\t ]+$/", "\n$depth", $t);
 	$phraseur->contenu[$depth] = "";
 	$att = '';
 	$sep = ' ';
 	foreach ($attrs as $k => $v) {
 	  $delim = strpos($v, "'") === false ? "'" : '"';
 	  $val = xml_entites_html($v);
-	  $att .= $sep .  $k . "=" . $delim
+	  $att .= $sep.$k."=".$delim
 	    . ($delim !== '"' ? str_replace('&quot;', '"', $val) : $val)
 	    . $delim;
 	  $sep = "\n $depth";
 	}
 	$phraseur->depth .= '  ';
 	$phraseur->contenu[$phraseur->depth] = "";
-	$phraseur->ouvrant[$phraseur->depth] = $name . $att;
+	$phraseur->ouvrant[$phraseur->depth] = $name.$att;
 	$phraseur->reperes[$phraseur->depth] = xml_get_current_line_number($phraseur->sax);
 }
 
 // http://doc.spip.org/@xml_finElement
-function xml_finElement($phraseur, $name, $fusion_bal=false)
+function xml_finElement($phraseur, $name, $fusion_bal = false)
 {
 	$ouv = $phraseur->ouvrant[$phraseur->depth];
 
 	if ($ouv[0] != ' ')
-		$phraseur->ouvrant[$phraseur->depth] = ' ' . $ouv;
-	else $ouv= "";
+		$phraseur->ouvrant[$phraseur->depth] = ' '.$ouv;
+	else $ouv = "";
 	$t = $phraseur->contenu[$phraseur->depth];
 	$phraseur->depth = substr($phraseur->depth, 2);
-	$t = preg_replace("/[\n\t ]+$/", "\n" . $phraseur->depth, $t);
+	$t = preg_replace("/[\n\t ]+$/", "\n".$phraseur->depth, $t);
 
   // fusion <balise></balise> en <balise />.
   // ATTENTION,  certains clients http croient que fusion ==> pas d'atttributs
@@ -82,16 +82,16 @@ 
   // (param fusion_bal)
 
 	if ($t || (($ouv != $name) AND !$fusion_bal))
-	  $phraseur->res .= ($ouv ? ('<' . $ouv . '>') : '') . $t . "</" . $name . ">";
+	  $phraseur->res .= ($ouv ? ('<'.$ouv.'>') : '').$t."</".$name.">";
 	else
-	  $phraseur->res .= ($ouv ? ('<' . $ouv  . ' />') : ("</" .  $name . ">"));
+	  $phraseur->res .= ($ouv ? ('<'.$ouv.' />') : ("</".$name.">"));
 }
 
 // http://doc.spip.org/@xml_textElement
 function xml_textElement($phraseur, $data)
 {
 	$depth = $phraseur->depth;
-	$phraseur->contenu[$depth] .= preg_match('/^script/',$phraseur->ouvrant[$depth])
+	$phraseur->contenu[$depth] .= preg_match('/^script/', $phraseur->ouvrant[$depth])
 	  ? $data
 	  : xml_entites_html($data);
 }
@@ -117,43 +117,42 @@ 
 {
 	$depth = $phraseur->depth;
 
-	if (!isset($phraseur->contenu[$depth])) $phraseur->contenu[$depth]='';
+	if (!isset($phraseur->contenu[$depth])) $phraseur->contenu[$depth] = '';
 	$phraseur->contenu[$depth] .= $data;
 }
 
 // http://doc.spip.org/@xml_parsestring
 function xml_parsestring($phraseur, $data)
 {
-	$phraseur->contenu[$phraseur->depth] ='';
+	$phraseur->contenu[$phraseur->depth] = '';
 
 	if (!xml_parse($phraseur->sax, $data, true)) {
 		coordonnees_erreur($phraseur,
 			xml_error_string(xml_get_error_code($phraseur->sax))
-			. "<br />\n" .
-			(!$phraseur->depth ? '' :
-			 ('(' .
-			  _T('erreur_balise_non_fermee') .
-			  " <tt>" .
-			  $phraseur->ouvrant[$phraseur->depth] .
-			  "</tt> " .
-			  _T('ligne') .
-			  " " .
-			  $phraseur->reperes[$phraseur->depth] .
-			  ") <br />\n" )));
+			. "<br />\n".
+			(!$phraseur->depth ? '' : ('('.
+			  _T('erreur_balise_non_fermee').
+			  " <tt>".
+			  $phraseur->ouvrant[$phraseur->depth].
+			  "</tt> ".
+			  _T('ligne').
+			  " ".
+			  $phraseur->reperes[$phraseur->depth].
+			  ") <br />\n")));
 	}
 }
 
 // http://doc.spip.org/@coordonnees_erreur
 function coordonnees_erreur($phraseur, $msg)
 {
-	$entete_length = substr_count($phraseur->entete,"\n");
+	$entete_length = substr_count($phraseur->entete, "\n");
 	$phraseur->err[] = array($msg,
 		xml_get_current_line_number($phraseur->sax) + $entete_length,
 		xml_get_current_column_number($phraseur->sax));
 }
 
 // http://doc.spip.org/@xml_sax_dist
-function xml_sax_dist($page, $apply=false, $phraseur=NULL, $doctype='', $charset=null)
+function xml_sax_dist($page, $apply = false, $phraseur = NULL, $doctype = '', $charset = null)
 {
 	if (is_null($charset))
 		$charset = $GLOBALS['meta']['charset'];
@@ -173,12 +172,12 @@ 
 	// et escamoter le doctype que sax mange en php5 mais pas en  php4
 	if (!$doctype) {
 		if (!$r = analyser_doctype($page)) {
-			$page = _MESSAGE_DOCTYPE . _DOCTYPE_ECRIRE
+			$page = _MESSAGE_DOCTYPE._DOCTYPE_ECRIRE
 			  . preg_replace(_REGEXP_DOCTYPE, '', $page);
-			$r =  analyser_doctype($page);
+			$r = analyser_doctype($page);
 		}
 		list($entete, $avail, $grammaire, $rotlvl) = array_pad($r, 4, null);
-		$page = substr($page,strlen($entete));
+		$page = substr($page, strlen($entete));
 	} else {
 		$avail = 'SYSTEM';
 		$grammaire = $doctype;
@@ -232,7 +231,7 @@ 
 // sinon on se rabat sur ce qu'en connait SPIP en standard.
 
 // http://doc.spip.org/@sax_bug
-function sax_bug($data, $dtc, $charset=null)
+function sax_bug($data, $dtc, $charset = null)
 {
 	if (is_null($charset))
 		$charset = $GLOBALS['meta']['charset'];
@@ -240,7 +239,7 @@ 
 	if ($dtc) {
 		$trans = array();
 		
-		foreach($dtc->entites as $k => $v) {
+		foreach ($dtc->entites as $k => $v) {
 			if (!strpos(" amp lt gt quot ", $k))
 			    $trans["&$k;"] = $v;
 		}
@@ -268,16 +267,16 @@ 
 				       _DOCTYPE_RSS,
 					     'rss-0.91.dtd');
 			else {
-				$dtd = $topelement . '.dtd';
+				$dtd = $topelement.'.dtd';
 				$f = find_in_path($dtd);
 				if (file_exists($f))
 				  return array($entete, 'SYSTEM', $f, $dtd);
 			}
 		}
-		spip_log("Dtd pas vu pour " . substr($data, 0, 100));
+		spip_log("Dtd pas vu pour ".substr($data, 0, 100));
 		return array();
 	}
-	list($entete,, $topelement, $avail,$suite) = $page;
+	list($entete,, $topelement, $avail, $suite) = $page;
 
 	if (!preg_match('/^"([^"]*)"\s*(.*)$/', $suite, $r))
 		if (!preg_match("/^'([^']*)'\s*(.*)$/", $suite, $r))

ecrire/xml/valider.php

Spacing +32 added lines, -32 removed lines

@@ -23,11 +23,11 @@ 
 {
 	if (!($p = isset($this->dtc->elements[$name]))) {
 		if ($p = strpos($name, ':')) {
-			$name = substr($name, $p+1);
+			$name = substr($name, $p + 1);
 			$p = isset($this->dtc->elements[$name]);
 		}
 		if (!$p) {
-			coordonnees_erreur($this," <b>$name</b>&nbsp;: "
+			coordonnees_erreur($this, " <b>$name</b>&nbsp;: "
 				. _T('zxml_inconnu_balise'));
 			return; 
 		}
@@ -45,21 +45,21 @@ 
 #		spip_log("rejeton $name fils " . @join(',',$fils));
 		if (!($p = @in_array($name, $fils))) {
 			if ($p = strpos($name, ':')) {
-				$p = substr($name, $p+1);
+				$p = substr($name, $p + 1);
 				$p = @in_array($p, $fils);
 			}
 		}
 		if (!$p) {
-	          $bons_peres = @join ('</b>, <b>', $this->dtc->peres[$name]);
+	          $bons_peres = @join('</b>, <b>', $this->dtc->peres[$name]);
 		  coordonnees_erreur($this, " <b>$name</b> "
 	            . _T('zxml_non_fils')
 	            . ' <b>'
 	            .  $pere
 	            . '</b>'
 	            . (!$bons_peres ? ''
-	               : ('<p style="font-size: 80%"> '._T('zxml_mais_de').' <b>'. $bons_peres . '</b></p>')));
-		} else if ($this->dtc->regles[$pere][0]=='/') {
-			$frat = substr($depth,2);
+	               : ('<p style="font-size: 80%"> '._T('zxml_mais_de').' <b>'.$bons_peres.'</b></p>')));
+		} else if ($this->dtc->regles[$pere][0] == '/') {
+			$frat = substr($depth, 2);
 			if (!isset($this->fratrie[$frat])) {
 				$this->fratrie[$frat] = '';
 			}
@@ -69,8 +69,8 @@ 
 	    }
 	}
 	// Init de la suite des balises a memoriser si regle difficile
-	if ($this->dtc->regles[$name][0]=='/')
-	    $this->fratrie[$depth]='';
+	if ($this->dtc->regles[$name][0] == '/')
+	    $this->fratrie[$depth] = '';
 	if (isset($this->dtc->attributs[$name])) {
 		  foreach ($this->dtc->attributs[$name] as $n => $v)
 		    { if (($v[1] == '#REQUIRED') AND (!isset($attrs[$n])))
@@ -87,16 +87,16 @@ 
 {
 	// Si la balise est inconnue, eviter d'insister
 	if (!isset($this->dtc->attributs[$bal]))
-		return ;
+		return;
 		
 	$a = $this->dtc->attributs[$bal];
 	if (!isset($a[$name])) {
-		$bons = join(', ',array_keys($a));
+		$bons = join(', ', array_keys($a));
 		if ($bons)
-		  $bons = " title=' " .
-		    _T('zxml_connus_attributs') .
-		    '&nbsp;: ' .
-		    $bons .
+		  $bons = " title=' ".
+		    _T('zxml_connus_attributs').
+		    '&nbsp;: '.
+		    $bons.
 		    "'";
 		$bons .= " style='font-weight: bold'";
 		coordonnees_erreur($this, " <b>$name</b> "
@@ -104,11 +104,11 @@ 
 		. " <a$bons>$bal</a> ("
 		. _T('zxml_survoler')
 		. ")");
-	} else{
-		$type =  $a[$name][0];
+	} else {
+		$type = $a[$name][0];
 		if (!preg_match('/^\w+$/', $type))
 			$this->valider_motif($phraseur, $name, $val, $bal, $type);
-		else if (method_exists($this, $f = 'validerAttribut_' . $type))
+		else if (method_exists($this, $f = 'validerAttribut_'.$type))
 			$this->$f($phraseur, $name, $val, $bal);
 #		else spip_log("$type type d'attribut inconnu");
 	}
@@ -128,7 +128,7 @@ 
 function validerAttribut_ID($phraseur, $name, $val, $bal)
 {
 	if (isset($this->ids[$val])) {
-		list($l,$c) = $this->ids[$val];
+		list($l, $c) = $this->ids[$val];
 		coordonnees_erreur($this, " <p><b>$val</b> "
 		      . _T('zxml_valeur_attribut')
 		      . " <b>$name</b> "
@@ -165,7 +165,7 @@ 
 		. " <b>$bal</b> "
 		. _T('zxml_non_conforme')
 		. "</p><p>"
-		. "<b>" . $motif . "</b>");
+		. "<b>".$motif."</b>");
 	}
 }
 
@@ -173,7 +173,7 @@ 
 function valider_idref($nom, $ligne, $col)
 {
 	if (!isset($this->ids[$nom]))
-		$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
+		$this->err[] = array(" <p><b>$nom</b> "._T('zxml_inconnu_id'), $ligne, $col);
 }
 
 // http://doc.spip.org/@valider_passe2
@@ -186,7 +186,7 @@ 
 		}
 		foreach ($this->idrefss as $idref) {
 			list($noms, $ligne, $col) = $idref;
-			foreach(preg_split('/\s+/', $noms) as $nom)
+			foreach (preg_split('/\s+/', $noms) as $nom)
 				$this->valider_idref($nom, $ligne, $col);
 		}
 	}
@@ -200,7 +200,7 @@ 
 
 	if ($f = $this->process['debut']) $f($this, $name, $attrs);
 	$depth = $this->depth;
-	$this->debuts[$depth] =  strlen($this->res);
+	$this->debuts[$depth] = strlen($this->res);
 	foreach ($attrs as $k => $v) {
 		$this->validerAttribut($phraseur, $k, $v, $name);
 	}
@@ -217,7 +217,7 @@ 
 	$k = $this->debuts[$depth];
 
 	$regle = $this->dtc->regles[$name];
-	$vide = ($regle  == 'EMPTY');
+	$vide = ($regle == 'EMPTY');
 	// controler que les balises devant etre vides le sont 
 	if ($vide) {
 		if ($n <> ($k + $c))
@@ -232,7 +232,7 @@ 
 				  . _T('zxml_vide_balise'));
 			}
 		} else {
-			$f = $this->fratrie[substr($depth,2)];
+			$f = $this->fratrie[substr($depth, 2)];
 			if (!preg_match($regle, $f)) {
 				coordonnees_erreur($this,
 				" <p>\n<b>$name</b> "
@@ -255,7 +255,7 @@ 
 		$d = $this->ouvrant[$d];
 		preg_match('/^\s*(\S+)/', $d, $m);
 		if ($this->dtc->pcdata[$m[1]]) {
-			coordonnees_erreur($this, " <p><b>". $m[1] . "</b> "
+			coordonnees_erreur($this, " <p><b>".$m[1]."</b> "
 			. _T('zxml_nonvide_balise') // message a affiner
 			);
 		}
@@ -281,7 +281,7 @@ 
 	if (!preg_match('/^<!--/', $data)
 	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER)))
 		foreach ($r as $m) {
-			list($t,$e) = $m;
+			list($t, $e) = $m;
 			if (!isset($this->dtc->entites[$e]))
 				coordonnees_erreur($this, " <b>$e</b> "
 				  . _T('zxml_inconnu_entite')
@@ -298,15 +298,15 @@ 
 { 
 	xml_parsestring($this, $data);
 
-	if (!$this->dtc OR preg_match(',^' . _MESSAGE_DOCTYPE . ',', $data)) {
-		$this->err[]= array('DOCTYPE ?', 0, 0);
+	if (!$this->dtc OR preg_match(',^'._MESSAGE_DOCTYPE.',', $data)) {
+		$this->err[] = array('DOCTYPE ?', 0, 0);
 	} else {
 		$this->valider_passe2($this);
 	}
 }
 
 // Init
-function ValidateurXML($process=array()) {
+function ValidateurXML($process = array()) {
 	if (is_array($process)) $this->process = $process;
 }
 
@@ -326,7 +326,7 @@ 
  var $contenu = array();
  var $ouvrant = array();
  var $reperes = array();
- var $process =  array(
+ var $process = array(
 			'debut' => 'xml_debutElement',
 			'fin' => 'xml_finElement',
 			'text' => 'xml_textElement',
@@ -339,7 +339,7 @@ 
 // ayant comme entrees des sous-tableaux [message, ligne, colonne]
 
 // http://doc.spip.org/@xml_valider_dist
-function xml_valider_dist($page, $apply=false, $process=false, $doctype='', $charset=null)
+function xml_valider_dist($page, $apply = false, $process = false, $doctype = '', $charset = null)
 {
 	$f = new ValidateurXML($process);
 	$sax = charger_fonction('sax', 'xml');