The lemon-workflow's discuss from lyuxiaobo

lyuxiaobo / lemon-workflow Goto Github PK

View Code? Open in Web Editor NEW

A Business Process Management (BPM) platform based on flowable 6, vue 2, bpmn-js.

License: Apache License 2.0

Java 15.89% Dockerfile 0.07% JavaScript 20.86% Vue 52.55% SCSS 2.78% HTML 5.01% CSS 2.84%

lemon-workflow's Issues

A UUID is not a valid ID for a BPMN element

lemon-workflow/lemon-workflow-frontend/package/process-designer/ProcessDesigner.vue

Line 362 in cb057e5

let xmlString = xml || DefaultEmptyXML(newId, newName, this.prefix);

When creating a new BPMN model the code above will use the model id (newId, a UUID) as the key parameter to the blank xml function. It in turns uses it as an id for the bpmn2:process element and its reference from BPMNPlane.

This results in a modeller that doesn't allow to drop elements to its canvas.

The patch below makes the modeller work again.

diff --git a/lemon-workflow-frontend/package/process-designer/plugins/defaultEmpty.js b/lemon-workflow-frontend/package/process-designer/plugins/defaultEmpty.js
index 21c6343..bf4c9b9 100644
--- a/lemon-workflow-frontend/package/process-designer/plugins/defaultEmpty.js
+++ b/lemon-workflow-frontend/package/process-designer/plugins/defaultEmpty.js
@@ -15,10 +15,10 @@ export default (key, name, type) => {
   xsi:schemaLocation="http://www.omg.org/spec/BPMN/20100524/MODEL BPMN20.xsd"
   id="diagram_${key}"
   targetNamespace="${TYPE_TARGET[type]}">
-  <bpmn2:process id="${key}" name="${name}" isExecutable="true">
+  <bpmn2:process id="process_${key}" name="${name}" isExecutable="true">
   </bpmn2:process>
   <bpmndi:BPMNDiagram id="BPMNDiagram_1">
-    <bpmndi:BPMNPlane id="BPMNPlane_1" bpmnElement="${key}">
+    <bpmndi:BPMNPlane id="BPMNPlane_1" bpmnElement="process_${key}">
     </bpmndi:BPMNPlane>
   </bpmndi:BPMNDiagram>
 </bpmn2:definitions>`;

But I don't know if UUID are used elsewhere.

Here is the relevant ticket in bpmn-io ( bpmn-io/moddle-xml#41 )

Recommend Projects