abaldwin88 / roamer

The Plain Text File Manager
MIT License
606 stars 15 forks source link

roamer should never delete a file on rename error #18

Open vimeitor opened 7 years ago

vimeitor commented 7 years ago

If renaming one or more files fails, the original file should stay in the original folder; you should never have to go under .roamer-data/ to find it.

I was bitten by this issue accidentally when using a slash on a file's name (which you can't use for obvious reasons). I think it's easy to overlook an error when you are renaming multiple files, and you might not even notice, which was the case for me for quite some time until I was looking for the file and couldn't find it.

Does that sound reasonable?

abaldwin88 commented 7 years ago

Yep very reasonable. Thanks for reporting!

EDIT: #16 should help with this