fix: add missing null check on relayCommandName

davidkallesen · davidkallesen · commit fc27d576ecb7 · 2025-02-16T23:47:52.000+01:00
diff --git a/src/Atc.Wpf.SourceGenerators/Extensions/StringExtensions.cs b/src/Atc.Wpf.SourceGenerators/Extensions/StringExtensions.cs
@@ -172,6 +172,11 @@ public static string StripPrefixFromField(
     public static string EnsureValidRelayCommandName(
         this string relayCommandName)
     {
+        if (relayCommandName is null)
+        {
+            throw new ArgumentNullException(nameof(relayCommandName));
+        }
+
         if (relayCommandName.EndsWith(NameConstants.Command + NameConstants.Command, StringComparison.Ordinal))
         {
             relayCommandName = relayCommandName.Substring(0, relayCommandName.Length - NameConstants.Command.Length);

Original file line number	Diff line number	Diff line change
`@@ -172,6 +172,11 @@ public static string StripPrefixFromField(`
`172`	`172`	`public static string EnsureValidRelayCommandName(`
`173`	`173`	`this string relayCommandName)`
`174`	`174`	`{`
	`175`	`+ if (relayCommandName is null)`
	`176`	`+ {`
	`177`	`+ throw new ArgumentNullException(nameof(relayCommandName));`
	`178`	`+ }`
	`179`	`+`
`175`	`180`	`if (relayCommandName.EndsWith(NameConstants.Command + NameConstants.Command, StringComparison.Ordinal))`
`176`	`181`	`{`
`177`	`182`	`relayCommandName = relayCommandName.Substring(0, relayCommandName.Length - NameConstants.Command.Length);`