spip / SPIP

ecrire/exec/plonger.php

Indentation +12 added lines, -12 removed lines

@@ -17,20 +17,20 @@
 // http://doc.spip.org/@exec_plonger_dist
 function exec_plonger_dist()
 {
-	include_spip('inc/actions');
+    include_spip('inc/actions');
 	
-	$rac = _request('rac');
-	$id = intval(_request('id'));
-	$exclus = intval(_request('exclus'));
-	$col = intval(_request('col'));
-	$do  = _request('do');
-	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) $do = 'aff';
+    $rac = _request('rac');
+    $id = intval(_request('id'));
+    $exclus = intval(_request('exclus'));
+    $col = intval(_request('col'));
+    $do  = _request('do');
+    if (preg_match('/^\w*$/', $do)) {
+        if (!$do) $do = 'aff';
 
-		$plonger = charger_fonction('plonger', 'inc');
-		$r = $plonger($id, spip_htmlentities($rac), array(), $col, $exclus, $do);
-	} else $r = '';
+        $plonger = charger_fonction('plonger', 'inc');
+        $r = $plonger($id, spip_htmlentities($rac), array(), $col, $exclus, $do);
+    } else $r = '';
 
-	ajax_retour($r);
+    ajax_retour($r);
 }
 ?>

Braces +9 added lines, -3 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;
+}
 
 # afficher les sous-rubriques d'une rubrique (composant du mini-navigateur)
 
@@ -25,11 +27,15 @@ 
 	$col = intval(_request('col'));
 	$do  = _request('do');
 	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) $do = 'aff';
+		if (!$do) {
+		    $do = 'aff';
+		}
 
 		$plonger = charger_fonction('plonger', 'inc');
 		$r = $plonger($id, spip_htmlentities($rac), array(), $col, $exclus, $do);
-	} else $r = '';
+	} else {
+	    $r = '';
+	}
 
 	ajax_retour($r);
 }

ecrire/exec/demande_mise_a_jour.php

Indentation +23 added lines, -23 removed lines

@@ -15,30 +15,30 @@
 
 // http://doc.spip.org/@exec_demande_mise_a_jour_dist
 function exec_demande_mise_a_jour_dist() {
-	// on fait la verif du path avant tout,
-	// et l'installation des qu'on est dans la colonne principale
-	// 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()){
-		include_spip('inc/headers');
-		redirige_par_entete(self());
-	}
+    // on fait la verif du path avant tout,
+    // et l'installation des qu'on est dans la colonne principale
+    // 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()){
+        include_spip('inc/headers');
+        redirige_par_entete(self());
+    }
 
-	include_spip('inc/presentation');
-	include_spip('inc/filtres_boites');
-	$commencer_page = charger_fonction('commencer_page', 'inc');
-	echo $commencer_page('','','','',true,false,false);
+    include_spip('inc/presentation');
+    include_spip('inc/filtres_boites');
+    $commencer_page = charger_fonction('commencer_page', 'inc');
+    echo $commencer_page('','','','',true,false,false);
 
-	echo debut_grand_cadre(true);
-	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 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>';
-	echo fin_grand_cadre(true);
-	echo fin_page();
+    echo debut_grand_cadre(true);
+    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 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>';
+    echo fin_grand_cadre(true);
+    echo fin_page();
 }
 ?>

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>';

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;
+}
 
 
 // http://doc.spip.org/@exec_demande_mise_a_jour_dist

ecrire/exec/selectionner.php

Indentation +11 added lines, -11 removed lines

@@ -19,18 +19,18 @@
 // http://doc.spip.org/@exec_selectionner_dist
 function exec_selectionner_dist()
 {
-	$id = intval(_request('id'));
-	$exclus = intval(_request('exclus'));
-	$type = _request('type');
-	$rac = _request('racine');
-	$do  = _request('do');
-	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) $do = 'aff';
+    $id = intval(_request('id'));
+    $exclus = intval(_request('exclus'));
+    $type = _request('type');
+    $rac = _request('racine');
+    $do  = _request('do');
+    if (preg_match('/^\w*$/', $do)) {
+        if (!$do) $do = 'aff';
 
-		$selectionner = charger_fonction('selectionner', 'inc');
+        $selectionner = charger_fonction('selectionner', 'inc');
 
-		$r = $selectionner($id, "choix_parent", $exclus, $rac, $type!='breve', $do);
-	} else $r = '';
-	ajax_retour($r);
+        $r = $selectionner($id, "choix_parent", $exclus, $rac, $type!='breve', $do);
+    } else $r = '';
+    ajax_retour($r);
 }
 ?>

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);
 }

Braces +9 added lines, -3 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/actions');
 
@@ -25,12 +27,16 @@ 
 	$rac = _request('racine');
 	$do  = _request('do');
 	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) $do = 'aff';
+		if (!$do) {
+		    $do = 'aff';
+		}
 
 		$selectionner = charger_fonction('selectionner', 'inc');
 
 		$r = $selectionner($id, "choix_parent", $exclus, $rac, $type!='breve', $do);
-	} else $r = '';
+	} else {
+	    $r = '';
+	}
 	ajax_retour($r);
 }
 ?>

ecrire/exec/admin_plugin.php

Indentation +166 added lines, -166 removed lines

@@ -22,135 +22,135 @@ 
 // http://doc.spip.org/@exec_admin_plugin_dist
 function exec_admin_plugin_dist($retour='') {
 
-	if (!autoriser('configurer', '_plugins')) {
-		include_spip('inc/minipres');
-		echo minipres();
-	} else {
-	// on fait la verif du path avant tout,
-	// et l'installation des qu'on est dans la colonne principale
-	// 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) {
-			include_spip('inc/headers');
-			redirige_par_entete(parametre_url(self(),'actualise',_request('actualise')+1,'&'));
-		}
-		else {
-			admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
-		}
-	}
+    if (!autoriser('configurer', '_plugins')) {
+        include_spip('inc/minipres');
+        echo minipres();
+    } else {
+    // on fait la verif du path avant tout,
+    // et l'installation des qu'on est dans la colonne principale
+    // 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) {
+            include_spip('inc/headers');
+            redirige_par_entete(parametre_url(self(),'actualise',_request('actualise')+1,'&'));
+        }
+        else {
+            admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
+        }
+    }
 }
 
 function admin_plug_args($quoi, $erreur, $format)
 {
-	if (!$quoi) $quoi = 'actifs';
-	// empecher l'affichage des erreurs dans le bandeau, on le donne ensuite
-	// format brut par plugin
-	$GLOBALS['erreurs_activation_raw'] = plugin_donne_erreurs(true, false);
-	// format resume mis en forme
-	$erreur_activation = plugin_donne_erreurs();
-	$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 pipeline('affiche_gauche',
-		array(
-		'args'=>array('exec'=>'admin_plugin'),
-		'data'=>afficher_librairies()
-		)
-	);
-
-	echo debut_droite('plugin', true);
-	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';
-	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);
-	echo fin_onglet();
-
-	// message d'erreur au retour d'une operation
-	if ($erreur)
-		echo "<div class='error'>$erreur</div>";
-	if ($erreur_activation){
-		echo "<div class='error'>$erreur_activation</div>";
-	}
-
-	// la mise a jour de cette meta a ete faite par ecrire_plugin_actifs
-	$actifs = unserialize($GLOBALS['meta']['plugin']);
-	$lcpa = $actifs + unserialize($GLOBALS['meta']['plugin_attente']);
-
-	// 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);
-	$lcpa = liste_chemin_plugin($lcpa);
+    if (!$quoi) $quoi = 'actifs';
+    // empecher l'affichage des erreurs dans le bandeau, on le donne ensuite
+    // format brut par plugin
+    $GLOBALS['erreurs_activation_raw'] = plugin_donne_erreurs(true, false);
+    // format resume mis en forme
+    $erreur_activation = plugin_donne_erreurs();
+    $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 pipeline('affiche_gauche',
+        array(
+        'args'=>array('exec'=>'admin_plugin'),
+        'data'=>afficher_librairies()
+        )
+    );
+
+    echo debut_droite('plugin', true);
+    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';
+    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);
+    echo fin_onglet();
+
+    // message d'erreur au retour d'une operation
+    if ($erreur)
+        echo "<div class='error'>$erreur</div>";
+    if ($erreur_activation){
+        echo "<div class='error'>$erreur_activation</div>";
+    }
+
+    // la mise a jour de cette meta a ete faite par ecrire_plugin_actifs
+    $actifs = unserialize($GLOBALS['meta']['plugin']);
+    $lcpa = $actifs + unserialize($GLOBALS['meta']['plugin_attente']);
+
+    // 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);
+    $lcpa = liste_chemin_plugin($lcpa);
 	
-	// on installe les plugins maintenant,
-	// cela permet aux scripts d'install de faire des affichages (moches...)
-	plugin_installes_meta();
-
-	echo "<div class='liste-plugins formulaire_spip'>";
-	echo debut_cadre_trait_couleur('plugin-24.png',true,'',_T('plugins_liste'), 'plugins');
-
-	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)))
-							. " &mdash; "._T('plugin_info_automatique_creer')."</p>";
-		}
-		$lcpaffiche = $lpf;
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
-	}
-	else {
-		// la liste
-		// $quoi=='actifs'
-		$lcpaffiche = $lcpa;
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$lcpaffichesup = $lcpas;
-	}
-
-	if ($quoi=='actifs' OR $lpf){
-		$nb = count($lcpa);
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$nb += count($lcpas);
-		echo "<h3>".sinon(singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'), _T('plugins_actif_aucun'))."</h3>";
-	}
-
-	if (empty($format))
-	  $format = 'liste';
-	elseif (!in_array($format,array('liste','repertoires')))
-		$format = 'repertoires';
-
-	$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);
-
-	if ($corps)
-	  $corps .= "\n<div class='boutons' style='display:none;'>"
-	    .  "<input type='submit' class='submit save' value='"._T('bouton_enregistrer')
-	    ."' />"
-	    . "</div>";
-
-	echo redirige_action_post('activer_plugins','activer','admin_plugin','', $corps);
-
-	echo fin_cadre_trait_couleur(true);
-
-	if ($quoi=='actifs')
-		echo affiche_les_plugins_verrouilles($actifs);
-	echo "</div>";
+    // on installe les plugins maintenant,
+    // cela permet aux scripts d'install de faire des affichages (moches...)
+    plugin_installes_meta();
+
+    echo "<div class='liste-plugins formulaire_spip'>";
+    echo debut_cadre_trait_couleur('plugin-24.png',true,'',_T('plugins_liste'), 'plugins');
+
+    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)))
+                            . " &mdash; "._T('plugin_info_automatique_creer')."</p>";
+        }
+        $lcpaffiche = $lpf;
+        if (defined('_DIR_PLUGINS_SUPPL'))
+            $lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
+    }
+    else {
+        // la liste
+        // $quoi=='actifs'
+        $lcpaffiche = $lcpa;
+        if (defined('_DIR_PLUGINS_SUPPL'))
+            $lcpaffichesup = $lcpas;
+    }
+
+    if ($quoi=='actifs' OR $lpf){
+        $nb = count($lcpa);
+        if (defined('_DIR_PLUGINS_SUPPL'))
+            $nb += count($lcpas);
+        echo "<h3>".sinon(singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'), _T('plugins_actif_aucun'))."</h3>";
+    }
+
+    if (empty($format))
+        $format = 'liste';
+    elseif (!in_array($format,array('liste','repertoires')))
+        $format = 'repertoires';
+
+    $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);
+
+    if ($corps)
+        $corps .= "\n<div class='boutons' style='display:none;'>"
+        .  "<input type='submit' class='submit save' value='"._T('bouton_enregistrer')
+        ."' />"
+        . "</div>";
+
+    echo redirige_action_post('activer_plugins','activer','admin_plugin','', $corps);
+
+    echo fin_cadre_trait_couleur(true);
+
+    if ($quoi=='actifs')
+        echo affiche_les_plugins_verrouilles($actifs);
+    echo "</div>";
 	
