bpmn-io / bpmn-js-differ

A diffing utility for BPMN 2.0 documents.
MIT License
45 stars 17 forks source link

Diff failing with bpmn:DataObjects #4

Closed nikku closed 6 years ago

nikku commented 6 years ago

The differ returns bpmn:DataObject instances when it actually should return drawable elements (the referenced bpmn:DataObjectReference instead.

Context

Diffing demo stack trace:

Uncaught TypeError: Cannot read property 'id' of undefined
    at Canvas._updateMarker (app.js:8785)
    at Canvas.addMarker (app.js:8832)
    at highlight (app.js:301)
    at ModdleElement.<anonymous> (app.js:153)
    at Function.each (app.js:29160)
    at showDiff (app.js:152)
    at diagramLoaded (app.js:117)
    at done (app.js:129)
    at app.js:724
    at importBpmnDiagram (app.js:4196)

Test Diagrams