[ 
https://issues.apache.org/jira/browse/TUSCANY-1109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12474146
 ] 

Kelvin Goodson commented on TUSCANY-1109:
-----------------------------------------

Yang,
   I see that the code looks a little fragile in the assumption that the 
delegate registry is the singleton instance,  but as far as I can see this is 
always the case unless someone is using the TypeHelperImpl constructor 
directly,  rather than the SDOUtil.createTypeHelper() (now deprecated) or 
SDOUtil.createHelperContext()  methods.  These are the only references in the 
impl codebase. Do I understand the issue you are raising correctly?  Is there a 
demonstrable failure for this? 

> TypeHelperImpl#getType(Class) queries wrong scope 
> (EPackage.Registry.INSTANCE) and misses delegate scopes
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: TUSCANY-1109
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-1109
>             Project: Tuscany
>          Issue Type: Bug
>          Components: Java SDO Implementation
>    Affects Versions: Java-SDO-Mx
>            Reporter: Yang ZHONG
>             Fix For: Java-SDO-Mx
>
>
> TypeHelperImpl#getType(Class) queries wrong scope: EPackage.Registry.INSTANCE.
> And TypeHelperImpl#getType(Class) misses delegate scopes, because 
> getRegistry().values() doesn't include delegateRegistry.values().

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to