]> git.sesse.net Git - kdenlive/blobdiff - src/clipitem.cpp
krazy code fixes
[kdenlive] / src / clipitem.cpp
index f78324034786bf6808584cde1af8768d268ec2d7..3f7c3d8dafbdcad337f0a9484a81fa1f7ae975e4 100644 (file)
 #include "kthumb.h"
 
 
-ClipItem::ClipItem(DocClipBase *clip, ItemInfo info, double fps, bool generateThumbs)
-        : AbstractClipItem(info, QRectF(), fps), m_clip(clip), m_resizeMode(NONE), m_grabPoint(0), m_maxTrack(0), m_hasThumbs(false), startThumbTimer(NULL), endThumbTimer(NULL), audioThumbWasDrawn(false), m_opacity(1.0), m_timeLine(0), m_startThumbRequested(false), m_endThumbRequested(false), m_startFade(0), m_endFade(0), m_hover(false), m_selectedEffect(-1), m_speed(1.0), framePixelWidth(0), m_startPix(QPixmap()), m_endPix(QPixmap()) {
+ClipItem::ClipItem(DocClipBase *clip, ItemInfo info, double fps, double speed, bool generateThumbs)
+        : AbstractClipItem(info, QRectF(), fps), m_clip(clip), m_resizeMode(NONE), m_grabPoint(0), m_maxTrack(0), m_hasThumbs(false), startThumbTimer(NULL), endThumbTimer(NULL), audioThumbWasDrawn(false), m_opacity(1.0), m_timeLine(0), m_startThumbRequested(false), m_endThumbRequested(false), m_startFade(0), m_endFade(0), m_hover(false), m_selectedEffect(-1), m_speed(speed), framePixelWidth(0), m_startPix(QPixmap()), m_endPix(QPixmap()) {
     setZValue(1);
     setRect(0, 0, (info.endPos - info.startPos).frames(fps) - 0.02, (double)(KdenliveSettings::trackheight() - 2));
     setPos(info.startPos.frames(fps), (double)(info.track * KdenliveSettings::trackheight()) + 1);
 
-    m_clipName = clip->name();
+    if (m_speed == 1.0) m_clipName = clip->name();
+    else {
+        m_clipName = clip->name() + " - " + QString::number(m_speed * 100, 'f', 0) + '%';
+        m_cropDuration = m_cropDuration * m_speed;
+    }
     m_producer = clip->getId();
     m_clipType = clip->clipType();
     m_cropStart = info.cropStart;
@@ -108,12 +112,12 @@ ClipItem::~ClipItem() {
 }
 
 ClipItem *ClipItem::clone(ItemInfo info) const {
-    ClipItem *duplicate = new ClipItem(m_clip, info, m_fps);
-    if (info.cropStart == cropStart()) duplicate->slotSetStartThumb(m_startPix);
+    ClipItem *duplicate = new ClipItem(m_clip, info, m_fps, m_speed);
+    if (info.cropStart == m_cropStart) duplicate->slotSetStartThumb(m_startPix);
     if (info.cropStart + (info.endPos - info.startPos) == m_cropStart + m_cropDuration) duplicate->slotSetEndThumb(m_endPix);
     kDebug() << "// CLoning clip: " << (info.cropStart + (info.endPos - info.startPos)).frames(m_fps) << ", CURRENT end: " << (cropStart() + duration()).frames(m_fps);
     duplicate->setEffectList(m_effectList.clone());
-    duplicate->setSpeed(m_speed);
+    //duplicate->setSpeed(m_speed);
     return duplicate;
 }
 
@@ -144,22 +148,48 @@ void ClipItem::initEffect(QDomElement effect) {
             QString def = e.attribute("default");
             // Effect has a keyframe type parameter, we need to set the values
             if (e.attribute("keyframes").isEmpty()) {
-                e.setAttribute("keyframes", QString::number(m_cropStart.frames(m_fps)) + ":" + def + ";" + QString::number((m_cropStart + m_cropDuration).frames(m_fps)) + ":" + def);
+                e.setAttribute("keyframes", QString::number(m_cropStart.frames(m_fps)) + ':' + def + ';' + QString::number((m_cropStart + m_cropDuration).frames(m_fps)) + ':' + def);
                 //kDebug() << "///// EFFECT KEYFRAMES INITED: " << e.attribute("keyframes");
                 break;
             }
         }
     }
 
-    if (effect.attribute("tag") == "volume") {
-        if (effect.attribute("id") == "fadeout") {
+    if (effect.attribute("tag") == "volume" || effect.attribute("tag") == "brightness") {
+        if (effect.attribute("id") == "fadeout" || effect.attribute("id") == "fade_to_black") {
             int end = (duration() + cropStart()).frames(m_fps);
-            int start = end - EffectsList::parameter(effect, "in").toInt();
+            int start = end;
+            if (effect.attribute("id") == "fadeout") {
+                if (m_effectList.hasEffect(QString(), "fade_to_black") == -1) {
+                    start -= EffectsList::parameter(effect, "in").toInt();
+                } else {
+                    QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fade_to_black");
+                    start -= EffectsList::parameter(fadeout, "out").toInt() - EffectsList::parameter(fadeout, "in").toInt();
+                }
+            } else if (effect.attribute("id") == "fade_to_black") {
+                if (m_effectList.hasEffect(QString(), "fadeout") == -1) {
+                    start -= EffectsList::parameter(effect, "in").toInt();
+                } else {
+                    QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fadeout");
+                    start -= EffectsList::parameter(fadeout, "out").toInt() - EffectsList::parameter(fadeout, "in").toInt();
+                }
+            }
             EffectsList::setParameter(effect, "in", QString::number(start));
             EffectsList::setParameter(effect, "out", QString::number(end));
-        } else if (effect.attribute("id") == "fadein") {
+        } else if (effect.attribute("id") == "fadein" || effect.attribute("id") == "fade_from_black") {
             int start = cropStart().frames(m_fps);
-            int end = start + EffectsList::parameter(effect, "out").toInt();
+            int end = start;
+            if (effect.attribute("id") == "fadein") {
+                if (m_effectList.hasEffect(QString(), "fade_from_black") == -1)
+                    end += EffectsList::parameter(effect, "out").toInt();
+                else
+                    end += EffectsList::parameter(m_effectList.getEffectByTag(QString(), "fade_from_black"), "out").toInt();
+            } else if (effect.attribute("id") == "fade_from_black") {
+                if (m_effectList.hasEffect(QString(), "fadein") == -1)
+                    end += EffectsList::parameter(effect, "out").toInt();
+                else
+                    end += EffectsList::parameter(m_effectList.getEffectByTag(QString(), "fadein"), "out").toInt();
+            }
             EffectsList::setParameter(effect, "in", QString::number(start));
             EffectsList::setParameter(effect, "out", QString::number(end));
         }
@@ -171,7 +201,7 @@ bool ClipItem::checkKeyFrames() {
     for (int ix = 0; ix < m_effectList.count(); ix ++) {
         QString kfr = keyframes(ix);
         if (!kfr.isEmpty()) {
-            const QStringList keyframes = kfr.split(";", QString::SkipEmptyParts);
+            const QStringList keyframes = kfr.split(';', QString::SkipEmptyParts);
             QStringList newKeyFrames;
             bool cutKeyFrame = false;
             bool modified = false;
@@ -179,9 +209,9 @@ bool ClipItem::checkKeyFrames() {
             double lastValue = -1;
             int start = m_cropStart.frames(m_fps);
             int end = (m_cropStart + m_cropDuration).frames(m_fps);
-            foreach(const QString str, keyframes) {
-                int pos = str.section(":", 0, 0).toInt();
-                double val = str.section(":", 1, 1).toDouble();
+            foreach(const QString &str, keyframes) {
+                int pos = str.section(':', 0, 0).toInt();
+                double val = str.section(':', 1, 1).toDouble();
                 if (pos - start < 0) {
                     // a keyframe is defined before the start of the clip
                     cutKeyFrame = true;
@@ -191,7 +221,7 @@ bool ClipItem::checkKeyFrames() {
                         int diff = pos - lastPos;
                         double ratio = (double)(start - lastPos) / diff;
                         double newValue = lastValue + (val - lastValue) * ratio;
-                        newKeyFrames.append(QString::number(start) + ":" + QString::number(newValue));
+                        newKeyFrames.append(QString::number(start) + ':' + QString::number(newValue));
                         modified = true;
                     }
                     cutKeyFrame = false;
@@ -203,12 +233,12 @@ bool ClipItem::checkKeyFrames() {
                         if (diff != 0) {
                             double ratio = (double)(end - lastPos) / diff;
                             double newValue = lastValue + (val - lastValue) * ratio;
-                            newKeyFrames.append(QString::number(end) + ":" + QString::number(newValue));
+                            newKeyFrames.append(QString::number(end) + ':' + QString::number(newValue));
                             modified = true;
                         }
                         break;
                     } else {
-                        newKeyFrames.append(QString::number(pos) + ":" + QString::number(val));
+                        newKeyFrames.append(QString::number(pos) + ':' + QString::number(val));
                     }
                 }
                 lastPos = pos;
@@ -239,10 +269,10 @@ void ClipItem::setKeyframes(const int ix, const QString keyframes) {
                 m_keyframeFactor = 100.0 / (max - min);
                 m_keyframeDefault = e.attribute("default").toDouble();
                 // parse keyframes
-                const QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
-                foreach(const QString str, keyframes) {
-                    int pos = str.section(":", 0, 0).toInt();
-                    double val = str.section(":", 1, 1).toDouble();
+                const QStringList keyframes = e.attribute("keyframes").split(';', QString::SkipEmptyParts);
+                foreach(const QString &str, keyframes) {
+                    int pos = str.section(':', 0, 0).toInt();
+                    double val = str.section(':', 1, 1).toDouble();
                     m_keyframes[pos] = val;
                 }
                 update();
@@ -268,10 +298,10 @@ void ClipItem::setSelectedEffect(const int ix) {
                 m_keyframeFactor = 100.0 / (max - min);
                 m_keyframeDefault = e.attribute("default").toDouble();
                 // parse keyframes
-                const QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
-                foreach(const QString str, keyframes) {
-                    int pos = str.section(":", 0, 0).toInt();
-                    double val = str.section(":", 1, 1).toDouble();
+                const QStringList keyframes = e.attribute("keyframes").split(';', QString::SkipEmptyParts);
+                foreach(const QString &str, keyframes) {
+                    int pos = str.section(':', 0, 0).toInt();
+                    double val = str.section(':', 1, 1).toDouble();
                     m_keyframes[pos] = val;
                 }
                 update();
@@ -314,7 +344,7 @@ void ClipItem::updateKeyframeEffect() {
                 double x1;
                 double y1;
                 while (i != m_keyframes.constEnd()) {
-                    keyframes.append(QString::number(i.key()) + ":" + QString::number(i.value()) + ";");
+                    keyframes.append(QString::number(i.key()) + ':' + QString::number(i.value()) + ';');
                     ++i;
                 }
             }
@@ -462,7 +492,9 @@ DocClipBase *ClipItem::baseClip() const {
 }
 
 QDomElement ClipItem::xml() const {
-    return m_clip->toXML();
+    QDomElement xml = m_clip->toXML();
+    if (m_speed != 1.0) xml.setAttribute("speed", m_speed);
+    return xml;
 }
 
 int ClipItem::clipType() const {
@@ -473,6 +505,10 @@ QString ClipItem::clipName() const {
     return m_clipName;
 }
 
+void ClipItem::setClipName(const QString &name) {
+    m_clipName = name;
+}
+
 const QString &ClipItem::clipProducer() const {
     return m_producer;
 }
@@ -729,6 +765,8 @@ void ClipItem::paint(QPainter *painter,
 
 
 OPERATIONTYPE ClipItem::operationMode(QPointF pos) {
+    if (isItemLocked()) return NONE;
+
     if (isSelected()) {
         m_editedKeyframe = mouseOverKeyFrames(pos);
         if (m_editedKeyframe != -1) return KEYFRAME;
@@ -929,6 +967,7 @@ void ClipItem::mouseReleaseEvent(QGraphicsSceneMouseEvent * event) {
 //virtual
 void ClipItem::hoverEnterEvent(QGraphicsSceneHoverEvent *e) {
     //if (e->pos().x() < 20) m_hover = true;
+    if (isItemLocked()) return;
     m_hover = true;
     QRectF r = boundingRect();
     double width = 35 / projectScene()->scale();
@@ -940,6 +979,7 @@ void ClipItem::hoverEnterEvent(QGraphicsSceneHoverEvent *e) {
 
 //virtual
 void ClipItem::hoverLeaveEvent(QGraphicsSceneHoverEvent *) {
+    if (isItemLocked()) return;
     m_hover = false;
     QRectF r = boundingRect();
     double width = 35 / projectScene()->scale();
@@ -955,10 +995,12 @@ void ClipItem::resizeStart(int posx, double speed) {
     if (posx == startPos().frames(m_fps)) return;
     const int previous = cropStart().frames(m_fps);
     AbstractClipItem::resizeStart(posx, m_speed);
-    checkEffectsKeyframesPos(previous, cropStart().frames(m_fps), true);
-    if (m_hasThumbs && KdenliveSettings::videothumbnails()) {
-        /*connect(m_clip->thumbProducer(), SIGNAL(thumbReady(int, QPixmap)), this, SLOT(slotThumbReady(int, QPixmap)));*/
-        startThumbTimer->start(100);
+    if ((int) cropStart().frames(m_fps) != previous) {
+        checkEffectsKeyframesPos(previous, cropStart().frames(m_fps), true);
+        if (m_hasThumbs && KdenliveSettings::videothumbnails()) {
+            /*connect(m_clip->thumbProducer(), SIGNAL(thumbReady(int, QPixmap)), this, SLOT(slotThumbReady(int, QPixmap)));*/
+            startThumbTimer->start(150);
+        }
     }
 }
 
@@ -969,10 +1011,12 @@ void ClipItem::resizeEnd(int posx, double speed, bool updateKeyFrames) {
     //kDebug() << "// NEW POS: " << posx << ", OLD END: " << endPos().frames(m_fps);
     const int previous = (cropStart() + duration()).frames(m_fps);
     AbstractClipItem::resizeEnd(posx, m_speed);
-    if (updateKeyFrames) checkEffectsKeyframesPos(previous, (cropStart() + duration()).frames(m_fps), false);
-    if (m_hasThumbs && KdenliveSettings::videothumbnails()) {
-        /*connect(m_clip->thumbProducer(), SIGNAL(thumbReady(int, QPixmap)), this, SLOT(slotThumbReady(int, QPixmap)));*/
-        endThumbTimer->start(100);
+    if ((int)(cropStart() + duration()).frames(m_fps) != previous) {
+        if (updateKeyFrames) checkEffectsKeyframesPos(previous, (cropStart() + duration()).frames(m_fps), false);
+        if (m_hasThumbs && KdenliveSettings::videothumbnails()) {
+            /*connect(m_clip->thumbProducer(), SIGNAL(thumbReady(int, QPixmap)), this, SLOT(slotThumbReady(int, QPixmap)));*/
+            endThumbTimer->start(150);
+        }
     }
 }
 
@@ -985,13 +1029,13 @@ void ClipItem::checkEffectsKeyframesPos(const int previous, const int current, b
             QDomElement e = params.item(i).toElement();
             if (e.attribute("type") == "keyframe") {
                 // parse keyframes and adjust values
-                const QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
+                const QStringList keyframes = e.attribute("keyframes").split(';', QString::SkipEmptyParts);
                 QMap <int, double> kfr;
                 int pos;
                 double val;
-                foreach(const QString str, keyframes) {
-                    pos = str.section(":", 0, 0).toInt();
-                    val = str.section(":", 1, 1).toDouble();
+                foreach(const QString &str, keyframes) {
+                    pos = str.section(':', 0, 0).toInt();
+                    val = str.section(':', 1, 1).toDouble();
                     if (pos == previous) kfr[current] = val;
                     else {
                         if (fromStart && pos >= current) kfr[pos] = val;
@@ -1001,7 +1045,7 @@ void ClipItem::checkEffectsKeyframesPos(const int previous, const int current, b
                 QString newkfr;
                 QMap<int, double>::const_iterator k = kfr.constBegin();
                 while (k != kfr.constEnd()) {
-                    newkfr.append(QString::number(k.key()) + ":" + QString::number(k.value()) + ";");
+                    newkfr.append(QString::number(k.key()) + ':' + QString::number(k.value()) + ';');
                     ++k;
                 }
                 e.setAttribute("keyframes", newkfr);
@@ -1042,7 +1086,7 @@ QVariant ClipItem::itemChange(GraphicsItemChange change, const QVariant &value)
                         return pos();
                     }
                     if (forwardMove) {
-                        offset = qMax(offset, (int)(newPos.x() - (static_cast < AbstractClipItem* >(items.at(i))->startPos() - m_cropDuration).frames(m_fps)));
+                        offset = qMax(offset, (int)(newPos.x() - (static_cast < AbstractClipItem* >(items.at(i))->startPos() - cropDuration()).frames(m_fps)));
                     } else {
                         offset = qMax(offset, (int)((static_cast < AbstractClipItem* >(items.at(i))->endPos().frames(m_fps)) - newPos.x()));
                     }
@@ -1058,7 +1102,10 @@ QVariant ClipItem::itemChange(GraphicsItemChange change, const QVariant &value)
                         QList<QGraphicsItem*> subitems = scene()->items(sceneShape, Qt::IntersectsItemShape);
                         subitems.removeAll(this);
                         for (int j = 0; j < subitems.count(); j++) {
-                            if (subitems.at(j)->type() == type()) return pos();
+                            if (subitems.at(j)->type() == type()) {
+                                m_startPos = GenTime((int) pos().x(), m_fps);
+                                return pos();
+                            }
                         }
                     }
 
@@ -1107,7 +1154,9 @@ void ClipItem::setEffectAt(int ix, QDomElement effect) {
     m_effectList.insert(ix, effect);
     m_effectList.removeAt(ix + 1);
     m_effectNames = m_effectList.effectNames().join(" / ");
-    if (effect.attribute("id") == "fadein" || effect.attribute("id") == "fadeout") update(boundingRect());
+    QString id = effect.attribute("id");
+    if (id == "fadein" || id == "fadeout" || id == "fade_from_black" || id == "fade_to_black")
+        update(boundingRect());
     else {
         QRectF r = boundingRect();
         r.setHeight(20);
@@ -1126,6 +1175,7 @@ EffectsParameterList ClipItem::addEffect(QDomElement effect, bool animate) {
     EffectsParameterList parameters;
     parameters.addParam("tag", effect.attribute("tag"));
     parameters.addParam("kdenlive_ix", effect.attribute("kdenlive_ix"));
+    if (effect.hasAttribute("src")) parameters.addParam("src", effect.attribute("src"));
 
     QString state = effect.attribute("disabled");
     if (!state.isEmpty()) {
@@ -1158,12 +1208,44 @@ EffectsParameterList ClipItem::addEffect(QDomElement effect, bool animate) {
                 // check if it is a fade effect
                 if (effectId == "fadein") {
                     needRepaint = true;
-                    if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
-                    else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
+                    if (m_effectList.hasEffect(QString(), "fade_from_black") == -1) {
+                        if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
+                        else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
+                    } else {
+                        QDomElement fadein = m_effectList.getEffectByTag(QString(), "fade_from_black");
+                        if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
+                        else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
+                    }
+                } else if (effectId == "fade_from_black") {
+                    needRepaint = true;
+                    if (m_effectList.hasEffect(QString(), "fadein") == -1) {
+                        if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
+                        else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
+                    } else {
+                        QDomElement fadein = m_effectList.getEffectByTag(QString(), "fadein");
+                        if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
+                        else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
+                    }
                 } else if (effectId == "fadeout") {
                     needRepaint = true;
-                    if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
-                    else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
+                    if (m_effectList.hasEffect(QString(), "fade_to_black") == -1) {
+                        if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
+                        else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
+                    } else {
+                        QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fade_to_black");
+                        if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
+                        else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
+                    }
+                } else if (effectId == "fade_to_black") {
+                    needRepaint = true;
+                    if (m_effectList.hasEffect(QString(), "fadeout") == -1) {
+                        if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
+                        else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
+                    } else {
+                        QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fadeout");
+                        if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
+                        else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
+                    }
                 }
             } else {
                 parameters.addParam(e.attribute("name"), QString::number(e.attribute("value").toDouble() / f));
@@ -1193,7 +1275,7 @@ EffectsParameterList ClipItem::getEffectArgs(QDomElement effect) {
     parameters.addParam("tag", effect.attribute("tag"));
     parameters.addParam("kdenlive_ix", effect.attribute("kdenlive_ix"));
     parameters.addParam("id", effect.attribute("id"));
-
+    if (effect.hasAttribute("src")) parameters.addParam("src", effect.attribute("src"));
     QString state = effect.attribute("disabled");
     if (!state.isEmpty()) {
         parameters.addParam("disabled", state);
@@ -1211,7 +1293,7 @@ EffectsParameterList ClipItem::getEffectArgs(QDomElement effect) {
             parameters.addParam("factor", e.attribute("factor", "1"));
             parameters.addParam("starttag", e.attribute("starttag", "start"));
             parameters.addParam("endtag", e.attribute("endtag", "end"));
-        } else if (e.attribute("namedesc").contains(";")) {
+        } else if (e.attribute("namedesc").contains(';')) {
             QString format = e.attribute("format");
             QStringList separators = format.split("%d", QString::SkipEmptyParts);
             QStringList values = e.attribute("value").split(QRegExp("[,:;x]"));
@@ -1238,18 +1320,25 @@ EffectsParameterList ClipItem::getEffectArgs(QDomElement effect) {
 void ClipItem::deleteEffect(QString index) {
     bool needRepaint = false;
     QString ix;
+
     for (int i = 0; i < m_effectList.size(); ++i) {
         ix = m_effectList.at(i).attribute("kdenlive_ix");
         if (ix == index) {
-            if (m_effectList.at(i).attribute("id") == "fadein") {
+            QString effectId = m_effectList.at(i).attribute("id");
+            if ((effectId == "fadein" && hasEffect(QString(), "fade_from_black") == -1) ||
+                    (effectId == "fade_from_black" && hasEffect(QString(), "fadein") == -1)) {
                 m_startFade = 0;
                 needRepaint = true;
-            } else if (m_effectList.at(i).attribute("id") == "fadeout") {
+            } else if ((effectId == "fadeout" && hasEffect(QString(), "fade_to_black") == -1) ||
+                       (effectId == "fade_to_black" && hasEffect(QString(), "fadeout") == -1)) {
                 m_endFade = 0;
                 needRepaint = true;
             }
             m_effectList.removeAt(i);
-        } else if (ix.toInt() > index.toInt()) m_effectList[i].setAttribute("kdenlive_ix", ix.toInt() - 1);
+            i--;
+        } else if (ix.toInt() > index.toInt()) {
+            m_effectList[i].setAttribute("kdenlive_ix", ix.toInt() - 1);
+        }
     }
     m_effectNames = m_effectList.effectNames().join(" / ");
     if (needRepaint) update(boundingRect());
@@ -1295,7 +1384,8 @@ void ClipItem::dropEvent(QGraphicsSceneDragDropEvent * event) {
 
 //virtual
 void ClipItem::dragEnterEvent(QGraphicsSceneDragDropEvent *event) {
-    event->setAccepted(event->mimeData()->hasFormat("kdenlive/effectslist"));
+    if (isItemLocked()) event->setAccepted(false);
+    else event->setAccepted(event->mimeData()->hasFormat("kdenlive/effectslist"));
 }
 
 void ClipItem::dragLeaveEvent(QGraphicsSceneDragDropEvent *event) {