mirror of https://github.com/ppy/osu
Only delete on successful import.
This commit is contained in:
parent
7e72911d3a
commit
eff66a1da6
|
@ -126,13 +126,7 @@ public void Import(IEnumerable<string> paths)
|
|||
BeatmapSetInfo set = importBeatmapSet(p);
|
||||
|
||||
sets.Push(set);
|
||||
}
|
||||
catch (Exception e)
|
||||
{
|
||||
Logger.Error(e, $@"Could not import beatmap set");
|
||||
}
|
||||
finally
|
||||
{
|
||||
|
||||
// We may or may not want to delete the file depending on where it is stored.
|
||||
// e.g. reconstructing/repairing database with beatmaps from default storage.
|
||||
// TODO: Add a check to prevent files from storage to be deleted.
|
||||
|
@ -145,6 +139,10 @@ public void Import(IEnumerable<string> paths)
|
|||
Logger.Error(e, $@"Could not delete file at {p}");
|
||||
}
|
||||
}
|
||||
catch (Exception e)
|
||||
{
|
||||
Logger.Error(e, $@"Could not import beatmap set");
|
||||
}
|
||||
|
||||
// Batch commit with multiple sets to database
|
||||
Import(sets);
|
||||
|
|
Loading…
Reference in New Issue