[Fwd: Re: [evolution-patches] Request review my patch for bug #46017]

[Antonio Xu <antonio xu sun com>]

Hello All
I have to resend this email to request you review my patch.

Anto
-------- Original Message --------
Subject: Re: [evolution-patches] Request review my patch for bug #46017
Date: Mon, 14 Jul 2003 13:02:51 +0800
From: Antonio Xu <antonio xu sun com>
To: Anna Marie Dirks <anna ximian com>
CC: Ettore Perazzoli <ettore ximian com>, evolution-patches ximian com
References: <1057851044 28496 82 camel anna boston ximian com>



I have created a new patch for bug #46017 according to Anna's advice, But I didn't add troubleshooting tips link in log, If you didn't think the error log was appropriate. Can you show me how I can wirte the formal error log? or you can file a new bug to complete error log.
Please review my new patch.

Thanks
Anto

Anna Marie Dirks wrote:

> Hi Antonio, Ettore, and hackers.
>
> So, regarding what label to use in the import wizard: 
>
> The only issue I have with Ettore's suggestion ("Evolution didn't find
> any settings that could be imported from other applications.") 
> is that it doesn't offer the user any information about what to do if
> she really does think that she has some importable data. That is, it
> doesn't offer any troubleshooting tips. It also doesn't explain what
> applications Evolution can import from. (Is that explained somewhere
> else in the wizard?) 
>
> Ideally, the message would be something like this: 
>
> "Evolution checked for settings to import from the following
> applications: Pine, Netscape, . No settings that
> could be imported where found. If you would like to try again, please
> click the "Back" button." (Or whatever troubleshooting tip might be
> appropriate, I am just guessing about the "Back" button.) If you wished,
> you could include a link to an online support document, in which case
> the last sentence would become "Please consult the troubleshooting tips
> in the Online Support Center (or whatever we are calling it these days)
> for more information about importing your old settings." 
>
> Does this make sense? 
>
> Anna
>
>
>
>
>  



Index: shell/e-shell-importer.c
===================================================================
RCS file: /cvs/gnome/evolution/shell/e-shell-importer.c,v
retrieving revision 1.56
diff -u -r1.56 e-shell-importer.c
--- shell/e-shell-importer.c	29 Apr 2003 02:40:56 -0000	1.56
+++ shell/e-shell-importer.c	14 Jul 2003 03:38:21 -0000
@@ -146,6 +146,12 @@
 	},
 	{ "intelligent_html",
 	  N_("Please select the information that you would like to import:")
+	},
+	{ "nodata_html",
+	  N_("Evolution checked for settings to import from the following\n"
+	     "applications: Pine, Netscape,Pine, Netscape, Elm, iCalendar.\n"
+	     "No settings that could be imported where found. If you would\n"
+	     "like to try again, please click the \"Back\" button.\n")
 	}
 };
 #define num_info (sizeof (info) / sizeof (info[0]))
@@ -734,9 +740,12 @@
 	ImportDialogImporterPage *import;
 	GList *l, *importers;
 	GtkWidget *table;
+	GtkLabel *no_data;
 	int running = 0;
 
 	if (data->importerpage->prepared == TRUE) {
+		if (data->importerpage->running == 0)
+			gnome_druid_set_buttons_sensitive(druid, TRUE, FALSE, TRUE, FALSE);
 		return TRUE;
 	}
 
@@ -763,8 +772,10 @@
 		/* No importers, go directly to finish, do not pass go
 		   Do not collect $200 */
 		import->running = 0;
-		gnome_druid_set_page (druid, GNOME_DRUID_PAGE (data->finish))
-			;
+		no_data = create_help ("nodata_html");
+		gtk_box_pack_start (GTK_BOX (data->importerpage->vbox), no_data,
+				    FALSE, TRUE, 0);
+		gnome_druid_set_buttons_sensitive(druid, TRUE, FALSE, TRUE, FALSE);
 		gtk_widget_destroy (dialog);
 		return TRUE;
 	}
@@ -887,7 +898,10 @@
 	gtk_widget_show_all (table);
 
 	if (running == 0) {
-		gnome_druid_set_page (druid, GNOME_DRUID_PAGE (data->finish));
+		no_data = create_help ("nodata_html");
+		gtk_box_pack_start (GTK_BOX (data->importerpage->vbox), no_data,
+				    FALSE, TRUE, 0);
+		gnome_druid_set_buttons_sensitive(druid, TRUE, FALSE, TRUE, FALSE);
 		gtk_widget_destroy (dialog);
 		return TRUE;
 	}
@@ -1194,7 +1208,7 @@
 			  G_CALLBACK (next_intelligent_page), data);
 	g_signal_connect (data->intelligent, "back",
 			  G_CALLBACK (back_intelligent_page), data);
-	g_signal_connect (data->intelligent, "prepare",
+	g_signal_connect_after (data->intelligent, "prepare",
 			  G_CALLBACK (prepare_intelligent_page), data);
 
 	data->importerpage = importer_importer_page_new (data);
@@ -1206,7 +1220,7 @@
 	
 
 	data->filedialog = glade_xml_get_widget (data->wizard, "page2-file");
-	g_signal_connect (data->filedialog, "prepare",
+	g_signal_connect_after (data->filedialog, "prepare",
 			  G_CALLBACK (prepare_file_page), data);
 	g_signal_connect (data->filedialog, "next",
 			  G_CALLBACK (next_file_page), data);
Index: shell/ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/shell/ChangeLog,v
retrieving revision 1.1287
diff -u -r1.1287 ChangeLog
--- shell/ChangeLog	9 Jul 2003 19:54:25 -0000	1.1287
+++ shell/ChangeLog	14 Jul 2003 03:48:20 -0000
@@ -1,3 +1,10 @@
+2003-07-10  Antonio Xu <antonio xu sun com>
+                                                                                        
+        * e-shell-importer.c: (show_import_wizard): use
+        g_signal_connect_after to connect "prepare" signal.
+        (prepare_intelligent_page): Display some error log and disable
+        forward button rather than jump to finish page.  [#46017]
+
 2003-07-09  Ettore Perazzoli  <ettore ximian com>
 
 	* e-shell-folder-commands.c (e_shell_command_rename_folder): Do