Update DCP name when some text changes from subtitle to ccap.
[dcpomatic.git] / src / wx / dcp_panel.cc
index d8aa3ac460986acb4e096cdd2b73ac2c31531184..d1de9aff7cd2512a40aa408bed2100feb8487106 100644 (file)
@@ -150,17 +150,70 @@ DCPPanel::DCPPanel(wxNotebook* n, shared_ptr<Film> film, FilmViewer& viewer)
        _reel_type->add(_("Split by video content"));
        /// TRANSLATORS: translate the word "Custom" here; do not include the "Reel|" prefix
        _reel_type->add(S_("Reel|Custom"));
+       _reel_type->SetToolTip(_("How the DCP should be split into parts internally.  If in doubt, choose 'Single reel'"));
 
        _reel_length->SetRange (1, 64);
 
-       _standard->add(_("SMPTE"));
-       _standard->add(_("Interop"));
+       add_standards();
+       _standard->SetToolTip(_("Which standard the DCP should use.  Interop is older and SMPTE is the modern standard.  If in doubt, choose 'SMPTE'"));
 
        Config::instance()->Changed.connect (boost::bind(&DCPPanel::config_changed, this, _1));
 
        add_to_grid ();
 }
 
+
+void
+DCPPanel::add_standards()
+{
+       _standard->add(_("SMPTE"), N_("smpte"));
+       if (Config::instance()->allow_smpte_bv20() || (_film && _film->limit_to_smpte_bv20())) {
+               _standard->add(_("SMPTE (Bv2.0 only)"), N_("smpte-bv20"));
+       }
+       _standard->add(_("Interop"), N_("interop"));
+       _sizer->Layout();
+}
+
+
+void
+DCPPanel::set_standard()
+{
+       DCPOMATIC_ASSERT(_film);
+       DCPOMATIC_ASSERT(!_film->limit_to_smpte_bv20() || _standard->GetCount() == 3);
+
+       if (_film->interop()) {
+               checked_set(_standard, "interop");
+       } else {
+               checked_set(_standard, _film->limit_to_smpte_bv20() ? "smpte-bv20" : "smpte");
+       }
+}
+
+
+void
+DCPPanel::standard_changed ()
+{
+       if (!_film || !_standard->get()) {
+               return;
+       }
+
+       auto const data = _standard->get_data();
+       if (!data) {
+               return;
+       }
+
+       if (*data == N_("interop")) {
+               _film->set_interop(true);
+               _film->set_limit_to_smpte_bv20(false);
+       } else if (*data == N_("smpte")) {
+               _film->set_interop(false);
+               _film->set_limit_to_smpte_bv20(false);
+       } else if (*data == N_("smpte-bv20")) {
+               _film->set_interop(false);
+               _film->set_limit_to_smpte_bv20(true);
+       }
+}
+
+
 void
 DCPPanel::add_to_grid ()
 {
@@ -313,26 +366,10 @@ DCPPanel::resolution_changed ()
 }
 
 
-void
-DCPPanel::standard_changed ()
-{
-       if (!_film || !_standard->get()) {
-               return;
-       }
-
-       _film->set_interop(*_standard->get() == 1);
-
-}
-
 void
 DCPPanel::markers_clicked ()
 {
-       if (_markers_dialog) {
-               _markers_dialog->Destroy ();
-               _markers_dialog = nullptr;
-       }
-
-       _markers_dialog = new MarkersDialog (_panel, _film, _viewer);
+       _markers_dialog.reset(_panel, _film, _viewer);
        _markers_dialog->Show();
 }
 
@@ -341,21 +378,11 @@ void
 DCPPanel::metadata_clicked ()
 {
        if (_film->interop()) {
-               if (_interop_metadata_dialog) {
-                       _interop_metadata_dialog->Destroy ();
-                       _interop_metadata_dialog = nullptr;
-               }
-
-               _interop_metadata_dialog = new InteropMetadataDialog (_panel, _film);
+               _interop_metadata_dialog.reset(_panel, _film);
                _interop_metadata_dialog->setup ();
                _interop_metadata_dialog->Show ();
        } else {
-               if (_smpte_metadata_dialog) {
-                       _smpte_metadata_dialog->Destroy ();
-                       _smpte_metadata_dialog = nullptr;
-               }
-
-               _smpte_metadata_dialog = new SMPTEMetadataDialog (_panel, _film);
+               _smpte_metadata_dialog.reset(_panel, _film);
                _smpte_metadata_dialog->setup ();
                _smpte_metadata_dialog->Show ();
        }
@@ -363,19 +390,19 @@ DCPPanel::metadata_clicked ()
 
 
 void
-DCPPanel::film_changed (Film::Property p)
+DCPPanel::film_changed(FilmProperty p)
 {
        switch (p) {
-       case Film::Property::NONE:
+       case FilmProperty::NONE:
                break;
-       case Film::Property::CONTAINER:
+       case FilmProperty::CONTAINER:
                setup_container ();
                break;
-       case Film::Property::NAME:
+       case FilmProperty::NAME:
                checked_set (_name, _film->name());
                setup_dcp_name ();
                break;
-       case Film::Property::DCP_CONTENT_TYPE:
+       case FilmProperty::DCP_CONTENT_TYPE:
        {
                auto index = DCPContentType::as_index(_film->dcp_content_type());
                DCPOMATIC_ASSERT (index);
@@ -383,18 +410,18 @@ DCPPanel::film_changed (Film::Property p)
                setup_dcp_name ();
                break;
        }
-       case Film::Property::ENCRYPTED:
+       case FilmProperty::ENCRYPTED:
                checked_set (_encrypted, _film->encrypted ());
                break;
-       case Film::Property::RESOLUTION:
+       case FilmProperty::RESOLUTION:
                checked_set (_resolution, _film->resolution() == Resolution::TWO_K ? 0 : 1);
                setup_container ();
                setup_dcp_name ();
                break;
-       case Film::Property::J2K_BANDWIDTH:
+       case FilmProperty::J2K_BANDWIDTH:
                checked_set (_j2k_bandwidth, _film->j2k_bandwidth() / 1000000);
                break;
-       case Film::Property::USE_ISDCF_NAME:
+       case FilmProperty::USE_ISDCF_NAME:
        {
                checked_set (_use_isdcf_name, _film->use_isdcf_name());
                if (_film->use_isdcf_name()) {
@@ -410,7 +437,7 @@ DCPPanel::film_changed (Film::Property p)
                setup_dcp_name ();
                break;
        }
-       case Film::Property::VIDEO_FRAME_RATE:
+       case FilmProperty::VIDEO_FRAME_RATE:
        {
                bool done = false;
                for (unsigned int i = 0; i < _frame_rate_choice->GetCount(); ++i) {
@@ -431,7 +458,7 @@ DCPPanel::film_changed (Film::Property p)
                setup_dcp_name ();
                break;
        }
-       case Film::Property::AUDIO_CHANNELS:
+       case FilmProperty::AUDIO_CHANNELS:
                if (_film->audio_channels() < minimum_allowed_audio_channels()) {
                        _film->set_audio_channels (minimum_allowed_audio_channels());
                } else {
@@ -439,39 +466,42 @@ DCPPanel::film_changed (Film::Property p)
                        setup_dcp_name ();
                }
                break;
-       case Film::Property::THREE_D:
+       case FilmProperty::THREE_D:
                checked_set (_three_d, _film->three_d());
                setup_dcp_name ();
                break;
-       case Film::Property::REENCODE_J2K:
+       case FilmProperty::REENCODE_J2K:
                checked_set (_reencode_j2k, _film->reencode_j2k());
                break;
-       case Film::Property::INTEROP:
-               checked_set (_standard, _film->interop() ? 1 : 0);
+       case FilmProperty::INTEROP:
+               set_standard();
                setup_dcp_name ();
                _markers->Enable (!_film->interop());
                break;
-       case Film::Property::AUDIO_PROCESSOR:
+       case FilmProperty::LIMIT_TO_SMPTE_BV20:
+               set_standard();
+               break;
+       case FilmProperty::AUDIO_PROCESSOR:
                if (_film->audio_processor()) {
                        checked_set (_audio_processor, _film->audio_processor()->id());
                } else {
                        checked_set (_audio_processor, 0);
                }
                setup_audio_channels_choice (_audio_channels, minimum_allowed_audio_channels());
-               film_changed (Film::Property::AUDIO_CHANNELS);
+               film_changed (FilmProperty::AUDIO_CHANNELS);
                break;
-       case Film::Property::REEL_TYPE:
+       case FilmProperty::REEL_TYPE:
                checked_set (_reel_type, static_cast<int>(_film->reel_type()));
                _reel_length->Enable (_film->reel_type() == ReelType::BY_LENGTH);
                break;
-       case Film::Property::REEL_LENGTH:
+       case FilmProperty::REEL_LENGTH:
                checked_set (_reel_length, _film->reel_length() / 1000000000LL);
                break;
-       case Film::Property::CONTENT:
+       case FilmProperty::CONTENT:
                setup_dcp_name ();
                setup_sensitivity ();
                break;
-       case Film::Property::AUDIO_LANGUAGE:
+       case FilmProperty::AUDIO_LANGUAGE:
        {
                auto al = _film->audio_language();
                checked_set (_enable_audio_language, static_cast<bool>(al));
@@ -480,23 +510,23 @@ DCPPanel::film_changed (Film::Property p)
                setup_sensitivity ();
                break;
        }
-       case Film::Property::AUDIO_FRAME_RATE:
+       case FilmProperty::AUDIO_FRAME_RATE:
                if (_audio_sample_rate) {
                        checked_set (_audio_sample_rate, _film->audio_frame_rate() == 48000 ? 0 : 1);
                }
                break;
-       case Film::Property::CONTENT_VERSIONS:
-       case Film::Property::VERSION_NUMBER:
-       case Film::Property::RELEASE_TERRITORY:
-       case Film::Property::RATINGS:
-       case Film::Property::FACILITY:
-       case Film::Property::STUDIO:
-       case Film::Property::TEMP_VERSION:
-       case Film::Property::PRE_RELEASE:
-       case Film::Property::RED_BAND:
-       case Film::Property::TWO_D_VERSION_OF_THREE_D:
-       case Film::Property::CHAIN:
-       case Film::Property::LUMINANCE:
+       case FilmProperty::CONTENT_VERSIONS:
+       case FilmProperty::VERSION_NUMBER:
+       case FilmProperty::RELEASE_TERRITORY:
+       case FilmProperty::RATINGS:
+       case FilmProperty::FACILITY:
+       case FilmProperty::STUDIO:
+       case FilmProperty::TEMP_VERSION:
+       case FilmProperty::PRE_RELEASE:
+       case FilmProperty::RED_BAND:
+       case FilmProperty::TWO_D_VERSION_OF_THREE_D:
+       case FilmProperty::CHAIN:
+       case FilmProperty::LUMINANCE:
                setup_dcp_name ();
                break;
        default:
@@ -513,6 +543,7 @@ DCPPanel::film_content_changed (int property)
            property == TextContentProperty::BURN ||
            property == TextContentProperty::LANGUAGE ||
            property == TextContentProperty::LANGUAGE_IS_ADDITIONAL ||
+           property == TextContentProperty::TYPE ||
            property == VideoContentProperty::CUSTOM_RATIO ||
            property == VideoContentProperty::CUSTOM_SIZE ||
            property == VideoContentProperty::BURNT_SUBTITLE_LANGUAGE ||
@@ -558,11 +589,10 @@ DCPPanel::container_changed ()
                return;
        }
 
-       auto const n = _container->get();
-       if (n) {
+       if (auto const container = _container->get()) {
                auto ratios = Ratio::containers ();
-               DCPOMATIC_ASSERT(*n < int(ratios.size()));
-               _film->set_container(ratios[*n]);
+               DCPOMATIC_ASSERT(*container < int(ratios.size()));
+               _film->set_container(ratios[*container]);
        }
 }
 
@@ -575,9 +605,8 @@ DCPPanel::dcp_content_type_changed ()
                return;
        }
 
-       auto n = _dcp_content_type->get();
-       if (n) {
-               _film->set_dcp_content_type(DCPContentType::from_index(*n));
+       if (auto const type = _dcp_content_type->get()) {
+               _film->set_dcp_content_type(DCPContentType::from_index(*type));
        }
 }
 
@@ -586,22 +615,10 @@ void
 DCPPanel::set_film (shared_ptr<Film> film)
 {
        /* We are changing film, so destroy any dialogs for the old one */
-       if (_audio_dialog) {
-               _audio_dialog->Destroy ();
-               _audio_dialog = nullptr;
-       }
-       if (_markers_dialog) {
-               _markers_dialog->Destroy ();
-               _markers_dialog = nullptr;
-       }
-       if (_interop_metadata_dialog) {
-               _interop_metadata_dialog->Destroy ();
-               _interop_metadata_dialog = nullptr;
-       }
-       if (_smpte_metadata_dialog) {
-               _smpte_metadata_dialog->Destroy ();
-               _smpte_metadata_dialog = nullptr;
-       }
+       _audio_dialog.reset();
+       _markers_dialog.reset();
+       _interop_metadata_dialog.reset();
+       _smpte_metadata_dialog.reset();
 
        _film = film;
 
@@ -612,25 +629,29 @@ DCPPanel::set_film (shared_ptr<Film> film)
                return;
        }
 
-       film_changed (Film::Property::NAME);
-       film_changed (Film::Property::USE_ISDCF_NAME);
-       film_changed (Film::Property::CONTENT);
-       film_changed (Film::Property::DCP_CONTENT_TYPE);
-       film_changed (Film::Property::CONTAINER);
-       film_changed (Film::Property::RESOLUTION);
-       film_changed (Film::Property::ENCRYPTED);
-       film_changed (Film::Property::J2K_BANDWIDTH);
-       film_changed (Film::Property::VIDEO_FRAME_RATE);
-       film_changed (Film::Property::AUDIO_CHANNELS);
-       film_changed (Film::Property::SEQUENCE);
-       film_changed (Film::Property::THREE_D);
-       film_changed (Film::Property::INTEROP);
-       film_changed (Film::Property::AUDIO_PROCESSOR);
-       film_changed (Film::Property::REEL_TYPE);
-       film_changed (Film::Property::REEL_LENGTH);
-       film_changed (Film::Property::REENCODE_J2K);
-       film_changed (Film::Property::AUDIO_LANGUAGE);
-       film_changed (Film::Property::AUDIO_FRAME_RATE);
+       _standard->Clear();
+       add_standards();
+
+       film_changed(FilmProperty::NAME);
+       film_changed(FilmProperty::USE_ISDCF_NAME);
+       film_changed(FilmProperty::CONTENT);
+       film_changed(FilmProperty::DCP_CONTENT_TYPE);
+       film_changed(FilmProperty::CONTAINER);
+       film_changed(FilmProperty::RESOLUTION);
+       film_changed(FilmProperty::ENCRYPTED);
+       film_changed(FilmProperty::J2K_BANDWIDTH);
+       film_changed(FilmProperty::VIDEO_FRAME_RATE);
+       film_changed(FilmProperty::AUDIO_CHANNELS);
+       film_changed(FilmProperty::SEQUENCE);
+       film_changed(FilmProperty::THREE_D);
+       film_changed(FilmProperty::INTEROP);
+       film_changed(FilmProperty::AUDIO_PROCESSOR);
+       film_changed(FilmProperty::REEL_TYPE);
+       film_changed(FilmProperty::REEL_LENGTH);
+       film_changed(FilmProperty::REENCODE_J2K);
+       film_changed(FilmProperty::AUDIO_LANGUAGE);
+       film_changed(FilmProperty::AUDIO_FRAME_RATE);
+       film_changed(FilmProperty::LIMIT_TO_SMPTE_BV20);
 
        set_general_sensitivity(static_cast<bool>(_film));
 }
@@ -749,8 +770,17 @@ DCPPanel::config_changed (Config::Property p)
                _audio_processor->Clear ();
                add_audio_processors ();
                if (_film) {
-                       film_changed (Film::Property::AUDIO_PROCESSOR);
+                       film_changed(FilmProperty::AUDIO_PROCESSOR);
+               }
+       } else if (p == Config::ALLOW_SMPTE_BV20) {
+               _standard->Clear();
+               add_standards();
+               if (_film) {
+                       film_changed(FilmProperty::INTEROP);
+                       film_changed(FilmProperty::LIMIT_TO_SMPTE_BV20);
                }
+       } else if (p == Config::ISDCF_NAME_PART_LENGTH) {
+               setup_dcp_name();
        }
 }
 
