Allow sending KDMs to more emails (#2244).

6 files changed, 150 insertions, 9 deletions

diff --git a/src/wx/dkdm_output_panel.cc b/src/wx/dkdm_output_panel.cc
index cc02721ed..9eb3084e3 100644
--- a/src/wx/dkdm_output_panel.cc
+++ b/src/wx/dkdm_output_panel.cc
@@ -178,7 +178,8 @@ DKDMOutputPanel::make (
 					kdms,
 					_filename_format->get(),
 					_filename_format->get(),
-					name
+					name,
+					{}
 					)
 				);
 		}
diff --git a/src/wx/extra_kdm_email_dialog.cc b/src/wx/extra_kdm_email_dialog.cc
new file mode 100644
index 000000000..8a8a37f0a
--- /dev/null
+++ b/src/wx/extra_kdm_email_dialog.cc
@@ -0,0 +1,81 @@
+/*
+    Copyright (C) 2022 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 "editable_list.h"
+#include "extra_kdm_email_dialog.h"
+#include "wx_util.h"
+#include <wx/gbsizer.h>
+#include <vector>
+
+
+using std::string;
+using std::vector;
+#if BOOST_VERSION >= 106100
+using namespace boost::placeholders;
+#endif
+
+
+ExtraKDMEmailDialog::ExtraKDMEmailDialog (wxWindow* parent, vector<string> emails)
+	: wxDialog (parent, wxID_ANY, _("Extra addresses for KDM delivery"))
+	, _emails(emails)
+{
+	auto overall_sizer = new wxBoxSizer (wxVERTICAL);
+	SetSizer (overall_sizer);
+
+	auto sizer = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+	int r = 0;
+
+	vector<EditableListColumn> columns;
+	columns.push_back (EditableListColumn(_("Address"), 500, true));
+	_email_list = new EditableList<string, EmailDialog> (
+		this, columns, bind(&ExtraKDMEmailDialog::get, this), bind(&ExtraKDMEmailDialog::set, this, _1), [](string s, int) {
+			return s;
+		}, false, EditableListButton::NEW | EditableListButton::EDIT | EditableListButton::REMOVE
+		);
+
+	sizer->Add (_email_list, wxGBPosition(r, 0), wxGBSpan(1, 2), wxEXPAND);
+	++r;
+
+	overall_sizer->Add (sizer, 1, wxEXPAND | wxALL, DCPOMATIC_DIALOG_BORDER);
+
+	auto buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL);
+	if (buttons) {
+		overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
+	}
+
+	overall_sizer->Layout ();
+	overall_sizer->SetSizeHints (this);
+}
+
+
+vector<string>
+ExtraKDMEmailDialog::get () const
+{
+	return _emails;
+}
+
+
+void
+ExtraKDMEmailDialog::set(vector<string> emails)
+{
+	_emails = emails;
+}
+
diff --git a/src/wx/extra_kdm_email_dialog.h b/src/wx/extra_kdm_email_dialog.h
new file mode 100644
index 000000000..e04541864
--- /dev/null
+++ b/src/wx/extra_kdm_email_dialog.h
@@ -0,0 +1,41 @@
+/*
+    Copyright (C) 2022 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 "editable_list.h"
+#include "email_dialog.h"
+#include <wx/wx.h>
+#include <vector>
+
+
+class ExtraKDMEmailDialog : public wxDialog
+{
+public:
+	ExtraKDMEmailDialog (wxWindow* parent, std::vector<std::string> emails);
+
+	std::vector<std::string> get () const;
+
+private:
+	void set(std::vector<std::string> emails);
+
+	EditableList<std::string, EmailDialog>* _email_list;
+	std::vector<std::string> _emails;
+};
+
diff --git a/src/wx/kdm_output_panel.cc b/src/wx/kdm_output_panel.cc
index 98f408135..75971c14b 100644
--- a/src/wx/kdm_output_panel.cc
+++ b/src/wx/kdm_output_panel.cc
@@ -22,6 +22,7 @@
 #include "check_box.h"
 #include "confirm_kdm_email_dialog.h"
 #include "dcpomatic_button.h"
+#include "extra_kdm_email_dialog.h"
 #include "kdm_advanced_dialog.h"
 #include "kdm_choice.h"
 #include "kdm_output_panel.h"
@@ -50,6 +51,7 @@ using std::exception;
 using std::function;
 using std::list;
 using std::make_pair;
+using std::make_shared;
 using std::pair;
 using std::shared_ptr;
 using std::string;
@@ -135,7 +137,8 @@ KDMOutputPanel::KDMOutputPanel (wxWindow* parent)
 
 	_email = new CheckBox (this, _("Send by email"));
 	table->Add (_email, 1, wxEXPAND);
-	table->AddSpacer (0);
+	auto add_email_addresses = new wxButton(this, wxID_ANY, _("Set additional email addresses..."));
+	table->Add (add_email_addresses);
 
 	switch (Config::instance()->last_kdm_write_type().get_value_or(Config::KDM_WRITE_FLAT)) {
 	case Config::KDM_WRITE_FLAT:
@@ -154,6 +157,7 @@ KDMOutputPanel::KDMOutputPanel (wxWindow* parent)
 
 	_write_to->Bind     (wxEVT_CHECKBOX, boost::bind (&KDMOutputPanel::write_to_changed, this));
 	_email->Bind        (wxEVT_CHECKBOX, boost::bind (&KDMOutputPanel::email_changed, this));
+	add_email_addresses->Bind (wxEVT_BUTTON, boost::bind(&KDMOutputPanel::add_email_addresses_clicked, this));
 	_write_flat->Bind   (wxEVT_RADIOBUTTON, boost::bind (&KDMOutputPanel::kdm_write_type_changed, this));
 	_write_folder->Bind (wxEVT_RADIOBUTTON, boost::bind (&KDMOutputPanel::kdm_write_type_changed, this));
 	_write_zip->Bind    (wxEVT_RADIOBUTTON, boost::bind (&KDMOutputPanel::kdm_write_type_changed, this));
@@ -301,13 +305,12 @@ KDMOutputPanel::make (
 		}
 
 		if (_email->GetValue ()) {
-			job.reset (
-				new SendKDMEmailJob (
-					cinema_kdms,
-					_container_name_format->get(),
-					_filename_format->get(),
-					name
-					)
+			job = make_shared<SendKDMEmailJob>(
+				cinema_kdms,
+				_container_name_format->get(),
+				_filename_format->get(),
+				name,
+				_extra_addresses
 				);
 		}
 
@@ -335,3 +338,15 @@ KDMOutputPanel::directory () const
 {
 	return wx_to_std (_folder->GetPath ());
 }
+
+
+void
+KDMOutputPanel::add_email_addresses_clicked ()
+{
+	auto dialog = new ExtraKDMEmailDialog (this, _extra_addresses);
+	if (dialog->ShowModal() == wxID_OK) {
+		_extra_addresses = dialog->get();
+	}
+	dialog->Destroy();
+}
+
diff --git a/src/wx/kdm_output_panel.h b/src/wx/kdm_output_panel.h
index 872d23a8f..9c30e286c 100644
--- a/src/wx/kdm_output_panel.h
+++ b/src/wx/kdm_output_panel.h
@@ -67,6 +67,7 @@ private:
 	void advanced_clicked ();
 	void write_to_changed ();
 	void email_changed ();
+	void add_email_addresses_clicked ();
 
 	KDMChoice* _type;
 	NameFormatEditor* _container_name_format;
@@ -84,4 +85,5 @@ private:
 	bool _forensic_mark_video;
 	bool _forensic_mark_audio;
 	boost::optional<int> _forensic_mark_audio_up_to;
+	std::vector<std::string> _extra_addresses;
 };
diff --git a/src/wx/wscript b/src/wx/wscript
index 8da9b546c..ad343cede 100644
--- a/src/wx/wscript
+++ b/src/wx/wscript
@@ -68,6 +68,7 @@ sources = """
           email_dialog.cc
           export_subtitles_dialog.cc
           export_video_file_dialog.cc
+          extra_kdm_email_dialog.cc
           file_picker_ctrl.cc
           film_editor.cc
           film_name_location_dialog.cc