Development #24869
Ne plus utiliser SubfieldBase
Début:
28 juin 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Déprécié en django 1.8 : https://docs.djangoproject.com/en/2.0/releases/1.8/#subfieldbase
Assez obscur, je ne comprends pas bien, mais on s'en sert dans authentic/saml/fields.py.
Fichiers
Demandes liées
Révisions associées
don't use SubfieldBase (#24869)
Historique
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Lié à Development #24868: Ne plus utiliser de fonctionnalités dépréciées en django 1.8 ajouté
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-test-custom-fields-using-SubfieldBase-24869.patch 0001-test-custom-fields-using-SubfieldBase-24869.patch ajouté
- Fichier 0002-don-t-use-SubfieldBase-24869.patch 0002-don-t-use-SubfieldBase-24869.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pas les idées complètement claires sur le sujet, j'essaie de compenser par de la prudence.
Ajout de tests qui passent. Suppression des __metaclass__ = models.SubfieldBase : les tests plantent. Ajout du fix : les tests passent.
Mis à jour par Anonyme il y a presque 6 ans
- Lié à Development #22638: tests pour authentic2.saml.fields (compatibilité django 1.11) ajouté
Mis à jour par Anonyme il y a presque 6 ans
- Fichier 0018-add-support-for-django-1.11-removing-models.Subfield.patch 0018-add-support-for-django-1.11-removing-models.Subfield.patch ajouté
Ça m'a l'air OK, j'en étais arrivé à la même proposition à l'époque, mais sans les tests (c'est pourquoi j'ai lié le ticket #22638 qui pourra être rejeté avec celui-là).
C'est un peu "touchy", je préfèrerais un autre relecture que la mienne sur le sujet.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Solution proposée à Solution validée
Ack.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 30708c0c7ce6b0749415d7fa76fb4cc8e8bf1d9f
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Thu Jul 12 16:59:26 2018 +0200
test custom fields using SubfieldBase (#24869)
commit af036b783d3b17f2424170d88a897d3542322ad9
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Thu Jul 12 17:00:29 2018 +0200
don't use SubfieldBase (#24869)
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution déployée à Fermé
test custom fields using SubfieldBase (#24869)