Bug #79157
toulouse-maelis: gérer les doublons à la validation du panier
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/112776/
UniqueViolation: duplicate key value violates unique constraint "toulouse_maelis_invoice_resource_id_regie_id_inv_b23a84d7_uniq"
DETAIL: Key (resource_id, regie_id, invoice_id)=(4, 101, 5) already exists.
File "django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql, params)
IntegrityError: duplicate key value violates unique constraint "toulouse_maelis_invoice_resource_id_regie_id_inv_b23a84d7_uniq"
DETAIL: Key (resource_id, regie_id, invoice_id)=(4, 101, 5) already exists.
(14 additional frame(s) were not displayed)
...
File "django/db/backends/utils.py", line 66, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "django/db/backends/utils.py", line 75, in _execute_with_wrappers
return executor(sql, params, many, context)
File "django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql, params)
File "django/db/utils.py", line 90, in __exit__
raise dj_exc_value.with_traceback(traceback) from exc_value
File "django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql, params)
Error occurred while processing request
Demandes liées
Historique
Mis à jour par Nicolas Roche il y a 10 mois
Duplique #79067 mais pas tout à fait non plus, ici on est sur le endpoint de validation du panier.
C'est sur ce endpoint que la facture générée est liée à une demande, donc ça vaut le coup de retourner un message pour préciser que le lien n'a pas pu être fait.
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche
Nicolas Roche (nroche) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/passerelle/pulls/303
- Titre : toulouse-maelis: gérer au mieux les factures renvoyées en double par maélis (#79157)
- Modifications : https://git.entrouvert.org/entrouvert/passerelle/pulls/303/files
Mis à jour par Nicolas Roche il y a 10 mois
- Sujet changé de toulouse-maelis: validate_basket error (IntegrityError: duplicate key value violates unique constraint "toulouse_maelis_invoice_resource_id_regie_id_inv_...) à toulouse-maelis: gérer les doublons à la valifdation du panier
Mis à jour par Nicolas Roche il y a 10 mois
- Sujet changé de toulouse-maelis: gérer les doublons à la valifdation du panier à toulouse-maelis: gérer les doublons à la validation du panier
Mis à jour par Benjamin Dauvergne il y a 8 mois
- Statut changé de Solution proposée à Nouveau
Je recopie mon commentaire de Gitlab ici:
Je suis du même avis que Fred, rien à faire ici, garder le code tel qu'il est (la validation d'un panier doit renvoyer une facture nouvelle, donc un id inconnu du système), mais prévoir une commande de ré-initialisation pendant recette pour supprimer tous les liens famille/facture (ou le faire à la main si c'est suffisamment rare).
Je propose donc de fermer plutôt ce ticket.
Mis à jour par Robot Gitea il y a 8 mois
- Statut changé de Nouveau à En cours
Nicolas Roche (nroche) a fermé une pull request sur Gitea concernant cette demande.
Mis à jour par Nicolas Roche il y a 8 mois
- Statut changé de En cours à Fermé
- Assigné à changé de Nicolas Roche à Benjamin Dauvergne
Mis à jour par Nicolas Roche il y a 8 mois
- Assigné à changé de Benjamin Dauvergne à Nicolas Roche
Mis à jour par Nicolas Roche il y a 7 mois
- Statut changé de Fermé à En cours
Je ré-ouvre parce qu'on a eu 3 fois la même erreur, mais cette fois-ci sur la récupération des factures.
https://sentry.entrouvert.org/entrouvert/publik/issues/114943/events/1028004/
Je ne vois pas d'incohérence sur la facture côté maélis et la date de l'erreur est la même que la date enregistrée sur l'objet du connecteur.
Je me demande si le connecteur n'aurais pas essayé de créer 2 fois la facture en même temps,
et qu'il faudrait protéger les création des objets factures par un verrou.
Mis à jour par Nicolas Roche il y a 6 mois
- Lié à Bug #82706: toulouse-maelis: ajouter un verrou sur la création des objets factures. ajouté
Mis à jour par Nicolas Roche il y a 4 mois
- Statut changé de En cours à Fermé
Je ferme, je ne vois plus l'erreur depuis.