You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Freeman Fang (JIRA)" <ji...@apache.org> on 2007/11/29 10:09:26 UTC

[jira] Created: (CAMEL-247) some issues with camel-cxf component

some issues with camel-cxf component
------------------------------------

                 Key: CAMEL-247
                 URL: https://issues.apache.org/activemq/browse/CAMEL-247
             Project: Apache Camel
          Issue Type: Bug
          Components: camel-cxf
    Affects Versions: 1.2.0
            Reporter: Freeman Fang
             Fix For: 1.3.0


1. input params List shouldn't be null, at lease it should be a empty List,
so use
List<Object> params = new ArrayList<Object>();
instead of 
List<Object> params = null;
in CamelInvoker.java
to avoid input params is null

2. Cxf Producer should only copy back exchange when ExchangePattern is not InOnly
so  use
if (exchange.getPattern() != ExchangePattern.InOnly) {
        	exchange.copyFrom(cxfExchange);
}
instead of 
    exchange.copyFrom(cxfExchange);

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


Re: [jira] Updated: (CAMEL-247) some issues with camel-cxf component

Posted by Freeman Fang <fr...@gmail.com>.
Can anyone review and apply this patch for me?
This is used for jaxws support in servicemix 4 which is based on 
camel-cxf component.
Thanks very much.
Freeman
Freeman Fang (JIRA) wrote:
>      [ https://issues.apache.org/activemq/browse/CAMEL-247?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>
> Freeman Fang updated CAMEL-247:
> -------------------------------
>
>     Attachment: CAMEL-247.patch
>
> address this issue and append test to verify it works
> Could anyone review and apply this patch for me?
>
> Thanks 
>
>   
>> some issues with camel-cxf component
>> ------------------------------------
>>
>>                 Key: CAMEL-247
>>                 URL: https://issues.apache.org/activemq/browse/CAMEL-247
>>             Project: Apache Camel
>>          Issue Type: Bug
>>          Components: camel-cxf
>>    Affects Versions: 1.2.0
>>            Reporter: Freeman Fang
>>             Fix For: 1.3.0
>>
>>         Attachments: CAMEL-247.patch
>>
>>
>> 1. input params List shouldn't be null, at lease it should be a empty List,
>> so use
>> List<Object> params = new ArrayList<Object>();
>> instead of 
>> List<Object> params = null;
>> in CamelInvoker.java
>> to avoid input params is null
>> 2. Cxf Producer should only copy back exchange when ExchangePattern is not InOnly
>> so  use
>> if (exchange.getPattern() != ExchangePattern.InOnly) {
>>         	exchange.copyFrom(cxfExchange);
>> }
>> instead of 
>>     exchange.copyFrom(cxfExchange);
>>     
>
>   

[jira] Updated: (CAMEL-247) some issues with camel-cxf component

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

Freeman Fang updated CAMEL-247:
-------------------------------

    Attachment: CAMEL-247.patch

address this issue and append test to verify it works
Could anyone review and apply this patch for me?

Thanks 

> some issues with camel-cxf component
> ------------------------------------
>
>                 Key: CAMEL-247
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-247
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-cxf
>    Affects Versions: 1.2.0
>            Reporter: Freeman Fang
>             Fix For: 1.3.0
>
>         Attachments: CAMEL-247.patch
>
>
> 1. input params List shouldn't be null, at lease it should be a empty List,
> so use
> List<Object> params = new ArrayList<Object>();
> instead of 
> List<Object> params = null;
> in CamelInvoker.java
> to avoid input params is null
> 2. Cxf Producer should only copy back exchange when ExchangePattern is not InOnly
> so  use
> if (exchange.getPattern() != ExchangePattern.InOnly) {
>         	exchange.copyFrom(cxfExchange);
> }
> instead of 
>     exchange.copyFrom(cxfExchange);

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


[jira] Resolved: (CAMEL-247) some issues with camel-cxf component

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

Hadrian Zbarcea resolved CAMEL-247.
-----------------------------------

    Resolution: Fixed

Patch applied with thanks!

> some issues with camel-cxf component
> ------------------------------------
>
>                 Key: CAMEL-247
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-247
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-cxf
>    Affects Versions: 1.2.0
>            Reporter: Freeman Fang
>             Fix For: 1.3.0
>
>         Attachments: CAMEL-247.patch
>
>
> 1. input params List shouldn't be null, at lease it should be a empty List,
> so use
> List<Object> params = new ArrayList<Object>();
> instead of 
> List<Object> params = null;
> in CamelInvoker.java
> to avoid input params is null
> 2. Cxf Producer should only copy back exchange when ExchangePattern is not InOnly
> so  use
> if (exchange.getPattern() != ExchangePattern.InOnly) {
>         	exchange.copyFrom(cxfExchange);
> }
> instead of 
>     exchange.copyFrom(cxfExchange);

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