3 files changed, 5 insertions, 100 deletions

diff --git a/core/settings/qPref.cpp b/core/settings/qPref.cpp
index ede230236..c5f2e663c 100644
--- a/core/settings/qPref.cpp
+++ b/core/settings/qPref.cpp
@@ -22,6 +22,7 @@ void qPref::loadSync(bool doSync)
 	// qPrefFaceook does not use disk.
 	qPrefProxy::instance()->loadSync(doSync);
 	qPrefUnits::instance()->loadSync(doSync);
+	qPrefUpdateManager::instance()->loadSync(doSync);
 }
 
 const QString qPref::canonical_version() const
diff --git a/core/subsurface-qt/SettingsObjectWrapper.cpp b/core/subsurface-qt/SettingsObjectWrapper.cpp
index c31d3831b..16a354dda 100644
--- a/core/subsurface-qt/SettingsObjectWrapper.cpp
+++ b/core/subsurface-qt/SettingsObjectWrapper.cpp
@@ -8,71 +8,6 @@
 #include "core/qthelper.h"
 #include "core/prefs-macros.h"
 
-UpdateManagerSettings::UpdateManagerSettings(QObject *parent) : QObject(parent)
-{
-
-}
-
-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;
-}
-
-QDate UpdateManagerSettings::nextCheck() const
-{
-	return QDate::fromString(QString(prefs.update_manager.next_check), "dd/MM/yyyy");
-}
-
-void UpdateManagerSettings::setDontCheckForUpdates(bool value)
-{
-	if (value == prefs.update_manager.dont_check_for_updates)
-		return;
-
-	QSettings s;
-	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);
-}
-
-void UpdateManagerSettings::setLastVersionUsed(const QString& value)
-{
-	if (value == prefs.update_manager.last_version_used)
-		return;
-
-	QSettings s;
-	s.beginGroup(group);
-	s.setValue("LastVersionUsed", value);
-	free((void *)prefs.update_manager.last_version_used);
-	prefs.update_manager.last_version_used = copy_qstring(value);
-	emit lastVersionUsedChanged(value);
-}
-
-void UpdateManagerSettings::setNextCheck(const QDate& date)
-{
-	if (date.toString() == prefs.update_manager.next_check)
-		return;
-
-	QSettings s;
-	s.beginGroup(group);
-	s.setValue("NextCheck", date);
-	free((void *)prefs.update_manager.next_check);
-	prefs.update_manager.next_check = copy_qstring(date.toString("dd/MM/yyyy"));
-	emit nextCheckChanged(date);
-}
-
-
 PartialPressureGasSettings::PartialPressureGasSettings(QObject* parent):
 	QObject(parent)
 {
@@ -1112,7 +1047,7 @@ QObject(parent),
 	language_settings(new LanguageSettingsObjectWrapper(this)),
 	animation_settings(new qPrefAnimations(this)),
 	location_settings(new LocationServiceSettingsObjectWrapper(this)),
-	update_manager_settings(new UpdateManagerSettings(this)),
+	update_manager_settings(new qPrefUpdateManager(this)),
 	dive_computer_settings(new qPrefDiveComputer(this))
 {
 }
@@ -1205,13 +1140,7 @@ void SettingsObjectWrapper::load()
 
 	qPrefDivePlanner::instance()->load();
 	qPrefDiveComputer::instance()->load();
-
-	s.beginGroup("UpdateManager");
-	prefs.update_manager.dont_check_exists = s.contains("DontCheckForUpdates");
-	GET_BOOL("DontCheckForUpdates", update_manager.dont_check_for_updates);
-	GET_TXT("LastVersionUsed", update_manager.last_version_used);
-	prefs.update_manager.next_check = copy_qstring(s.value("NextCheck").toDate().toString("dd/MM/yyyy"));
-	s.endGroup();
+	qPrefUpdateManager::instance()->load();
 
 	s.beginGroup("Language");
 	GET_BOOL("UseSystemLanguage", locale.use_system_language);
diff --git a/core/subsurface-qt/SettingsObjectWrapper.h b/core/subsurface-qt/SettingsObjectWrapper.h
index 54add731d..a648658f1 100644
--- a/core/subsurface-qt/SettingsObjectWrapper.h
+++ b/core/subsurface-qt/SettingsObjectWrapper.h
@@ -13,31 +13,6 @@
  * and QWidget frontends. This class will be huge, since
  * I need tons of properties, one for each option. */
 
-class UpdateManagerSettings : public QObject {
-	Q_OBJECT
-	Q_PROPERTY(bool dont_check_for_updates READ dontCheckForUpdates WRITE setDontCheckForUpdates NOTIFY dontCheckForUpdatesChanged)
-	Q_PROPERTY(QString last_version_used READ lastVersionUsed WRITE setLastVersionUsed NOTIFY lastVersionUsedChanged)
-	Q_PROPERTY(QDate next_check READ nextCheck WRITE nextCheckChanged)
-public:
-	UpdateManagerSettings(QObject *parent);
-	bool dontCheckForUpdates() const;
-	bool dontCheckExists() const;
-	QString lastVersionUsed() const;
-	QDate nextCheck() const;
-
-public slots:
-	void setDontCheckForUpdates(bool value);
-	void setLastVersionUsed(const QString& value);
-	void setNextCheck(const QDate& date);
-
-signals:
-	void dontCheckForUpdatesChanged(bool value);
-	void lastVersionUsedChanged(const QString& value);
-	void nextCheckChanged(const QDate& date);
-private:
-	const QString group = QStringLiteral("UpdateManager");
-};
-
 /* Control the state of the Partial Pressure Graphs preferences */
 class PartialPressureGasSettings : public QObject {
 	Q_OBJECT
@@ -366,7 +341,7 @@ class SettingsObjectWrapper : public QObject {
 	Q_PROPERTY(qPrefAnimations*      animation MEMBER animation_settings CONSTANT)
 	Q_PROPERTY(LocationServiceSettingsObjectWrapper* Location  MEMBER location_settings CONSTANT)
 
-	Q_PROPERTY(UpdateManagerSettings* update MEMBER update_manager_settings CONSTANT)
+	Q_PROPERTY(qPrefUpdateManager* update MEMBER update_manager_settings CONSTANT)
 	Q_PROPERTY(qPrefDiveComputer* dive_computer MEMBER dive_computer_settings CONSTANT)
 public:
 	static SettingsObjectWrapper *instance();
@@ -384,7 +359,7 @@ public:
 	LanguageSettingsObjectWrapper *language_settings;
 	qPrefAnimations *animation_settings;
 	LocationServiceSettingsObjectWrapper *location_settings;
-	UpdateManagerSettings *update_manager_settings;
+	qPrefUpdateManager *update_manager_settings;
 	qPrefDiveComputer *dive_computer_settings;
 
 	void sync();