UI tweaks for still image DCPs with audio.
[dcpomatic.git] / src / wx / film_editor.cc
index d1b341bb7a57e00f1f3011d03f9737ad0d38f2da..999ea8753f68bd45b9141bfc9a2c536da3b38e23 100644 (file)
@@ -35,8 +35,9 @@
 #include "lib/ab_transcode_job.h"
 #include "lib/job_manager.h"
 #include "lib/filter.h"
-#include "lib/screen.h"
 #include "lib/config.h"
+#include "lib/ffmpeg_decoder.h"
+#include "lib/external_audio_decoder.h"
 #include "filter_dialog.h"
 #include "wx_util.h"
 #include "film_editor.h"
@@ -54,6 +55,7 @@ using std::setprecision;
 using std::list;
 using std::vector;
 using boost::shared_ptr;
+using boost::dynamic_pointer_cast;
 
 /** @param f Film to edit */
 FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
@@ -65,7 +67,7 @@ FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
        SetSizer (s);
        _notebook = new wxNotebook (this, wxID_ANY);
        s->Add (_notebook, 1);
-       
+
        make_film_panel ();
        _notebook->AddPage (_film_panel, _("Film"), true);
        make_video_panel ();
@@ -91,7 +93,9 @@ FilmEditor::make_film_panel ()
 {
        _film_panel = new wxPanel (_notebook);
        _film_sizer = new wxFlexGridSizer (2, 4, 4);
-       _film_panel->SetSizer (_film_sizer);
+       wxBoxSizer* pad = new wxBoxSizer (wxVERTICAL);
+       pad->Add (_film_sizer, 0, wxALL, 8);
+       _film_panel->SetSizer (pad);
 
        add_label_to_sizer (_film_sizer, _film_panel, "Name");
        _name = new wxTextCtrl (_film_panel, wxID_ANY);
@@ -110,6 +114,11 @@ FilmEditor::make_film_panel ()
        _content = new wxFilePickerCtrl (_film_panel, wxID_ANY, wxT (""), wxT ("Select Content File"), wxT("*.*"));
        _film_sizer->Add (_content, 1, wxEXPAND);
 
+       _trust_content_header = new wxCheckBox (_film_panel, wxID_ANY, wxT ("Trust content's header"));
+       video_control (_trust_content_header);
+       _film_sizer->Add (_trust_content_header, 1);
+       _film_sizer->AddSpacer (0);
+
        add_label_to_sizer (_film_sizer, _film_panel, "Content Type");
        _dcp_content_type = new wxComboBox (_film_panel, wxID_ANY, wxT (""), wxDefaultPosition, wxDefaultSize, 0, 0, wxCB_READONLY);
        _film_sizer->Add (_dcp_content_type);
@@ -130,12 +139,12 @@ FilmEditor::make_film_panel ()
        {
                video_control (add_label_to_sizer (_film_sizer, _film_panel, "Trim frames"));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               add_label_to_sizer (s, _film_panel, "Start");
+               video_control (add_label_to_sizer (s, _film_panel, "Start"));
                _dcp_trim_start = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_dcp_trim_start);
-               add_label_to_sizer (s, _film_panel, "End");
+               s->Add (video_control (_dcp_trim_start));
+               video_control (add_label_to_sizer (s, _film_panel, "End"));
                _dcp_trim_end = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_dcp_trim_end);
+               s->Add (video_control (_dcp_trim_end));
 
                _film_sizer->Add (s);
        }
@@ -170,6 +179,7 @@ FilmEditor::connect_to_widgets ()
        _edit_dci_button->Connect (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (FilmEditor::edit_dci_button_clicked), 0, this);
        _format->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::format_changed), 0, this);
        _content->Connect (wxID_ANY, wxEVT_COMMAND_FILEPICKER_CHANGED, wxCommandEventHandler (FilmEditor::content_changed), 0, this);
+       _trust_content_header->Connect (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler (FilmEditor::trust_content_header_changed), 0, this);
        _left_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::left_crop_changed), 0, this);
        _right_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::right_crop_changed), 0, this);
        _top_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::top_crop_changed), 0, this);
