Merge pull request #545 from ipax77/dev

ipax77 · web-flow · commit 1eb70bf77f42 · 2025-07-22T20:13:17.000+02:00
DsBuilds - fix unit placement / add Abathur
diff --git a/README.md b/README.md
@@ -30,7 +30,13 @@ We really like people helping us with the project. Nevertheless, take your time
 
 ## ChangeLog
 
-<details open="open"><summary>v2.0.9</summary>
+<details open="open"><summary>v2.0.10</summary>
+
+>- DsBuilds - fix unit placement / add Abathur
+
+</details>
+
+<details><summary>v2.0.9</summary>
 
 >- dotnet update to 8.0.17
 >- DsBuilds - build units in the Direct Strike Tutorial map based on replay data
diff --git a/src/dsstats.db8services/Challenge/ChallengeDbService.cs b/src/dsstats.db8services/Challenge/ChallengeDbService.cs
@@ -87,7 +87,7 @@ public async Task<bool> SaveSubmission(ChallengeResponse response, int spChallen
         var submission = await context.SpChallengeSubmissions
             .FirstOrDefaultAsync(s => s.SpChallengeId == spChallengeId && s.PlayerId == playerId)
 ;
-        if (submission == null)
+        if (submission == null || submission.Commander == response.Commander)
         {
             submission = new SpChallengeSubmission
             {
diff --git a/src/dsstats.db8services/Import/ImportService.Import.cs b/src/dsstats.db8services/Import/ImportService.Import.cs
@@ -26,6 +26,11 @@ public async Task<ImportResult> Import(List<ReplayDto> replayDtos, List<PlayerId
             await Init();
         }
 
+        if (!IsMaui)
+        {
+            replayDtos = replayDtos.Where(x => x.GameMode != GameMode.Tutorial).ToList();
+        }
+
         Dictionary<PlayerId, string> playerInfos = new();
 
         for (int i = 0; i < replayDtos.Count; i++)
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.de.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.de.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Konservativer Wiederholungsimport (etwas langsamer, aber robuster)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Zusammenstellung</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>FEN-Zeichenkette ist leer. Bitte wenden Sie eine gültige FEN an, bevor Sie die Zusammenstellung starten.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.en.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.en.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Conservative Replay Import (slightly slower, but more robust)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Build</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>FEN string is empty. Please apply a valid FEN before starting the build.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.es.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.es.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Importación de repeticiones conservadora (ligeramente más lenta, pero más robusta)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Composición</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>La cadena FEN está vacía. Por favor, aplique un FEN válido antes de comenzar la composición.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.fr.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.fr.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Importation de rediffusions conservatrice (légèrement plus lente, mais plus robuste)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Composition</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>La chaîne FEN est vide. Veuillez appliquer un FEN valide avant de commencer la composition.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Conservative Replay Import (slightly slower, but more robust)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Build</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>FEN string is empty. Please apply a valid FEN before starting the build.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.ru.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.ru.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Консервативный импорт повторов (немного медленнее, но более надежный)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Компоновка</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>Строка FEN пуста. Пожалуйста, примените действительный FEN перед началом компоновки.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/DsstatsLoc.uk.resx b/src/dsstats.maui/dsstats.localization/DsstatsLoc.uk.resx
@@ -406,4 +406,10 @@
   <data name="NoBatchImport" xml:space="preserve">
     <value>Консервативний імпорт повторів (трохи повільніше, але надійніше)</value>
   </data>
+  <data name="Build" xml:space="preserve">
+    <value>Композиція</value>
+  </data>
+  <data name="FENempty" xml:space="preserve">
+    <value>Рядок FEN порожній. Будь ласка, застосуйте дійсний FEN перед початком компоновки.</value>
+  </data>
 </root>
diff --git a/src/dsstats.maui/dsstats.localization/dsstats_languages.csv b/src/dsstats.maui/dsstats.localization/dsstats_languages.csv
@@ -114,4 +114,6 @@ Should a backup be created at this location now?;Should a backup be created at t
 Restore Database?;Restore Database?; Datenbank wiederherstellen?; ¿Restaurar base de datos?; Восстановить базу данных?; Restaurer la base de données ?; Відновити базу даних?
 The current data will be overwritten.;The current data will be overwritten.; Die aktuellen Daten werden überschrieben.; Los datos actuales serán sobrescritos.; Текущие данные будут перезаписаны.; Les données actuelles seront écrasées.; Поточні дані будуть перезаписані.
 Please check if the settings are as expected.;Please check if the settings are as expected.; Bitte überprüfen Sie, ob die Einstellungen wie erwartet sind.; Por favor, compruebe si la configuración es la esperada.; Пожалуйста, проверьте, соответствуют ли настройки ожидаемым.; Veuillez vérifier si les paramètres sont conformes aux attentes.; Будь ласка, перевірте, чи налаштування відповідають очікуванням.
-NoBatchImport;Conservative Replay Import (slightly slower, but more robust);Konservativer Wiederholungsimport (etwas langsamer, aber robuster); Importación de repeticiones conservadora (ligeramente más lenta, pero más robusta); Консервативный импорт повторов (немного медленнее, но более надежный); Importation de rediffusions conservatrice (légèrement plus lente, mais plus robuste); Консервативний імпорт повторів (трохи повільніше, але надійніше)
+NoBatchImport;Conservative Replay Import (slightly slower, but more robust);Konservativer Wiederholungsimport (etwas langsamer, aber robuster); Importación de repeticiones conservadora (ligeramente más lenta, pero más robusta); Консервативный импорт повторов (немного медленнее, но более надежный); Importation de rediffusions conservatrice (légèrement plus lente, mais plus robuste); Консервативний імпорт повторів (трохи повільніше, але надійніше)
+Build;Build; Zusammenstellung; Composición; Компоновка; Composition; Композиція
+FENempty;FEN string is empty. Please apply a valid FEN before starting the build.; FEN-Zeichenkette ist leer. Bitte wenden Sie eine gültige FEN an, bevor Sie die Zusammenstellung starten.; La cadena FEN está vacía. Por favor, aplique un FEN válido antes de comenzar la composición.; Строка FEN пуста. Пожалуйста, примените действительный FEN перед началом компоновки.; La chaîne FEN est vide. Veuillez appliquer un FEN valide avant de commencer la composition.; Рядок FEN порожній. Будь ласка, застосуйте дійсний FEN перед початком компоновки.
diff --git a/src/dsstats.maui/dsstats.maui8/Components/Pages/Home.razor b/src/dsstats.maui/dsstats.maui8/Components/Pages/Home.razor
@@ -26,6 +26,7 @@
     }
     <button type="button" class="btn btn-outline-success" @onclick="e => dsstatsService.ScanForNewReplays()">@Loc["Scan"]</button>
     <button type="button" class="btn btn-outline-info" @onclick="Upload">@Loc["Upload"]</button>
+    <button type="button" class="btn btn-outline-light" @onclick="e => buildModal?.Show(new SpawnDto(), [], Commander.None, 0)">@Loc["Build"]</button>
 </div>
 <div class="row justify-content-between">
     <div class="col-auto">
@@ -105,4 +106,6 @@
 }
 <div style="height: 50px; width: 50px; position: relative;">
     <dsstats.razorlib.Players.Profile.AnnotationChartComponent OnAnnotationsRegistered="AnnotationsRegistered" />
-</div>
+</div>
+
+<dsstats.razorlib.Modals.BuildModal @ref="buildModal" />
diff --git a/src/dsstats.maui/dsstats.maui8/Components/Pages/Home.razor.cs b/src/dsstats.maui/dsstats.maui8/Components/Pages/Home.razor.cs
@@ -1,6 +1,7 @@
 ﻿using Blazored.Toast.Services;
 using dsstats.db8services;
 using dsstats.maui8.Services;
+using dsstats.razorlib.Modals;
 using dsstats.razorlib.Players.Profile;
 using dsstats.razorlib.Replays;
 using dsstats.shared;
@@ -39,6 +40,7 @@ public partial class Home : ComponentBase, IDisposable
     ReplayComponent? replayComponent;
     bool DEBUG = false;
     bool isChartAnnotationPluginRegistered;
+    BuildModal? buildModal;
 
     protected override void OnInitialized()
     {
diff --git a/src/dsstats.razorlib/Modals/BuildModal.razor b/src/dsstats.razorlib/Modals/BuildModal.razor
@@ -96,6 +96,11 @@
 
     private void StartTimerPl()
     {
+        if (string.IsNullOrEmpty(fen))
+        {
+            ToastService.ShowError("FEN string is empty. Please apply a valid FEN before starting the build.");
+            return;
+        }
         starting = true;
         timerComponent?.Start(5);
     }
diff --git a/src/dsstats.razorlib/Replays/ReplayPlayerDetails.razor b/src/dsstats.razorlib/Replays/ReplayPlayerDetails.razor
@@ -2,6 +2,7 @@
 @using dsstats.razorlib.Modals
 @using dsstats.shared
 @using dsstats.razorlib.Services
+@using dsstats.shared.DsFen
 @using dsstats.shared.Interfaces
 @using static dsstats.razorlib.Replays.ReplayPlayersDetailContainer
 @inject IDsDataService dsDataService
@@ -92,7 +93,7 @@
                     </h5>
                 </div>
             }
-            @if (Spawn != null && Player.Race != Commander.None && (int)Player.Race <= 3)
+            @if (Spawn != null && CmdrBuildFactory.IsSupported(Player.Race))
             {
                 <div class="col-auto">
                     <button type="button" class="btn btn-sm btn-outline-light"
diff --git a/src/dsstats.shared/DsFen/AbathurBuild.cs b/src/dsstats.shared/DsFen/AbathurBuild.cs
@@ -0,0 +1,64 @@
+﻿namespace dsstats.shared.DsFen;
+
+public class AbathurBuild : CmdrBuild
+{
+    public AbathurBuild()
+    {
+        UnitMap = new Dictionary<string, BuildOption>
+            {
+                { "VileRoach", new('q') },
+                { "SwarmQueen", new('w') },
+
+                { "Mutalisk", new('a', 1, true) },
+                { "Ravager", new('s', 2) },
+                { "Devourer", new('d', 2, true) },
+                { "Overseer", new('f', 2, true) },
+                { "SwarmHost", new('g', 2) },
+
+                { "Viper", new('z', 2, true) },
+                { "Guardian", new('x', 2, true) },
+                { "Brutalisk", new('c', 2) },
+                { "Leviathan", new('v', 3, true) },
+            };
+
+        AbilityMap = new Dictionary<string, BuildOption>
+            {
+                { "GlialReconstitution", new('q') },
+                { "TunnelingClaws", new('w') },
+                { "VileRoachHydriodicBile", new('r') },
+                { "VileRoachAdaptivePlating", new('e') },
+                { "SwarmQueenBioMechanicalTransfusion", new('t') },
+
+                { "RavagerBloatedBileDucts", new('a') },
+                { "RavagerPotentBile", new('s') },
+                { "GuardianProlongedDispersion", new('d') },
+                { "DevourerCorrosiveSpray", new('f') },
+                { "overlordspeed", new('g') },
+
+                { "BroodMutaliskViciousGlave", new('z') },
+                { "BroodMutaliskSunderingGlave", new('x') },
+                { "SwarmHostPressurizedGlands", new('c') },
+                { "ViperVirulentMicrobes", new('v') },
+            };
+
+        UpgradeMap = new Dictionary<string, BuildOption>
+            {
+                { "ZergMeleeWeaponsLevel1", new('a') },
+                { "ZergMeleeWeaponsLevel2", new('a') },
+                { "ZergMeleeWeaponsLevel3", new('a') },
+                { "ZergGroundArmorsLevel1", new('s') },
+                { "ZergGroundArmorsLevel2", new('s') },
+                { "ZergGroundArmorsLevel3", new('s') },
+                { "ZergMissileWeaponsLevel1", new('d') },
+                { "ZergMissileWeaponsLevel2", new('d') },
+                { "ZergMissileWeaponsLevel3", new('d') },
+                { "ZergFlyerWeaponsLevel1", new('f') },
+                { "ZergFlyerWeaponsLevel2", new('f') },
+                { "ZergFlyerWeaponsLevel3", new('f') },
+                { "ZergFlyerArmorsLevel1", new('g') },
+                { "ZergFlyerArmorsLevel2", new('g') },
+                { "ZergFlyerArmorsLevel3", new('g') },
+            };
+        CreateActiveUnits();
+    }
+}
diff --git a/src/dsstats.shared/DsFen/CmdrBuild.cs b/src/dsstats.shared/DsFen/CmdrBuild.cs
@@ -1,4 +1,6 @@
-﻿namespace dsstats.shared.DsFen;
+﻿using System.Collections.Frozen;
+
+namespace dsstats.shared.DsFen;
 
 public abstract class CmdrBuild
 {
@@ -89,14 +91,29 @@ public sealed record BuildOption(char Key, int UnitSize = 1, bool IsAir = false,
 
 public static class CmdrBuildFactory
 {
+    private static readonly HashSet<Commander> SupportedCommanders = new()
+    {
+        Commander.Protoss,
+        Commander.Terran,
+        Commander.Zerg,
+        Commander.Abathur
+    };
+
     public static CmdrBuild? Create(Commander commander)
     {
         return commander switch
         {
             Commander.Protoss => new ProtossBuild(),
             Commander.Terran => new TerranBuild(),
             Commander.Zerg => new ZergBuild(),
+
+            Commander.Abathur => new AbathurBuild(),
             _ => null
         };
     }
+
+    public static bool IsSupported(Commander commander)
+    {
+        return SupportedCommanders.Contains(commander);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ public async Task<bool> SaveSubmission(ChallengeResponse response, int spChallen`
`87`	`87`	`var submission = await context.SpChallengeSubmissions`
`88`	`88`	`.FirstOrDefaultAsync(s => s.SpChallengeId == spChallengeId && s.PlayerId == playerId)`
`89`	`89`	`;`
`90`		`- if (submission == null)`
	`90`	`+ if (submission == null \|\| submission.Commander == response.Commander)`
`91`	`91`	`{`
`92`	`92`	`submission = new SpChallengeSubmission`
`93`	`93`	`{`