Project

General

Profile

Development #44940

wscall et data_sources : ajouter une condition d'éxecution

Added by Thomas Noël (congés → 2 septembre) about 4 years ago. Updated over 3 years ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
08 July 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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.


Related issues

Related to w.c.s. - Development #42686: meilleur message de trace quand un appel webservice est fait avec une url videFermé10 May 2020

Actions
Related to w.c.s. - Development #86992: Condition de sortie de page: ne pas les tester si la page actuelle est déjà en erreurRejeté15 February 2024

Actions

History

#1

Updated by Frédéric Péters about 4 years ago

  • Related to Development #42686: meilleur message de trace quand un appel webservice est fait avec une url vide added
#2

Updated by Thomas Noël (congés → 2 septembre) about 4 years ago

(et sans doute pareil pour les data_source)

#3

Updated by Frédéric Péters about 4 years ago

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".

#4

Updated by Thomas Noël (congés → 2 septembre) about 4 years ago

  • Priority changed from Normal to 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.

#5

Updated by Thomas Noël (congés → 2 septembre) about 4 years ago

  • Subject changed from wscall: ne pas lancer d'appel si l'URL est vide to wscall et data_sources : ajouter une condition d'éxecution
#6

Updated by Frédéric Péters over 3 years ago

  • Priority changed from Bas to 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.

#8

Updated by Frédéric Péters 5 months ago

  • Related to Development #86992: Condition de sortie de page: ne pas les tester si la page actuelle est déjà en erreur added

Also available in: Atom PDF