Fix: Fixed potential COMException when updating preview pane selection (#13919)

yaira2 · yaira2 · commit ee2fb9eedf46 · 2023-11-17T11:57:09.000-05:00
diff --git a/src/Files.App/Views/LayoutModes/BaseLayout.cs b/src/Files.App/Views/LayoutModes/BaseLayout.cs
@@ -1448,7 +1448,7 @@ protected void UpdatePreviewPaneSelection(List<ListedItem>? value)
 				InfoPaneViewModel.SelectedItem = value?.Count == 1 ? value.First() : null;
 
 				// Check if the preview pane is open before updating the model
-				if (InfoPaneViewModel.IsEnabled)
+				if (InfoPaneViewModel.IsEnabled && !App.AppModel.IsMainWindowClosed)
 				{
 					var isPaneEnabled = ((MainWindow.Instance.Content as Frame)?.Content as MainPage)?.ShouldPreviewPaneBeActive ?? false;
 					if (isPaneEnabled)

Original file line number	Diff line number	Diff line change
`@@ -1448,7 +1448,7 @@ protected void UpdatePreviewPaneSelection(List<ListedItem>? value)`
`1448`	`1448`	`InfoPaneViewModel.SelectedItem = value?.Count == 1 ? value.First() : null;`
`1449`	`1449`
`1450`	`1450`	`// Check if the preview pane is open before updating the model`
`1451`		`- if (InfoPaneViewModel.IsEnabled)`
	`1451`	`+ if (InfoPaneViewModel.IsEnabled && !App.AppModel.IsMainWindowClosed)`
`1452`	`1452`	`{`
`1453`	`1453`	`var isPaneEnabled = ((MainWindow.Instance.Content as Frame)?.Content as MainPage)?.ShouldPreviewPaneBeActive ?? false;`
`1454`	`1454`	`if (isPaneEnabled)`