Fix sensitivity of custom scaled edit button.
[dcpomatic.git] / src / wx / screens_panel.cc
index 6bc821feb42a829b74dd741bb8e8062a560eb0ae..c034903129a0bf48217532459926efb213782e7c 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2015-2021 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2022 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
 */
 
 
-#include "screens_panel.h"
-#include "wx_util.h"
 #include "cinema_dialog.h"
-#include "screen_dialog.h"
 #include "dcpomatic_button.h"
-#include "lib/config.h"
+#include "screen_dialog.h"
+#include "screens_panel.h"
+#include "wx_util.h"
 #include "lib/cinema.h"
+#include "lib/config.h"
 #include "lib/screen.h"
 
 
-using std::list;
-using std::pair;
 using std::cout;
-using std::map;
-using std::string;
 using std::make_pair;
 using std::make_shared;
+using std::map;
+using std::pair;
 using std::shared_ptr;
+using std::string;
+using std::vector;
 using boost::optional;
 using namespace dcpomatic;
 
@@ -45,15 +45,9 @@ ScreensPanel::ScreensPanel (wxWindow* parent)
        : wxPanel (parent, wxID_ANY)
        , _ignore_selection_change (false)
 {
-       wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
+       auto sizer = new wxBoxSizer (wxVERTICAL);
 
-#ifdef __WXGTK3__
-       int const height = 30;
-#else
-       int const height = -1;
-#endif
-
-       _search = new wxSearchCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(200, height));
+       _search = new wxSearchCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(200, search_ctrl_height()));
 #ifndef __WXGTK3__
        /* The cancel button seems to be strangely broken in GTK3; clicking on it twice sometimes works */
        _search->ShowCancelButton (true);
@@ -61,7 +55,8 @@ ScreensPanel::ScreensPanel (wxWindow* parent)
        sizer->Add (_search, 0, wxBOTTOM, DCPOMATIC_SIZER_GAP);
 
        auto targets = new wxBoxSizer (wxHORIZONTAL);
-       _targets = new wxTreeCtrl (this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxTR_HIDE_ROOT | wxTR_MULTIPLE | wxTR_HAS_BUTTONS | wxTR_LINES_AT_ROOT);
+       _targets = new TreeListCtrl (this);
+       _targets->AppendColumn (wxT("foo"));
        targets->Add (_targets, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
 
        add_cinemas ();
@@ -86,7 +81,8 @@ ScreensPanel::ScreensPanel (wxWindow* parent)
        sizer->Add (targets, 1, wxEXPAND);
 
        _search->Bind        (wxEVT_TEXT, boost::bind (&ScreensPanel::search_changed, this));
-       _targets->Bind       (wxEVT_TREE_SEL_CHANGED, &ScreensPanel::selection_changed_shim, this);
+       _targets->Bind       (wxEVT_TREELIST_SELECTION_CHANGED, &ScreensPanel::selection_changed_shim, this);
+       _targets->Bind       (wxEVT_TREELIST_ITEM_CHECKED, &ScreensPanel::checkbox_changed, this);
 
        _add_cinema->Bind    (wxEVT_BUTTON, boost::bind (&ScreensPanel::add_cinema_clicked, this));
        _edit_cinema->Bind   (wxEVT_BUTTON, boost::bind (&ScreensPanel::edit_cinema_clicked, this));
@@ -102,7 +98,8 @@ ScreensPanel::ScreensPanel (wxWindow* parent)
 
 ScreensPanel::~ScreensPanel ()
 {
-       _targets->Unbind (wxEVT_TREE_SEL_CHANGED, &ScreensPanel::selection_changed_shim, this);
+       _targets->Unbind (wxEVT_TREELIST_SELECTION_CHANGED, &ScreensPanel::selection_changed_shim, this);
+       _targets->Unbind (wxEVT_TREELIST_ITEM_CHECKED, &ScreensPanel::checkbox_changed, this);
 }
 
 
@@ -112,16 +109,16 @@ ScreensPanel::setup_sensitivity ()
        bool const sc = _selected_cinemas.size() == 1;
        bool const ss = _selected_screens.size() == 1;
 
-       _edit_cinema->Enable (sc);
+       _edit_cinema->Enable (sc || ss);
        _remove_cinema->Enable (_selected_cinemas.size() >= 1);
 
-       _add_screen->Enable (sc);
+       _add_screen->Enable (sc || ss);
        _edit_screen->Enable (ss);
        _remove_screen->Enable (_selected_screens.size() >= 1);
 }
 
 
