After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 594495 - annoying dialog
annoying dialog
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 594798 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-09-08 13:42 UTC by Matthias Clasen
Modified: 2010-07-12 13:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Do not display error dialog when cancelling the password dialog when mounting shares (1.25 KB, patch)
2009-10-11 19:27 UTC, Marcus Carlson
none Details | Review

Description Matthias Clasen 2009-09-08 13:42:00 UTC
When I click on a location in network:// that has been shared via gnome-user-share and protected with a password, I get a password dialog. Which is good.
But if I cancel that password dialog (e.g. because I don't know the password), I get an annoying error dialog that says:

Unable to mount location
HTML Error: Cancelled

There is no reason to tell me that it was cancelled, I just pressed the cancel button myself...
Comment 1 Marcus Carlson 2009-10-11 19:27:40 UTC
Created attachment 145253 [details] [review]
Do not display error dialog when cancelling the password dialog when mounting shares
Comment 2 Marcus Carlson 2009-10-11 19:30:18 UTC
The last patch will not work as we do not have something called G_IO_ERROR_USER_CANCELLED (yet). But once we do this will prevent the error dialog from appearing.

Also note that we need to patch the gvfs backends to use the new code as well.

Alex; Company will talk to you about this.
Comment 3 Alexander Larsson 2009-10-12 08:42:28 UTC
I don't see why G_IO_ERROR_CANCELLED can not be reported by a user requestion. In fact, almost all current uses of G_IO_ERROR_CANCELLED is from a user cancelling something. Why would you need a separate code for this?
Comment 4 Benjamin Otte (Company) 2009-10-29 13:52:04 UTC
*** Bug 594798 has been marked as a duplicate of this bug. ***
Comment 5 Benjamin Otte (Company) 2009-10-29 13:58:51 UTC
G_IO_ERROR_CANCELLED is a special operation that happens only if a cancellable is attached to an operation and that cancellable has been cancelled.
It seems to me that just returning CANCELLED in other cases is a bad idea, my code probably relies on that behavior (though not in this exact case).

I made gvfs return G_IO_ERROR_FAILED_HANDLED for these cases in http://git.gnome.org/cgit/gvfs/commit/?id=bcc92d779f6e5d1d2bd960ff20d8b4c944cebe91 (the fix is in 1.4.1, too) so it's an error that applications can check for.
Alex said on IRC that he wanted to use a new error code for explicit user cancellation, so I'm keeping this bug open.
Comment 6 Marcus Carlson 2010-07-11 20:24:11 UTC
Didn't this get fixed in [1], bug #601096 ?

[1] http://git.gnome.org/browse/nautilus/commit/?id=ae1e7474830fb98eea93e1f4aadbf6e2b6caefc2
Comment 7 Matthias Clasen 2010-07-12 13:21:37 UTC
Looks like it, yes.