Merge branch 'main' into spelling-mistakes-from-curtmgray

MrHinsh · web-flow · commit 288b70525c9d · 2025-03-07T08:55:54.000Z
diff --git a/src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs b/src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs
@@ -895,6 +895,12 @@ private void CheckClosedDateIsValid(WorkItemData sourceWorkItem, WorkItemData ta
             {
                 closedDateField = "Microsoft.VSTS.Common.ClosedDate";
             }
+            else if (!targetWorkItem.ToWorkItem().Fields.Contains("System.ClosedDate"))
+            {
+                Log.LogDebug("CheckClosedDateIsValid::ClosedDate field doesn't exist in targetWorkItem: {targetWorkItem} - nothing to validate.", targetWorkItem);
+                return;
+            }
+
             Log.LogDebug("CheckClosedDateIsValid::ClosedDate field is {closedDateField}", closedDateField);
             if (targetWorkItem.ToWorkItem().Fields[closedDateField].Value == null && (targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Closed" || targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Done"))
             {
@@ -921,7 +927,6 @@ private void CheckClosedDateIsValid(WorkItemData sourceWorkItem, WorkItemData ta
                     Log.LogWarning("Target ClosedDate Field: ", "System.ClosedDate");
                 }
             }
-
         }
 
         private bool SkipRevisionWithInvalidIterationPath(WorkItemData targetWorkItemData)

Original file line number	Diff line number	Diff line change
`@@ -895,6 +895,12 @@ private void CheckClosedDateIsValid(WorkItemData sourceWorkItem, WorkItemData ta`
`895`	`895`	`{`
`896`	`896`	`closedDateField = "Microsoft.VSTS.Common.ClosedDate";`
`897`	`897`	`}`
	`898`	`+ else if (!targetWorkItem.ToWorkItem().Fields.Contains("System.ClosedDate"))`
	`899`	`+ {`
	`900`	`+ Log.LogDebug("CheckClosedDateIsValid::ClosedDate field doesn't exist in targetWorkItem: {targetWorkItem} - nothing to validate.", targetWorkItem);`
	`901`	`+ return;`
	`902`	`+ }`
	`903`	`+`
`898`	`904`	`Log.LogDebug("CheckClosedDateIsValid::ClosedDate field is {closedDateField}", closedDateField);`
`899`	`905`	`if (targetWorkItem.ToWorkItem().Fields[closedDateField].Value == null && (targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Closed" \|\| targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Done"))`
`900`	`906`	`{`
`@@ -921,7 +927,6 @@ private void CheckClosedDateIsValid(WorkItemData sourceWorkItem, WorkItemData ta`
`921`	`927`	`Log.LogWarning("Target ClosedDate Field: ", "System.ClosedDate");`
`922`	`928`	`}`
`923`	`929`	`}`
`924`		`-`
`925`	`930`	`}`
`926`	`931`
`927`	`932`	`private bool SkipRevisionWithInvalidIterationPath(WorkItemData targetWorkItemData)`