Skip to content

Commit e7076cf

Browse files
authored
Do not crash on CheckClosedDateIsValid if source has no closedDate
1 parent 00a2e81 commit e7076cf

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -901,7 +901,17 @@ private void CheckClosedDateIsValid(WorkItemData sourceWorkItem, WorkItemData ta
901901
if (targetWorkItem.ToWorkItem().Fields[closedDateField].Value == null && (targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Closed" || targetWorkItem.ToWorkItem().Fields["System.State"].Value.ToString() == "Done"))
902902
{
903903
Log.LogWarning("The field {closedDateField} is set to Null and will revert to the current date on save! ", closedDateField);
904-
Log.LogWarning("Source Closed Date [#{sourceId}][Rev{sourceRev}]: {sourceClosedDate} ", sourceWorkItem.ToWorkItem().Id, sourceWorkItem.ToWorkItem().Rev, sourceWorkItem.ToWorkItem().Fields[closedDateField].Value);
904+
if (sourceWorkItem.ToWorkItem().Fields.Contains(closedDateField))
905+
{
906+
Log.LogWarning("Source Closed Date [#{sourceId}][Rev{sourceRev}]: {sourceClosedDate} ",
907+
sourceWorkItem.ToWorkItem().Id, sourceWorkItem.ToWorkItem().Rev,
908+
sourceWorkItem.ToWorkItem().Fields[closedDateField].Value);
909+
}
910+
else
911+
{
912+
Log.LogWarning("Source Closed Date [#{sourceId}][Rev{sourceRev}] is not Available ",
913+
sourceWorkItem.ToWorkItem().Id, sourceWorkItem.ToWorkItem().Rev);
914+
}
905915
}
906916
if (!sourceWorkItem.ToWorkItem().Fields.Contains(closedDateField))
907917
{

0 commit comments

Comments
 (0)