From: Jean-Baptiste Mardelle <jb@kdenlive.org>
Date: Fri, 10 Jun 2011 23:35:47 +0000 (+0000)
Subject: Check document for invalid (overlapping) transitions
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=659aee5913ceae2bc046cc9491a9c974824f2cf7;p=kdenlive

Check document for invalid (overlapping) transitions

svn path=/trunk/kdenlive/; revision=5697
---

diff --git a/src/documentvalidator.cpp b/src/documentvalidator.cpp
index 1f66b5e4..e6f7317f 100644
--- a/src/documentvalidator.cpp
+++ b/src/documentvalidator.cpp
@@ -28,6 +28,8 @@
 
 #include <QFile>
 #include <QColor>
+#include <QBitmap>
+
 
 DocumentValidator::DocumentValidator(QDomDocument doc):
         m_doc(doc),
@@ -143,9 +145,44 @@ bool DocumentValidator::validate(const double currentVersion)
                 }
             }
         }
+        
+        // Make sure transitions do not overlap
+        QDomNodeList transitions = m_doc.elementsByTagName("transition");
+        QPolygon scenelist;
+        QStringList overlappingTransitions;
+        for (int i = 0; i < transitions.count(); i++) {
+            QDomElement t = transitions.at(i).toElement();
+            QDomNodeList props = t.elementsByTagName("property");
+            bool testTransition = true;
+            int track = -1;
+            for (int k = 0; k < props.count(); k++) {
+                QDomElement p = props.at(k).toElement();
+                QString name = p.attribute("name");
+                if (name == "mlt_service" && p.firstChild().nodeValue() == "mix") testTransition = false;
+                else if (name == "b_track") track = p.firstChild().nodeValue().toInt();
+            }
+            if (testTransition) {
+                QRect r(t.attribute("in").toInt(), 3 * track, t.attribute("out").toInt() - t.attribute("in").toInt(), 1);
+                QPolygon p(r);
+                if (scenelist.intersected(p).isEmpty()) {
+                    scenelist = scenelist.united(p);
+                }
+                else {
+                    // Transition is overlapping, should be removed
+                    overlappingTransitions << t.attribute("id");
+                    tractor.removeChild(t);
+                    i--;
+                }
+            }
+        }
+        if (!overlappingTransitions.isEmpty()) {
+            KMessageBox::informationList(kapp->activeWindow(), i18n("The following transitions were corrupted (overlapping)\n and removed from project."), overlappingTransitions, i18n("Invalid Transitions"));
+            m_modified = true;
+        }
 
         // TODO: check the tracks references
         // TODO: check internal mix transitions
+        
     }
 
     return true;