! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Thunar shows useless dialog on smb auth cancel
Status:
RESOLVED: FIXED

Comments

Description Rion 2012-04-14 09:57:23 CEST
Pressing "Cancel" in password prompt dialog, while connecting to windows share,
shows additional dialog with message that password was canceled.
its stupid and annoying.

Expected result: just silently close password prompt dialog
Comment 1 arbe 2016-06-26 06:24:00 CEST
Created attachment 6714 
Do not display error dialog for already handled errors

Docs for G_IO_ERROR_FAILED_HANDLED (int: 30) at [1] document it as:
Operation failed and a helper program has already interacted with the user. Do not display any error dialog. 

I've attached a patch to the error dialog function to do nothing for this "error".

[1] https://www.freedesktop.org/software/gstreamer-sdk/data/docs/latest/gio/gio-GIOError.html#GIOErrorEnum
Comment 2 Landry Breuil editbugs 2016-06-28 13:36:32 CEST
Makes sense to me, harald what do you think about this one ?
Comment 3 Harald Judt editbugs 2016-06-28 21:25:48 CEST
I have not tested it, but it makes sense.
Comment 4 Landry Breuil editbugs 2016-06-28 22:16:05 CEST
@arbe: have you tested it in the 'canceling connection to windows share' case ?
Comment 5 Landry Breuil editbugs 2016-06-28 22:26:10 CEST
I havent figured out the proper steps to reproduce the bug here..
Comment 6 arbe 2016-06-29 09:54:40 CEST
@Landry: I have - I simply went to "Browse Network" > WORKGROUP > My router's samba share and when the dialog popped up I simply opted "Cancel". No further dialog was shown (as expected).
Comment 7 Landry Breuil editbugs 2016-06-29 10:25:04 CEST
Okay, been able to test it the same way here, and indeed your patch removes the extraneous/annoying error dialog. I'll push it tonight!
Comment 8 Landry Breuil editbugs 2016-06-29 21:07:09 CEST
Thanks armin, applied in http://git.xfce.org/xfce/thunar/commit/?id=febb41736615d76d73bf8156290296e3de49349b

Bug #8695

Reported by:
Rion
Reported on: 2012-04-14
Last modified on: 2016-06-29

People

CC List:
4 users

Version

Attachments

Additional information