Fix failure to remove markers when the checkbox is unticked.
[dcpomatic.git] / src / wx / markers_dialog.cc
1 /*
2     Copyright (C) 2019-2021 Carl Hetherington <cth@carlh.net>
3
4     This file is part of DCP-o-matic.
5
6     DCP-o-matic is free software; you can redistribute it and/or modify
7     it under the terms of the GNU General Public License as published by
8     the Free Software Foundation; either version 2 of the License, or
9     (at your option) any later version.
10
11     DCP-o-matic is distributed in the hope that it will be useful,
12     but WITHOUT ANY WARRANTY; without even the implied warranty of
13     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
14     GNU General Public License for more details.
15
16     You should have received a copy of the GNU General Public License
17     along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
18
19 */
20
21
22 #include "markers_dialog.h"
23 #include "wx_util.h"
24 #include "timecode.h"
25 #include "static_text.h"
26 #include "dcpomatic_button.h"
27 #include "check_box.h"
28 #include "film_viewer.h"
29 #include "lib/film.h"
30 #include <dcp/types.h>
31 #include <wx/gbsizer.h>
32 #include <boost/bind/bind.hpp>
33 #include <iostream>
34
35
36 using std::cout;
37 using std::shared_ptr;
38 using std::weak_ptr;
39 using std::make_shared;
40 using boost::optional;
41 using boost::bind;
42 using dcpomatic::DCPTime;
43
44
45 class Marker
46 {
47 public:
48         Marker (wxWindow* parent, wxGridBagSizer* grid, int row, weak_ptr<Film> film_, weak_ptr<FilmViewer> viewer_, wxString name, dcp::Marker type_)
49                 : film (film_)
50                 , viewer (viewer_)
51                 , type (type_)
52         {
53                 checkbox = new CheckBox(parent, name);
54                 grid->Add (checkbox, wxGBPosition(row, 0), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
55                 timecode = new Timecode<DCPTime> (parent);
56                 grid->Add (timecode, wxGBPosition(row, 1));
57                 set_button = new Button (parent, _("Set from current position"));
58                 grid->Add (set_button, wxGBPosition(row, 2));
59
60                 auto f = film.lock ();
61                 DCPOMATIC_ASSERT (f);
62
63                 auto t = f->marker (type);
64                 checkbox->SetValue (static_cast<bool>(t));
65                 if (t) {
66                         timecode->set (*t, f->video_frame_rate());
67                 }
68
69                 set_sensitivity ();
70
71                 set_button->Bind (wxEVT_BUTTON, bind(&Marker::set, this));
72                 checkbox->Bind (wxEVT_CHECKBOX, bind(&Marker::checkbox_clicked, this));
73                 timecode->Changed.connect (bind(&Marker::changed, this));
74         }
75
76 private:
77         void checkbox_clicked ()
78         {
79                 set_sensitivity ();
80                 changed ();
81         }
82
83         void set_sensitivity ()
84         {
85                 timecode->Enable (checkbox->GetValue());
86                 set_button->Enable (checkbox->GetValue());
87         }
88
89         void set ()
90         {
91                 auto f = film.lock ();
92                 DCPOMATIC_ASSERT (f);
93                 auto v = viewer.lock ();
94                 DCPOMATIC_ASSERT (v);
95                 timecode->set (v->position(), f->video_frame_rate());
96                 changed ();
97         }
98
99         void changed ()
100         {
101                 auto f = film.lock ();
102                 DCPOMATIC_ASSERT (f);
103                 auto vfr = f->video_frame_rate();
104                 auto tc = timecode->get(vfr);
105                 if (tc >= f->length()) {
106                         tc = f->length() - DCPTime::from_frames(1, vfr);
107                         timecode->set (tc, vfr);
108                 }
109                 if (checkbox->GetValue()) {
110                         f->set_marker (type, tc);
111                 } else {
112                         f->unset_marker (type);
113                 }
114         }
115
116         weak_ptr<Film> film;
117         weak_ptr<FilmViewer> viewer;
118         dcp::Marker type;
119         CheckBox* checkbox;
120         Timecode<dcpomatic::DCPTime>* timecode;
121         Button* set_button;
122 };
123
124
125 MarkersDialog::MarkersDialog (wxWindow* parent, weak_ptr<Film> film, weak_ptr<FilmViewer> viewer)
126         : wxDialog (parent, wxID_ANY, _("Markers"))
127         , _film (film)
128 {
129         auto sizer = new wxBoxSizer (wxVERTICAL);
130         auto grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
131
132         int r = 0;
133         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of composition"), dcp::Marker::FFOC));
134         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of composition"), dcp::Marker::LFOC));
135         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of title credits"), dcp::Marker::FFTC));
136         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of title credits"), dcp::Marker::LFTC));
137         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of intermission"), dcp::Marker::FFOI));
138         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of intermission"), dcp::Marker::LFOI));
139         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of end credits"), dcp::Marker::FFEC));
140         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of end credits"), dcp::Marker::LFEC));
141         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of moving credits"), dcp::Marker::FFMC));
142         _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of moving credits"), dcp::Marker::LFMC));
143
144         sizer->Add (grid, 0, wxALL, 8);
145
146         auto buttons = CreateSeparatedButtonSizer (wxCLOSE);
147         if (buttons) {
148                 sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
149         }
150
151         SetSizerAndFit (sizer);
152 }