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

Composite History Event Handler

XMLWordPrintable

      AT:

      • provide a default implementation of a composite history event handler such that users can more easily add multiple history event handler implementations to the engine

        This is the controller panel for Smart Panels app

            [CAM-2716] Composite History Event Handler

            Thorben Lindhauer created issue -
            Daniel Meyer made changes -
            Labels Original: EasyPick New: AD EasyPick
            Daniel Meyer made changes -
            Fix Version/s New: 7.3.0 [ 13391 ]
            Fix Version/s Original: 7.2.0 [ 12997 ]
            Robert Gimbel made changes -
            Fix Version/s Original: 7.3.0 [ 13391 ]

            Alexander Tyatenkov added a comment -

            Hello,

            Can I also try to implement this ticket?

            For example, I could implement 2 composite classes CompositeHistoryEventHandler and CompositeDbHistoryEventHandler inside the package org.camunda.bpm.engine.impl.history.handler.
            Where:

            • CompositeHistoryEventHandler is the simple composite class implemented HistoryEventHandler interface
            • CompositeDbHistoryEventHandler extends CompositeHistoryEventHandler adding the new instance of DbHistoryEventHandler into the inner Collection/List

            Br,
            Alexander

            Alexander Tyatenkov added a comment - Hello, Can I also try to implement this ticket? For example, I could implement 2 composite classes CompositeHistoryEventHandler and CompositeDbHistoryEventHandler inside the package org.camunda.bpm.engine.impl.history.handler . Where: CompositeHistoryEventHandler is the simple composite class implemented HistoryEventHandler interface CompositeDbHistoryEventHandler extends CompositeHistoryEventHandler adding the new instance of DbHistoryEventHandler into the inner Collection/List Br, Alexander

            Thorben Lindhauer added a comment -

            Sure, feel free to provide a pull request.

            Cheers,
            Thorben

            Thorben Lindhauer added a comment - Sure, feel free to provide a pull request. Cheers, Thorben

            Alexander Tyatenkov added a comment - - edited

            Thanks,

            I will try to write unit tests, if it is possible and they are needed for this task, and then create the pull request.
            Anyway writing tests should help me better to learn Camunda, you have plenty of examples.

            Br,
            Alexander

            Alexander Tyatenkov added a comment - - edited Thanks, I will try to write unit tests, if it is possible and they are needed for this task, and then create the pull request. Anyway writing tests should help me better to learn Camunda, you have plenty of examples. Br, Alexander

            Thorben Lindhauer added a comment -

            Yes, please write tests. Feel free to ask if you need assistance.

            Cheers,
            Thorben

            Thorben Lindhauer added a comment - Yes, please write tests. Feel free to ask if you need assistance. Cheers, Thorben
            Thorben Lindhauer made changes -
            Fix Version/s New: 7.4.0 [ 13505 ]

            Thorben Lindhauer added a comment -

            Hi Alexander,

            Thanks for the pull request (for reference: https://github.com/camunda/camunda-bpm-platform/pull/156/). We'll review it in the next days.

            Cheers,
            Thorben

            Thorben Lindhauer added a comment - Hi Alexander, Thanks for the pull request (for reference: https://github.com/camunda/camunda-bpm-platform/pull/156/ ). We'll review it in the next days. Cheers, Thorben
            Thorben Lindhauer made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            Remaining Estimate New: 0 minutes [ 0 ]
            Original Estimate New: 0 minutes [ 0 ]

              Unassigned Unassigned
              thorben.lindhauer Thorben Lindhauer
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: