network-manager-applet r1247 - in trunk: . src/connection-editor

[dcbw svn gnome org]

Author: dcbw
Date: Sat Apr  4 15:25:54 2009
New Revision: 1247
URL: http://svn.gnome.org/viewvc/network-manager-applet?rev=1247&view=rev

Log:
2009-04-04  Dan Williams  <dcbw redhat com>

	* src/connection-editor/page-wired-security.c
		- (ce_page_wired_security_new): don't connect to the "use 802.1x"
			checkbox's 'toggled' signal here, do it after setting the
			checkbox's initial value in finish_setup().  Fixes issue
			where validation would be triggered prematurely, before the
			editor was initialized
		- (finish_setup): enable_toggled() already calls ce_page_changed()



Modified:
   trunk/ChangeLog
   trunk/src/connection-editor/page-wired-security.c

Modified: trunk/src/connection-editor/page-wired-security.c
==============================================================================
--- trunk/src/connection-editor/page-wired-security.c	(original)
+++ trunk/src/connection-editor/page-wired-security.c	Sat Apr  4 15:25:54 2009
@@ -87,7 +87,7 @@
 	priv->security_widget = wireless_security_get_widget (priv->security);
 
 	gtk_toggle_button_set_active (priv->enabled, priv->initial_have_8021x);
-	g_signal_connect_swapped (priv->enabled, "toggled", G_CALLBACK (ce_page_changed), self);
+	g_signal_connect (priv->enabled, "toggled", G_CALLBACK (enable_toggled), self);
 	gtk_widget_set_sensitive (priv->security_widget, priv->initial_have_8021x);
 
 	gtk_box_pack_start (GTK_BOX (parent->page), GTK_WIDGET (priv->enabled), FALSE, TRUE, 12);
@@ -118,8 +118,6 @@
 		priv->initial_have_8021x = TRUE;
 
 	priv->enabled = GTK_TOGGLE_BUTTON (gtk_check_button_new_with_label (_("Use 802.1X security for this connection")));
-	g_signal_connect (priv->enabled, "toggled",
-					  G_CALLBACK (enable_toggled), self);
 
 	g_signal_connect (self, "initialized", G_CALLBACK (finish_setup), NULL);
 	if (!ce_page_initialize (parent, NM_SETTING_802_1X_SETTING_NAME, error)) {