Les fonctions appelées par inc_admin reçoivent deux arguments, il est...

Les fonctions appelées par inc_admin reçoivent deux arguments, il est largement temps d'avoir systématiquement les 2 paramètres correspondants, car ça permet d'empêcher leur execution si elles ne sont pas appelées à travers elle (notamment un testeur automatique avec les autorisations sans confirmation dans le mes_options).

ecrire/base/admin_repair.php

@@ -13,13 +13,14 @@
 if (!defined("_ECRIRE_INC_VERSION")) return;
 
 // http://doc.spip.org/@base_admin_repair_dist
-function base_admin_repair_dist() {
+function base_admin_repair_dist($titre='', $reprise='') {
 
 	$desc = spip_connect();
-	if (function_exists($f = @$desc['repair'])) {
+	if (function_exists($f = @$desc['repair']) AND $titre) {
 		$res = admin_repair_tables();
 	} else {
-		spip_log("Pas d'instruction REPAIR dans ce serveur SQL");
+		if ($titre)
+		  spip_log("Pas d'instruction REPAIR dans ce serveur SQL");
 		$res = '     ';
 	}
 
@@ -45,7 +46,7 @@ function admin_repair_tables() {
 	if ($res1) { while ($r = sql_fetch($res1)) {
 		$tab = array_shift($r);
 
-		$res .= "<p><b>$tab</b> ";
+		$res .= "<br /><b>$tab</b> ";
 		spip_log("Repare $tab");
 		$result_repair = sql_repair($tab);
 		if (!$result_repair) return false;

ecrire/base/convert_sql_utf8.php

@@ -14,8 +14,9 @@ if (!defined("_ECRIRE_INC_VERSION")) return;
 
 
 // http://doc.spip.org/@base_convert_sql_utf8_dist
-function base_convert_sql_utf8_dist($titre, $reprise=false)
+function base_convert_sql_utf8_dist($titre='', $reprise=false)
 {
+	if (!$titre) return; // anti-testeur automatique
 	ecrire_meta('convert_sql_utf8','oui','non');
 	echo install_debut_html($titre);
 	

ecrire/base/convert_utf8.php

@@ -34,8 +34,9 @@ function convert_utf8_init($tables_a_convertir)
 }
 
 // http://doc.spip.org/@base_convert_utf8_dist
-function base_convert_utf8_dist($titre, $reprise=false)
+function base_convert_utf8_dist($titre='', $reprise=false)
 {
+	if (!$titre) return; // anti-testeur automatique
 	// une liste des tables a convertir, avec le champ dans lequel on
 	// indique '<CONVERT charset>' ; on commence par les rubriques sinon
 	// ca fait desordre dans l'interface privee

ecrire/base/delete_all.php

@@ -23,16 +23,16 @@ function base_delete_all_dist($titre)
 				$res[] = $table;
 			else spip_log("SPIP n'a pas pu detruire $table.");
 		}
-	}
 
 	// un pipeline pour detruire les tables installees par les plugins
-	pipeline('delete_tables', '');
+		pipeline('delete_tables', '');
 
-	spip_unlink(_FILE_CONNECT);
-	spip_unlink(_FILE_CHMOD);
-	spip_unlink(_FILE_META);
-	spip_unlink(_ACCESS_FILE_NAME);
-	spip_unlink(_CACHE_RUBRIQUES);
+		spip_unlink(_FILE_CONNECT);
+		spip_unlink(_FILE_CHMOD);
+		spip_unlink(_FILE_META);
+		spip_unlink(_ACCESS_FILE_NAME);
+		spip_unlink(_CACHE_RUBRIQUES);
+	}
 	$d = count($delete);
 	$r = count($res);
 	spip_log("Tables detruites: $r sur $d: " . join(', ',$res));

ecrire/base/delete_statistiques.php

@@ -18,8 +18,9 @@ if (!defined("_ECRIRE_INC_VERSION")) return; // securiser
 // ca vaudrait mieux
 
 // http://doc.spip.org/@base_delete_statistiques_dist
-function base_delete_statistiques_dist($titre)
+function base_delete_statistiques_dist($titre='', $reprise='')
 {
+	if (!$titre) return; // anti-testeur automatique
 	sql_delete("spip_visites");
 	sql_delete("spip_visites_articles");
 	sql_delete("spip_referers");

ecrire/base/import_all.php

@@ -38,8 +38,9 @@ if ($f = include_spip('mes_fonctions', false)) {
 }
 
 // http://doc.spip.org/@base_import_all_dist
-function base_import_all_dist($titre, $reprise=false)
+function base_import_all_dist($titre='', $reprise=false)
 {
+	if (!$titre) return; // anti-testeur automatique
 	if (!$reprise) import_all_debut();
 
 	$request = unserialize($GLOBALS['meta']['import_all']);

ecrire/base/upgrade.php

@@ -13,8 +13,9 @@
 if (!defined("_ECRIRE_INC_VERSION")) return;
 
 // http://doc.spip.org/@base_upgrade_dist
-function base_upgrade_dist($titre)
+function base_upgrade_dist($titre='', $reprise='')
 {
+	if (!$titre) return; // anti-testeur automatique
 	if ($GLOBALS['spip_version']!=$GLOBALS['meta']['version_installee']) {
 		if (!is_numeric(_request('reinstall'))) {
 			include_spip('base/create');