You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tuscany.apache.org by "ant elder (JIRA)" <de...@tuscany.apache.org> on 2009/08/25 15:31:59 UTC

[jira] Commented: (TUSCANY-3231) OASIS otest ASM_5029_TestCase fails as Tuscany is not noticing that a composite property with isMany=false has multiple elements

    [ https://issues.apache.org/jira/browse/TUSCANY-3231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12747401#action_12747401 ] 

ant elder commented on TUSCANY-3231:
------------------------------------

Code has been fixed to spot the multiple value elements hwoever with the structure of the OASIS tests having the test composite fail with an error causes unrelated tests to fail which pick up the Test_ASM_5029.composite. So for now i've set it to use warning, see the comment in BaseAssemblyProcessor: // TODO: TUSCANY-3231


> OASIS otest ASM_5029_TestCase fails as Tuscany is not noticing that a composite property with isMany=false has multiple <value> elements 
> -----------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: TUSCANY-3231
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-3231
>             Project: Tuscany
>          Issue Type: Bug
>          Components: OASIS Compliance - OASIS
>            Reporter: ant elder
>             Fix For: Java-SCA-2.0
>
>
> OASIS otest ASM_5029_TestCase fails as Tuscany is not noticing that a composite property with isMany=false has multiple <value> elements 
> The code goes wrong in CompositeProcessor.read line 372
>                                 Document value = readPropertyValue(property.getXSDElement(), property.getXSDType(), property.isMany(), reader);
> and in readPropertyValue line 589 it does 
>                         loadElement(reader, root);
> and nothing ends up checking that the the next start element is another <value> element which should be an error when isMany is false.
> The stax parser reading code around all this is quite convoluted so no fix for this yet.

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