Ditch optional name argument. Make Name property not throw, but default empty.

Author: Bazeloth

src/ProgressOnderwijsUtils/Data/SiteBase.cs

@@ -16,5 +16,5 @@ public virtual IContainer? Container
     public virtual bool DesignMode
         => throw new NotSupportedException();
 
-    public virtual string? Name { get; set; }
+    public string? Name { get; set; }
 }

src/ProgressOnderwijsUtils/Data/SqlCommandCreationContext.cs

@@ -10,15 +10,8 @@ public interface IHasDefaultCommandTimeout
     CommandTimeoutDefaults TimeoutDefaults { get; }
 }
 
-public sealed class SqlConnectionContext : SiteBase, IAttachedToTracer, IHasDefaultCommandTimeout
+public sealed class SqlConnectionContext(ISqlCommandTracer tracer, CommandTimeoutDefaults timeoutDefaults) : SiteBase, IAttachedToTracer, IHasDefaultCommandTimeout
 {
-    public SqlConnectionContext(ISqlCommandTracer tracer, CommandTimeoutDefaults timeoutDefaults, string? name = null)
-    {
-        Tracer = tracer;
-        TimeoutDefaults = timeoutDefaults;
-        Name = name;
-    }
-
-    public ISqlCommandTracer Tracer { get; }
-    public CommandTimeoutDefaults TimeoutDefaults { get; set; }
+    public ISqlCommandTracer Tracer { get; } = tracer;
+    public CommandTimeoutDefaults TimeoutDefaults { get; set; } = timeoutDefaults;
 }

src/ProgressOnderwijsUtils/ProgressOnderwijsUtils.csproj

@@ -2,7 +2,7 @@
   <Import Project="..\NugetPackagesCommon.props" />
   <PropertyGroup Label="Configuration">
     <Version>110.3.0</Version>
-    <PackageReleaseNotes>Give the possibility to optionally set a name for your SqlConnectionContext.</PackageReleaseNotes>
+    <PackageReleaseNotes>Do not throw on accessing the Name of SqlConnectionContext to prevent errors on disposing.</PackageReleaseNotes>
     <Title>ProgressOnderwijsUtils</Title>
     <Description>Collection of utilities developed by ProgressOnderwijs</Description>
     <PackageTags>ProgressOnderwijs</PackageTags>