More renaming; text -> plain.

[dcpomatic.git] / src / wx / content_panel.cc

diff --git a/src/wx/content_panel.cc b/src/wx/content_panel.cc
index 7b69791634478f27277c1d7d5c0b8a09a3b4dce3..225ac5d6b22ebf6a6b836766df2697555c2d4ee7 100644 (file)
--- a/src/wx/content_panel.cc
+++ b/src/wx/content_panel.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -28,7 +28,7 @@
 #include "image_sequence_dialog.h"
 #include "film_viewer.h"
 #include "lib/audio_content.h"
-#include "lib/subtitle_content.h"
+#include "lib/text_content.h"
 #include "lib/video_content.h"
 #include "lib/ffmpeg_content.h"
 #include "lib/content_factory.h"
@@ -37,6 +37,10 @@
 #include "lib/case_insensitive_sorter.h"
 #include "lib/playlist.h"
 #include "lib/config.h"
+#include "lib/log.h"
+#include "lib/compose.hpp"
+#include "lib/plain_text_content.h"
+#include "lib/text_subtitle.h"
 #include <wx/wx.h>
 #include <wx/notebook.h>
 #include <wx/listctrl.h>
@@ -54,6 +58,8 @@ using boost::weak_ptr;
 using boost::dynamic_pointer_cast;
 using boost::optional;
 
+#define LOG_GENERAL(...) _film->log()->log (String::compose (__VA_ARGS__), LogEntry::TYPE_GENERAL);
+
 ContentPanel::ContentPanel (wxNotebook* n, boost::shared_ptr<Film> film, FilmViewer* viewer)
        : _timeline_dialog (0)
        , _parent (n)
@@ -250,18 +256,31 @@ ContentPanel::selection_changed ()
 
        optional<DCPTime> go_to;
        BOOST_FOREACH (shared_ptr<Content> i, selected ()) {
-               if (!go_to || i->position() < go_to.get()) {
-                       go_to = i->position ();
+               DCPTime p;
+               p = i->position();
+               if (dynamic_pointer_cast<PlainText>(i) && i->paths_valid()) {
+                       /* Rather special case; if we select a text subtitle file jump to its
+                          first subtitle.
+                       */
+                       TextSubtitle ts (dynamic_pointer_cast<PlainText>(i));
+                       if (ts.first()) {
+                               p += DCPTime(ts.first().get(), _film->active_frame_rate_change(i->position()));
+                       }
+               }
+               if (!go_to || p < go_to.get()) {
+                       go_to = p;
                }
        }
 
        if (go_to && Config::instance()->jump_to_selected ()) {
-               _film_viewer->set_position (go_to.get ());
+               _film_viewer->set_position (go_to.get().ceil(_film->video_frame_rate()));
        }
 
        if (_timeline_dialog) {
                _timeline_dialog->set_selection (selected ());
        }
+
+       SelectionChanged ();
 }
 
 void
@@ -390,6 +409,10 @@ ContentPanel::remove_clicked (bool hotkey)
 void
 ContentPanel::timeline_clicked ()
 {
+       if (!_film) {
+               return;
+       }
+
        if (_timeline_dialog) {
                _timeline_dialog->Destroy ();
                _timeline_dialog = 0;
@@ -417,14 +440,14 @@ ContentPanel::setup_sensitivity ()
        ContentList video_selection = selected_video ();
        ContentList audio_selection = selected_audio ();
 
-       _remove->Enable   (!selection.empty() && _generally_sensitive);
-       _earlier->Enable  (selection.size() == 1 && _generally_sensitive);
-       _later->Enable    (selection.size() == 1 && _generally_sensitive);
-       _timeline->Enable (!_film->content().empty() && _generally_sensitive);
+       _remove->Enable   (_generally_sensitive && !selection.empty());
+       _earlier->Enable  (_generally_sensitive && selection.size() == 1);
+       _later->Enable    (_generally_sensitive && selection.size() == 1);
+       _timeline->Enable (_generally_sensitive && _film && !_film->content().empty());
 
-       _video_panel->Enable    (video_selection.size() > 0 && _generally_sensitive);
-       _audio_panel->Enable    (audio_selection.size() > 0 && _generally_sensitive);
-       _subtitle_panel->Enable (selection.size() == 1 && selection.front()->subtitle && _generally_sensitive);
+       _video_panel->Enable    (_generally_sensitive && video_selection.size() > 0);
+       _audio_panel->Enable    (_generally_sensitive && audio_selection.size() > 0);
+       _subtitle_panel->Enable (_generally_sensitive && selection.size() == 1 && selection.front()->subtitle);
        _timing_panel->Enable   (_generally_sensitive);
 }
 
@@ -445,20 +468,7 @@ void
 ContentPanel::set_general_sensitivity (bool s)
 {
        _generally_sensitive = s;
-
-       _content->Enable (s);
-       _add_file->Enable (s);
-       _add_folder->Enable (s);
-       _add_dcp->Enable (s);
-       _remove->Enable (s);
-       _earlier->Enable (s);
-       _later->Enable (s);
-       _timeline->Enable (s);
-
-       /* Set the panels in the content notebook */
-       BOOST_FOREACH (ContentSubPanel* i, _panels) {
-               i->Enable (s);
-       }
+       setup_sensitivity ();
 }
 
 void
@@ -545,6 +555,12 @@ ContentPanel::setup ()
        BOOST_FOREACH (shared_ptr<Content> i, content) {
                int const t = _content->GetItemCount ();
                bool const valid = i->paths_valid ();
+
+               /* Temporary debugging for Igor */
+               BOOST_FOREACH (boost::filesystem::path j, i->paths()) {
+                       LOG_GENERAL ("Check %1 %2 answer %3", j.string(), boost::filesystem::exists(j) ? "yes" : "no", valid ? "yes" : "no");
+               }
+
                shared_ptr<DCPContent> dcp = dynamic_pointer_cast<DCPContent> (i);
                bool const needs_kdm = dcp && dcp->needs_kdm ();
                bool const needs_assets = dcp && dcp->needs_assets ();
@@ -582,6 +598,8 @@ ContentPanel::setup ()
                /* Select the item of content if none was selected before */
                _content->SetItemState (0, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED);
        }
+
+       setup_sensitivity ();
 }
 
 void
@@ -612,9 +630,13 @@ ContentPanel::add_files (list<boost::filesystem::path> paths)
 
        /* XXX: check for lots of files here and do something */
 
-       BOOST_FOREACH (boost::filesystem::path i, paths) {
-               BOOST_FOREACH (shared_ptr<Content> j, content_factory (_film, i)) {
-                       _film->examine_and_add_content (j);
+       try {
+               BOOST_FOREACH (boost::filesystem::path i, paths) {
+                       BOOST_FOREACH (shared_ptr<Content> j, content_factory (_film, i)) {
+                               _film->examine_and_add_content (j);
+                       }
                }
+       } catch (exception& e) {
+               error_dialog (_parent, e.what());
        }
 }