failj/ntspi-app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(vikon): use rename() instead of copy() for syncFiles - atomic, no extra disk space

Browse Source
This commit is contained in:
F4ilji
2026-07-04 19:09:13 +05:00
parent e502225e9c
commit 93bd3963bf
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/Containers/VikonIntegration/Actions/UpdateCoreAction.php
+5 -5
View File
@@ -119,10 +119,10 @@ class UpdateCoreAction
if (File::exists($targetPath)) {
$oldPath = $targetPath . self::OLD_SUFFIX;
File::delete($oldPath);
@unlink($oldPath);
rename($targetPath, $oldPath);
}
copy($file->getPathname(), $targetPath);
rename($file->getPathname(), $targetPath);
}
foreach (File::directories($source) as $dir) {
@@ -131,11 +131,11 @@ class UpdateCoreAction
if (File::exists($targetPath)) {
$oldPath = $targetPath . self::OLD_SUFFIX;
File::deleteDirectory($oldPath);
File::move($targetPath, $oldPath);
$this->fs->safeRemove($oldPath, $target, true);
rename($targetPath, $oldPath);
}
File::copyDirectory($dir, $targetPath);
rename($dir, $targetPath);
}
}