-	echo 	http_script("
+    echo 	http_script("
 	jQuery(function(){
 		jQuery('.plugins li.item a[rel=info]').click(function(){
 			var li = jQuery(this).parents('li').eq(0);
@@ -176,33 +176,33 @@ 
 	});
 	");
 
-	echo pipeline('affiche_milieu',
-		array(
-		'args'=>array('exec'=>'admin_plugin'),
-		'data'=>''
-		)
-	);
+    echo pipeline('affiche_milieu',
+        array(
+        'args'=>array('exec'=>'admin_plugin'),
+        'data'=>''
+        )
+    );
 
-	echo fin_gauche(), fin_page();
+    echo fin_gauche(), fin_page();
 }
 
 function affiche_les_plugins_verrouilles($actifs)
 {
-	if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) return '';
-
-	$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')
-		. "<p>"
-		. _T('plugin_info_plugins_dist_1', array('plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)))
-		. '<br />'. _T('plugin_info_plugins_dist_2')
-		. "</p>"
-		. $liste
-		. fin_cadre_trait_couleur(true)
-		. "</div>\n";
+    if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) return '';
+
+    $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')
+        . "<p>"
+        . _T('plugin_info_plugins_dist_1', array('plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)))
+        . '<br />'. _T('plugin_info_plugins_dist_2')
+        . "</p>"
+        . $liste
+        . fin_cadre_trait_couleur(true)
+        . "</div>\n";
 }
 
 /**
@@ -212,15 +212,15 @@ 
  */
 function afficher_librairies(){
 
-	if (!$libs = liste_librairies()) return '';
-	ksort($libs);
-	$res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
-	$res .= '<dl>';
-	foreach ($libs as $lib => $rep)
-		$res .= "<dt>$lib</dt><dd>".joli_repertoire($rep)."</dd>\n";
-	$res .= '</dl>';
-	$res .= fin_cadre_enfonce(true);
-	return $res;
+    if (!$libs = liste_librairies()) return '';
+    ksort($libs);
+    $res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
+    $res .= '<dl>';
+    foreach ($libs as $lib => $rep)
+        $res .= "<dt>$lib</dt><dd>".joli_repertoire($rep)."</dd>\n";
+    $res .= '</dl>';
+    $res .= fin_cadre_enfonce(true);
+    return $res;
 }
 
 
@@ -232,17 +232,17 @@ 
  */
 // http://doc.spip.org/@liste_librairies
 function liste_librairies() {
-	$libs = array();
-	foreach (array_reverse(creer_chemin()) as $d) {
-		if (is_dir($dir = $d.'lib/')
-		AND $t = @opendir($dir)) {
-			while (($f = readdir($t)) !== false) {
-				if ($f[0] != '.'
-				AND is_dir("$dir/$f"))
-					$libs[$f] = $dir;
-			}
-		}
-	}
-	return $libs;
+    $libs = array();
+    foreach (array_reverse(creer_chemin()) as $d) {
+        if (is_dir($dir = $d.'lib/')
+        AND $t = @opendir($dir)) {
+            while (($f = readdir($t)) !== false) {
+                if ($f[0] != '.'
+                AND is_dir("$dir/$f"))
+                    $libs[$f] = $dir;
+            }
+        }
+    }
+    return $libs;
 }
 ?>

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);

Braces +55 added lines, -37 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');
 include_spip('inc/plugin');
@@ -34,8 +36,7 @@ 
 		if ($new AND _request('actualise')<2) {
 			include_spip('inc/headers');
 			redirige_par_entete(parametre_url(self(),'actualise',_request('actualise')+1,'&'));
-		}
-		else {
+		} else {
 			admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
 		}
 	}
@@ -43,7 +44,9 @@ 
 
 function admin_plug_args($quoi, $erreur, $format)
 {
-	if (!$quoi) $quoi = 'actifs';
+	if (!$quoi) {
+	    $quoi = 'actifs';
+	}
 	// empecher l'affichage des erreurs dans le bandeau, on le donne ensuite
 	// format brut par plugin
 	$GLOBALS['erreurs_activation_raw'] = plugin_donne_erreurs(true, false);
@@ -75,8 +78,9 @@ 
 	echo fin_onglet();
 
 	// message d'erreur au retour d'une operation
-	if ($erreur)
-		echo "<div class='error'>$erreur</div>";
+	if ($erreur) {
+			echo "<div class='error'>$erreur</div>";
+	}
 	if ($erreur_activation){
 		echo "<div class='error'>$erreur_activation</div>";
 	}
@@ -87,8 +91,9 @@ 
 
 	// 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);
+	if (defined('_DIR_PLUGINS_SUPPL')) {
+			$lcpas = liste_chemin_plugin($lcpa,_DIR_PLUGINS_SUPPL);
+	}
 	$lcpa = liste_chemin_plugin($lcpa);
 	
 	// on installe les plugins maintenant,
@@ -100,54 +105,61 @@ 
 
 	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)))
+		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)))
 							. " &mdash; "._T('plugin_info_automatique_creer')."</p>";
+			}
 		}
 		$lcpaffiche = $lpf;
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
-	}
-	else {
+		if (defined('_DIR_PLUGINS_SUPPL')) {
+					$lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
+		}
+	} else {
 		// la liste
 		// $quoi=='actifs'
 		$lcpaffiche = $lcpa;
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$lcpaffichesup = $lcpas;
+		if (defined('_DIR_PLUGINS_SUPPL')) {
+					$lcpaffichesup = $lcpas;
+		}
 	}
 
 	if ($quoi=='actifs' OR $lpf){
 		$nb = count($lcpa);
-		if (defined('_DIR_PLUGINS_SUPPL'))
-			$nb += count($lcpas);
+		if (defined('_DIR_PLUGINS_SUPPL')) {
+					$nb += count($lcpas);
+		}
 		echo "<h3>".sinon(singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'), _T('plugins_actif_aucun'))."</h3>";
 	}
 
-	if (empty($format))
-	  $format = 'liste';
-	elseif (!in_array($format,array('liste','repertoires')))
-		$format = 'repertoires';
+	if (empty($format)) {
+		  $format = 'liste';
+	} elseif (!in_array($format,array('liste','repertoires'))) {
+			$format = 'repertoires';
+	}
 
 	$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);
+	if (defined('_DIR_PLUGINS_SUPPL')) {
+			$corps .= $afficher(self(),$lcpaffichesup, $lcpas, $actifs, _DIR_PLUGINS_SUPPL);
+	}
 
-	if ($corps)
-	  $corps .= "\n<div class='boutons' style='display:none;'>"
+	if ($corps) {
+		  $corps .= "\n<div class='boutons' style='display:none;'>"
 	    .  "<input type='submit' class='submit save' value='"._T('bouton_enregistrer')
 	    ."' />"
 	    . "</div>";
+	}
 
 	echo redirige_action_post('activer_plugins','activer','admin_plugin','', $corps);
 
 	echo fin_cadre_trait_couleur(true);
 
-	if ($quoi=='actifs')
-		echo affiche_les_plugins_verrouilles($actifs);
+	if ($quoi=='actifs') {
+			echo affiche_les_plugins_verrouilles($actifs);
+	}
 	echo "</div>";
 	
 	echo 	http_script("
@@ -188,7 +200,9 @@ 
 
 function affiche_les_plugins_verrouilles($actifs)
 {
-	if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) return '';
+	if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) {
+	    return '';
+	}
 
 	$afficher = charger_fonction("afficher_liste",'plugins');
 	$liste = $afficher(self(), $liste, array(), $actifs, _DIR_PLUGINS_DIST);
@@ -212,12 +226,15 @@ 
  */
 function afficher_librairies(){
 
-	if (!$libs = liste_librairies()) return '';
+	if (!$libs = liste_librairies()) {
+	    return '';
+	}
 	ksort($libs);
 	$res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
 	$res .= '<dl>';
-	foreach ($libs as $lib => $rep)
-		$res .= "<dt>$lib</dt><dd>".joli_repertoire($rep)."</dd>\n";
+	foreach ($libs as $lib => $rep) {
+			$res .= "<dt>$lib</dt><dd>".joli_repertoire($rep)."</dd>\n";
+	}
 	$res .= '</dl>';
 	$res .= fin_cadre_enfonce(true);
 	return $res;
@@ -238,8 +255,9 @@ 
 		AND $t = @opendir($dir)) {
 			while (($f = readdir($t)) !== false) {
 				if ($f[0] != '.'
-				AND is_dir("$dir/$f"))
-					$libs[$f] = $dir;
+				AND is_dir("$dir/$f")) {
+									$libs[$f] = $dir;
+				}
 			}
 		}
 	}

ecrire/xml/indenter.php

Indentation +17 added lines, -17 removed lines

@@ -36,30 +36,30 @@
 // http://doc.spip.org/@phraserTout
 function phraserTout($phraseur, $data)
 {
-	xml_parsestring($this, $data);
+    xml_parsestring($this, $data);
 }
 
- var $depth = "";
- var $res = "";
- var $err = array();
- var $contenu = array();
- var $ouvrant = array();
- var $reperes = array();
- var $entete = '';
- var $page = '';
- var $dtc = NULL;
- var $sax = NULL;
+    var $depth = "";
+    var $res = "";
+    var $err = array();
+    var $contenu = array();
+    var $ouvrant = array();
+    var $reperes = array();
+    var $entete = '';
+    var $page = '';
+    var $dtc = NULL;
+    var $sax = NULL;
 }
 
 // http://doc.spip.org/@xml_indenter_dist
 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;
+    $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;
 }
 
 ?>

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;
 }
 
 ?>

