[Kde-pim] Review Request 108884: fix character encoding for body in composer window when creating mail from command line

[Laurent Montel]

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

Ship it!


Seems good for me.
Work fine here.
Please apply in 4.10 branch and we will merge after that.
(don't cherry pick please)

Regards.

- Laurent Montel


On Feb. 10, 2013, 10:30 a.m., Martin Koller wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108884/
> -----------------------------------------------------------
> 
> (Updated Feb. 10, 2013, 10:30 a.m.)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Description
> -------
> 
> assemble message before it is passed to the composer window, which makes sure that also the ContentType header is used which contains the charset.
> 
> I added the assemble() call also to other methods in this file where I thought it's equally missing but as I do not know when these methods are called, I could not test these.
> 
> 
> This addresses bug 259039.
>     http://bugs.kde.org/show_bug.cgi?id=259039
> 
> 
> Diffs
> -----
> 
>   kmail/kmkernel.cpp 243ba8f 
> 
> Diff: http://git.reviewboard.kde.org/r/108884/diff/
> 
> 
> Testing
> -------
> 
> commandline call with --body and also --msg and given text with some asian characters
> 
> 
> Thanks,
> 
> Martin Koller
> 
>

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