Ajoute une contrainte d'unicité sur PrivateTopicRead

[philippemilink]

Sentry nous rapporte que l'erreur suivante s'est produite :

PrivateTopicRead.MultipleObjectsReturned: get() returned more than one PrivateTopicRead -- it returned 2!
[...]
File "zds/mp/views.py", line 215, in get_context_data
    mark_read(self.object, self.request.user)
File "zds/mp/models.py", line 560, in mark_read
    topic = PrivateTopicRead.objects.filter(privatetopic=privatetopic, user=user).get()

En effet, la base de données contient 4 couples privatetopic et user en double.

À défaut de comprendre comment ils ont pu être créés, je rajoute une contrainte d'unicité sur ce couple.

Instructions de QA

• la CI passe
• jouer avec les MPs et s'assurer que tout fonctionne comme attendu

[coveralls]

coverage: 88.811% (-0.02%) from 88.827%
when pulling 6f288e9 on philippemilink:unique_privatetopicread
into fbdd980 on zestedesavoir:dev.

[Arnaud-D]

Comment ça va se passer avec les potentiels doublons dans la base de prod ? Il faut les gérer avec la migration ?

[philippemilink]

Comment ça va se passer avec les potentiels doublons dans la base de prod ? Il faut les gérer avec la migration ?

Bonne question ! :) Pour chaque couple en double, je vais en supprimer un manuellement.

[Arnaud-D]

QA OK ✔️

Tu as déjà supprimé les doublons où tu le rajoutes sur ta liste de courses pour le déploiement ?

Je ne fusionne pas tout de suite pour ne pas oublier de noter ça quelque part.

[philippemilink]

Tu as déjà supprimé les doublons où tu le rajoutes sur ta liste de courses pour le déploiement ?

Je ne fusionne pas tout de suite pour ne pas oublier de noter ça quelque part.

Je viens de supprimer les doublons (enfin, un de chaque doublon, pour que ce ne soit plus des doublons). Pour en garder une trace, voilà les commandes effectuées :

SELECT 
    t.*
FROM mp_privatetopicread t 
INNER JOIN (
    SELECT 
        user_id,
        privatetopic_id,
        COUNT(*) AS nb
    FROM mp_privatetopicread
    GROUP BY user_id, privatetopic_id
    HAVING nb > 1
) g ON t.user_id=g.user_id AND t.privatetopic_id=g.privatetopic_id;

DELETE FROM mp_privatetopicread WHERE id IN(<ids des lignes à supprimer>);

J'ai sauvegardé les lignes supprimées ; si jamais il y a un problème, on pourra les restaurer.

Je rebase et je merge.