Braces +6 added lines, -2 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;
+}
 
 class IndenteurXML {
 
@@ -57,7 +59,9 @@ 
 	$sax = charger_fonction('sax', 'xml');
 	$f = new IndenteurXML();
 	$sax($page, $apply, $f);
-	if (!$f->err) return $f->entete . $f->res;
+	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/valider.php

Indentation +219 added lines, -219 removed lines

@@ -21,251 +21,251 @@ 
 // http://doc.spip.org/@validerElement
 function validerElement($phraseur, $name, $attrs)
 {
-	if (!($p = isset($this->dtc->elements[$name]))) {
-		if ($p = strpos($name, ':')) {
-			$name = substr($name, $p+1);
-			$p = isset($this->dtc->elements[$name]);
-		}
-		if (!$p) {
-			coordonnees_erreur($this," <b>$name</b>&nbsp;: "
-				. _T('zxml_inconnu_balise'));
-			return; 
-		}
-	}
-	// controler les filles illegitimes, ca suffit 
-	$depth = $this->depth;
-	$ouvrant = $this->ouvrant;
+    if (!($p = isset($this->dtc->elements[$name]))) {
+        if ($p = strpos($name, ':')) {
+            $name = substr($name, $p+1);
+            $p = isset($this->dtc->elements[$name]);
+        }
+        if (!$p) {
+            coordonnees_erreur($this," <b>$name</b>&nbsp;: "
+                . _T('zxml_inconnu_balise'));
+            return; 
+        }
+    }
+    // controler les filles illegitimes, ca suffit 
+    $depth = $this->depth;
+    $ouvrant = $this->ouvrant;
 #	spip_log("trouve $name apres " . $ouvrant[$depth]);
-	if (isset($ouvrant[$depth])) {
-	    if (preg_match('/^\s*(\w+)/', $ouvrant[$depth], $r)) {
-	      $pere = $r[1];
+    if (isset($ouvrant[$depth])) {
+        if (preg_match('/^\s*(\w+)/', $ouvrant[$depth], $r)) {
+            $pere = $r[1];
 #	      spip_log("pere $pere");
-	      if (isset($this->dtc->elements[$pere])) {
-		$fils = $this->dtc->elements[$pere];
+            if (isset($this->dtc->elements[$pere])) {
+        $fils = $this->dtc->elements[$pere];
 #		spip_log("rejeton $name fils " . @join(',',$fils));
-		if (!($p = @in_array($name, $fils))) {
-			if ($p = strpos($name, ':')) {
-				$p = substr($name, $p+1);
-				$p = @in_array($p, $fils);
-			}
-		}
-		if (!$p) {
-	          $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);
-			if (!isset($this->fratrie[$frat])) {
-				$this->fratrie[$frat] = '';
-			}
-			$this->fratrie[$frat] .= "$name ";
-		}
-	      }
-	    }
-	}
-	// Init de la suite des balises a memoriser si regle difficile
-	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])))
-			coordonnees_erreur($this, " <b>$n</b>"
-			  . '&nbsp;:&nbsp;'
-			  . _T('zxml_obligatoire_attribut')
-			  . " <b>$name</b>");
-		    }
-	}
+        if (!($p = @in_array($name, $fils))) {
+            if ($p = strpos($name, ':')) {
+                $p = substr($name, $p+1);
+                $p = @in_array($p, $fils);
+            }
+        }
+        if (!$p) {
+                $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);
+            if (!isset($this->fratrie[$frat])) {
+                $this->fratrie[$frat] = '';
+            }
+            $this->fratrie[$frat] .= "$name ";
+        }
+            }
+        }
+    }
+    // Init de la suite des balises a memoriser si regle difficile
+    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])))
+            coordonnees_erreur($this, " <b>$n</b>"
+                . '&nbsp;:&nbsp;'
+                . _T('zxml_obligatoire_attribut')
+                . " <b>$name</b>");
+            }
+    }
 }
 
 // http://doc.spip.org/@validerAttribut
 function validerAttribut($phraseur, $name, $val, $bal)
 {
-	// Si la balise est inconnue, eviter d'insister
-	if (!isset($this->dtc->attributs[$bal]))
-		return ;
+    // Si la balise est inconnue, eviter d'insister
+    if (!isset($this->dtc->attributs[$bal]))
+        return ;
 		
-	$a = $this->dtc->attributs[$bal];
-	if (!isset($a[$name])) {
-		$bons = join(', ',array_keys($a));
-		if ($bons)
-		  $bons = " title=' " .
-		    _T('zxml_connus_attributs') .
-		    '&nbsp;: ' .
-		    $bons .
-		    "'";
-		$bons .= " style='font-weight: bold'";
-		coordonnees_erreur($this, " <b>$name</b> "
-		. _T('zxml_inconnu_attribut').' '._T('zxml_de')
-		. " <a$bons>$bal</a> ("
-		. _T('zxml_survoler')
-		. ")");
-	} 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))
-			$this->$f($phraseur, $name, $val, $bal);
+    $a = $this->dtc->attributs[$bal];
+    if (!isset($a[$name])) {
+        $bons = join(', ',array_keys($a));
+        if ($bons)
+            $bons = " title=' " .
+            _T('zxml_connus_attributs') .
+            '&nbsp;: ' .
+            $bons .
+            "'";
+        $bons .= " style='font-weight: bold'";
+        coordonnees_erreur($this, " <b>$name</b> "
+        . _T('zxml_inconnu_attribut').' '._T('zxml_de')
+        . " <a$bons>$bal</a> ("
+        . _T('zxml_survoler')
+        . ")");
+    } 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))
+            $this->$f($phraseur, $name, $val, $bal);
 #		else spip_log("$type type d'attribut inconnu");
-	}
+    }
 }
 
 function validerAttribut_NMTOKEN($phraseur, $name, $val, $bal)
 {
-	$this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_NMTOKEN);
+    $this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_NMTOKEN);
 }
 
 function validerAttribut_NMTOKENS($phraseur, $name, $val, $bal)
 {
-	$this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_NMTOKENS);
+    $this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_NMTOKENS);
 }
 
 // http://doc.spip.org/@validerAttribut_ID
 function validerAttribut_ID($phraseur, $name, $val, $bal)
 {
-	if (isset($this->ids[$val])) {
-		list($l,$c) = $this->ids[$val];
-		coordonnees_erreur($this, " <p><b>$val</b> "
-		      . _T('zxml_valeur_attribut')
-		      . " <b>$name</b> "
-		      . _T('zxml_de')
-		      . " <b>$bal</b> "
-		      . _T('zxml_vu')
-		      . " (L$l,C$c)");
-	} else {
-		$this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_ID);
-		$this->ids[$val] = array(xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
-	}
+    if (isset($this->ids[$val])) {
+        list($l,$c) = $this->ids[$val];
+        coordonnees_erreur($this, " <p><b>$val</b> "
+                . _T('zxml_valeur_attribut')
+                . " <b>$name</b> "
+                . _T('zxml_de')
+                . " <b>$bal</b> "
+                . _T('zxml_vu')
+                . " (L$l,C$c)");
+    } else {
+        $this->valider_motif($phraseur, $name, $val, $bal, _REGEXP_ID);
+        $this->ids[$val] = array(xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
+    }
 }
 
 // http://doc.spip.org/@validerAttribut_IDREF
 function validerAttribut_IDREF($phraseur, $name, $val, $bal)
 {
-	$this->idrefs[] = array($val, xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
+    $this->idrefs[] = array($val, xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
 }
 
 // http://doc.spip.org/@validerAttribut_IDREFS
 function validerAttribut_IDREFS($phraseur, $name, $val, $bal)
 {
-	$this->idrefss[] = array($val, xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
+    $this->idrefss[] = array($val, xml_get_current_line_number($phraseur), xml_get_current_column_number($phraseur));
 }
 
 // http://doc.spip.org/@valider_motif
 function valider_motif($phraseur, $name, $val, $bal, $motif)
 {
-	if (!preg_match($motif, $val)) {
-		coordonnees_erreur($this, "<b>$val</b> "
-		. _T('zxml_valeur_attribut')
-		. " <b>$name</b> "
-		. _T('zxml_de')
-		. " <b>$bal</b> "
-		. _T('zxml_non_conforme')
-		. "</p><p>"
-		. "<b>" . $motif . "</b>");
-	}
+    if (!preg_match($motif, $val)) {
+        coordonnees_erreur($this, "<b>$val</b> "
+        . _T('zxml_valeur_attribut')
+        . " <b>$name</b> "
+        . _T('zxml_de')
+        . " <b>$bal</b> "
+        . _T('zxml_non_conforme')
+        . "</p><p>"
+        . "<b>" . $motif . "</b>");
+    }
 }
 
 // http://doc.spip.org/@valider_idref
 function valider_idref($nom, $ligne, $col)
 {
-	if (!isset($this->ids[$nom]))
-		$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
+    if (!isset($this->ids[$nom]))
+        $this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
 }
 
 // http://doc.spip.org/@valider_passe2
 function valider_passe2()
 {
-	if (!$this->err) {
-		foreach ($this->idrefs as $idref) {
-			list($nom, $ligne, $col) = $idref;
-			$this->valider_idref($nom, $ligne, $col);
-		}
-		foreach ($this->idrefss as $idref) {
-			list($noms, $ligne, $col) = $idref;
-			foreach(preg_split('/\s+/', $noms) as $nom)
-				$this->valider_idref($nom, $ligne, $col);
-		}
-	}
+    if (!$this->err) {
+        foreach ($this->idrefs as $idref) {
+            list($nom, $ligne, $col) = $idref;
+            $this->valider_idref($nom, $ligne, $col);
+        }
+        foreach ($this->idrefss as $idref) {
+            list($noms, $ligne, $col) = $idref;
+            foreach(preg_split('/\s+/', $noms) as $nom)
+                $this->valider_idref($nom, $ligne, $col);
+        }
+    }
 }
 
 // http://doc.spip.org/@debutElement
 function debutElement($phraseur, $name, $attrs)
 { 
-	if ($this->dtc->elements)
-		$this->validerElement($phraseur, $name, $attrs);
+    if ($this->dtc->elements)
+        $this->validerElement($phraseur, $name, $attrs);
 
-	if ($f = $this->process['debut']) $f($this, $name, $attrs);
-	$depth = $this->depth;
-	$this->debuts[$depth] =  strlen($this->res);
-	foreach ($attrs as $k => $v) {
-		$this->validerAttribut($phraseur, $k, $v, $name);
-	}
+    if ($f = $this->process['debut']) $f($this, $name, $attrs);
+    $depth = $this->depth;
+    $this->debuts[$depth] =  strlen($this->res);
+    foreach ($attrs as $k => $v) {
+        $this->validerAttribut($phraseur, $k, $v, $name);
+    }
 }
 
 // http://doc.spip.org/@finElement
 function finElement($phraseur, $name)
 {
-	$depth = $this->depth;
-	$contenu = $this->contenu;
+    $depth = $this->depth;
+    $contenu = $this->contenu;
 
-	$n = strlen($this->res);
-	$c = strlen(trim($contenu[$depth]));
-	$k = $this->debuts[$depth];
+    $n = strlen($this->res);
+    $c = strlen(trim($contenu[$depth]));
+    $k = $this->debuts[$depth];
 
-	$regle = $this->dtc->regles[$name];
-	$vide = ($regle  == 'EMPTY');
-	// controler que les balises devant etre vides le sont 
-	if ($vide) {
-		if ($n <> ($k + $c))
-			coordonnees_erreur($this, " <p><b>$name</b> "
-					   . _T('zxml_nonvide_balise'));
-	// pour les regles PCDATA ou iteration de disjonction, tout est fait
-	} elseif ($regle AND ($regle != '*')) {
-		if ($regle == '+') {
-		    // iteration de disjonction non vide: 1 balise au -
-			if ($n == $k) {
-				coordonnees_erreur($this, "<p>\n<b>$name</b> "
-				  . _T('zxml_vide_balise'));
-			}
-		} else {
-			$f = $this->fratrie[substr($depth,2)];
-			if (!preg_match($regle, $f)) {
-				coordonnees_erreur($this,
-				" <p>\n<b>$name</b> "
-				  .  _T('zxml_succession_fils_incorrecte')
-				  . '&nbsp;: <b>'
-				  . $f
-				  . '</b>');
-			}
-		}
+    $regle = $this->dtc->regles[$name];
+    $vide = ($regle  == 'EMPTY');
+    // controler que les balises devant etre vides le sont 
+    if ($vide) {
+        if ($n <> ($k + $c))
+            coordonnees_erreur($this, " <p><b>$name</b> "
+                        . _T('zxml_nonvide_balise'));
+    // pour les regles PCDATA ou iteration de disjonction, tout est fait
+    } elseif ($regle AND ($regle != '*')) {
+        if ($regle == '+') {
+            // iteration de disjonction non vide: 1 balise au -
+            if ($n == $k) {
+                coordonnees_erreur($this, "<p>\n<b>$name</b> "
+                    . _T('zxml_vide_balise'));
+            }
+        } else {
+            $f = $this->fratrie[substr($depth,2)];
+            if (!preg_match($regle, $f)) {
+                coordonnees_erreur($this,
+                " <p>\n<b>$name</b> "
+                    .  _T('zxml_succession_fils_incorrecte')
+                    . '&nbsp;: <b>'
+                    . $f
+                    . '</b>');
+            }
+        }
 
-	}
-	if ($f = $this->process['fin']) $f($this, $name, $vide);
+    }
+    if ($f = $this->process['fin']) $f($this, $name, $vide);
 }
 
 // http://doc.spip.org/@textElement
 function textElement($phraseur, $data)
 {	
-	if (trim($data)) {
-		$d = $this->depth;
-		$d = $this->ouvrant[$d];
-		preg_match('/^\s*(\S+)/', $d, $m);
-		if ($this->dtc->pcdata[$m[1]]) {
-			coordonnees_erreur($this, " <p><b>". $m[1] . "</b> "
-			. _T('zxml_nonvide_balise') // message a affiner
-			);
-		}
-	}
-	if ($f = $this->process['text']) $f($this, $data);
+    if (trim($data)) {
+        $d = $this->depth;
+        $d = $this->ouvrant[$d];
+        preg_match('/^\s*(\S+)/', $d, $m);
+        if ($this->dtc->pcdata[$m[1]]) {
+            coordonnees_erreur($this, " <p><b>". $m[1] . "</b> "
+            . _T('zxml_nonvide_balise') // message a affiner
+            );
+        }
+    }
+    if ($f = $this->process['text']) $f($this, $data);
 }
 
 function piElement($phraseur, $target, $data)
 {
-	if ($f = $this->process['pi']) $f($this, $target, $data);
+    if ($f = $this->process['pi']) $f($this, $target, $data);
 }
 
 // Denonciation des entitees XML inconnues
@@ -278,61 +278,61 @@ 
 // http://doc.spip.org/@defautElement
 function defaultElement($phraseur, $data)
 {
-	if (!preg_match('/^<!--/', $data)
-	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER)))
-		foreach ($r as $m) {
-			list($t,$e) = $m;
-			if (!isset($this->dtc->entites[$e]))
-				coordonnees_erreur($this, " <b>$e</b> "
-				  . _T('zxml_inconnu_entite')
-				  . ' '
-				  );
-		}
-	if (isset($this->process['default']) AND ($f = $this->process['default'])) {
-		$f($this, $data);
-	}
+    if (!preg_match('/^<!--/', $data)
+    AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER)))
+        foreach ($r as $m) {
+            list($t,$e) = $m;
+            if (!isset($this->dtc->entites[$e]))
+                coordonnees_erreur($this, " <b>$e</b> "
+                    . _T('zxml_inconnu_entite')
+                    . ' '
+                    );
+        }
+    if (isset($this->process['default']) AND ($f = $this->process['default'])) {
+        $f($this, $data);
+    }
 }
 
 // http://doc.spip.org/@phraserTout
 function phraserTout($phraseur, $data)
 { 
-	xml_parsestring($this, $data);
+    xml_parsestring($this, $data);
 
-	if (!$this->dtc OR preg_match(',^' . _MESSAGE_DOCTYPE . ',', $data)) {
-		$this->err[]= array('DOCTYPE ?', 0, 0);
-	} else {
-		$this->valider_passe2($this);
-	}
+    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()) {
-	if (is_array($process)) $this->process = $process;
+    if (is_array($process)) $this->process = $process;
 }
 
