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 367760 - Evolution Mail: The Save / Save All Butt...
Evolution Mail: The Save / Save All Butt...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
2.8.x (obsolete)
Other All
: High critical
: ---
Assigned To: Milan Crha
Evolution QA team
: 495606 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-10-30 16:42 UTC by Dave Croal
Modified: 2007-11-22 12:31 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
proposed eds patch (998 bytes, patch)
2007-08-17 15:19 UTC, Milan Crha
committed Details | Review
proposed evo patch (3.64 KB, patch)
2007-08-17 15:24 UTC, Milan Crha
committed Details | Review

Description Dave Croal 2006-10-30 16:42:14 UTC
What were you doing when the application crashed?
The Save / Save All Button for attachments in Evolution 2.8.1 does not work as well as it did in previous versions.

-It does not remember the directory chosen on previous saves, instead it always defaults to the users home directory.
-It overwrites files without prompting. If you save the same file via right-click "Save As..." it asks if you want to overwrite.
-File names with certain characters such as # are truncated on save. Again, if you save the same file via right-click "Save As..." the file name is saved correctly.

IIRC the previous version of Evolution had none of these problems.

Thanks,
Dave Croal


Distribution: Ubuntu 6.10 (edgy)
Gnome Release: 2.16.1 2006-10-02 (Ubuntu)
BugBuddy Version: 2.16.0
Comment 1 Kjartan Maraas 2007-01-15 23:00:05 UTC
Can you still reproduce this problem or has it been fixed since?
Comment 2 Dave Croal 2007-01-15 23:32:02 UTC
I checked again and all the problems listed above still exist.
Comment 3 Dave Croal 2007-03-13 16:17:38 UTC
I am now using Evolution 2.9.92 with Ubuntu Feisty and the problem with the Save/Save All button is mostly still there:

Fixed:
1. The problem where "it does not remember the directory chosen on previous saves, instead it always defaults to the users home directory" appears to be fixed now.

Still broken:
2. Save/Save All overwrites files without prompting. If you save the same file via
right-click "Save As..." it asks if you want to overwrite.

3. Attachment file names with certain characters such as # are truncated on save. If an email has several such attachments the first attachment is is saved with a truncated file name and the other attachments are not saved at all. Eg. "Letter of thanks # 1.doc" saves as "Letter of thanks".

A new but probably related problem has cropped up in 2.9.92, when an attachment with a file name containing spaces is saved via right-click, Save As... the saved file name contains %20 instead of spaces, making for a very ugly looking file name. Eg. "Letter of thanks # 1.doc" saves as "Letter%20of%20thanks%20%23%201.doc"

Thanks,
Dave Croal
Comment 4 Dave Croal 2007-03-28 14:45:25 UTC
The problems still exist with Evolution 2.10.0 from Ubuntu Feisty.
Comment 5 Milan Crha 2007-08-17 15:19:19 UTC
Created attachment 93852 [details] [review]
proposed eds patch

for evolution-data-server;

gnome-vfs doesn't like '#' character too, so serve with it as with non-safe character.
Comment 6 Milan Crha 2007-08-17 15:24:18 UTC
Created attachment 93853 [details] [review]
proposed evo patch

for evolution;

also added '#' as non-safe character into em_filename_make_safe and using this function when saving attachments. Removed gnome_vfs_escape_string from e_file_get_save_filesel, it makes a mess there :)
Comment 7 Matthew Barnes 2007-08-17 15:50:52 UTC
Milan, this is looking good.  For the "overwrite without prompting" problem, I think we just need to call gtk_file_chooser_set_do_overwrite_confirmation().

http://developer.gnome.org/doc/API/2.0/gtk/GtkFileChooser.html#gtk-file-chooser-set-do-overwrite-confirmation
Comment 8 Srinivasa Ragavan 2007-08-20 06:07:09 UTC
It currently asks for overwrite in 2.11.90 (I patched it sometime back). 

From comment #7, approving it.
Comment 9 Milan Crha 2007-08-20 07:14:00 UTC
eds part committed to trunk. Committed revision 7977.
evo part committed to trunk. Committed revision 34042.
Comment 10 Milan Crha 2007-11-22 12:31:08 UTC
*** Bug 495606 has been marked as a duplicate of this bug. ***