From 716c7fa07a6c6607b85ae6730fbf192160ec73a1 Mon Sep 17 00:00:00 2001
From: mcendu <nathandu@outlook.com>
Date: Wed, 1 Apr 2020 11:04:29 +0800
Subject: [PATCH] Add check to detect whether mania is skinned

---
 .../Skinning/ManiaLegacySkinTransformer.cs           | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/osu.Game.Rulesets.Mania/Skinning/ManiaLegacySkinTransformer.cs b/osu.Game.Rulesets.Mania/Skinning/ManiaLegacySkinTransformer.cs
index 69e6a0d238..88eb6e0d2f 100644
--- a/osu.Game.Rulesets.Mania/Skinning/ManiaLegacySkinTransformer.cs
+++ b/osu.Game.Rulesets.Mania/Skinning/ManiaLegacySkinTransformer.cs
@@ -18,6 +18,12 @@ public class ManiaLegacySkinTransformer : ISkin
 
         private Lazy<bool> isLegacySkin;
 
+        /// <summary>
+        /// Whether texture for the keys exists.
+        /// Used to determine if the mania ruleset is skinned.
+        /// </summary>
+        private Lazy<bool> hasKeyTexture;
+
         public ManiaLegacySkinTransformer(ISkinSource source)
         {
             this.source = source;
@@ -29,6 +35,10 @@ public ManiaLegacySkinTransformer(ISkinSource source)
         private void sourceChanged()
         {
             isLegacySkin = new Lazy<bool>(() => source.GetConfig<LegacySkinConfiguration.LegacySetting, decimal>(LegacySkinConfiguration.LegacySetting.Version) != null);
+            hasKeyTexture = new Lazy<bool>(() => source.GetTexture(
+                source.GetConfig<LegacyManiaSkinConfigurationLookup, string>(
+                    new LegacyManiaSkinConfigurationLookup(4, LegacyManiaSkinConfigurationLookups.KeyImage, 0))?.Value
+                    ?? $"mania-key1") != null);
         }
 
         public Drawable GetDrawableComponent(ISkinComponent component)
@@ -39,7 +49,7 @@ public Drawable GetDrawableComponent(ISkinComponent component)
                     return getResult(resultComponent);
 
                 case ManiaSkinComponent maniaComponent:
-                    if (!isLegacySkin.Value)
+                    if (!isLegacySkin.Value || !hasKeyTexture.Value)
                         return null;
 
                     switch (maniaComponent.Component)