@@ -205,7 +215,9 @@ FilmEditor::make_video_panel ()
 {
        _video_panel = new wxPanel (_notebook);
        _video_sizer = new wxFlexGridSizer (2, 4, 4);
-       _video_panel->SetSizer (_video_sizer);
+       wxBoxSizer* pad = new wxBoxSizer (wxVERTICAL);
+       pad->Add (_video_sizer, 0, wxALL, 8);
+       _video_panel->SetSizer (pad);
 
        add_label_to_sizer (_video_sizer, _video_panel, "Format");
        _format = new wxComboBox (_video_panel, wxID_ANY, wxT (""), wxDefaultPosition, wxDefaultSize, 0, 0, wxCB_READONLY);
@@ -257,7 +269,7 @@ FilmEditor::make_video_panel ()
        _top_crop->SetRange (0, 1024);
        _right_crop->SetRange (0, 1024);
        _bottom_crop->SetRange (0, 1024);
-       _still_duration->SetRange (0, 60 * 60);
+       _still_duration->SetRange (1, 60 * 60);
        _dcp_trim_start->SetRange (0, 100);
        _dcp_trim_end->SetRange (0, 100);
 }
@@ -267,7 +279,9 @@ FilmEditor::make_audio_panel ()
 {
        _audio_panel = new wxPanel (_notebook);
        _audio_sizer = new wxFlexGridSizer (2, 4, 4);
-       _audio_panel->SetSizer (_audio_sizer);
+       wxBoxSizer* pad = new wxBoxSizer (wxVERTICAL);
+       pad->Add (_audio_sizer, 0, wxALL, 8);
+       _audio_panel->SetSizer (pad);
 
        {
                video_control (add_label_to_sizer (_audio_sizer, _audio_panel, "Audio Gain"));
@@ -302,7 +316,7 @@ FilmEditor::make_audio_panel ()
        }
 
        _use_external_audio = new wxRadioButton (_audio_panel, wxID_ANY, _("Use external audio"));
-       _audio_sizer->Add (video_control (_use_external_audio));
+       _audio_sizer->Add (_use_external_audio);
        _audio_sizer->AddSpacer (0);
 
        assert (MAX_AUDIO_CHANNELS == 6);
@@ -319,7 +333,7 @@ FilmEditor::make_audio_panel ()
        for (int i = 0; i < MAX_AUDIO_CHANNELS; ++i) {
                add_label_to_sizer (_audio_sizer, _audio_panel, channels[i]);
                _external_audio[i] = new wxFilePickerCtrl (_audio_panel, wxID_ANY, wxT (""), wxT ("Select Audio File"), wxT ("*.wav"));
-               _audio_sizer->Add (video_control (_external_audio[i]), 1, wxEXPAND);
+               _audio_sizer->Add (_external_audio[i], 1, wxEXPAND);
        }
 
        _audio_gain->SetRange (-60, 60);
@@ -331,14 +345,16 @@ FilmEditor::make_subtitle_panel ()
 {
        _subtitle_panel = new wxPanel (_notebook);
        _subtitle_sizer = new wxFlexGridSizer (2, 4, 4);
-       _subtitle_panel->SetSizer (_subtitle_sizer);
+       wxBoxSizer* pad = new wxBoxSizer (wxVERTICAL);
+       pad->Add (_subtitle_sizer, 0, wxALL, 8);
+       _subtitle_panel->SetSizer (pad);
 
        _with_subtitles = new wxCheckBox (_subtitle_panel, wxID_ANY, wxT("With Subtitles"));
        video_control (_with_subtitles);
        _subtitle_sizer->Add (_with_subtitles, 1);
        
        _subtitle_stream = new wxComboBox (_subtitle_panel, wxID_ANY, wxT (""), wxDefaultPosition, wxDefaultSize, 0, 0, wxCB_READONLY);
-       _subtitle_sizer->Add (_subtitle_stream);
+       _subtitle_sizer->Add (video_control (_subtitle_stream));
 
        video_control (add_label_to_sizer (_subtitle_sizer, _subtitle_panel, "Subtitle Offset"));
        _subtitle_offset = new wxSpinCtrl (_subtitle_panel);
@@ -417,6 +433,16 @@ FilmEditor::content_changed (wxCommandEvent &)
        }
 }
 
+void
+FilmEditor::trust_content_header_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
+       }
+
+       _film->set_trust_content_header (_trust_content_header->GetValue ());
+}
+
 /** Called when the DCP A/B switch has been toggled */
 void
 FilmEditor::dcp_ab_toggled (wxCommandEvent &)
@@ -485,11 +511,16 @@ FilmEditor::film_changed (Film::Property p)
                setup_subtitle_control_sensitivity ();
                setup_streams ();
                break;
