From 7a71ecd8af0454e405b25080a4b266fc99306269 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= Date: Sat, 7 Feb 2015 12:55:18 +0100 Subject: NOISSUE fix notification checker --- logic/notifications/NotificationChecker.cpp | 55 +++++++++++++++++------------ 1 file changed, 32 insertions(+), 23 deletions(-) (limited to 'logic/notifications/NotificationChecker.cpp') diff --git a/logic/notifications/NotificationChecker.cpp b/logic/notifications/NotificationChecker.cpp index 3f8b17ba..77721770 100644 --- a/logic/notifications/NotificationChecker.cpp +++ b/logic/notifications/NotificationChecker.cpp @@ -3,28 +3,37 @@ #include #include #include +#include #include "logic/Env.h" -#include "BuildConfig.h" #include "logic/net/CacheDownload.h" -#include + NotificationChecker::NotificationChecker(QObject *parent) - : QObject(parent), m_notificationsUrl(QUrl(BuildConfig.NOTIFICATION_URL)) + : QObject(parent) { - // this will call checkForNotifications once the event loop is running - QMetaObject::invokeMethod(this, "checkForNotifications", Qt::QueuedConnection); } -QUrl NotificationChecker::notificationsUrl() const -{ - return m_notificationsUrl; -} void NotificationChecker::setNotificationsUrl(const QUrl ¬ificationsUrl) { m_notificationsUrl = notificationsUrl; } +void NotificationChecker::setApplicationChannel(QString channel) +{ + m_appVersionChannel = channel; +} + +void NotificationChecker::setApplicationFullVersion(QString version) +{ + m_appFullVersion = version; +} + +void NotificationChecker::setApplicationPlatform(QString platform) +{ + m_appPlatform = platform; +} + QList NotificationChecker::notificationEntries() const { return m_entries; @@ -83,7 +92,8 @@ void NotificationChecker::downloadSucceeded(int) { entry.type = NotificationEntry::Information; } - m_entries.append(entry); + if(entryApplies(entry)) + m_entries.append(entry); } } @@ -92,19 +102,7 @@ void NotificationChecker::downloadSucceeded(int) emit notificationCheckFinished(); } -bool NotificationChecker::NotificationEntry::applies() const -{ - bool channelApplies = channel.isEmpty() || channel == BuildConfig.VERSION_CHANNEL; - bool platformApplies = platform.isEmpty() || platform == BuildConfig.BUILD_PLATFORM; - bool fromApplies = - from.isEmpty() || from == BuildConfig.FULL_VERSION_STR || !versionLessThan(BuildConfig.FULL_VERSION_STR, from); - bool toApplies = - to.isEmpty() || to == BuildConfig.FULL_VERSION_STR || !versionLessThan(to, BuildConfig.FULL_VERSION_STR); - return channelApplies && platformApplies && fromApplies && toApplies; -} - -bool NotificationChecker::NotificationEntry::versionLessThan(const QString &v1, - const QString &v2) +bool versionLessThan(const QString &v1, const QString &v2) { QStringList l1 = v1.split('.'); QStringList l2 = v2.split('.'); @@ -119,3 +117,14 @@ bool NotificationChecker::NotificationEntry::versionLessThan(const QString &v1, } return false; } + +bool NotificationChecker::entryApplies(const NotificationChecker::NotificationEntry& entry) const +{ + bool channelApplies = entry.channel.isEmpty() || entry.channel == m_appVersionChannel; + bool platformApplies = entry.platform.isEmpty() || entry.platform == m_appPlatform; + bool fromApplies = + entry.from.isEmpty() || entry.from == m_appFullVersion || !versionLessThan(m_appFullVersion, entry.from); + bool toApplies = + entry.to.isEmpty() || entry.to == m_appFullVersion || !versionLessThan(entry.to, m_appFullVersion); + return channelApplies && platformApplies && fromApplies && toApplies; +} -- cgit v1.2.3