]> git.sesse.net Git - kdenlive/blobdiff - src/projectlist.cpp
Improve handling of missing clips:
[kdenlive] / src / projectlist.cpp
index 5ff295cfcedfa61b89e5321f5ed6bfd480255640..20ab1d4b3c6e23fb888cbec8a7b87423bad806e1 100644 (file)
 #include <QProcess>
 #include <QHeaderView>
 
-ProjectList::ProjectList(QWidget *parent)
-        : QWidget(parent), m_render(NULL), m_fps(-1), m_commandStack(NULL), m_selectedItem(NULL), m_infoQueue(QMap <QString, QDomElement> ()), m_thumbnailQueue(QList <QString> ()), m_refreshed(false), m_editAction(NULL), m_openAction(NULL), m_deleteAction(NULL), m_reloadAction(NULL)
-{
-
-    listView = new ProjectListView(this);;
+ProjectList::ProjectList(QWidget *parent) :
+        QWidget(parent),
+        m_render(NULL),
+        m_fps(-1),
+        m_commandStack(NULL),
+        m_editAction(NULL),
+        m_deleteAction(NULL),
+        m_openAction(NULL),
+        m_reloadAction(NULL),
+        m_selectedItem(NULL),
+        m_refreshed(false),
+        m_infoQueue(),
+        m_thumbnailQueue()
+{
+
+    m_listView = new ProjectListView(this);;
     QVBoxLayout *layout = new QVBoxLayout;
     layout->setContentsMargins(0, 0, 0, 0);
 
     // setup toolbar
-    searchView = new KTreeWidgetSearchLine(this);
+    KTreeWidgetSearchLine *searchView = new KTreeWidgetSearchLine(this);
     m_toolbar = new QToolBar("projectToolBar", this);
     m_toolbar->addWidget(searchView);
+    searchView->setTreeWidget(m_listView);
 
     m_addButton = new QToolButton(m_toolbar);
     m_addButton->setPopupMode(QToolButton::MenuButtonPopup);
     m_toolbar->addWidget(m_addButton);
 
     layout->addWidget(m_toolbar);
-    layout->addWidget(listView);
+    layout->addWidget(m_listView);
     setLayout(layout);
-    //m_toolbar->setEnabled(false);
 
-    searchView->setTreeWidget(listView);
 
-    connect(listView, SIGNAL(itemSelectionChanged()), this, SLOT(slotClipSelected()));
-    connect(listView, SIGNAL(focusMonitor()), this, SLOT(slotClipSelected()));
-    connect(listView, SIGNAL(pauseMonitor()), this, SLOT(slotPauseMonitor()));
-    connect(listView, SIGNAL(requestMenu(const QPoint &, QTreeWidgetItem *)), this, SLOT(slotContextMenu(const QPoint &, QTreeWidgetItem *)));
-    connect(listView, SIGNAL(addClip()), this, SLOT(slotAddClip()));
-    connect(listView, SIGNAL(addClip(const QList <QUrl>, const QString &)), this, SLOT(slotAddClip(const QList <QUrl>, const QString &)));
-    connect(listView, SIGNAL(itemChanged(QTreeWidgetItem *, int)), this, SLOT(slotItemEdited(QTreeWidgetItem *, int)));
-    connect(listView, SIGNAL(showProperties(DocClipBase *)), this, SIGNAL(showClipProperties(DocClipBase *)));
 
-    ItemDelegate *listViewDelegate = new ItemDelegate(listView);
-    listView->setItemDelegate(listViewDelegate);
+    connect(m_listView, SIGNAL(itemSelectionChanged()), this, SLOT(slotClipSelected()));
+    connect(m_listView, SIGNAL(focusMonitor()), this, SLOT(slotClipSelected()));
+    connect(m_listView, SIGNAL(pauseMonitor()), this, SLOT(slotPauseMonitor()));
+    connect(m_listView, SIGNAL(requestMenu(const QPoint &, QTreeWidgetItem *)), this, SLOT(slotContextMenu(const QPoint &, QTreeWidgetItem *)));
+    connect(m_listView, SIGNAL(addClip()), this, SLOT(slotAddClip()));
+    connect(m_listView, SIGNAL(addClip(const QList <QUrl>, const QString &)), this, SLOT(slotAddClip(const QList <QUrl>, const QString &)));
+    connect(m_listView, SIGNAL(itemChanged(QTreeWidgetItem *, int)), this, SLOT(slotItemEdited(QTreeWidgetItem *, int)));
+    connect(m_listView, SIGNAL(showProperties(DocClipBase *)), this, SIGNAL(showClipProperties(DocClipBase *)));
+
+    ItemDelegate *listViewDelegate = new ItemDelegate(m_listView);
+    m_listView->setItemDelegate(listViewDelegate);
 
     if (KdenliveSettings::activate_nepomuk()) {
         Nepomuk::ResourceManager::instance()->init();
@@ -157,17 +168,18 @@ void ProjectList::setupGeneratorMenu(QMenu *addMenu)
 
 QByteArray ProjectList::headerInfo()
 {
-    return listView->header()->saveState();
+    return m_listView->header()->saveState();
 }
 
 void ProjectList::setHeaderInfo(const QByteArray &state)
 {
-    listView->header()->restoreState(state);
+    m_listView->header()->restoreState(state);
 }
 
 void ProjectList::slotEditClip()
 {
-    ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+    ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
+    if (!(item->flags() & Qt::ItemIsDragEnabled)) return;
     if (item && !item->isGroup()) {
         emit clipSelected(item->referencedClip());
         emit showClipProperties(item->referencedClip());
@@ -176,7 +188,7 @@ void ProjectList::slotEditClip()
 
 void ProjectList::slotOpenClip()
 {
-    ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+    ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
     if (item && !item->isGroup()) {
         if (item->clipType() == IMAGE) {
             if (KdenliveSettings::defaultimageapp().isEmpty()) KMessageBox::sorry(this, i18n("Please set a default application to open images in the Settings dialog"));
@@ -191,7 +203,7 @@ void ProjectList::slotOpenClip()
 
 void ProjectList::slotReloadClip()
 {
-    ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+    ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
     if (item && !item->isGroup()) {
         if (item->clipType() == IMAGE) {
             item->referencedClip()->producer()->set("force_reload", 1);
@@ -203,13 +215,13 @@ void ProjectList::slotReloadClip()
 void ProjectList::setRenderer(Render *projectRender)
 {
     m_render = projectRender;
-    listView->setIconSize(QSize(40 * m_render->dar(), 40));
+    m_listView->setIconSize(QSize(40 * m_render->dar(), 40));
 }
 
 void ProjectList::slotClipSelected()
 {
-    if (listView->currentItem()) {
-        ProjectItem *clip = static_cast <ProjectItem*>(listView->currentItem());
+    if (m_listView->currentItem()) {
+        ProjectItem *clip = static_cast <ProjectItem*>(m_listView->currentItem());
         if (!clip->isGroup()) {
             m_selectedItem = clip;
             emit clipSelected(clip->referencedClip());
@@ -253,16 +265,16 @@ void ProjectList::slotUpdateClipProperties(ProjectItem *clip, QMap <QString, QSt
     if (!clip) return;
     if (!clip->isGroup()) clip->setProperties(properties);
     if (properties.contains("name")) {
-        listView->blockSignals(true);
+        m_listView->blockSignals(true);
         clip->setText(1, properties.value("name"));
-        listView->blockSignals(false);
+        m_listView->blockSignals(false);
         emit clipNameChanged(clip->clipId(), properties.value("name"));
     }
     if (properties.contains("description")) {
         CLIPTYPE type = clip->clipType();
-        listView->blockSignals(true);
+        m_listView->blockSignals(true);
         clip->setText(2, properties.value("description"));
-        listView->blockSignals(false);
+        m_listView->blockSignals(false);
         if (KdenliveSettings::activate_nepomuk() && (type == AUDIO || type == VIDEO || type == AV || type == IMAGE || type == PLAYLIST)) {
             // Use Nepomuk system to store clip description
             Nepomuk::Resource f(clip->clipUrl().path());
@@ -334,20 +346,20 @@ void ProjectList::slotContextMenu(const QPoint &pos, QTreeWidgetItem *item)
 
 void ProjectList::slotRemoveClip()
 {
-    if (!listView->currentItem()) return;
+    if (!m_listView->currentItem()) return;
     QList <QString> ids;
     QMap <QString, QString> folderids;
-    QList<QTreeWidgetItem *> selected = listView->selectedItems();
+    QList<QTreeWidgetItem *> selected = m_listView->selectedItems();
     ProjectItem *item;
     for (int i = 0; i < selected.count(); i++) {
         item = static_cast <ProjectItem *>(selected.at(i));
         if (item->isGroup()) folderids[item->groupName()] = item->clipId();
         else ids << item->clipId();
         if (item->numReferences() > 0) {
-            if (KMessageBox::questionYesNo(this, i18np("Delete clip <b>%2</b> ?<br>This will also remove the clip in timeline", "Delete clip <b>%2</b> ?<br>This will also remove its %1 clips in timeline", item->numReferences(), item->names().at(1)), i18n("Delete Clip")) != KMessageBox::Yes) return;
+            if (KMessageBox::questionYesNo(this, i18np("Delete clip <b>%2</b>?<br>This will also remove the clip in timeline", "Delete clip <b>%2</b>?<br>This will also remove its %1 clips in timeline", item->numReferences(), item->names().at(1)), i18n("Delete Clip")) != KMessageBox::Yes) return;
         } else if (item->isGroup() && item->childCount() > 0) {
             int children = item->childCount();
-            if (KMessageBox::questionYesNo(this, i18n("Delete folder <b>%2</b> ?<br>This will also remove the %1 clips in that folder", children, item->names().at(1)), i18n("Delete Folder")) != KMessageBox::Yes) return;
+            if (KMessageBox::questionYesNo(this, i18n("Delete folder <b>%2</b>?<br>This will also remove the %1 clips in that folder", children, item->names().at(1)), i18n("Delete Folder")) != KMessageBox::Yes) return;
             for (int i = 0; i < children; ++i) {
                 ProjectItem *child = static_cast <ProjectItem *>(item->child(i));
                 ids << child->clipId();
@@ -356,7 +368,7 @@ void ProjectList::slotRemoveClip()
     }
     if (!ids.isEmpty()) m_doc->deleteProjectClip(ids);
     if (!folderids.isEmpty()) deleteProjectFolder(folderids);
-    if (listView->topLevelItemCount() == 0) {
+    if (m_listView->topLevelItemCount() == 0) {
         m_editAction->setEnabled(false);
         m_deleteAction->setEnabled(false);
         m_openAction->setEnabled(false);
@@ -367,7 +379,7 @@ void ProjectList::slotRemoveClip()
 void ProjectList::selectItemById(const QString &clipId)
 {
     ProjectItem *item = getItemById(clipId);
-    if (item) listView->setCurrentItem(item);
+    if (item) m_listView->setCurrentItem(item);
 }
 
 
@@ -406,11 +418,11 @@ void ProjectList::slotAddFolder(const QString foldername, const QString &clipId,
     } else {
         if (edit) {
             ProjectItem *item = getFolderItemById(clipId);
-            QTreeWidgetItemIterator it(listView);
+            QTreeWidgetItemIterator it(m_listView);
             if (item) {
-                listView->blockSignals(true);
+                m_listView->blockSignals(true);
                 item->setGroupName(foldername);
-                listView->blockSignals(false);
+                m_listView->blockSignals(false);
                 m_doc->clipManager()->addFolder(clipId, foldername);
                 const int children = item->childCount();
                 for (int i = 0; i < children; i++) {
@@ -421,10 +433,10 @@ void ProjectList::slotAddFolder(const QString foldername, const QString &clipId,
         } else {
             QStringList text;
             text << QString() << foldername;
-            listView->blockSignals(true);
-            (void) new ProjectItem(listView, text, clipId);
+            m_listView->blockSignals(true);
+            (void) new ProjectItem(m_listView, text, clipId);
             m_doc->clipManager()->addFolder(clipId, foldername);
-            listView->blockSignals(false);
+            m_listView->blockSignals(false);
         }
     }
 }
@@ -446,8 +458,8 @@ void ProjectList::deleteProjectFolder(QMap <QString, QString> map)
 
 void ProjectList::slotAddClip(DocClipBase *clip, bool getProperties)
 {
-    if (getProperties) listView->setEnabled(false);
-    listView->blockSignals(true);
+    if (getProperties) m_listView->setEnabled(false);
+    m_listView->blockSignals(true);
     const QString parent = clip->getProperty("groupid");
     kDebug() << "Adding clip with groupid: " << parent;
     ProjectItem *item = NULL;
@@ -459,13 +471,13 @@ void ProjectList::slotAddClip(DocClipBase *clip, bool getProperties)
             //kDebug() << "Adding clip to new group: " << groupName;
             if (groupName.isEmpty()) groupName = i18n("Folder");
             text << QString() << groupName;
-            parentitem = new ProjectItem(listView, text, parent);
+            parentitem = new ProjectItem(m_listView, text, parent);
         } else {
             //kDebug() << "Adding clip to existing group: " << parentitem->groupName();
         }
         if (parentitem) item = new ProjectItem(parentitem, clip);
     }
-    if (item == NULL) item = new ProjectItem(listView, clip);
+    if (item == NULL) item = new ProjectItem(m_listView, clip);
 
     KUrl url = clip->fileURL();
     if (!url.isEmpty() && KdenliveSettings::activate_nepomuk()) {
@@ -475,12 +487,12 @@ void ProjectList::slotAddClip(DocClipBase *clip, bool getProperties)
         if (!annotation.isEmpty()) item->setText(2, annotation);
         item->setText(3, QString::number(f.rating()));
     }
-    listView->blockSignals(false);
+    m_listView->blockSignals(false);
 }
 
 void ProjectList::slotResetProjectList()
 {
-    listView->clear();
+    m_listView->clear();
     emit clipSelected(NULL);
     m_thumbnailQueue.clear();
     m_infoQueue.clear();
@@ -491,14 +503,14 @@ void ProjectList::requestClipInfo(const QDomElement xml, const QString id)
 {
     kDebug() << " PRG LIST REQUEST CLP INFO: " << id;
     m_infoQueue.insert(id, xml);
-    listView->setEnabled(false);
+    m_listView->setEnabled(false);
     if (m_infoQueue.count() == 1) QTimer::singleShot(300, this, SLOT(slotProcessNextClipInQueue()));
 }
 
 void ProjectList::slotProcessNextClipInQueue()
 {
     if (m_infoQueue.isEmpty()) {
-        listView->setEnabled(true);
+        m_listView->setEnabled(true);
         return;
     }
     QMap<QString, QDomElement>::const_iterator i = m_infoQueue.constBegin();
@@ -508,20 +520,20 @@ void ProjectList::slotProcessNextClipInQueue()
         m_infoQueue.remove(i.key());
         emit getFileProperties(dom, id, true);
     }
-    if (m_infoQueue.isEmpty()) listView->setEnabled(true);
+    if (m_infoQueue.isEmpty()) m_listView->setEnabled(true);
 }
 
 void ProjectList::slotUpdateClip(const QString &id)
 {
     ProjectItem *item = getItemById(id);
-    listView->blockSignals(true);
+    m_listView->blockSignals(true);
     if (item) item->setData(1, UsageRole, QString::number(item->numReferences()));
-    listView->blockSignals(false);
+    m_listView->blockSignals(false);
 }
 
 void ProjectList::updateAllClips()
 {
-    QTreeWidgetItemIterator it(listView);
+    QTreeWidgetItemIterator it(m_listView);
     while (*it) {
         ProjectItem *item = static_cast <ProjectItem *>(*it);
         if (!item->isGroup()) {
@@ -537,25 +549,26 @@ void ProjectList::updateAllClips()
                     pix.save(clip->fileURL().path());
                     delete dia_ui;
                 }
-                requestClipInfo(clip->toXML(), clip->getId());
+                if (clip->isPlaceHolder() == false) requestClipInfo(clip->toXML(), clip->getId());
+                else item->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled);
             } else {
                 QString cachedPixmap = m_doc->projectFolder().path() + "/thumbs/" + item->getClipHash() + ".png";
                 if (QFile::exists(cachedPixmap)) {
                     //kDebug()<<"// USING CACHED PIX: "<<cachedPixmap;
-                    listView->blockSignals(true);
+                    m_listView->blockSignals(true);
                     item->setIcon(0, QPixmap(cachedPixmap));
-                    listView->blockSignals(false);
+                    m_listView->blockSignals(false);
                 } else requestClipThumbnail(item->clipId());
 
                 if (item->data(1, DurationRole).toString().isEmpty()) {
-                    listView->blockSignals(true);
+                    m_listView->blockSignals(true);
                     item->changeDuration(item->referencedClip()->producer()->get_playtime());
-                    listView->blockSignals(false);
+                    m_listView->blockSignals(false);
                 }
             }
-            listView->blockSignals(true);
+            m_listView->blockSignals(true);
             item->setData(1, UsageRole, QString::number(item->numReferences()));
-            listView->blockSignals(false);
+            m_listView->blockSignals(false);
             qApp->processEvents();
         }
         ++it;
@@ -569,7 +582,7 @@ void ProjectList::slotAddClip(const QList <QUrl> givenList, QString group)
     KUrl::List list;
     if (givenList.isEmpty()) {
         // Build list of mime types
-        QStringList mimeTypes = QStringList() << "application/x-kdenlive" << "video/x-flv" << "application/vnd.rn-realmedia" << "video/x-dv" << "video/dv" << "video/x-msvideo" << "video/x-matroska" << "video/mpeg" << "video/x-ms-wmv" << "audio/mpeg" << "audio/x-mp3" << "audio/x-wav" << "application/ogg" << "video/mp4" << "video/quicktime" << "image/gif" << "image/jpeg" << "image/png" << "image/x-tga" << "image/x-bmp" << "image/svg+xml" << "image/tiff" << "image/x-xcf-gimp" << "image/x-vnd.adobe.photoshop" << "image/x-pcx" << "image/x-exr" << "video/mlt-playlist" << "audio/x-flac" << "audio/mp4" << "video/x-matroska" << "audio/x-matroska";
+        QStringList mimeTypes = QStringList() << "application/x-kdenlive" << "video/x-flv" << "application/vnd.rn-realmedia" << "video/x-dv" << "video/dv" << "video/x-msvideo" << "video/mpeg" << "video/x-ms-wmv" << "audio/mpeg" << "audio/x-mp3" << "audio/x-wav" << "application/ogg" << "video/mp4" << "video/quicktime" << "image/gif" << "image/jpeg" << "image/png" << "image/x-tga" << "image/x-bmp" << "image/svg+xml" << "image/tiff" << "image/x-xcf-gimp" << "image/x-vnd.adobe.photoshop" << "image/x-pcx" << "image/x-exr" << "video/mlt-playlist" << "audio/x-flac" << "audio/mp4" << "video/x-matroska" << "audio/x-matroska";
 
         QString allExtensions;
         foreach(const QString& mimeType, mimeTypes) {
@@ -591,7 +604,7 @@ void ProjectList::slotAddClip(const QList <QUrl> givenList, QString group)
 
     QString groupId;
     if (group.isEmpty()) {
-        ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+        ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
         if (item && !item->isGroup()) {
             while (item->parent()) {
                 item = static_cast <ProjectItem*>(item->parent());
@@ -606,18 +619,23 @@ void ProjectList::slotAddClip(const QList <QUrl> givenList, QString group)
     m_doc->slotAddClipList(list, group, groupId);
 }
 
-void ProjectList::slotRemoveInvalidClip(const QString &id)
+void ProjectList::slotRemoveInvalidClip(const QString &id, bool replace)
 {
     ProjectItem *item = getItemById(id);
     if (item) {
         const QString path = item->referencedClip()->fileURL().path();
-        if (!path.isEmpty()) KMessageBox::sorry(this, i18n("Clip <b>%1</b><br>is invalid, will be removed from project.", path));
+        if (!path.isEmpty()) {
+            if (replace) KMessageBox::sorry(this, i18n("Clip <b>%1</b><br>is invalid, will be removed from project.", path));
+            else {
+                if (KMessageBox::questionYesNo(this, i18n("Clip <b>%1</b><br>is missing or invalid. Remove it from project?", path), i18n("Invalid clip")) == KMessageBox::Yes) replace = true;
+            }
+        }
         QList <QString> ids;
         ids << id;
-        m_doc->deleteProjectClip(ids);
+        if (replace) m_doc->deleteProjectClip(ids);
     }
     if (!m_infoQueue.isEmpty()) QTimer::singleShot(300, this, SLOT(slotProcessNextClipInQueue()));
-    else listView->setEnabled(true);
+    else m_listView->setEnabled(true);
 }
 
 void ProjectList::slotAddColorClip()
@@ -634,7 +652,7 @@ void ProjectList::slotAddColorClip()
 
         QString group;
         QString groupId;
-        ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+        ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
         if (item && !item->isGroup()) {
             while (item->parent()) {
                 item = static_cast <ProjectItem*>(item->parent());
@@ -663,7 +681,7 @@ void ProjectList::slotAddSlideshowClip()
 
         QString group;
         QString groupId;
-        ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+        ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
         if (item && !item->isGroup()) {
             while (item->parent()) {
                 item = static_cast <ProjectItem*>(item->parent());
@@ -685,7 +703,7 @@ void ProjectList::slotAddTitleClip()
 {
     QString group;
     QString groupId;
-    ProjectItem *item = static_cast <ProjectItem*>(listView->currentItem());
+    ProjectItem *item = static_cast <ProjectItem*>(m_listView->currentItem());
     if (item && !item->isGroup()) {
         while (item->parent()) {
             item = static_cast <ProjectItem*>(item->parent());
@@ -702,8 +720,8 @@ void ProjectList::slotAddTitleClip()
 
 void ProjectList::setDocument(KdenliveDoc *doc)
 {
-    listView->blockSignals(true);
-    listView->clear();
+    m_listView->blockSignals(true);
+    m_listView->clear();
     emit clipSelected(NULL);
     m_thumbnailQueue.clear();
     m_infoQueue.clear();
@@ -712,7 +730,7 @@ void ProjectList::setDocument(KdenliveDoc *doc)
     QMapIterator<QString, QString> f(flist);
     while (f.hasNext()) {
         f.next();
-        (void) new ProjectItem(listView, QStringList() << QString() << f.value(), f.key());
+        (void) new ProjectItem(m_listView, QStringList() << QString() << f.value(), f.key());
     }
 
     QList <DocClipBase*> list = doc->clipManager()->documentClipList();
@@ -724,9 +742,9 @@ void ProjectList::setDocument(KdenliveDoc *doc)
     m_timecode = doc->timecode();
     m_commandStack = doc->commandStack();
     m_doc = doc;
-    QTreeWidgetItem *first = listView->topLevelItem(0);
-    if (first) listView->setCurrentItem(first);
-    listView->blockSignals(false);
+    QTreeWidgetItem *first = m_listView->topLevelItem(0);
+    if (first) m_listView->setCurrentItem(first);
+    m_listView->blockSignals(false);
     m_toolbar->setEnabled(true);
 }
 
@@ -736,7 +754,7 @@ QDomElement ProjectList::producersList()
     QDomElement prods = doc.createElement("producerlist");
     doc.appendChild(prods);
     kDebug() << "////////////  PRO LIST BUILD PRDSLIST ";
-    QTreeWidgetItemIterator it(listView);
+    QTreeWidgetItemIterator it(m_listView);
     while (*it) {
         if (!((ProjectItem *)(*it))->isGroup())
             prods.appendChild(doc.importNode(((ProjectItem *)(*it))->toXml(), true));
@@ -785,9 +803,9 @@ void ProjectList::slotRefreshClipThumbnail(ProjectItem *item, bool update)
         QPixmap pix;
         if (clip->clipType() == AUDIO) pix = KIcon("audio-x-generic").pixmap(QSize(width, height));
         else pix = item->referencedClip()->thumbProducer()->extractImage(item->referencedClip()->getClipThumbFrame(), width, height);
-        listView->blockSignals(true);
+        m_listView->blockSignals(true);
         item->setIcon(0, pix);
-        listView->blockSignals(false);
+        m_listView->blockSignals(false);
         m_doc->cachePixmap(item->getClipHash(), pix);
         if (update) emit projectModified();
         if (!m_thumbnailQueue.isEmpty()) QTimer::singleShot(300, this, SLOT(slotProcessNextThumbnail()));
@@ -798,32 +816,36 @@ void ProjectList::slotReplyGetFileProperties(const QString &clipId, Mlt::Produce
 {
     ProjectItem *item = getItemById(clipId);
     if (item && producer) {
-        listView->blockSignals(true);
+        m_listView->blockSignals(true);
         item->setProperties(properties, metadata);
         Q_ASSERT_X(item->referencedClip(), "void ProjectList::slotReplyGetFileProperties", QString("Item with groupName %1 does not have a clip associated").arg(item->groupName()).toLatin1());
         if (replace) item->referencedClip()->setProducer(producer);
-        emit receivedClipDuration(clipId, item->clipMaxDuration());
-        listView->blockSignals(false);
+        else {
+            // Check if duration changed.
+            emit receivedClipDuration(clipId);
+            delete producer;
+        }
+        m_listView->blockSignals(false);
     } else kDebug() << "////////  COULD NOT FIND CLIP TO UPDATE PRPS...";
     if (!m_infoQueue.isEmpty()) QTimer::singleShot(300, this, SLOT(slotProcessNextClipInQueue()));
-    else listView->setEnabled(true);
+    else m_listView->setEnabled(true);
 }
 
 void ProjectList::slotReplyGetImage(const QString &clipId, const QPixmap &pix)
 {
     ProjectItem *item = getItemById(clipId);
     if (item) {
-        listView->blockSignals(true);
+        m_listView->blockSignals(true);
         item->setIcon(0, pix);
         m_doc->cachePixmap(item->getClipHash(), pix);
-        listView->blockSignals(false);
+        m_listView->blockSignals(false);
     }
 }
 
 ProjectItem *ProjectList::getItemById(const QString &id)
 {
     ProjectItem *item;
-    QTreeWidgetItemIterator it(listView);
+    QTreeWidgetItemIterator it(m_listView);
     while (*it) {
         item = static_cast<ProjectItem *>(*it);
         if (item->clipId() == id && item->clipType() != FOLDER)
@@ -836,7 +858,7 @@ ProjectItem *ProjectList::getItemById(const QString &id)
 ProjectItem *ProjectList::getFolderItemById(const QString &id)
 {
     ProjectItem *item;
-    QTreeWidgetItemIterator it(listView);
+    QTreeWidgetItemIterator it(m_listView);
     while (*it) {
         item = static_cast<ProjectItem *>(*it);
         if (item->clipId() == id && item->clipType() == FOLDER)
@@ -850,8 +872,8 @@ void ProjectList::slotSelectClip(const QString &ix)
 {
     ProjectItem *clip = getItemById(ix);
     if (clip) {
-        listView->setCurrentItem(clip);
-        listView->scrollToItem(clip);
+        m_listView->setCurrentItem(clip);
+        m_listView->scrollToItem(clip);
         m_editAction->setEnabled(true);
         m_deleteAction->setEnabled(true);
         m_reloadAction->setEnabled(true);