-       case Film::AUDIO_STREAMS:
+       case Film::TRUST_CONTENT_HEADER:
+               checked_set (_trust_content_header, _film->trust_content_header ());
+               break;
        case Film::SUBTITLE_STREAMS:
                setup_subtitle_control_sensitivity ();
                setup_streams ();
                break;
+       case Film::CONTENT_AUDIO_STREAMS:
+               setup_streams ();
+               break;
        case Film::FORMAT:
        {
                int n = 0;
@@ -498,7 +529,7 @@ FilmEditor::film_changed (Film::Property p)
                        ++i;
                        ++n;
                }
-               if (i != _formats.end()) {
+               if (i == _formats.end()) {
                        checked_set (_format, -1);
                } else {
                        checked_set (_format, n);
@@ -557,8 +588,6 @@ FilmEditor::film_changed (Film::Property p)
                checked_set (_dcp_content_type, DCPContentType::as_index (_film->dcp_content_type ()));
                _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
                break;
-       case Film::THUMBS:
-               break;
        case Film::DCP_AB:
                checked_set (_dcp_ab, _film->dcp_ab ());
                break;
@@ -600,31 +629,33 @@ FilmEditor::film_changed (Film::Property p)
        case Film::DCI_METADATA:
                _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
                break;
-       case Film::AUDIO_STREAM:
-               if (_film->audio_stream()) {
-                       checked_set (_audio_stream, _film->audio_stream().get().to_string());
-               } else {
-                       checked_set (_audio_stream, wxNOT_FOUND);
+       case Film::CONTENT_AUDIO_STREAM:
+               if (_film->content_audio_stream()) {
+                       checked_set (_audio_stream, _film->content_audio_stream()->to_string());
                }
                _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
                setup_audio_details ();
+               setup_audio_control_sensitivity ();
+               break;
+       case Film::USE_CONTENT_AUDIO:
+               checked_set (_use_content_audio, _film->use_content_audio());
+               checked_set (_use_external_audio, !_film->use_content_audio());
+               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               setup_audio_details ();
+               setup_audio_control_sensitivity ();
                break;
        case Film::SUBTITLE_STREAM:
                if (_film->subtitle_stream()) {
-                       checked_set (_subtitle_stream, _film->subtitle_stream().get().to_string());
+                       checked_set (_subtitle_stream, _film->subtitle_stream()->to_string());
                }
                break;
-       case Film::USE_CONTENT_AUDIO:
-               checked_set (_use_content_audio, _film->use_content_audio ());
-               checked_set (_use_external_audio, !_film->use_content_audio ());
-               setup_audio_control_sensitivity ();
-               break;
        case Film::EXTERNAL_AUDIO:
        {
                vector<string> a = _film->external_audio ();
                for (size_t i = 0; i < a.size() && i < MAX_AUDIO_CHANNELS; ++i) {
                        checked_set (_external_audio[i], a[i]);
                }
+               setup_audio_details ();
                break;
        }
        }
@@ -680,6 +711,7 @@ FilmEditor::set_film (shared_ptr<Film> f)
        film_changed (Film::NAME);
        film_changed (Film::USE_DCI_NAME);
        film_changed (Film::CONTENT);
+       film_changed (Film::TRUST_CONTENT_HEADER);
        film_changed (Film::DCP_CONTENT_TYPE);
        film_changed (Film::FORMAT);
        film_changed (Film::CROP);
@@ -688,9 +720,9 @@ FilmEditor::set_film (shared_ptr<Film> f)
        film_changed (Film::DCP_TRIM_START);
        film_changed (Film::DCP_TRIM_END);
        film_changed (Film::DCP_AB);
-       film_changed (Film::USE_CONTENT_AUDIO);
-       film_changed (Film::AUDIO_STREAM);
+       film_changed (Film::CONTENT_AUDIO_STREAM);
        film_changed (Film::EXTERNAL_AUDIO);
+       film_changed (Film::USE_CONTENT_AUDIO);
        film_changed (Film::AUDIO_GAIN);
        film_changed (Film::AUDIO_DELAY);
        film_changed (Film::WITH_SUBTITLES);
@@ -699,7 +731,7 @@ FilmEditor::set_film (shared_ptr<Film> f)
        film_changed (Film::DCI_METADATA);
        film_changed (Film::SIZE);
        film_changed (Film::LENGTH);
-       film_changed (Film::AUDIO_STREAMS);
+       film_changed (Film::CONTENT_AUDIO_STREAMS);
        film_changed (Film::SUBTITLE_STREAMS);
        film_changed (Film::FRAMES_PER_SECOND);
 }
@@ -717,6 +749,7 @@ FilmEditor::set_things_sensitive (bool s)
        _edit_dci_button->Enable (s);
        _format->Enable (s);
        _content->Enable (s);
+       _trust_content_header->Enable (s);
        _left_crop->Enable (s);
        _right_crop->Enable (s);
        _top_crop->Enable (s);
@@ -965,24 +998,23 @@ void
 FilmEditor::setup_streams ()
 {
        _audio_stream->Clear ();
-       vector<AudioStream> a = _film->audio_streams ();
-       for (vector<AudioStream>::iterator i = a.begin(); i != a.end(); ++i) {
-               _audio_stream->Append (std_to_wx (i->name()), new wxStringClientData (std_to_wx (i->to_string ())));
+       vector<shared_ptr<AudioStream> > a = _film->content_audio_streams ();
+       for (vector<shared_ptr<AudioStream> >::iterator i = a.begin(); i != a.end(); ++i) {
+               shared_ptr<FFmpegAudioStream> ffa = dynamic_pointer_cast<FFmpegAudioStream> (*i);
+               _audio_stream->Append (std_to_wx (ffa->name()), new wxStringClientData (std_to_wx (ffa->to_string ())));
        }
        
-       if (_film->audio_stream()) {
-               checked_set (_audio_stream, _film->audio_stream().get().to_string());
-       } else {
-               _audio_stream->SetValue (wxT (""));
+       if (_film->use_content_audio() && _film->audio_stream()) {
+               checked_set (_audio_stream, _film->audio_stream()->to_string());
        }
 
        _subtitle_stream->Clear ();
-       vector<SubtitleStream> s = _film->subtitle_streams ();
-       for (vector<SubtitleStream>::iterator i = s.begin(); i != s.end(); ++i) {
-               _subtitle_stream->Append (std_to_wx (i->name()), new wxStringClientData (std_to_wx (i->to_string ())));
+       vector<shared_ptr<SubtitleStream> > s = _film->subtitle_streams ();
+       for (vector<shared_ptr<SubtitleStream> >::iterator i = s.begin(); i != s.end(); ++i) {
+               _subtitle_stream->Append (std_to_wx ((*i)->name()), new wxStringClientData (std_to_wx ((*i)->to_string ())));
        }
        if (_film->subtitle_stream()) {
-               checked_set (_subtitle_stream, _film->subtitle_stream().get().to_string());
+               checked_set (_subtitle_stream, _film->subtitle_stream()->to_string());
        } else {
                _subtitle_stream->SetValue (wxT (""));
        }
@@ -995,7 +1027,12 @@ FilmEditor::audio_stream_changed (wxCommandEvent &)
                return;
        }
 
-       _film->set_audio_stream (AudioStream (string_client_data (_audio_stream->GetClientObject (_audio_stream->GetSelection ()))));
+       _film->set_content_audio_stream (
+               audio_stream_factory (
+                       string_client_data (_audio_stream->GetClientObject (_audio_stream->GetSelection ())),
+                       Film::state_version
+                       )
+               );
 }
 
 void
@@ -1005,7 +1042,12 @@ FilmEditor::subtitle_stream_changed (wxCommandEvent &)
                return;
        }
 
-       _film->set_subtitle_stream (SubtitleStream (string_client_data (_subtitle_stream->GetClientObject (_subtitle_stream->GetSelection ()))));
+       _film->set_subtitle_stream (
+               subtitle_stream_factory (
+                       string_client_data (_subtitle_stream->GetClientObject (_subtitle_stream->GetSelection ())),
+                       Film::state_version
+                       )
+               );
 }
 
 void
@@ -1015,7 +1057,7 @@ FilmEditor::setup_audio_details ()
                _audio->SetLabel (wxT (""));
        } else {
                stringstream s;
-               s << _film->audio_stream().get().channels () << " channels, " << _film->audio_stream().get().sample_rate() << "Hz";
+               s << _film->audio_stream()->channels () << " channels, " << _film->audio_stream()->sample_rate() << "Hz";
                _audio->SetLabel (std_to_wx (s.str ()));
        }
 }
@@ -1029,8 +1071,7 @@ FilmEditor::active_jobs_changed (bool a)
 void
 FilmEditor::use_audio_changed (wxCommandEvent &)
 {
-       _film->set_use_content_audio (_use_content_audio->GetValue ());
-       setup_audio_control_sensitivity ();
+       _film->set_use_content_audio (_use_content_audio->GetValue());
 }
 
 void