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 327754 - run dialog(alt-f2) is giving non-usable error message
run dialog(alt-f2) is giving non-usable error message
Status: RESOLVED FIXED
Product: libgnome
Classification: Deprecated
Component: general
2.13.x
Other All
: Normal normal
: 2.14.x
Assigned To: libgnome maintainer
libgnome maintainer
: 330306 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-01-19 21:28 UTC by Sebastien Bacher
Modified: 2006-11-15 11:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
handle GNOME_VFS_ERROR_INVALID_URI error (1.31 KB, patch)
2006-01-24 00:37 UTC, Claudio Saavedra
committed Details | Review

Description Sebastien Bacher 2006-01-19 21:28:25 UTC
This bug has been described on https://launchpad.net/distros/ubuntu/+source/gnome-panel/+bug/29005

"When entering a non-existant command into the run dialog and pressing enter, the error message

Could not open location 'file://asd'

Details: Unknown error code: 13

Pops up - that's rather cryptic and non-informative for a random user. I would expect either a shaking window like in gnome-screensaver or a more useful message."
Comment 1 Claudio Saavedra 2006-01-24 00:30:23 UTC
This is actually a libgnome bug. gnome_url_show_with_env () doesn't recognize the GNOME_VFS_ERROR_INVALID_URI error and therefore sends the default message to the panel.

Reassigning to libgnome.
Comment 2 Claudio Saavedra 2006-01-24 00:37:47 UTC
Created attachment 57971 [details] [review]
handle GNOME_VFS_ERROR_INVALID_URI error

The message could not be the best, but you get the idea.
Comment 3 Vincent Untz 2006-01-24 07:38:09 UTC
Cool (I'm always happy to see bugs move away from gnome-panel). Anyone to review the patch? :-)

This is a new string so we have to add it soon.
Comment 4 Claudio Saavedra 2006-01-27 14:27:21 UTC
Adding string keyword, hopefully someone will interest on it.
Comment 5 Sebastien Bacher 2006-02-07 22:38:38 UTC
*** Bug 330306 has been marked as a duplicate of this bug. ***
Comment 6 Claudio Saavedra 2006-02-10 19:23:33 UTC
Ok to commit? String freeze is getting closer and closer!
Comment 7 Christian Neumair 2006-02-11 00:03:54 UTC
I agree that this issue has to be tackled, but aren't we opening a can of worms here? gnome_url_show_with_env is just a wrapper around gnome_vfs_url_show_with_env converting its results into a retval and a g_set_error, and the last few commits all deal with missing retvals. We'd just reach partity if the error enum matches GnomeVFSResult.

We could deprecate gnome_vfs_url_show_with_env and directly call the gnome-vfs helper.
Comment 8 Vincent Untz 2006-02-11 08:39:50 UTC
Christian: the error strings are more explicit in libgnome than in gnome-vfs, though.
Comment 9 Christian Neumair 2006-02-11 10:47:24 UTC
Vincent: Maybe we need helpers for getting more explicit strings from gnome-vfs which are ready to be displayed to the user?

Kjartan, are you OK with this patch?
Comment 10 Kjartan Maraas 2006-02-12 14:48:39 UTC
Looks good to me, but we're breaking the freeze now I guess?
Comment 11 Vincent Untz 2006-02-12 16:54:20 UTC
String freeze starts tomorrow :-)
Claudio: if you have a cvs account, I guess this means "go go go".
Comment 12 Claudio Saavedra 2006-02-12 17:01:36 UTC
Vincent,

I can commit before the string freeze, that is not a problem :-)

I would love to hear suggestions for the string. I am not sure if "The location or file could not be found." is good enough. What do you think?

I will commit it before the string freeze if nobody suggest something better.
Comment 13 Claudio Saavedra 2006-02-12 22:45:46 UTC
Fixed. Am about to mail everyone.

2006-02-12  Claudio Saavedra  <csaavedra@alumnos.utalca.cl>

        * libgnome/gnome-url.c: (gnome_url_show_with_env): Add handling of
        GNOME_VFS_ERROR_INVALID_URI error and a user readable message
        (Fixes #327754).
Comment 14 Sebastien Bacher 2006-02-28 11:21:45 UTC
Still happening, reopening. the call returns an GNOME_VFS_ERROR_NOT_FOUND not a GNOME_VFS_ERROR_INVALID_URI and anyway the dialog is to run an application, so the message should be than the command is not found and not that the location is incorrect
Comment 15 Claudio Saavedra 2006-02-28 12:41:44 UTC
(In reply to comment #14)
> Still happening, reopening. the call returns an GNOME_VFS_ERROR_NOT_FOUND not a
> GNOME_VFS_ERROR_INVALID_URI

Which command you entered to get a GNOME_VFS_ERROR_NOT_FOUND? I still get the GNOME_VFS_ERROR_INVALID_URI.

> and anyway the dialog is to run an application, so
> the message should be than the command is not found and not that the location
> is incorrect
> 

When entering a filename (and not an application) in the dialog this opens it with the default associated application. I agree that probably that is not what the dialog is intended for, but it is the way it is currently working. Maybe the dialog should autocomplete only executable files in the path and not everything that matches in $HOME.
Comment 16 Victor Osadci (Vic) 2006-02-28 13:08:18 UTC
I can get different 'unknown error code' messages by typing the protocol in the run dialogue, Try the following ones
ftp://ftp.gnome.org
sftp://gnome.org
/home/foo
Comment 17 Kjartan Maraas 2006-11-15 11:56:52 UTC
I don't see this any more. ftp:// and sftp:// opens nautilus correctly and /home/foo just says that it can't open it because it doesn't exist. An existing folder will open just fine with no error message. Please reopen if you can still reproduce this somehow.