X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fhints_dialog.cc;h=cb0de450a73ad7d141941536c003f4f057cfddf7;hb=d6e899f5f9e3f3db10dbe17cf68508556ad5a83c;hp=cb971a671a02f9d82ca1eae733d977acc1e43b1d;hpb=b74bc6135a6a690bf5674638a8bea56e5580fd08;p=dcpomatic.git

diff --git a/src/wx/hints_dialog.cc b/src/wx/hints_dialog.cc
index cb971a671..cb0de450a 100644
--- a/src/wx/hints_dialog.cc
+++ b/src/wx/hints_dialog.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.
 
@@ -20,6 +20,8 @@
 
 #include "hints_dialog.h"
 #include "wx_util.h"
+#include "static_text.h"
+#include "check_box.h"
 #include "lib/film.h"
 #include "lib/hints.h"
 #include "lib/config.h"
@@ -29,22 +31,32 @@
 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 (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);
 
 	if (!ok) {
-		wxCheckBox* b = new wxCheckBox (this, wxID_ANY, _("Don't show hints again"));
+		wxCheckBox* b = new CheckBox (this, _("Don't show hints again"));
 		sizer->Add (b, 0, wxALL, 6);
-		b->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, bind (&HintsDialog::shut_up, this, _1));
+		b->Bind (wxEVT_CHECKBOX, bind (&HintsDialog::shut_up, this, _1));
 	}
 
 	wxStdDialogButtonSizer* buttons = CreateStdDialogButtonSizer (0);
@@ -52,7 +64,7 @@ HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> film, bool ok)
 	if (ok) {
 		buttons->SetAffirmativeButton (new wxButton (this, wxID_OK));
 	} else {
-		buttons->SetAffirmativeButton (new wxButton (this, wxID_OK, _("Make DCP anyway")));
+		buttons->SetAffirmativeButton (new wxButton (this, wxID_OK, _("Make DCP")));
 		buttons->SetNegativeButton (new wxButton (this, wxID_CANCEL, _("Go back")));
 	}
 
@@ -66,30 +78,62 @@ HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> film, bool ok)
 
 	boost::shared_ptr<Film> locked_film = _film.lock ();
 	if (locked_film) {
-		_film_changed_connection = locked_film->Changed.connect (boost::bind (&HintsDialog::film_changed, this));
-		_film_content_changed_connection = locked_film->ContentChanged.connect (boost::bind (&HintsDialog::film_changed, this));
+		_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 ();
+	_current.clear ();
 
 	boost::shared_ptr<Film> film = _film.lock ();
 	if (!film) {
 		return;
 	}
 
-	vector<string> hints = get_hints (film);
+	_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 ();
+}
+
+void
+HintsDialog::film_content_change (ChangeType type)
+{
+	film_change (type);
+}
 
-	if (hints.empty ()) {
-		_text->WriteText (_("There are no hints: everything looks good!"));
+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, hints) {
+		BOOST_FOREACH (string i, _current) {
 			_text->WriteText (std_to_wx (i));
 			_text->Newline ();
 		}
@@ -97,8 +141,43 @@ HintsDialog::film_changed ()
 	}
 }
 
+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());
 }
+
+void
+HintsDialog::pulse ()
+{
+	_gauge->Pulse ();
+}
+
+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));
+}