add cdmid to pdf filename to prevent overwriting index.pdf files

UNC-Libraries · Nov 20, 2024 · 0115921 · 0115921
1 parent 25833e3
commit 0115921
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/java/edu/unc/lib/boxc/migration/cdm/services/CdmExportFilesService.java b/src/main/java/edu/unc/lib/boxc/migration/cdm/services/CdmExportFilesService.java
@@ -84,7 +84,8 @@ public String exportUnmapped() throws IOException {
                         String filePath;
                         // Pdf and image cpd objects are located in different places
                         if (CdmIndexService.ENTRY_TYPE_DOCUMENT_PDF.equals(entryTypeField)) {
-                            filename = "index.pdf";
+                            // add cdmid to filename to prevent overwriting
+                            filename = origMapping.getCdmId() + "_index.pdf";
                             filePath = pdfDir.resolve(origMapping.getCdmId() + "/index.pdf").toString();
                         } else {
                             filename = fileInfo.get(0);

diff --git a/src/test/java/edu/unc/lib/boxc/migration/cdm/ExportUnmappedSourceFilesIT.java b/src/test/java/edu/unc/lib/boxc/migration/cdm/ExportUnmappedSourceFilesIT.java
@@ -128,7 +128,7 @@ public void withUnmappedPdfFilesTest() throws Exception {
         var sourceFileService = testHelper.getSourceFileService();
         var mappingInfo = sourceFileService.loadMappings();
         var mapping1 = mappingInfo.getMappingByCdmId("17940");
-        assertEquals(exportedSourceFilesPath.resolve("index.pdf"), mapping1.getFirstSourcePath());
+        assertEquals(exportedSourceFilesPath.resolve("17940_index.pdf"), mapping1.getFirstSourcePath());
     }
 
     @Test