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

Add support to validate XML against XSD 1.1 #110

Open
mkordas opened this issue Jan 24, 2016 · 13 comments
Open

Add support to validate XML against XSD 1.1 #110

mkordas opened this issue Jan 24, 2016 · 13 comments

Comments

@mkordas
Copy link

mkordas commented Jan 24, 2016

Currently method com.jcabi.xml.XSDDocument#validate uses XSD 1.0 parser. XSD 1.1 has couple of new features added, like possibility to require exactly one attribute out of two that will be useful in other Teamed projects.

I'd expect just to replace 1.0 parser with 1.1, as version 1.1 is just superset of 1.0.

@dmarkov
Copy link

dmarkov commented Jan 28, 2016

@yegor256 please take a look at it and dispatch (see par.21)

@yegor256
Copy link
Member

@mkordas but we're not using any specific parser. we're just using JDK. parser is provided in runtime. or I didn't get the problem?

@mkordas
Copy link
Author

mkordas commented Jan 29, 2016

@yegor256 you are right! Do you think it is in scope of integration test, Javadoc or http://xml.jcabi.com/example-xsd.html to provide instruction how to enable XSD 1.1 parsing?

@yegor256 yegor256 added the bug label Jan 29, 2016
@yegor256
Copy link
Member

@mkordas yes, let's document it

@dmarkov
Copy link

dmarkov commented Feb 1, 2016

@mkordas thanks a lot for reporting, 15 mins added to your acc, pmt ID 56af291dc2680807d7000398

@mkordas
Copy link
Author

mkordas commented Mar 10, 2016

@yegor256 actually I've realized that we may have to modify line https://github.com/jcabi/jcabi-xml/blob/master/src/main/java/com/jcabi/xml/XSDDocument.java#L158. We should set it to http://www.w3.org/XML/XMLSchema/v1.1 according to https://xerces.apache.org/xerces2-j/faq-xs.html

@longtimeago
Copy link

@yegor256 can we proceed here?

@longtimeago
Copy link

@yegor256 could you please respond to messages above?

@longtimeago
Copy link

@yegor256 ping :)

@longtimeago
Copy link

@yegor256 please, proceed here :)

@yegor256
Copy link
Member

@dmarkov pls assign @westonized to this ticket

@dmarkov
Copy link

dmarkov commented Jun 28, 2016

@dmarkov pls assign @westonized to this ticket

@yegor256 OK @westonized please proceed, this task is yours

@longtimeago
Copy link

@westonized any progress so far?

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

No branches or pull requests

5 participants