From 828d449aa44d5626c658d9cc82e89166a08679d6 Mon Sep 17 00:00:00 2001
From: sawinglogz <3787776-sawinglogz@users.noreply.gitlab.com>
Date: Thu, 26 Mar 2020 19:59:54 -0400
Subject: [PATCH] Update summary cache so that purging oximetry persists
 between launches.

---
 oscar/mainwindow.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/oscar/mainwindow.cpp b/oscar/mainwindow.cpp
index ed8e6def..53e8a30e 100644
--- a/oscar/mainwindow.cpp
+++ b/oscar/mainwindow.cpp
@@ -2587,16 +2587,19 @@ void MainWindow::on_actionPurgeCurrentDaysOximetry_triggered()
         }
 
         QList<Session *> sessionlist=day->getSessions(MT_OXIMETER);
+        QSet<Machine *> machines;
 
         for (auto & sess : sessionlist) {
+            machines += sess->machine();
             sess->Destroy();
             delete sess;
         }
-        // TODO: Fix this. It deletes the underlying session data file in the machine,
-        // but not from the machine's summary cache. This results in future launches
-        // of OSCAR thinking the day has oximetry data, but then it isn't really there.
-        // Currently this is only useful for reimporting a single day, which the purge
-        // permits, and which in turn creates a new data file for that day.
+        
+        // We have to update the summary cache for the affected machine(s),
+        // otherwise OSCAR will still think this day has oximetry data at next launch.
+        for (auto & mach : machines) {
+            mach->SaveSummaryCache();
+        }
 
 
         if (daily) {