Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

json2xml adds unneccesary xsi:type #21

Open
JDziurlaj opened this issue Apr 28, 2023 · 1 comment
Open

json2xml adds unneccesary xsi:type #21

JDziurlaj opened this issue Apr 28, 2023 · 1 comment
Labels
wontfix This will not be worked on

Comments

@JDziurlaj
Copy link
Contributor

There are two scenarios where subtypes can be used. An association can point to an abstract or non-abstract type, in which the concrete subtype is allowed or required. The second scenario is when an association points to a particular non-abstract subtype. In this case, the subtype is always called for, and there is no need for xsi:type. For example, CVR::ImageData points to CVR::Image, a subtype of CVR::File, in this case there is no need to use xsi:type.

@JDziurlaj
Copy link
Contributor Author

This is not a super easy fix, because the application of xsi:type is on the xsl:template associated with each class, but we need the xsi:type to apply contextually.

@JDziurlaj JDziurlaj added the wontfix This will not be worked on label Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant