Move UTC offset for KDMs from the cinema to the point of KDM creation (#2300).
[dcpomatic.git] / src / wx / kdm_timing_panel.cc
index 53f674e2ecbe01f6edc83ec80461cd68c2259467..7759d39496e8dec87e9c7cc4136fb85279605c6a 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2015-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2020 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
 
 */
 
+
+#include "dcpomatic_choice.h"
 #include "kdm_timing_panel.h"
-#include "wx_util.h"
+#include "static_text.h"
 #include "time_picker.h"
+#include "wx_util.h"
+#include "lib/config.h"
+#include <dcp/utc_offset.h>
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
 #include <wx/datectrl.h>
 #include <wx/dateevt.h>
+LIBDCP_ENABLE_WARNINGS
+
 
 using std::cout;
 using boost::bind;
 
+
 KDMTimingPanel::KDMTimingPanel (wxWindow* parent)
        : wxPanel (parent, wxID_ANY)
 {
-       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+       auto overall_sizer = new wxBoxSizer (wxVERTICAL);
+
+#ifdef __WXGTK3__
+       /* wxDatePickerCtrl is too small with the GTK3 backend so we need to make it bigger with some fudge factors */
+       wxClientDC dc (parent);
+       auto size = dc.GetTextExtent(wxT("99/99/9999"));
+       size.SetWidth (size.GetWidth() * 1.75);
+       size.SetHeight (-1);
+#else
+       auto size = wxDefaultSize;
+#endif
 
-       wxSizer* table = new wxBoxSizer (wxHORIZONTAL);
-       add_label_to_sizer (table, this, _("From"), true);
+       auto table = new wxBoxSizer (wxHORIZONTAL);
+       add_label_to_sizer (table, this, _("From"), false, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL);
        wxDateTime from;
        from.SetToCurrent ();
-       _from_date = new wxDatePickerCtrl (this, wxID_ANY, from);
-       table->Add (_from_date, 0, wxALIGN_CENTER_VERTICAL);
-       _from_time = new TimePicker (this, from);
+       _from_date = new wxDatePickerCtrl (this, wxID_ANY, from, wxDefaultPosition, size);
 #ifdef DCPOMATIC_OSX
        /* Hack to tweak alignment, which I can't get right by "proper" means for some reason */
-       table->Add (_from_time, 0, wxALIGN_CENTER_VERTICAL | wxTOP, 4);
+       table->Add (_from_date, 0, wxALIGN_CENTER_VERTICAL | wxBOTTOM, 4);
 #else
-       table->Add (_from_time, 0, wxALIGN_CENTER_VERTICAL);
+       table->Add (_from_date, 0, wxALIGN_CENTER_VERTICAL);
 #endif
 
-       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, 0, wxALIGN_CENTER_VERTICAL);
-       _until_time = new TimePicker (this, to);
+#ifdef __WXGTK3__
+       _from_time = new TimePickerText (this, from);
+#else
+       _from_time = new TimePickerSpin (this, from);
+#endif
+
+       table->Add (_from_time, 0, wxALIGN_CENTER_VERTICAL);
+
+       add_label_to_sizer (table, this, _("until"), false, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL);
+       auto to = from;
+
+       auto const duration = Config::instance()->default_kdm_duration();
+       switch (duration.unit) {
+       case RoughDuration::Unit::DAYS:
+               to.Add(wxDateSpan(0, 0, 0, duration.duration));
+               break;
+       case RoughDuration::Unit::WEEKS:
+               to.Add(wxDateSpan(0, 0, duration.duration, 0));
+               break;
+       case RoughDuration::Unit::MONTHS:
+               to.Add(wxDateSpan(0, duration.duration, 0, 0));
+               break;
+       case RoughDuration::Unit::YEARS:
+               to.Add(wxDateSpan(duration.duration, 0, 0, 0));
+               break;
+       }
+
+       _until_date = new wxDatePickerCtrl (this, wxID_ANY, to, wxDefaultPosition, size);
 #ifdef DCPOMATIC_OSX
-       table->Add (_until_time, 0, wxALIGN_CENTER_VERTICAL | wxTOP, 4);
+       /* Hack to tweak alignment, which I can't get right by "proper" means for some reason */
+       table->Add (_until_date, 0, wxALIGN_CENTER_VERTICAL | wxBOTTOM, 4);
 #else
-       table->Add (_until_time, 0, wxALIGN_CENTER_VERTICAL);
+       table->Add (_until_date, 0, wxALIGN_CENTER_VERTICAL);
 #endif
 
-       overall_sizer->Add (table);
+#ifdef __WXGTK3__
+       _until_time = new TimePickerText (this, to);
+#else
+       _until_time = new TimePickerSpin (this, to);
+#endif
 
-       _warning = new wxStaticText (this, wxID_ANY, wxT (""));
+       table->Add (_until_time, 0, wxALIGN_CENTER_VERTICAL);
+
+       add_label_to_sizer(table, this, _("UTC offset (time zone)"), true, 1, wxALIGN_CENTRE_VERTICAL);
+       _utc_offset = new Choice(this);
+       table->Add(_utc_offset, 0, wxALIGN_CENTRE_VERTICAL | wxLEFT, DCPOMATIC_SIZER_X_GAP);
+
+       overall_sizer->Add (table, 0, wxTOP, DCPOMATIC_SIZER_GAP);
+
+       _warning = new StaticText (this, wxT(""));
        overall_sizer->Add (_warning, 0, wxTOP, DCPOMATIC_SIZER_GAP);
        wxFont font = _warning->GetFont();
        font.SetStyle(wxFONTSTYLE_ITALIC);
@@ -69,42 +121,69 @@ KDMTimingPanel::KDMTimingPanel (wxWindow* parent)
        _warning->SetForegroundColour (wxColour (255, 0, 0));
        _warning->SetFont(font);
 
+       /* Default to UTC */
+       size_t sel = 0;
+       for (size_t i = 0; i < _offsets.size(); ++i) {
+               _utc_offset->add(_offsets[i].name);
+               if (_offsets[i].hour == 0 && _offsets[i].minute == 0) {
+                       sel = i;
+               }
+       }
+
+       _utc_offset->set(sel);
+
+       /* I said I've been to the year 3000.  Not much has changed but they live underwater.  And your In-in-in-interop DCP
+           is pretty fine.
+        */
+       _from_date->SetRange(wxDateTime(1, wxDateTime::Jan, 1900, 0, 0, 0, 0), wxDateTime(31, wxDateTime::Dec, 3000, 0, 0, 0, 0));
+       _until_date->SetRange(wxDateTime(1, wxDateTime::Jan, 1900, 0, 0, 0, 0), wxDateTime(31, wxDateTime::Dec, 3000, 0, 0, 0, 0));
+
        _from_date->Bind (wxEVT_DATE_CHANGED, bind (&KDMTimingPanel::changed, this));
        _until_date->Bind (wxEVT_DATE_CHANGED, bind (&KDMTimingPanel::changed, this));
        _from_time->Changed.connect (bind (&KDMTimingPanel::changed, this));
        _until_time->Changed.connect (bind (&KDMTimingPanel::changed, this));
+       _utc_offset->bind(&KDMTimingPanel::changed, this);
 
        SetSizer (overall_sizer);
 }
 
