Made Before<,>.Other a required property (#764)

* Made Before<,>.Other a required property * Adapted code to the change
Nexus-Mods · Nov 10, 2023 · f699d8e · f699d8e
1 parent bec16f5
commit f699d8e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/src/Games/NexusMods.Games.BethesdaGameStudios/PluginSorter.cs b/src/Games/NexusMods.Games.BethesdaGameStudios/PluginSorter.cs
@@ -91,7 +91,7 @@ private IEnumerable<ISortRule<RuleTuple, RelativePath>> GenerateRules(PluginAnal
                     foreach (var itm in allPlugins)
                     {
                         if (DefaultOrdering.Contains(itm.FileName)) continue;
-                        yield return new Before<RuleTuple, RelativePath>(itm.FileName);
+                        yield return new Before<RuleTuple, RelativePath> { Other = itm.FileName };
                     }
 
                     break;

diff --git a/src/NexusMods.DataModel/Sorting/Rules/Before.cs b/src/NexusMods.DataModel/Sorting/Rules/Before.cs
@@ -4,4 +4,7 @@ namespace NexusMods.DataModel.Sorting.Rules;
 
 /// <summary />
 [JsonName("Before")]
-public record Before<TType, TId>(TId Other) : ISortRule<TType, TId>;
+public record Before<TType, TId>: ISortRule<TType, TId>
+{
+    public required TId Other { get; init; }
+}
diff --git a/tests/NexusMods.DataModel.Tests/Harness/ALoadoutSynrconizerTest.cs b/tests/NexusMods.DataModel.Tests/Harness/ALoadoutSynrconizerTest.cs
@@ -171,7 +171,7 @@ public async IAsyncEnumerable<ISortRule<Mod, ModId>> GenerateSortRules(ModId sel
             if (modId.Equals(selfId)) continue;
             if (string.Compare(other.Name, thisMod.Name, StringComparison.Ordinal) > 0)
             {
-                yield return new Before<Mod, ModId>(other.Id);
+                yield return new Before<Mod, ModId> { Other = other.Id };
             }
             else
             {

diff --git a/tests/NexusMods.DataModel.Tests/SortTests.cs b/tests/NexusMods.DataModel.Tests/SortTests.cs
@@ -33,7 +33,7 @@ public void BeforeAndAfterWorks()
             new Item {Id = "B", Rules = new()},
             new Item {Id = "A", Rules = new()
             {
-                new Before<Item, string>("B")
+                new Before<Item, string> { Other = "B" }
             }},
             new Item {Id = "C", Rules = new()
             {