Issue462

Title bkp savegame files not deleted
Priority important Status resolved
Assigned To fluzz Keywords
Linked issues saving: savegame size wrongly calculated?
View: 461
Watchers fluzz

Submitted on 2011-09-09 13h07 by matthiaskrgr, last changed by fluzz.

Messages
Author: matthiaskrgr Date: 2011-09-09   13h07
When I delete a savegame, the bkp files still remain which may cause a lot of
unwanted files over time.

 r5034
Author: matthiaskrgr Date: 2011-11-02   21h08
Any ideas what to do about this? If we do something here, we should do it before
0.15 in my opinion.

Raising prio.
Author: fluzz Date: 2011-11-03   08h25
Assigning to myself
Author: ahuillet Date: 2011-11-03   08h36
It's very simple to fix - I have the fix on my eeePC but it's powered off.
Just change DeleteGame to also delete .bkp files.
Author: matthiaskrgr Date: 2011-11-03   13h50
At the other hand, these are "backup" files. Probably to be still accessed even
if we accidentally delete a savegame.
However, we do already have a warning (Do you really want to...) when a backup
savegame is going to be deleted, so I assume the argument that backups are not
to be deleted is invalid.(?)
Author: fluzz Date: 2011-11-03   22h39
On 11/03/2011 09:36 AM, Arthur Huillet - Roundup wrote:
>
> Arthur Huillet<ahuillet@freedroid.org>  added the comment:
>
> It's very simple to fix - I have the fix on my eeePC but it's powered off.
> Just change DeleteGame to also delete .bkp files.
>

That's exactly what I added.
I'll commit it tomorrow.
Author: fluzz Date: 2011-11-04   06h37
Fixed in svn r5118
History
Date User Action Args
2011-11-04 06:37:10fluzzsetstatus: open -> resolved
messages: + msg1883
2011-11-03 22:39:04fluzzsetmessages: + msg1882
2011-11-03 13:50:28matthiaskrgrsetmessages: + msg1868
2011-11-03 08:36:02ahuilletsetmessages: + msg1863
2011-11-03 08:25:09fluzzsetassignedto: fluzz
messages: + msg1862
nosy: + fluzz
2011-11-02 21:08:24matthiaskrgrsetpriority: bug -> important
messages: + msg1855
2011-09-09 13:07:48matthiaskrgrcreate