X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Fwx%2Fjob_view.cc;h=77620142473af4a6d0823da4c283d2ffefbad0d9;hp=8e7040c3b855cb1d0f3ae87db05da65b7b7b5d6e;hb=5e74e417e8b3d4e9773d0fad97cf11a5c4eeeccf;hpb=ad5c8849fc3ef5aad88201f28db5474a60db4436 diff --git a/src/wx/job_view.cc b/src/wx/job_view.cc index 8e7040c3b..776201424 100644 --- a/src/wx/job_view.cc +++ b/src/wx/job_view.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2012-2015 Carl Hetherington + Copyright (C) 2012-2019 Carl Hetherington This file is part of DCP-o-matic. @@ -18,25 +18,42 @@ */ + #include "job_view.h" #include "wx_util.h" +#include "message_dialog.h" +#include "static_text.h" +#include "check_box.h" +#include "dcpomatic_button.h" #include "lib/job.h" +#include "lib/job_manager.h" #include "lib/compose.hpp" +#include "lib/config.h" +#include "lib/send_notification_email_job.h" +#include "lib/transcode_job.h" +#include "lib/analyse_audio_job.h" #include +#include + using std::string; using std::min; -using boost::shared_ptr; +using std::shared_ptr; +using boost::bind; +using std::dynamic_pointer_cast; + JobView::JobView (shared_ptr job, wxWindow* parent, wxWindow* container, wxFlexGridSizer* table) : _job (job) , _table (table) , _parent (parent) , _container (container) + , _gauge (0) { } + void JobView::setup () { @@ -47,25 +64,32 @@ JobView::setup () /* This seems to be required to allow the gauge to shrink under OS X */ _gauge->SetMinSize (wxSize (0, -1)); _gauge_message->Add (_gauge, 0, wxEXPAND | wxLEFT | wxRIGHT); - _message = new wxStaticText (_container, wxID_ANY, wxT (" \n "), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_MIDDLE); - _gauge_message->Add (_message, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6); + _message = new StaticText (_container, wxT(" \n "), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_MIDDLE); + _gauge_message->Add (_message, 1, wxEXPAND | wxALL, 6); _table->Insert (n, _gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT); ++n; - wxBoxSizer* buttons = new wxBoxSizer (wxHORIZONTAL); + _buttons = new wxBoxSizer (wxHORIZONTAL); - _cancel = new wxButton (_container, wxID_ANY, _("Cancel")); + _cancel = new Button (_container, _("Cancel")); _cancel->Bind (wxEVT_BUTTON, &JobView::cancel_clicked, this); - buttons->Add (_cancel, 1, wxALIGN_CENTER_VERTICAL); + _buttons->Add (_cancel, 1, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_BUTTON_STACK_GAP); - _details = new wxButton (_container, wxID_ANY, _("Details...")); + _details = new Button (_container, _("Details...")); _details->Bind (wxEVT_BUTTON, &JobView::details_clicked, this); _details->Enable (false); - buttons->Add (_details, 1, wxALIGN_CENTER_VERTICAL); + _buttons->Add (_details, 1, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_BUTTON_STACK_GAP); - finish_setup (_container, buttons); + finish_setup (_container, _buttons); - _table->Insert (n, buttons, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3); + _controls = new wxBoxSizer (wxVERTICAL); + _controls->Add (_buttons); + _notify = new CheckBox (_container, _("Notify when complete")); + _notify->Bind (wxEVT_CHECKBOX, bind (&JobView::notify_clicked, this)); + _notify->SetValue (Config::instance()->default_notify()); + _controls->Add (_notify, 0, wxTOP, DCPOMATIC_BUTTON_STACK_GAP); + + _table->Insert (n, _controls, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3); _progress_connection = _job->Progress.connect (boost::bind (&JobView::progress, this)); _finished_connection = _job->Finished.connect (boost::bind (&JobView::finished, this)); @@ -75,14 +99,19 @@ JobView::setup () _table->Layout (); } + void JobView::maybe_pulse () { - if (_job->running() && !_job->progress ()) { - _gauge->Pulse (); + if (_gauge && _job->running()) { + auto elapsed = _job->seconds_since_last_progress_update(); + if (!_job->progress() || !elapsed || *elapsed > 2) { + _gauge->Pulse (); + } } } + void JobView::progress () { @@ -90,7 +119,11 @@ JobView::progress () if (!_job->sub_name().empty ()) { whole += _job->sub_name() + " "; } - whole += _job->status (); + auto s = _job->status (); + /* Watch out for < > in the error string */ + boost::algorithm::replace_all (s, "<", "<"); + boost::algorithm::replace_all (s, ">", ">"); + whole += s; if (whole != _last_message) { _message->SetLabelMarkup (std_to_wx (whole)); /* This hack fixes the size of _message on OS X */ @@ -104,6 +137,7 @@ JobView::progress () } } + void JobView::finished () { @@ -114,19 +148,40 @@ JobView::finished () } _cancel->Enable (false); + _notify->Enable (false); if (!_job->error_details().empty ()) { _details->Enable (true); } + + if (_job->message()) { + auto d = new MessageDialog (_parent, std_to_wx(_job->name()), std_to_wx(_job->message().get())); + d->ShowModal (); + d->Destroy (); + } + + if ((dynamic_pointer_cast(_job) || dynamic_pointer_cast(_job)) && _notify->GetValue()) { + if (Config::instance()->notification(Config::MESSAGE_BOX)) { + wxMessageBox (std_to_wx(_job->name() + ": " + _job->status()), _("DCP-o-matic"), wxICON_INFORMATION); + } + if (Config::instance()->notification(Config::EMAIL)) { + string body = Config::instance()->notification_email(); + boost::algorithm::replace_all (body, "$JOB_NAME", _job->name()); + boost::algorithm::replace_all (body, "$JOB_STATUS", _job->status()); + JobManager::instance()->add_after (_job, shared_ptr (new SendNotificationEmailJob (body))); + } + } } + void JobView::details_clicked (wxCommandEvent &) { - string s = _job->error_summary(); + auto s = _job->error_summary(); s[0] = toupper (s[0]); - error_dialog (_parent, std_to_wx (String::compose ("%1.\n\n%2", s, _job->error_details()))); + error_dialog (_parent, std_to_wx(s), std_to_wx(_job->error_details())); } + void JobView::cancel_clicked (wxCommandEvent &) { @@ -134,3 +189,27 @@ JobView::cancel_clicked (wxCommandEvent &) _job->cancel (); } } + + +void +JobView::insert (int pos) +{ + _table->Insert (pos, _gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT); + _table->Insert (pos + 1, _controls, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3); + _table->Layout (); +} + + +void +JobView::detach () +{ + _table->Detach (_gauge_message); + _table->Detach (_controls); +} + + +void +JobView::notify_clicked () +{ + Config::instance()->set_default_notify (_notify->GetValue ()); +}