Set 'i' earlier in name_values.
[dcpomatic.git] / src / wx / kdm_dialog.cc
index 3dd1c2029bafcb9d0cbd86263b48ce6bf9e518c2..d74741871a63213fefbdd8419b506f7f7b9146e6 100644 (file)
 /*
-    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2019 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    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.
 
-    This program is distributed in the hope that it will be useful,
+    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 this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
 
 */
 
 #include "kdm_dialog.h"
-#include "cinema_dialog.h"
-#include "screen_dialog.h"
 #include "wx_util.h"
-#include "lib/cinema.h"
-#include "lib/config.h"
+#include "screens_panel.h"
+#include "kdm_timing_panel.h"
+#include "kdm_output_panel.h"
+#include "kdm_cpl_panel.h"
+#include "confirm_kdm_email_dialog.h"
+#include "static_text.h"
+#include "dcpomatic_button.h"
 #include "lib/film.h"
+#include "lib/screen.h"
+#include "lib/kdm_with_metadata.h"
+#include "lib/job_manager.h"
+#include "lib/config.h"
+#include "lib/cinema.h"
 #include <libcxml/cxml.h>
-#ifdef DCPOMATIC_USE_OWN_DIR_PICKER
-#include "dir_picker_ctrl.h"
-#else
-#include <wx/filepicker.h>
-#endif
+#include <dcp/exceptions.h>
 #include <wx/treectrl.h>
-#include <wx/datectrl.h>
-#include <wx/timectrl.h>
-#include <wx/stdpaths.h>
 #include <wx/listctrl.h>
+#include <iostream>
 
 using std::string;
+using std::exception;
 using std::map;
 using std::list;
 using std::pair;
 using std::cout;
 using std::vector;
 using std::make_pair;
+using std::runtime_error;
 using boost::shared_ptr;
+using boost::bind;
+using boost::optional;
 
-KDMDialog::KDMDialog (wxWindow* parent, boost::shared_ptr<const Film> film)
+KDMDialog::KDMDialog (wxWindow* parent, shared_ptr<const Film> film)
        : wxDialog (parent, wxID_ANY, _("Make KDMs"))
