fix(UpdateChecker): Use OS move function instead of copying

Skytils · Feb 11, 2025 · e9ef6ff · e9ef6ff
1 parent d4bc147
commit e9ef6ff
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/kotlin/gg/skytils/skytilsmod/core/UpdateChecker.kt b/src/main/kotlin/gg/skytils/skytilsmod/core/UpdateChecker.kt
@@ -40,6 +40,8 @@ import net.minecraftforge.fml.common.eventhandler.EventPriority
 import net.minecraftforge.fml.common.eventhandler.SubscribeEvent
 import net.minecraftforge.fml.common.versioning.DefaultArtifactVersion
 import java.io.File
+import java.nio.file.Files
+import java.nio.file.StandardCopyOption
 
 object UpdateChecker {
     val updateGetter = UpdateGetter()
@@ -73,9 +75,7 @@ object UpdateChecker {
                     oldJar.parent,
                     "${if (oldJar.name.startsWith("!")) "!" else ""}${nameNoExtension}${if (oldJar.endsWith(".temp.jar") && newExtension == oldJar.extension) ".temp.jar" else ".$newExtension"}"
                 )
-                newLocation.createNewFile()
-                newJar.copyTo(newLocation, true)
-                newJar.delete()
+                Files.move(newJar.toPath(), newLocation.toPath(), StandardCopyOption.REPLACE_EXISTING, StandardCopyOption.ATOMIC_MOVE)
                 if (oldJar.delete()) {
                     println("successfully deleted the files. skipping install tasks")
                     return@Thread