Created attachment 567917 [details] process definition In order to import any process definition into Guvnor BPMN designer, each node has to have corresponding bpmndi:BPMNShape element inside BPMNDiagram definition. BPMN2 specification says that diagram depicts all model elements as well as only part of them. Steps to Reproduce: 1. Start the Guvnor, create new BPMN2 process. 2. Choose "Import from BPMN2" in BPMN designer and load attached process definition (BPMNShape element is missing for dataObject element). 3. Error window appears without any kind of information what is wrong (see attached sceenshot) and no process definition is imported.
Created attachment 567919 [details] screenshot
Yes this is a requirement for Designer - to have valid BPMN2 which can be parsed and presented properly. Thanks.
(In reply to comment #2) > Yes this is a requirement for Designer - to have valid BPMN2 which can be > parsed and presented properly. In our understanding of the spec, the document is valid. See comment 0: "BPMN2 specification says that diagram depicts all model elements as well as only part of them."
Ok for model elements, but here we are talking about the notation of the model elements, which is required for a editor to be able to draw the diagram for you. Otherwise you may want to use a text-based editor.
Designer has visual support for Data Objects, Lanes, and Groups. Data Objects are model elements and thus have a notation section as well. I understand that possibly in Eclipse or via text-based editing you can create a Data Object but it cannot be represented in Designer without its notation (DI) info. What we do with lanes now when you import a process from Eclipse that has lanes, we ignore these lanes and you have to add them back visually in Designer. With data objects we could do the same if you so wish.
(In reply to comment #5) > What we do with lanes now when you import a process from Eclipse that has > lanes, we ignore these lanes and you have to add them back visually in > Designer. With data objects we could do the same if you so wish. I do consider it better than just rejecting a valid BPMN file.
Added code to not marshall Data Objects if no DI info can be found. Code in master and 2.1.x Designer branch. Let me know if I need to provide a war for testing or if you guys can build. Server logs also indicate what was removed, for example for the process attached to this BZ: 15:14:35,825 INFO [org.jbpm.designer.bpmn2.impl.Bpmn2JsonMarshaller] (http-localhost-127.0.0.1-8080-1) Could not find BPMNDI information for org.eclipse.bpmn2.impl.DataObjectImpl@69a585cf (id: employee, anyAttribute: []) (name: null) (isCollection: false) 15:14:35,826 INFO [org.jbpm.designer.bpmn2.impl.Bpmn2JsonMarshaller] (http-localhost-127.0.0.1-8080-1) Could not marshall Data Object org.eclipse.bpmn2.impl.DataObjectImpl@69a585cf (id: employee, anyAttribute: []) (name: null) (isCollection: false) because no DI information could be found.
Please verify this issue on ER5. Thanks!