search

jmix-framework / jmix

Jmix framework
https://www.jmix.io
Apache License 2.0
695 stars 124 forks source link

Support for "retry cycle" property on service task and other elements that support retrying mecanism #2563

Closed KartnDev closed 9 months ago

KartnDev commented 12 months ago

Feature Description

For details check flowable documentation: https://www.flowable.com/open-source/docs/bpmn/ch07b-BPMN-Constructs#fail-retry

gorbunkov commented 12 months ago

The issue must be ported to 1.6

konyashkina commented 9 months ago

After clearing the field, the tag remains <extensionElements/>

Tested on Jmix version: 2.2.999-SNAPSHOT Jmix Studio plugin version: 2.2.SNAPSHOT6050-233

KartnDev commented 9 months ago

After clearing the field, the tag remains <extensionElements/>

Tested on Jmix version: 2.2.999-SNAPSHOT Jmix Studio plugin version: 2.2.SNAPSHOT6050-233

My thoughts:

Seems like is part of internal "features" inside flowable that doesnt break process and its ok for BPMN

I did check:

  1. create schema and add service task with any of implementation (spring bean or expression, etc) => deployable
  2. add retry cycle and then remove it
  3. got the schema, where <extensionElements/> is present: 
    <process id="process" name="Process" isExecutable="true">
<startEvent id="startEvent1" />
<serviceTask id="Activity_11htt4l">
  <extensionElements />
</serviceTask>
</process>
  4. deploy it => still deployable
  5. close modeler
  6. open again and import my new schema
  7. deploy it => still deployable
  8. conclusion that this <extensionElements/> is normal behavior for bpmn.io "remove" command inside modeler
konyashkina commented 9 months ago

Tested on Jmix version: 2.2.999-SNAPSHOT Jmix Studio plugin version: 2.2.SNAPSHOT6060-233 IntelliJ version: IntelliJ IDEA 2023.3.4 RC

knstvk commented 9 months ago

Studio issue: https://youtrack.jmix.io/issue/JST-4663/Support-FailedJobRetryCycle-on-service-task-properties