Merge branch 'main' into copilot/update-regex-for-unicode-support

MrHinsh · web-flow · commit 24c2d8aea475 · 2025-10-09T15:24:58.000Z
diff --git a/src/MigrationTools.Clients.TfsObjectModel/Tools/TfsWorkItemTypeValidatorTool.cs b/src/MigrationTools.Clients.TfsObjectModel/Tools/TfsWorkItemTypeValidatorTool.cs
@@ -370,7 +370,14 @@ private bool ValidateAllowedValues(
                 {
                     if (valueMaps.TryGetValue(missingValue, out string mappedValue))
                     {
-                        if (targetAllowedValues.Contains(mappedValue, StringComparer.OrdinalIgnoreCase))
+                        if (string.IsNullOrWhiteSpace(mappedValue))
+                        {
+                            mappedValues.Add(missingValue);
+                            Log.LogInformation("      Value '{missingValue}' is mapped to empty string ('{mappedValue}')."
+                                + " This is not checked in target and considered valid.",
+                                missingValue, mappedValue);
+                        }
+                        else if (targetAllowedValues.Contains(mappedValue, StringComparer.OrdinalIgnoreCase))
                         {
                             mappedValues.Add(missingValue);
                             Log.LogDebug("      Value '{missingValue}' is mapped to '{mappedValue}', which exists in target.",

Original file line number	Diff line number	Diff line change
`@@ -370,7 +370,14 @@ private bool ValidateAllowedValues(`
`370`	`370`	`{`
`371`	`371`	`if (valueMaps.TryGetValue(missingValue, out string mappedValue))`
`372`	`372`	`{`
`373`		`- if (targetAllowedValues.Contains(mappedValue, StringComparer.OrdinalIgnoreCase))`
	`373`	`+ if (string.IsNullOrWhiteSpace(mappedValue))`
	`374`	`+ {`
	`375`	`+ mappedValues.Add(missingValue);`
	`376`	`+ Log.LogInformation(" Value '{missingValue}' is mapped to empty string ('{mappedValue}')."`
	`377`	`+ + " This is not checked in target and considered valid.",`
	`378`	`+ missingValue, mappedValue);`
	`379`	`+ }`
	`380`	`+ else if (targetAllowedValues.Contains(mappedValue, StringComparer.OrdinalIgnoreCase))`
`374`	`381`	`{`
`375`	`382`	`mappedValues.Add(missingValue);`
`376`	`383`	`Log.LogDebug(" Value '{missingValue}' is mapped to '{mappedValue}', which exists in target.",`