Added replace existing option to Files.copy in order to fix attachments not being copied properly

Peyton-Hill-CLS · Peyton-Hill-CLS · commit f68b6fd4e27a · 2025-06-19T08:24:59.000-06:00
diff --git a/app/logbook/inmemory/src/main/java/org/phoebus/applications/logbook/InMemoryLogClient.java b/app/logbook/inmemory/src/main/java/org/phoebus/applications/logbook/InMemoryLogClient.java
@@ -128,7 +128,7 @@ public LogEntry set(LogEntry log) {
                     ext = file.getName().substring(i);
                 }
                 File tempFile = File.createTempFile(prefix, ext);
-                Files.copy(file.toPath(), tempFile.toPath());
+                Files.copy(file.toPath(), tempFile.toPath(), StandardCopyOption.REPLACE_EXISTING);
                 tempFile.deleteOnExit();
                 String mimeType = URLConnection.guessContentTypeFromName(tempFile.getName());
                 return AttachmentImpl.of(tempFile, mimeType != null ? mimeType : ext, false);

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ public LogEntry set(LogEntry log) {`
`128`	`128`	`ext = file.getName().substring(i);`
`129`	`129`	`}`
`130`	`130`	`File tempFile = File.createTempFile(prefix, ext);`
`131`		`- Files.copy(file.toPath(), tempFile.toPath());`
	`131`	`+ Files.copy(file.toPath(), tempFile.toPath(), StandardCopyOption.REPLACE_EXISTING);`
`132`	`132`	`tempFile.deleteOnExit();`
`133`	`133`	`String mimeType = URLConnection.guessContentTypeFromName(tempFile.getName());`
`134`	`134`	`return AttachmentImpl.of(tempFile, mimeType != null ? mimeType : ext, false);`