Development #33783
appel avec cancel_booking_id vide
0%
Description
De #33489,
Aussi, en fait, au lieu de juste faire « if 'cancel_booking_id' in payload » on devrait plutôt faire un « if payload.get('cancel_booking_id'): » dans l'idée qu'un appel webservice avec un cancel_booking_id vide, c'est acceptable, ça veut juste dire "y'a rien à annuler". Ca simplifiera l'appel via les workflows -- bon, j'imagine que le « cancel_booking_id = serializers.IntegerField() » va nous faire chier.
Au premier usage il s'avère que ce serait de fait bien utile d'accepter un cancel_booking_id vide.
Fichiers
Révisions associées
Historique
Mis à jour par Pierre Cros il y a presque 5 ans
Et je suis désolé de pas avoir vu/compris la question explicite de Manu sur le sujet (tant que je teste pas, je comprends pas).
Mis à jour par Thomas Noël il y a presque 5 ans
- Fichier 0001-api-allow-empty-cancel_booking_id-33783.patch 0001-api-allow-empty-cancel_booking_id-33783.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(j'ai fait ça car j'avais déjà commencé à regarder...)
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
J'ai l'impression que le try/except suffit et que les deux if au dessus ne servent à rien, non ?
Mis à jour par Thomas Noël il y a presque 5 ans
Emmanuel Cazenave a écrit :
J'ai l'impression que le try/except suffit et que les deux if au dessus ne servent à rien, non ?
En fait c'est un peu idiot, je cherche à gérer sans planter le cas d'un cancel_booking_id avec juste des espaces... sans doute qu'on peut dire que ça planterait dans ce cas (mais c'est au niveau de mon vécu, genre tu tapes dans l'appel wcs un {{ cancel_truc }} et paf y'a une espace avant le {, et boum)
On peut sans doute éliminer ce cas un peu bête et se dire que les gens feront attention et que ça sera facile à corriger...?
Mis à jour par Frédéric Péters il y a presque 5 ans
On peut sans doute éliminer ce cas un peu bête et se dire que les gens feront attention et que ça sera facile à corriger...?
(oui, et de manière générale dans nos applications à mon sens on devrait automatiquement dégager les espaces devant/derrière quand on reçoit des données d'un formulaire)
Mis à jour par Thomas Noël il y a presque 5 ans
- Fichier 0001-api-allow-empty-cancel_booking_id-33783.patch 0001-api-allow-empty-cancel_booking_id-33783.patch ajouté
Yep yep ; ainsi simplifié, donc.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit af3b8ffc44e478382e3756f86af7c03192983122
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Tue Jun 11 09:56:03 2019 +0200
api: allow empty cancel_booking_id (#33783)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Pierre Cros il y a presque 5 ans
Et ça marche nickel quand on met un |default:"" sur sa variable qui contient un None
api: allow empty cancel_booking_id (#33783)