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

Transitive includes not supported #27

Closed
lfrancke opened this issue Sep 7, 2020 · 2 comments
Closed

Transitive includes not supported #27

lfrancke opened this issue Sep 7, 2020 · 2 comments

Comments

@lfrancke
Copy link

lfrancke commented Sep 7, 2020

I decided to open a new issue but it is related to this comment #25 (comment)

The problem is that we have a XSD (A) that includes another one (B) which in turn imports a third one (C).
Currently when A references something from C it is treated as a UnsolvedReference.

According to this post on StackOverflow it's not entirely clear whether this should be supported or not: https://stackoverflow.com/questions/13735887/use-type-from-xsd-imported-from-import

So this is a feature request: It'd be great if XsdParser could (maybe optionally) process transitive includes.

@lcduarte
Copy link
Member

Hello,

I'm also not sure if XSD should support that or not. I'm using Intellij and if I'm using:

A including B
B including C

It shows the elements of C in A.

On the other hand if I have:

A importing B
B importing C

It only shows elements from B on A.

Apart from that, I've had to improve some stuff to solve #25 which have also "resolved" this problem.

I've introduced some use cases and tests (Transitive Includes, Transitive Imports) related with this issue, you can take a look if you want.

I've also performed a new version where all of this is available, (1.1.0), give it a shot.

Thanks for using the library and for the feedback!

@lfrancke
Copy link
Author

lfrancke commented Oct 5, 2020

Thank you for implementing this! Unfortunately, I'm not currently able to test this with my original data as I've lost access to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants