search

POE-Addon-Launcher / PAL2

Path of Exile Addon Launcher and Manager
116 stars 20 forks source link

Temp_downloads handling #37

Open POE-Addon-Launcher opened 5 years ago

POE-Addon-Launcher commented 5 years ago

Add a check that ensures the temp_download folder is an empty folder when setting it.

Dissolator commented 5 years ago

It's better to use temp_downloads path as parent for your app's temporary folder, which you can create, fill with files and delete without any risk to harm other users files. Also you can name it with timestamp to minimize risk of coincidence with already existing folders (like "pal"). Something like %temp_downloads%\pal2_05_06_2019\ should be enough

POE-Addon-Launcher commented 5 years ago

Yeah, that's also what I was thinking off, I was also thinking of when a user specifies a directory and it isn't empty I just create another folder inside of to use for PAL