evolution r37333 - in trunk/smime: . gui

[mcrha svn gnome org]

Author: mcrha
Date: Wed Feb 25 11:28:08 2009
New Revision: 37333
URL: http://svn.gnome.org/viewvc/evolution?rev=37333&view=rev

Log:
2009-02-25  Milan Crha  <mcrha redhat com>

	** Fix for bug #572975

	* gui/smime-ui.glade: Center dialogs.
	* gui/certificate-manager.c: (import_your), (import_contact),
	(import_ca): Destroy file chooser dialog early enough to not have
	hidden any other windows below it.



Modified:
   trunk/smime/ChangeLog
   trunk/smime/gui/certificate-manager.c
   trunk/smime/gui/smime-ui.glade

Modified: trunk/smime/gui/certificate-manager.c
==============================================================================
--- trunk/smime/gui/certificate-manager.c	(original)
+++ trunk/smime/gui/certificate-manager.c	Wed Feb 25 11:28:08 2009
@@ -130,7 +130,6 @@
 import_your (GtkWidget *widget, CertificateManagerData *cfm)
 {
 	GtkWidget *filesel;
-	const char *filename;
 
 	GtkFileFilter* filter;
 
@@ -153,7 +152,10 @@
 	gtk_file_chooser_add_filter (GTK_FILE_CHOOSER (filesel), filter);
 
 	if (GTK_RESPONSE_OK == gtk_dialog_run (GTK_DIALOG (filesel))) {
-		filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+		char *filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+
+		/* destroy dialog to get rid of it in the GUI */
+		gtk_widget_destroy (filesel);
 
 		if (e_cert_db_import_pkcs12_file (e_cert_db_peek (),
 						  filename, NULL /* XXX */)) {
@@ -163,9 +165,10 @@
 			load_certs (cfm, E_CERT_USER, add_user_cert);
 			gtk_tree_view_expand_all (GTK_TREE_VIEW (cfm->yourcerts_treeview));
 		}
-	}
 
-	gtk_widget_destroy (filesel);
+		g_free (filename);
+	} else
+		gtk_widget_destroy (filesel);
 }
 
 static void
@@ -373,7 +376,6 @@
 import_contact (GtkWidget *widget, CertificateManagerData *cfm)
 {
 	GtkWidget *filesel;
-	const char *filename;
 
 	GtkFileFilter *filter;
 
@@ -396,7 +398,10 @@
 	gtk_file_chooser_add_filter (GTK_FILE_CHOOSER (filesel), filter);
 
 	if (GTK_RESPONSE_OK == gtk_dialog_run (GTK_DIALOG (filesel))) {
-		filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+		char *filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+
+		/* destroy dialog to get rid of it in the GUI */
+		gtk_widget_destroy (filesel);
 
 		if (e_cert_db_import_certs_from_file (e_cert_db_peek (),
 						      filename,
@@ -409,9 +414,10 @@
 			load_certs (cfm, E_CERT_CONTACT, add_contact_cert);
 			gtk_tree_view_expand_all (GTK_TREE_VIEW (cfm->contactcerts_treeview));
 		}
-	}
 
-	gtk_widget_destroy (filesel);
+		g_free (filename);
+	} else
+		gtk_widget_destroy (filesel);
 }
 
 static void
@@ -597,7 +603,6 @@
 import_ca (GtkWidget *widget, CertificateManagerData *cfm)
 {
 	GtkWidget *filesel;
-	const char *filename;
 
 	GtkFileFilter *filter;
 
@@ -620,7 +625,10 @@
 	gtk_file_chooser_add_filter (GTK_FILE_CHOOSER (filesel), filter);
 
 	if (GTK_RESPONSE_OK == gtk_dialog_run (GTK_DIALOG (filesel))) {
-		filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+		char *filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filesel));
+
+		/* destroy dialog to get rid of it in the GUI */
+		gtk_widget_destroy (filesel);
 
 		if (e_cert_db_import_certs_from_file (e_cert_db_peek (),
 						      filename,
@@ -632,9 +640,10 @@
 			unload_certs (cfm, E_CERT_CA);
 			load_certs (cfm, E_CERT_CA, add_ca_cert);
 		}
-	}
 
-	gtk_widget_destroy (filesel);
+		g_free (filename);
+	} else
+		gtk_widget_destroy (filesel);
 }
 
 static void

Modified: trunk/smime/gui/smime-ui.glade
==============================================================================
--- trunk/smime/gui/smime-ui.glade	(original)
+++ trunk/smime/gui/smime-ui.glade	Wed Feb 25 11:28:08 2009
@@ -6,7 +6,7 @@
 <widget class="GtkDialog" id="certificate-viewer-dialog">
   <property name="title" translatable="no">dialog1</property>
   <property name="type">GTK_WINDOW_TOPLEVEL</property>
-  <property name="window_position">GTK_WIN_POS_NONE</property>
+  <property name="window_position">GTK_WIN_POS_CENTER</property>
   <property name="modal">False</property>
   <property name="resizable">True</property>
   <property name="destroy_with_parent">False</property>
@@ -1850,7 +1850,7 @@
 <widget class="GtkDialog" id="ca-trust-dialog">
   <property name="title" translatable="yes">Certificate Authority Trust</property>
   <property name="type">GTK_WINDOW_TOPLEVEL</property>
-  <property name="window_position">GTK_WIN_POS_NONE</property>
+  <property name="window_position">GTK_WIN_POS_CENTER</property>
   <property name="modal">False</property>
   <property name="resizable">True</property>
   <property name="destroy_with_parent">False</property>
@@ -2173,7 +2173,7 @@
 <widget class="GtkDialog" id="cert-trust-dialog">
   <property name="title" translatable="yes">Email Certificate Trust Settings</property>
   <property name="type">GTK_WINDOW_TOPLEVEL</property>
-  <property name="window_position">GTK_WIN_POS_NONE</property>
+  <property name="window_position">GTK_WIN_POS_CENTER</property>
   <property name="modal">False</property>
   <property name="resizable">True</property>
   <property name="destroy_with_parent">False</property>