You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Willem Jiang (JIRA)" <ji...@apache.org> on 2010/01/02 06:12:15 UTC

[jira] Updated: (CAMEL-2327) The NP check of the remote service is wrong in CamelServiceExporter

     [ https://issues.apache.org/activemq/browse/CAMEL-2327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Willem Jiang updated CAMEL-2327:
--------------------------------

    Summary: The NP check of the remote service is wrong in CamelServiceExporter   (was: The NPE check of the remote service is wrong in CamelServiceExporter )

> The NP check of the remote service is wrong in CamelServiceExporter 
> --------------------------------------------------------------------
>
>                 Key: CAMEL-2327
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2327
>             Project: Apache Camel
>          Issue Type: Bug
>    Affects Versions: 1.4.0, 1.5.0, 1.6.0, 2.0-M1, 1.6.1, 1.6.2, 2.0-M2, 2.0-M3, 2.0.0, 2.1.0
>            Reporter: Willem Jiang
>            Assignee: Willem Jiang
>            Priority: Minor
>             Fix For: 1.6.3, 2.2.0
>
>
> We should check the export service instead of the camelContext.
> --- components/camel-spring/src/main/java/org/apache/camel/spring/remoting/CamelServiceExporter.jav(revision 895109)
> +++ components/camel-spring/src/main/java/org/apache/camel/spring/remoting/CamelServiceExporter.jav(working copy)
> @@ -83,7 +83,7 @@
>          }
>  
>          Endpoint endpoint = CamelContextHelper.getMandatoryEndpoint(camelContext, uri);
> -        notNull(camelContext, "service");
> +        notNull(getService(), "service");
>          Object proxy = getProxyForService();
>  
>          consumer = endpoint.createConsumer(new BeanProcessor(proxy, camelContext));

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