Development #74089
paramètre de type "date" pas possible optionnel ?
0%
Description
D'un commentaire https://gitea.entrouvert.org/entrouvert/passerelle/pulls/59#issuecomment-3108
C'était pour utiliser le type date sur ce paramètre pour ne pas à avoir à gérer les cas d'erreur, mais c'était une mauvaise idée, parce [que] ce type ne fonctionne pas avec un paramètre optionnel.
Ça n'est "optionnel" mais "vide" mais il me semble qu'il y aurait à gérer ça; i.e. pour partir de tests/test_generic_endpoint.py :
def test_endpoint_typed_params(app, db, monkeypatch): @endpoint( methods=['get'], parameters={ [...] 'date': { 'type': 'date', }, }, ) def httpcall(obj, request, boolean=False, integer=1, floating=1.1, date=None): [...] - json_res = app.get('/stub-invoices/fake/httpcall?date=', status=400).json - assert json_res['err'] == 1 + json_res = app.get('/stub-invoices/fake/httpcall?date=', status=200).json + assert json_res['date'] is None
modifier ce cas, considérer que la présence du =None dans la signature vaut acceptation d'un paramètre vide.
Révisions associées
Historique
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche
Nicolas Roche (nroche) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/78
- Titre : utils: accept empty value on date type with None default value (#74089)
- Modifications : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/78/files
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Agate Berriot (aberriot) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Nicolas Roche (nroche) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/78
- Titre : utils: accept empty value on date type with None default value (#74089)
- Modifications : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/78/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
utils: accept empty value on date type with None default value (#74089)