Feat: make MaxItemsPerTrade being less than 8192 be a config validati…

…on error
SmallTailTeam · Jul 5, 2024 · 852756e · 852756e
1 parent 05295ed
commit 852756e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/BotLooter/Looting/LootClient.cs b/BotLooter/Looting/LootClient.cs
@@ -213,7 +213,7 @@ public async Task<LootResult> TryLoot(TradeOfferUrl tradeOfferUrl, Configuration
         }
 
         assets = assets
-            .Take(Math.Min(configuration.MaxItemsPerTrade, 8192))
+            .Take(configuration.MaxItemsPerTrade)
             .ToList();
 
         return (assets, "");

diff --git a/BotLooter/Resources/Configuration.cs b/BotLooter/Resources/Configuration.cs
@@ -95,6 +95,11 @@ В параметре конфига 'Inventories' не указаны инве
             ...
             """);
         }
+
+        if (config.MaxItemsPerTrade < 8192)
+        {
+            return (null, $"Параметр конфига 'MaxItemsPerTrade' не должен быть меньше 8192, текущее значение: {config.MaxItemsPerTrade}");
+        }
 
         return (config, "");
     }
-Original file line number
+Diff line change
@@ Expand Up @@
                 ...
                 """);
             }
+            if (config.MaxItemsPerTrade < 8192)
+            {
+                return (null, $"Параметр конфига 'MaxItemsPerTrade' не должен быть меньше 8192, текущее значение: {config.MaxItemsPerTrade}");
+            }
             return (config, "");
         }
@@ Expand Down @@