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 326571 - Evolution converts *space* in filename to *underscore* when saving attachments
Evolution converts *space* in filename to *underscore* when saving attachments
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.6.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2006-01-11 10:12 UTC by Xiurong Simon Zheng
Modified: 2013-09-13 00:48 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
Now filenames will retain spaces when saving attachments (1.30 KB, patch)
2006-02-02 13:13 UTC, Rajeev R
needs-work Details | Review
clone a copy in local file (2.29 KB, patch)
2006-03-08 11:44 UTC, Xiurong Simon Zheng
accepted-commit_now Details | Review

Description Xiurong Simon Zheng 2006-01-11 10:12:26 UTC
Please describe the problem:
If text attachment filenames contain *space* or printable characters,
such as "/'\"`&();|<>$%{}!", they will be automatically converted to
*underscore* when saving attachments. These characters are valid on Unix.

Steps to reproduce:
1. Send and receive an e-mail with attachment whose orginal filename contains
spaces.
2. Save this file somewhere.



Actual results:
Spaces in the filename are replaced with underscores.

Expected results:
Keep the spaces in filename.

Does this happen every time?
yes.

Other information:
This conversation is done by e_filename_make_safe()
(e-d-s/libedataserver/e-data-server-util.c). I think some printable character is
uncessary to limit.
Comment 1 André Klapper 2006-01-11 10:17:20 UTC
i've already seen this in bugzilla, but i cannot find the duplicate.
Comment 2 Rajeev R 2006-02-02 13:13:46 UTC
Created attachment 58582 [details] [review]
Now filenames will retain spaces when saving attachments
Comment 3 Srinivasa Ragavan 2006-02-02 16:29:20 UTC
I doesnt say 'invalid' it is said 'unsafe'. Even if we want to fix this, I wouldnt prefer doing this way, since this fn is used in many other functions and I would prefer cloning this fucntion for saving attachments along locally in that file.

Im rejecting the patch.
Comment 4 Srinivasa Ragavan 2006-02-02 16:30:27 UTC
Adding rajeev to CC to rework on that patch.
Comment 5 Xiurong Simon Zheng 2006-03-08 11:44:39 UTC
Created attachment 60906 [details] [review]
clone a copy in local file

Srag, I clone a copy from e-d-s and make a bit changes, then use it to allow attachment names contain more valid characters when saving attachment.
Comment 6 Srinivasa Ragavan 2006-03-20 04:00:13 UTC
Looks good to commit to both HEAD and 2-14 branch.
Comment 7 Xiurong Simon Zheng 2006-03-21 03:14:16 UTC
Committed the patch to CVS HEAD and gnome-2-14 branch.