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

clarify error about XML ID #533

Closed
TzviyaSiegman opened this issue May 5, 2015 · 4 comments
Closed

clarify error about XML ID #533

TzviyaSiegman opened this issue May 5, 2015 · 4 comments
Labels
status: wontfix The issue is rejected due to limitations (of scope or dev resources)

Comments

@TzviyaSiegman
Copy link
Collaborator

I received this error about ncx "Error while parsing file 'value of attribute "id" is invalid; must be an XML name without colons'." The ID in ncx was the same as the path to the content ("L1-ESN2014J-R12.xhtml#r12-sec-0003). The hash triggered the error. (Earlier IDs included hyphens, and no error reported). A clearer error message that does not single out colon would be helpful. There are many disallowed characters in XML IDs (http://www.w3.org/TR/REC-xml/#sec-common-syn).

@mattgarrish
Copy link
Member

mattgarrish commented May 5, 2015

Hi Tzviya,

A colon is a valid character in an xml name, though; it's xml namespaces that assigns it a purpose. To exclude it as invalid, It has to be singled out.

Matt

@TzviyaSiegman
Copy link
Collaborator Author

Makes sense, but I git this error only on ids with hashes (abd no colons)
On May 5, 2015 7:23 PM, "Matt Garrish" notifications@github.com wrote:

Hi Tzviya,

A colon is a valid character in an xml name, though; it's xml namespaces
that assigns it a purpose. To exclude it as invalid, It has to be singled
out.

Matt

From: Tzviya [mailto:notifications@github.com]
Sent: May 5, 2015 4:11 PM
To: IDPF/epubcheck
Subject: [epubcheck] clarify error about XML ID (#533)

I received this error about ncx "Error while parsing file 'value of
attribute "id" is invalid; must be an XML name without colons'." The ID in
ncx was the same as the path to the content
("L1-ESN2014J-R12.xhtml#r12-sec-0003). The hash triggered the error.
(Earlier IDs included hyphens, and no error reported). A clearer error
message that does not single out colon would be helpful. There are many
disallowed characters in XML IDs (
http://www.w3.org/TR/REC-xml/#sec-common-syn).


Reply to this email directly or view it on GitHub <
https://github.com/IDPF/epubcheck/issues/533> .


Reply to this email directly or view it on GitHub
#533 (comment).

@mattgarrish
Copy link
Member

mattgarrish commented May 5, 2015

Right, it's explaining the production rule you have to follow for them, not telling you what specific character is wrong. But you can't remove that exclusion without requiring people to go look up what an ID is supposed to conform to.

I don't think epucheck's ID validation rules try to single out the offending character(s), but I'll let others speak to whether that's feasible.

Matt

@rdeltour rdeltour added the status: wontfix The issue is rejected due to limitations (of scope or dev resources) label Aug 28, 2015
@rdeltour
Copy link
Member

I don't think we can single out the offending character. As far as I know, this message is built in the 3d party RelaxNG schema validator (Jing).
Closing the issue as wontfix, feel free to reopen if you disagree!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: wontfix The issue is rejected due to limitations (of scope or dev resources)
Projects
None yet
Development

No branches or pull requests

3 participants