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

Issue in signature of p:validate-with-relax-ng #183

Closed
xml-project opened this issue Aug 2, 2019 · 2 comments

Comments

@xml-project
Copy link
Contributor

commented Aug 2, 2019

There is an issue with port schema defined as `content-type="text xml".
The prose then says, that a document with media type "application/relax-ng-compact-syntax" is also accepted and triggers the compact syntax mode.

Using a document with this media type on port schema will always raise "XD0038" (unexpected media type).

Solutions:

  1. We drop the special treat for this media type: If it comes in, authors have to use p:cast-content-type to make it a text document.
  2. We explicitly allow documents with this media type on port schema.
  3. We add this media type to our overall definition of "text document".

I have a slight preference for (3) because compact syntax documents could be treated as text (in text steps). But I could also life with (2). (1) would be inconvenient to authors.

@gimsieke

This comment has been minimized.

Copy link
Contributor

commented Aug 2, 2019

I’d prefer 3, too. We can then rephrase the prose to:

If the schema document has a “text” media type, which comprises “application/relax-ng-compact-syntax,” then it must be interpreted as a RELAX NG Compact Syntax document for validation.

@ndw

This comment has been minimized.

Copy link
Collaborator

commented Aug 5, 2019

Seems like this is something implementors are going to have to let users customize.

I don't object to enumerating a few more types as "text".

@xml-project xml-project self-assigned this Aug 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.