From 2a87b851fae5d3fb783c936782155968006f1900 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Mon, 28 May 2018 18:38:42 +0900
Subject: [PATCH] Add proper transaction rollback logic on exception

---
 osu.Game/Database/DatabaseWriteUsage.cs | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/osu.Game/Database/DatabaseWriteUsage.cs b/osu.Game/Database/DatabaseWriteUsage.cs
index 7858c1a0d1..07bebbf0c3 100644
--- a/osu.Game/Database/DatabaseWriteUsage.cs
+++ b/osu.Game/Database/DatabaseWriteUsage.cs
@@ -28,8 +28,19 @@ namespace osu.Game.Database
             if (isDisposed) return;
             isDisposed = true;
 
-            PerformedWrite |= Context.SaveChanges(transaction) > 0;
-            usageCompleted?.Invoke(this);
+            try
+            {
+                PerformedWrite |= Context.SaveChanges(transaction) > 0;
+            }
+            catch (Exception e)
+            {
+                transaction?.Rollback();
+                throw;
+            }
+            finally
+            {
+                usageCompleted?.Invoke(this);
+            }
         }
 
         public void Dispose()