aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--core/pref.h1
-rw-r--r--core/settings/qPrefUpdateManager.cpp14
-rw-r--r--core/settings/qPrefUpdateManager.h4
-rw-r--r--tests/testqPrefUpdateManager.cpp39
4 files changed, 8 insertions, 50 deletions
diff --git a/core/pref.h b/core/pref.h
index 308aa9555..410280a10 100644
--- a/core/pref.h
+++ b/core/pref.h
@@ -47,7 +47,6 @@ enum def_file_behavior {
typedef struct {
bool dont_check_for_updates;
- bool dont_check_exists;
const char *last_version_used;
int next_check;
} update_manager_prefs_t;
diff --git a/core/settings/qPrefUpdateManager.cpp b/core/settings/qPrefUpdateManager.cpp
index 34b843d08..deee1c890 100644
--- a/core/settings/qPrefUpdateManager.cpp
+++ b/core/settings/qPrefUpdateManager.cpp
@@ -14,7 +14,6 @@ qPrefUpdateManager *qPrefUpdateManager::instance()
return self;
}
-
void qPrefUpdateManager::loadSync(bool doSync)
{
disk_dont_check_for_updates(doSync);
@@ -25,22 +24,9 @@ void qPrefUpdateManager::loadSync(bool doSync)
}
}
-
HANDLE_PREFERENCE_BOOL_EXT(UpdateManager, "DontCheckForUpdates", dont_check_for_updates, update_manager.);
-
-void qPrefUpdateManager::set_dont_check_exists(bool value)
-{
- if (value != prefs.update_manager.dont_check_exists) {
- prefs.update_manager.dont_check_exists = value;
- emit instance()->dont_check_existsChanged(value);
- }
- // DO NOT STORE ON DISK
-}
-
-
HANDLE_PREFERENCE_TXT_EXT(UpdateManager, "LastVersionUsed", last_version_used, update_manager.);
-
void qPrefUpdateManager::set_next_check(const QDate& value)
{
long time_value = value.toJulianDay();
diff --git a/core/settings/qPrefUpdateManager.h b/core/settings/qPrefUpdateManager.h
index ff5f3b53f..15bcd3c6e 100644
--- a/core/settings/qPrefUpdateManager.h
+++ b/core/settings/qPrefUpdateManager.h
@@ -9,7 +9,6 @@
class qPrefUpdateManager : public QObject {
Q_OBJECT
Q_PROPERTY(bool dont_check_for_updates READ dont_check_for_updates WRITE set_dont_check_for_updates NOTIFY dont_check_for_updatesChanged)
- Q_PROPERTY(bool dont_check_exists READ dont_check_exists WRITE set_dont_check_exists NOTIFY dont_check_existsChanged)
Q_PROPERTY(const QString last_version_used READ last_version_used WRITE set_last_version_used NOTIFY last_version_usedChanged)
Q_PROPERTY(const QDate next_check READ next_check WRITE set_next_check NOTIFY next_checkChanged)
Q_PROPERTY(const QString uuidString READ uuidString WRITE set_uuidString NOTIFY uuidStringChanged)
@@ -24,21 +23,18 @@ public:
public:
static bool dont_check_for_updates() { return prefs.update_manager.dont_check_for_updates; }
- static bool dont_check_exists() { return prefs.update_manager.dont_check_exists; }
static const QString last_version_used() { return prefs.update_manager.last_version_used; }
static const QDate next_check() { return QDate::fromJulianDay(prefs.update_manager.next_check); }
static const QString uuidString() { return st_uuidString; }
public slots:
static void set_dont_check_for_updates(bool value);
- static void set_dont_check_exists(bool value);
static void set_last_version_used(const QString& value);
static void set_next_check(const QDate& value);
static void set_uuidString(const QString& value);
signals:
void dont_check_for_updatesChanged(bool value);
- void dont_check_existsChanged(bool value);
void last_version_usedChanged(const QString& value);
void next_checkChanged(const QDate& value);
void uuidStringChanged(const QString& value);
diff --git a/tests/testqPrefUpdateManager.cpp b/tests/testqPrefUpdateManager.cpp
index 8e3047fdb..dae0de555 100644
--- a/tests/testqPrefUpdateManager.cpp
+++ b/tests/testqPrefUpdateManager.cpp
@@ -22,14 +22,12 @@ void TestQPrefUpdateManager::test_struct_get()
auto tst = qPrefUpdateManager::instance();
prefs.update_manager.dont_check_for_updates = true;
- prefs.update_manager.dont_check_exists = true;
prefs.update_manager.last_version_used = copy_qstring("last_version");
prefs.update_manager.next_check = QDate::fromString("11/09/1957", "dd/MM/yyyy").toJulianDay();
QCOMPARE(tst->dont_check_for_updates(), true);
- QCOMPARE(tst->dont_check_exists(), true);
QCOMPARE(tst->last_version_used(), QString("last_version"));
- QCOMPARE(tst->next_check(), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
+ QCOMPARE(tst->next_check(), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
}
void TestQPrefUpdateManager::test_set_struct()
@@ -39,15 +37,13 @@ void TestQPrefUpdateManager::test_set_struct()
auto tst = qPrefUpdateManager::instance();
tst->set_dont_check_for_updates(false);
- tst->set_dont_check_exists(false);
tst->set_last_version_used("last_version2");
- tst->set_next_check(QDate::fromString("11/09/1957", "dd/MM/yyyy"));
+ tst->set_next_check(QDate::fromString("11/09/1957", "dd/MM/yyyy"));
tst->set_uuidString("uuid");
QCOMPARE(prefs.update_manager.dont_check_for_updates, false);
- QCOMPARE(prefs.update_manager.dont_check_exists, false);
QCOMPARE(QString(prefs.update_manager.last_version_used), QString("last_version2"));
- QCOMPARE(QDate::fromJulianDay(prefs.update_manager.next_check), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
+ QCOMPARE(QDate::fromJulianDay(prefs.update_manager.next_check), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
QCOMPARE(tst->uuidString(), QString("uuid"));
}
@@ -59,28 +55,22 @@ void TestQPrefUpdateManager::test_set_load_struct()
// secure set_ stores on disk
prefs.update_manager.dont_check_for_updates = true;
- prefs.update_manager.dont_check_exists = true;
prefs.update_manager.next_check = 100;
tst->set_dont_check_for_updates(false);
- tst->set_dont_check_exists(false);
tst->set_last_version_used("last_version2");
- tst->set_next_check(QDate::fromString("11/09/1957", "dd/MM/yyyy"));
+ tst->set_next_check(QDate::fromString("11/09/1957", "dd/MM/yyyy"));
tst->set_uuidString("uuid2");
prefs.update_manager.dont_check_for_updates = true;
- prefs.update_manager.dont_check_exists = true;
prefs.update_manager.last_version_used = copy_qstring("last_version");
prefs.update_manager.next_check = 1000;
tst->load();
QCOMPARE(prefs.update_manager.dont_check_for_updates, false);
QCOMPARE(QString(prefs.update_manager.last_version_used), QString("last_version2"));
- QCOMPARE(QDate::fromJulianDay(prefs.update_manager.next_check), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
+ QCOMPARE(QDate::fromJulianDay(prefs.update_manager.next_check), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
QCOMPARE(tst->uuidString(), QString("uuid2"));
-
- // dont_check_exists is NOT stored on disk
- QCOMPARE(prefs.update_manager.dont_check_exists, true);
}
void TestQPrefUpdateManager::test_struct_disk()
@@ -90,13 +80,11 @@ void TestQPrefUpdateManager::test_struct_disk()
auto tst = qPrefUpdateManager::instance();
prefs.update_manager.dont_check_for_updates = true;
- prefs.update_manager.dont_check_exists = true;
prefs.update_manager.last_version_used = copy_qstring("last_version");
prefs.update_manager.next_check = QDate::fromString("11/09/1957", "dd/MM/yyyy").toJulianDay();
tst->sync();
prefs.update_manager.dont_check_for_updates = false;
- prefs.update_manager.dont_check_exists = false;
prefs.update_manager.last_version_used = copy_qstring("");
prefs.update_manager.next_check = 1000;
@@ -104,9 +92,6 @@ void TestQPrefUpdateManager::test_struct_disk()
QCOMPARE(tst->dont_check_for_updates(), true);
QCOMPARE(tst->last_version_used(), QString("last_version"));
QCOMPARE(tst->next_check(), QDate::fromString("11/09/1957", "dd/MM/yyyy"));
-
- // dont_check_exists is NOT stored on disk
- QCOMPARE(tst->dont_check_exists(), false);
}
void TestQPrefUpdateManager::test_multiple()
@@ -115,13 +100,10 @@ void TestQPrefUpdateManager::test_multiple()
prefs.update_manager.dont_check_for_updates = false;
- prefs.update_manager.dont_check_exists = false;
auto tst = qPrefUpdateManager::instance();
QCOMPARE(tst->dont_check_for_updates(), qPrefUpdateManager::dont_check_for_updates());
QCOMPARE(tst->dont_check_for_updates(), false);
- QCOMPARE(tst->dont_check_exists(), qPrefUpdateManager::dont_check_exists());
- QCOMPARE(qPrefUpdateManager::dont_check_exists(), false);
}
void TestQPrefUpdateManager::test_next_check()
@@ -131,7 +113,7 @@ void TestQPrefUpdateManager::test_next_check()
prefs.update_manager.next_check = QDate::fromString("11/09/1957", "dd/MM/yyyy").toJulianDay();
prefs.update_manager.next_check++;
- QCOMPARE(tst->next_check(), QDate::fromString("12/09/1957", "dd/MM/yyyy"));
+ QCOMPARE(tst->next_check(), QDate::fromString("12/09/1957", "dd/MM/yyyy"));
}
#define TEST(METHOD, VALUE) \
@@ -166,29 +148,24 @@ void TestQPrefUpdateManager::test_oldPreferences()
void TestQPrefUpdateManager::test_signals()
{
QSignalSpy spy1(qPrefUpdateManager::instance(), &qPrefUpdateManager::dont_check_for_updatesChanged);
- QSignalSpy spy2(qPrefUpdateManager::instance(), &qPrefUpdateManager::dont_check_existsChanged);
QSignalSpy spy3(qPrefUpdateManager::instance(), &qPrefUpdateManager::last_version_usedChanged);
QSignalSpy spy4(qPrefUpdateManager::instance(), &qPrefUpdateManager::next_checkChanged);
QSignalSpy spy5(qPrefUpdateManager::instance(), &qPrefUpdateManager::uuidStringChanged);
prefs.update_manager.dont_check_for_updates = true;
qPrefUpdateManager::set_dont_check_for_updates(false);
- prefs.update_manager.dont_check_exists = true;
- qPrefUpdateManager::set_dont_check_exists(false);
qPrefUpdateManager::set_last_version_used("signal last_version2");
- qPrefUpdateManager::set_next_check(QDate::fromString("11/09/1967", "dd/MM/yyyy"));
+ qPrefUpdateManager::set_next_check(QDate::fromString("11/09/1967", "dd/MM/yyyy"));
qPrefUpdateManager::set_uuidString("signal uuid");
QCOMPARE(spy1.count(), 1);
- QCOMPARE(spy2.count(), 1);
QCOMPARE(spy3.count(), 1);
QCOMPARE(spy4.count(), 1);
QCOMPARE(spy5.count(), 1);
QVERIFY(spy1.takeFirst().at(0).toBool() == false);
- QVERIFY(spy2.takeFirst().at(0).toBool() == false);
QVERIFY(spy3.takeFirst().at(0).toString() == "signal last_version2");
- QVERIFY(spy4.takeFirst().at(0).toDate() == QDate::fromString("11/09/1967", "dd/MM/yyyy"));
+ QVERIFY(spy4.takeFirst().at(0).toDate() == QDate::fromString("11/09/1967", "dd/MM/yyyy"));
QVERIFY(spy5.takeFirst().at(0).toString() == "signal uuid");
}
generated by cgit v1.2.3-70-g09d2 (git 2.47.0) at 2024-12-25 04:23:46 +0000