]> git.sesse.net Git - kdenlive/blobdiff - src/projectlist.cpp
Some proxy / job fixes
[kdenlive] / src / projectlist.cpp
index 08a81dacbe161721c293387d1aede9ddb5b3ffbb..683500185314d9ebe8dd338421412fc38919c88e 100644 (file)
@@ -290,10 +290,13 @@ ProjectList::ProjectList(QWidget *parent) :
     connect(m_listView, SIGNAL(showProperties(DocClipBase *)), this, SIGNAL(showClipProperties(DocClipBase *)));
     
     connect(this, SIGNAL(cancelRunningJob(const QString, stringMap )), this, SLOT(slotCancelRunningJob(const QString, stringMap)));
-    connect(this, SIGNAL(processLog(ProjectItem *, int , int)), this, SLOT(slotProcessLog(ProjectItem *, int , int)));
+    connect(this, SIGNAL(processLog(const QString, int , int)), this, SLOT(slotProcessLog(const QString, int , int)));
+    
+    connect(this, SIGNAL(updateJobStatus(const QString &, int, int, const QString &, const QString &, const QString)), this, SLOT(slotUpdateJobStatus(const QString &, int, int, const QString &, const QString &, const QString)));
+    
+    connect(this, SIGNAL(checkJobProcess()), this, SLOT(slotCheckJobProcess()));
+    connect(this, SIGNAL(gotProxy(const QString)), this, SLOT(slotGotProxyForId(const QString)));
     
-    connect(this, SIGNAL(jobCrashed(ProjectItem *, const QString &, const QString &, const QString)), this, SLOT(slotJobCrashed(ProjectItem *, const QString &, const QString &, const QString)));
-
     m_listViewDelegate = new ItemDelegate(m_listView);
     m_listView->setItemDelegate(m_listViewDelegate);
 #ifdef NEPOMUK
@@ -312,6 +315,8 @@ ProjectList::~ProjectList()
     m_abortAllJobs = true;
     m_closing = true;
     m_thumbnailQueue.clear();
+    m_jobThreads.waitForFinished();
+    m_jobThreads.clearFutures();
     if (!m_jobList.isEmpty()) qDeleteAll(m_jobList);
     m_jobList.clear();
     delete m_menu;
