Fixed nullability warnings

riganti · Nov 3, 2024 · 36615b8 · 36615b8
1 parent 11784bb
commit 36615b8
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 8 deletions.
diff --git a/src/Framework/Framework/Compilation/DotHtmlFileInfo.cs b/src/Framework/Framework/Compilation/DotHtmlFileInfo.cs
@@ -14,7 +14,7 @@ public sealed class DotHtmlFileInfo
         public ImmutableArray<CompilationDiagnosticViewModel> Warnings { get; internal set; } = ImmutableArray<CompilationDiagnosticViewModel>.Empty;
 
         /// <summary>Gets or sets the virtual path to the view.</summary>
-        public string VirtualPath { get; }
+        public string? VirtualPath { get; }
 
         public string? TagName { get; }
         public string? Namespace { get; }
@@ -25,7 +25,7 @@ public sealed class DotHtmlFileInfo
         public ImmutableArray<string>? DefaultValues { get; }
         public bool? HasParameters { get; }
 
-        public DotHtmlFileInfo(string virtualPath, string? tagName = null, string? nameSpace = null, string? assembly = null, string? tagPrefix = null, string? url = null, string? routeName = null, ImmutableArray<string>? defaultValues = null, bool? hasParameters = null)
+        public DotHtmlFileInfo(string? virtualPath, string? tagName = null, string? nameSpace = null, string? assembly = null, string? tagPrefix = null, string? url = null, string? routeName = null, ImmutableArray<string>? defaultValues = null, bool? hasParameters = null)
         {
             VirtualPath = virtualPath;
             Status = IsDothtmlFile(virtualPath) ? CompilationState.None : CompilationState.NonCompilable;
@@ -40,7 +40,7 @@ public DotHtmlFileInfo(string virtualPath, string? tagName = null, string? nameS
             HasParameters = hasParameters;
         }
 
-        private static bool IsDothtmlFile(string virtualPath)
+        private static bool IsDothtmlFile(string? virtualPath)
         {
             return !string.IsNullOrWhiteSpace(virtualPath) &&
                 (

diff --git a/src/Framework/Framework/Compilation/Static/StaticViewCompiler.cs b/src/Framework/Framework/Compilation/Static/StaticViewCompiler.cs
@@ -10,6 +10,7 @@
 using DotVVM.Framework.Configuration;
 using DotVVM.Framework.Hosting;
 using DotVVM.Framework.Security;
+using DotVVM.Framework.Utils;
 using Microsoft.CodeAnalysis;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
@@ -33,7 +34,7 @@ public static ImmutableArray<DotvvmCompilationDiagnostic> CompileAll(
                 diagnostics.AddRange(CompileNoThrow(configuration, markupControl!));
             }
 
-            var views = configuration.RouteTable.Select(r => r.VirtualPath).ToImmutableArray();
+            var views = configuration.RouteTable.Select(r => r.VirtualPath).WhereNotNull().ToImmutableArray();
             foreach(var view in views)
             {
                 diagnostics.AddRange(CompileNoThrow(configuration, view));

diff --git a/src/Framework/Framework/Hosting/AggregateMarkupFileLoader.cs b/src/Framework/Framework/Hosting/AggregateMarkupFileLoader.cs
@@ -3,6 +3,7 @@
 using System.Linq;
 using System.Text;
 using DotVVM.Framework.Configuration;
+using DotVVM.Framework.Utils;
 
 namespace DotVVM.Framework.Hosting
 {
@@ -39,7 +40,8 @@ public AggregateMarkupFileLoader()
         /// </summary>
         public string GetMarkupFileVirtualPath(IDotvvmRequestContext context)
         {
-            return context.Route!.VirtualPath;
+            return context.Route!.VirtualPath
+                ?? throw new Exception($"The route {context.Route.RouteName} must have a non-null virtual path.");
         }
     }
 }
diff --git a/src/Framework/Framework/Hosting/DefaultMarkupFileLoader.cs b/src/Framework/Framework/Hosting/DefaultMarkupFileLoader.cs
@@ -38,7 +38,8 @@ public class DefaultMarkupFileLoader : IMarkupFileLoader
         /// </summary>
         public string GetMarkupFileVirtualPath(IDotvvmRequestContext context)
         {
-            return context.Route!.VirtualPath;
+            return context.Route!.VirtualPath
+                ?? throw new Exception($"The route {context.Route.RouteName} must have a non-null virtual path.");
         }
     }
 }
diff --git a/src/Framework/Framework/Hosting/EmbeddedMarkupFileLoader.cs b/src/Framework/Framework/Hosting/EmbeddedMarkupFileLoader.cs
@@ -58,7 +58,8 @@ public class EmbeddedMarkupFileLoader : IMarkupFileLoader
         /// </summary>
         public string GetMarkupFileVirtualPath(IDotvvmRequestContext context)
         {
-            return context.Route!.VirtualPath;
+            return context.Route!.VirtualPath
+                ?? throw new Exception($"The route {context.Route.RouteName} must have a non-null virtual path.");
         }
     }
 }

diff --git a/src/Framework/Framework/Routing/RouteHelper.cs b/src/Framework/Framework/Routing/RouteHelper.cs
@@ -52,7 +52,7 @@ public static void AssertConfigurationIsValid(this DotvvmConfiguration config)
                     invalidRoutes.Add(new DotvvmConfigurationAssertResult<RouteBase>(route, DotvvmConfigurationAssertReason.MissingRouteName));
                 }
 
-                var content = loader.GetMarkup(config, route.VirtualPath);
+                var content = loader.GetMarkup(config, route.VirtualPath!);
                 if (content == null)
                 {
                     invalidRoutes.Add(new DotvvmConfigurationAssertResult<RouteBase>(route, DotvvmConfigurationAssertReason.MissingFile));