@@ -765,6 +795,7 @@ DCPPanel::setup_frame_rate_widget ()
                _frame_rate_choice->Show ();
                _frame_rate_spin->Hide ();
        }
+       _frame_rate_sizer->Layout();
 }
 
 
@@ -787,7 +818,6 @@ DCPPanel::make_video_panel ()
        _frame_rate_label = create_label (panel, _("Frame Rate"), true);
        _frame_rate_choice = new Choice(panel);
        _frame_rate_spin = new SpinCtrl (panel, DCPOMATIC_SPIN_CTRL_WIDTH);
-       setup_frame_rate_widget ();
        _best_frame_rate = new Button (panel, _("Use best"));
 
        _three_d = new CheckBox (panel, _("3D"));
@@ -824,6 +854,7 @@ DCPPanel::make_video_panel ()
        _resolution->add(_("4K"));
 
        add_video_panel_to_grid ();
+       setup_frame_rate_widget();
 
        return panel;
 }
@@ -979,13 +1010,8 @@ DCPPanel::show_audio_clicked ()
                return;
        }
 
-       if (_audio_dialog) {
-               _audio_dialog->Destroy ();
-               _audio_dialog = nullptr;
-       }
-
-       auto d = new AudioDialog (_panel, _film, _viewer);
-       d->Show ();
+       _audio_dialog.reset(_panel, _film, _viewer);
+       _audio_dialog->Show();
 }
 
 
@@ -1039,11 +1065,10 @@ void
 DCPPanel::edit_audio_language_clicked ()
 {
        DCPOMATIC_ASSERT (_film->audio_language());
-       auto d = new LanguageTagDialog (_panel, *_film->audio_language());
+       auto d = make_wx<LanguageTagDialog>(_panel, *_film->audio_language());
        if (d->ShowModal() == wxID_OK) {
               _film->set_audio_language(d->get());
        }
-       d->Destroy ();
 }