From 796555a4f812a467129323f13b8849204e3e0492 Mon Sep 17 00:00:00 2001
From: Jean-Baptiste Mardelle <jb@kdenlive.org>
Date: Tue, 19 Jan 2010 09:14:54 +0000
Subject: [PATCH] Fix crash on custom effect:
 http://kdenlive.org/mantis/view.php?id=1391

svn path=/trunk/kdenlive/; revision=4238
---
 src/mainwindow.cpp | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 5b6782d6..4d089edd 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -584,19 +584,24 @@ void MainWindow::readProperties(const KConfigGroup &config)
 
 void MainWindow::slotReloadEffects()
 {
-    kDebug() << "START RELOAD; COUNR: " << m_customEffectsMenu->actions().count();
     m_customEffectsMenu->clear();
-    kDebug() << "START RELOAD; CLR: " << m_customEffectsMenu->actions().count();
     initEffects::parseCustomEffectsFile();
-    const QStringList effects = customEffects.effectNames();
-    kDebug() << "NEW EFFS: " << effects;
     QAction *action;
-    if (effects.isEmpty()) m_customEffectsMenu->setEnabled(false);
-    else m_customEffectsMenu->setEnabled(true);
+    QStringList effectInfo;
+    QMap<QString, QStringList> effectsList;
 
-    foreach(const QString &name, effects) {
-        action = new QAction(name, this);
-        action->setData(name);
+    for (int ix = 0; ix < customEffects.count(); ix++) {
+        effectInfo = customEffects.effectIdInfo(ix);
+        effectsList.insert(effectInfo.at(0).toLower(), effectInfo);
+    }
+    if (effectsList.isEmpty()) {
+        m_customEffectsMenu->setEnabled(false);
+        return;
+    } else m_customEffectsMenu->setEnabled(true);
+
+    foreach(const QStringList &value, effectsList) {
+        action = new QAction(value.at(0), this);
+        action->setData(value);
         m_customEffectsMenu->addAction(action);
     }
     m_effectList->reloadEffectList();
@@ -1960,7 +1965,7 @@ void MainWindow::connectDocument(TrackView *trackView, KdenliveDoc *doc)   //cha
             disconnect(m_activeTimeline, SIGNAL(zoneMoved(int, int)), this, SLOT(slotZoneMoved(int, int)));
             disconnect(m_projectList, SIGNAL(loadingIsOver()), m_activeTimeline->projectView(), SLOT(slotUpdateAllThumbs()));
             disconnect(m_projectList, SIGNAL(displayMessage(const QString&, int)), this, SLOT(slotGotProgressInfo(const QString&, int)));
-	    disconnect(m_projectList, SIGNAL(clipNeedsReload(const QString&, bool)), m_activeTimeline->projectView(), SLOT(slotUpdateClip(const QString &, bool)));
+            disconnect(m_projectList, SIGNAL(clipNeedsReload(const QString&, bool)), m_activeTimeline->projectView(), SLOT(slotUpdateClip(const QString &, bool)));
             m_effectStack->clear();
         }
         //m_activeDocument->setRenderer(NULL);
-- 
2.39.2