ZipIntaller: use file timestamp if version string is missing.

Instead of using some arbitrary and not really useful string use the timestamp
of the downloaded file if version string is provided by the caller. This also
makes it possible to check if the downloaded file is actually a different one.

Use it for manual and game file downloads, as this gives more reasonable values
than using the date transmitted for daily (archived) build and as support for
archived builds will be removed shortly.

Change-Id: I0c751fabe7bb516edca93a5f73f077a611d4ef87

4 files changed, 11 insertions, 5 deletions

diff --git a/rbutil/rbutilqt/base/zipinstaller.cpp b/rbutil/rbutilqt/base/zipinstaller.cpp
index 3f3179f..117fa2d 100644
--- a/rbutil/rbutilqt/base/zipinstaller.cpp
+++ b/rbutil/rbutilqt/base/zipinstaller.cpp
@@ -59,7 +59,7 @@ void ZipInstaller::installContinue()
         m_url = m_urllist.at(runner);
         m_logsection = m_loglist.at(runner);
         if(runner < m_verlist.size()) m_logver = m_verlist.at(runner);
-        else m_logver = "0";
+        else m_logver = "";
         installStart();
     }
     else {
@@ -176,6 +176,10 @@ void ZipInstaller::downloadDone(bool error)
         // add file to log
         zipContents.append(m_target);
     }
+    if(m_logver.isEmpty()) {
+        // if no version info is set use the timestamp of the server file.
+        m_logver = getter->timestamp().toString(Qt::ISODate);
+    }
 
     emit logItem(tr("Creating installation log"),LOGINFO);
     QSettings installlog(m_mountpoint + "/.rockbox/rbutil.log", QSettings::IniFormat, 0);
diff --git a/rbutil/rbutilqt/base/zipinstaller.h b/rbutil/rbutilqt/base/zipinstaller.h
index cb154c0..e15bbcf 100644
--- a/rbutil/rbutilqt/base/zipinstaller.h
+++ b/rbutil/rbutilqt/base/zipinstaller.h
@@ -41,8 +41,10 @@ public:
     void setUrl(QStringList url) { m_urllist = url; }
     void setLogSection(QString name) {m_loglist = QStringList(name);}
     void setLogSection(QStringList name) { m_loglist = name; }
-    void setLogVersion(QString v) { m_verlist = QStringList(v); qDebug() << m_verlist;}
-    void setLogVersion(QStringList v) { m_verlist = v; qDebug() << m_verlist;}
+    void setLogVersion(QString v = "")
+    { m_verlist = QStringList(v); qDebug() << m_verlist;}
+    void setLogVersion(QStringList v)
+    { m_verlist = v; qDebug() << m_verlist;}
     void setUnzip(bool i) { m_unzip = i; }
     void setTarget(QString t) { m_target = t; }
     void setCache(QDir c) { m_cache = c; m_usecache = true; };
diff --git a/rbutil/rbutilqt/gui/manualwidget.cpp b/rbutil/rbutilqt/gui/manualwidget.cpp
index 3bc795f..ae38980 100644
--- a/rbutil/rbutilqt/gui/manualwidget.cpp
+++ b/rbutil/rbutilqt/gui/manualwidget.cpp
@@ -127,7 +127,7 @@ void ManualWidget::downloadManual(void)
     }
     qDebug() << "[ManualWidget] Manual URL:" << manualurl;
 
-    installer->setLogVersion(ServerInfo::value(ServerInfo::DailyDate).toString());
+    installer->setLogVersion();
     installer->setUrl(manualurl);
     installer->setUnzip(false);
 
diff --git a/rbutil/rbutilqt/rbutilqt.cpp b/rbutil/rbutilqt/rbutilqt.cpp
index 5c49658..ae45068 100644
--- a/rbutil/rbutilqt/rbutilqt.cpp
+++ b/rbutil/rbutilqt/rbutilqt.cpp
@@ -994,7 +994,7 @@ void RbUtilQt::installDoom()
 
     installer->setUrl(SystemInfo::value(SystemInfo::DoomUrl).toString());
     installer->setLogSection("Game Addons");
-    installer->setLogVersion(ServerInfo::value(ServerInfo::DailyDate).toString());
+    installer->setLogVersion();
     installer->setMountPoint(RbSettings::value(RbSettings::Mountpoint).toString());
     if(!RbSettings::value(RbSettings::CacheDisabled).toBool())
         installer->setCache(true);