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

Conformance of Sense IDs to xsd:ID in XML #749

Closed
1313ou opened this issue Sep 5, 2021 · 0 comments
Closed

Conformance of Sense IDs to xsd:ID in XML #749

1313ou opened this issue Sep 5, 2021 · 0 comments
Labels
release format This issue refers to the WNDB or RDF export, so no changes will be made to this repository
Milestone

Comments

@1313ou
Copy link
Contributor

1313ou commented Sep 5, 2021

Sense IDs conform to the old ID definition but not the more recent xsd:ID. The former are Names, the latter must be NCNames (NonColonizedName) which are defined by production:

NCName ::= Name - (Char* ':' Char*)
which is Name minus the colon.

see https://www.w3.org/TR/xmlschema-2/#ID

Validators which use the Java std runtime (1), Woodstox (2), RNG (3), ... will raise errors. I can confirm (1) and (2). (3) by https://stackoverflow.com/questions/6811188

(1)
[E] org.xml.sax.SAXParseException; systemId: file://src/xml/wn-noun.body.xml; lineNumber: 13; columnNumber: 82; cvc-datatype-valid.1.2.1: 'ewn-organ_of_hearing__1:08:00::' is not a valid value for 'NCName'. javax.xml.transform.stream.StreamSource@7530d0a

(2)
[E] attribute "id" has a bad value: "ewn-organ_of_hearing__1:08:00::" does not satisfy the "ID" type [row,col,system-id]: [13,7,"file:src/xml/wn-noun.body.xml"]

@1313ou 1313ou added the release format This issue refers to the WNDB or RDF export, so no changes will be made to this repository label Sep 5, 2021
@jmccrae jmccrae closed this as completed in 3acb02e Sep 9, 2021
@jmccrae jmccrae added this to the 2021 Release milestone Sep 9, 2021
This was referenced Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release format This issue refers to the WNDB or RDF export, so no changes will be made to this repository
Projects
None yet
Development

No branches or pull requests

2 participants