@@ -916,7 +921,8 @@ void ProjectList::slotUpdateClipProperties(ProjectItem *clip, QMap <QString, QSt
     clip->setProperties(properties);
     if (properties.contains("proxy")) {
         if (properties.value("proxy") == "-" || properties.value("proxy").isEmpty())
-            clip->setJobStatus(NOJOB);
+            // this should only apply to proxy jobs
+            clip->setConditionalJobStatus(NOJOB, PROXYJOB);
     }
     if (properties.contains("name")) {
         monitorItemEditing(false);
@@ -1138,7 +1144,6 @@ void ProjectList::slotDeleteClip(const QString &clipId)
         return;
     }
     deleteJobsForClip(clipId);
-    if (item->isProxyRunning()) m_abortProxy.append(item->referencedClip()->getProperty("proxy"));
     m_listView->blockSignals(true);
     QTreeWidgetItem *newSelectedItem = m_listView->itemAbove(item);
     if (!newSelectedItem)
@@ -1312,7 +1317,7 @@ void ProjectList::slotGotProxy(const QString &proxyPath)
     QTreeWidgetItemIterator it(m_listView);
     ProjectItem *item;
 
-    while (*it && !m_abortAllJobs) {
+    while (*it && !m_closing) {
         if ((*it)->type() == PROJECTCLIPTYPE) {
             item = static_cast <ProjectItem *>(*it);
             if (item->referencedClip()->getProperty("proxy") == proxyPath)
@@ -1322,6 +1327,13 @@ void ProjectList::slotGotProxy(const QString &proxyPath)
     }
 }
 
+void ProjectList::slotGotProxyForId(const QString id)
+{
+    if (m_closing) return;
+    ProjectItem *item = getItemById(id);
+    slotGotProxy(item);
+}
+
 void ProjectList::slotGotProxy(ProjectItem *item)
 {
     if (item == NULL) return;
@@ -1352,8 +1364,8 @@ void ProjectList::slotResetProjectList()
     m_listView->blockSignals(true);
     m_abortAllJobs = true;
     m_closing = true;
-    m_proxyThreads.waitForFinished();
-    m_proxyThreads.clearFutures();
+    m_jobThreads.waitForFinished();
+    m_jobThreads.clearFutures();
     m_thumbnailQueue.clear();
     if (!m_jobList.isEmpty()) qDeleteAll(m_jobList);
     m_jobList.clear();
@@ -1463,7 +1475,8 @@ void ProjectList::updateAllClips(bool displayRatioChanged, bool fpsChanged, QStr
                 bool replace = false;
                 if (brokenClips.contains(item->clipId())) {
                     // if this is a proxy clip, disable proxy
-                    item->setJobStatus(NOJOB);
+                    item->setConditionalJobStatus(NOJOB, PROXYJOB);
+                    discardJobs(item->clipId(), PROXYJOB);
                     clip->setProperty("proxy", "-");
                     replace = true;
                 }
@@ -1683,7 +1696,7 @@ void ProjectList::slotRemoveInvalidProxy(const QString &id, bool durationError)
             kDebug() << "Proxy duration is wrong, try changing transcoding parameters.";
             emit displayMessage(i18n("Proxy clip unusable (duration is different from original)."), -2);
         }
-        slotJobCrashed(item, i18n("Failed to create proxy for %1. check parameters", item->text(0)), "project_settings");
+        slotUpdateJobStatus(item, PROXYJOB, JOBCRASHED, i18n("Failed to create proxy for %1. check parameters", item->text(0)), "project_settings");
         QString path = item->referencedClip()->getProperty("proxy");
         KUrl proxyFolder(m_doc->projectFolder().path( KUrl::AddTrailingSlash) + "proxy/");
 
@@ -1811,8 +1824,8 @@ void ProjectList::setDocument(KdenliveDoc *doc)
     m_listView->blockSignals(true);
     m_abortAllJobs = true;
     m_closing = true;
-    m_proxyThreads.waitForFinished();
-    m_proxyThreads.clearFutures();
+    m_jobThreads.waitForFinished();
+    m_jobThreads.clearFutures();
     m_thumbnailQueue.clear();
     m_listView->clear();
     m_processingClips.clear();
@@ -1998,7 +2011,6 @@ void ProjectList::slotReplyGetFileProperties(const QString &clipId, Mlt::Produce
 {
     QString toReload;
     ProjectItem *item = getItemById(clipId);
-
     int queue = m_render->processingItems();
     if (item && producer) {
         monitorItemEditing(false);
@@ -2015,8 +2027,14 @@ void ProjectList::slotReplyGetFileProperties(const QString &clipId, Mlt::Produce
 
         // Proxy stuff
         QString size = properties.value("frame_size");
-        if (!useProxy() && clip->getProperty("proxy").isEmpty()) setJobStatus(item, NOJOB);
-        if (useProxy() && generateProxy() && clip->getProperty("proxy") == "-") setJobStatus(item, NOJOB);
+        if (!useProxy() && clip->getProperty("proxy").isEmpty()) {
+            item->setConditionalJobStatus(NOJOB, PROXYJOB);
+            discardJobs(clipId, PROXYJOB);
+        }
+        if (useProxy() && generateProxy() && clip->getProperty("proxy") == "-") {
+            item->setConditionalJobStatus(NOJOB, PROXYJOB);
+            discardJobs(clipId, PROXYJOB);
+        }
         else if (useProxy() && !item->hasProxy() && !hasPendingProxy(item)) {
             // proxy video and image clips
             int maxSize;
@@ -2509,28 +2527,26 @@ void ProjectList::slotCreateProxy(const QString id)
     if (!item || hasPendingProxy(item) || item->referencedClip()->isPlaceHolder()) return;
     QString path = item->referencedClip()->getProperty("proxy");
     if (path.isEmpty()) {
-        jobCrashed(item, i18n("Failed to create proxy, empty path."));
+        slotUpdateJobStatus(item, PROXYJOB, JOBCRASHED, i18n("Failed to create proxy, empty path."));
         return;
     }
 
-    if (m_abortProxy.contains(path)) m_abortProxy.removeAll(path);
     if (m_processingProxy.contains(path)) {
         // Proxy is already being generated
         return;
     }
     if (QFileInfo(path).size() > 0) {
         // Proxy already created
-        setJobStatus(item, JOBDONE);
+        setJobStatus(item, PROXYJOB, JOBDONE);
         slotGotProxy(path);
         return;
     }
-    if (!item->isJobRunning()) setJobStatus(item, JOBWAITING);
     m_processingProxy.append(path);
 
     ProxyJob *job = new ProxyJob(item->clipType(), id, QStringList() << path << item->clipUrl().path() << item->referencedClip()->producerProperty("_exif_orientation") << m_doc->getDocumentProperty("proxyparams").simplified() << QString::number(m_render->frameRenderWidth()) << QString::number(m_render->renderHeight()));
     m_jobList.append(job);
-    
-    startJobProcess();
+    setJobStatus(item, job->jobType, JOBWAITING);
+    slotCheckJobProcess();
 }
 
 void ProjectList::slotCutClipJob(const QString &id, QPoint zone)
@@ -2589,185 +2605,176 @@ void ProjectList::slotCutClipJob(const QString &id, QPoint zone)
     KdenliveSettings::setAdd_clip_cut(ui.add_clip->isChecked());
     delete d;
 
-    if (!item->isJobRunning()) setJobStatus(item, JOBWAITING);
     m_processingProxy.append(dest);
     QStringList jobParams;
     jobParams << dest << item->clipUrl().path() << timeIn << timeOut << QString::number(duration) << QString::number(KdenliveSettings::add_clip_cut());
     if (!extraParams.isEmpty()) jobParams << extraParams;
     CutClipJob *job = new CutClipJob(item->clipType(), id, jobParams);
     m_jobList.append(job);
-    
-    startJobProcess();
+    setJobStatus(item, job->jobType, JOBWAITING);
+
+    slotCheckJobProcess();
 }
 
 
-void ProjectList::startJobProcess()
-{
-    int ct = 0;
-    if (!m_proxyThreads.futures().isEmpty()) {
+void ProjectList::slotCheckJobProcess()
+{        
+    if (!m_jobThreads.futures().isEmpty()) {
         // Remove inactive threads
-        QList <QFuture<void> > futures = m_proxyThreads.futures();
-        m_proxyThreads.clearFutures();
+        QList <QFuture<void> > futures = m_jobThreads.futures();
+        m_jobThreads.clearFutures();
         for (int i = 0; i < futures.count(); i++)
             if (!futures.at(i).isFinished()) {
-                m_proxyThreads.addFuture(futures.at(i));
-                ct++;
+                m_jobThreads.addFuture(futures.at(i));
             }
     }
-    emit jobCount(ct + m_jobList.count());
+    if (m_jobList.isEmpty()) return;
+    int count = 0;
+    m_jobMutex.lock();
+    for (int i = 0; i < m_jobList.count(); i++) {
+        if (m_jobList.at(i)->jobStatus == JOBWORKING || m_jobList.at(i)->jobStatus == JOBWAITING)
+            count ++;
+        else {
+            // remove finished jobs
+            AbstractClipJob *job = m_jobList.takeAt(i);
+            delete job;
+            i--;
+        }
+    }
+
+    emit jobCount(count);
+    m_jobMutex.unlock();
     
-    if (m_proxyThreads.futures().isEmpty() || m_proxyThreads.futures().count() < KdenliveSettings::proxythreads()) m_proxyThreads.addFuture(QtConcurrent::run(this, &ProjectList::slotProcessJobs));
+    if (m_jobThreads.futures().isEmpty() || m_jobThreads.futures().count() < KdenliveSettings::proxythreads()) m_jobThreads.addFuture(QtConcurrent::run(this, &ProjectList::slotProcessJobs));
 }
 
 void ProjectList::slotAbortProxy(const QString id, const QString path)
 {
-    QTreeWidgetItemIterator it(m_listView);
     ProjectItem *item = getItemById(id);
     if (!item) return;
-    if (!path.isEmpty() && m_processingProxy.contains(path)) {
-        m_abortProxy << path;
-    }
-    else {
-        // Should only be done if we were already using a proxy producer
-        if (!item->isProxyRunning()) slotGotProxy(item);
-    }
+    if (!item->isProxyRunning()) slotGotProxy(item);
+    item->setConditionalJobStatus(NOJOB, PROXYJOB);
+    discardJobs(id, PROXYJOB);
 }
 
 void ProjectList::slotProcessJobs()
 {
     while (!m_jobList.isEmpty() && !m_abortAllJobs) {
         emit projectModified();
-        AbstractClipJob *job = m_jobList.takeFirst();
-        // Get jobs count
-        int ct = 0;
-        QList <QFuture<void> > futures = m_proxyThreads.futures();
-        for (int i = 0; i < futures.count(); i++)
-            if (futures.at(i).isRunning()) {
-                ct++;
-            }
-        emit jobCount(ct + m_jobList.count());
-
-        if (job->jobType == PROXYJOB) {
-            //ProxyJob *pjob = static_cast<ProxyJob *> (job);
-            kDebug()<<"// STARTING JOB: "<<job->destination();
-            if (m_abortProxy.contains(job->destination())) {
-                m_abortProxy.removeAll(job->destination());
-                m_processingProxy.removeAll(job->destination());
-                emit cancelRunningJob(job->clipId(), job->cancelProperties());
-                delete job;
-                continue;
+        AbstractClipJob *job = NULL;
+        int count = 0;
+        m_jobMutex.lock();
+        for (int i = 0; i < m_jobList.count(); i++) {
+            if (m_jobList.at(i)->jobStatus == JOBWAITING) {
+                if (job == NULL) {
+                    m_jobList.at(i)->jobStatus = JOBWORKING;
+                    job = m_jobList.at(i);
+                }
+                count++;
             }
+            else if (m_jobList.at(i)->jobStatus == JOBWORKING)
+                count ++;
         }
-        
+        // Set jobs count
+        emit jobCount(count);
+        m_jobMutex.unlock();
+
+        if (job == NULL) {
+            break;
+        }
+        QString destination = job->destination();
+       
         // Get the list of clips that will need to get progress info
         ProjectItem *processingItem = getItemById(job->clipId());
         if (processingItem == NULL) {
-            delete job;
+            job->setStatus(JOBDONE);
             continue;
         }
-        
-        /*while (*it && !m_abortAllJobs) {
-            if ((*it)->type() == PROJECTCLIPTYPE) {
-                ProjectItem *item = static_cast <ProjectItem *>(*it);
-                if (item->referencedClip()->getProperty("proxy") == job->destination()) {
-                    processingItems.append(item);
-                }
-            }
-            ++it;
-        }*/
 
         // Make sure destination path is writable
-        QFile file(job->destination());
+        QFile file(destination);
         if (!file.open(QIODevice::WriteOnly)) {
-            emit jobCrashed(processingItem, i18n("Cannot write to path: %1", job->destination()));
-            m_processingProxy.removeAll(job->destination());
-            delete job;
+            emit updateJobStatus(job->clipId(), job->jobType, JOBCRASHED, i18n("Cannot write to path: %1", destination));
+            m_processingProxy.removeAll(destination);
+            job->setStatus(JOBCRASHED);
             continue;
         }
         file.close();
-        QFile::remove(job->destination());
+        QFile::remove(destination);
     
-        setJobStatus(processingItem, CREATINGJOB, 0, job->jobType);
+        //setJobStatus(processingItem, JOBWORKING, 0, job->jobType, job->statusMessage());
 
         bool success;
         QProcess *jobProcess = job->startJob(&success);
-            
         int result = -1;
         if (jobProcess == NULL) {
             // job is finished
             if (success) {
-                setJobStatus(processingItem, JOBDONE);
-                if (job->jobType == PROXYJOB) slotGotProxy(job->destination());
+                emit updateJobStatus(job->clipId(), job->jobType, JOBDONE);
+                if (job->jobType == PROXYJOB) emit gotProxy(job->clipId());
                 //TODO: set folder for transcoded clips
-                else if (job->jobType == CUTJOB) emit addClip(job->destination(), QString(), QString());
+                else if (job->jobType == CUTJOB) emit addClip(destination, QString(), QString());
+                job->setStatus(JOBDONE);
             }
             else {
-                QFile::remove(job->destination());
-                emit jobCrashed(processingItem, job->errorMessage());
+                QFile::remove(destination);
+                emit updateJobStatus(job->clipId(), job->jobType, JOBCRASHED, job->errorMessage());
+                job->setStatus(JOBCRASHED);
             }
-            m_abortProxy.removeAll(job->destination());
-            m_processingProxy.removeAll(job->destination());
-            delete job;
+            m_processingProxy.removeAll(destination);
             continue;
         }
         else while (jobProcess->state() != QProcess::NotRunning) {
             // building proxy file
-            if (m_abortProxy.contains(job->destination()) || m_abortAllJobs) {
+            if (m_abortAllJobs || job->jobStatus == JOBABORTED) {
+                job->jobStatus = JOBABORTED;
                 jobProcess->close();
                 jobProcess->waitForFinished();
-                QFile::remove(job->destination());
-                m_abortProxy.removeAll(job->destination());
-                m_processingProxy.removeAll(job->destination());
+                QFile::remove(destination);
+                m_processingProxy.removeAll(destination);
                 if (!m_closing) {
                     emit cancelRunningJob(job->clipId(), job->cancelProperties());
-                    /*for (int i = 0; i < processingItems.count(); i++)
-                        setJobStatus(processingItems.at(i), NOJOB);*/
                 }
-                else continue;
                 result = -2;
+                continue;
             }
             else {
                 int progress = job->processLogInfo();
-                if (progress > -1) emit processLog(processingItem, progress, job->jobType); 
+                if (progress > 0) emit processLog(job->clipId(), progress, job->jobType); 
             }
             jobProcess->waitForFinished(500);
         }
         jobProcess->waitForFinished();
-        m_processingProxy.removeAll(job->destination());
+        m_processingProxy.removeAll(destination);
         if (result == -1) result = jobProcess->exitStatus();
         
-        if (result != -2 && QFileInfo(job->destination()).size() == 0) {
+        if (result != -2 && QFileInfo(destination).size() == 0) {
             job->processLogInfo();
-            emit jobCrashed(processingItem, i18n("Failed to create file"), QString(), job->errorMessage());
+            emit updateJobStatus(job->clipId(), job->jobType, JOBCRASHED, i18n("Failed to create file"), QString(), job->errorMessage());
+            job->setStatus(JOBCRASHED);
             result = -2;
         }
-        
         if (result == QProcess::NormalExit) {
             // proxy successfully created
-            setJobStatus(processingItem, JOBDONE);
-            if (job->jobType == PROXYJOB) slotGotProxy(job->destination());
+            emit updateJobStatus(job->clipId(), job->jobType, JOBDONE);
+            if (job->jobType == PROXYJOB) emit gotProxy(job->clipId());
             //TODO: set folder for transcoded clips
             else if (job->jobType == CUTJOB) {
                 CutClipJob *cutJob = static_cast<CutClipJob *>(job);
-                if (cutJob->addClipToProject) emit addClip(job->destination(), QString(), QString());
+                if (cutJob->addClipToProject) emit addClip(destination, QString(), QString());
             }
+            job->setStatus(JOBDONE);
         }
         else if (result == QProcess::CrashExit) {
             // Proxy process crashed
-            QFile::remove(job->destination());
-            emit jobCrashed(processingItem, i18n("Job crashed"), QString(), job->errorMessage());
+            QFile::remove(destination);
+            emit updateJobStatus(job->clipId(), job->jobType, JOBCRASHED, i18n("Job crashed"), QString(), job->errorMessage());
+            job->setStatus(JOBCRASHED);
         }
-        delete job;
         continue;
     }
-    // Thread finished, update count
-    int ct = -1; // -1 because we don't want to count this terminating thread
-    QList <QFuture<void> > futures = m_proxyThreads.futures();
-    for (int i = 0; i < futures.count(); i++)
-        if (futures.at(i).isRunning()) {
-            ct++;
-        }
-    emit jobCount(ct + m_jobList.count());
+    // Thread finished, cleanup & update count
+    QTimer::singleShot(200, this, SIGNAL(checkJobProcess()));
 }
 
 
@@ -2842,9 +2849,10 @@ void ProjectList::updateProxyConfig()
     else delete command;
 }
 
-void ProjectList::slotProcessLog(ProjectItem *item, int progress, int type)
+void ProjectList::slotProcessLog(const QString id, int progress, int type)
 {
-    setJobStatus(item, CREATINGJOB, progress, (JOBTYPE) type);
+    ProjectItem *item = getItemById(id);
+    setJobStatus(item, (JOBTYPE) type, JOBWORKING, progress);
 }
 
 void ProjectList::slotProxyCurrentItem(bool doProxy, ProjectItem *itemToProxy)
@@ -2954,11 +2962,11 @@ void ProjectList::slotDeleteProxy(const QString proxyPath)
     QFile::remove(proxyPath);
 }
 
-void ProjectList::setJobStatus(ProjectItem *item, CLIPJOBSTATUS status, int progress, JOBTYPE jobType)
+void ProjectList::setJobStatus(ProjectItem *item, JOBTYPE jobType, CLIPJOBSTATUS status, int progress, const QString &statusMessage)
 {
-    if (item == NULL || m_abortAllJobs) return;
+    if (item == NULL || (m_abortAllJobs && m_closing)) return;
     monitorItemEditing(false);
-    item->setJobStatus(status, progress, jobType);
+    item->setJobStatus(jobType, status, progress);
     if (status == JOBCRASHED) {
         DocClipBase *clip = item->referencedClip();
         if (!clip) {
@@ -3022,10 +3030,11 @@ void ProjectList::processThumbOverlays(ProjectItem *item, QPixmap &pix)
 void ProjectList::slotCancelJobs()
 {
     m_abortAllJobs = true;
-    m_proxyThreads.waitForFinished();
-    m_proxyThreads.clearFutures();
+    m_jobThreads.waitForFinished();
+    m_jobThreads.clearFutures();
     QUndoCommand *command = new QUndoCommand();
     command->setText(i18np("Cancel job", "Cancel jobs", m_jobList.count()));
+    m_jobMutex.lock();
     for (int i = 0; i < m_jobList.count(); i++) {
         ProjectItem *item = getItemById(m_jobList.at(i)->clipId());
         if (!item || !item->referencedClip()) continue;
@@ -3034,6 +3043,7 @@ void ProjectList::slotCancelJobs()
         QMap <QString, QString> oldProps = item->referencedClip()->currentProperties(newProps);
         new EditClipCommand(this, m_jobList.at(i)->clipId(), oldProps, newProps, true, command);
     }
+    m_jobMutex.unlock();
     if (command->childCount() > 0) {
         m_doc->commandStack()->push(command);
     }
@@ -3060,32 +3070,42 @@ void ProjectList::slotCancelRunningJob(const QString id, stringMap newProps)
 bool ProjectList::hasPendingProxy(ProjectItem *item)
 {
     if (!item || !item->referencedClip() || m_abortAllJobs) return false;
+    AbstractClipJob *job;
+    QMutexLocker lock(&m_jobMutex);
     for (int i = 0; i < m_jobList.count(); i++) {
         if (m_abortAllJobs) break;
-        if (m_jobList.at(i)->clipId() == item->clipId() && m_jobList.at(i)->jobType == PROXYJOB) return true;
+        job = m_jobList.at(i);
+        if (job->clipId() == item->clipId() && job->jobType == PROXYJOB && (job->jobStatus == JOBWAITING || job->jobStatus == JOBWORKING)) return true;
     }
-    if (item->isProxyRunning()) return true;
+    
     return false;
 }
 
 void ProjectList::deleteJobsForClip(const QString &clipId)
 {
-    QList <AbstractClipJob *> jobsToDelete;
+    QMutexLocker lock(&m_jobMutex);
     for (int i = 0; i < m_jobList.count(); i++) {
         if (m_abortAllJobs) break;
         if (m_jobList.at(i)->clipId() == clipId) {
-            AbstractClipJob *job = m_jobList.takeAt(i);
-            delete job;
-            i--;
+            m_jobList.at(i)->setStatus(JOBABORTED);
         }
     }
 }
 
-void ProjectList::slotJobCrashed(ProjectItem *item, const QString &label, const QString &actionName, const QString details)
+void ProjectList::slotUpdateJobStatus(const QString &id, int type, int status, const QString &label, const QString &actionName, const QString details)
+{
+    ProjectItem *item = getItemById(id);
+    if (!item) return;
+    slotUpdateJobStatus(item, type, status, label, actionName, details);
+    
+}
+
+void ProjectList::slotUpdateJobStatus(ProjectItem *item, int type, int status, const QString &label, const QString &actionName, const QString details)
 {
+    item->setJobStatus((JOBTYPE) type, (CLIPJOBSTATUS) status);
+    if (status != JOBCRASHED) return;
 #if KDE_IS_VERSION(4,7,0)
     m_infoMessage->animatedHide();
-    item->setJobStatus(NOJOB);
     m_errorLog.clear();
     m_infoMessage->setText(label);
     m_infoMessage->setWordWrap(label.length() > 35);
@@ -3096,7 +3116,7 @@ void ProjectList::slotJobCrashed(ProjectItem *item, const QString &label, const
     }
     
     if (!actionName.isEmpty()) {
-        QAction *action;
+        QAction *action = NULL;
         QList< KActionCollection * > collections = KActionCollection::allCollections();
         for (int i = 0; i < collections.count(); i++) {
             KActionCollection *coll = collections.at(i);
@@ -3110,13 +3130,12 @@ void ProjectList::slotJobCrashed(ProjectItem *item, const QString &label, const
         m_infoMessage->addAction(m_logAction);
     }
     m_infoMessage->animatedShow();
-#else 
-    item->setJobStatus(JOBCRASHED);
 #endif
 }
 
 void ProjectList::slotShowJobLog()
 {
+#if KDE_IS_VERSION(4,7,0)
     KDialog d(this);
     d.setButtons(KDialog::Close);
     QTextEdit t(&d);
@@ -3124,6 +3143,18 @@ void ProjectList::slotShowJobLog()
     t.setReadOnly(true);
     d.setMainWidget(&t);
     d.exec();
+#endif
+}
+
+void ProjectList::discardJobs(const QString &id, JOBTYPE type) {
+    QMutexLocker lock(&m_jobMutex);
+    for (int i = 0; i < m_jobList.count(); i++) {
+        if (m_jobList.at(i)->clipId() == id && m_jobList.at(i)->jobType == type) {
+            // discard this job
+            m_jobList.at(i)->setStatus(JOBABORTED);
+        }
+    }
 }
 
+
 #include "projectlist.moc"