From 102b88f303c9635500e42465ebe92f92a87270f8 Mon Sep 17 00:00:00 2001
From: atravita-mods <94934860+atravita-mods@users.noreply.github.com>
Date: Sat, 1 Jan 2022 20:55:49 -0500
Subject: [PATCH] Account for the fact that RecentSOManager can give a null
 when not ready.

---
 SpecialOrdersExtended/DataModels/RecentCompletedSO.cs |  1 +
 SpecialOrdersExtended/DialogueManager.cs              | 10 +++++++---
 SpecialOrdersExtended/SpecialOrdersExtended.csproj    |  4 +++-
 SpecialOrdersExtended/StatsManager.cs                 |  1 +
 4 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/SpecialOrdersExtended/DataModels/RecentCompletedSO.cs b/SpecialOrdersExtended/DataModels/RecentCompletedSO.cs
index 4cb465c..39d9e73 100644
--- a/SpecialOrdersExtended/DataModels/RecentCompletedSO.cs
+++ b/SpecialOrdersExtended/DataModels/RecentCompletedSO.cs
@@ -25,6 +25,7 @@ public void Save()
         base.Save(identifier);
     }
 
+    [SuppressMessage("ReSharper", "IDE1006", Justification = "Method naming follows convention used in-game")]
     public void dayUpdate(uint daysPlayed)
     {
         foreach (string key in RecentOrdersCompleted.Keys)
diff --git a/SpecialOrdersExtended/DialogueManager.cs b/SpecialOrdersExtended/DialogueManager.cs
index 32f206c..7f2d1a4 100644
--- a/SpecialOrdersExtended/DialogueManager.cs
+++ b/SpecialOrdersExtended/DialogueManager.cs
@@ -103,10 +103,14 @@ public static void PostfixCheckDialogue(ref bool __result, ref NPC __instance, i
                 __result = FindBestDialogue(baseKey, __instance, __0);
                 if (__result) { return; }
             }
-            foreach (string cacheOrder in RecentSOManager.GetKeys(1u))
+            List<string> cacheOrders = RecentSOManager.GetKeys(1u)?.ToList();
+            if (cacheOrders is not null)
             {
-                __result = FindBestDialogue(cacheOrder + "_Completed", __instance, __0);
-                if (__result) { return; }
+                foreach (string cacheOrder in cacheOrders)
+                {
+                    __result = FindBestDialogue(cacheOrder + "_Completed", __instance, __0);
+                    if (__result) { return; }
+                }
             }
         }
         catch (Exception ex)
diff --git a/SpecialOrdersExtended/SpecialOrdersExtended.csproj b/SpecialOrdersExtended/SpecialOrdersExtended.csproj
index 9309223..000c67c 100644
--- a/SpecialOrdersExtended/SpecialOrdersExtended.csproj
+++ b/SpecialOrdersExtended/SpecialOrdersExtended.csproj
@@ -7,12 +7,14 @@
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Pathoschild.Stardew.ModBuildConfig" Version="4.0.0-beta.20210916" />
+    <PackageReference Include="Microsoft.Win32.Registry" Version="6.0.0-preview.5.21301.5" />
+    <PackageReference Include="Pathoschild.Stardew.ModBuildConfig" Version="4.0.0" />
     <PackageReference Include="Pathoschild.Stardew.ModTranslationClassBuilder" Version="2.0.0" />
   </ItemGroup>
 
   <ItemGroup>
     <Using Include="StardewValley" />
     <Using Include="StardewModdingAPI" />
+    <Using Include="System.Diagnostics.CodeAnalysis" />
   </ItemGroup>
 </Project>
\ No newline at end of file
diff --git a/SpecialOrdersExtended/StatsManager.cs b/SpecialOrdersExtended/StatsManager.cs
index dc375ea..5c4e53a 100644
--- a/SpecialOrdersExtended/StatsManager.cs
+++ b/SpecialOrdersExtended/StatsManager.cs
@@ -41,6 +41,7 @@ public uint GrabBasicProperty(string key, Stats stats)
         return 0u;
     }
 
+    [SuppressMessage("ReSharper", "IDE0060", Justification = "Format expected by console commands")]
     public void ConsoleListProperties(string command, string[] args)
     {
         if (propertyInfos.Count.Equals(0)) { GrabProperties(); }