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

Allow setting bpmnParseFactory in configuration

XMLWordPrintable

      Background
      Currently, the field bpmnParseFactory in ProcessEngineConfigurationImpl mark is protected, and there are no setters/getters. To specify my implementation of BpmnParseFactory, I need to create custom config ProcessEngineConfigurationImpl, but this is inconvenient.

      Given:
      A custom BpmnParseFactory implementation.

      When:
      An instance of the custom BpmnParseFactory needs to be set in the ProcessEngineConfigurationImpl

      Then:
      A custom config of ProcessEngineConfigurationImpl needs to be created.

      Expected:
      A Setter exists for the protected ProcessEngineConfigurationImpl.bpmnParseFactory property that allows replacing the factory through a plugin.

        This is the controller panel for Smart Panels app

            [CAM-12179] Allow setting bpmnParseFactory in configuration

            Pavel Pletnev created issue -
            Nikola Koevski made changes -
            Description Original: Currently field bpmnParseFactory in ProcessEngineConfigurationImpl mark is protecred and there are no setters/getters. And to specify my implementation of bpmnParseFactory I need to create custom config ProcessEngineConfigurationImpl, but this is inconvenient.
            Setter allows replacing factroty in plugin.
            You can add setter for bpmnParseFactory?             		New: *Background*
            Currently, the field {{bpmnParseFactory}} in {{ProcessEngineConfigurationImpl}} mark is {{protected}}, and there are no setters/getters. To specify my implementation of {{BpmnParseFactory}}, I need to create custom config ProcessEngineConfigurationImpl, but this is inconvenient.

            *Given:*
            A custom {{BpmnParseFactory}} implementation.

            *When:*
            An instance of the custom {{BpmnParseFactory}} needs to be set in the {{ProcessEngineConfigurationImpl}}

            *Then:*
            A custom config of {{ProcessEngineConfigurationImpl}} needs to be created.

            *Expected:*
            A Setter exists for the protected {{ProcessEngineConfigurationImpl.bpmnParseFactory}} property that allows replacing the factory through a plugin.
            Nikola Koevski made changes -
            Mentioned Roles
            Nikola Koevski made changes -
            Assignee New: Nikola Koevski [ nikola.koevski ]
            Nikola Koevski made changes -
            Mentioned Groups

            Nikola Koevski added a comment - - edited

            Hey bespaltovyj,

            That's a cool suggestion. I reformatted the ticket description to better describe the problem and will forward it for a decision.

            Since this might not be immediately resolved, you can speed up the process by contributing a PR that adds the necessary getter and setter in the ProcessEngineConfigurationImpl in the Camunda BPM Platgorm GitHub repo here: https://github.com/camunda/camunda-bpm-platform

            Best,
            Nikola

            Nikola Koevski added a comment - - edited Hey bespaltovyj , That's a cool suggestion. I reformatted the ticket description to better describe the problem and will forward it for a decision. Since this might not be immediately resolved, you can speed up the process by contributing a PR that adds the necessary getter and setter in the ProcessEngineConfigurationImpl in the Camunda BPM Platgorm GitHub repo here: https://github.com/camunda/camunda-bpm-platform Best, Nikola
            Nikola Koevski made changes -
            Mentioned Roles
            Nikola Koevski made changes -
            Mentioned Groups
            Nikola Koevski made changes -
            Mentioned Roles
            Nikola Koevski made changes -
            Mentioned Groups

              miklas.boskamp Miklas Boskamp
              bespaltovyj Pavel Pletnev
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: