Rename beast tribe to allied society in code
authorLiza Carvelli <liza@carvel.li>
Sat, 17 Aug 2024 17:16:33 +0000 (19:16 +0200)
committerLiza Carvelli <liza@carvel.li>
Sat, 17 Aug 2024 17:16:33 +0000 (19:16 +0200)
Questionable/Model/EAlliedSociety.cs [new file with mode: 0644]
Questionable/Model/EBeastTribe.cs [deleted file]
Questionable/Model/IQuestInfo.cs
Questionable/Model/LeveInfo.cs
Questionable/Model/QuestInfo.cs
Questionable/Model/SatisfactionSupplyInfo.cs
Questionable/Validation/QuestValidator.cs
Questionable/Validation/ValidationIssue.cs
Questionable/Windows/QuestValidationWindow.cs

diff --git a/Questionable/Model/EAlliedSociety.cs b/Questionable/Model/EAlliedSociety.cs
new file mode 100644 (file)
index 0000000..2f7281f
--- /dev/null
@@ -0,0 +1,28 @@
+using System.Diagnostics.CodeAnalysis;
+using JetBrains.Annotations;
+
+namespace Questionable.Model;
+
+[SuppressMessage("Design", "CA1028", Justification = "Game type")]
+[UsedImplicitly(ImplicitUseTargetFlags.Members)]
+public enum EAlliedSociety : byte
+{
+    None = 0,
+    Amaljaa = 1,
+    Sylphs = 2,
+    Kobolds = 3,
+    Sahagin = 4,
+    Ixal = 5,
+    VanuVanu = 6,
+    Vath = 7,
+    Moogles = 8,
+    Kojin = 9,
+    Ananta = 10,
+    Namazu = 11,
+    Pixies = 12,
+    Qitari = 13,
+    Dwarves = 14,
+    Arkasodara = 15,
+    Omicrons = 16,
+    Loporrits = 17,
+}
diff --git a/Questionable/Model/EBeastTribe.cs b/Questionable/Model/EBeastTribe.cs
deleted file mode 100644 (file)
index e26fb67..0000000
+++ /dev/null
@@ -1,28 +0,0 @@
-using System.Diagnostics.CodeAnalysis;
-using JetBrains.Annotations;
-
-namespace Questionable.Model;
-
-[SuppressMessage("Design", "CA1028", Justification = "Game type")]
-[UsedImplicitly(ImplicitUseTargetFlags.Members)]
-public enum EBeastTribe : byte
-{
-    None = 0,
-    Amaljaa = 1,
-    Sylphs = 2,
-    Kobolds = 3,
-    Sahagin = 4,
-    Ixal = 5,
-    VanuVanu = 6,
-    Vath = 7,
-    Moogles = 8,
-    Kojin = 9,
-    Ananta = 10,
-    Namazu = 11,
-    Pixies = 12,
-    Qitari = 13,
-    Dwarves = 14,
-    Arkasodara = 15,
-    Omicrons = 16,
-    Loporrits = 17,
-}
index 1a2f557a0d7b1ea2e3978ea8c2aa1282b4d69f7f..5bc8d5f985252f5f5e92fb1ace3fe8db46bdc460 100644 (file)
@@ -13,7 +13,7 @@ public interface IQuestInfo
     public uint IssuerDataId { get; }
     public bool IsRepeatable { get; }
     public ushort Level { get; }
-    public EBeastTribe BeastTribe { get; }
+    public EAlliedSociety AlliedSociety { get; }
     public uint? JournalGenre { get; }
     public ushort SortKey { get; }
     public bool IsMainScenarioQuest { get; }
index 8d454e5b181a6a1d782901e86dbf35d6a9d35565..b446da9a77fea4ffd0ee787baf7b19c096546c3b 100644 (file)
@@ -24,7 +24,7 @@ internal sealed class LeveInfo : IQuestInfo
     public uint IssuerDataId { get; }
     public bool IsRepeatable => true;
     public ushort Level { get; }
-    public EBeastTribe BeastTribe => EBeastTribe.None;
+    public EAlliedSociety AlliedSociety => EAlliedSociety.None;
     public uint? JournalGenre { get; }
     public ushort SortKey { get; }
     public bool IsMainScenarioQuest => false;
index e9a70ea3ab419190db72fc2b85bba995be996bcf..2c9c295174b835fd8a57758fc8da99d4f2ad4b0d 100644 (file)
@@ -51,7 +51,7 @@ internal sealed class QuestInfo : IQuestInfo
         PreviousInstanceContent = quest.InstanceContent.Select(x => (ushort)x.Row).Where(x => x != 0).ToList();
         PreviousInstanceContentJoin = (QuestJoin)quest.InstanceContentJoin;
         GrandCompany = (GrandCompany)quest.GrandCompany.Row;
-        BeastTribe = (EBeastTribe)quest.BeastTribe.Row;
+        AlliedSociety = (EAlliedSociety)quest.BeastTribe.Row;
         ClassJobs = QuestInfoUtils.AsList(quest.ClassJobCategory0.Value!);
         IsSeasonalEvent = quest.Festival.Row != 0;
         NewGamePlusChapter = newGamePlusChapter;
