]> git.sesse.net Git - kdenlive/blobdiff - src/renderwidget.cpp
comments added
[kdenlive] / src / renderwidget.cpp
index 7b624a31033e08aa0a9264477a965d40b3f5f695..84ab9f6c77e4e7d63a53072e017b8edc6fdae45d 100644 (file)
@@ -241,12 +241,17 @@ void RenderWidget::setGuides(QDomElement guidesxml, double duration)
         m_view.guide_end->addItem(i18n("End"), QString::number(duration));
 }
 
-// Will be called when the user selects an output file via the file dialog.
-// File extension will be added automatically.
+/** 
+ * Will be called when the user selects an output file via the file dialog.
+ * File extension will be added automatically.
+ */
 void RenderWidget::slotUpdateButtons(KUrl url)
 {
     if (m_view.out_file->url().isEmpty()) m_view.buttonStart->setEnabled(false);
-    else m_view.buttonStart->setEnabled(true);
+    else {
+               updateButtons(); // This also checks whether the selected format is available
+               //m_view.buttonStart->setEnabled(true);
+       }
     if (url != 0) {
         QListWidgetItem *item = m_view.size_list->currentItem();
         QString extension = item->data(ExtensionRole).toString();
@@ -255,12 +260,15 @@ void RenderWidget::slotUpdateButtons(KUrl url)
     }
 }
 
-// Will be called when the user changes the output file path in the text line.
-// File extension must NOT be added, would make editing impossible!
+/**
+ * Will be called when the user changes the output file path in the text line.
+ * File extension must NOT be added, would make editing impossible!
+ */
 void RenderWidget::slotUpdateButtons()
 {
     if (m_view.out_file->url().isEmpty()) m_view.buttonStart->setEnabled(false);
-    else m_view.buttonStart->setEnabled(true);
+    else updateButtons(); // This also checks whether the selected format is available
+       //else m_view.buttonStart->setEnabled(true);
 }
 
 void RenderWidget::slotSaveProfile()
@@ -568,34 +576,24 @@ void RenderWidget::slotExport(bool scriptExport)
     QListWidgetItem *item = m_view.size_list->currentItem();
     if (!item) return;
 
-    const QString dest = m_view.out_file->url().path();
+    QString dest = m_view.out_file->url().path();
     if (dest.isEmpty()) return;
+       
+       // Check whether target file has an extension.
+       // If not, ask whether extension should be added or not.
+       QString extension = item->data(ExtensionRole).toString();
+       if (!dest.endsWith(extension)) {
+               if (KMessageBox::questionYesNo(this, i18n("File has no extension. Add extension (%1)?", extension)) == KMessageBox::Yes) {
+                       dest.append("." + extension);
+               }
+       }
+       
     QFile f(dest);
     if (f.exists()) {
         if (KMessageBox::warningYesNo(this, i18n("Output file already exists. Do you want to overwrite it?")) != KMessageBox::Yes)
             return;
     }
 
