Don't allow clip moving over another one

[kdenlive] / src / effectslistview.cpp

diff --git a/src/effectslistview.cpp b/src/effectslistview.cpp
index 36a26afdeeffd5680707407a42ef654bf130dc05..8c1d88648e08f8ffb3d7ff313771daa07e9b5cbf 100644 (file)
--- a/src/effectslistview.cpp
+++ b/src/effectslistview.cpp
@@ -101,27 +101,27 @@ void EffectsListView::reloadEffectList() {
 void EffectsListView::slotRemoveEffect() {
     QListWidgetItem *item = m_effectsList->currentItem();
     QString effectId = item->text();
-    QString path = KStandardDirs::locateLocal("data", "kdenlive/effects/", true);
+    QString path = KStandardDirs::locateLocal("appdata", "effects/", true);
 
     QDir directory = QDir(path);
-    QStringList fileList = directory.entryList(QDir::Files);
+    QStringList filter;
+    filter << "*.xml";
+    const QStringList fileList = directory.entryList(filter, QDir::Files);
     QString itemName;
-    foreach(QString filename, fileList) {
+    foreach(const QString filename, fileList) {
         itemName = KUrl(path + filename).path();
-        if (itemName.endsWith(".xml")) {
-            QDomDocument doc;
-            QFile file(itemName);
-            doc.setContent(&file, false);
-            file.close();
-            QDomNodeList effects = doc.elementsByTagName("effect");
-            if (effects.count() != 1) {
-                kDebug() << "More than one effect in file " << itemName << ", NOT SUPPORTED YET";
-            } else {
-                QDomElement e = effects.item(0).toElement();
-                if (e.attribute("id") == effectId) {
-                    QFile::remove(itemName);
-                    break;
-                }
+        QDomDocument doc;
+        QFile file(itemName);
+        doc.setContent(&file, false);
+        file.close();
+        QDomNodeList effects = doc.elementsByTagName("effect");
+        if (effects.count() != 1) {
+            kDebug() << "More than one effect in file " << itemName << ", NOT SUPPORTED YET";
+        } else {
+            QDomElement e = effects.item(0).toElement();
+            if (e.attribute("id") == effectId) {
+                QFile::remove(itemName);
+                break;
             }
         }
     }