@@ -75,7 +75,7 @@ internal sealed class QuestInfo : IQuestInfo
     public bool IsMainScenarioQuest { get; }
     public bool CompletesInstantly { get; }
     public GrandCompany GrandCompany { get; }
-    public EBeastTribe BeastTribe { get; }
+    public EAlliedSociety AlliedSociety { get; }
     public IReadOnlyList<EClassJob> ClassJobs { get; }
     public bool IsSeasonalEvent { get; }
     public ushort NewGamePlusChapter { get; }
index 17c56772fc1de1514aec3d2dd007f043b516c9a7..9b3a7a6d694c2ed77df1d792e2f54fd7d20d25ec 100644 (file)
@@ -22,7 +22,7 @@ internal sealed class SatisfactionSupplyInfo : IQuestInfo
     public uint IssuerDataId { get; }
     public bool IsRepeatable => true;
     public ushort Level { get; }
-    public EBeastTribe BeastTribe => EBeastTribe.None;
+    public EAlliedSociety AlliedSociety => EAlliedSociety.None;
     public uint? JournalGenre => null;
     public ushort SortKey { get; }
     public bool IsMainScenarioQuest => false;
index 44f952f34e2af97b9309a76689a4cb74a0d98b5f..c91f0df2fe4219325a0cc6a4c6116b8a2a2ca1fb 100644 (file)
@@ -44,7 +44,7 @@ internal sealed class QuestValidator
                 _validationIssues.Clear();
 
                 List<ValidationIssue> issues = [];
-                Dictionary<EBeastTribe, int> disabledTribeQuests = new();
+                Dictionary<EAlliedSociety, int> disabledTribeQuests = new();
                 foreach (var quest in quests)
                 {
                     foreach (var validator in _validators)
@@ -57,10 +57,10 @@ internal sealed class QuestValidator
                             _logger.Log(level,
                                 "Validation failed: {QuestId} ({QuestName}) / {QuestSequence} / {QuestStep} - {Description}",
                                 issue.ElementId, quest.Info.Name, issue.Sequence, issue.Step, issue.Description);
-                            if (issue.Type == EIssueType.QuestDisabled && quest.Info.BeastTribe != EBeastTribe.None)
+                            if (issue.Type == EIssueType.QuestDisabled && quest.Info.AlliedSociety != EAlliedSociety.None)
                             {
-                                disabledTribeQuests.TryAdd(quest.Info.BeastTribe, 0);
-                                disabledTribeQuests[quest.Info.BeastTribe]++;
+                                disabledTribeQuests.TryAdd(quest.Info.AlliedSociety, 0);
+                                disabledTribeQuests[quest.Info.AlliedSociety]++;
                             }
                             else
                                 issues.Add(issue);
@@ -89,7 +89,7 @@ internal sealed class QuestValidator
         }, CancellationToken.None, TaskCreationOptions.LongRunning, TaskScheduler.Default);
     }
 
-    private static IEnumerable<ValidationIssue> DisabledTribesAsIssues(Dictionary<EBeastTribe, int> disabledTribeQuests)
+    private static IEnumerable<ValidationIssue> DisabledTribesAsIssues(Dictionary<EAlliedSociety, int> disabledTribeQuests)
     {
         return disabledTribeQuests
             .OrderBy(x => x.Key)
@@ -98,7 +98,7 @@ internal sealed class QuestValidator
                 ElementId = null,
                 Sequence = null,
                 Step = null,
-                BeastTribe = x.Key,
+                AlliedSociety = x.Key,
                 Type = EIssueType.QuestDisabled,
                 Severity = EIssueSeverity.None,
                 Description = $"{x.Value} disabled quest(s)",
index 654ac514bbd3410f75cefe01ea4503328994fce5..b77c21fe9bab58d1387b816b9c598330797a0d57 100644 (file)
@@ -8,7 +8,7 @@ internal sealed record ValidationIssue
     public required ElementId? ElementId { get; init; }
     public required byte? Sequence { get; init; }
     public required int? Step { get; init; }
-    public EBeastTribe BeastTribe { get; init; } = EBeastTribe.None;
+    public EAlliedSociety AlliedSociety { get; init; } = EAlliedSociety.None;
     public required EIssueType Type { get; init; }
     public required EIssueSeverity Severity { get; init; }
     public required string Description { get; init; }
index 8f3eb3ac5581101a7d3e1640e2fc84a3fa6c4d73..173c82f37ac03f2f1f9274d06f5ba8fffb71e846 100644 (file)
@@ -61,7 +61,7 @@ internal sealed class QuestValidationWindow : LWindow
             if (ImGui.TableNextColumn())
                 ImGui.TextUnformatted(validationIssue.ElementId != null
                     ? _questData.GetQuestInfo(validationIssue.ElementId).Name
-                    : validationIssue.BeastTribe.ToString());
+                    : validationIssue.AlliedSociety.ToString());
 
             if (ImGui.TableNextColumn())
                 ImGui.TextUnformatted(validationIssue.Sequence?.ToString(CultureInfo.InvariantCulture) ?? string.Empty);