Re: [PATCH] Don't display authentication windows again if cancelled



Am Mittwoch, den 19.10.2005, 14:50 +0200 schrieb Alexander Larsson:
> On Fri, 2005-10-07 at 20:53 +0200, Christian Neumair wrote:
> > Together with [1] this patch ensures that authentication windows are not
> > shown again if they're cancelled.
> 
>          if (vfs_result_code == GNOME_VFS_OK
>              || vfs_result_code == GNOME_VFS_ERROR_NOT_SUPPORTED
> -            || vfs_result_code == GNOME_VFS_ERROR_INVALID_URI) {
> +            || vfs_result_code == GNOME_VFS_ERROR_INVALID_URI
> +	    || vfs_result_code != GNOME_VFS_ERROR_CANCELLED) {
> 
> This part looks wrong (if result is a, b, or c, or result is anything
> but cancelled, the last bit overrides the first). I don't think its
> needed at all, is it?

Yes, you are right. Dunno how this crap made it into the patch.
Reattaching new version, which doesn't have it.

-- 
Christian Neumair <chris gnome-de org>
Index: libnautilus-private/nautilus-program-choosing.c
===================================================================
RCS file: /cvs/gnome/nautilus/libnautilus-private/nautilus-program-choosing.c,v
retrieving revision 1.79
diff -u -p -r1.79 nautilus-program-choosing.c
--- libnautilus-private/nautilus-program-choosing.c	12 Aug 2005 18:11:29 -0000	1.79
+++ libnautilus-private/nautilus-program-choosing.c	7 Oct 2005 18:48:10 -0000
@@ -628,6 +628,8 @@ void nautilus_launch_show_file (Nautilus
 			detail_message = g_strdup 
 				(_("Please check the spelling and try again."));
 			break;
+		case GNOME_VFS_ERROR_CANCELLED:
+			break;
 		case GNOME_VFS_OK:
 		default:
 #if NEW_MIME_COMPLETE
Index: src/nautilus-window-manage-views.c
===================================================================
RCS file: /cvs/gnome/nautilus/src/nautilus-window-manage-views.c,v
retrieving revision 1.354
diff -u -p -r1.354 nautilus-window-manage-views.c
--- src/nautilus-window-manage-views.c	9 Jul 2005 20:09:28 -0000	1.354
+++ src/nautilus-window-manage-views.c	7 Oct 2005 18:48:15 -0000
@@ -1436,7 +1437,11 @@ display_view_selection_failure (Nautilus
 		detail_message = g_strdup
 			(_("Check that an SMB server is running in the local network."));
 		break;
-		
+
+	case GNOME_VFS_ERROR_CANCELLED:
+		g_free (uri_for_display);
+		return;
+
 	case GNOME_VFS_ERROR_SERVICE_NOT_AVAILABLE:
 	default:
 		error_message = g_strdup_printf (_("Nautilus cannot display \"%s\"."),
Index: src/file-manager/fm-directory-view.c
===================================================================
RCS file: /cvs/gnome/nautilus/src/file-manager/fm-directory-view.c,v
retrieving revision 1.711
diff -u -p -r1.711 fm-directory-view.c
--- src/file-manager/fm-directory-view.c	27 Sep 2005 08:34:37 -0000	1.711
+++ src/file-manager/fm-directory-view.c	7 Oct 2005 18:48:15 -0000
@@ -7248,7 +7269,8 @@ activate_callback (NautilusFile *file, g
 
 	view = FM_DIRECTORY_VIEW (parameters->view);
 
-	if (!activate_check_mime_types (view, file, TRUE)) {
+	if (!activate_check_mime_types (view, file, TRUE)
+	    || nautilus_file_get_file_info_result (file) == GNOME_VFS_ERROR_CANCELLED) {
 		nautilus_file_unref (file);
 		g_free (parameters);
 		
@@ -7379,6 +7401,13 @@ activate_activation_uri_ready_callback (
 	if (nautilus_file_is_broken_symbolic_link (file)) {
 		stop_activate (parameters);
 		report_broken_symbolic_link (parameters->view, file);
+		nautilus_file_unref (parameters->file);
+		g_free (parameters);
+		return;
+	}
+
+	if (nautilus_file_get_file_info_result (file) == GNOME_VFS_ERROR_CANCELLED) {
+		stop_activate (parameters);
 		nautilus_file_unref (parameters->file);
 		g_free (parameters);
 		return;

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]