From e2c4c94993e61234c42b08d359681ce12df39fa0 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Thu, 7 Jul 2022 17:37:46 +0900
Subject: [PATCH] Simplify `BeatmapUpdater` transaction handling using nested
 transaction support

---
 osu.Game/Beatmaps/BeatmapUpdater.cs | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/osu.Game/Beatmaps/BeatmapUpdater.cs b/osu.Game/Beatmaps/BeatmapUpdater.cs
index d0ec44a034..978c6de35c 100644
--- a/osu.Game/Beatmaps/BeatmapUpdater.cs
+++ b/osu.Game/Beatmaps/BeatmapUpdater.cs
@@ -52,15 +52,7 @@ namespace osu.Game.Beatmaps
         /// <summary>
         /// Run all processing on a beatmap immediately.
         /// </summary>
-        public void Process(BeatmapSetInfo beatmapSet)
-        {
-            if (beatmapSet.Realm.IsInTransaction)
-                Process(beatmapSet, beatmapSet.Realm);
-            else
-                beatmapSet.Realm.Write(r => Process(beatmapSet, r));
-        }
-
-        public void Process(BeatmapSetInfo beatmapSet, Realm realm)
+        public void Process(BeatmapSetInfo beatmapSet) => beatmapSet.Realm.Write(r =>
         {
             // Before we use below, we want to invalidate.
             workingBeatmapCache.Invalidate(beatmapSet);
@@ -85,7 +77,7 @@ namespace osu.Game.Beatmaps
 
             // And invalidate again afterwards as re-fetching the most up-to-date database metadata will be required.
             workingBeatmapCache.Invalidate(beatmapSet);
-        }
+        });
 
         private double calculateLength(IBeatmap b)
         {