/*
- Copyright (C) 2012-2013 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
- This program is free software; you can redistribute it and/or modify
+ This file is part of DCP-o-matic.
+
+ DCP-o-matic is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
- This program is distributed in the hope that it will be useful,
+ DCP-o-matic is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
- along with this program; if not, write to the Free Software
- Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
*/
-#include <boost/algorithm/string.hpp>
-#include <wx/richtext/richtextctrl.h>
-#include "lib/film.h"
#include "hints_dialog.h"
-
-HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> f)
+#include "wx_util.h"
+#include "static_text.h"
+#include "check_box.h"
+#include "lib/film.h"
+#include "lib/hints.h"
+#include "lib/config.h"
+#include <wx/richtext/richtextctrl.h>
+#include <boost/foreach.hpp>
+
+using std::max;
+using std::vector;
+using std::string;
+using std::cout;
+using boost::shared_ptr;
+using boost::optional;
+using boost::bind;
+using boost::dynamic_pointer_cast;
+
+HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> film, bool ok)
: wxDialog (parent, wxID_ANY, _("Hints"))
- , _film (f)
+ , _film (film)
+ , _hints (0)
+ , _finished (false)
{
wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
+
+ _gauge = new wxGauge (this, wxID_ANY, 100);
+ sizer->Add (_gauge, 0, wxALL | wxEXPAND, DCPOMATIC_SIZER_GAP);
+ _gauge_message = new StaticText (this, wxT(""));
+ sizer->Add (_gauge_message, 0, wxALL | wxEXPAND, DCPOMATIC_SIZER_GAP);
+
_text = new wxRichTextCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (400, 300), wxRE_READONLY);
sizer->Add (_text, 1, wxEXPAND | wxALL, 6);
- wxSizer* buttons = CreateSeparatedButtonSizer (wxOK);
- if (buttons) {
- sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
+ if (!ok) {
+ wxCheckBox* b = new CheckBox (this, _("Don't show hints again"));
+ sizer->Add (b, 0, wxALL, 6);
+ b->Bind (wxEVT_CHECKBOX, bind (&HintsDialog::shut_up, this, _1));
+ }
+
+ wxStdDialogButtonSizer* buttons = CreateStdDialogButtonSizer (0);
+ sizer->Add (CreateSeparatedSizer(buttons), wxSizerFlags().Expand().DoubleBorder());
+ if (ok) {
+ buttons->SetAffirmativeButton (new wxButton (this, wxID_OK));
+ } else {
+ buttons->SetAffirmativeButton (new wxButton (this, wxID_OK, _("Make DCP")));
+ buttons->SetNegativeButton (new wxButton (this, wxID_CANCEL, _("Go back")));
}
+ buttons->Realize ();
+
SetSizer (sizer);
sizer->Layout ();
sizer->SetSizeHints (this);
_text->GetCaret()->Hide ();
- boost::shared_ptr<Film> film = _film.lock ();
- if (film) {
- film->Changed.connect (boost::bind (&HintsDialog::film_changed, this));
+ boost::shared_ptr<Film> locked_film = _film.lock ();
+ if (locked_film) {
+ _film_change_connection = locked_film->Change.connect (boost::bind (&HintsDialog::film_change, this, _1));
+ _film_content_change_connection = locked_film->ContentChange.connect (boost::bind (&HintsDialog::film_content_change, this, _1));
}
- film_changed ();
+ film_change (CHANGE_TYPE_DONE);
}
void
-HintsDialog::film_changed ()
+HintsDialog::film_change (ChangeType type)
{
+ if (type != CHANGE_TYPE_DONE) {
+ return;
+ }
+
_text->Clear ();
- bool hint = false;
-
+ _current.clear ();
+
boost::shared_ptr<Film> film = _film.lock ();
if (!film) {
return;
}
- _text->BeginStandardBullet (N_("standard/circle"), 1, 50);
- if (film->audio_channels() % 2) {
- hint = true;
- _text->WriteText (_("Your DCP has an odd number of audio channels. This is very likely to cause problems on playback."));
- _text->Newline ();
- } else if (film->audio_channels() < 6) {
- hint = true;
- _text->WriteText (_("Your DCP has fewer than 6 audio channels. This may cause problems on some projectors."));
- _text->Newline ();
- }
+ _gauge->Show ();
+ _gauge_message->Show ();
+ Layout ();
+ _gauge->SetValue (0);
+ update ();
+ _finished = false;
+
+ _hints.reset (new Hints (_film));
+ _hints->Hint.connect (bind (&HintsDialog::hint, this, _1));
+ _hints->Progress.connect (bind (&HintsDialog::progress, this, _1));
+ _hints->Pulse.connect (bind (&HintsDialog::pulse, this));
+ _hints->Finished.connect (bind (&HintsDialog::finished, this));
+ _hints->start ();
+}
- if (film->video_frame_rate() != 24 && film->video_frame_rate() != 48) {
- hint = true;
- _text->WriteText (wxString::Format (_("Your DCP frame rate (%d fps) may cause problems in a few (mostly older) projectors. Use 24 or 48 frames per second to be on the safe side."), film->video_frame_rate()));
- _text->Newline ();
- }
+void
+HintsDialog::film_content_change (ChangeType type)
+{
+ film_change (type);
+}
- ContentList content = film->content ();
- int vob = 0;
- for (ContentList::const_iterator i = content.begin(); i != content.end(); ++i) {
- if (boost::algorithm::starts_with ((*i)->path(0).filename().string(), "VTS_")) {
- ++vob;
+void
+HintsDialog::update ()
+{
+ _text->Clear ();
+ if (_current.empty ()) {
+ if (_finished) {
+ _text->WriteText (_("There are no hints: everything looks good!"));
+ } else {
+ _text->WriteText (_("There are no hints yet: project check in progress."));
+ }
+ } else {
+ _text->BeginStandardBullet (N_("standard/circle"), 1, 50);
+ BOOST_FOREACH (string i, _current) {
+ _text->WriteText (std_to_wx (i));
+ _text->Newline ();
}
+ _text->EndSymbolBullet ();
}
+}
- if (vob > 1) {
- hint = true;
- _text->WriteText (wxString::Format (_("You have %d files that look like they are VOB files from DVD. You should join them to ensure smooth joins between the files."), vob));
- _text->Newline ();
- }
+void
+HintsDialog::hint (string text)
+{
+ _current.push_back (text);
+ update ();
+}
+
+void
+HintsDialog::shut_up (wxCommandEvent& ev)
+{
+ Config::instance()->set_show_hints_before_make_dcp (!ev.IsChecked());
+}
- _text->EndSymbolBullet ();
+void
+HintsDialog::pulse ()
+{
+ _gauge->Pulse ();
+}
- if (!hint) {
- _text->WriteText (_("There are no hints: everything looks good!"));
+void
+HintsDialog::finished ()
+{
+ try {
+ _hints->rethrow ();
+ } catch (std::exception& e) {
+ error_dialog (this, wxString::Format(_("A problem occurred when looking for hints (%s)"), std_to_wx(e.what())));
}
+
+ _finished = true;
+ update ();
+ _gauge->Hide ();
+ _gauge_message->Hide ();
+ Layout ();
+}
+
+void
+HintsDialog::progress (string m)
+{
+ _gauge_message->SetLabel (std_to_wx(m));
}
projects / dcpomatic.git / blobdiff