Merge pull request #7 from typinghard/feature/43

DiegoGalante · web-flow · commit 7145f33bd69e · 2022-06-27T19:07:59.000-03:00
Validate if State Field exists before add Workitem Change
diff --git a/AzureDevopsTracker/Services/AzureDevopsTrackerService.cs b/AzureDevopsTracker/Services/AzureDevopsTrackerService.cs
@@ -242,6 +242,9 @@ public void AddWorkItemChange(WorkItem workItem, CreateWorkItemDTO create)
 
         public void AddWorkItemChange(WorkItem workItem, UpdatedWorkItemDTO update)
         {
+            if (update.Resource.Fields.State is null) return;
+            if (update.Resource.Fields.StateChangeDate is null) return;
+
             var changedBy = update.Resource.Revision.Fields.ChangedBy ?? update.Resource.Fields.ChangedBy.NewValue;
             var workItemChange = ToWorkItemChange(workItem.Id,
                                       changedBy,

Original file line number	Diff line number	Diff line change
`@@ -242,6 +242,9 @@ public void AddWorkItemChange(WorkItem workItem, CreateWorkItemDTO create)`
`242`	`242`
`243`	`243`	`public void AddWorkItemChange(WorkItem workItem, UpdatedWorkItemDTO update)`
`244`	`244`	`{`
	`245`	`+ if (update.Resource.Fields.State is null) return;`
	`246`	`+ if (update.Resource.Fields.StateChangeDate is null) return;`
	`247`	`+`
`245`	`248`	`var changedBy = update.Resource.Revision.Fields.ChangedBy ?? update.Resource.Fields.ChangedBy.NewValue;`
`246`	`249`	`var workItemChange = ToWorkItemChange(workItem.Id,`
`247`	`250`	`changedBy,`