Deux requêtes SQL incomplètement abstraites, et déclaration plus régulière des...

Deux requêtes SQL incomplètement abstraites, et déclaration plus régulière des fonctions d'abstraction pour distinguer celles qui prennent en argument une ressource, et celles ont de l'analyse d'argument à faire.

ecrire/base/abstract_sql.php

@@ -76,8 +76,7 @@ function sql_set_charset($charset,$serveur='', $option=true){
 // afin de disposer du texte brut.
 
 // http://doc.spip.org/@sql_select
-function sql_select (
-	$select = array(), $from = array(), $where = array(),
+function sql_select ($select = array(), $from = array(), $where = array(),
 	$groupby = array(), $orderby = array(), $limit = '', $having = array(),
 	$serveur='', $option=true) {
 	$f = sql_serveur('select', $serveur,  $option==='continue' OR $option===false);
@@ -109,8 +108,7 @@ function sql_select (
 // Recupere la syntaxe de la requete select sans l'executer
 // simplement $option = false au lieu de true
 // http://doc.spip.org/@sql_get_select
-function sql_get_select	(
-	$select = array(), $from = array(), $where = array(),
+function sql_get_select($select = array(), $from = array(), $where = array(),
 	$groupby = array(), $orderby = array(), $limit = '', $having = array(),
 	$serveur='') {
 	return sql_select ($select, $from, $where, $groupby, $orderby, $limit, $having, $serveur, false);
@@ -174,11 +172,11 @@ function sql_listdbs($serveur='', $option=true) {
 }
 
 // http://doc.spip.org/@sql_selectdb
-function sql_selectdb($res, $serveur='', $option=true)
+function sql_selectdb($nom, $serveur='', $option=true)
 {
 	$f = sql_serveur('selectdb', $serveur,  $option==='continue' OR $option===false);
 	if (!is_string($f) OR !$f) return false;
-	$r = $f($res, $serveur, $option!==false);
+	$r = $f($nom, $serveur, $option!==false);
 	if ($r===false) spip_sql_erreur($serveur);
 	return $r;
 }
@@ -445,12 +443,10 @@ function sql_query($ins, $serveur='', $option=true) {
 
 # une composition tellement frequente...
 // http://doc.spip.org/@sql_fetsel
-function sql_fetsel(
-	$select = array(), $from = array(), $where = array(),
+function sql_fetsel($select = array(), $from = array(), $where = array(),
 	$groupby = array(), $orderby = array(), $limit = '',
 	$having = array(), $serveur='', $option=true) {
 	$q = sql_select($select, $from, $where,	$groupby, $orderby, $limit, $having, $serveur, $option);
-	spip_log("fetsel $select $from $where '$option' $q");
 	if ($option===false) return $q;
 	if (!$q) return array();
 	$r = sql_fetch($q, $serveur, $option);
@@ -460,8 +456,7 @@ function sql_fetsel(
 
 // Retourne le tableau de toutes les lignes d'une requete Select
 // http://doc.spip.org/@sql_allfetsel
-function sql_allfetsel(
-	$select = array(), $from = array(), $where = array(),
+function sql_allfetsel($select = array(), $from = array(), $where = array(),
 	$groupby = array(), $orderby = array(), $limit = '',
 	$having = array(), $serveur='', $option=true) {
 	$q = sql_select($select, $from, $where,	$groupby, $orderby, $limit, $having, $serveur, $option);
@@ -471,8 +466,7 @@ function sql_allfetsel(
 
 # Retourne l'unique champ demande dans une requete Select a resultat unique
 // http://doc.spip.org/@sql_getfetsel
-function sql_getfetsel(
-		       $select, $from = array(), $where = array(), $groupby = array(), 
+function sql_getfetsel($select, $from = array(), $where = array(), $groupby = array(), 
 	$orderby = array(), $limit = '', $having = array(), $serveur='', $option=true) {
 	if (preg_match('/\s+as\s+(\w+)$/i', $select, $c)) $id = $c[1];
 	elseif (!preg_match('/\W/', $select)) $id = $select;

ecrire/install/etape_3.php

@@ -135,14 +135,14 @@ function install_bases($adresse_db, $login_db, $pass_db,  $server_db, $choix_db,
 	  }
 	  // eliminer la derniere operation d'admin mal terminee
 	  // notamment la mise a jour 
-	  sql_query("DELETE FROM spip_meta WHERE nom='import_all' OR  nom='admin'", $server_db);
+	  sql_delete('spip_meta', "nom='import_all' OR  nom='admin'", $server_db);
 	}
 
 	$ligne_rappel = ($server_db != 'mysql') ? ''
 	: (test_rappel_nom_base_mysql($server_db)
 	  .test_sql_mode_mysql($server_db)	);
 
-	$result_ok = sql_query("SELECT COUNT(*) FROM spip_meta", $server_db);
+	$result_ok = sql_countsel('spip_meta', '','','',$server_db);
 	if (!$result_ok) return "<!--\nvielle = $old rappel= $ligne_rappel\n-->";
 
 	if ($chmod_db) {