-    QString scriptName;
-    if (scriptExport) {
-        bool ok;
-        int ix = 0;
-        QString scriptsFolder = m_projectFolder + "/scripts/";
-        KStandardDirs::makeDir(scriptsFolder);
-        QString path = scriptsFolder + i18n("script") + QString::number(ix).rightJustified(3, '0', false) + ".sh";
-        while (QFile::exists(path)) {
-            ix++;
-            path = scriptsFolder + i18n("script") + QString::number(ix).rightJustified(3, '0', false) + ".sh";
-        }
-        scriptName = QInputDialog::getText(this, i18n("Create Render Script"), i18n("Script name (will be saved in: %1)", scriptsFolder), QLineEdit::Normal, KUrl(path).fileName(), &ok);
-        if (!ok || scriptName.isEmpty()) return;
-        scriptName.prepend(scriptsFolder);
-        QFile f(scriptName);
-        if (f.exists()) {
-            if (KMessageBox::warningYesNo(this, i18n("Script file already exists. Do you want to overwrite it?")) != KMessageBox::Yes)
-                return;
-        }
-    }
 
     QStringList overlayargs;
     if (m_view.tc_overlay->isChecked()) {
@@ -634,24 +632,60 @@ void RenderWidget::slotExport(bool scriptExport)
 
     // Check if the rendering profile is different from project profile,
     // in which case we need to use the producer_comsumer from MLT
-    bool resizeProfile = false;
-
     QString std = renderArgs;
     QString destination = m_view.destination_list->itemData(m_view.destination_list->currentIndex()).toString();
+    const QString currentSize = QString::number(width) + 'x' + QString::number(height);
+    QString subsize = currentSize;
     if (std.startsWith("s=")) {
-        QString subsize = std.section(' ', 0, 0).toLower();
+        subsize = std.section(' ', 0, 0).toLower();
         subsize = subsize.section("=", 1, 1);
-        const QString currentSize = QString::number(width) + 'x' + QString::number(height);
-        if (subsize != currentSize) resizeProfile = true;
     } else if (std.contains(" s=")) {
-        QString subsize = std.section(" s=", 1, 1);
+        subsize = std.section(" s=", 1, 1);
         subsize = subsize.section(' ', 0, 0).toLower();
-        const QString currentSize = QString::number(width) + 'x' + QString::number(height);
-        if (subsize != currentSize) resizeProfile = true;
-    } else if (destination != "audioonly") {
-        // Add current size parametrer
-        renderArgs.append(QString(" s=%1x%2").arg(width).arg(height));
+    } else if (destination != "audioonly" && m_view.rescale->isChecked() && m_view.rescale->isEnabled()) 
+    {
+        subsize = QString(" s=%1x%2").arg(width).arg(height);
+        // Add current size parameter
+        renderArgs.append(subsize);
     }
+    bool resizeProfile = (subsize != currentSize);
+
+    QString group = m_view.size_list->currentItem()->data(MetaGroupRole).toString();
+
+    QStringList renderParameters;
+    renderParameters << dest << item->data(RenderRole).toString() << renderArgs.simplified();
+    renderParameters << QString::number(m_view.render_zone->isChecked()) << QString::number(m_view.play_after->isChecked());
+    renderParameters << QString::number(startPos) << QString::number(endPos) << QString::number(resizeProfile);
+
+    QString scriptName;
+    if (scriptExport) {
+        bool ok;
+        int ix = 0;
+        QString scriptsFolder = m_projectFolder + "/scripts/";
+        KStandardDirs::makeDir(scriptsFolder);
+        QString path = scriptsFolder + i18n("script") + QString::number(ix).rightJustified(3, '0', false) + ".sh";
+        while (QFile::exists(path)) {
+            ix++;
+            path = scriptsFolder + i18n("script") + QString::number(ix).rightJustified(3, '0', false) + ".sh";
+        }
+        scriptName = QInputDialog::getText(this, i18n("Create Render Script"), i18n("Script name (will be saved in: %1)", scriptsFolder), QLineEdit::Normal, KUrl(path).fileName(), &ok);
+        if (!ok || scriptName.isEmpty()) return;
+        scriptName.prepend(scriptsFolder);
+        QFile f(scriptName);
+        if (f.exists()) {
+            if (KMessageBox::warningYesNo(this, i18n("Script file already exists. Do you want to overwrite it?")) != KMessageBox::Yes)
+                return;
+        }
+        renderParameters << scriptName;
+        if (group == "dvd") renderParameters << QString::number(m_view.create_chapter->isChecked());
+        else renderParameters << QString::number(false);
+        emit doRender(renderParameters, overlayargs);
+        QTimer::singleShot(400, this, SLOT(parseScriptFiles()));
+        m_view.tabWidget->setCurrentIndex(2);
+        return;
+    }
+    renderParameters << scriptName;
+    m_view.tabWidget->setCurrentIndex(1);
 
     // insert item in running jobs list
     QTreeWidgetItem *renderItem;
@@ -662,6 +696,7 @@ void RenderWidget::slotExport(bool scriptExport)
             KMessageBox::information(this, i18n("There is already a job writing file:<br><b>%1</b><br>Abort the job if you want to overwrite it...", dest), i18n("Already running"));
             return;
         }
