Settings update: Fix broken UpdateManger logic

In commit b76c1846bbfd ("Settings update: Simplify Update Manager")
the logic for when to show the UpdateManger question to the user got
broken. Unintuitively, a boolean setting actually has three possible
values. True, False, and Unset. This patch fixes things to work as
designed again.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 7 insertions, 0 deletions

diff --git a/core/subsurface-qt/SettingsObjectWrapper.cpp b/core/subsurface-qt/SettingsObjectWrapper.cpp
index 16e7ab7f9..e71e1b1e8 100644
--- a/core/subsurface-qt/SettingsObjectWrapper.cpp
+++ b/core/subsurface-qt/SettingsObjectWrapper.cpp
@@ -78,6 +78,11 @@ bool UpdateManagerSettings::dontCheckForUpdates() const
 	return prefs.update_manager.dont_check_for_updates;
 }
 
+bool UpdateManagerSettings::dontCheckExists() const
+{
+	return prefs.update_manager.dont_check_exists;
+}
+
 QString UpdateManagerSettings::lastVersionUsed() const
 {
 	return prefs.update_manager.last_version_used;
@@ -94,6 +99,7 @@ void UpdateManagerSettings::setDontCheckForUpdates(bool value)
 	s.beginGroup(group);
 	s.setValue("DontCheckForUpdates", value);
 	prefs.update_manager.dont_check_for_updates = value;
+	prefs.update_manager.dont_check_exists = true;
 	emit dontCheckForUpdatesChanged(value);
 }
 
@@ -1934,6 +1940,7 @@ void SettingsObjectWrapper::load()
 	s.endGroup();
 
 	s.beginGroup("UpdateManager");
+	prefs.update_manager.dont_check_exists = s.contains("DontCheckForUpdates");
 	prefs.update_manager.dont_check_for_updates = s.value("DontCheckForUpdates").toBool();
 	prefs.update_manager.last_version_used = copy_string(qPrintable(s.value("LastVersionUsed").toString()));
 	prefs.update_manager.next_check = copy_string(qPrintable(s.value("NextCheck").toString()));