-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce Open-BPMN-Meta Model #823
Labels
Milestone
Comments
rsoika
added a commit
that referenced
this issue
Aug 5, 2024
rsoika
added a commit
that referenced
this issue
Aug 9, 2024
rsoika
added a commit
that referenced
this issue
Aug 10, 2024
rsoika
added a commit
that referenced
this issue
Aug 11, 2024
rsoika
added a commit
that referenced
this issue
Aug 15, 2024
rsoika
added a commit
that referenced
this issue
Aug 16, 2024
rsoika
added a commit
that referenced
this issue
Aug 17, 2024
rsoika
added a commit
that referenced
this issue
Aug 17, 2024
rsoika
added a commit
that referenced
this issue
Aug 21, 2024
Verify the case process workitem with $modelversion = "" and $workflowGroup="Some Groupname" |
rsoika
added a commit
that referenced
this issue
Aug 27, 2024
rsoika
added a commit
that referenced
this issue
Aug 28, 2024
rsoika
added a commit
that referenced
this issue
Aug 30, 2024
rsoika
added a commit
that referenced
this issue
Aug 31, 2024
rsoika
added a commit
that referenced
this issue
Aug 31, 2024
rsoika
added a commit
that referenced
this issue
Sep 1, 2024
rsoika
added a commit
that referenced
this issue
Sep 2, 2024
rsoika
added a commit
that referenced
this issue
Sep 2, 2024
rsoika
added a commit
that referenced
this issue
Sep 4, 2024
rsoika
added a commit
that referenced
this issue
Sep 6, 2024
rsoika
added a commit
that referenced
this issue
Sep 7, 2024
rsoika
added a commit
that referenced
this issue
Sep 8, 2024
rsoika
added a commit
that referenced
this issue
Sep 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current model implementation is using a sax parser and a ModelHandler. This implementation is not stable. In complex model situations duplicate event ids can occur. This is not detected by the
org.imixs.workflow.bpmn.BPMNModelHandler
.In the following example, both the escalate event and the case2 event have the ID 100
As a result the following junit test will fail (task 2200 should have 5 events! but results in 3!!!)
See the corresponding attached BPMN model in
complex-model0.txt
Solution
To resolve this issue we should get rid of the
org.imixs.workflow.bpmn.BPMNModelHandler
and replace it with the Open-BPMN Meta model library.The new implementation should follow a complete new approach:
org.imixs.workflow.Model
directly withorg.openbpmn.bpmn.BPMNModel
findVersionsByRegEx
from the engine directly into theModelManager
core classfindNextTask
from theWorkflowKernel
- use a OpenBPMN method insteadBPMNModelHandler
,BPMNParser
andBPMNModel
form the packageorg.imixs.workflow.bpmn
As a result, the new model implementation based on the Open-BPMN meta model should detect the issue with the two identical IDs in the example model.
New Methods to be implemented
Consequences
The WorkflowKernel need to be refactored
findNextTask
loadEvent
updateEventList
it should askhasNextEvent
It looks like this methods should become part of the ModelManger
The text was updated successfully, but these errors were encountered: