Utiliser systématiquement sql_fetsel plutôt que sql_fetch quand c'est...

Utiliser systématiquement sql_fetsel plutôt que sql_fetch quand c'est possible, afin de profiter de l'optimisation de [11738], qui évite en particulier des opérations bêtement interdites par SQLite à cause de son compteur de références.

ecrire/action/copier_local.php

@@ -38,8 +38,7 @@ function action_copier_local_dist() {
 function action_copier_local_post($id_document) {
 
 	// Il faut la source du document pour le copier
-	$s = sql_select("fichier, descriptif", "spip_documents", "id_document=$id_document");
-	$row = sql_fetch($s);
+	$ow = sql_fetsel("fichier, descriptif", "spip_documents", "id_document=$id_document");
 	$source = $row['fichier'];
 
 	include_spip('inc/distant'); // pour 'copie_locale'

ecrire/action/editer_site.php

@@ -176,8 +176,7 @@ function revisions_sites ($id_syndic, $c=false) {
 		$c);
 
 
-	$s = sql_select("statut, id_rubrique, id_secteur", "spip_syndic", "id_syndic=$id_syndic");
-	$row = sql_fetch($s);
+	$row = sql_fetsel("statut, id_rubrique, id_secteur", "spip_syndic", "id_syndic=$id_syndic");
 	$id_rubrique = $row['id_rubrique'];
 	$statut_ancien = $row['statut'];
 	$id_secteur_old = $row['id_secteur'];

ecrire/action/instituer_forum.php

@@ -20,9 +20,8 @@ function action_instituer_forum_dist() {
 
 	list($id_forum, $statut) = preg_split('/\W/', $arg);
 	$id_forum = intval($id_forum);
-	$result = sql_select("*", "spip_forum", "id_forum=$id_forum");
-	if (!($row = sql_fetch($result)))
-		return;
+	$row = sql_fetsel("*", "spip_forum", "id_forum=$id_forum");
+	if (!$row) return;
 
 	// invalider les pages comportant ce forum
 	include_spip('inc/invalideur');

ecrire/action/reorganiser.php

@@ -36,8 +36,7 @@ function reorganiser_article_rubrique($id_article, $id_rubrique)
 
 		include_spip('action/editer_article');
 		include_spip('inc/rubriques');
-		$s = sql_select("statut, id_rubrique", "spip_articles", "id_article=$id_article");
-		$s = sql_fetch($s);
+		$s = sql_fetsel("statut, id_rubrique", "spip_articles", "id_article=$id_article");
 		editer_article_heritage($id_article,
 					$s['id_rubrique'], 
 					$s['statut'],
@@ -54,12 +53,10 @@ function reorganiser_rubrique_rubrique($id_quoi, $id_cible)
 		if (!$id_cible)
 			$id_secteur = $id_quoi;
 		else {
-			$s = sql_select("id_secteur", "spip_rubriques", "id_rubrique=$id_cible");
-			$s = sql_fetch($s);
-			$id_secteur = $s['id_secteur'];
+			$id_secteur = sql_getfetsel("id_secteur", "spip_rubriques", "id_rubrique=$id_cible");
 		}
 
-		$s = sql_select("statut, id_parent", "spip_rubriques", "id_rubrique=$id_quoi");
+		$s = sql_fetsel("statut, id_parent", "spip_rubriques", "id_rubrique=$id_quoi");
 
 		sql_updateq('spip_rubriques', array('id_parent' => $id_cible, 'id_secteur'=>$id_secteur),  "id_rubrique=".sql_quote($id_quoi));
 

ecrire/action/supprimer.php

@@ -77,8 +77,8 @@ function action_supprimer_rubrique($r)
 // http://doc.spip.org/@supprimer_document_et_vignette
 function supprimer_document_et_vignette($arg)
 {
-	$result = sql_select("id_vignette, fichier", "spip_documents", "id_document=$arg");
-	if ($row = sql_fetch($result)) {
+	$row = sql_fetsel("id_vignette, fichier", "spip_documents", "id_document=$arg");
+	if ($row) {
 		spip_unlink(get_spip_doc($row['fichier']));
 		sql_delete("spip_documents", "id_document=$arg");
 		sql_updateq("spip_documents", array("id_vignette" => 0), "id_vignette=$arg");
@@ -87,11 +87,9 @@ function supprimer_document_et_vignette($arg)
 		sql_delete("spip_documents_breves", "id_document=$arg");
 		$id_vignette = $row['id_vignette'];
 		if ($id_vignette > 0) {
-			$result = sql_select("fichier", "spip_documents	", "id_document=$id_vignette");
+			$f = sql_getfetsel("fichier", "spip_documents	", "id_document=$id_vignette");
 
-			if ($row = sql_fetch($result)) {
-				spip_unlink(get_spip_doc($row['fichier']));
-			}
+			if ($f) spip_unlink(get_spip_doc($f));
 			sql_delete("spip_documents", "id_document=$id_vignette");
 			sql_delete("spip_documents_articles", "id_document=$id_vignette");
 			sql_delete("spip_documents_rubriques", "id_document=$id_vignette");

ecrire/action/tester_taille.php

@@ -49,10 +49,8 @@ function action_tester_taille_dist() {
 			$dummy = propre("<doc1>");
 
 			$image_source = chemin_image("test.png");
-			$res = sql_select("valeur", "spip_meta", "nom='max_taille_vignettes_test'");
-			if ($row = sql_fetch($res))
-				$max_size_test = $row['valeur'];
-			unset ($res); // sinon SQLite ne peut pas ecrire dans la table spip_meta (locked)
+			$row = sql_fetsel("valeur", "spip_meta", "nom='max_taille_vignettes_test'");
+			if ($row) $max_size_test = $row['valeur'];
 			if (!$max_size_test OR $max_size_test>$s)
 				ecrire_meta('max_taille_vignettes_test',$s,'non');
 			$result = filtrer('image_recadre',$image_source,$taille,$taille);

ecrire/action/tourner.php

@@ -28,10 +28,9 @@ function action_tourner_dist() {
 function action_tourner_post($r)
 {
 	$arg = $r[1];
-	$result = sql_select("fichier", "spip_documents", "id_document=$arg");
+	$row = sql_fetsel("fichier", "spip_documents", "id_document=$arg");
 
-	if (!$row = sql_fetch($result))
-		return;
+	if (!$row) return;
 
 	include_spip('inc/charsets');	# pour le nom de fichier
 	include_spip('inc/documents'); 

ecrire/balise/formulaire_inscription.php

@@ -113,8 +113,7 @@ function message_inscription($mail, $nom, $mode, $id=0) {
 	if (is_string($declaration))
 		return  $declaration;
 
-	$row = sql_select("statut, id_auteur, login, email", "spip_auteurs", "email=" . sql_quote($declaration['email']));
-	$row = sql_fetch($row);
+	$row = sql_fetsel("statut, id_auteur, login, email", "spip_auteurs", "email=" . sql_quote($declaration['email']));
 
 	if (!$row) 
 		// il n'existe pas, creer les identifiants  

ecrire/balise/formulaire_signature.php

@@ -203,10 +203,9 @@ function inc_controler_signature_dist($id_article, $nom, $mail, $message, $site,
 
 	// tout le monde est la.
 
-	$result_petition = sql_select('*', 'spip_petitions', "id_article=$id_article");
+	$row = sql_fetsel('*', 'spip_petitions', "id_article=$id_article");
 
-	if (!$row = sql_fetch($result_petition)) 
-		return _T('form_pet_probleme_technique');
+	if (!$row) return _T('form_pet_probleme_technique');
 
 	if ($row['site_obli'] == "oui") {
 		if (!strlen($site)