1
0
mirror of https://github.com/ppy/osu.git synced 2024-11-06 06:57:39 +08:00

Report any error during import to the write context to allow for rollback

This commit is contained in:
Dean Herbert 2018-05-29 13:48:14 +09:00
parent cc081cad5a
commit 3d3026a80c

View File

@ -181,21 +181,29 @@ namespace osu.Game.Database
{
using (var write = ContextFactory.GetForWrite()) // used to share a context for full import. keep in mind this will block all writes.
{
if (!write.IsTransactionLeader) throw new InvalidOperationException($"Ensure there is no parent transaction so errors can correctly be handled by {this}");
try
{
if (!write.IsTransactionLeader) throw new InvalidOperationException($"Ensure there is no parent transaction so errors can correctly be handled by {this}");
// create a new model (don't yet add to database)
item = CreateModel(archive);
// create a new model (don't yet add to database)
item = CreateModel(archive);
var existing = CheckForExisting(item);
var existing = CheckForExisting(item);
if (existing != null) return existing;
if (existing != null) return existing;
item.Files = createFileInfos(archive, Files);
item.Files = createFileInfos(archive, Files);
Populate(item, archive);
Populate(item, archive);
// import to store
ModelStore.Add(item);
// import to store
ModelStore.Add(item);
}
catch (Exception e)
{
write.Errors.Add(e);
throw;
}
}
}
catch