williamjcm/MassBuilderSaveTool
williamjcm/MassBuilderSaveTool
1
0
Fork 0
Code Issues 16 Pull requests Releases 18 Wiki Activity

UESaveFile: don't create a backup if the file is already temporary.

Browse source 
Also improve error handling.
This commit is contained in:
Guillaume Jacquemin 2022-04-04 09:21:31 +02:00
parent 704f6e2f49
commit 70ddb0ce39
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/UESaveFile/UESaveFile.cpp
View file

@ -67,6 +67,13 @@ auto UESaveFile::props() -> Containers::ArrayView<UnrealPropertyBase::ptr> {
} }
auto UESaveFile::saveToFile() -> bool { auto UESaveFile::saveToFile() -> bool {
bool temp_file = _filepath.hasSuffix(".tmp");
if(!temp_file && !Utility::Path::copy(_filepath, _filepath + ".bak"_s)) {
_lastError = "Couldn't create a backup for " + _filepath;
return false;
}
BinaryWriter writer{_filepath + ".tmp"_s}; BinaryWriter writer{_filepath + ".tmp"_s};
if(!writer.open()) { if(!writer.open()) {
@ -125,12 +132,8 @@ auto UESaveFile::saveToFile() -> bool {
writer.closeFile(); writer.closeFile();
if(!Utility::Path::copy(_filepath, _filepath + ".bak"_s)) { if(!temp_file && !Utility::Path::copy(_filepath + ".tmp"_s, _filepath)) {
return false; _lastError = "Couldn't save the file properly.";
}
if(!Utility::Path::copy(_filepath + ".tmp"_s, _filepath)) {
Utility::Path::copy(_filepath + ".bak"_s, _filepath);
return false; return false;
} }
else { else {