+       , _film (film)
 {
-       /* Main sizer */
-       wxBoxSizer* vertical = new wxBoxSizer (wxVERTICAL);
+       /* Main sizers */
+       wxBoxSizer* horizontal = new wxBoxSizer (wxHORIZONTAL);
+       wxBoxSizer* left = new wxBoxSizer (wxVERTICAL);
+       wxBoxSizer* right = new wxBoxSizer (wxVERTICAL);
+
+       horizontal->Add (left, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_X_GAP * 4);
+       horizontal->Add (right, 1, wxEXPAND);
 
        /* Font for sub-headings */
        wxFont subheading_font (*wxNORMAL_FONT);
        subheading_font.SetWeight (wxFONTWEIGHT_BOLD);
 
-
        /* Sub-heading: Screens */
-       wxStaticText* h = new wxStaticText (this, wxID_ANY, _("Screens"));
+       wxStaticText* h = new StaticText (this, _("Screens"));
        h->SetFont (subheading_font);
-       vertical->Add (h, 0, wxALIGN_CENTER_VERTICAL);
-
-       wxBoxSizer* targets = new wxBoxSizer (wxHORIZONTAL);
-       _targets = new wxTreeCtrl (this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxTR_HIDE_ROOT | wxTR_MULTIPLE | wxTR_HAS_BUTTONS);
-       targets->Add (_targets, 1, wxEXPAND | wxTOP | wxRIGHT, DCPOMATIC_SIZER_GAP);
-
-       _root = _targets->AddRoot ("Foo");
-
-       list<shared_ptr<Cinema> > c = Config::instance()->cinemas ();
-       for (list<shared_ptr<Cinema> >::iterator i = c.begin(); i != c.end(); ++i) {
-               add_cinema (*i);
-       }
-
-       _targets->ExpandAll ();
-
-       wxBoxSizer* target_buttons = new wxBoxSizer (wxVERTICAL);
-
-       _add_cinema = new wxButton (this, wxID_ANY, _("Add Cinema..."));
-       target_buttons->Add (_add_cinema, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-       _edit_cinema = new wxButton (this, wxID_ANY, _("Edit Cinema..."));
-       target_buttons->Add (_edit_cinema, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-       _remove_cinema = new wxButton (this, wxID_ANY, _("Remove Cinema"));
-       target_buttons->Add (_remove_cinema, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-
-       _add_screen = new wxButton (this, wxID_ANY, _("Add Screen..."));
-       target_buttons->Add (_add_screen, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-       _edit_screen = new wxButton (this, wxID_ANY, _("Edit Screen..."));
-       target_buttons->Add (_edit_screen, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-       _remove_screen = new wxButton (this, wxID_ANY, _("Remove Screen"));
-       target_buttons->Add (_remove_screen, 1, wxEXPAND | wxALL, DCPOMATIC_BUTTON_STACK_GAP);
-
-       targets->Add (target_buttons, 0, 0);
-
-       vertical->Add (targets, 1, wxEXPAND);
-
+       left->Add (h, 0, wxALIGN_CENTER_VERTICAL | wxBOTTOM, DCPOMATIC_SIZER_Y_GAP);
+       _screens = new ScreensPanel (this);
+       left->Add (_screens, 1, wxEXPAND | wxBOTTOM, DCPOMATIC_SIZER_Y_GAP);
 
        /* Sub-heading: Timing */
-       h = new wxStaticText (this, wxID_ANY, S_("KDM|Timing"));
+       /// TRANSLATORS: translate the word "Timing" here; do not include the "KDM|" prefix
+       h = new StaticText (this, S_("KDM|Timing"));
        h->SetFont (subheading_font);
-       vertical->Add (h, 0, wxALIGN_CENTER_VERTICAL | wxTOP, DCPOMATIC_SIZER_Y_GAP * 2);
-
-       wxFlexGridSizer* table = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       add_label_to_sizer (table, this, _("From"), true);
-       wxDateTime from;
-       from.SetToCurrent ();
-       _from_date = new wxDatePickerCtrl (this, wxID_ANY, from);
-       table->Add (_from_date, 1, wxEXPAND);
-       _from_time = new wxTimePickerCtrl (this, wxID_ANY, from);
-       table->Add (_from_time, 1, wxEXPAND);
-
-       add_label_to_sizer (table, this, _("Until"), true);
-       wxDateTime to = from;
-       /* 1 week from now */
-       to.Add (wxDateSpan (0, 0, 1, 0));
-       _until_date = new wxDatePickerCtrl (this, wxID_ANY, to);
-       table->Add (_until_date, 1, wxEXPAND);
-       _until_time = new wxTimePickerCtrl (this, wxID_ANY, to);
-       table->Add (_until_time, 1, wxEXPAND);
-
-       vertical->Add (table, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP);
-
+       right->Add (h, 0, wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_Y_GAP * 2);
+       _timing = new KDMTimingPanel (this);
+       right->Add (_timing);
 
        /* Sub-heading: CPL */
-       h = new wxStaticText (this, wxID_ANY, _("CPL"));
-       h->SetFont (subheading_font);
-       vertical->Add (h, 0, wxALIGN_CENTER_VERTICAL | wxTOP, DCPOMATIC_SIZER_Y_GAP * 2);
-
-       /* CPL choice */
-       wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-       add_label_to_sizer (s, this, _("CPL"), true);
-       _cpl = new wxChoice (this, wxID_ANY);
-       s->Add (_cpl, 1, wxEXPAND);
-       _cpl_browse = new wxButton (this, wxID_ANY, _("Browse..."));
-       s->Add (_cpl_browse, 0);
-       vertical->Add (s, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP + 2);
-
-       /* CPL details */
-       table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       add_label_to_sizer (table, this, _("DCP directory"), true);
-       _dcp_directory = new wxStaticText (this, wxID_ANY, "");
-       table->Add (_dcp_directory);
-       add_label_to_sizer (table, this, _("CPL ID"), true);
-       _cpl_id = new wxStaticText (this, wxID_ANY, "");
-       table->Add (_cpl_id);
-       add_label_to_sizer (table, this, _("CPL annotation text"), true);
-       _cpl_annotation_text = new wxStaticText (this, wxID_ANY, "");
-       table->Add (_cpl_annotation_text);
-       vertical->Add (table, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP + 2);
-
-       _cpls = film->cpls ();
-       update_cpl_choice ();
-
-
-       /* Sub-heading: Output */
-       h = new wxStaticText (this, wxID_ANY, _("Output"));
+       h = new StaticText (this, _("CPL"));
        h->SetFont (subheading_font);
-       vertical->Add (h, 0, wxALIGN_CENTER_VERTICAL | wxTOP, DCPOMATIC_SIZER_Y_GAP * 2);
+       right->Add (h, 0, wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_Y_GAP * 2);
 
-       table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, 0);
-
-       add_label_to_sizer (table, this, _("KDM type"), true);
-       _type = new wxChoice (this, wxID_ANY);
-       _type->Append ("Modified Transitional 1", ((void *) dcp::MODIFIED_TRANSITIONAL_1));
-       if (!film->interop ()) {
-               _type->Append ("DCI Any", ((void *) dcp::DCI_ANY));
-               _type->Append ("DCI Specific", ((void *) dcp::DCI_SPECIFIC));
+       vector<CPLSummary> cpls;
+       BOOST_FOREACH (CPLSummary const & i, film->cpls()) {
+               if (i.encrypted) {
+                       cpls.push_back (i);
+               }
        }
-       table->Add (_type, 1, wxEXPAND);
-       _type->SetSelection (0);
-
-       _write_to = new wxRadioButton (this, wxID_ANY, _("Write to"));
-       table->Add (_write_to, 1, wxEXPAND);
 
-#ifdef DCPOMATIC_USE_OWN_DIR_PICKER
-       _folder = new DirPickerCtrl (this);
-#else
-       _folder = new wxDirPickerCtrl (this, wxID_ANY, wxEmptyString, wxDirSelectorPromptStr, wxDefaultPosition, wxSize (300, -1));
-#endif
+       _cpl = new KDMCPLPanel (this, cpls);
+       right->Add (_cpl, 0, wxEXPAND);
 
-       _folder->SetPath (wxStandardPaths::Get().GetDocumentsDir());
-
-       table->Add (_folder, 1, wxEXPAND);
-
-       _email = new wxRadioButton (this, wxID_ANY, _("Send by email"));
-       table->Add (_email, 1, wxEXPAND);
-       table->AddSpacer (0);
+       /* Sub-heading: Output */
+       h = new StaticText (this, _("Output"));
+       h->SetFont (subheading_font);
+       right->Add (h, 0, wxALIGN_CENTER_VERTICAL | wxTOP, DCPOMATIC_SIZER_Y_GAP * 2);
+       _output = new KDMOutputPanel (this, film->interop ());
+       right->Add (_output, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP);
 
-       vertical->Add (table, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP);
+       _make = new Button (this, _("Make KDMs"));
+       right->Add (_make, 0, wxTOP | wxBOTTOM, DCPOMATIC_SIZER_GAP);
 
-       /* Make an overall sizer to get a nice border, and put some buttons in */
+       /* Make an overall sizer to get a nice border */
 
        wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
-       overall_sizer->Add (vertical, 0, wxEXPAND | wxTOP | wxLEFT | wxRIGHT, DCPOMATIC_DIALOG_BORDER);
-
-       wxSizer* buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL);
-       if (buttons) {
-               overall_sizer->Add (buttons, 0, wxEXPAND | wxALL, DCPOMATIC_SIZER_Y_GAP);
-       }
-
-       _write_to->SetValue (true);
+       overall_sizer->Add (horizontal, 0, wxEXPAND | wxTOP | wxLEFT | wxRIGHT, DCPOMATIC_DIALOG_BORDER);
 
        /* Bind */
 
-       _targets->Bind       (wxEVT_COMMAND_TREE_SEL_CHANGED, boost::bind (&KDMDialog::setup_sensitivity, this));
-
-       _add_cinema->Bind    (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::add_cinema_clicked, this));
-       _edit_cinema->Bind   (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::edit_cinema_clicked, this));
-       _remove_cinema->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::remove_cinema_clicked, this));
-
-       _add_screen->Bind    (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::add_screen_clicked, this));
-       _edit_screen->Bind   (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::edit_screen_clicked, this));
-       _remove_screen->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&KDMDialog::remove_screen_clicked, this));
-
-       _cpl->Bind           (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&KDMDialog::update_cpl_summary, this));
-       _cpl_browse->Bind    (wxEVT_COMMAND_BUTTON_CLICKED,  boost::bind (&KDMDialog::cpl_browse_clicked, this));
-
-       _write_to->Bind      (wxEVT_COMMAND_RADIOBUTTON_SELECTED, boost::bind (&KDMDialog::setup_sensitivity, this));
-       _email->Bind         (wxEVT_COMMAND_RADIOBUTTON_SELECTED, boost::bind (&KDMDialog::setup_sensitivity, this));
+       _screens->ScreensChanged.connect (boost::bind (&KDMDialog::setup_sensitivity, this));
+       _timing->TimingChanged.connect (boost::bind (&KDMDialog::setup_sensitivity, this));
+       _make->Bind (wxEVT_BUTTON, boost::bind (&KDMDialog::make_clicked, this));
 
        setup_sensitivity ();
 
