[Kde-pim] Re: Review Request: Kmail: Custom drafts and templates are lost

[Kevin Krammer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101391/#review3424
-----------------------------------------------------------



messagecomposer/composerviewbase.cpp
<http://git.reviewboard.kde.org/r/101391/#comment2882>

    Since saving becomes asynchronous, it might be better to add the message to the job as a custom property.
    Maybe already as an item



messagecomposer/composerviewbase.cpp
<http://git.reviewboard.kde.org/r/101391/#comment2883>

    I know this is not part of your change, but overwriting target (which has been initialized with suitable default) without checking looks bad to me.
    Should probably be done in an extra review



messagecomposer/composerviewbase.cpp
<http://git.reviewboard.kde.org/r/101391/#comment2884>

    This is only necessary if target has been overwritten (no longer one of the two default collections).



messagecomposer/composerviewbase.cpp
<http://git.reviewboard.kde.org/r/101391/#comment2885>

    I'd say check for SaveInTemplates for consistence with check in the other method above


- Kevin


On May 20, 2011, 2:55 p.m., Antonis Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101391/
> -----------------------------------------------------------
> 
> (Updated May 20, 2011, 2:55 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Summary
> -------
> 
> Hello
> 
> If someone is trying to save a draft or a template mail and the custom collection doesn't exist then the mail is lost. With this patch, if the custom collection doesn't exist then the mail is saved in the default draft and template folders. (Which are created by the Akonadi::SpecialMailCollections)
> 
> 
> Diffs
> -----
> 
>   messagecomposer/composerviewbase.h 3b0a315 
>   messagecomposer/composerviewbase.cpp 9a7d5c3 
> 
> Diff: http://git.reviewboard.kde.org/r/101391/diff
> 
> 
> Testing
> -------
> 
> kdepim compiles and runs without any issue.
> 
> 
> Thanks,
> 
> Antonis
> 
>

_______________________________________________
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/