- int const r = run ();
-
- if (r == Gtk::RESPONSE_OK) {
- _group->set_name (_name.get_text ());
- _group->set_active (_active.get_active (), this);
-
- _group->set_property (RouteGroup::Gain, _gain.get_active ());
- _group->set_property (RouteGroup::Mute, _mute.get_active ());
- _group->set_property (RouteGroup::Solo, _solo.get_active ());
- _group->set_property (RouteGroup::RecEnable, _rec_enable.get_active ());
- _group->set_property (RouteGroup::Select, _select.get_active ());
- _group->set_property (RouteGroup::Edit, _edit.get_active ());
- _group->set_relative (_relative.get_active(), this);
+ while (1) {
+ int const r = run ();
+ if (r == Gtk::RESPONSE_CANCEL) {
+ /* cancel, so just bail now */
+ return Gtk::RESPONSE_CANCEL;
+ }
+
+ if (unique_name ()) {
+ /* not cancelled and the name is ok, so all is well */
+ return false;
+ }
+
+ _group->set_name (_initial_name);
+ MessageDialog msg (
+ _("A route group of this name already exists. Please use a different name."),
+ false,
+ Gtk::MESSAGE_ERROR,
+ Gtk::BUTTONS_OK,
+ true
+ );
+
+ msg.run ();
+ }
+
+ /* NOTREACHED */
+ return false;
+}
+
+void
+RouteGroupDialog::update ()
+{
+ PropertyList plist;
+
+ plist.add (Properties::gain, _gain.get_active());
+ plist.add (Properties::recenable, _rec_enable.get_active());
+ plist.add (Properties::mute, _mute.get_active());
+ plist.add (Properties::solo, _solo.get_active ());
+ plist.add (Properties::select, _select.get_active());
+ plist.add (Properties::edit, _edit.get_active());
+ plist.add (Properties::route_active, _route_active.get_active());
+ plist.add (Properties::relative, _relative.get_active());
+ plist.add (Properties::active, _active.get_active());
+ plist.add (Properties::name, string (_name.get_text()));
+ plist.add (Properties::color, _share_color.get_active());
+ plist.add (Properties::monitoring, _share_monitoring.get_active());
+
+ _group->apply_changes (plist);
+
+ GroupTabs::set_group_color (_group, _color.get_color ());
+}
+
+void
+RouteGroupDialog::gain_toggled ()
+{
+ _relative.set_sensitive (_gain.get_active ());
+}
+
+/** @return true if the current group's name is unique accross the session */
+bool
+RouteGroupDialog::unique_name () const
+{
+ list<RouteGroup*> route_groups = _group->session().route_groups ();
+ list<RouteGroup*>::iterator i = route_groups.begin ();
+ while (i != route_groups.end() && ((*i)->name() != _name.get_text() || *i == _group)) {
+ ++i;