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

Make the various Query classes more consistent in their APIs

    • Icon: Feature Request Feature Request
    • Resolution: Won't Do
    • Icon: L3 - Default L3 - Default
    • None
    • None
    • engine
    • None

      User Story (Required on creation):

      Just notice that the various xxxQuery objects exposed in the RuntimeService and the RepositoryService (I haven't checked all of them) are all over the place

      VariableInstanceQuery processInstanceIdIn(String... processInstanceIds);

      and

      ProcessDefinitionQuery processDefinitionIdIn(String... ids);

      vs

      ProcessInstanceQuery processInstanceIds(Set<String> processInstanceIds);

       

      So, in order to find all process definitions for a "collection" of process instance IDs, I need a java.util.Set, but to find all deployments from which those definitions came, I need a String array.

      Then, to delete multiple process instances, I need a java.util.List.

       

      All this could be a little more consistent to make the code I use to interact with that API a bit less "hacky".

      Functional Requirements (Required before implementation):

      No new functionality required

      Technical Requirements (Required before implementation):

      Limitations of Scope (Optional):

      Hints (optional):

        This is the controller panel for Smart Panels app

            [CAM-14211] Make the various Query classes more consistent in their APIs

            Jaap Sperling created issue -
            Jaap Sperling made changes -
            Description Original: h3. User Story (Required on creation):

            Just notice that the various xxxQuery objects exposed in the RuntimeService and the RepositoryService (I haven't checked all of them) are all over the place

            VariableInstanceQuery processInstanceIdIn(String... processInstanceIds);

            and

            ProcessDefinitionQuery processDefinitionIdIn(String... ids);

            vs

            ProcessInstanceQuery processInstanceIds(Set<String> processInstanceIds);
            h3. Functional Requirements (Required before implementation):

            No new functionality required
            h3. Technical Requirements (Required before implementation):
            h3. Limitations of Scope (Optional):
            h3. Hints (optional):
            New: h3. User Story (Required on creation):

            Just notice that the various xxxQuery objects exposed in the RuntimeService and the RepositoryService (I haven't checked all of them) are all over the place

            VariableInstanceQuery processInstanceIdIn(String... processInstanceIds);

            and

            ProcessDefinitionQuery processDefinitionIdIn(String... ids);

            vs

            ProcessInstanceQuery processInstanceIds(Set<String> processInstanceIds);

             

            So, in order to find all process definitions for a "collection" of process instance IDs, I need a java.util.Set, but to find all deployments from which those definitions came, I need a String array.

            Then, to delete multiple process instances, I need a java.util.List.

             

            All this could be a little more consistent to make the code I use to interact with that API a bit less "hacky".
            h3. Functional Requirements (Required before implementation):

            No new functionality required
            h3. Technical Requirements (Required before implementation):
            h3. Limitations of Scope (Optional):
            h3. Hints (optional):
            Jaap Sperling made changes -
            Mentioned Roles
            Jaap Sperling made changes -
            Mentioned Groups
            Miklas Boskamp made changes -
            Mentioned Roles
            Miklas Boskamp made changes -
            Mentioned Groups
            Miklas Boskamp made changes -
            Assignee New: Miklas Boskamp [ miklas.boskamp ]
            Miklas Boskamp made changes -
            Resolution New: Won't Do [ 10101 ]
            Status Original: Open [ 1 ] New: Closed [ 6 ]
            Jaap Sperling made changes -
            Mentioned Roles
            Jaap Sperling made changes -
            Mentioned Groups

              miklas.boskamp Miklas Boskamp
              Gwaptiva Jaap Sperling
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: