X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Froute_group_dialog.cc;h=9f50d7a3f653fc031d4cb34d5b9f638e301b1949;hb=965a9740835a77cc2bf9e6cea237e56ae41150d8;hp=176c23d8e2ae468a069b76e2f278715002efdeaf;hpb=1b9bf21aa40eee32eb4c7ab49d1ad8ab51d09b11;p=ardour.git

diff --git a/gtk2_ardour/route_group_dialog.cc b/gtk2_ardour/route_group_dialog.cc
index 176c23d8e2..9f50d7a3f6 100644
--- a/gtk2_ardour/route_group_dialog.cc
+++ b/gtk2_ardour/route_group_dialog.cc
@@ -19,7 +19,9 @@
 
 #include <gtkmm/table.h>
 #include <gtkmm/stock.h>
+#include <gtkmm/messagedialog.h>
 #include "ardour/route_group.h"
+#include "ardour/session.h"
 #include "route_group_dialog.h"
 #include "i18n.h"
 #include <iostream>
@@ -29,17 +31,19 @@ using namespace ARDOUR;
 using namespace std;
 using namespace PBD;
 
-RouteGroupDialog::RouteGroupDialog (RouteGroup* g, StockID const & s)
-	: ArdourDialog (_("Route Group")),
-	  _group (g),
-	  _active (_("Active")),
-	  _gain (_("Gain")),
-	  _relative (_("Relative")),
-	  _mute (_("Muting")),
-	  _solo (_("Soloing")),
-	  _rec_enable (_("Record enable")),
-	  _select (_("Selection")),
-	  _edit (_("Editing"))
+RouteGroupDialog::RouteGroupDialog (RouteGroup* g, bool creating_new)
+	: ArdourDialog (_("Route Group"))
+	, _group (g)
+	, _initial_name (g->name ())
+	, _active (_("Active"))
+	, _gain (_("Gain"))
+	, _relative (_("Relative"))
+	, _mute (_("Muting"))
+	, _solo (_("Soloing"))
+	, _rec_enable (_("Record enable"))
+	, _select (_("Selection"))
+	, _edit (_("Editing"))
+	, _route_active (_("Route active state"))
 {
 	set_modal (true);
 	set_skip_taskbar_hint (true);
@@ -83,6 +87,18 @@ RouteGroupDialog::RouteGroupDialog (RouteGroup* g, StockID const & s)
 	_rec_enable.set_active (_group->is_recenable());
 	_select.set_active (_group->is_select());
 	_edit.set_active (_group->is_edit());
+	_route_active.set_active (_group->is_route_active());
+
+	_name.signal_changed().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+	_active.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+	_gain.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_relative.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_mute.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_solo.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_rec_enable.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_select.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_edit.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
+ 	_route_active.signal_toggled().connect (sigc::mem_fun (*this, &RouteGroupDialog::update));
 
 	gain_toggled ();
 
@@ -106,6 +122,7 @@ RouteGroupDialog::RouteGroupDialog (RouteGroup* g, StockID const & s)
 	table->attach (_rec_enable, 1, 3, 5, 6, Gtk::FILL, Gtk::FILL, 0, 0);
 	table->attach (_select, 1, 3, 6, 7, Gtk::FILL, Gtk::FILL, 0, 0);
 	table->attach (_edit, 1, 3, 7, 8, Gtk::FILL, Gtk::FILL, 0, 0);
+	table->attach (_route_active, 1, 3, 8, 9, Gtk::FILL, Gtk::FILL, 0, 0);
 
 	options_box->pack_start (*table, false, true);
 	vbox->pack_start (*options_box, false, true);
@@ -114,36 +131,67 @@ RouteGroupDialog::RouteGroupDialog (RouteGroup* g, StockID const & s)
 
 	_gain.signal_toggled().connect(sigc::mem_fun (*this, &RouteGroupDialog::gain_toggled));
 
-	add_button (Stock::CANCEL, RESPONSE_CANCEL);
-	add_button (s, RESPONSE_OK);
-	set_default_response (RESPONSE_OK);
-
+	if (creating_new) {
+		add_button (Stock::CANCEL, RESPONSE_CANCEL);
+		add_button (Stock::NEW, RESPONSE_OK);
+		set_default_response (RESPONSE_OK);
+	} else {
+		add_button (Stock::CLOSE, RESPONSE_CLOSE);
+		set_default_response (RESPONSE_CLOSE);
+	}
+	
 	show_all_children ();
 }
 
-int
+/** @return true if the route group edit was cancelled, otherwise false */
+bool
 RouteGroupDialog::do_run ()
 {
-	int const r = run ();
-
-	if (r == Gtk::RESPONSE_OK || r == Gtk::RESPONSE_ACCEPT) {
-
-		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::relative, _relative.get_active());
-		plist.add (Properties::active, _active.get_active());
-		plist.add (Properties::name, string (_name.get_text()));
-
-		_group->set_properties (plist);
+	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 ();
 	}
 
-	return r;
+	/* 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()));
+
+	_group->apply_changes (plist);
 }
 
 void
@@ -152,3 +200,15 @@ 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;
+	}
+
+	return (i == route_groups.end ());
+}