@@ -225,339 +126,94 @@ KDMDialog::KDMDialog (wxWindow* parent, boost::shared_ptr<const Film> film)
        overall_sizer->SetSizeHints (this);
 }
 
-list<pair<wxTreeItemId, shared_ptr<Cinema> > >
-KDMDialog::selected_cinemas () const
-{
-       wxArrayTreeItemIds s;
-       _targets->GetSelections (s);
-
-       list<pair<wxTreeItemId, shared_ptr<Cinema> > > c;
-       for (size_t i = 0; i < s.GetCount(); ++i) {
-               map<wxTreeItemId, shared_ptr<Cinema> >::const_iterator j = _cinemas.find (s[i]);
-               if (j != _cinemas.end ()) {
-                       c.push_back (make_pair (j->first, j->second));
-               }
-       }
-
-       return c;
-}
-
-list<pair<wxTreeItemId, shared_ptr<Screen> > >
-KDMDialog::selected_screens () const
-{
-       wxArrayTreeItemIds s;
-       _targets->GetSelections (s);
-
-       list<pair<wxTreeItemId, shared_ptr<Screen> > > c;
-       for (size_t i = 0; i < s.GetCount(); ++i) {
-               map<wxTreeItemId, shared_ptr<Screen> >::const_iterator j = _screens.find (s[i]);
-               if (j != _screens.end ()) {
-                       c.push_back (make_pair (j->first, j->second));
-               }
-       }
-
-       return c;
-}
-
 void
 KDMDialog::setup_sensitivity ()
 {
-       bool const sc = selected_cinemas().size() == 1;
-       bool const ss = selected_screens().size() == 1;
-       bool const sd = _cpl->GetSelection() != -1;
-
-       _edit_cinema->Enable (sc);
-       _remove_cinema->Enable (sc);
-
-       _add_screen->Enable (sc);
-       _edit_screen->Enable (ss);
-       _remove_screen->Enable (ss);
-
-       wxButton* ok = dynamic_cast<wxButton *> (FindWindowById (wxID_OK, this));
-       if (ok) {
-               ok->Enable ((selected_cinemas().size() > 0 || selected_screens().size() > 0) && sd);
-       }
-
-       _folder->Enable (_write_to->GetValue ());
-}
-
-void
-KDMDialog::add_cinema (shared_ptr<Cinema> c)
-{
-       _cinemas[_targets->AppendItem (_root, std_to_wx (c->name))] = c;
-
-       list<shared_ptr<Screen> > sc = c->screens ();
-       for (list<shared_ptr<Screen> >::iterator i = sc.begin(); i != sc.end(); ++i) {
-               add_screen (c, *i);
-       }
-}
-
-void
-KDMDialog::add_screen (shared_ptr<Cinema> c, shared_ptr<Screen> s)
-{
-       map<wxTreeItemId, shared_ptr<Cinema> >::const_iterator i = _cinemas.begin();
-       while (i != _cinemas.end() && i->second != c) {
-               ++i;
-       }
-
-       if (i == _cinemas.end()) {
-               return;
-       }
-
-       _screens[_targets->AppendItem (i->first, std_to_wx (s->name))] = s;
-}
-
-void
-KDMDialog::add_cinema_clicked ()
-{
-       CinemaDialog* d = new CinemaDialog (this, "Add Cinema");
-       if (d->ShowModal () == wxID_OK) {
-               shared_ptr<Cinema> c (new Cinema (d->name(), d->email()));
-               Config::instance()->add_cinema (c);
-               add_cinema (c);
-       }
-
-       d->Destroy ();
-}
-
-void
-KDMDialog::edit_cinema_clicked ()
-{
-       if (selected_cinemas().size() != 1) {
-               return;
-       }
-
-       pair<wxTreeItemId, shared_ptr<Cinema> > c = selected_cinemas().front();
-
-       CinemaDialog* d = new CinemaDialog (this, "Edit cinema", c.second->name, c.second->email);
-       if (d->ShowModal () == wxID_OK) {
-               c.second->name = d->name ();
-               c.second->email = d->email ();
-               _targets->SetItemText (c.first, std_to_wx (d->name()));
-               Config::instance()->changed ();
-       }
-
-       d->Destroy ();
-}
-
-void
-KDMDialog::remove_cinema_clicked ()
-{
-       if (selected_cinemas().size() != 1) {
-               return;
-       }
-
-       pair<wxTreeItemId, shared_ptr<Cinema> > c = selected_cinemas().front();
-
-       Config::instance()->remove_cinema (c.second);
-       _targets->Delete (c.first);
+       _screens->setup_sensitivity ();
+       _output->setup_sensitivity ();
+       _make->Enable (!_screens->screens().empty() && _timing->valid() && _cpl->has_selected());
 }
 
