Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-2066

Retierate Command class hierarchy

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • L3 - Default
    • 2.7.0
    • None
    • backend
    • None

    Description

      AT:

      • the current command class hierarchy is reevaluated in terms of it it serves the current purpose well and redisgned where appropriate
      • all the parts of the command are modular and can be reuused
      • there is a command builder that reuuses the modules to build the command
      • every command is a chain of modules that is used to build the elasticsearch query and to extract the result from the Elastisearch query result
      • the command key generation is adjusted so that the configuration parameter do not depend on the view and group by

      Context:
      The current class hierarchy makes it hard to implement result type specific abstractions e.g. as the sorting done with https://github.com/camunda/camunda-optimize/pull/425.
      The command structure became confusing and hard to understand. Also the creation of the command is now using configuration fields which depend on the view/group by, which is really prone to errors.

      mgm-controller-panel

        This is the controller panel for Smart Panels app

        Attachments

          Issue Links

            is duplicated by

            Task - A task that needs to be done. OPT-2383 Refactor how reports are mapped to its respective command

            • L3 - Default - Minor issue or issue with existing workaround, feature requests and questions
            • Done
            1.
            		 Merge frequency and duration result maps and numbers Sub-task Done Unassigned
            2.
            		 Unify instance count fields in report results Sub-task Done Unassigned
            3.
            		 Use new instanceCount field to retrieve the instance count from proces/decsision report results Sub-task Done Unassigned
            4.
            		 Use new simplified report result types Sub-task Done Unassigned
            5.
            		 Merge decision and process report result dto's Sub-task Done Unassigned
            6.
            		 First iteration of command class hierarchy Sub-task Done Unassigned
            7.
            		 Refactor remaining process commands to new command structure Sub-task Done Unassigned
            8.
            		 Refactor decision commands to new command structure Sub-task Done Unassigned
            9.
            		 Cleanup unused remnants of the command class hierarchy refactoring Sub-task Done Unassigned
            10.
            		 Refactor first process duration commands to new command class hierarchy Sub-task Done Unassigned
            11.
            		 Refactor first process user taskcommands to new command class hierarchy Sub-task Done Unassigned
            12.
            		 Extend existing process commands by distributed by part Sub-task Done Unassigned
            13.
            		 Refactor first process distributed by commands to new command class hierarchy Sub-task Done Unassigned
            14.
            		 Refactor GroupByCandidateGroup and GroupByAssignee classes Sub-task Done Unassigned
            15.
            		 Move command sorting functionality to the CompositeCommandResult Sub-task Done Unassigned
            16.
            		 Merge process and decision group by none parts Sub-task Done Unassigned
            17.
            		 Merge code duplication in process and decision group by variable parts Sub-task Done Unassigned
            18.
            		 Refactor raw data commands to new command class hierarchy Sub-task Done Unassigned
            19.
            		 Enable automatic interval selection for combined reports again Sub-task Done Unassigned

            Activity

              People

                Unassigned Unassigned
                sebastian.bathke Sebastian Bathke
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Salesforce