You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Daniel Kulp (JIRA)" <ji...@apache.org> on 2007/01/24 20:41:49 UTC
[jira] Updated: (CXF-389) Make org.apache.cxf.jaxws.EndpointImpl
constructor generic
[ https://issues.apache.org/jira/browse/CXF-389?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel Kulp updated CXF-389:
----------------------------
Fix Version/s: 2.0-RC
Assignee: Daniel Kulp
Affects Version/s: (was: 2.0-RC)
> Make org.apache.cxf.jaxws.EndpointImpl constructor generic
> ----------------------------------------------------------
>
> Key: CXF-389
> URL: https://issues.apache.org/jira/browse/CXF-389
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime
> Affects Versions: 2.0-M1
> Reporter: Bharath Ganesh
> Assigned To: Daniel Kulp
> Fix For: 2.0-RC
>
>
> Bharath:
> Currently (in M1 release) the constructor of EndpointImpl looks like
> *public EndpointImpl(Bus b, Object implementor, JaxWsServiceFactoryBean
> serviceFactory).*
> Instead it would be better if it takes an AbstractJaxWsServiceFactoryBean
> instead of JaxWsServiceFactoryBean. This would be helpful for Provider based
> endpoints to use this contructor, since JaxWsServiceFactoryBean and
> ProviderServiceFactoryBean are subclasses of
> AbstractJaxWsServiceFactoryBean.
> Dan:
> I think that would be fine. I don't see any code specific to to
> JaxWsServiceFactoryBean in the constructor. Could you create a JIRA issue
> for this please?
> Thanks,
> - Dan
> Daniel:
> Not only possible, but definitely more "correct". The internal field that is
> set actually is an AbstractJaxWsServiceFactoryBean:
> private AbstractJaxWsServiceFactoryBean serviceFactory;
> .....
> this.serviceFactory = serviceFactory;
> Nothing in the constructor requires it to be the specific subclass.
> I'll go ahead and make that change.
> Thanks!
> --
> J. Daniel Kulp
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.