Fix NRE on trying to update all when there's nothing to update

KSP-CKAN · Mar 10, 2024 · 59a7152 · 59a7152
1 parent 1cd1901
commit 59a7152
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/GUI/Controls/ManageMods.cs b/GUI/Controls/ManageMods.cs
@@ -154,17 +154,17 @@ private void ChangeSetUpdated()
                 var removing = changeIdentifiersOfType(GUIModChangeType.Remove)
                                .Except(changeIdentifiersOfType(GUIModChangeType.Install))
                                .ToHashSet();
-                foreach (var kvp in mainModList.full_list_of_mod_rows)
+                foreach ((string ident, DataGridViewRow row) in mainModList.full_list_of_mod_rows)
                 {
-                    if (removing.Contains(kvp.Key))
+                    if (removing.Contains(ident))
                     {
                         // Set strikeout font for rows being uninstalled
-                        kvp.Value.DefaultCellStyle.Font = uninstallingFont;
+                        row.DefaultCellStyle.Font = uninstallingFont;
                     }
-                    else if (kvp.Value.DefaultCellStyle.Font != null)
+                    else if (row.DefaultCellStyle.Font != null)
                     {
                         // Clear strikeout font for rows not being uninstalled
-                        kvp.Value.DefaultCellStyle.Font = null;
+                        row.DefaultCellStyle.Font = null;
                     }
                 }
             });

diff --git a/GUI/Model/ModList.cs b/GUI/Model/ModList.cs
@@ -43,7 +43,7 @@ public enum GUIModFilter
     public class ModList
     {
         //identifier, row
-        internal Dictionary<string, DataGridViewRow> full_list_of_mod_rows;
+        internal Dictionary<string, DataGridViewRow> full_list_of_mod_rows = new Dictionary<string, DataGridViewRow>();
 
         public event Action ModFiltersUpdated;
         public IReadOnlyCollection<GUIMod> Modules { get; private set; } =