Produits Entr'ouvert » Authentic 2

Faire et défaire c'est toujours travailler :/

0001, remarques de forme :

Plutôt utiliser transaction.atomic comme décorateur pour s'économiser un niveau d'indentation ?

+    def delete(self, **kwargs):
+        with transaction.atomic():

Indentation bizarre juste après :

+            deleted_user = DeletedUser(
+                old_user_id=self.id)

0002, si la migration est exécutée au mauvais moment ça va désupprimer des utilisateurs, ce serait plus safe d'avoir un RunPython qui crée les DeletedUser nécessaires avant.

Sur la forme, un sed trop rapide ici

     qs = User.objects.all()

-    qs = qs.filter(deleted__isnull=True)
+    qs = qs.filter()

Tu es sûr que le user.delete() dans la migration c'est bien la méthode delete surchargée qui crée un DeletedUser ? Il me semble que dans une migration on a pas accès aux méthodes custom mais je me trompe peut-être, si tu as testé et que c'est bon je te crois.