Prevent duplicate screen names within a cinema (#1007).

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

diff --git a/src/wx/screens_panel.cc b/src/wx/screens_panel.cc
index 042fcc5ab58184b9f39041c800b03b4daa81f275..3fabba46a567221f312e658c01350ca934e8df80 100644 (file)
--- a/src/wx/screens_panel.cc
+++ b/src/wx/screens_panel.cc
@@ -147,7 +147,7 @@ ScreensPanel::add_screen (shared_ptr<Cinema> c, shared_ptr<Screen> s)
 void
 ScreensPanel::add_cinema_clicked ()
 {
-       CinemaDialog* d = new CinemaDialog (this, _("Add Cinema"));
+       CinemaDialog* d = new CinemaDialog (GetParent(), _("Add Cinema"));
        if (d->ShowModal () == wxID_OK) {
                shared_ptr<Cinema> c (new Cinema (d->name(), d->emails(), d->notes(), d->utc_offset_hour(), d->utc_offset_minute()));
                Config::instance()->add_cinema (c);
@@ -167,7 +167,7 @@ ScreensPanel::edit_cinema_clicked ()
        pair<wxTreeItemId, shared_ptr<Cinema> > c = *_selected_cinemas.begin();
 
        CinemaDialog* d = new CinemaDialog (
-               this, _("Edit cinema"), c.second->name, c.second->emails, c.second->notes, c.second->utc_offset_hour(), c.second->utc_offset_minute()
+               GetParent(), _("Edit cinema"), c.second->name, c.second->emails, c.second->notes, c.second->utc_offset_hour(), c.second->utc_offset_minute()
                );
 
        if (d->ShowModal () == wxID_OK) {
@@ -203,11 +203,25 @@ ScreensPanel::add_screen_clicked ()
 
        shared_ptr<Cinema> c = _selected_cinemas.begin()->second;
 
-       ScreenDialog* d = new ScreenDialog (this, _("Add Screen"));
+       ScreenDialog* d = new ScreenDialog (GetParent(), _("Add Screen"));
        if (d->ShowModal () != wxID_OK) {
+               d->Destroy ();
                return;
        }
 
+       BOOST_FOREACH (shared_ptr<Screen> i, c->screens ()) {
+               if (i->name == d->name()) {
+                       error_dialog (
+                               GetParent(),
+                               wxString::Format (
+                                       _("You cannot add a screen called '%s' as the cinema already has a screen with this name."),
+                                       std_to_wx(d->name()).data()
+                                       )
+                               );
+                       return;
+               }
+       }
+
        shared_ptr<Screen> s (new Screen (d->name(), d->recipient(), d->trusted_devices()));
        c->add_screen (s);
        optional<wxTreeItemId> id = add_screen (c, s);
@@ -229,16 +243,33 @@ ScreensPanel::edit_screen_clicked ()
 
        pair<wxTreeItemId, shared_ptr<Screen> > s = *_selected_screens.begin();
 
-       ScreenDialog* d = new ScreenDialog (this, _("Edit screen"), s.second->name, s.second->notes, s.second->recipient, s.second->trusted_devices);
-       if (d->ShowModal () == wxID_OK) {
-               s.second->name = d->name ();
-               s.second->notes = d->notes ();
-               s.second->recipient = d->recipient ();
-               s.second->trusted_devices = d->trusted_devices ();
-               _targets->SetItemText (s.first, std_to_wx (d->name()));
-               Config::instance()->changed ();
+       ScreenDialog* d = new ScreenDialog (GetParent(), _("Edit screen"), s.second->name, s.second->notes, s.second->recipient, s.second->trusted_devices);
+       if (d->ShowModal () != wxID_OK) {
+               d->Destroy ();
+               return;
        }
 
+       shared_ptr<Cinema> c = s.second->cinema;
+       BOOST_FOREACH (shared_ptr<Screen> i, c->screens ()) {
+               if (i != s.second && i->name == d->name()) {
+                       error_dialog (
+                               GetParent(),
+                               wxString::Format (
+                                       _("You cannot change this screen's name to '%s' as the cinema already has a screen with this name."),
+                                       std_to_wx(d->name()).data()
+                                       )
+                               );
+                       return;
+               }
+       }
+
+       s.second->name = d->name ();
+       s.second->notes = d->notes ();
+       s.second->recipient = d->recipient ();
+       s.second->trusted_devices = d->trusted_devices ();
+       _targets->SetItemText (s.first, std_to_wx (d->name()));
+       Config::instance()->changed ();
+
        d->Destroy ();
 }