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 457936 - Directory names with '#' character confuse gthumb
Directory names with '#' character confuse gthumb
Status: RESOLVED FIXED
Product: gthumb
Classification: Other
Component: general
2.10.x
Other All
: Normal normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks:
 
 
Reported: 2007-07-18 12:37 UTC by Pavel
Modified: 2007-07-19 14:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
At apply_transformation_jpeg, escape the path for moving. (1.18 KB, patch)
2007-07-19 12:00 UTC, manuel braga
none Details | Review

Description Pavel 2007-07-18 12:37:45 UTC
Please describe the problem:
When gThumb directed to rotate an image placed in a directory, whose name has a
'#' character, it does not perform rotation of an original image. Instead, the
rotated image is saved in a parent directory with a file name equal to a part of
 its directory name before '#'.

Steps to reproduce:
1. Create a directory with # character (/home/username/picture#1) and place an
image into that directory.
2. Browse to that directory with gThumb and try to rotate an image.


Actual results:
Original image remains non-rotated and file named 'picture' holding rotated image is created in a parent directory (/home/username).


Expected results:
Rotate an original image.

Does this happen every time?
yes

Other information:
Comment 1 manuel braga 2007-07-19 12:00:20 UTC
Created attachment 91972 [details] [review]
At apply_transformation_jpeg, escape the path for moving.

This only happens with jpeg images.
The problem is that the # is a special gnomevfs character.
Comment 2 Michael Chudobiak 2007-07-19 12:11:40 UTC
Manuel,

Is that the right place for escaping? Or should escaping be added to the underlying function (xfer_file)?

- Mike
Comment 3 manuel braga 2007-07-19 12:41:51 UTC
For doing the escaping there, all calls have to be rechecked.
Because, at least in move_to_catalog_dir__ok_cb, escaped paths are passed to file_move funtion.
Comment 4 Michael Chudobiak 2007-07-19 13:05:47 UTC
To be honest, I'm pretty confused about what is escaped and what isn't. Is there an easy way to tell?

- Mike
Comment 5 Michael Chudobiak 2007-07-19 13:17:11 UTC
In other words... I'm a little confused why the rotation file has unescaped uris. I'm looking into that.

- Mike
Comment 6 manuel braga 2007-07-19 13:59:39 UTC
I don't know if i am understanding.
In apply_transformation_jpeg, the jpegtran needs unescaped paths. And the gnomevfs functions need escaped paths because some characters have special meanings.

But you are right, this is confusing.
Comment 7 Michael Chudobiak 2007-07-19 14:16:56 UTC
Anyway, I've committed the patch (svn rev 1807, should appear in 2.10.6). Thanks!

I guess that gvfs is supposed to replace gnome-vfs and make URI handling much easier. I hope it's ready soon...

- Mike