Drag&Drop Modify with collapsed subprocess with more than one process instance don´t work

[venetrius]

Environment (Required on creation)

Reproduces issue on Tomcat distro on version 7.20.4, 7.20.5 and 7.21.0

Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket)

User can't use drag and drop functionality for collapsed subprocess to start instance before/after an other step.

Steps to reproduce (Required on creation)

• Deploy process.bpmn
• Start a few instances of the process setting a single variable to:

{"var1":
   { "value":0,
     "type":"Integer"
   }
}

• Go to the process-definition page of this process and select modify
• Try to drag and drop the three dots to move instances from collapsed subprocess to the user tasks
• It should create a startBefore and a Cancel action
• Click on Select Instances and select a few instances
• Hit modify instances
• See error: Could not apply modification : For instruction type 'cancel': 'activityId' must be set

Observed Behavior (Required on creation)

After following the above steps, modification/executeAsync endpoint returns

{
    "type": "InvalidRequestException",
    "message": "For instruction type 'cancel': 'activityId' must be set",
    "code": null
}

Expected behavior (Required on creation)

Should successfully send a modification/executeAsync which creates a batch job after the batch job run the tokens are in the user task

Root Cause (Required on prioritization)

webapps/camunda-webapp/plugins/frontend/plugin/cockpit/modification/app/diagramPlugins/definitionModification.js

In onDragAndDrop we are checking if the element.id is present as a key in the activityIdToInstancesMap to see if it is a multi instance and then we set the id. However activityIdToInstancesMap does not contain collapsed subprocess. This id is used to create the cancel action. When creating the request the empty field is filtered out this is how we end up with the For instruction type 'cancel': 'activityId' must be set error.

I think this has never worked.

Solution Ideas

One simple fix is to check if the element type is bpmn:SubProcess then set MI to false and id to element.id

Hints

Links

https://jira.camunda.com/browse/SUPPORT-17986

Breakdown

Pull Requests

Beta Give feedback

1. https://github.com/camunda/camunda-bpm-platform-ee/pull/933

Dev2QA handover

• Does this ticket need a QA test and the testing goals are not clear from the description? Add a Dev2QA handover comment