RED-7384: also migrate annotation Ids with unprocessed manual changes #355
@ -101,8 +101,14 @@ public class RedactionLogToEntityLogMigrationService {
|
|||||||
throw new AssertionError(message);
|
throw new AssertionError(message);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Set<String> entitiesWithUnprocessedChanges = manualRedactions.buildAll()
|
||||||
|
.stream()
|
||||||
|
.filter(manualRedaction -> manualRedaction.getProcessedDate() == null)
|
||||||
|
.map(BaseAnnotation::getAnnotationId)
|
||||||
|
.collect(Collectors.toSet());
|
||||||
|
|
||||||
MigratedIds idsToMigrateInDb = entitiesToMigrate.stream()
|
MigratedIds idsToMigrateInDb = entitiesToMigrate.stream()
|
||||||
.filter(migrationEntity -> migrationEntity.hasManualChangesOrComments(entitiesWithComments))
|
.filter(migrationEntity -> migrationEntity.hasManualChangesOrComments(entitiesWithComments, entitiesWithUnprocessedChanges))
|
||||||
.filter(m -> !m.getOldId().equals(m.getNewId()))
|
.filter(m -> !m.getOldId().equals(m.getNewId()))
|
||||||
.collect(new MigratedIdsCollector());
|
.collect(new MigratedIdsCollector());
|
||||||
|
|
||||||
|
|||||||
@ -347,11 +347,11 @@ public final class MigrationEntity {
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
public boolean hasManualChangesOrComments(Set<String> entitiesWithComments) {
|
public boolean hasManualChangesOrComments(Set<String> entitiesWithComments, Set<String> entitiesWithUnprocessedChanges) {
|
||||||
|
|
||||||
return !(redactionLogEntry.getManualChanges() == null || redactionLogEntry.getManualChanges().isEmpty()) || //
|
return !(redactionLogEntry.getManualChanges() == null || redactionLogEntry.getManualChanges().isEmpty()) || //
|
||||||
!(redactionLogEntry.getComments() == null || redactionLogEntry.getComments().isEmpty()) //
|
!(redactionLogEntry.getComments() == null || redactionLogEntry.getComments().isEmpty()) //
|
||||||
|| hasManualChanges() || entitiesWithComments.contains(oldId);
|
|| hasManualChanges() || entitiesWithComments.contains(oldId) || entitiesWithUnprocessedChanges.contains(oldId);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user