Merge #2585 Fix crash on selecting filtered-out provider module

KSP-CKAN · Nov 29, 2018 · 5c0c97d · 5c0c97d
2 parents 960c51d + dfc7fb0
commit 5c0c97d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -46,6 +46,7 @@ All notable changes to this project will be documented in this file.
 - [Netkan] Treat AVC min without max as open range (#2571 by: HebaruSan; reviewed: politas)
 - [GUI] Restore uninstallation of dependencies (#2579 by: HebaruSan; reviewed: politas)
 - [GUI] Fix tray icon menu position and color on Linux (Gnome) (#2587 by: HebaruSan; reviewed: politas)
+- [GUI] Fix crash on selecting filtered-out provider module (#2585 by: HebaruSan; reviewed: politas)
 
 ## v1.25.3 (Woomera)
 

diff --git a/GUI/GUIMod.cs b/GUI/GUIMod.cs
@@ -279,10 +279,13 @@ public void SetInstallChecked(DataGridViewRow row, bool? set_value_to = null)
                 if ((bool)install_cell.Value != IsInstallChecked)
                 {
                     install_cell.Value = IsInstallChecked;
-                    // These calls are needed to force the UI to update,
-                    // otherwise the checkbox will look checked when it's unchecked or vice versa
-                    row.DataGridView.RefreshEdit();
-                    row.DataGridView.Refresh();
+                    if (row.DataGridView != null)
+                    {
+                        // These calls are needed to force the UI to update,
+                        // otherwise the checkbox will look checked when it's unchecked or vice versa
+                        row.DataGridView.RefreshEdit();
+                        row.DataGridView.Refresh();
+                    }
                 }
             }
         }