Check if message box is null before loggign

jvyden · jvyden · commit 1d32279b387e · 2024-10-29T18:09:27.000-04:00
diff --git a/Refresher/UI/PipelineForm.cs b/Refresher/UI/PipelineForm.cs
@@ -505,6 +505,12 @@ private void OnLog(RefresherLog log)
     {
         Application.Instance.Invoke(() =>
         {
+            // somehow getting hit on sentry?
+            // id REFRESHER-88
+            // ReSharper disable once ConditionIsAlwaysTrueOrFalseAccordingToNullableAPIContract
+            if (this._messages == null)
+                return;
+            
             this._messages.Items.Add($"[{log.Level}] [{log.Category}] {log.Content}");
         
             // automatically scroll to the bottom by highlighting the last item temporarily

Original file line number	Diff line number	Diff line change
`@@ -505,6 +505,12 @@ private void OnLog(RefresherLog log)`
`505`	`505`	`{`
`506`	`506`	`Application.Instance.Invoke(() =>`
`507`	`507`	`{`
	`508`	`+ // somehow getting hit on sentry?`
	`509`	`+ // id REFRESHER-88`
	`510`	`+ // ReSharper disable once ConditionIsAlwaysTrueOrFalseAccordingToNullableAPIContract`
	`511`	`+ if (this._messages == null)`
	`512`	`+ return;`
	`513`	`+`
`508`	`514`	`this._messages.Items.Add($"[{log.Level}] [{log.Category}] {log.Content}");`
`509`	`515`
`510`	`516`	`// automatically scroll to the bottom by highlighting the last item temporarily`