Produits Entr'ouvert » Authentic 2

Paul Marillonnet a écrit :

Commencé à taper un truc dans la branche, erreur cryptique à l’application des migrations, je regarde.

Le moteur de migration n’arrive pas à appliquer la migration générée.
À l’exécution de "django.db.backends.base.schema.BaseDatabaseSchemaEditor.alter_field", ce bout de code

        elif old_type is None and new_type is None and (
                old_field.remote_field.through and new_field.remote_field.through and
                old_field.remote_field.through._meta.auto_created and
                new_field.remote_field.through._meta.auto_created):
            return self._alter_many_to_many(model, old_field, new_field, strict)

explose parce que new_field.remote_field.through vaut la chaîne 'django_rbac.RoleMember' (i.e. le nom de la classe qui contient les doublets de m2m) et n’a donc pas d’attribut _meta.

Je creuse l’affaire.

Paul Marillonnet a écrit :

Je creuse l’affaire.

Bon, en fait il fallait la même mécanique de duplication que pour la classe Role, présente dans django_rbac et a2_rbac.
Cette duplication est visible dans la branche.

En revanche, j’ai l’impression que django ne tolère plus la manipulation du billet de blog en description de ce ticket. Ce qui était décrit comme une feature dans le billet semble maintenant être un bug. Je me mange un :

$ authentic2-multitenant-manage makemigrations
SystemCheckError: System check identified some issues:

ERRORS:
a2_rbac.Role.members: (fields.E340) The field's intermediary table 'a2_rbac_role_members' clashes with the table name of 'a2_rbac.RoleMember'.

Ok, je vais regarder un peu aussi.

Bon, je me disais que le souci venait du fait qu’on gère des modèles similaires dans django_rbac et authentic2.a2_rbac. J’ai essayé, par curiosité, de le faire seulement dans a2 mais encore une fois on dirait que django n’est pas prévu pour faire ce genre de chose.
Le postulat du billet de blog selon lequel le moteur de migration django n’irait pas altérer la relation m2m me semble faux avec notre version de django : en ajoutant les champs through et through_fields sur le modèle de départ, la relation est forcément altérée même avec l’étape intermédiaire décrite dans le billet de blog.

• création du nouveau modèle intermédiaire m2m
• copie des données de m2m dans le nouveau modèle
• suppression de l’ancienne table
• désignation du nouveau modèle intermédiaire comme celui stockant les Role.members

Je vais y porter un regard frais, en me basant sur ce qui a été fait dans #57500, passé dans la dernière release.