From 4571c985cf3fdb28cd6e4881053e1d4a4b8e0bc6 Mon Sep 17 00:00:00 2001 From: Drew Naylor Date: Thu, 8 Oct 2020 02:37:52 -0400 Subject: [PATCH] Don't crash when trying to load manifests if their cell is Nothing. --- guinget/MainWindow.vb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/guinget/MainWindow.vb b/guinget/MainWindow.vb index 2aec74a2..e87e55db 100644 --- a/guinget/MainWindow.vb +++ b/guinget/MainWindow.vb @@ -372,8 +372,11 @@ Public Class aaformMainWindow End Sub Private Sub datagridviewPackageList_SelectionChanged(sender As Object, e As EventArgs) Handles datagridviewPackageList.SelectionChanged - ' Get package details if only one package is selected. - If datagridviewPackageList.SelectedRows.Count = 1 AndAlso IO.File.Exists(datagridviewPackageList.SelectedRows.Item(0).Cells(7).Value.ToString) Then + ' Get package details if only one package is selected, if the manifest cell isn't Nothing, + ' and the manifest exists. This prevents crashes in case the database is broken. + If datagridviewPackageList.SelectedRows.Count = 1 AndAlso + datagridviewPackageList.SelectedRows.Item(0).Cells(7).Value IsNot Nothing AndAlso + IO.File.Exists(datagridviewPackageList.SelectedRows.Item(0).Cells(7).Value.ToString) Then ' If only one is selected, get its details into the details textbox. ' Set the textbox to say "Loading..." so it doesn't look like it's hanging. textboxPackageDetails.Text = "Loading, please wait..."