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!