Projet

Général

Profil

Development #74089

paramètre de type "date" pas possible optionnel ?

Ajouté par Frédéric Péters il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
01 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

Révision 91fa1266 (diff)
Ajouté par Nicolas Roche il y a environ un an

utils: accept empty value on date type with None default value (#74089)

Historique

#1

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 :

#2

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 :

#3

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 :

#4

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#5

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF