Merge lp:~michael-sheldon/ubuntu-download-manager/fix-1461444 into lp:ubuntu-download-manager

[Michael Sheldon]

Clean up old log files

Clean up old log files

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:351
http://jenkins.qa.ubuntu.com/job/ubuntu-download-manager-ci/800/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-download-manager-wily-amd64-ci/31
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-download-manager-wily-armhf-ci/30
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-download-manager-wily-armhf-ci/30/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-download-manager-ci/800/rebuild

review: Approve (continuous-integration)

=== modified file 'src/common/public/ubuntu/transfers/system/logger.cpp'
--- src/common/public/ubuntu/transfers/system/logger.cpp	2014-07-21 12:30:56 +0000
+++ src/common/public/ubuntu/transfers/system/logger.cpp	2016-03-17 14:04:15 +0000
@@ -105,18 +105,31 @@
 void
 Logger::setupLogging(const QString logDir) {
     auto path = logDir;
+    QDir *dir;
     if (path == "" ){
         path = getLogDir() + QDir::separator() + LOG_NAME;
+        dir = new QDir(getLogDir());
     } else {
         bool wasCreated = QDir().mkpath(logDir);
         if (wasCreated) {
             path = QDir(logDir).absoluteFilePath(LOG_NAME);
+            dir = new QDir(logDir);
         } else {
             qCritical() << "Could not create the logging path" << logDir;
             path = getLogDir() + QDir::separator() + LOG_NAME;
+            dir = new QDir(getLogDir());
         }
     }
 
+    // Only keep the last 100 most recent logs
+    QStringList nameFilter;
+    nameFilter << LOG_NAME + "*";
+    QStringList logFiles = dir->entryList(nameFilter, QDir::Files | QDir::Hidden | QDir::Writable, QDir::Time);
+    for (int i = 99; i < logFiles.count(); i++) {
+        QFile logFile(dir->absoluteFilePath(logFiles[i]));
+        logFile.remove();
+    }
+
     auto appName = QCoreApplication::instance()->applicationName();
 
     if (!_init) {