- var $ids = array();
- var $idrefs = array();
- var $idrefss = array();
- var $debuts = array();
- var $fratrie = array();
+    var $ids = array();
+    var $idrefs = array();
+    var $idrefss = array();
+    var $debuts = array();
+    var $fratrie = array();
 
- var $dtc = NULL;
- var $sax = NULL;
- var $depth = "";
- var $entete = '';
- var $page = '';
- var $res = "";
- var $err = array();
- var $contenu = array();
- var $ouvrant = array();
- var $reperes = array();
- var $process =  array(
-			'debut' => 'xml_debutElement',
-			'fin' => 'xml_finElement',
-			'text' => 'xml_textElement',
-			'pi' => 'xml_piElement',
-			'default' => 'xml_defaultElement'
-				 );
+    var $dtc = NULL;
+    var $sax = NULL;
+    var $depth = "";
+    var $entete = '';
+    var $page = '';
+    var $res = "";
+    var $err = array();
+    var $contenu = array();
+    var $ouvrant = array();
+    var $reperes = array();
+    var $process =  array(
+            'debut' => 'xml_debutElement',
+            'fin' => 'xml_finElement',
+            'text' => 'xml_textElement',
+            'pi' => 'xml_piElement',
+            'default' => 'xml_defaultElement'
+                    );
 }
 
 // Retourne une structure ValidateurXML, dont le champ "err" est un tableau
@@ -341,8 +341,8 @@ 
 // http://doc.spip.org/@xml_valider_dist
 function xml_valider_dist($page, $apply=false, $process=false, $doctype='', $charset=null)
 {
-	$f = new ValidateurXML($process);
-	$sax = charger_fonction('sax', 'xml');
-	return $sax($page, $apply, $f, $doctype, $charset);
+    $f = new ValidateurXML($process);
+    $sax = charger_fonction('sax', 'xml');
+    return $sax($page, $apply, $f, $doctype, $charset);
 }
 ?>

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');

Braces +59 added lines, -36 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;
+}
 
 // Validateur XML en deux passes, fonde sur SAX pour la premiere
 // Faudrait faire deux classes car pour la premiere passe
@@ -69,16 +71,18 @@ 
 	    }
 	}
 	// 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])))
-			coordonnees_erreur($this, " <b>$n</b>"
+		    { if (($v[1] == '#REQUIRED') AND (!isset($attrs[$n]))) {
+		    			coordonnees_erreur($this, " <b>$n</b>"
 			  . '&nbsp;:&nbsp;'
 			  . _T('zxml_obligatoire_attribut')
 			  . " <b>$name</b>");
 		    }
+		    }
 	}
 }
 
@@ -86,18 +90,20 @@ 
 function validerAttribut($phraseur, $name, $val, $bal)
 {
 	// Si la balise est inconnue, eviter d'insister
-	if (!isset($this->dtc->attributs[$bal]))
-		return ;
+	if (!isset($this->dtc->attributs[$bal])) {
+			return ;
+	}
 		
 	$a = $this->dtc->attributs[$bal];
 	if (!isset($a[$name])) {
 		$bons = join(', ',array_keys($a));
-		if ($bons)
-		  $bons = " title=' " .
+		if ($bons) {
+				  $bons = " title=' " .
 		    _T('zxml_connus_attributs') .
 		    '&nbsp;: ' .
 		    $bons .
 		    "'";
+		}
 		$bons .= " style='font-weight: bold'";
 		coordonnees_erreur($this, " <b>$name</b> "
 		. _T('zxml_inconnu_attribut').' '._T('zxml_de')
@@ -106,11 +112,12 @@ 
 		. ")");
 	} 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))
-			$this->$f($phraseur, $name, $val, $bal);
-#		else spip_log("$type type d'attribut inconnu");
+		if (!preg_match('/^\w+$/', $type)) {
+					$this->valider_motif($phraseur, $name, $val, $bal, $type);
+		} else if (method_exists($this, $f = 'validerAttribut_' . $type)) {
+					$this->$f($phraseur, $name, $val, $bal);
+		}
+		#		else spip_log("$type type d'attribut inconnu");
 	}
 }
 
@@ -172,9 +179,10 @@ 
 // http://doc.spip.org/@valider_idref
 function valider_idref($nom, $ligne, $col)
 {
-	if (!isset($this->ids[$nom]))
-		$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
-}
+	if (!isset($this->ids[$nom])) {
+			$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
+	}
+	}
 
 // http://doc.spip.org/@valider_passe2
 function valider_passe2()
@@ -186,8 +194,9 @@ 
 		}
 		foreach ($this->idrefss as $idref) {
 			list($noms, $ligne, $col) = $idref;
-			foreach(preg_split('/\s+/', $noms) as $nom)
-				$this->valider_idref($nom, $ligne, $col);
+			foreach(preg_split('/\s+/', $noms) as $nom) {
+							$this->valider_idref($nom, $ligne, $col);
+			}
 		}
 	}
 }
@@ -195,10 +204,13 @@ 
 // http://doc.spip.org/@debutElement
 function debutElement($phraseur, $name, $attrs)
 { 
-	if ($this->dtc->elements)
-		$this->validerElement($phraseur, $name, $attrs);
+	if ($this->dtc->elements) {
+			$this->validerElement($phraseur, $name, $attrs);
+	}
 
-	if ($f = $this->process['debut']) $f($this, $name, $attrs);
+	if ($f = $this->process['debut']) {
+	    $f($this, $name, $attrs);
+	}
 	$depth = $this->depth;
 	$this->debuts[$depth] =  strlen($this->res);
 	foreach ($attrs as $k => $v) {
@@ -220,9 +232,10 @@ 
 	$vide = ($regle  == 'EMPTY');
 	// controler que les balises devant etre vides le sont 
 	if ($vide) {
-		if ($n <> ($k + $c))
-			coordonnees_erreur($this, " <p><b>$name</b> "
+		if ($n <> ($k + $c)) {
+					coordonnees_erreur($this, " <p><b>$name</b> "
 					   . _T('zxml_nonvide_balise'));
+		}
 	// pour les regles PCDATA ou iteration de disjonction, tout est fait
 	} elseif ($regle AND ($regle != '*')) {
 		if ($regle == '+') {
@@ -244,8 +257,10 @@ 
 		}
 
 	}
-	if ($f = $this->process['fin']) $f($this, $name, $vide);
-}
+	if ($f = $this->process['fin']) {
+	    $f($this, $name, $vide);
+	}
+	}
 
 // http://doc.spip.org/@textElement
 function textElement($phraseur, $data)
@@ -260,13 +275,17 @@ 
 			);
 		}
 	}
