Project

General

Profile

Development #83418

Ne pas permettre de renseigner un identifiant d'appel webservice sur un saut automatique

Added by Emmanuel Cazenave 7 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
13 November 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Ça n'a grosso modo aucun sens de faire ça.

Mais si le champ est renseigné et qu'il y a aussi une expiration sur ce saut il ne sera jamais exécuté, voir le code JumpWorkflowStatusItem.check_condition.


Related issues

Related to w.c.s. - Development #15661: gestion des rôles dans l'action de saut automatiqueFermé28 March 2017

Actions

Associated revisions

Revision 53659837 (diff)
Added by Frédéric Péters about 1 month ago

workflows: be explicit about jump modes (#83418)

Revision 617e170a (diff)
Added by Frédéric Péters about 1 month ago

workflows: remove unnecessary handling of exception in timeout_parse (#83418)

Revision 01716e72 (diff)
Added by Frédéric Péters about 1 month ago

tests: add trigger jump + redirect test (#83418)

History

#2

Updated by Emmanuel Cazenave 7 months ago

  • Description updated (diff)
#3

Updated by Emmanuel Cazenave 7 months ago

Emmanuel Cazenave a écrit :

Mais si le champ est renseigné et qu'il y a aussi une expiration sur ce saut il ne sera jamais exécuté, voir le code JumpWorkflowStatusItem.check_condition.

Pour être plus précis, le saut peut-être exécuté après expiration s'il y a eu d'abord un appel au trigger. Mais pas ce cas d'usage n'existe pas à ma connaissance, quand on veut un déclenchement externe on met un saut manuel.

#4

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

Aujourd'hui ça permettrait à un système externe de taper à répétition sur le trigger, sans se soucier de l'expiration, et que le saut effectif ait lieu uniquement quand le temps nécessaire est passé, mais ok pour considérer ça rare voire inexistant, et se dire que ça peut au pire être reproduit en deux temps.

Mais pour ici la mise en place, je pense qu'alors il faut revoir la présentation de la configuration,

Exécution
  (*)  tout de suite    ( )  après expiration    ( )  sur appel webservice

et dessous le champ expiration qui apparaitrait quand le choix correspondant est fait, et les champs identifiant d'appel / rôles autorisés quand le choix d'appel webservice est fait.

#5

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

En passant sur de vieux tickets, la question était de donner un sens à la combinaison automatique/rôle : #15661, ma proposition d'interface ne permettrait pas ça.

#6

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

  • Related to Development #15661: gestion des rôles dans l'action de saut automatique added
#7

Updated by Robot Gitea 2 months ago

  • Status changed from Nouveau to En cours
  • Assignee set to Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#10

Updated by Robot Gitea about 2 months ago

  • Status changed from En cours to Solution proposée
#11

Updated by Robot Gitea about 2 months ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#13

Updated by Robot Gitea about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#14

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF