Merge branch 'hotfix/0.2.1' into stable

Author: trollixx

src/core/application.cpp

@@ -147,12 +147,14 @@ void Application::extract(const QString &filePath, const QString &destination, c
 
 QNetworkReply *Application::download(const QUrl &url)
 {
-    static const QString userAgent = userAgentJson();
+    static const QString ua = userAgent();
+    static const QString uaJson = userAgentJson();
 
     QNetworkRequest request(url);
+    request.setHeader(QNetworkRequest::UserAgentHeader, ua);
 
     if (url.host().endsWith(QLatin1String(".zealdocs.org", Qt::CaseInsensitive)))
-        request.setRawHeader("X-Zeal-User-Agent", userAgent.toUtf8());
+        request.setRawHeader("X-Zeal-User-Agent", uaJson.toUtf8());
 
     return m_networkManager->get(request);
 }
@@ -218,6 +220,10 @@ void Application::applySettings()
     }
 }
 
+QString Application::userAgent()
+{
+    return QString("Zeal/%1").arg(QCoreApplication::applicationVersion());
+}
 
 QString Application::userAgentJson() const
 {

src/core/application.h

@@ -74,6 +74,7 @@ private slots:
     void applySettings();
 
 private:
+    static inline QString userAgent();
     QString userAgentJson() const;
 
     static Application *m_instance;

src/core/settings.cpp

@@ -94,7 +94,8 @@ void Settings::load()
     m_settings->endGroup();
 
     m_settings->beginGroup(GroupProxy);
-    proxyType = m_settings->value(QStringLiteral("type"), ProxyType::System).value<ProxyType>();
+    proxyType = static_cast<ProxyType>(m_settings->value(QStringLiteral("type"),
+                                                         ProxyType::System).toUInt());
     proxyHost = m_settings->value(QStringLiteral("host")).toString();
     proxyPort = m_settings->value(QStringLiteral("port"), 0).toInt();
     proxyAuthenticate = m_settings->value(QStringLiteral("authenticate"), false).toBool();
@@ -126,7 +127,7 @@ void Settings::load()
                                   // Avoid curly braces (QTBUG-885)
                                   QUuid::createUuid().toString().mid(1, 36)).toString();
     version = m_settings->value(QStringLiteral("version"),
-                                  QCoreApplication::applicationVersion()).toString();
+                                QCoreApplication::applicationVersion()).toString();
     m_settings->endGroup();
 }
 
@@ -170,11 +171,10 @@ void Settings::save()
 
     m_settings->beginGroup(GroupInternal);
     m_settings->setValue(QStringLiteral("install_id"), installId);
-    m_settings->setValue(QStringLiteral("version"), version);
+    m_settings->setValue(QStringLiteral("version"), QCoreApplication::applicationVersion());
     m_settings->endGroup();
 
     m_settings->sync();
 
     emit updated();
 }
-

src/core/settings.h

@@ -58,7 +58,7 @@ class Settings : public QObject
     /// TODO: QString customCss;
 
     // Network
-    enum ProxyType {
+    enum ProxyType : unsigned int {
         None,
         System,
         UserDefined

src/registry/docset.cpp

@@ -105,8 +105,10 @@ Docset::Docset(const QString &path) :
     }
 
     /// TODO: Verify if this is needed
-    if (plist[InfoPlist::DashDocSetFamily].toString() == QLatin1String("cheatsheet"))
+    if (plist.contains(InfoPlist::DashDocSetFamily)
+            && plist[InfoPlist::DashDocSetFamily].toString() == QLatin1String("cheatsheet")) {
         m_name = m_name + QLatin1String("cheats");
+    }
 
     if (!dir.cd(QStringLiteral("Resources")) || !dir.exists(QStringLiteral("docSet.dsidx")))
         return;

src/src.pro

@@ -15,7 +15,7 @@ portable {
     DEFINES += PORTABLE_BUILD
 }
 
-VERSION = 0.2.0
+VERSION = 0.2.1
 DEFINES += ZEAL_VERSION=\\\"$${VERSION}\\\"
 
 HEADERS += \

src/ui/widgets/searchedit.cpp

@@ -140,7 +140,7 @@ void SearchEdit::showCompletions(const QString &newValue)
     const int frameWidth = style()->pixelMetric(QStyle::PM_DefaultFrameWidth);
     const int textWidth = fontMetrics().width(newValue);
 
-    if (!m_prefixCompleter)
+    if (m_prefixCompleter)
         m_prefixCompleter->setCompletionPrefix(text());
 
     const QString completed = currentCompletion(newValue).mid(newValue.size());