[BUG] `BpmnElementsRegistry.getElementsByIds` generates error when the HTML BPMN container has no id
See original GitHub issueI’ve started working with bpmn-vsualization on svelte, and then to implement it I didn’t need to register to an element via ID element query selector. But it leads to some unexpected behaviors I’ve recently fixed by putting an id to the container. For example htmlElement
for all element keeps getting null because the query selector build starts with “# > …” and is not valid.
there’s no problem if there’s already stated somewhere around docs and I missed, but maybe to better debug this kind of error there could be an warning “No id detected on root html element”
I’ll later get an repro here, but right now I’m just reporting to get it out of my mind
Thanks for all your efforts on this awesome library
Issue Analytics
- State:
- Created a year ago
- Reactions:1
- Comments:7 (6 by maintainers)
Top Results From Across the Web
Error on use bmpn-js With Primefaces - Forum - bpmn.io
I'm getting this error: VM5197:73 could not import BPMN 2.0 diagram Error: invalid element specified at Overlays.add (<anonymous>:20845:12) ...
Read more >Undo of invalid process ID in the XML does not work #1925
Open the Modeler · Open a new BPMN diagram · Go to the XML view · Change the process ID to an invalid...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Hi @csouchet !
Just made a quick repro (using rxjs for easier copy pastes) https://codesandbox.io/s/bpmn-visualization-svelte-id-bug-mvp1it
Remove attribute
id
from element then it will crashLater I can do another repro without all the bloats if it is not useful right now.
And I would be happy to help as I can on the svelte examples
ℹ️ The fix is now available as part of version
0.27.1