Restore time zone to Cinema and improve UI to use it (#2473).
[dcpomatic.git] / src / tools / dcpomatic_kdm.cc
index 9bb8e20a86ea1c2c208e583326a75cb46f85e6d5..7838e145d71e81396d6cd577f5e6a5308d8330b0 100644 (file)
@@ -22,6 +22,7 @@
 #include "wx/about_dialog.h"
 #include "wx/dcpomatic_button.h"
 #include "wx/editable_list.h"
+#include "wx/id.h"
 #include "wx/invalid_certificate_period_dialog.h"
 #include "wx/file_dialog.h"
 #include "wx/file_picker_ctrl.h"
@@ -36,6 +37,7 @@
 #include "wx/static_text.h"
 #include "wx/wx_signal_manager.h"
 #include "wx/wx_util.h"
+#include "wx/wx_variant.h"
 #include "lib/cinema.h"
 #include "lib/collator.h"
 #include "lib/compose.hpp"
@@ -51,6 +53,8 @@
 #include "lib/kdm_with_metadata.h"
 #include "lib/screen.h"
 #include "lib/send_kdm_email_job.h"
+#include "lib/util.h"
+#include "lib/variant.h"
 #include <dcp/encrypted_kdm.h>
 #include <dcp/decrypted_kdm.h>
 #include <dcp/exceptions.h>
@@ -96,7 +100,7 @@ using namespace dcpomatic;
 
 
 enum {
-       ID_help_report_a_problem = 1,
+       ID_help_report_a_problem = DCPOMATIC_MAIN_MENU,
 };
 
 
@@ -124,7 +128,7 @@ public:
                        setvbuf(hf_in, NULL, _IONBF, 128);
                        *stdin = *hf_in;
 
-                       std::cout << "DCP-o-matic KDM creator is starting." << "\n";
+                       std::cout << variant::insert_dcpomatic_kdm_creator("%1 is starting.\n");
                }
 #endif
 
@@ -233,7 +237,7 @@ public:
                /* Instantly save any config changes when using a DCP-o-matic GUI */
                Config::instance()->Changed.connect (boost::bind (&Config::write, Config::instance ()));
 
-               _screens->ScreensChanged.connect (boost::bind (&DOMFrame::setup_sensitivity, this));
+               _screens->ScreensChanged.connect(boost::bind(&DOMFrame::screens_changed, this));
                _create->Bind (wxEVT_BUTTON, bind (&DOMFrame::create_kdms, this));
                _dkdm->Bind(wxEVT_TREE_SEL_CHANGED, boost::bind(&DOMFrame::dkdm_selection_changed, this));
                _dkdm->Bind (wxEVT_TREE_BEGIN_DRAG, boost::bind (&DOMFrame::dkdm_begin_drag, this, _1));
@@ -246,6 +250,7 @@ public:
                _export_dkdm->Bind (wxEVT_BUTTON, bind (&DOMFrame::export_dkdm_clicked, this));
                _dkdm_search->Bind(wxEVT_TEXT, boost::bind(&DOMFrame::dkdm_search_changed, this));
                _timing->TimingChanged.connect(boost::bind(&DOMFrame::setup_sensitivity, this));
+               _output->MethodChanged.connect(boost::bind(&DOMFrame::setup_sensitivity, this));
 
                setup_sensitivity ();
 
@@ -292,7 +297,7 @@ private:
 #endif
 
 #ifdef __WXOSX__
-               file->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
+               file->Append(wxID_PREFERENCES, _("&Preferences...\tCtrl-,"));
 #else
                wxMenu* edit = new wxMenu;
                edit->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
@@ -300,7 +305,7 @@ private:
 
                wxMenu* help = new wxMenu;
 #ifdef __WXOSX__
-               help->Append (wxID_ABOUT, _("About DCP-o-matic"));
+               help->Append(wxID_ABOUT, variant::wx::insert_dcpomatic_kdm_creator(_("About %s")));
 #else
                help->Append (wxID_ABOUT, _("About"));
 #endif
@@ -380,12 +385,12 @@ private:
 
                        vector<KDMCertificatePeriod> period_checks;
 
