Development #44940
wscall et data_sources : ajouter une condition d'éxecution
0%
Description
Quand on configure un webservice, on peut être amené à le faire avec une URL comme
{% if form_var_point }https://sig/?lat={{ form_var_point_lat }}{ endif %}qui signifie en fait qu'on ne veut pas faire d'appel au webservice si form_var_point n'existe pas.
Mais si on fait cela, on se retrouve avec une erreur « invalid schema » car l'appel vers l'URL vide est lancé...
Il faudrait éviter de lancer l'appel quand l'URL calculée est vide.
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Lié à Development #42686: meilleur message de trace quand un appel webservice est fait avec une url vide ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
Je trouve que ça peut trop facilement cacher des erreurs, genre typo dans {{form_var_creneau_api_status}} et un message où il y a {% if webservice.event_status.full }oh non{ endif %}, qui ne sera jamais vu.
Si le ticket est sur une volonté de conditionner l'exécution des appels webservices, je serais pour réfléchir à un attribut condition explicite (comme dans les actions de workflow), plutôt que définir "url vide = on zappe".
Mis à jour par Thomas Noël il y a presque 4 ans
- Priorité changé de Normal à Bas
Frédéric Péters a écrit :
Si le ticket est sur une volonté de conditionner l'exécution des appels webservices, je serais pour réfléchir à un attribut condition explicite (comme dans les actions de workflow), plutôt que définir "url vide = on zappe".
De fait, sur les webservices je pense que c'est inutile, car c'est toujours (?) utilisé dans des gabarits qui peuvent faire des {% if ... }{{ webservice.foo ... }}{ endif %}
Restent donc éventuellement les sources de données mais là encore on a des conditions sur les champs qui les utilisent.
Donc on peut toujours éviter le truc.
Reste que l'idée d'un attribut "conditions d'exécution" explicite me plait quand même, alors je ne rejette pas ce ticket, j'en change le titre et je le passe en priorité basse.
Mis à jour par Thomas Noël il y a presque 4 ans
- Sujet changé de wscall: ne pas lancer d'appel si l'URL est vide à wscall et data_sources : ajouter une condition d'éxecution
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Priorité changé de Bas à Normal
Parce que traces via #49377 et qu'il aurait été simple de mettre une condition dans l'appel webservice, et qu'il n'est pas facile d'aller éditer tous les champs qui peuvent faire appel au webservice.
Mis à jour par Frédéric Péters il y a 2 mois
- Lié à Development #86992: Condition de sortie de page: ne pas les tester si la page actuelle est déjà en erreur ajouté