-boost::posix_time::ptime
+
+dcp::LocalTime
 KDMTimingPanel::from () const
 {
-       return posix_time (_from_date, _from_time);
+       return local_time(_from_date, _from_time, utc_offset());
 }
 
-boost::posix_time::ptime
-KDMTimingPanel::posix_time (wxDatePickerCtrl* date_picker, TimePicker* time_picker)
+
+dcp::LocalTime
+KDMTimingPanel::local_time(wxDatePickerCtrl* date_picker, TimePicker* time_picker, dcp::UTCOffset offset)
 {
-       wxDateTime const date = date_picker->GetValue ();
-       return boost::posix_time::ptime (
-               boost::gregorian::date (date.GetYear(), date.GetMonth() + 1, date.GetDay()),
-               boost::posix_time::time_duration (time_picker->hours(), time_picker->minutes(), 0)
+       auto const date = date_picker->GetValue ();
+       return dcp::LocalTime(
+               date.GetYear(),
+               date.GetMonth() + 1,
+               date.GetDay(),
+               time_picker->hours(),
+               time_picker->minutes(),
+               offset
                );
 }
 
-boost::posix_time::ptime
+
+dcp::LocalTime
 KDMTimingPanel::until () const
 {
-       return posix_time (_until_date, _until_time);
+       return local_time(_until_date, _until_time, utc_offset());
 }
 
+
 bool
 KDMTimingPanel::valid () const
 {
        return until() > from();
 }
 
+
 void
 KDMTimingPanel::changed () const
 {
@@ -116,3 +195,20 @@ KDMTimingPanel::changed () const
 
        TimingChanged ();
 }
+
+
+dcp::UTCOffset
+KDMTimingPanel::utc_offset() const
+{
+       auto const sel = _utc_offset->get();
+       if (!sel || *sel >= int(_offsets.size())) {
+               return {};
+       }
+
+       auto const& offset = _offsets[*sel];
+       int const minute_scale = offset.hour < 0 ? -1 : 1;
+
+       return { offset.hour, minute_scale * offset.minute };
+}
+
+