Cleanup: use new CheckBox::bind().

[dcpomatic.git] / src / wx / smpte_metadata_dialog.cc

diff --git a/src/wx/smpte_metadata_dialog.cc b/src/wx/smpte_metadata_dialog.cc
index 54b6da0d4651de53968774b6542c04ecd7c2ae73..f5d63e329be924085f39c96043e0771993b600a1 100644 (file)
--- a/src/wx/smpte_metadata_dialog.cc
+++ b/src/wx/smpte_metadata_dialog.cc
@@ -19,6 +19,7 @@
 */
 
 
+#include "check_box.h"
 #include "content_version_dialog.h"
 #include "editable_list.h"
 #include "language_tag_dialog.h"
@@ -44,17 +45,6 @@ using namespace boost::placeholders;
 #endif
 
 
-static string
-ratings_column (dcp::Rating r, int c)
-{
-       if (c == 0) {
-               return r.agency;
-       }
-
-       return r.label;
-}
-
-
 static string
 content_versions_column (string v, int)
 {
@@ -84,18 +74,6 @@ SMPTEMetadataDialog::setup_standard (wxPanel* panel, wxSizer* sizer)
                sizer->Add (m, 0, flags, DCPOMATIC_SIZER_GAP);
        }
 
-       vector<EditableListColumn> columns;
-       columns.push_back (EditableListColumn("Agency", 200, true));
-       columns.push_back (EditableListColumn("Label", 50, true));
-       _ratings = new EditableList<dcp::Rating, RatingDialog> (
-               panel,
-               columns,
-               boost::bind(&SMPTEMetadataDialog::ratings, this),
-               boost::bind(&SMPTEMetadataDialog::set_ratings, this, _1),
-               boost::bind(&ratings_column, _1, _2),
-               true,
-               false
-               );
        sizer->Add (_ratings, 1, wxEXPAND);
 }
 
@@ -113,7 +91,7 @@ SMPTEMetadataDialog::setup_advanced (wxPanel* panel, wxSizer* sizer)
        _status = new wxChoice (panel, wxID_ANY);
        sizer->Add (_status, 0);
 
-       _enable_distributor = new wxCheckBox (panel, wxID_ANY, _("Distributor"));
+       _enable_distributor = new CheckBox(panel, _("Distributor"));
        sizer->Add (_enable_distributor, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL);
        _distributor = new wxTextCtrl (panel, wxID_ANY);
        sizer->Add (_distributor, 1, wxEXPAND);
@@ -135,8 +113,8 @@ SMPTEMetadataDialog::setup_advanced (wxPanel* panel, wxSizer* sizer)
                boost::bind(&SMPTEMetadataDialog::content_versions, this),
                boost::bind(&SMPTEMetadataDialog::set_content_versions, this, _1),
                boost::bind(&content_versions_column, _1, _2),
-               true,
-               false
+               EditableListTitle::INVISIBLE,
+               EditableListButton::NEW | EditableListButton::REMOVE
                );
        sizer->Add (_content_versions, 1, wxEXPAND);
 }
@@ -161,7 +139,7 @@ SMPTEMetadataDialog::setup ()
        _name_language->Changed.connect (boost::bind(&SMPTEMetadataDialog::name_language_changed, this, _1));
        _version_number->Bind (wxEVT_SPINCTRL, boost::bind(&SMPTEMetadataDialog::version_number_changed, this));
        _status->Bind (wxEVT_CHOICE, boost::bind(&SMPTEMetadataDialog::status_changed, this));
-       _enable_distributor->Bind (wxEVT_CHECKBOX, boost::bind(&SMPTEMetadataDialog::enable_distributor_changed, this));
+       _enable_distributor->bind(&SMPTEMetadataDialog::enable_distributor_changed, this);
        _distributor->Bind (wxEVT_TEXT, boost::bind(&SMPTEMetadataDialog::distributor_changed, this));
 
        film_changed (ChangeType::DONE, Film::Property::NAME_LANGUAGE);
@@ -208,20 +186,6 @@ SMPTEMetadataDialog::film_changed (ChangeType type, Film::Property property)
 }
 
 
-vector<dcp::Rating>
-SMPTEMetadataDialog::ratings () const
-{
-       return film()->ratings ();
-}
-
-
-void
-SMPTEMetadataDialog::set_ratings (vector<dcp::Rating> r)
-{
-       film()->set_ratings (r);
-}
-
-
 vector<string>
 SMPTEMetadataDialog::content_versions () const
 {