diff --git a/backend/src/main/java/org/raddatz/familienarchiv/controller/TranscriptionBlockController.java b/backend/src/main/java/org/raddatz/familienarchiv/controller/TranscriptionBlockController.java index ed622826..227713d0 100644 --- a/backend/src/main/java/org/raddatz/familienarchiv/controller/TranscriptionBlockController.java +++ b/backend/src/main/java/org/raddatz/familienarchiv/controller/TranscriptionBlockController.java @@ -77,7 +77,8 @@ public class TranscriptionBlockController { public List reorderBlocks( @PathVariable UUID documentId, @RequestBody ReorderTranscriptionBlocksDTO dto) { - return transcriptionService.reorderBlocks(documentId, dto); + transcriptionService.reorderBlocks(documentId, dto); + return transcriptionService.listBlocks(documentId); } @GetMapping("/{blockId}/history") diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/TranscriptionService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/TranscriptionService.java index caf8604c..2aff91bb 100644 --- a/backend/src/main/java/org/raddatz/familienarchiv/service/TranscriptionService.java +++ b/backend/src/main/java/org/raddatz/familienarchiv/service/TranscriptionService.java @@ -107,14 +107,13 @@ public class TranscriptionService { } @Transactional - public List reorderBlocks(UUID documentId, ReorderTranscriptionBlocksDTO dto) { + public void reorderBlocks(UUID documentId, ReorderTranscriptionBlocksDTO dto) { List blockIds = dto.getBlockIds(); for (int i = 0; i < blockIds.size(); i++) { TranscriptionBlock block = getBlock(documentId, blockIds.get(i)); block.setSortOrder(i); blockRepository.save(block); } - return blockRepository.findByDocumentIdOrderBySortOrderAsc(documentId); } public List getBlockHistory(UUID documentId, UUID blockId) { diff --git a/backend/src/test/java/org/raddatz/familienarchiv/service/TranscriptionServiceTest.java b/backend/src/test/java/org/raddatz/familienarchiv/service/TranscriptionServiceTest.java index efc6de6a..ebe02d10 100644 --- a/backend/src/test/java/org/raddatz/familienarchiv/service/TranscriptionServiceTest.java +++ b/backend/src/test/java/org/raddatz/familienarchiv/service/TranscriptionServiceTest.java @@ -184,7 +184,6 @@ class TranscriptionServiceTest { when(blockRepository.findByIdAndDocumentId(id2, docId)).thenReturn(Optional.of(block2)); when(blockRepository.findByIdAndDocumentId(id1, docId)).thenReturn(Optional.of(block1)); when(blockRepository.save(any())).thenAnswer(inv -> inv.getArgument(0)); - when(blockRepository.findByDocumentIdOrderBySortOrderAsc(docId)).thenReturn(List.of(block2, block1)); ReorderTranscriptionBlocksDTO dto = new ReorderTranscriptionBlocksDTO(List.of(id2, id1));