[Kde-pim] Re: Review Request: Output error message when saving an attachment to a disk, which becomes full - Bug 127981

[George Metaxas]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6104/
-----------------------------------------------------------

(Updated 2010-12-12 19:17:09.945659)


Review request for KDE PIM.


Changes
-------

This should address all comments made by Ingo. I merged the two cases into one, and removed the unnecessary new string.


Summary
-------

As discussed in Bug 127981, when one attempts to save an attachment to a disk which has some space available, but not enough to fit the attachment, KMail will not output an error message informing the user about this. Instead, the attachment will be partially saved and the user will be oblivious to what really happened. This patch will show an error message to the user and will also remove the partially written file. KMail does indeed correctly inform the user if the disk has no space left, but it leaves behind a zero-sized file. This patch also removes this unnecessary file.


This addresses bug 127981.
    https://bugs.kde.org/show_bug.cgi?id=127981


Diffs (updated)
-----

  /trunk/KDE/kdepim/messageviewer/util.cpp 1205704 

Diff: http://svn.reviewboard.kde.org/r/6104/diff


Testing
-------

Tested saving an attachment with:
- No space left
- Some space left, but not enough to fit the attachment
- Enough space to fit the attachment


Thanks,

George

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/