]> git.sesse.net Git - kdenlive/blobdiff - src/documentvalidator.cpp
cppcheck fixes, patch by Mikko Rapeli [2/27]
[kdenlive] / src / documentvalidator.cpp
index e6f7317fb79c8fef05e5d6a01cebb18cf4184dff..7a6b5208391df9f8c8c0a0d9b9289f5575951ee8 100644 (file)
@@ -20,6 +20,7 @@
 
 #include "documentvalidator.h"
 #include "definitions.h"
+#include "initeffects.h"
 
 #include <KDebug>
 #include <KMessageBox>
 
 #include <QFile>
 #include <QColor>
-#include <QBitmap>
+#include <QString>
+
+#include <mlt++/Mlt.h>
+
+#include "locale.h"
 
 
 DocumentValidator::DocumentValidator(QDomDocument doc):
@@ -47,8 +52,29 @@ bool DocumentValidator::validate(const double currentVersion)
     // Check if we're validating a Kdenlive project
     if (kdenliveDoc.isNull())
         return false;
+
+    // Previous MLT / Kdenlive versions used C locale by default
+    QLocale documentLocale = QLocale::c();
+    
+    if (mlt.hasAttribute("LC_NUMERIC")) {
+        // Set locale for the document
+        // WARNING: what should be done in case the locale does not exist on the system?
+        setlocale(LC_NUMERIC, mlt.attribute("LC_NUMERIC").toUtf8().constData());
+        documentLocale = QLocale(mlt.attribute("LC_NUMERIC"));
+    }
+
+    if (documentLocale != QLocale()) {
+        QLocale::setDefault(documentLocale);
+        // locale conversion might need to be redone
+        initEffects::parseEffectFiles();
+    }
+
+    // TODO: remove after string freeze
+    if (0)
+        KMessageBox::sorry(kapp->activeWindow(), i18n("The document you are opening uses a different locale (%1) than your system. You can only open and render it, no editing is supported unless you change your system's locale.", mlt.attribute("LC_NUMERIC")), i18n("Read only project"));
+
     // Upgrade the document to the latest version
-    if (!upgrade(kdenliveDoc.attribute("version").toDouble(), currentVersion))
+    if (!upgrade(documentLocale.toDouble(kdenliveDoc.attribute("version")), currentVersion))
         return false;
 
     /*
@@ -144,41 +170,7 @@ bool DocumentValidator::validate(const double currentVersion)
                     tracksinfoElm.appendChild(trackinfo);
                 }
             }
-        }
-        
-        // 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
@@ -873,6 +865,59 @@ bool DocumentValidator::upgrade(double version, const double currentVersion)
         }
     }
 
+    if (version <= 0.86) {
+        // Make sure we don't have avformat-novalidate producers, since it caused crashes
+        QDomNodeList producers = m_doc.elementsByTagName("producer");
+        int max = producers.count();
+        for (int i = 0; i < max; i++) {
+            QDomElement prod = producers.at(i).toElement();
+            if (EffectsList::property(prod, "mlt_service") == "avformat-novalidate")
+                EffectsList::setProperty(prod, "mlt_service", "avformat");
+        }
+
+        // There was a mistake in Geometry transitions where the last keyframe was created one frame after the end of transition, so fix it and move last keyframe to real end of transition
+
+        // Get profile info (width / height)
+        int profileWidth;
+        int profileHeight;
+        QDomElement profile = m_doc.firstChildElement("profile");
+        if (profile.isNull()) profile = infoXml.firstChildElement("profileinfo");
+        if (profile.isNull()) {
+            // could not find profile info, set PAL
+            profileWidth = 720;
+            profileHeight = 576;
+        }
+        else {
+            profileWidth = profile.attribute("width").toInt();
+            profileHeight = profile.attribute("height").toInt();
+        }
+        QDomNodeList transitions = m_doc.elementsByTagName("transition");
+        max = transitions.count();
+        int out;
+        for (int i = 0; i < max; i++) {
+            QDomElement trans = transitions.at(i).toElement();
+            out = trans.attribute("out").toInt() - trans.attribute("in").toInt();
+            QString geom = EffectsList::property(trans, "geometry");
+            Mlt::Geometry *g = new Mlt::Geometry(geom.toUtf8().data(), out, profileWidth, profileHeight);
+            Mlt::GeometryItem item;
+            if (g->next_key(&item, out) == 0) {
+                // We have a keyframe just after last frame, try to move it to last frame
+                if (item.frame() == out + 1) {
+                    item.frame(out);
+                    g->insert(item);
+                    g->remove(out + 1);
+                    EffectsList::setProperty(trans, "geometry", g->serialise());
+                }
+            }
+            delete g;
+        }
+    }
+
+    if (version <= 0.87) {
+        if (!m_doc.firstChildElement("mlt").hasAttribute("LC_NUMERIC")) {
+            m_doc.firstChildElement("mlt").setAttribute("LC_NUMERIC", "C");
+        }
+    }
 
     // The document has been converted: mark it as modified
     infoXml.setAttribute("version", currentVersion);