-void
-KDMDialog::add_screen_clicked ()
-{
-       if (selected_cinemas().size() != 1) {
-               return;
-       }
-
-       shared_ptr<Cinema> c = selected_cinemas().front().second;
-
-       ScreenDialog* d = new ScreenDialog (this, "Add Screen");
-       if (d->ShowModal () != wxID_OK) {
-               return;
-       }
-
-       shared_ptr<Screen> s (new Screen (d->name(), d->certificate()));
-       c->add_screen (s);
-       add_screen (c, s);
-
-       Config::instance()->changed ();
-
-       d->Destroy ();
-}
-
-void
-KDMDialog::edit_screen_clicked ()
+bool
+KDMDialog::confirm_overwrite (boost::filesystem::path path)
 {
-       if (selected_screens().size() != 1) {
-               return;
-       }
-
-       pair<wxTreeItemId, shared_ptr<Screen> > s = selected_screens().front();
-
-       ScreenDialog* d = new ScreenDialog (this, "Edit screen", s.second->name, s.second->certificate);
-       if (d->ShowModal () == wxID_OK) {
-               s.second->name = d->name ();
-               s.second->certificate = d->certificate ();
-               _targets->SetItemText (s.first, std_to_wx (d->name()));
-               Config::instance()->changed ();
-       }
-
-       d->Destroy ();
+       return confirm_dialog (
+               this,
+               wxString::Format (_("File %s already exists.  Do you want to overwrite it?"), std_to_wx(path.string()).data())
+               );
 }
 
 void
