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

Paul Golick commented on TUSCANY-1062:
--------------------------------------

I have verified this issue is fixed in revision 509219.  Please close this issue

> DataGraph.createRootObject implementation does not match 2.1 spec and also is 
> not helpful when given bad parameters
> -------------------------------------------------------------------------------------------------------------------
>
>                 Key: TUSCANY-1062
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-1062
>             Project: Tuscany
>          Issue Type: Bug
>          Components: Java SDO Implementation
>    Affects Versions: Java-M2
>            Reporter: Paul Golick
>             Fix For: Java-SDO-Mx
>
>         Attachments: DataGraphImpl_patch.txt, DataGraphTestCase.jar
>
>
> When createRootObject is called and a root object has already been created, 
> the SDO 2.1 spec calls for an IllegalStateException but the current 
> implementation replaces the existing root.
> Also, when either a type is not provided or when a type cannot be determined 
> from the URI and typeName provided, a NullPointerException is thrown from 
> org.eclipse.emf.ecore.util.EcoreUtil.create() which is not helpful to a user 
> who is not literate in emf utilities:  an exception with a helpful error 
> message should be provided.
> Attached are:
> a) a jar containing a new test case file for DataGraph 
> (sdo/impl/src/test/java/org/apache/tuscany/sdo/test/DataGraphTestCase.java)
> b) a patch that corrects these problems in 
> sdo/impl/src/main/java/org/apache/tuscany/sdo/impl/DataGraphImpl.java

-- 
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