From 988f9b98a153d95b5f6de42dc3782d08912c125f Mon Sep 17 00:00:00 2001
From: Salman Ahmed <frenzibyte@gmail.com>
Date: Fri, 1 Jan 2021 16:16:00 +0300
Subject: [PATCH] Split button mods updating to private method

---
 osu.Game/Overlays/Mods/ModSection.cs | 30 ++++++++++++++--------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/osu.Game/Overlays/Mods/ModSection.cs b/osu.Game/Overlays/Mods/ModSection.cs
index e29ed4f5c4..573d1e5355 100644
--- a/osu.Game/Overlays/Mods/ModSection.cs
+++ b/osu.Game/Overlays/Mods/ModSection.cs
@@ -133,24 +133,24 @@ namespace osu.Game.Overlays.Mods
         public void UpdateSelectedMods(IReadOnlyList<Mod> newSelectedMods)
         {
             foreach (var button in buttons)
+                updateButtonMods(button, newSelectedMods);
+        }
+
+        private void updateButtonMods(ModButton button, IReadOnlyList<Mod> newSelectedMods)
+        {
+            foreach (var mod in newSelectedMods)
             {
-                int index = -1;
-
-                foreach (var mod in newSelectedMods)
-                {
-                    index = Array.FindIndex(button.Mods, m1 => mod.GetType() == m1.GetType());
-                    if (index < 0)
-                        continue;
-
-                    var buttonMod = button.Mods[index];
-                    buttonMod.CopyFrom(mod);
-                    button.SelectAt(index);
-                    break;
-                }
-
+                var index = Array.FindIndex(button.Mods, m1 => mod.GetType() == m1.GetType());
                 if (index < 0)
-                    button.Deselect();
+                    continue;
+
+                var buttonMod = button.Mods[index];
+                buttonMod.CopyFrom(mod);
+                button.SelectAt(index);
+                return;
             }
+
+            button.Deselect();
         }
 
         protected ModSection()