-optional<wxTreeItemId>
+optional<wxTreeListItem>
 ScreensPanel::add_cinema (shared_ptr<Cinema> c)
 {
        auto search = wx_to_std (_search->GetValue ());
@@ -131,11 +128,11 @@ ScreensPanel::add_cinema (shared_ptr<Cinema> c)
                auto name = c->name;
                transform (name.begin(), name.end(), name.begin(), ::tolower);
                if (name.find (search) == string::npos) {
-                       return optional<wxTreeItemId>();
+                       return {};
                }
        }
 
-       auto id = _targets->AppendItem(_root, std_to_wx(c->name));
+       auto id = _targets->AppendItem(_targets->GetRootItem(), std_to_wx(c->name));
 
        _cinemas[id] = c;
 
@@ -143,13 +140,13 @@ ScreensPanel::add_cinema (shared_ptr<Cinema> c)
                add_screen (c, i);
        }
 
-       _targets->SortChildren (_root);
+       _targets->SetSortColumn (0);
 
        return id;
 }
 
 
-optional<wxTreeItemId>
+optional<wxTreeListItem>
 ScreensPanel::add_screen (shared_ptr<Cinema> c, shared_ptr<Screen> s)
 {
        auto i = _cinemas.begin();
@@ -175,8 +172,8 @@ ScreensPanel::add_cinema_clicked ()
                Config::instance()->add_cinema (c);
                auto id = add_cinema (c);
                if (id) {
-                       _targets->Unselect ();
-                       _targets->SelectItem (*id);
+                       _targets->UnselectAll ();
+                       _targets->Select (*id);
                }
        }
 
@@ -184,26 +181,38 @@ ScreensPanel::add_cinema_clicked ()
 }
 
 