-	if ($f = $this->process['text']) $f($this, $data);
-}
+	if ($f = $this->process['text']) {
+	    $f($this, $data);
+	}
+	}
 
 function piElement($phraseur, $target, $data)
 {
-	if ($f = $this->process['pi']) $f($this, $target, $data);
-}
+	if ($f = $this->process['pi']) {
+	    $f($this, $target, $data);
+	}
+	}
 
 // Denonciation des entitees XML inconnues
 // Pour contourner le bug de conception de SAX qui ne signale pas si elles
@@ -279,14 +298,16 @@ 
 function defaultElement($phraseur, $data)
 {
 	if (!preg_match('/^<!--/', $data)
-	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER)))
-		foreach ($r as $m) {
+	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER))) {
+			foreach ($r as $m) {
 			list($t,$e) = $m;
-			if (!isset($this->dtc->entites[$e]))
-				coordonnees_erreur($this, " <b>$e</b> "
+	}
+			if (!isset($this->dtc->entites[$e])) {
+							coordonnees_erreur($this, " <b>$e</b> "
 				  . _T('zxml_inconnu_entite')
 				  . ' '
 				  );
+			}
 		}
 	if (isset($this->process['default']) AND ($f = $this->process['default'])) {
 		$f($this, $data);
@@ -307,8 +328,10 @@ 
 
 // Init
 function ValidateurXML($process=array()) {
-	if (is_array($process)) $this->process = $process;
-}
+	if (is_array($process)) {
+	    $this->process = $process;
+	}
+	}
 
  var $ids = array();
  var $idrefs = array();

ecrire/xml/interfaces.php

Indentation +15 added lines, -15 removed lines

@@ -13,7 +13,7 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 define('_REGEXP_DOCTYPE',
-	'/^((?:<\001?[?][^>]*>\s*)*(?:<!--.*?-->\s*)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]*)>\s*/s');
+    '/^((?:<\001?[?][^>]*>\s*)*(?:<!--.*?-->\s*)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]*)>\s*/s');
 
 define('_REGEXP_XML', '/^(\s*(?:<[?][^x>][^>]*>\s*)?(?:<[?]xml[^>]*>)?\s*(?:<!--.*?-->\s*)*)<(\w+)/s');
 
@@ -31,14 +31,14 @@ 
 define('_REGEXP_ENTITY_DEF', '/^%('  . _SUB_REGEXP_SYMBOL . '+);/');
 define('_REGEXP_TYPE_XML', 'PUBLIC|SYSTEM|INCLUDE|IGNORE|CDATA');
 define('_REGEXP_ENTITY_DECL', '/^<!ENTITY\s+(%?)\s*(' .
-		_SUB_REGEXP_SYMBOL .
-		'+;?)\s+(' .
-		_REGEXP_TYPE_XML .
-		')?\s*(' .
-		"('([^']*)')" .
-		'|("([^"]*)")' .
+        _SUB_REGEXP_SYMBOL .
+        '+;?)\s+(' .
+        _REGEXP_TYPE_XML .
+        ')?\s*(' .
+        "('([^']*)')" .
+        '|("([^"]*)")' .
                 '|\s*(%' . _SUB_REGEXP_SYMBOL . '+;)\s*' .
-       		')\s*(--.*?--)?("([^"]*)")?\s*>\s*(.*)$/s');
+                ')\s*(--.*?--)?("([^"]*)")?\s*>\s*(.*)$/s');
 
 define('_REGEXP_INCLUDE_USE', '/^<!\[\s*%\s*([^;]*);\s*\[\s*(.*)$/s');
 
@@ -47,12 +47,12 @@ 
 // Document Type Compilation
 
 class DTC {
-	var	$macros = array();
-	var 	$elements = array();
-	var 	$peres = array();
-	var 	$attributs = array();
-	var	$entites = array();
-	var	$regles = array();
-	var	$pcdata = array();
+    var	$macros = array();
+    var 	$elements = array();
+    var 	$peres = array();
+    var 	$attributs = array();
+    var	$entites = array();
+    var	$regles = array();
+    var	$pcdata = array();
 }
 ?>

Spacing +13 added lines, -13 removed lines

@@ -21,23 +21,23 @@
 
 define('_SUB_REGEXP_SYMBOL', '[\w_:.-]');
 
-define('_REGEXP_NMTOKEN', '/^'  . _SUB_REGEXP_SYMBOL . '+$/');
+define('_REGEXP_NMTOKEN', '/^'._SUB_REGEXP_SYMBOL.'+$/');
 
-define('_REGEXP_NMTOKENS', '/^('  . _SUB_REGEXP_SYMBOL . '+\s*)*$/');
+define('_REGEXP_NMTOKENS', '/^('._SUB_REGEXP_SYMBOL.'+\s*)*$/');
 
-define('_REGEXP_ID', '/^[A-Za-z_:]' . _SUB_REGEXP_SYMBOL . '*$/');
+define('_REGEXP_ID', '/^[A-Za-z_:]'._SUB_REGEXP_SYMBOL.'*$/');
 
-define('_REGEXP_ENTITY_USE', '/%('  . _SUB_REGEXP_SYMBOL . '+);/');
-define('_REGEXP_ENTITY_DEF', '/^%('  . _SUB_REGEXP_SYMBOL . '+);/');
+define('_REGEXP_ENTITY_USE', '/%('._SUB_REGEXP_SYMBOL.'+);/');
+define('_REGEXP_ENTITY_DEF', '/^%('._SUB_REGEXP_SYMBOL.'+);/');
 define('_REGEXP_TYPE_XML', 'PUBLIC|SYSTEM|INCLUDE|IGNORE|CDATA');
-define('_REGEXP_ENTITY_DECL', '/^<!ENTITY\s+(%?)\s*(' .
-		_SUB_REGEXP_SYMBOL .
-		'+;?)\s+(' .
-		_REGEXP_TYPE_XML .
-		')?\s*(' .
-		"('([^']*)')" .
-		'|("([^"]*)")' .
-                '|\s*(%' . _SUB_REGEXP_SYMBOL . '+;)\s*' .
+define('_REGEXP_ENTITY_DECL', '/^<!ENTITY\s+(%?)\s*('.
+		_SUB_REGEXP_SYMBOL.
+		'+;?)\s+('.
+		_REGEXP_TYPE_XML.
+		')?\s*('.
+		"('([^']*)')".
+		'|("([^"]*)")'.
+                '|\s*(%'._SUB_REGEXP_SYMBOL.'+;)\s*'.
        		')\s*(--.*?--)?("([^"]*)")?\s*>\s*(.*)$/s');
 
 define('_REGEXP_INCLUDE_USE', '/^<!\[\s*%\s*([^;]*);\s*\[\s*(.*)$/s');

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;
+}
 
 define('_REGEXP_DOCTYPE',
 	'/^((?:<\001?[?][^>]*>\s*)*(?:<!--.*?-->\s*)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]*)>\s*/s');

ecrire/base/trouver_table.php

Indentation +115 added lines, -115 removed lines

