Resolve circulair includes#190
Conversation
c763ad6 to
a842c24
Compare
|
When merging circulair dependencies, types will be re-added to the schema, this fix merged and deduplicate things. |
|
P.S. NeTEx also has a name-clash coming from case-sensitivity: <xsd:element name="FlexibleServiceProperties" type="flexibleServiceProperties" maxOccurs="unbounded"/>GoName() turns both names to |
|
Thank You for your contribution, @friedelschoen . Don't worry about the lint. Let me find some time later to review this in depth. In the mean time, would you perhaps like to add a minimalist test for this? |
There is an open semi-abandoned pr #45, to rework name clash resolver. You can take a look to see whether you like that path. |
|
I'll add some tests next week 😄 I want to look into #45 as NeTEx has many such shenanigans but cannot promise anything. |
6bb3490 to
f8d4aab
Compare
|
Hey! I've added some short unit tests for circular includes and imports, is that alright? |
By definition XSD does allow circulair includes, but trying to do so results in a "hot-potato" loop. This fix holds a loaded schema in memory which won't be re-resolved over and over.