Development #11059
csv datasource : supporter les intérogations en POST
0%
Fichiers
Demandes liées
Historique
Mis à jour par Josué Kouka il y a presque 8 ans
- Lié à Development #11023: connecteur csv (Nancy) ajouté
Mis à jour par Frédéric Péters il y a presque 8 ans
Pourquoi ? (Les paramètres peuvent déjà être mis dans l'URL, dans l'appel webservice de wcs.)
Mis à jour par Thomas Noël il y a presque 8 ans
Frédéric Péters a écrit :
Pourquoi ? (Les paramètres peuvent déjà être mis dans l'URL, dans l'appel webservice de wcs.)
On va avoir en entrée un numéro, et il va falloir selectionner la liste "I"mpaire ou "P"aire : ça ne peut pas se faire dans l'URL en ezt.
J'ai donc proposé que le CSV sache répondre aussi à un POST, où wscall saura construire des arguments (='P' if form_var_num % 2 else 'I')
Maintenant en REST, POST a une autre signification, mais dans le cadre du connecteur CSV, je dirais que m'en fiche.
Mis à jour par Josué Kouka il y a presque 8 ans
Frédéric Péters a écrit :
Pourquoi ? (Les paramètres peuvent déjà être mis dans l'URL, dans l'appel webservice de wcs.)
Certes, mais pour l'instant dans wcs en GET on peut pas passer d'expression python alors qu'en POST on peut le faire. C'est pratique quand la valeur d'un filtre est fonction du champs d'un formulaire.
Un cas concret (Nancy), serait de déterminer le bureau de vote d'un usager habitant au 45 Rue Charlemagne. On peut facilement déterminer que le numéro est impair et envoyé un filtre du genre street_side=I par exemple.
Mis à jour par Frédéric Péters il y a presque 8 ans
ok pour l'explication.
(sur le cas précis, je continue à penser que vouloir faire assurer ce boulot précis au connecteur CSV est une erreur).
Mis à jour par Josué Kouka il y a presque 8 ans
- Fichier 0001-csv-datasource-add-POST-method-support.patch 0001-csv-datasource-add-POST-method-support.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a presque 8 ans
Il faudrait rajouter des tests avec le méthode POST, non?
Mis à jour par Frédéric Péters il y a presque 8 ans
Maintenant en REST, POST a une autre signification, mais dans le cadre du connecteur CSV, je dirais que m'en fiche.
Je reviens là-dessus, je pense que ce n'est pas une bonne idée d'ainsi utiliser le POST dans ce connecteur; qu'on devrait plutôt réserver son usage pour un moment qui lui correspondra mieux, par exemple quand on voudra permettre l'écriture dans le fichier.
On va avoir en entrée un numéro, et il va falloir selectionner la liste "I"mpaire ou "P"aire : ça ne peut pas se faire dans l'URL en ezt.
Et je suggérerais d'améliorer les choses là, #11207.
~~
Pour moi ce ticket doit être rejeté.