@@ -62,120 +62,120 @@
  *
 **/
 function base_trouver_table_dist($nom, $serveur='', $table_spip = true){
-	static $nom_cache_desc_sql=array();
-	global $tables_principales, $tables_auxiliaires, $table_des_tables;
-
-	if (!spip_connect($serveur)
-	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom))
-		return null;
-
-	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
-	$objets_sql = lister_tables_objets_sql("::md5");
-
-	// le nom du cache depend du serveur mais aussi du nom de la db et du prefixe
-	// ce qui permet une auto invalidation en cas de modif manuelle du fichier
-	// de connexion, et tout risque d'ambiguite
-	if (!isset($nom_cache_desc_sql[$serveur][$objets_sql])){
-		$nom_cache_desc_sql[$serveur][$objets_sql] =
-		  _DIR_CACHE . 'sql_desc_'
-		  . ($serveur ? "{$serveur}_":"")
-		  . substr(md5($connexion['db'].":".$connexion['prefixe'].":$objets_sql"),0,8)
-			.'.txt';
-		// nouveau nom de cache = nouvelle version en memoire
-		unset($connexion['tables']);
-	}
-
-	// un appel avec $nom vide est une demande explicite de vidange du cache des descriptions
-	if (!$nom){
-		spip_unlink($nom_cache_desc_sql[$serveur][$objets_sql]);
-		$connexion['tables'] = array();
-		return null;
-	}
-
-	$nom_sql = $nom;
-	if (preg_match('/\.(.*)$/', $nom, $s))
-		$nom_sql = $s[1];
-	else
-		$nom_sql = $nom;
-
-	$fdesc = $desc = '';
-	$connexion = &$GLOBALS['connexions'][$serveur ? $serveur : 0];
-
-	// base sous SPIP: gerer les abreviations explicites des noms de table
-	if ($connexion['spip_connect_version']) {
-		if ($table_spip AND isset($table_des_tables[$nom])) {
-			$nom = $table_des_tables[$nom];
-			$nom_sql = 'spip_' . $nom;
-		}
-	}
-
-	// si c'est la premiere table qu'on cherche
-	// et si on est pas explicitement en recalcul
-	// on essaye de recharger le cache des decriptions de ce serveur
-	// dans le fichier cache
-	if (!isset($connexion['tables'][$nom_sql])
-	  AND defined('_VAR_MODE') AND _VAR_MODE!=='recalcul'
-	  AND (!isset($connexion['tables']) OR !$connexion['tables'])) {
-		if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],$desc_cache)
-		  AND $desc_cache=unserialize($desc_cache))
-		  $connexion['tables'] = $desc_cache;
-	}
-	if ($table_spip AND !isset($connexion['tables'][$nom_sql])) {
-
-		if (isset($tables_principales[$nom_sql]))
-			$fdesc = $tables_principales[$nom_sql];
-		// meme si pas d'abreviation declaree, trouver la table spip_$nom
-		// si c'est une table principale,
-		// puisqu'on le fait aussi pour les tables auxiliaires
-		elseif ($nom_sql==$nom AND isset($tables_principales['spip_' .$nom])){
-			$nom_sql = 'spip_' . $nom;
-			$fdesc = &$tables_principales[$nom_sql];
-		}
-		elseif (isset($tables_auxiliaires[$n=$nom])
-		  OR isset($tables_auxiliaires[$n='spip_'.$nom])) {
-			$nom_sql = $n;
-			$fdesc = &$tables_auxiliaires[$n];
-		}  # table locale a cote de SPIP, comme non SPIP:
-	}
-	if (!isset($connexion['tables'][$nom_sql])) {
-
-		// La *vraie* base a la priorite
-		$desc = sql_showtable($nom_sql, $table_spip, $serveur);
-		if (!$desc OR !$desc['field']) {
-			if (!$fdesc) {
-				spip_log("trouver_table: table inconnue '$serveur' '$nom'",_LOG_INFO_IMPORTANTE);
-				return null;
-			}
-			// on ne sait pas lire la structure de la table :
-			// on retombe sur la description donnee dans les fichiers spip
-			$desc = $fdesc;
-		}
-		else {
-			$desc['exist'] = true;
-		}
-
-		$desc['table'] = $desc['table_sql'] = $nom_sql;
-		$desc['connexion']= $serveur;
-
-		// charger les infos declarees pour cette table
-		// en lui passant les infos connues
-		// $desc est prioritaire pour la description de la table
-		$desc = array_merge(lister_tables_objets_sql($nom_sql,$desc),$desc);
-
-		// si tables_objets_sql est bien fini d'init, on peut cacher
-		$connexion['tables'][$nom_sql] = $desc;
-		$res = &$connexion['tables'][$nom_sql];
-		// une nouvelle table a ete decrite
-		// mettons donc a jour le cache des descriptions de ce serveur
-		if (is_writeable(_DIR_CACHE))
-			ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
-	}
-	else
-		$res = &$connexion['tables'][$nom_sql];
-
-	// toujours retourner $nom dans id_table
-	$res['id_table']=$nom;
-
-	return $res;
+    static $nom_cache_desc_sql=array();
+    global $tables_principales, $tables_auxiliaires, $table_des_tables;
+
+    if (!spip_connect($serveur)
+    OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom))
+        return null;
+
+    $connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
+    $objets_sql = lister_tables_objets_sql("::md5");
+
+    // le nom du cache depend du serveur mais aussi du nom de la db et du prefixe
+    // ce qui permet une auto invalidation en cas de modif manuelle du fichier
+    // de connexion, et tout risque d'ambiguite
+    if (!isset($nom_cache_desc_sql[$serveur][$objets_sql])){
+        $nom_cache_desc_sql[$serveur][$objets_sql] =
+            _DIR_CACHE . 'sql_desc_'
+            . ($serveur ? "{$serveur}_":"")
+            . substr(md5($connexion['db'].":".$connexion['prefixe'].":$objets_sql"),0,8)
+            .'.txt';
+        // nouveau nom de cache = nouvelle version en memoire
+        unset($connexion['tables']);
+    }
+
+    // un appel avec $nom vide est une demande explicite de vidange du cache des descriptions
+    if (!$nom){
+        spip_unlink($nom_cache_desc_sql[$serveur][$objets_sql]);
+        $connexion['tables'] = array();
+        return null;
+    }
+
+    $nom_sql = $nom;
+    if (preg_match('/\.(.*)$/', $nom, $s))
+        $nom_sql = $s[1];
+    else
+        $nom_sql = $nom;
+
+    $fdesc = $desc = '';
+    $connexion = &$GLOBALS['connexions'][$serveur ? $serveur : 0];
+
+    // base sous SPIP: gerer les abreviations explicites des noms de table
+    if ($connexion['spip_connect_version']) {
+        if ($table_spip AND isset($table_des_tables[$nom])) {
+            $nom = $table_des_tables[$nom];
+            $nom_sql = 'spip_' . $nom;
+        }
+    }
+
+    // si c'est la premiere table qu'on cherche
+    // et si on est pas explicitement en recalcul
+    // on essaye de recharger le cache des decriptions de ce serveur
+    // dans le fichier cache
+    if (!isset($connexion['tables'][$nom_sql])
+      AND defined('_VAR_MODE') AND _VAR_MODE!=='recalcul'
+      AND (!isset($connexion['tables']) OR !$connexion['tables'])) {
+        if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],$desc_cache)
+          AND $desc_cache=unserialize($desc_cache))
+            $connexion['tables'] = $desc_cache;
+    }
+    if ($table_spip AND !isset($connexion['tables'][$nom_sql])) {
+
+        if (isset($tables_principales[$nom_sql]))
+            $fdesc = $tables_principales[$nom_sql];
+        // meme si pas d'abreviation declaree, trouver la table spip_$nom
+        // si c'est une table principale,
+        // puisqu'on le fait aussi pour les tables auxiliaires
+        elseif ($nom_sql==$nom AND isset($tables_principales['spip_' .$nom])){
+            $nom_sql = 'spip_' . $nom;
+            $fdesc = &$tables_principales[$nom_sql];
+        }
+        elseif (isset($tables_auxiliaires[$n=$nom])
+          OR isset($tables_auxiliaires[$n='spip_'.$nom])) {
+            $nom_sql = $n;
+            $fdesc = &$tables_auxiliaires[$n];
+        }  # table locale a cote de SPIP, comme non SPIP:
+    }
+    if (!isset($connexion['tables'][$nom_sql])) {
+
+        // La *vraie* base a la priorite
+        $desc = sql_showtable($nom_sql, $table_spip, $serveur);
+        if (!$desc OR !$desc['field']) {
+            if (!$fdesc) {
+                spip_log("trouver_table: table inconnue '$serveur' '$nom'",_LOG_INFO_IMPORTANTE);
+                return null;
+            }
+            // on ne sait pas lire la structure de la table :
+            // on retombe sur la description donnee dans les fichiers spip
+            $desc = $fdesc;
+        }
+        else {
+            $desc['exist'] = true;
+        }
+
+        $desc['table'] = $desc['table_sql'] = $nom_sql;
+        $desc['connexion']= $serveur;
+
+        // charger les infos declarees pour cette table
+        // en lui passant les infos connues
+        // $desc est prioritaire pour la description de la table
+        $desc = array_merge(lister_tables_objets_sql($nom_sql,$desc),$desc);
+
+        // si tables_objets_sql est bien fini d'init, on peut cacher
+        $connexion['tables'][$nom_sql] = $desc;
+        $res = &$connexion['tables'][$nom_sql];
+        // une nouvelle table a ete decrite
+        // mettons donc a jour le cache des descriptions de ce serveur
+        if (is_writeable(_DIR_CACHE))
+            ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+    }
+    else
+        $res = &$connexion['tables'][$nom_sql];
+
+    // toujours retourner $nom dans id_table
+    $res['id_table']=$nom;
+
+    return $res;
 }
 ?>

Spacing +21 added lines, -21 removed lines

@@ -61,12 +61,12 @@ 
  *     
  *
 **/
-function base_trouver_table_dist($nom, $serveur='', $table_spip = true){
-	static $nom_cache_desc_sql=array();
+function base_trouver_table_dist($nom, $serveur = '', $table_spip = true) {
+	static $nom_cache_desc_sql = array();
 	global $tables_principales, $tables_auxiliaires, $table_des_tables;
 
 	if (!spip_connect($serveur)
-	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom))
+	OR !preg_match('/^[a-zA-Z0-9._-]*/', $nom))
 		return null;
 
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
@@ -75,18 +75,18 @@ 
 	// le nom du cache depend du serveur mais aussi du nom de la db et du prefixe
 	// ce qui permet une auto invalidation en cas de modif manuelle du fichier
 	// de connexion, et tout risque d'ambiguite
-	if (!isset($nom_cache_desc_sql[$serveur][$objets_sql])){
+	if (!isset($nom_cache_desc_sql[$serveur][$objets_sql])) {
 		$nom_cache_desc_sql[$serveur][$objets_sql] =
-		  _DIR_CACHE . 'sql_desc_'
-		  . ($serveur ? "{$serveur}_":"")
-		  . substr(md5($connexion['db'].":".$connexion['prefixe'].":$objets_sql"),0,8)
+		  _DIR_CACHE.'sql_desc_'
+		  . ($serveur ? "{$serveur}_" : "")
+		  . substr(md5($connexion['db'].":".$connexion['prefixe'].":$objets_sql"), 0, 8)
 			.'.txt';
 		// nouveau nom de cache = nouvelle version en memoire
 		unset($connexion['tables']);
 	}
 
 	// un appel avec $nom vide est une demande explicite de vidange du cache des descriptions
-	if (!$nom){
+	if (!$nom) {
 		spip_unlink($nom_cache_desc_sql[$serveur][$objets_sql]);
 		$connexion['tables'] = array();
 		return null;
@@ -105,7 +105,7 @@ 
 	if ($connexion['spip_connect_version']) {
 		if ($table_spip AND isset($table_des_tables[$nom])) {
 			$nom = $table_des_tables[$nom];
-			$nom_sql = 'spip_' . $nom;
+			$nom_sql = 'spip_'.$nom;
 		}
 	}
 
@@ -114,10 +114,10 @@ 
 	// on essaye de recharger le cache des decriptions de ce serveur
 	// dans le fichier cache
 	if (!isset($connexion['tables'][$nom_sql])
-	  AND defined('_VAR_MODE') AND _VAR_MODE!=='recalcul'
+	  AND defined('_VAR_MODE') AND _VAR_MODE !== 'recalcul'
 	  AND (!isset($connexion['tables']) OR !$connexion['tables'])) {
-		if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],$desc_cache)
-		  AND $desc_cache=unserialize($desc_cache))
+		if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql], $desc_cache)
+		  AND $desc_cache = unserialize($desc_cache))
 		  $connexion['tables'] = $desc_cache;
 	}
 	if ($table_spip AND !isset($connexion['tables'][$nom_sql])) {
@@ -127,12 +127,12 @@ 
 		// meme si pas d'abreviation declaree, trouver la table spip_$nom
 		// si c'est une table principale,
 		// puisqu'on le fait aussi pour les tables auxiliaires
-		elseif ($nom_sql==$nom AND isset($tables_principales['spip_' .$nom])){
-			$nom_sql = 'spip_' . $nom;
+		elseif ($nom_sql == $nom AND isset($tables_principales['spip_'.$nom])) {
+			$nom_sql = 'spip_'.$nom;
 			$fdesc = &$tables_principales[$nom_sql];
 		}
-		elseif (isset($tables_auxiliaires[$n=$nom])
-		  OR isset($tables_auxiliaires[$n='spip_'.$nom])) {
+		elseif (isset($tables_auxiliaires[$n = $nom])
+		  OR isset($tables_auxiliaires[$n = 'spip_'.$nom])) {
 			$nom_sql = $n;
 			$fdesc = &$tables_auxiliaires[$n];
 		}  # table locale a cote de SPIP, comme non SPIP:
@@ -143,7 +143,7 @@ 
 		$desc = sql_showtable($nom_sql, $table_spip, $serveur);
 		if (!$desc OR !$desc['field']) {
 			if (!$fdesc) {
-				spip_log("trouver_table: table inconnue '$serveur' '$nom'",_LOG_INFO_IMPORTANTE);
+				spip_log("trouver_table: table inconnue '$serveur' '$nom'", _LOG_INFO_IMPORTANTE);
 				return null;
 			}
 			// on ne sait pas lire la structure de la table :
@@ -155,12 +155,12 @@ 
 		}
 
 		$desc['table'] = $desc['table_sql'] = $nom_sql;
-		$desc['connexion']= $serveur;
+		$desc['connexion'] = $serveur;
 
 		// charger les infos declarees pour cette table
 		// en lui passant les infos connues
 		// $desc est prioritaire pour la description de la table
-		$desc = array_merge(lister_tables_objets_sql($nom_sql,$desc),$desc);
+		$desc = array_merge(lister_tables_objets_sql($nom_sql, $desc), $desc);
 
 		// si tables_objets_sql est bien fini d'init, on peut cacher
 		$connexion['tables'][$nom_sql] = $desc;
@@ -168,13 +168,13 @@ 
 		// une nouvelle table a ete decrite
 		// mettons donc a jour le cache des descriptions de ce serveur
 		if (is_writeable(_DIR_CACHE))
-			ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+			ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql], serialize($connexion['tables']), true);
 	}
 	else
 		$res = &$connexion['tables'][$nom_sql];
 
 	// toujours retourner $nom dans id_table
