Fix a possible race and some wrong assumptions about file ownership.

The download archive job does not need to cleanup the files afterwards,
that's taken care of by the meta data download. That one will provide
the infrastructure where the files are downloaded at. Also prevent the
meta job from removing the files once they are fetched in case of a
cancel event (which does not entirely mean we are going to shutdown).

Change-Id: I66eeff30ef4cabb485dd4f300b2917deb7557867
Reviewed-by: Kai Koehne <kai.koehne@digia.com>
Reviewed-by: Niels Weber <niels.weber@digia.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/src/libs/installer/downloadarchivesjob.h b/src/libs/installer/downloadarchivesjob.h
index 765d2c564..25d3daaec 100644
--- a/src/libs/installer/downloadarchivesjob.h
+++ b/src/libs/installer/downloadarchivesjob.h
@@ -45,7 +45,6 @@
 #include <kdjob.h>
 
 #include <QtCore/QPair>
-#include <QtCore/QSet>
 
 QT_BEGIN_NAMESPACE
 class QTimerEvent;
@@ -65,9 +64,10 @@ class DownloadArchivesJob : public KDJob
     Q_OBJECT
 
 public:
-    explicit DownloadArchivesJob(PackageManagerCore *core = 0);
+    explicit DownloadArchivesJob(PackageManagerCore *core);
     ~DownloadArchivesJob();
 
+    int numberOfDownloads() const { return m_archivesDownloaded; }
     void setArchivesToDownload(const QList<QPair<QString, QString> > &archives);
 
 Q_SIGNALS:
@@ -102,7 +102,6 @@ private:
     QList<QPair<QString, QString> > m_archivesToDownload;
 
     bool m_canceled;
-    QSet<QString> m_temporaryFiles;
     QByteArray m_currentHash;
     double m_lastFileProgress;
     int m_progressChangedTimerId;