From 6f6e2a72a2ebf753635e15347fc62d972b4f63f5 Mon Sep 17 00:00:00 2001 From: Erik Darling <2136037+erikdarlingdata@users.noreply.github.com> Date: Fri, 6 Mar 2026 18:11:54 -0600 Subject: [PATCH] Fix nullable databaseName parameter in ActualPlanExecutor The method already guards against null with IsNullOrEmpty, but the parameter type was non-nullable, causing CS8604 at the MainWindow call site where database can legitimately be null. Co-Authored-By: Claude Opus 4.6 --- src/PlanViewer.Core/Services/ActualPlanExecutor.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PlanViewer.Core/Services/ActualPlanExecutor.cs b/src/PlanViewer.Core/Services/ActualPlanExecutor.cs index d5fc41a..d77debb 100644 --- a/src/PlanViewer.Core/Services/ActualPlanExecutor.cs +++ b/src/PlanViewer.Core/Services/ActualPlanExecutor.cs @@ -36,7 +36,7 @@ public static class ActualPlanExecutor /// The actual execution plan XML, or null if no plan was captured. public static async Task ExecuteForActualPlanAsync( string connectionString, - string databaseName, + string? databaseName, string queryText, string? planXml, string? isolationLevel,