Development #33948
Hobo - Development #33875: rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elleaà abouti une première fois
rendre la commande import_site atomique
0%
Description
Si l'import échoue après avoir chargé les cartes,
alors le rejeu via l'option '--if-empty' n'a pas lieu.
cf: combo/data/utils.py::import_site()
if if_empty and (Page.objects.count() or MapLayer.objects.count()): return
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-data-make-import_site-an-atomic-query-33948.patch 0001-data-make-import_site-an-atomic-query-33948.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Proposition naïve.
cf: https://www.tutorialspoint.com/postgresql/postgresql_transactions.htm
Practically, you will club many PostgreSQL queries into a group and you will execute all of them together as a part of a transaction.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à En cours
Oui mais j'incorpererais aussi la partie présente dans if clean
, ainsi, un import qui échouerait sur l'ajout de contenu ne laisserait pas un site vide.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-import_site-add-import_site-management-command-34503.patch ajouté
- Statut changé de En cours à Solution proposée
oui : inclusion de toutes les modifications induites dans une unique transaction.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Nicolas Roche a écrit :
oui : inclusion de toutes les modifications induites dans une unique transaction.
Mauvais ticket.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-data-make-import_site-an-atomic-query-33948.patch 0001-data-make-import_site-an-atomic-query-33948.patch ajouté
désolé, voilà le bon patch.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier
0001-import_site-add-import_site-management-command-34503.patchsupprimé
Mis à jour par Nicolas Roche il y a presque 5 ans
- Statut changé de Rejeté à Solution proposée
Pardon, j'ai rejeté le ticket par erreur (du fait que j'ai rejeté le ticket chapeau).
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8df192e4552201fd99ea9fa479469f3bc729c22a Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Jun 13 14:16:12 2019 +0200 data: make import_site an atomic query (#33948)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
data: make import_site an atomic query (#33948)