From 5384ac4c9f806cda2a13a7ea4bab2eb608696788 Mon Sep 17 00:00:00 2001
From: SwiftlyNoah <devnoahapps@gmail.com>
Date: Wed, 16 Aug 2023 12:48:36 -0400
Subject: [PATCH] Fix predicted BG graph only showing sometimes

---
 Loop/Managers/LoopDataManager.swift | 27 ++++++++++++++-------------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/Loop/Managers/LoopDataManager.swift b/Loop/Managers/LoopDataManager.swift
index 083c41e203..a97944e9ea 100644
--- a/Loop/Managers/LoopDataManager.swift
+++ b/Loop/Managers/LoopDataManager.swift
@@ -197,7 +197,7 @@ final class LoopDataManager {
                 self.dataAccessQueue.async {
                     self.logger.default("Received notification of dosing changing")
 
-                    self.insulinEffect = nil
+                    self.clearCachedInsulinEffects()
                     self.remoteRecommendationNeedsUpdating = true
 
                     self.notify(forChange: .insulin)
@@ -314,7 +314,7 @@ final class LoopDataManager {
                 // Invalidate cached effects based on this schedule
                 self.carbEffect = nil
                 self.carbsOnBoard = nil
-                self.insulinEffect = nil
+                self.clearCachedInsulinEffects()
             }
         }
 
@@ -339,12 +339,7 @@ final class LoopDataManager {
         }
     }
 
-    private var insulinEffect: [GlucoseEffect]? {
-        didSet {
-            insulinEffectIncludingPendingInsulin = nil
-            predictedGlucose = nil
-        }
-    }
+    private var insulinEffect: [GlucoseEffect]?
 
     private var insulinEffectIncludingPendingInsulin: [GlucoseEffect]? {
         didSet {
@@ -454,6 +449,12 @@ final class LoopDataManager {
         
         return cachedRetrospectiveCorrection!
     }
+    
+    func clearCachedInsulinEffects() {
+        insulinEffect = nil
+        insulinEffectIncludingPendingInsulin = nil
+        predictedGlucose = nil
+    }
 
     // MARK: - Background task management
 
@@ -717,7 +718,7 @@ extension LoopDataManager {
             self.logger.debug("bolusConfirmed")
             self.lastRequestedBolus = nil
             self.recommendedAutomaticDose = nil
-            self.insulinEffect = nil
+            self.clearCachedInsulinEffects()
             self.notify(forChange: .insulin)
 
             completion?()
@@ -733,7 +734,7 @@ extension LoopDataManager {
         self.dataAccessQueue.async {
             self.logger.debug("bolusRequestFailed")
             self.lastRequestedBolus = nil
-            self.insulinEffect = nil
+            self.clearCachedInsulinEffects()
             self.notify(forChange: .insulin)
 
             completion?()
@@ -754,7 +755,7 @@ extension LoopDataManager {
             
             self.dataAccessQueue.async {
                 if error == nil {
-                    self.insulinEffect = nil
+                    self.clearCachedInsulinEffects()
                 }
             }
         }
@@ -773,7 +774,7 @@ extension LoopDataManager {
         doseStore.addDoses([dose], from: nil) { (error) in
             if error == nil {
                 self.recommendedAutomaticDose = nil
-                self.insulinEffect = nil
+                self.clearCachedInsulinEffects()
                 self.notify(forChange: .insulin)
             }
         }
@@ -795,7 +796,7 @@ extension LoopDataManager {
                 completion(.failure(error))
             } else if let newValue = newValue {
                 self.dataAccessQueue.async {
-                    self.insulinEffect = nil
+                    self.clearCachedInsulinEffects()
 
                     if let newDoseStartDate = previousValue?.startDate {
                         // Prune back any counteraction effects for recomputation, after the effect delay