-KDMDialog::remove_screen_clicked ()
+KDMDialog::make_clicked ()
 {
-       if (selected_screens().size() != 1) {
-               return;
-       }
+       shared_ptr<const Film> film = _film.lock ();
+       DCPOMATIC_ASSERT (film);
 
-       pair<wxTreeItemId, shared_ptr<Screen> > s = selected_screens().front();
-
-       map<wxTreeItemId, shared_ptr<Cinema> >::iterator i = _cinemas.begin ();
-       while (i != _cinemas.end ()) {
-               list<shared_ptr<Screen> > sc = i->second->screens ();
-               if (find (sc.begin(), sc.end(), s.second) != sc.end ()) {
-                       break;
+       list<KDMWithMetadataPtr> kdms;
+       try {
+               /* Start off by enabling forensic marking for all */
+               optional<int> for_audio;
+               if (!_output->forensic_mark_audio()) {
+                       /* No forensic marking for audio */
+                       for_audio = 0;
+               } else if (_output->forensic_mark_audio_up_to()) {
+                       /* Forensic mark up to this channel; disabled on channels greater than this */
+                       for_audio = _output->forensic_mark_audio_up_to();
                }
-       }
-
-       if (i == _cinemas.end()) {
-               return;
-       }
 
-       i->second->remove_screen (s.second);
-       _targets->Delete (s.first);
-
-       Config::instance()->changed ();
-}
-
-list<shared_ptr<Screen> >
-KDMDialog::screens () const
-{
-       list<shared_ptr<Screen> > s;
-
-       list<pair<wxTreeItemId, shared_ptr<Cinema> > > cinemas = selected_cinemas ();
-       for (list<pair<wxTreeItemId, shared_ptr<Cinema> > >::iterator i = cinemas.begin(); i != cinemas.end(); ++i) {
-               list<shared_ptr<Screen> > sc = i->second->screens ();
-               for (list<shared_ptr<Screen> >::const_iterator j = sc.begin(); j != sc.end(); ++j) {
-                       s.push_back (*j);
+               BOOST_FOREACH (shared_ptr<dcpomatic::Screen> i, _screens->screens()) {
+                       if (i->recipient) {
+                               dcp::LocalTime const begin(_timing->from(),  i->cinema ? i->cinema->utc_offset_hour() : 0, i->cinema ? i->cinema->utc_offset_minute() : 0);
+                               dcp::LocalTime const end(_timing->until(), i->cinema ? i->cinema->utc_offset_hour() : 0, i->cinema ? i->cinema->utc_offset_minute() : 0);
+
+                               dcp::EncryptedKDM const kdm = film->make_kdm (
+                                               i->recipient.get(),
+                                               i->trusted_device_thumbprints(),
+                                               _cpl->cpl(),
+                                               begin,
+                                               end,
+                                               _output->formulation(),
+                                               !_output->forensic_mark_video(),
+                                               for_audio
+                                               );
+
+                               dcp::NameFormat::Map name_values;
+                               if (i->cinema) {
+                                       name_values['c'] = i->cinema->name;
+                               }
+                               name_values['s'] = i->name;
+                               name_values['f'] = film->name();
+                               name_values['b'] = dcp::LocalTime(begin).date() + " " + dcp::LocalTime(begin).time_of_day(false, false);
+                               name_values['e'] = dcp::LocalTime(end).date() + " " + dcp::LocalTime(end).time_of_day(false, false);
+                               name_values['i'] = kdm.cpl_id();
+
+                               kdms.push_back (KDMWithMetadataPtr(new DCPKDMWithMetadata(name_values, i->cinema, kdm)));
+                       }
                }
-       }
-
-       list<pair<wxTreeItemId, shared_ptr<Screen> > > screens = selected_screens ();
-       for (list<pair<wxTreeItemId, shared_ptr<Screen> > >::iterator i = screens.begin(); i != screens.end(); ++i) {
-               s.push_back (i->second);
-       }
-
-       s.sort ();
-       s.unique ();
-
-       return s;
-}
 
-boost::posix_time::ptime
-KDMDialog::from () const
-{
-       return posix_time (_from_date, _from_time);
-}
-
-boost::posix_time::ptime
-KDMDialog::posix_time (wxDatePickerCtrl* date_picker, wxTimePickerCtrl* time_picker)
-{
-       wxDateTime const date = date_picker->GetValue ();
-       wxDateTime const time = time_picker->GetValue ();
-       return boost::posix_time::ptime (
-               boost::gregorian::date (date.GetYear(), date.GetMonth() + 1, date.GetDay()),
-               boost::posix_time::time_duration (time.GetHour(), time.GetMinute(), time.GetSecond())
-               );
-}
-
-boost::posix_time::ptime
-KDMDialog::until () const
-{
-       return posix_time (_until_date, _until_time);
-}
-
-boost::filesystem::path
-KDMDialog::cpl () const
-{
-       int const item = _cpl->GetSelection ();
-       DCPOMATIC_ASSERT (item >= 0);
-       return _cpls[item].cpl_file;
-}
-
-boost::filesystem::path
-KDMDialog::directory () const
-{
-       return wx_to_std (_folder->GetPath ());
-}
-
-bool
-KDMDialog::write_to () const
-{
-       return _write_to->GetValue ();
-}
-
-dcp::Formulation
-KDMDialog::formulation () const
-{
-       return (dcp::Formulation) reinterpret_cast<intptr_t> (_type->GetClientData (_type->GetSelection()));
-}
-
-void
-KDMDialog::update_cpl_choice ()
-{
-       _cpl->Clear ();
-
-       for (vector<CPLSummary>::const_iterator i = _cpls.begin(); i != _cpls.end(); ++i) {
-               _cpl->Append (std_to_wx (i->cpl_id));
-
-               if (_cpls.size() > 0) {
-                       _cpl->SetSelection (0);
+       } catch (dcp::BadKDMDateError& e) {
+               if (e.starts_too_early()) {
+                       error_dialog (this, _("The KDM start period is before (or close to) the start of the signing certificate's validity period.  Use a later start time for this KDM."));
+               } else {
+                       error_dialog (this, _("The KDM end period is after (or close to) the end of the signing certficates' validity period.  Either use an earlier end time for this KDM or re-create your signing certificates in the DCP-o-matic preferences window."));
                }
-       }
-
-       update_cpl_summary ();
-}
-
-void
-KDMDialog::update_cpl_summary ()
-{
-       int const n = _cpl->GetSelection();
-       if (n == wxNOT_FOUND) {
                return;
-       }
-
-       _dcp_directory->SetLabel (std_to_wx (_cpls[n].dcp_directory));
-       _cpl_id->SetLabel (std_to_wx (_cpls[n].cpl_id));
-       _cpl_annotation_text->SetLabel (std_to_wx (_cpls[n].cpl_annotation_text));
-}
-
-void
-KDMDialog::cpl_browse_clicked ()
-{
-       wxFileDialog* d = new wxFileDialog (this, _("Select CPL XML file"), wxEmptyString, wxEmptyString, "*.xml");
-       if (d->ShowModal() == wxID_CANCEL) {
-               d->Destroy ();
+       } catch (runtime_error& e) {
+               error_dialog (this, std_to_wx(e.what()));
                return;
        }
 
-       boost::filesystem::path cpl_file (wx_to_std (d->GetPath ()));
-       boost::filesystem::path dcp_dir = cpl_file.parent_path ();
-
-       d->Destroy ();
-
-       /* XXX: hack alert */
-       cxml::Document cpl_document ("CompositionPlaylist");
-       cpl_document.read_file (cpl_file);
+       pair<shared_ptr<Job>, int> result = _output->make (kdms, film->name(), bind (&KDMDialog::confirm_overwrite, this, _1));
+       if (result.first) {
+               JobManager::instance()->add (result.first);
+       }
 
-       try {
-               _cpls.push_back (
-                       CPLSummary (
-                               dcp_dir.filename().string(),
-                               cpl_document.string_child("Id").substr (9),
-                               cpl_document.string_child ("ContentTitleText"),
-                               cpl_file
-                               )
+       if (result.second > 0) {
+               /* XXX: proper plural form support in wxWidgets? */
+               wxString s = result.second == 1 ? _("%d KDM written to %s") : _("%d KDMs written to %s");
+               message_dialog (
+                       this,
+                       wxString::Format (s, result.second, std_to_wx(_output->directory().string()).data())
                        );
-       } catch (cxml::Error) {
-               error_dialog (this, _("This is not a valid CPL file"));
-               return;
        }
-
-       update_cpl_choice ();
-       _cpl->SetSelection (_cpls.size() - 1);
-       update_cpl_summary ();
 }