Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-2276

When an outgoing SequenceFlow is removed from a FlowNode, the diagram element of SequenceFlow loses bpmnElement attribute

XMLWordPrintable

      AT:

      • only the outgoing reference and not the actual sequence flow should be removed from the model

      Failing test case:
      https://github.com/camunda/camunda-bpmn-model/commit/67c469a6871f0a06ac0828de5c85040d2804d7e7

      ==================
      floNode.getOutgoing().remove(flow) somehow clears association between edge and flow

      Try these three lines to validate the bug:

      FlowNode source = <someNode>;
      BpmnEdge flowDi = flow.getDiagramElement();
      source.getOutgoing().remove(flow);
      // flowDi has no bpmnElement attribute now
      // a workaround I am currently using in our project:
      flowDi.setBpmnElement(flow);

        This is the controller panel for Smart Panels app

            [CAM-2276] When an outgoing SequenceFlow is removed from a FlowNode, the diagram element of SequenceFlow loses bpmnElement attribute

            Sebastian Menski added a comment -

            Hi Ganesh,

            again thank you for your feedback. I could reproduce the bug and commited a failing test case to a branch (https://github.com/camunda/camunda-bpmn-model/commit/67c469a6871f0a06ac0828de5c85040d2804d7e7). It seems that this is a general problem of the reference removal.

            Cheers,
            Sebastian

            Sebastian Menski added a comment - Hi Ganesh, again thank you for your feedback. I could reproduce the bug and commited a failing test case to a branch ( https://github.com/camunda/camunda-bpmn-model/commit/67c469a6871f0a06ac0828de5c85040d2804d7e7 ). It seems that this is a general problem of the reference removal. Cheers, Sebastian

            Sebastian Menski added a comment -

            commited on branch CAM-2276 (bpmn/xml-model repo) and already backported

            Sebastian Menski added a comment - commited on branch CAM-2276 (bpmn/xml-model repo) and already backported

            Sebastian Menski added a comment -

            Please notice that if you remove a sequence flow from the incoming/outgoing collection of a flow node it will not alter the sourceRef/targetRef attribute of the sequence flow element and vice versa.

            Sebastian Menski added a comment - Please notice that if you remove a sequence flow from the incoming/outgoing collection of a flow node it will not alter the sourceRef/targetRef attribute of the sequence flow element and vice versa.

            Sebastian Menski added a comment -

            merged on master

            Sebastian Menski added a comment - merged on master

              meyer Daniel Meyer
              ganesh Ganesh Ingle
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: