Hi,

I just have a look at the patch, and I can't just copy the java file into the repository, as there are some change on the MailBinding.java since Camel 2.2.0 release, so please send a diff patch which is based on the current trunk.

Thanks for your contribution.

Willem

tide08 wrote:
I already had patch with JIRA, and I just added a TestCase.
Thanks!


willem.jiang wrote:
+1 for this change, can you submit a patch with an unit test.
I will be happy to apply it into the trunk :)

Willem

tide08 wrote:
Just wanted to check, if this patch/fix made sense? Anyone?

Thanks!



tide08 wrote:
It looks like the mail component does not support charset handling for
alternative part? I can set Content-Type for main body but MailBinding
does not extract out charset from Content-Type?

Class: MailBinding

{code}
protected void createMultipartAlternativeMessage(MimeMessage
mimeMessage,
MailConfiguration configuration, Exchange exchange)
        throws MessagingException, IOException {

        MimeMultipart multipartAlternative = new
MimeMultipart("alternative");
        mimeMessage.setContent(multipartAlternative);

        BodyPart plainText = new MimeBodyPart();
        plainText.setText(getAlternativeBody(configuration, exchange));
        // remove the header with the alternative mail now that we got
it
        // otherwise it might end up twice in the mail reader
exchange.getIn().removeHeader(configuration.getAlternativeBodyHeader());
        multipartAlternative.addBodyPart(plainText);
{code}

But it should instead be -

 plainText.setText(getAlternativeBody(configuration, exchange),
charset);

Please let me know if I am missing something obvious and there is way to
specify it?

Thanks!






Reply via email to