You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "slad (JIRA)" <ji...@apache.org> on 2010/03/12 23:51:44 UTC

[jira] Created: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Camel-Mail: Alternative body part does not handle charset
---------------------------------------------------------

                 Key: CAMEL-2545
                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
             Project: Apache Camel
          Issue Type: Bug
          Components: camel-mail
    Affects Versions: 2.2.0
         Environment: All
            Reporter: slad


Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment:     (was: MailBinding-Patch.java)

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>         Attachments: MailBinding-Patch.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Willem Jiang resolved CAMEL-2545.
---------------------------------

       Resolution: Fixed
    Fix Version/s: 2.3.0

Applied patch with thanks to slad.

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>             Fix For: 2.3.0
>
>         Attachments: bindingpatch.txt, MailBinding.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58325#action_58325 ] 

slad edited comment on CAMEL-2545 at 3/19/10 5:53 AM:
------------------------------------------------------

@willem

Sorry, I was using different logger, which was causing compilation issue. I have updated the source and also submitted the diff as you had requested.



Test Result:
----
Running org.apache.camel.component.mail.MimeMultipartAlternativeWithContentTypeTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.198 sec

      was (Author: tide08):
    @willem

Sorry, I was using different logger, which was causing compilation issue. I have updated the source and also submitted the diff as you had requested.
  
> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: bindingpatch.txt, MailBinding.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment: MailBinding-Patch.java

Applied patch to trunk version

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment: MailBinding-Patch.java

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>         Attachments: MailBinding-Patch.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment: MimeMultipartAlternativeWithContentTypeTest.java

TestCase for content-type fix.

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58320#action_58320 ] 

Willem Jiang commented on CAMEL-2545:
-------------------------------------

@slad,
Can you use diff to create the patch?
I copy the patched MailBinding.java into current trunk, there are some compile errors in the file.
So I doubt you've tested it.

If you want me to apply the patch quickly, please send the diff file, so I can easily tell which part change that you made. 

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Willem Jiang reassigned CAMEL-2545:
-----------------------------------

    Assignee: Willem Jiang

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment:     (was: MailBinding-Patch.java)

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58325#action_58325 ] 

slad commented on CAMEL-2545:
-----------------------------

@willem

Sorry, I was using different logger, which was causing compilation issue. I have updated the source and also submitted the diff as you had requested.

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: bindingpatch.txt, MailBinding.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment:     (was: MailBinding-Patch.java)

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58262#action_58262 ] 

Willem Jiang commented on CAMEL-2545:
-------------------------------------

@slad,
Can you create a patch based on the trunk code? As there is some change (CAMEL-2467) in the MailBinding.java.
 

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: MailBinding-Patch.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment: bindingpatch.txt
                MailBinding.java

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>            Assignee: Willem Jiang
>         Attachments: bindingpatch.txt, MailBinding.java, MimeMultipartAlternativeWithContentTypeTest.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CAMEL-2545) Camel-Mail: Alternative body part does not handle charset

Posted by "slad (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/CAMEL-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

slad updated CAMEL-2545:
------------------------

    Attachment: MailBinding-Patch.java

patch to set charset for alternative part. 

> Camel-Mail: Alternative body part does not handle charset
> ---------------------------------------------------------
>
>                 Key: CAMEL-2545
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2545
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.2.0
>         Environment: All
>            Reporter: slad
>         Attachments: MailBinding-Patch.java
>
>
> Mail component does not set charset for alternative body part. See - http://old.nabble.com/Camel-Mail:-Alternative-part-does-not-handle-charset--td27882178.html for more information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.