Bug #29144
permettre l'utilisation du champ natif JSONField au lieu de celui issu de django-jsonfield
0%
Description
Parce que, dans #20695, je reproduis très exactement https://code.djangoproject.com/ticket/27675 (dont la conclusion est "Short story: Django's JSONField cannot be mixed with django-jsonfield's JSONField to use."), et que je suis à peu près certain que c'est l'utilisation de django-jsonfield dans authentic2_auth_oidc qui fout le bazar.
Modifier le modèle ne suffit pas, il faut aussi, pour que le bogue disparaisse, enlever tout import erroné de django-field dans les migrations.
Fichiers
Demandes liées
Historique
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Fichier 0001-let-a2-use-django.contrib.postgres-native-jsonfield-.patch 0001-let-a2-use-django.contrib.postgres-native-jsonfield-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà, pas certain que ce soit la façon la plus élégante.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Frileux là-dessus, tu peux pas utiliser le django-jsonfield comme partout ailleurs, et laisser la migration à autre chose, plus tard ?
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Lié à Development #20695: Avoir sur les objets un journal des modifications et sur les utilisateurs, en plus, un journal des actions ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
partout ailleurs
En fait, utilisé uniquement là dans OIDCProvider, et ce champ m'a l'air souvent vide, donc bon, pourquoi pas. (mais except ImportError:, pas except: nu).
Mis à jour par Paul Marillonnet il y a plus de 5 ans
Frédéric Péters a écrit :
Et faut que ça tourne avec sqlite, toujours.
Oui en effet, merci, j'ai posé le patch un peu trop rapidement. Je revois ça et je reviens ici.
Mis à jour par Paul Marillonnet il y a plus de 5 ans
Frédéric Péters a écrit :
Frileux là-dessus, tu peux pas utiliser le django-jsonfield comme partout ailleurs, et laisser la migration à autre chose, plus tard ?
Oups, je n'avais pas vu cette remarque. C'est peut-être la décision la plus sage en effet.
Je continue #20695 en utilisant django-jsonfield uniquement, et on verra si on adapte plus tard en tirant profit de django.contrib.postgres.fields.jsonb, ok ?
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Benjamin Dauvergne a écrit :
Une piste https://github.com/kbussell/django-jsonfield-compat
Et donc j'ai intégré une partie de ce code dans #29193.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Rejeté
- Assigné à mis à Paul Marillonnet
Annulé au profit de #29193.