Development #17929
Permettre des kwargs pour les champs REST attribués aux types d'attributs
100%
Description
- la nécessité d'avoir une taille maximum par défaut pour les champs textes (ici 128 caractères pour coller à une demande du GL)
- la nécessité d'accepter les champs vides (mais pas null)
- la possibilité de restreindre cette taille maximum ou d'interdire les valeurs vides
Fichiers
Demandes liées
Révisions associées
attribute_kinds: enlarge default width for default REST field of attributes (fixes #17929)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Lié à Bug #15621: api: dans l'API des utilisateurs ne pas forcer allow_blank=True sur tous les champs générés pour les attributs étendus ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Lié à Bug #17901: attributs qui ne peuvent être vides lors d'un PATCH ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-attribute_kinds-add-a-rest_framework_field_kwargs-fi.patch 0001-attribute_kinds-add-a-rest_framework_field_kwargs-fi.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 6 ans
rest_field_kwargs.setdefault('max_length', 128)
128, je pense que ça va m'embêter pour les adresses... ou bien c'est une limite qui existe déjà ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Non elle n'existe pas encore, mais dans l'absolu il en faut une même grande, pour les API et pour les widgets en FO, pour l'instant on accepte n'importe quoi et comme c'est stocké en TextField ça passe, 256 ça irait pour les adresses ?
Mis à jour par Thomas Noël il y a plus de 6 ans
Et ne pas mettre de limite, ça serait un soucis ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Oui, c'est mal de ne pas valider ses entrées. 256 caractères ça semble suffisant pour une adresse, la poste limiterait à 4 lignes de 32 caractères maximum.
Mis à jour par Thomas Noël il y a plus de 6 ans
Benjamin Dauvergne a écrit :
Oui, c'est mal de ne pas valider ses entrées.
Je pensais DRF suffisamment intelligent pour remonter une erreur "propre" quand il aura un dépassement vu par le modèle Django, mais TextField, j'ai compris donc.
Ack pour 256
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|0eb65d673e6e5679a4f8b326e11898c5ea9983af.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Poussé pour se poser la question d'un transfert rapide vers testing/prod.
commit 0eb65d673e6e5679a4f8b326e11898c5ea9983af Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Aug 8 22:34:33 2017 +0200 attribute_kinds: add a rest_framework_field_kwargs field (fixes #17929) This field allow customizing the definition of Django REST framework field corresponding to user attirbutes, default parameter for the default field CharField are allow_blank=True, max_length=128.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ça m'embête un peu car je n'avais pas fait la modification vers la longueur 256... je vais pousser un surpatch.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Appliqué par commit authentic2|76d09366d8a8dfe6643fe1723e83967247dea21c.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Yep en plus je m'en suis rendu compte après mais j'avais déjà commencé les installations en prod, et j'ai oublié de le noter ici.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Résolu (à déployer) à Fermé
attribute_kinds: add a rest_framework_field_kwargs field (fixes #17929)
This field allow customizing the definition of Django REST framework field
corresponding to user attirbutes, default parameter for the default field
CharField are allow_blank=True, max_length=128.