flowable / flowable/flowable-engine

batchMigrateCaseInstancesOfCaseDefinition has not been implemented using case definition id

Offen
#2,835 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
9.5k
Forks
2.9k
Ø Merge
7 Std. 8 Min.
Gemergte PRs (30 T.)
2

Beschreibung

**Describe the bug**
Please check the class CaseInstanceMigrationManagerImpl seems like the following method has not been implemented:
batchMigrateCaseInstancesOfCaseDefinition(String caseDefinitionId, CaseInstanceMigrationDocument document, CommandContext commandContext)

**Expected behavior**
Should behave similarly to batchMigrateCaseInstancesOfCaseDefinition(String caseDefinitionKey, int caseDefinitionVersion, String caseDefinitionTenantId, CaseInstanceMigrationDocument document, CommandContext commandContext)

If we compare the mentioned methods only difference is, how are we getting the caseDefinitionId, but once we have the same the behavior should be same.

**Code**

@Override
public Batch batchMigrateCaseInstancesOfCaseDefinition(String caseDefinitionKey, int caseDefinitionVersion, String caseDefinitionTenantId, CaseInstanceMigrationDocument document, CommandContext commandContext) {
CaseDefinition caseDefinition = resolveCaseDefinition(caseDefinitionKey, caseDefinitionVersion, caseDefinitionTenantId, commandContext);
return batchMigrateCaseInstancesOfCaseDefinition(caseDefinition.getId(), document, commandContext);
}

@Override
public Batch batchMigrateCaseInstancesOfCaseDefinition(String caseDefinitionId, CaseInstanceMigrationDocument document, CommandContext commandContext) {
throw new UnsupportedOperationException("not implemented"); // TODO
}

**Additional context**
It is difficult to use batchMigrateCaseInstancesOfCaseDefinition(String caseDefinitionKey, int caseDefinitionVersion, String caseDefinitionTenantId, CaseInstanceMigrationDocument document, CommandContext commandContext), when no tenet setup. As in db, there will be null value and following the same if we pass null in the methods, then will get exception while creating the command context, so it's not possible. Please check following classes as caseDefinitionId always set to to null considering the methods have not been implemented CaseInstanceMigrationValidationCmd and CaseInstanceMigrationBatchCmd.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne in CaseInstanceMigrationManagerImpl und vergleiche die Überladung für case-definition-key mit der Überladung für case-definition-id. Untersuche anschließend CaseInstanceMigrationValidationCmd und CaseInstanceMigrationBatchCmd, bei denen das Issue meldet, dass caseDefinitionId immer null ist. Als abgeschlossen gilt die Aufgabe, wenn sich die ID-basierte Methode wie die Methode für Key/Version/Tenant verhält und die zugehörigen Befehle die Case-Definition-ID beibehalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.