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

Non conditional event sub process is not triggered multiple times in one transaction

XMLWordPrintable

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 7.6.0, 7.6.0-alpha6
    • None
    • engine
    • None

      Setting of multiple variables in one transaction like via runtimeService should trigger the
      non conditional event sub process multiple times.

      See failing test case EventSubProcessStartConditionalEventTest#testNonInterruptingSetMultipleVariables

        This is the controller panel for Smart Panels app

            [CAM-6933] Non conditional event sub process is not triggered multiple times in one transaction

            Christopher Kujawa added a comment -

            Perhaps we have to check in the getActivityInstanceId if the targetScope is compacted and return the parent activity instance id.

              protected boolean isCompacted(PvmExecutionImpl execution) {
    return execution.isScope() && execution.getActivity() != null && !execution.getActivity().isScope();
  }

            Christopher Kujawa added a comment - Perhaps we have to check in the getActivityInstanceId if the targetScope is compacted and return the parent activity instance id. protected boolean isCompacted(PvmExecutionImpl execution) { return execution.isScope() && execution.getActivity() != null && !execution.getActivity().isScope(); }

            Christopher Kujawa added a comment -

            Partly fixed with this change.

            Christopher Kujawa added a comment - Partly fixed with this change.

            Roman Smirnov added a comment -

            As a discussed, I pushed a failing test case on the following branch: https://github.com/camunda/camunda-bpm-platform/tree/CAM-6933-test

            Roman Smirnov added a comment - As a discussed, I pushed a failing test case on the following branch: https://github.com/camunda/camunda-bpm-platform/tree/CAM-6933-test

              roman.smirnov Roman Smirnov
              christopher.zell Christopher Kujawa
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: