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

Validate XML Schema with Xerces XSD validator. #390

Merged
merged 1 commit into from
May 29, 2019
Merged

Conversation

angelozerr
Copy link
Contributor

@angelozerr angelozerr commented May 24, 2019

See #190

Signed-off-by: azerr [email protected]

@angelozerr
Copy link
Contributor Author

@angelozerr
Copy link
Contributor Author

Please note that I tested with JDK 11 and it works (test was done with JDK 11 because I'm using Java Reflection because Xerces cannot work with reporter (only error handler) for XSD validator)

@fbricon
Copy link
Contributor

fbricon commented May 27, 2019

Error ranges are off
Screen Shot 2019-05-27 at 4 32 46 PM

using:

<?xml version='1.0'?>
<xs:schema xmlns:xs='http://www.w3.org/2001/XMLSchema'>
    <xs:element name='note'>
        <xs:complexType>
            <xs:sequence>
                <xs:element nhame='to' type='xs:string' nillable='false' />
                <xs:element name='from' type='xs:string' />
                <xs:element name='heading' type='xs:string' />
                <xs:element name='body' type='xs:string' nillable='false' />
            </xs:sequence>
        </xs:complexType>
    </xs:element>
</xs:schema>

@angelozerr
Copy link
Contributor Author

Error ranges are off

Error ranges was implemented like WTP XML Editor, but there are some error ranges which ar emissing like you have found. I hav eimplemented it but we need other error I think. Perhaps it could be an another PR?

@NikolasKomonen NikolasKomonen changed the title Validate XML Schema with Xerces XSD validator. See #190 Validate XML Schema with Xerces XSD validator. May 28, 2019
@NikolasKomonen
Copy link
Contributor

Works well for me, and if any error ranges were missed we can create new issues for them.

@NikolasKomonen NikolasKomonen merged commit 12f97c0 into master May 29, 2019
@angelozerr angelozerr deleted the xsdvalidator branch May 29, 2019 21:12
@angelozerr angelozerr restored the xsdvalidator branch May 31, 2019 14:07
@angelozerr angelozerr deleted the xsdvalidator branch May 31, 2019 14:08
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

Successfully merging this pull request may close these issues.

3 participants