Skip to content
Extraits de code Groupes Projets
Valider aa23037d rédigé par esj's avatar esj
Parcourir les fichiers

#209: abstration de l'instruction SQL REPLACE dans le script de fusion.... 

#209: abstration de l'instruction SQL REPLACE dans le script de fusion. L'obtention du numéro de l'insertion dans une table à clé primaire multiple (problème évoqué dans [9989]) n'est en fait pas nécessaire à SPIP, on renonce à ça dans la fonction d'abstraction pour le moment. Cela permet à la fusion de fonctionner en PG, toutefois il y a toujours le problème du verrou qui suit dans calculer_rubriques. Par ailleurs, blindage de situations provoquant des erreurs SQL sans réelle gravité mais inutilement alarmantes.

A part ça: Champagne!!!!
parent f9b24df0
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
Masquer les modifications d'espaces
En ligne Côte à côte
ecrire/base/db_pg.php
+ 7
− 3
Voir le fichier @ aa23037d
...@@ -305,9 +305,13 @@ function spip_pg_insert($table, $champs, $valeurs, $desc=array()) { ...@@ -305,9 +305,13 @@ function spip_pg_insert($table, $champs, $valeurs, $desc=array()) {
include_spip('base/serial'); include_spip('base/serial');
$desc = @$tables_principales[$table]; $desc = @$tables_principales[$table];
} }
if (isset($desc['key']["PRIMARY KEY"]))
$ret = " RETURNING " . $desc['key']["PRIMARY KEY"]; // Dans les tables principales de SPIP, le numero de l'insertion
else $ret = ''; // est la valeur de l'unique et atomique cle primaire ===> RETURNING
// Le code actuel n'a pas besoin de ce numero dans les autres cas
// mais il faudra surement amelioer ca un jour.
$seq = @$desc['key']["PRIMARY KEY"];
$ret = preg_match('/\w+/', $seq) ? " RETURNING $seq" : '';
if ($GLOBALS['table_prefix']) if ($GLOBALS['table_prefix'])
$table = preg_replace('/^spip/', $table = preg_replace('/^spip/',
......
ecrire/inc/grouper_mots.php
+ 1
− 1
Voir le fichier @ aa23037d
...@@ -136,7 +136,7 @@ function afficher_groupe_mots_boucle($row, $occurrences, $total, $deb_aff) ...@@ -136,7 +136,7 @@ function afficher_groupe_mots_boucle($row, $occurrences, $total, $deb_aff)
} }
// http://doc.spip.org/@generer_supprimer_mot // http://doc.spip.org/@generer_supprimer_mot
function generer_supprimer_mot($id_mot, $id_groupe, $clic, $total, $deb_aff) function generer_supprimer_mot($id_mot, $id_groupe, $clic, $total, $deb_aff='')
{ {
$cont = ($total > 1) $cont = ($total > 1)
? '' ? ''
......
ecrire/inc/import_insere.php
+ 49
− 17
Voir le fichier @ aa23037d
...@@ -221,12 +221,14 @@ function importe_translate_maj($k, $v) ...@@ -221,12 +221,14 @@ function importe_translate_maj($k, $v)
list($g, $titre, $ajout) = $trans[$k][$v]; list($g, $titre, $ajout) = $trans[$k][$v];
if ($g <= 0) { if ($g <= 0) {
$f = 'import_identifie_parent_' . $k; $f = 'import_identifie_parent_' . $k;
$g = $f($g, $titre, $v); if (function_exists($f)) {
if ($g > 0) $g = $f($g, $titre, $v);
if ($g > 0)
// memoriser qu'on insere // memoriser qu'on insere
$trans[$k][$v][2]=1; $trans[$k][$v][2]=1;
else $g = (0-$g); else $g = (0-$g);
$trans[$k][$v][0] = $g; $trans[$k][$v][0] = $g;
} else spip_log("$f manquante");
} }
return $g; return $g;
} }
...@@ -236,7 +238,6 @@ define('_RACCOURCI_MODELE_ALL', '@' . _RACCOURCI_MODELE .'@isS'); ...@@ -236,7 +238,6 @@ define('_RACCOURCI_MODELE_ALL', '@' . _RACCOURCI_MODELE .'@isS');
// http://doc.spip.org/@importe_raccourci // http://doc.spip.org/@importe_raccourci
function importe_raccourci($k, $v) function importe_raccourci($k, $v)
{ {
if (preg_match_all(_RACCOURCI_LIEN, $v, $m, PREG_SET_ORDER)) { if (preg_match_all(_RACCOURCI_LIEN, $v, $m, PREG_SET_ORDER)) {
foreach ($m as $regs) { foreach ($m as $regs) {
// supprimer 'http://' ou 'mailto:' // supprimer 'http://' ou 'mailto:'
...@@ -274,7 +275,7 @@ function import_identifie_id_document($values, $table, $desc, $request) { ...@@ -274,7 +275,7 @@ function import_identifie_id_document($values, $table, $desc, $request) {
$f = $values['fichier']; $f = $values['fichier'];
$h = $request['url_site'] . $f; $h = $request['url_site'] . $f;
$r = sql_fetch(spip_query("SELECT id_document AS id, fichier AS titre FROM spip_documents WHERE taille=" . _q($t) . " AND (fichier=" . _q($f) . " OR fichier= " . _q($h) . ')')); $r = sql_fetch(spip_query("SELECT id_document AS id, fichier AS titre FROM spip_documents WHERE taille=" . _q($t) . " AND (fichier=" . _q($f) . " OR fichier= " . _q($h) . ')'));
return array($r['id'], $r['titre']); return $r ? array($r['id'], $r['titre']) : false;
} }
// un type de document importe est considere comme identique a un type present // un type de document importe est considere comme identique a un type present
...@@ -285,14 +286,14 @@ function import_identifie_id_type($values, $table, $desc, $request) { ...@@ -285,14 +286,14 @@ function import_identifie_id_type($values, $table, $desc, $request) {
$e = $values['extension']; $e = $values['extension'];
$t = $values['titre']; $t = $values['titre'];
$r = sql_fetch(spip_query("SELECT id_type AS id, titre FROM spip_types_documents WHERE extension=" . _q($e) . " AND titre=" . _q($t))); $r = sql_fetch(spip_query("SELECT id_type AS id, titre FROM spip_types_documents WHERE extension=" . _q($e) . " AND titre=" . _q($t)));
return array($r['id'], $r['titre']); return $r ? array($r['id'], $r['titre']) : false;
} }
// deux groupes de mots ne peuvent avoir le meme titre ==> identification // deux groupes de mots ne peuvent avoir le meme titre ==> identification
// http://doc.spip.org/@import_identifie_id_groupe // http://doc.spip.org/@import_identifie_id_groupe
function import_identifie_id_groupe($values, $table, $desc, $request) { function import_identifie_id_groupe($values, $table, $desc, $request) {
$r = sql_fetch(spip_query("SELECT id_groupe AS id, titre FROM spip_groupes_mots WHERE titre=" . _q($values['titre']))); $r = sql_fetch(spip_query("SELECT id_groupe AS id, titre FROM spip_groupes_mots WHERE titre=" . _q($values['titre'])));
return array($r['id'], $r['titre']); return $r ? array($r['id'], $r['titre']) : false;
} }
// pour un mot le titre est insuffisant, il faut aussi l'identite du groupe. // pour un mot le titre est insuffisant, il faut aussi l'identite du groupe.
...@@ -315,8 +316,16 @@ function import_identifie_parent_id_mot($id_groupe, $titre, $v) ...@@ -315,8 +316,16 @@ function import_identifie_parent_id_mot($id_groupe, $titre, $v)
$r = sql_fetch(spip_query("SELECT id_mot FROM spip_mots WHERE titre=$titre AND id_groupe=$new" )); $r = sql_fetch(spip_query("SELECT id_mot FROM spip_mots WHERE titre=$titre AND id_groupe=$new" ));
if ($r) return (0 - $r['id_mot']); if ($r) return (0 - $r['id_mot']);
} }
$r = sql_insert('spip_mots', '()', '()'); if ($r = sql_insert('spip_mots', '()', '()'))
spip_query("REPLACE spip_translate (id_old, id_new, titre, type, ajout) VALUES ($v,$r,$titre,'id_mot',1)"); sql_replace('spip_translate', array(
'id_old' => $v,
'id_new' => $r,
'titre' => $titre,
'type' => 'id_mot',
'ajout' => 1),
$GLOBALS['tables_principales']['spip_translate']
);
else spip_log("Impossible d'inserer dans spip_mots");
return $r; return $r;
} }
...@@ -337,8 +346,16 @@ function import_identifie_parent_id_article($id_parent, $titre, $v) ...@@ -337,8 +346,16 @@ function import_identifie_parent_id_article($id_parent, $titre, $v)
$r = sql_fetch(spip_query("SELECT id_article FROM spip_articles WHERE titre=$titre AND id_rubrique=$id_parent AND statut<>'poubelle'" )); $r = sql_fetch(spip_query("SELECT id_article FROM spip_articles WHERE titre=$titre AND id_rubrique=$id_parent AND statut<>'poubelle'" ));
if ($r) return (0 - $r['id_article']); if ($r) return (0 - $r['id_article']);
$r = sql_insert('spip_articles', '()', '()'); if ($r = sql_insert('spip_articles', '()', '()'))
spip_query("REPLACE spip_translate (id_old, id_new, titre, type, ajout) VALUES ($v,$r,$titre,'id_article',1)"); sql_replace('spip_translate', array(
'id_old' => $v,
'id_new' => $r,
'titre' => $titre,
'type' => 'id_article',
'ajout' => 1),
$GLOBALS['tables_principales']['spip_translate']
);
else spip_log("Impossible d'inserer dans spip_articles");
return $r; return $r;
} }
...@@ -359,8 +376,16 @@ function import_identifie_parent_id_breve($id_parent, $titre, $v) ...@@ -359,8 +376,16 @@ function import_identifie_parent_id_breve($id_parent, $titre, $v)
$r = sql_fetch(spip_query("SELECT id_breve FROM spip_breves WHERE titre=$titre AND id_rubrique=$id_parent AND statut<>'refuse'" )); $r = sql_fetch(spip_query("SELECT id_breve FROM spip_breves WHERE titre=$titre AND id_rubrique=$id_parent AND statut<>'refuse'" ));
if ($r) return (0 - $r['id_breve']); if ($r) return (0 - $r['id_breve']);
$r = sql_insert('spip_breves', '()', '()'); if ($r = sql_insert('spip_breves', '()', '()'))
spip_query("REPLACE spip_translate (id_old, id_new, titre, type, ajout) VALUES ($v,$r,$titre,'id_breve',1)"); sql_replace('spip_translate', array(
'id_old' => $v,
'id_new' => $r,
'titre' => $titre,
'type' => 'id_breve',
'ajout' => 1),
$GLOBALS['tables_principales']['spip_translate']
);
else spip_log("Impossible d'inserer dans spip_breves");
return $r; return $r;
} }
...@@ -413,8 +438,15 @@ function import_identifie_parent_id_rubrique($id_parent, $titre, $v) ...@@ -413,8 +438,15 @@ function import_identifie_parent_id_rubrique($id_parent, $titre, $v)
// http://doc.spip.org/@import_alloue_id_rubrique // http://doc.spip.org/@import_alloue_id_rubrique
function import_alloue_id_rubrique($id_parent, $titre, $v) { function import_alloue_id_rubrique($id_parent, $titre, $v) {
$titre = _q($titre); $titre = _q($titre);
$r = sql_insert('spip_rubriques', '(titre, id_parent)', "($titre,$id_parent)"); if ($r = sql_insert('spip_rubriques', '(titre, id_parent)', "($titre,$id_parent)"))
spip_query("REPLACE spip_translate (id_old, id_new, titre, type, ajout) VALUES ($v,$r,$titre,'id_rubrique',1)"); sql_replace('spip_translate', array(
'id_old' => $v,
'id_new' => $r,
'titre' => $titre,
'type' => 'id_rubrique',
'ajout' => 1),
$GLOBALS['tables_principales']['spip_translate']);
else spip_log("Impossible d'inserer dans spip_articles");
return $r; return $r;
} }
?> ?>
0% ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site