Skip to content
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

Start events of nested event-subprocesses cannot be triggered #118

Open
rajgoel opened this issue Apr 28, 2022 · 3 comments
Open

Start events of nested event-subprocesses cannot be triggered #118

rajgoel opened this issue Apr 28, 2022 · 3 comments
Labels
backlog Queued in backlog bpmn support Flags missing or to be extended BPMN support bug Something isn't working pr welcome We rely on a community contribution to improve this. spring cleaning Could be cleaned up one day

Comments

@rajgoel
Copy link

rajgoel commented Apr 28, 2022

Describe the Bug

Token simulation allows to click the Play button to trigger events (message, timer, ...). In the main process, the start event of an event-subprocess can also be triggered. However, if the event-subprocess is nested within a subprocess, there is no Play button.

Steps to Reproduce

  1. Create this main process

Screenshot from 2022-04-28 13-54-43

and the respective subprocess:

Screenshot from 2022-04-28 13-54-52

  1. Token simulation shows the Play button only for the start event of the event-subprocess in the main process, but not for the start event of the event-subprocess in the subprocess.

Expected Behavior

The start event of the event-subprocess in the subprocess should also show.

Environment

  • Host (Browser/Node version), if applicable: FF99
  • OS: Ubuntu 18.04
  • Library version:
 "dependencies": {
    "bpmn-js": "^9.0.3",
    "@bpmn-io/properties-panel": "^0.13.2",
    "bpmn-js-properties-panel": "^1.0.0",
    "bpmn-js-token-simulation": "^0.27.0",
    "bpmn-moddle": "^7.1.2",
    "diagram-js": "^7.x",
    "tiny-svg": "^2.2.2"
  },
@rajgoel rajgoel added the bug Something isn't working label Apr 28, 2022
@nikku
Copy link
Member

nikku commented Apr 28, 2022

Related to #114.

We need to catch up with the simulator on the recent nested sub-process additions.

@nikku nikku added the backlog Queued in backlog label Apr 28, 2022
@rajgoel
Copy link
Author

rajgoel commented Oct 31, 2022

@nikku Do you have a recommendation where to search when trying to fix this issue? I would like to help, but so-far couldn't really figure out how to track down the cause of this bug.

@nikku nikku added spring cleaning Could be cleaned up one day bpmn support Flags missing or to be extended BPMN support labels Feb 16, 2023
@nikku
Copy link
Member

nikku commented Feb 16, 2023

@rajgoel Sorry for leaving you hanging for so long. Just now realized your response 🙈.

If you are still eager to fix this, please consider create a test case first. That will help you to debug the tool and see where things go awry.

@nikku nikku added the pr welcome We rely on a community contribution to improve this. label Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog bpmn support Flags missing or to be extended BPMN support bug Something isn't working pr welcome We rely on a community contribution to improve this. spring cleaning Could be cleaned up one day
Development

No branches or pull requests

2 participants