-	$res['id_table']=$nom;
+	$res['id_table'] = $nom;
 
 	return $res;
 }

Braces +24 added lines, -19 removed lines

@@ -15,7 +15,9 @@ 
  *
  * @package SPIP\SQL\Tables
 **/
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 include_spip('base/objets');
 
 /**
@@ -66,8 +68,9 @@ 
 	global $tables_principales, $tables_auxiliaires, $table_des_tables;
 
 	if (!spip_connect($serveur)
-	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom))
-		return null;
+	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom)) {
+			return null;
+	}
 
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 	$objets_sql = lister_tables_objets_sql("::md5");
@@ -93,10 +96,11 @@ 
 	}
 
 	$nom_sql = $nom;
-	if (preg_match('/\.(.*)$/', $nom, $s))
-		$nom_sql = $s[1];
-	else
-		$nom_sql = $nom;
+	if (preg_match('/\.(.*)$/', $nom, $s)) {
+			$nom_sql = $s[1];
+	} else {
+			$nom_sql = $nom;
+	}
 
 	$fdesc = $desc = '';
 	$connexion = &$GLOBALS['connexions'][$serveur ? $serveur : 0];
@@ -117,21 +121,22 @@ 
 	  AND defined('_VAR_MODE') AND _VAR_MODE!=='recalcul'
 	  AND (!isset($connexion['tables']) OR !$connexion['tables'])) {
 		if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],$desc_cache)
-		  AND $desc_cache=unserialize($desc_cache))
-		  $connexion['tables'] = $desc_cache;
+		  AND $desc_cache=unserialize($desc_cache)) {
+				  $connexion['tables'] = $desc_cache;
+		}
 	}
 	if ($table_spip AND !isset($connexion['tables'][$nom_sql])) {
 
-		if (isset($tables_principales[$nom_sql]))
-			$fdesc = $tables_principales[$nom_sql];
+		if (isset($tables_principales[$nom_sql])) {
+					$fdesc = $tables_principales[$nom_sql];
+		}
 		// meme si pas d'abreviation declaree, trouver la table spip_$nom
 		// si c'est une table principale,
 		// puisqu'on le fait aussi pour les tables auxiliaires
 		elseif ($nom_sql==$nom AND isset($tables_principales['spip_' .$nom])){
 			$nom_sql = 'spip_' . $nom;
 			$fdesc = &$tables_principales[$nom_sql];
-		}
-		elseif (isset($tables_auxiliaires[$n=$nom])
+		} elseif (isset($tables_auxiliaires[$n=$nom])
 		  OR isset($tables_auxiliaires[$n='spip_'.$nom])) {
 			$nom_sql = $n;
 			$fdesc = &$tables_auxiliaires[$n];
@@ -149,8 +154,7 @@ 
 			// on ne sait pas lire la structure de la table :
 			// on retombe sur la description donnee dans les fichiers spip
 			$desc = $fdesc;
-		}
-		else {
+		} else {
 			$desc['exist'] = true;
 		}
 
@@ -167,11 +171,12 @@ 
 		$res = &$connexion['tables'][$nom_sql];
 		// une nouvelle table a ete decrite
 		// mettons donc a jour le cache des descriptions de ce serveur
-		if (is_writeable(_DIR_CACHE))
-			ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+		if (is_writeable(_DIR_CACHE)) {
+					ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+		}
+	} else {
+			$res = &$connexion['tables'][$nom_sql];
 	}
-	else
-		$res = &$connexion['tables'][$nom_sql];
 
 	// toujours retourner $nom dans id_table
 	$res['id_table']=$nom;

ecrire/base/create.php

Indentation +79 added lines, -79 removed lines

@@ -26,17 +26,17 @@ 
  * @return bool
  */
 function base_determine_autoinc($table,$desc=array()){
-	if ($t=lister_tables_principales() AND isset($t[$table]))
-		$autoinc = true;
-	elseif ($t=lister_tables_auxiliaires() AND isset($t[$table]))
-		$autoinc = false;
-	else {
-		// essayer de faire au mieux !
-		$autoinc = (isset($desc['key']['PRIMARY KEY'])
-						AND strpos($desc['key']['PRIMARY KEY'],',')===false
-						AND strpos($desc['field'][$desc['key']['PRIMARY KEY']],'default')===false);
-	}
-	return $autoinc;
+    if ($t=lister_tables_principales() AND isset($t[$table]))
+        $autoinc = true;
+    elseif ($t=lister_tables_auxiliaires() AND isset($t[$table]))
+        $autoinc = false;
+    else {
+        // essayer de faire au mieux !
+        $autoinc = (isset($desc['key']['PRIMARY KEY'])
+                        AND strpos($desc['key']['PRIMARY KEY'],',')===false
+                        AND strpos($desc['field'][$desc['key']['PRIMARY KEY']],'default')===false);
+    }
+    return $autoinc;
 }
 
 /**
@@ -55,51 +55,51 @@ 
  * @return void
  */
 function creer_ou_upgrader_table($table,$desc,$autoinc,$upgrade=false,$serveur='') {
-	#spip_log("creer_ou_upgrader_table table=$table autoinc=$autoinc upgrade=$upgrade","dbinstall"._LOG_INFO_IMPORTANTE);
-	$sql_desc = $upgrade ? sql_showtable($table,true,$serveur) : false;
-	#if (!$sql_desc) $sql_desc = false;
-	#spip_log("table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
-	if (!$sql_desc) {
-		if ($autoinc==='auto')
-			$autoinc = base_determine_autoinc($table,$desc);
-		#spip_log("sql_create $table autoinc=$autoinc","dbinstall"._LOG_INFO_IMPORTANTE);
-		sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
-		// verifier la bonne installation de la table (php-fpm es-tu la ?)
-		$sql_desc = sql_showtable($table,true,$serveur);
-		#if (!$sql_desc) $sql_desc = false;
-		#spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
-		if (!$sql_desc){
-			// on retente avec un sleep ?
-			sleep(1);
-			sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
-			$sql_desc = sql_showtable($table,true,$serveur);
-			#if (!$sql_desc) $sql_desc = false;
-			#spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
-			if (!$sql_desc){
-				spip_log("Echec creation table $table","maj"._LOG_CRITIQUE);
-			}
-		}
-	}
-	else {
-		#spip_log("sql_alter $table ... (on s'en fiche)","dbinstall"._LOG_INFO_IMPORTANTE);
-		// ajouter les champs manquants
-		// on ne supprime jamais les champs, car c'est dangereux
-		// c'est toujours a faire manuellement
-		$last = '';
-		foreach($desc['field'] as $field=>$type){
-			if (!isset($sql_desc['field'][$field]))
-				sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
-			$last = $field;
-		}
-		foreach($desc['key'] as $key=>$type){
-			// Ne pas oublier les cas des cles non nommees dans la declaration et qui sont retournees
-			// par le showtable sous la forme d'un index de tableau "KEY $type" et non "KEY"
-			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"]))
-				sql_alter("TABLE $table ADD $key ($type)",$serveur);
-			$last = $field;
-		}
+    #spip_log("creer_ou_upgrader_table table=$table autoinc=$autoinc upgrade=$upgrade","dbinstall"._LOG_INFO_IMPORTANTE);
+    $sql_desc = $upgrade ? sql_showtable($table,true,$serveur) : false;
+    #if (!$sql_desc) $sql_desc = false;
+    #spip_log("table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
+    if (!$sql_desc) {
+        if ($autoinc==='auto')
+            $autoinc = base_determine_autoinc($table,$desc);
+        #spip_log("sql_create $table autoinc=$autoinc","dbinstall"._LOG_INFO_IMPORTANTE);
+        sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
+        // verifier la bonne installation de la table (php-fpm es-tu la ?)
+        $sql_desc = sql_showtable($table,true,$serveur);
+        #if (!$sql_desc) $sql_desc = false;
+        #spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
+        if (!$sql_desc){
+            // on retente avec un sleep ?
+            sleep(1);
+            sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
+            $sql_desc = sql_showtable($table,true,$serveur);
+            #if (!$sql_desc) $sql_desc = false;
+            #spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
+            if (!$sql_desc){
+                spip_log("Echec creation table $table","maj"._LOG_CRITIQUE);
+            }
+        }
+    }
+    else {
+        #spip_log("sql_alter $table ... (on s'en fiche)","dbinstall"._LOG_INFO_IMPORTANTE);
+        // ajouter les champs manquants
+        // on ne supprime jamais les champs, car c'est dangereux
+        // c'est toujours a faire manuellement
+        $last = '';
+        foreach($desc['field'] as $field=>$type){
+            if (!isset($sql_desc['field'][$field]))
+                sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+            $last = $field;
+        }
+        foreach($desc['key'] as $key=>$type){
+            // Ne pas oublier les cas des cles non nommees dans la declaration et qui sont retournees
+            // par le showtable sous la forme d'un index de tableau "KEY $type" et non "KEY"
+            if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"]))
+                sql_alter("TABLE $table ADD $key ($type)",$serveur);
+            $last = $field;
+        }
 
-	}
+    }
 }
 
 /**
@@ -120,20 +120,20 @@ 
  */
 function alterer_base($tables_inc, $tables_noinc, $up=false, $serveur='')
 {
-	if ($up === false) {
-		$old = false;
-		$up = array();
-	} else {
-		$old = true;
-		if (!is_array($up)) $up = array($up);
-	}
-	foreach($tables_inc as $k => $v)
-		if (!$old OR in_array($k, $up))
-			creer_ou_upgrader_table($k,$v,true,$old,$serveur);
+    if ($up === false) {
+        $old = false;
+        $up = array();
+    } else {
+        $old = true;
+        if (!is_array($up)) $up = array($up);
+    }
+    foreach($tables_inc as $k => $v)
+        if (!$old OR in_array($k, $up))
+            creer_ou_upgrader_table($k,$v,true,$old,$serveur);
 
-	foreach($tables_noinc as $k => $v)
-		if (!$old OR in_array($k, $up))
-			creer_ou_upgrader_table($k,$v,false,$old,$serveur);
+    foreach($tables_noinc as $k => $v)
+        if (!$old OR in_array($k, $up))
+            creer_ou_upgrader_table($k,$v,false,$old,$serveur);
 }
 
 /**
@@ -147,14 +147,14 @@ 
  */
 function creer_base($serveur='') {
 
-	// Note: les mises a jour reexecutent ce code pour s'assurer
-	// de la conformite de la base
-	// pas de panique sur  "already exists" et "duplicate entry" donc.
+    // Note: les mises a jour reexecutent ce code pour s'assurer
+    // de la conformite de la base
+    // pas de panique sur  "already exists" et "duplicate entry" donc.
 
-	alterer_base(lister_tables_principales(),
-		     lister_tables_auxiliaires(),
-		     false,
-		     $serveur);
+    alterer_base(lister_tables_principales(),
+                lister_tables_auxiliaires(),
+                false,
+                $serveur);
 }
 
 /**
@@ -167,10 +167,10 @@ 
  * @return void
  */
 function maj_tables($upgrade_tables=array(),$serveur=''){
-	alterer_base(lister_tables_principales(),
-		     lister_tables_auxiliaires(),
-		     $upgrade_tables,
-		     $serveur);
+    alterer_base(lister_tables_principales(),
+                lister_tables_auxiliaires(),
+                $upgrade_tables,
+                $serveur);
 }
 
 ?>