+        renderItem->setData(1, Qt::UserRole + 4, QString());
     } else {
         renderItem = new QTreeWidgetItem(m_view.running_jobs, QStringList() << QString() << dest << QString());
         renderItem->setData(1, Qt::UserRole + 2, WAITINGJOB);
@@ -671,14 +706,8 @@ void RenderWidget::slotExport(bool scriptExport)
     renderItem->setSizeHint(1, QSize(m_view.running_jobs->columnWidth(1), fontMetrics().height() * 2));
     renderItem->setData(1, Qt::UserRole + 1, QTime::currentTime());
     renderItem->setData(1, Qt::UserRole + 2, overlayargs);
-    QStringList renderParameters;
-    renderParameters << dest << item->data(RenderRole).toString() << renderArgs.simplified();
-    renderParameters << QString::number(m_view.render_zone->isChecked()) << QString::number(m_view.play_after->isChecked());
-    renderParameters << QString::number(startPos) << QString::number(endPos) << QString::number(resizeProfile);
-    renderParameters << scriptName;
 
     // Set rendering type
-    QString group = m_view.size_list->currentItem()->data(MetaGroupRole).toString();
     if (group == "dvd") {
         renderParameters << QString::number(m_view.create_chapter->isChecked());
         if (m_view.open_dvd->isChecked()) {
@@ -703,13 +732,7 @@ void RenderWidget::slotExport(bool scriptExport)
 
 
     renderItem->setData(1, Qt::UserRole + 3, renderParameters);
-
     checkRenderStatus();
-    if (scriptName.isEmpty()) m_view.tabWidget->setCurrentIndex(1);
-    else {
-        QTimer::singleShot(400, this, SLOT(parseScriptFiles()));
-        m_view.tabWidget->setCurrentIndex(2);
-    }
 }
 
 void RenderWidget::checkRenderStatus()
@@ -719,9 +742,12 @@ void RenderWidget::checkRenderStatus()
         if (item->data(1, Qt::UserRole + 2).toInt() == RUNNINGJOB) break;
         else if (item->data(1, Qt::UserRole + 2).toInt() == WAITINGJOB) {
             item->setData(1, Qt::UserRole + 1, QTime::currentTime());
-            if (item->data(1, Qt::UserRole + 4).isNull())
+            if (item->data(1, Qt::UserRole + 4).isNull()) {
                 emit doRender(item->data(1, Qt::UserRole + 3).toStringList(), item->data(1, Qt::UserRole + 2).toStringList());
-            else QProcess::startDetached(item->data(1, Qt::UserRole + 3).toString());
+            } else {
+                // Script item
+                QProcess::startDetached(item->data(1, Qt::UserRole + 3).toString());
+            }
             break;
         }
         item = m_view.running_jobs->itemBelow(item);
@@ -888,8 +914,12 @@ KUrl RenderWidget::filenameWithExtension(KUrl url, QString extension)
 }
 
 
+/**
+ * Called when a new format or size has been selected.
+ */
 void RenderWidget::refreshParams()
 {
+       // Format not available (e.g. codec not installed); Disable start button
     QListWidgetItem *item = m_view.size_list->currentItem();
     if (!item || item->isHidden()) {
         m_view.advanced_params->clear();
@@ -1341,6 +1371,7 @@ void RenderWidget::slotStartScript()
         // Insert new job in queue
         QTreeWidgetItem *renderItem;
         QList<QTreeWidgetItem *> existing = m_view.running_jobs->findItems(destination, Qt::MatchExactly, 1);
+        kDebug() << "------  START SCRIPT";
         if (!existing.isEmpty()) {
             renderItem = existing.at(0);
             if (renderItem->data(1, Qt::UserRole + 2).toInt() == RUNNINGJOB) {
@@ -1348,6 +1379,8 @@ void RenderWidget::slotStartScript()
                 return;
             }
         } else renderItem = new QTreeWidgetItem(m_view.running_jobs, QStringList() << QString() << destination << QString());
+        kDebug() << "------  START SCRIPT 2";
+        renderItem->setData(2, Qt::UserRole, 0);
         renderItem->setData(1, Qt::UserRole + 2, WAITINGJOB);
         renderItem->setIcon(0, KIcon("media-playback-pause"));
         renderItem->setData(1, Qt::UserRole, i18n("Waiting..."));