tobias/radicale
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #472 from Unrud/patch-30

Browse Source 
Use os.replace instead of os.rename
This commit is contained in:
Guillaume Ayoub 2016-08-11 04:23:56 +02:00 committed by GitHub
commit 35d12ee97e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
radicale/storage.py
View File

@ -404,7 +404,7 @@ class Collection(BaseCollection):
else:
os.fsync(tmp.fileno())
tmp.close()
os.rename(tmp.name, path)
os.replace(tmp.name, path)
except:
tmp.close()
os.remove(tmp.name)
@ -574,7 +574,7 @@ class Collection(BaseCollection):
@classmethod
def move(cls, item, to_collection, to_href):
os.rename(
os.replace(
path_to_filesystem(item.collection._filesystem_path, item.href),
path_to_filesystem(to_collection._filesystem_path, to_href))
cls._sync_directory(to_collection._filesystem_path)