X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Fwx%2Fjob_manager_view.cc;h=42d5f9dbeb5e5aade18cec6c3216abc3ff4a5234;hp=7361f29a815896d1021a5a1ed06a1500cc293f8c;hb=9001a63be211fd8e97431f8fc07c66af01554f5a;hpb=51b4acfbaf0e2cd8b87bb675e023d9e940febc89 diff --git a/src/wx/job_manager_view.cc b/src/wx/job_manager_view.cc index 7361f29a8..42d5f9dbe 100644 --- a/src/wx/job_manager_view.cc +++ b/src/wx/job_manager_view.cc @@ -1,19 +1,20 @@ /* - Copyright (C) 2012 Carl Hetherington + Copyright (C) 2012-2017 Carl Hetherington - 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 . */ @@ -21,118 +22,117 @@ * @brief Class generating a GTK widget to show the progress of jobs. */ +#include "job_manager_view.h" +#include "batch_job_view.h" +#include "normal_job_view.h" +#include "wx_util.h" #include "lib/job_manager.h" #include "lib/job.h" #include "lib/util.h" #include "lib/exceptions.h" -#include "job_manager_view.h" -#include "wx_util.h" +#include "lib/compose.hpp" +#include +#include using std::string; using std::list; using std::map; +using std::min; +using std::cout; using boost::shared_ptr; +using boost::weak_ptr; +using boost::bind; -/** Must be called in the GUI thread */ -JobManagerView::JobManagerView (wxWindow* parent) +/** @param parent Parent window. + * @param batch true to use BatchJobView, false to use NormalJobView. + * + * Must be called in the GUI thread. + */ +JobManagerView::JobManagerView (wxWindow* parent, bool batch) : wxScrolledWindow (parent) + , _batch (batch) { _panel = new wxPanel (this); wxSizer* sizer = new wxBoxSizer (wxVERTICAL); sizer->Add (_panel, 1, wxEXPAND); SetSizer (sizer); - - _table = new wxFlexGridSizer (4, 6, 6); - _table->AddGrowableCol (1, 1); + + _table = new wxFlexGridSizer (2, 6, 6); + _table->AddGrowableCol (0, 1); _panel->SetSizer (_table); SetScrollRate (0, 32); + EnableScrolling (false, true); - Connect (wxID_ANY, wxEVT_TIMER, wxTimerEventHandler (JobManagerView::periodic), 0, this); + Bind (wxEVT_TIMER, boost::bind (&JobManagerView::periodic, this)); _timer.reset (new wxTimer (this)); _timer->Start (1000); - update (); + JobManager::instance()->JobAdded.connect (bind (&JobManagerView::job_added, this, _1)); + JobManager::instance()->JobsReordered.connect (bind (&JobManagerView::replace, this)); } void -JobManagerView::periodic (wxTimerEvent &) +JobManagerView::job_added (weak_ptr j) { - update (); + shared_ptr job = j.lock (); + if (job) { + shared_ptr v; + if (_batch) { + v.reset (new BatchJobView (job, this, _panel, _table)); + } else { + v.reset (new NormalJobView (job, this, _panel, _table)); + } + v->setup (); + _job_records.push_back (v); + } + + FitInside(); + job_list_changed (); } -/** Update the view by examining the state of each job. - * Must be called in the GUI thread. - */ void -JobManagerView::update () +JobManagerView::periodic () { - list > jobs = JobManager::instance()->get (); - - int index = 0; - - for (list >::iterator i = jobs.begin(); i != jobs.end(); ++i) { - - if (_job_records.find (*i) == _job_records.end ()) { - wxStaticText* m = new wxStaticText (_panel, wxID_ANY, std_to_wx ((*i)->name ())); - _table->Insert (index, m, 0, wxALIGN_CENTER_VERTICAL | wxALL, 6); - - JobRecord r; - r.finalised = false; - r.gauge = new wxGauge (_panel, wxID_ANY, 100); - _table->Insert (index + 1, r.gauge, 1, wxEXPAND | wxLEFT | wxRIGHT); - - r.message = new wxStaticText (_panel, wxID_ANY, std_to_wx ("")); - _table->Insert (index + 2, r.message, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6); - - r.details = new wxButton (_panel, wxID_ANY, _("Details...")); - r.details->Connect (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (JobManagerView::details_clicked), 0, this); - r.details->Enable (false); - _table->Insert (index + 3, r.details, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6); - - _job_records[*i] = r; - } + for (list >::iterator i = _job_records.begin(); i != _job_records.end(); ++i) { + (*i)->maybe_pulse (); + } +} - string const st = (*i)->status (); +void +JobManagerView::replace () +{ + /* Make a new version of _job_records which reflects the order in JobManager's job list */ - if (!(*i)->finished ()) { - float const p = (*i)->overall_progress (); - if (p >= 0) { - checked_set (_job_records[*i].message, st); - _job_records[*i].gauge->SetValue (p * 100); - } else { - checked_set (_job_records[*i].message, wx_to_std (_("Running"))); - _job_records[*i].gauge->Pulse (); - } - } - - if ((*i)->finished() && !_job_records[*i].finalised) { - _job_records[*i].gauge->SetValue (100); - checked_set (_job_records[*i].message, st); - (*i)->Finished (); - _job_records[*i].finalised = true; - if (!(*i)->error_details().empty ()) { - _job_records[*i].details->Enable (true); + list > new_job_records; + + BOOST_FOREACH (shared_ptr i, JobManager::instance()->get()) { + /* Find this job's JobView */ + BOOST_FOREACH (shared_ptr j, _job_records) { + if (j->job() == i) { + new_job_records.push_back (j); + break; } } + } + + BOOST_FOREACH (shared_ptr i, _job_records) { + i->detach (); + } - index += 4; + _job_records = new_job_records; + + BOOST_FOREACH (shared_ptr i, _job_records) { + i->insert (i->insert_position ()); } - _table->Layout (); - FitInside (); + job_list_changed (); } void -JobManagerView::details_clicked (wxCommandEvent& ev) +JobManagerView::job_list_changed () { - wxObject* o = ev.GetEventObject (); - - for (map, JobRecord>::iterator i = _job_records.begin(); i != _job_records.end(); ++i) { - if (i->second.details == o) { - string s = i->first->error_summary(); - s[0] = toupper (s[0]); - error_dialog (this, std_to_wx (String::compose ("%1.\n\n%2", s, i->first->error_details()))); - } + BOOST_FOREACH (shared_ptr i, _job_records) { + i->job_list_changed (); } }