You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Jonathan Anstey (JIRA)" <ji...@apache.org> on 2009/01/02 18:44:05 UTC
[jira] Commented: (CAMEL-1213) camel-jms consumer does not set
camel message id based on JMS message id if the next node is not JMS based
[ https://issues.apache.org/activemq/browse/CAMEL-1213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=48427#action_48427 ]
Jonathan Anstey commented on CAMEL-1213:
----------------------------------------
Claus,
The fix makes sense to me. I think the ! was just a typo when JmsMessage was originally created.
> camel-jms consumer does not set camel message id based on JMS message id if the next node is not JMS based
> ----------------------------------------------------------------------------------------------------------
>
> Key: CAMEL-1213
> URL: https://issues.apache.org/activemq/browse/CAMEL-1213
> Project: Apache Camel
> Issue Type: Bug
> Components: camel-jms
> Affects Versions: 1.5.0
> Reporter: Claus Ibsen
> Assignee: Claus Ibsen
> Fix For: 1.5.1, 2.0.0
>
> Attachments: CAMEL-1213.patch
>
>
> The code in org.apache.camel.component.jms.copyFrom looks odd
> I think the ! should be removed
> BEFORE
> {code}
> if (!copyMessageId) {
> setMessageId(that.getMessageId());
> }
> {code}
> AFTER
> {code}
> if (copyMessageId) {
> setMessageId(that.getMessageId());
> }
> {code}
> I will add a unit test (JmsToFileMessageIdTest) based on end user having some trouble with it
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.