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

Execute conditional event dispatching only if conditional events exist in process definition

XMLWordPrintable

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

      Right now the check will be evaluated every time a variable is set:

      The process definition should contain a flag or property which indicates if a conditional event exist in the process

        This is the controller panel for Smart Panels app

            [CAM-6966] Execute conditional event dispatching only if conditional events exist in process definition

            Thorben Lindhauer added a comment - - edited

            Review feedback:

            • let's pull the check for the property a little bit up. Either to the point where variable events are delayed or to the start point where events are consumed
            • use typed properties
            • remove Corba import
            • test cases: Either write no additional tests (since this is a refactoring, that is ok in our testing approach) or unit test the internal behavior entirely (i.e. on top of testing the property, test that this property is respected when events are delayed)
            • comment "must not be executed" => "need not be executed"

            Thorben Lindhauer added a comment - - edited Review feedback: let's pull the check for the property a little bit up. Either to the point where variable events are delayed or to the start point where events are consumed use typed properties remove Corba import test cases: Either write no additional tests (since this is a refactoring, that is ok in our testing approach) or unit test the internal behavior entirely (i.e. on top of testing the property, test that this property is respected when events are delayed) comment "must not be executed" => "need not be executed"

            Thorben Lindhauer added a comment - - edited

            Can't strike through smiley

            Thorben Lindhauer added a comment - - edited Can't strike through smiley

            Christopher Kujawa added a comment -

            Christopher Kujawa added a comment -

              thorben.lindhauer Thorben Lindhauer
              christopher.zell Christopher Kujawa
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: