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

Fixed error range for p-props-correct.2.1 #436

Closed
angelozerr opened this issue Jun 13, 2019 · 3 comments · Fixed by #440
Closed

Fixed error range for p-props-correct.2.1 #436

angelozerr opened this issue Jun 13, 2019 · 3 comments · Fixed by #440
Assignees
Labels
bug Something isn't working validation XSD
Milestone

Comments

@angelozerr
Copy link
Contributor

if minOccurs > maxOccurs in xs:element, xerces report a p-props-correct.2.1 and the range is not managed:

image

Here the XSD sample:

<?xml version="1.0" encoding="UTF-8"?>
<xs:schema 
	xmlns:xs="http://www.w3.org/2001/XMLSchema">
	<xs:complexType name="testType">     
		<xs:all>
			<xs:element name="testEle" minOccurs="1" maxOccurs="0" type="xs:string"/>
		</xs:all>
	</xs:complexType>
</xs:schema>
@xorye
Copy link

xorye commented Jun 13, 2019

Should the error range cover "xs:element" or "minOccurs="1""?
Either one, I am able to do very easily.

@xorye xorye self-assigned this Jun 13, 2019
@angelozerr
Copy link
Contributor Author

I would say minOccurs="1"" @NikolasKomonen @fbricon what do you think?

@xorye
Copy link

xorye commented Jun 13, 2019

I just spoke with @fbricon, it should be "minOccurs="1""

@angelozerr angelozerr added this to the v0.8.0 milestone Jun 14, 2019
@angelozerr angelozerr added bug Something isn't working validation XSD labels Jul 23, 2019
@angelozerr angelozerr changed the title Error range for p-props-correct.2.1 Error range for p-props-correct.2.1 Jul 23, 2019
@angelozerr angelozerr changed the title Error range for p-props-correct.2.1 Fixed error range for p-props-correct.2.1 Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working validation XSD
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants