Merge pull request #617 from ColinKinloch/wake_up_mode_setting_change

JF002 · web-flow · commit c9aeef94ea82 · 2021-09-12T18:41:26.000+02:00
Correct change test for wake up mode setting
diff --git a/src/components/settings/Settings.h b/src/components/settings/Settings.h
@@ -114,7 +114,7 @@ namespace Pinetime {
       };
 
       void setWakeUpMode(WakeUpMode wakeUp, bool enabled) {
-        if (!isWakeUpModeOn(wakeUp)) {
+        if (enabled != isWakeUpModeOn(wakeUp)) {
           settingsChanged = true;
         }
         settings.wakeUpMode.set(static_cast<size_t>(wakeUp), enabled);

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ namespace Pinetime {`
`114`	`114`	`};`
`115`	`115`
`116`	`116`	`void setWakeUpMode(WakeUpMode wakeUp, bool enabled) {`
`117`		`- if (!isWakeUpModeOn(wakeUp)) {`
	`117`	`+ if (enabled != isWakeUpModeOn(wakeUp)) {`
`118`	`118`	`settingsChanged = true;`
`119`	`119`	`}`
`120`	`120`	`settings.wakeUpMode.set(static_cast<size_t>(wakeUp), enabled);`