+optional<pair<wxTreeListItem, shared_ptr<Cinema>>>
+ScreensPanel::cinema_for_operation () const
+{
+       if (_selected_cinemas.size() == 1) {
+               return make_pair(_selected_cinemas.begin()->first, _selected_cinemas.begin()->second);
+       } else if (_selected_screens.size() == 1) {
+               return make_pair(_targets->GetItemParent(_selected_screens.begin()->first), _selected_screens.begin()->second->cinema);
+       }
+
+       return {};
+}
+
+
 void
 ScreensPanel::edit_cinema_clicked ()
 {
-       if (_selected_cinemas.size() != 1) {
+       auto cinema = cinema_for_operation ();
+       if (!cinema) {
                return;
        }
 
-       auto c = *_selected_cinemas.begin();
-
        auto d = new CinemaDialog (
-               GetParent(), _("Edit cinema"), c.second->name, c.second->emails, c.second->notes, c.second->utc_offset_hour(), c.second->utc_offset_minute()
+               GetParent(), _("Edit cinema"), cinema->second->name, cinema->second->emails, cinema->second->notes, cinema->second->utc_offset_hour(), cinema->second->utc_offset_minute()
                );
 
        if (d->ShowModal() == wxID_OK) {
-               c.second->name = d->name ();
-               c.second->emails = d->emails ();
-               c.second->notes = d->notes ();
-               c.second->set_utc_offset_hour (d->utc_offset_hour ());
-               c.second->set_utc_offset_minute (d->utc_offset_minute ());
-               _targets->SetItemText (c.first, std_to_wx (d->name()));
+               cinema->second->name = d->name ();
+               cinema->second->emails = d->emails ();
+               cinema->second->notes = d->notes ();
+               cinema->second->set_utc_offset_hour (d->utc_offset_hour ());
+               cinema->second->set_utc_offset_minute (d->utc_offset_minute ());
+               _targets->SetItemText (cinema->first, std_to_wx(d->name()));
                Config::instance()->changed (Config::CINEMAS);
        }
 
@@ -226,7 +235,7 @@ ScreensPanel::remove_cinema_clicked ()
 
        for (auto const& i: _selected_cinemas) {
                Config::instance()->remove_cinema (i.second);
-               _targets->Delete (i.first);
+               _targets->DeleteItem (i.first);
        }
 
        selection_changed ();
@@ -236,19 +245,18 @@ ScreensPanel::remove_cinema_clicked ()
 void
 ScreensPanel::add_screen_clicked ()
 {
-       if (_selected_cinemas.size() != 1) {
+       auto cinema = cinema_for_operation ();
+       if (!cinema) {
                return;
        }
 
-       auto c = _selected_cinemas.begin()->second;
-
        auto d = new ScreenDialog (GetParent(), _("Add Screen"));
        if (d->ShowModal () != wxID_OK) {
                d->Destroy ();
                return;
        }
 
-       for (auto i: c->screens ()) {
+       for (auto i: cinema->second->screens()) {
                if (i->name == d->name()) {
                        error_dialog (
                                GetParent(),
@@ -261,9 +269,9 @@ ScreensPanel::add_screen_clicked ()
                }
        }
 
-       auto s = std::make_shared<Screen>(d->name(), d->notes(), d->recipient(), d->trusted_devices());
-       c->add_screen (s);
-       auto id = add_screen (c, s);
+       auto screen = std::make_shared<Screen>(d->name(), d->notes(), d->recipient(), d->recipient_file(), d->trusted_devices());
+       cinema->second->add_screen (screen);
+       auto id = add_screen (cinema->second, screen);
        if (id) {
                _targets->Expand (id.get ());
        }
@@ -283,7 +291,7 @@ ScreensPanel::edit_screen_clicked ()
 
        auto s = *_selected_screens.begin();
 
-       auto d = new ScreenDialog (GetParent(), _("Edit screen"), s.second->name, s.second->notes, s.second->recipient, s.second->trusted_devices);
+       auto d = new ScreenDialog (GetParent(), _("Edit screen"), s.second->name, s.second->notes, s.second->recipient, s.second->recipient_file, s.second->trusted_devices);
        if (d->ShowModal() != wxID_OK) {
                d->Destroy ();
                return;
@@ -306,6 +314,7 @@ ScreensPanel::edit_screen_clicked ()
        s.second->name = d->name ();
        s.second->notes = d->notes ();
        s.second->recipient = d->recipient ();
+       s.second->recipient_file = d->recipient_file ();
        s.second->trusted_devices = d->trusted_devices ();
        _targets->SetItemText (s.first, std_to_wx (d->name()));
        Config::instance()->changed (Config::CINEMAS);
@@ -343,37 +352,33 @@ ScreensPanel::remove_screen_clicked ()
                }
 
                j->second->remove_screen (i.second);
-               _targets->Delete (i.first);
+               _targets->DeleteItem (i.first);
        }
 
        Config::instance()->changed (Config::CINEMAS);
 }
 
 
-list<shared_ptr<Screen>>
+vector<shared_ptr<Screen>>
 ScreensPanel::screens () const
 {
-       list<shared_ptr<Screen>> s;
+       vector<shared_ptr<Screen>> output;
 
-       for (auto const& i: _selected_cinemas) {
-               for (auto j: i.second->screens()) {
-                       s.push_back (j);
+       for (auto item = _targets->GetFirstItem(); item.IsOk(); item = _targets->GetNextItem(item)) {
+               if (_targets->GetCheckedState(item) == wxCHK_CHECKED) {
+                       auto screen_iter = _screens.find(item);
+                       if (screen_iter != _screens.end()) {
+                               output.push_back (screen_iter->second);
+                       }
                }
        }
 
-       for (auto const& i: _selected_screens) {
-               s.push_back (i.second);
-       }
-
-       s.sort ();
-       s.unique ();
-
-       return s;
+       return output;
 }
 
 
 void
-ScreensPanel::selection_changed_shim (wxTreeEvent &)
+ScreensPanel::selection_changed_shim (wxTreeListEvent &)
 {
        selection_changed ();
 }
@@ -386,13 +391,13 @@ ScreensPanel::selection_changed ()
                return;
        }
 
-       wxArrayTreeItemIds s;
+       wxTreeListItems s;
        _targets->GetSelections (s);
 
        _selected_cinemas.clear ();
        _selected_screens.clear ();
 
-       for (size_t i = 0; i < s.GetCount(); ++i) {
+       for (size_t i = 0; i < s.size(); ++i) {
                auto j = _cinemas.find (s[i]);
                if (j != _cinemas.end ()) {
                        _selected_cinemas[j->first] = j->second;
@@ -404,15 +409,12 @@ ScreensPanel::selection_changed ()
        }
 
        setup_sensitivity ();
-       ScreensChanged ();
 }
 
 
 void
 ScreensPanel::add_cinemas ()
 {
-       _root = _targets->AddRoot ("Foo");
-
        for (auto i: Config::instance()->cinemas()) {
                add_cinema (i);
        }
@@ -431,14 +433,14 @@ ScreensPanel::search_changed ()
        _ignore_selection_change = true;
 
        for (auto const& i: _selected_cinemas) {
-               /* The wxTreeItemIds will now be different, so we must search by cinema */
+               /* The wxTreeListItems will now be different, so we must search by cinema */
                auto j = _cinemas.begin ();
                while (j != _cinemas.end() && j->second != i.second) {
                        ++j;
                }
 
                if (j != _cinemas.end()) {
-                       _targets->SelectItem (j->first);
+                       _targets->Select (j->first);
                }
        }
 
@@ -449,9 +451,57 @@ ScreensPanel::search_changed ()
                }
 
                if (j != _screens.end()) {
-                       _targets->SelectItem (j->first);
+                       _targets->Select (j->first);
                }
        }
 
        _ignore_selection_change = false;
 }
+
+
+void
+ScreensPanel::checkbox_changed (wxTreeListEvent& ev)
+{
+       if (_cinemas.find(ev.GetItem()) != _cinemas.end()) {
+               /* Cinema: check/uncheck all children */
+               auto const checked = _targets->GetCheckedState(ev.GetItem());
+               for (auto child = _targets->GetFirstChild(ev.GetItem()); child.IsOk(); child = _targets->GetNextSibling(child)) {
+                       _targets->CheckItem(child, checked);
+               }
+       } else {
+               /* Screen: set cinema to checked/unchecked/3state */
+               auto parent = _targets->GetItemParent(ev.GetItem());
+               DCPOMATIC_ASSERT (parent.IsOk());
+               int checked = 0;
+               int unchecked = 0;
+               for (auto child = _targets->GetFirstChild(parent); child.IsOk(); child = _targets->GetNextSibling(child)) {
+                       if (_targets->GetCheckedState(child) == wxCHK_CHECKED) {
+                           ++checked;
+                       } else {
+                           ++unchecked;
+                       }
+               }
+               if (checked == 0) {
+                       _targets->CheckItem(parent, wxCHK_UNCHECKED);
+               } else if (unchecked == 0) {
+                       _targets->CheckItem(parent, wxCHK_CHECKED);
+               } else {
+                       _targets->CheckItem(parent, wxCHK_UNDETERMINED);
+               }
+       }
+
+       ScreensChanged ();
+}
+
+
+
+wxIMPLEMENT_DYNAMIC_CLASS (TreeListCtrl, wxTreeListCtrl);
+
+
+int
+TreeListCtrl::OnCompareItems (wxTreeListItem const& a, wxTreeListItem const& b)
+{
+       return strcoll (wx_to_std(GetItemText(a)).c_str(), wx_to_std(GetItemText(b)).c_str());
+}
+
+