Development #36098
api/users/, envoi de chaine vide sur des attributs non obligatoires
Début:
13 septembre 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
--- a/tests/test_api.py +++ b/tests/test_api.py @@ -1373,3 +1373,11 @@ def test_api_user_required_drf_attribute(settings, app, admin, simple_user): params=payload, headers=headers, status=400) assert resp.json['result'] == 0 assert resp.json['errors'] == {'prefered_color': ["This field may not be blank."]} + + color_attribute = Attribute.objects.get(name='prefered_color') + color_attribute.required = False + color_attribute.save() + + resp = app.put_json('/api/users/{}/'.format(simple_user.uuid), + params=payload, headers=headers, status=200) + assert resp.json['result'] == 0
L'attribut couleur est marqué comme n'étant pas obligatoire, et pourtant ce dernier appel échoue,
E AppError: Bad response: 400 BAD REQUEST (not 200) E '{"errors":{"prefered_color":["This field may not be blank."]},"result":0}'
Fichiers
Révisions associées
Copy attribute's DRF field kwargs before use (#36098)
Without it we get parameters from previous calls to
Attribute.get_kind().
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Assigné à changé de Nicolas Roche à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-Copy-attribute-s-DRF-field-kwargs-before-use-36098.patch 0001-Copy-attribute-s-DRF-field-kwargs-before-use-36098.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le bug était déjà là mais le allow_blank=False ajouté par Nicolas l'a révélé.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b7b0b93e7aa9d25f129191ed4292bce03b7c781d Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Sep 13 11:14:41 2019 +0200 Copy attribute's DRF field kwargs before use (#36098) Without it we get parameters from previous calls to Attribute.get_kind().
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Copy attribute's DRF field kwargs before use (#36098)
Without it we get parameters from previous calls to
Attribute.get_kind().