-                       std::function<dcp::DecryptedKDM (dcp::LocalTime, dcp::LocalTime)> make_kdm = [decrypted, title](dcp::LocalTime begin, dcp::LocalTime end) {
+                       std::function<dcp::DecryptedKDM (dcp::LocalTime, dcp::LocalTime)> make_kdm = [this, decrypted, title](dcp::LocalTime begin, dcp::LocalTime end) {
                                /* Make an empty KDM */
                                dcp::DecryptedKDM kdm (
                                        begin,
                                        end,
-                                       decrypted.annotation_text().get_value_or(""),
+                                       _output->annotation_text(),
                                        title,
                                        dcp::LocalTime().as_string()
                                        );
@@ -458,7 +463,13 @@ private:
                        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 certificates' validity period.  Either use an earlier end time for this KDM or re-create your signing certificates in the DCP-o-matic preferences window."));
+                               error_dialog(
+                                       this,
+                                       variant::wx::insert_dcpomatic_kdm_creator(
+                                               _("The KDM end period is after (or close to) the end of the signing certificates' validity period.  "
+                                                 "Either use an earlier end time for this KDM or re-create your signing certificates in the %s preferences window.")
+                                               )
+                                       );
                        }
                        return;
                } catch (dcp::NotEncryptedError& e) {
@@ -478,7 +489,7 @@ private:
                _dkdm->GetSelections (sel);
                auto group = dynamic_pointer_cast<DKDMGroup>(selected_dkdm());
                auto dkdm = dynamic_pointer_cast<DKDM>(selected_dkdm());
-               _create->Enable(!_screens->screens().empty() && _timing->valid() && sel.GetCount() > 0 && dkdm);
+               _create->Enable(!_screens->screens().empty() && _timing->valid() && sel.GetCount() > 0 && dkdm && _output->method_selected());
                _remove_dkdm->Enable (sel.GetCount() > 0 && (!group || group->name() != "root"));
                _export_dkdm->Enable (sel.GetCount() > 0 && dkdm);
        }
@@ -486,6 +497,17 @@ private:
        void dkdm_selection_changed()
        {
                _selected_dkdm = selected_dkdm();
+               if (_selected_dkdm) {
+                       auto dkdm = std::dynamic_pointer_cast<DKDM>(_selected_dkdm);
+                       if (dkdm) {
+                               try {
+                                       dcp::DecryptedKDM decrypted(dkdm->dkdm(), Config::instance()->decryption_chain()->key().get());
+                                       if (decrypted.annotation_text()) {
+                                               _output->set_annotation_text(*decrypted.annotation_text());
+                                       }
+                               } catch (...) {}
+                       }
+               }
                setup_sensitivity();
        }
 
@@ -775,6 +797,12 @@ private:
                update_dkdm_view();
        }
 
+       void screens_changed()
+       {
+               _timing->suggest_utc_offset(_screens->best_utc_offset());
+               setup_sensitivity();
+       }
+
        wxPreferencesEditor* _config_dialog;
        ScreensPanel* _screens;
        KDMTimingPanel* _timing;
@@ -828,7 +856,7 @@ private:
 
                        splash = maybe_show_splash ();
 
-                       SetAppName (_("DCP-o-matic KDM Creator"));
+                       SetAppName(variant::wx::dcpomatic_kdm_creator());
 
                        if (!wxApp::OnInit()) {
                                return false;
@@ -862,7 +890,7 @@ private:
                        */
                        Config::drop ();
 
-                       _frame = new DOMFrame (_("DCP-o-matic KDM Creator"));
+                       _frame = new DOMFrame(variant::wx::dcpomatic_kdm_creator());
                        SetTopWindow (_frame);
                        _frame->Maximize ();
                        if (splash) {
@@ -879,7 +907,7 @@ private:
                        if (splash) {
                                splash->Destroy ();
                        }
-                       error_dialog (0, _("DCP-o-matic could not start"), std_to_wx(e.what()));
+                       error_dialog(nullptr, variant::wx::insert_dcpomatic_kdm_creator(_("%s could not start")), std_to_wx(e.what()));
                }
 
                return true;