Spacing +25 added lines, -25 removed lines

@@ -25,16 +25,16 @@ 
  * @param array $desc
  * @return bool
  */
-function base_determine_autoinc($table,$desc=array()){
-	if ($t=lister_tables_principales() AND isset($t[$table]))
+function base_determine_autoinc($table, $desc = array()) {
+	if ($t = lister_tables_principales() AND isset($t[$table]))
 		$autoinc = true;
-	elseif ($t=lister_tables_auxiliaires() AND isset($t[$table]))
+	elseif ($t = lister_tables_auxiliaires() AND isset($t[$table]))
 		$autoinc = false;
 	else {
 		// essayer de faire au mieux !
 		$autoinc = (isset($desc['key']['PRIMARY KEY'])
-						AND strpos($desc['key']['PRIMARY KEY'],',')===false
-						AND strpos($desc['field'][$desc['key']['PRIMARY KEY']],'default')===false);
+						AND strpos($desc['key']['PRIMARY KEY'], ',') === false
+						AND strpos($desc['field'][$desc['key']['PRIMARY KEY']], 'default') === false);
 	}
 	return $autoinc;
 }
@@ -54,29 +54,29 @@ 
  * @param string $serveur
  * @return void
  */
-function creer_ou_upgrader_table($table,$desc,$autoinc,$upgrade=false,$serveur='') {
+function creer_ou_upgrader_table($table, $desc, $autoinc, $upgrade = false, $serveur = '') {
 	#spip_log("creer_ou_upgrader_table table=$table autoinc=$autoinc upgrade=$upgrade","dbinstall"._LOG_INFO_IMPORTANTE);
-	$sql_desc = $upgrade ? sql_showtable($table,true,$serveur) : false;
+	$sql_desc = $upgrade ? sql_showtable($table, true, $serveur) : false;
 	#if (!$sql_desc) $sql_desc = false;
 	#spip_log("table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
 	if (!$sql_desc) {
-		if ($autoinc==='auto')
-			$autoinc = base_determine_autoinc($table,$desc);
+		if ($autoinc === 'auto')
+			$autoinc = base_determine_autoinc($table, $desc);
 		#spip_log("sql_create $table autoinc=$autoinc","dbinstall"._LOG_INFO_IMPORTANTE);
 		sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
 		// verifier la bonne installation de la table (php-fpm es-tu la ?)
-		$sql_desc = sql_showtable($table,true,$serveur);
+		$sql_desc = sql_showtable($table, true, $serveur);
 		#if (!$sql_desc) $sql_desc = false;
 		#spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
-		if (!$sql_desc){
+		if (!$sql_desc) {
 			// on retente avec un sleep ?
 			sleep(1);
 			sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
-			$sql_desc = sql_showtable($table,true,$serveur);
+			$sql_desc = sql_showtable($table, true, $serveur);
 			#if (!$sql_desc) $sql_desc = false;
 			#spip_log("Resultat table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
-			if (!$sql_desc){
-				spip_log("Echec creation table $table","maj"._LOG_CRITIQUE);
+			if (!$sql_desc) {
+				spip_log("Echec creation table $table", "maj"._LOG_CRITIQUE);
 			}
 		}
 	}
@@ -86,16 +86,16 @@ 
 		// on ne supprime jamais les champs, car c'est dangereux
 		// c'est toujours a faire manuellement
 		$last = '';
-		foreach($desc['field'] as $field=>$type){
+		foreach ($desc['field'] as $field=>$type) {
 			if (!isset($sql_desc['field'][$field]))
-				sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+				sql_alter("TABLE $table ADD $field $type".($last ? " AFTER $last" : ""), $serveur);
 			$last = $field;
 		}
-		foreach($desc['key'] as $key=>$type){
+		foreach ($desc['key'] as $key=>$type) {
 			// Ne pas oublier les cas des cles non nommees dans la declaration et qui sont retournees
 			// par le showtable sous la forme d'un index de tableau "KEY $type" et non "KEY"
 			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"]))
-				sql_alter("TABLE $table ADD $key ($type)",$serveur);
+				sql_alter("TABLE $table ADD $key ($type)", $serveur);
 			$last = $field;
 		}
 
@@ -118,7 +118,7 @@ 
  *   serveur sql
  * @return void
  */
-function alterer_base($tables_inc, $tables_noinc, $up=false, $serveur='')
+function alterer_base($tables_inc, $tables_noinc, $up = false, $serveur = '')
 {
 	if ($up === false) {
 		$old = false;
@@ -127,13 +127,13 @@ 
 		$old = true;
 		if (!is_array($up)) $up = array($up);
 	}
-	foreach($tables_inc as $k => $v)
+	foreach ($tables_inc as $k => $v)
 		if (!$old OR in_array($k, $up))
-			creer_ou_upgrader_table($k,$v,true,$old,$serveur);
+			creer_ou_upgrader_table($k, $v, true, $old, $serveur);
 
-	foreach($tables_noinc as $k => $v)
+	foreach ($tables_noinc as $k => $v)
 		if (!$old OR in_array($k, $up))
-			creer_ou_upgrader_table($k,$v,false,$old,$serveur);
+			creer_ou_upgrader_table($k, $v, false, $old, $serveur);
 }
 
 /**
@@ -145,7 +145,7 @@ 
  * @param string $serveur
  * @return void
  */
-function creer_base($serveur='') {
+function creer_base($serveur = '') {
 
 	// Note: les mises a jour reexecutent ce code pour s'assurer
 	// de la conformite de la base
@@ -166,7 +166,7 @@ 
  * @param string $serveur
  * @return void
  */
-function maj_tables($upgrade_tables=array(),$serveur=''){
+function maj_tables($upgrade_tables = array(), $serveur = '') {
 	alterer_base(lister_tables_principales(),
 		     lister_tables_auxiliaires(),
 		     $upgrade_tables,

Braces +28 added lines, -20 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/acces');
 include_spip('base/objets');
@@ -26,11 +28,11 @@ 
  * @return bool
  */
 function base_determine_autoinc($table,$desc=array()){
-	if ($t=lister_tables_principales() AND isset($t[$table]))
-		$autoinc = true;
-	elseif ($t=lister_tables_auxiliaires() AND isset($t[$table]))
-		$autoinc = false;
-	else {
+	if ($t=lister_tables_principales() AND isset($t[$table])) {
+			$autoinc = true;
+	} elseif ($t=lister_tables_auxiliaires() AND isset($t[$table])) {
+			$autoinc = false;
+	} else {
 		// essayer de faire au mieux !
 		$autoinc = (isset($desc['key']['PRIMARY KEY'])
 						AND strpos($desc['key']['PRIMARY KEY'],',')===false
@@ -60,8 +62,9 @@ 
 	#if (!$sql_desc) $sql_desc = false;
 	#spip_log("table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
 	if (!$sql_desc) {
-		if ($autoinc==='auto')
-			$autoinc = base_determine_autoinc($table,$desc);
+		if ($autoinc==='auto') {
+					$autoinc = base_determine_autoinc($table,$desc);
+		}
 		#spip_log("sql_create $table autoinc=$autoinc","dbinstall"._LOG_INFO_IMPORTANTE);
 		sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
 		// verifier la bonne installation de la table (php-fpm es-tu la ?)
@@ -79,23 +82,24 @@ 
 				spip_log("Echec creation table $table","maj"._LOG_CRITIQUE);
 			}
 		}
-	}
-	else {
+	} else {
 		#spip_log("sql_alter $table ... (on s'en fiche)","dbinstall"._LOG_INFO_IMPORTANTE);
 		// ajouter les champs manquants
 		// on ne supprime jamais les champs, car c'est dangereux
 		// c'est toujours a faire manuellement
 		$last = '';
 		foreach($desc['field'] as $field=>$type){
-			if (!isset($sql_desc['field'][$field]))
-				sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+			if (!isset($sql_desc['field'][$field])) {
+							sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+			}
 			$last = $field;
 		}
 		foreach($desc['key'] as $key=>$type){
 			// Ne pas oublier les cas des cles non nommees dans la declaration et qui sont retournees
 			// par le showtable sous la forme d'un index de tableau "KEY $type" et non "KEY"
-			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"]))
-				sql_alter("TABLE $table ADD $key ($type)",$serveur);
+			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"])) {
+							sql_alter("TABLE $table ADD $key ($type)",$serveur);
+			}
 			$last = $field;
 		}
 
@@ -125,16 +129,20 @@ 
 		$up = array();
 	} else {
 		$old = true;
-		if (!is_array($up)) $up = array($up);
+		if (!is_array($up)) {
+		    $up = array($up);
+		}
 	}
-	foreach($tables_inc as $k => $v)
-		if (!$old OR in_array($k, $up))
+	foreach($tables_inc as $k => $v) {
+			if (!$old OR in_array($k, $up))
 			creer_ou_upgrader_table($k,$v,true,$old,$serveur);
+	}
 
-	foreach($tables_noinc as $k => $v)
-		if (!$old OR in_array($k, $up))
+	foreach($tables_noinc as $k => $v) {
+			if (!$old OR in_array($k, $up))
 			creer_ou_upgrader_table($k,$v,false,$old,$serveur);
-}
+	}
+	}
 
 /**
  * Creer une base de donnee