Fixed parsing of uncut gems

Author: leMicin

Game Localization.xlsm

@@ -1,7 +1,6 @@
 using Microsoft.Extensions.Logging;
 using Sidekick.Apis.Poe.Trade.Clients;
 using Sidekick.Apis.Poe.Trade.Items.Models;
-using Sidekick.Common;
 using Sidekick.Common.Cache;
 using Sidekick.Common.Enums;
 using Sidekick.Common.Extensions;
@@ -25,7 +24,9 @@ ISettingsService settingsService
 
     public Dictionary<string, ApiItem> NameDictionary { get; } = new();
 
-    public List<string> UncutGemIds { get; } = [];
+    public string UncutSkillGemId { get; private set; } = string.Empty;
+    public string UncutSupportGemId { get; private set; } = string.Empty;
+    public string UncutSpiritGemId { get; private set; } = string.Empty;
 
     /// <inheritdoc/>
     public int Priority => 100;
@@ -76,13 +77,19 @@ private void FillCategoryItems(GameType game, List<ApiCategory> categories, stri
 
     private void InitializeUncutGemIds()
     {
-        UncutGemIds.Clear();
-
         foreach (var item in IdDictionary)
         {
-            if (item.Value.Type == "Uncut Skill Gem" || item.Value.Type == "Uncut Spirit Gem" || item.Value.Type == "Uncut Support Gem")
+            switch (item.Value.Type)
             {
-                UncutGemIds.Add(item.Key);
+                case "Uncut Skill Gem":
+                    UncutSkillGemId = item.Key;
+                    break;
+                case "Uncut Spirit Gem":
+                    UncutSpiritGemId = item.Key;
+                    break;
+                case "Uncut Support Gem":
+                    UncutSupportGemId = item.Key;
+                    break;
             }
         }
     }

src/Sidekick.Apis.Poe.Trade/Items/ApiInvariantItemProvider.cs

@@ -2,7 +2,6 @@
 using Microsoft.Extensions.Logging;
 using Sidekick.Apis.Poe.Trade.Clients;
 using Sidekick.Apis.Poe.Trade.Items.Models;
-using Sidekick.Common;
 using Sidekick.Common.Cache;
 using Sidekick.Common.Enums;
 using Sidekick.Common.Extensions;
@@ -26,6 +25,8 @@ ISettingsService settingsService
 
     public Dictionary<string, List<ApiItem>> NameAndTypeDictionary { get; } = new();
 
+    public Dictionary<string, ApiItem> IdDictionary { get; } = new();
+
     public List<(Regex Regex, ApiItem Item)> NameAndTypeRegex { get; } = new();
 
     /// <inheritdoc/>
@@ -35,6 +36,7 @@ ISettingsService settingsService
     public async Task Initialize()
     {
         NameAndTypeDictionary.Clear();
+        IdDictionary.Clear();
         NameAndTypeRegex.Clear();
 
         var leagueId = await settingsService.GetString(SettingKeys.LeagueId);
@@ -73,6 +75,8 @@ private void FillCategoryItems(GameType game, List<ApiCategory> categories, stri
             entry.Game = game;
             entry.Category = category;
 
+            IdDictionary.Add(entry.Id, entry);
+
             if (!string.IsNullOrEmpty(entry.Name))
             {
                 FillDictionary(entry.Name, entry);

src/Sidekick.Apis.Poe.Trade/Items/ApiItemProvider.cs

@@ -9,5 +9,7 @@ public interface IApiInvariantItemProvider : IInitializableService
 
     Dictionary<string, ApiItem> NameDictionary { get; }
 
-    List<string> UncutGemIds { get; }
+    string UncutSkillGemId { get; }
+    string UncutSupportGemId { get; }
+    string UncutSpiritGemId { get; }
 }

src/Sidekick.Apis.Poe.Trade/Items/IApiInvariantItemProvider.cs

@@ -10,5 +10,7 @@ public interface IApiItemProvider : IInitializableService
 
     Dictionary<string, List<ApiItem>> NameAndTypeDictionary { get; }
 
+    Dictionary<string, ApiItem> IdDictionary { get; }
+
     List<(Regex Regex, ApiItem Item)> NameAndTypeRegex { get; }
 }