Development #37874
Toulouse Axel - supprimer la contrainte d'unicité ('resource', 'name_id', 'dui', 'person_id')
Début:
22 novembre 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Suite de #37659
On a déjà une contrainte d'unicité sur ('resource', 'name_id'), il n'est pas nécessaire d'avoir en plus ('resource', 'name_id', 'dui', 'person_id')
revoir le get_or_create du endpoint link: le get doit se faire sur le tuple ('resource', 'name_id') uniquement, et non sur le tuple ('resource', 'name_id', 'dui', 'person_id'). Vérifier par contre, si on trouve un Link existant pour ('resource', 'name_id'), que 'dui' et 'person_id' correspondent à ce qu'on a trouvé.
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Fichier 0001-toulouse_axel-fix-unique_together-and-get_or_create-.patch 0001-toulouse_axel-fix-unique_together-and-get_or_create-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Ça se fait normalement en ne touchant pas au get_or_create() mais en interceptant django.db.IntegrityError, mais je te laisse décider ce que tu préfères au niveau du style.
Mis à jour par Lauréline Guérin il y a plus de 4 ans
je préfère faire le get_or_create sur le tuple d'unicité :)
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c354c9460753f16dc913ed661e883658dc718fdd Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Nov 22 10:05:48 2019 +0100 toulouse_axel: fix unique_together and get_or_create call (#37874)
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
toulouse_axel: fix unique_together and get_or_create call (#37874)