Move subtitle export to its own dialogue.

4 files changed, 129 insertions, 5 deletions

diff --git a/src/wx/export_dialog.cc b/src/wx/export_dialog.cc
index ca723ddeb..3553f3c7e 100644
--- a/src/wx/export_dialog.cc
+++ b/src/wx/export_dialog.cc
@@ -31,30 +31,26 @@ DCPOMATIC_ENABLE_WARNINGS
 using std::string;
 using boost::bind;
 
-#define FORMATS 3
+#define FORMATS 2
 
 wxString format_names[] = {
 	_("ProRes"),
 	_("MP4 / H.264"),
-	_("DCP subtitles")
 };
 
 wxString format_filters[] = {
 	_("MOV files (*.mov)|*.mov"),
 	_("MP4 files (*.mp4)|*.mp4"),
-	_("Subtitle files (*.xml)|*.xml"),
 };
 
 wxString format_extensions[] = {
 	"mov",
 	"mp4",
-	"xml",
 };
 
 ExportFormat formats[] = {
 	EXPORT_FORMAT_PRORES,
 	EXPORT_FORMAT_H264_AAC,
-	EXPORT_FORMAT_SUBTITLES_DCP
 };
 
 ExportDialog::ExportDialog (wxWindow* parent, string name)
diff --git a/src/wx/export_subtitles_dialog.cc b/src/wx/export_subtitles_dialog.cc
new file mode 100644
index 000000000..83e08feb4
--- /dev/null
+++ b/src/wx/export_subtitles_dialog.cc
@@ -0,0 +1,86 @@
+/*
+    Copyright (C) 2017-2020 Carl Hetherington <cth@carlh.net>
+
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    DCP-o-matic is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
+
+*/
+
+
+#include "check_box.h"
+#include "export_subtitles_dialog.h"
+#include "file_picker_ctrl.h"
+#include "wx_util.h"
+#include "lib/warnings.h"
+DCPOMATIC_DISABLE_WARNINGS
+#include <wx/filepicker.h>
+DCPOMATIC_ENABLE_WARNINGS
+#include <boost/bind.hpp>
+
+
+using std::string;
+using boost::bind;
+
+
+ExportSubtitlesDialog::ExportSubtitlesDialog (wxWindow* parent, string name)
+	: TableDialog (parent, _("Export subtitles"), 2, 1, true)
+	, _initial_name (name)
+{
+	_split_reels = new CheckBox (this, _("Write reels into separate files"));
+	add (_split_reels, false);
+	add_spacer ();
+
+	add (_("Output file"), true);
+	/* Don't warn overwrite here, because on Linux (at least) if we specify a filename like foo
+	   the wxFileDialog will check that foo exists, but we will add an extension so we actually
+	   need to check if foo.mov (or similar) exists.  I can't find a way to make wxWidgets do this,
+	   so disable its check and the caller will have to do it themselves.
+	*/
+	_file = new FilePickerCtrl (this, _("Select output file"), _("Subtitle files (.xml)|*.xml"), false, false);
+	_file->SetPath (_initial_name);
+	add (_file);
+
+	_file->Bind (wxEVT_FILEPICKER_CHANGED, bind(&ExportSubtitlesDialog::file_changed, this));
+
+	layout ();
+
+	wxButton* ok = dynamic_cast<wxButton *>(FindWindowById(wxID_OK, this));
+	ok->Enable (false);
+}
+
+
+boost::filesystem::path
+ExportSubtitlesDialog::path () const
+{
+	wxFileName fn (_file->GetPath());
+	fn.SetExt (".xml");
+	return wx_to_std (fn.GetFullPath());
+}
+
+
+bool
+ExportSubtitlesDialog::split_reels () const
+{
+	return _split_reels->GetValue ();
+}
+
+
+void
+ExportSubtitlesDialog::file_changed ()
+{
+	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById (wxID_OK, this));
+	DCPOMATIC_ASSERT (ok);
+	ok->Enable (path().is_absolute());
+}
diff --git a/src/wx/export_subtitles_dialog.h b/src/wx/export_subtitles_dialog.h
new file mode 100644
index 000000000..d4405cb29
--- /dev/null
+++ b/src/wx/export_subtitles_dialog.h
@@ -0,0 +1,41 @@
+/*
+    Copyright (C) 2017-2020 Carl Hetherington <cth@carlh.net>
+
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    DCP-o-matic is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
+
+*/
+
+#include "table_dialog.h"
+#include <wx/wx.h>
+#include <boost/filesystem.hpp>
+
+class FilePickerCtrl;
+
+class ExportSubtitlesDialog : public TableDialog
+{
+public:
+	ExportSubtitlesDialog (wxWindow* parent, std::string name);
+
+	boost::filesystem::path path () const;
+	bool split_reels () const;
+
+private:
+	void file_changed ();
+
+	std::string _initial_name;
+	wxCheckBox* _split_reels;
+	FilePickerCtrl* _file;
+};
diff --git a/src/wx/wscript b/src/wx/wscript
index 7b4b96653..5d8fc16bb 100644
--- a/src/wx/wscript
+++ b/src/wx/wscript
@@ -66,6 +66,7 @@ sources = """
           download_certificate_dialog.cc
           download_certificate_panel.cc
           export_dialog.cc
+          export_subtitles_dialog.cc
           file_picker_ctrl.cc
           film_editor.cc
           film_name_location_dialog.cc