Persist app zone history in real time (microsoft#1257)

* save settings every time changes are performed * settings are not saved on app exit
udit3333 · Feb 20, 2020 · 4df0d0c · 4df0d0c
1 parent a457899
commit 4df0d0c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/modules/fancyzones/dll/dllmain.cpp b/src/modules/fancyzones/dll/dllmain.cpp
@@ -136,8 +136,6 @@ class FancyZonesModule : public PowertoyModuleIface
     void Disable(bool const traceEvent)
     {
         if (m_app) {
-            const auto& fancyZonesData = JSONHelpers::FancyZonesDataInstance();
-            fancyZonesData.SaveFancyZonesData();
             if (traceEvent) 
             {
                 Trace::FancyZones::EnableFancyZones(false);

diff --git a/src/modules/fancyzones/lib/JsonHelpers.cpp b/src/modules/fancyzones/lib/JsonHelpers.cpp
@@ -207,6 +207,7 @@ namespace JSONHelpers
                 if (data.zoneSetUuid == zoneSetId && data.deviceId == deviceId)
                 {
                     appZoneHistoryMap.erase(processPath);
+                    SaveFancyZonesData();
                     return true;
                 }
             }
@@ -224,6 +225,7 @@ namespace JSONHelpers
         }
 
         appZoneHistoryMap[processPath] = AppZoneHistoryData{ .zoneSetUuid = zoneSetId, .deviceId = deviceId, .zoneIndex = zoneIndex };
+        SaveFancyZonesData();
         return true;
     }
 
@@ -538,6 +540,7 @@ namespace JSONHelpers
         if (appliedZoneSetsMap.contains(std::wstring{ activeZoneSetId }))
         {
             deviceInfoMap[deviceId] = DeviceInfoData{ appliedZoneSetsMap.at(std::wstring{ activeZoneSetId }), static_cast<bool>(showSpacing), static_cast<int>(spacing), static_cast<int>(zoneCount) };
+            SaveFancyZonesData();
         }
     }