Decrease metadata dowload amount

If there are no Script, Translations, Licenses nor UserInterfaces
-tags in Updates.xml, there is no metadata. In such case, do not download
the empty metadata to decrease the time used in downloading metadata.

Task-number: QTIFW-975
Change-Id: I1b81a07b5ad43677e190fb058ece8b06382d215b
Reviewed-by: Iikka Eklund <iikka.eklund@qt.io>

1 files changed, 26 insertions, 12 deletions

diff --git a/src/libs/installer/metadatajob.cpp b/src/libs/installer/metadatajob.cpp
index bd577fecf..f56eb43a3 100644
--- a/src/libs/installer/metadatajob.cpp
+++ b/src/libs/installer/metadatajob.cpp
@@ -39,6 +39,8 @@
 #include <QTemporaryDir>
 #include <QtMath>
 
+const QStringList metaElements = {QLatin1String("Script"), QLatin1String("Licenses"), QLatin1String("UserInterfaces"), QLatin1String("Translations")};
+
 namespace QInstaller {
 
 static QUrl resolveUrl(const FileTaskResult &result, const QString &url)
@@ -534,6 +536,7 @@ MetadataJob::Status MetadataJob::parseUpdatesXml(const QList<FileTaskResult> &re
             if (!el.isNull() && el.tagName() == QLatin1String("PackageUpdate")) {
                 const QDomNodeList c2 = el.childNodes();
                 QString packageName, packageVersion, packageHash;
+                bool metaFound = false;
                 for (int j = 0; j < c2.count(); ++j) {
                     if (c2.at(j).toElement().tagName() == scName)
                         packageName = c2.at(j).toElement().text();
@@ -541,21 +544,32 @@ MetadataJob::Status MetadataJob::parseUpdatesXml(const QList<FileTaskResult> &re
                         packageVersion = (online ? c2.at(j).toElement().text() : QString());
                     else if ((c2.at(j).toElement().tagName() == QLatin1String("SHA1")) && testCheckSum)
                         packageHash = c2.at(j).toElement().text();
+                    else {
+                        foreach (QString meta, metaElements) {
+                            if (c2.at(j).toElement().tagName() == meta) {
+                                metaFound = true;
+                                break;
+                            }
+                        }
+                    }
                 }
 
                 const QString repoUrl = metadata.repository.url().toString();
-                FileTaskItem item(QString::fromLatin1("%1/%2/%3meta.7z").arg(repoUrl, packageName,
-                    packageVersion), metadata.directory + QString::fromLatin1("/%1-%2-meta.7z")
-                    .arg(packageName, packageVersion));
-
-                QAuthenticator authenticator;
-                authenticator.setUser(metadata.repository.username());
-                authenticator.setPassword(metadata.repository.password());
-
-                item.insert(TaskRole::UserRole, metadata.directory);
-                item.insert(TaskRole::Checksum, packageHash.toLatin1());
-                item.insert(TaskRole::Authenticator, QVariant::fromValue(authenticator));
-                m_packages.append(item);
+                //If script element is not found, no need to fetch metadata
+                if (metaFound) {
+                    FileTaskItem item(QString::fromLatin1("%1/%2/%3meta.7z").arg(repoUrl, packageName,
+                        packageVersion), metadata.directory + QString::fromLatin1("/%1-%2-meta.7z")
+                        .arg(packageName, packageVersion));
+
+                    QAuthenticator authenticator;
+                    authenticator.setUser(metadata.repository.username());
+                    authenticator.setPassword(metadata.repository.password());
+
+                    item.insert(TaskRole::UserRole, metadata.directory);
+                    item.insert(TaskRole::Checksum, packageHash.toLatin1());
+                    item.insert(TaskRole::Authenticator, QVariant::fromValue(authenticator));
+                    m_packages.append(item);
+                }
             }
         }
         m_metadata.insert(metadata.directory, metadata);