Skip to content

Conversation

joewiz
Copy link
Member

@joewiz joewiz commented Feb 6, 2022

The snippet with the XQuery version declaration contained a syntax error.

The snippet with the HTML5 serialization is updated to conform to the XSLT and XQuery Serialization 3.1 spec, at https://www.w3.org/TR/xslt-xquery-serialization-31/#HTML_VERSION. While eXist is forgiving about output:method="html5", this formulation will throw an error in Saxon and BaseX.

@joewiz joewiz added the bug label Feb 6, 2022
@joewiz
Copy link
Member Author

joewiz commented Feb 6, 2022

@wolfgangmm I'm curious to know: Is the snippets.xml used to generate the source of xquery.snippets? In testing this PR, my edits - which were only to xquery.snippets - had the desired effect in eXide. But I just stumbled upon snippets.xml and realized that perhaps I should be editing that and somehow generating the xquery.snippets file from it. I'm happy to adjust the PR, but I would appreciate any guidance on how to generate the xquery.snippets file. I wasn't able to find anything in source about this. Alternatively, we could delete the snippets.xml file. Thanks!

@joewiz joewiz requested a review from a team February 14, 2022 03:48
@line-o line-o merged commit bf82d73 into eXist-db:develop Feb 21, 2022
@joewiz joewiz deleted the update-xq-snippets branch February 22, 2022 15:31
@joewiz
Copy link
Member Author

joewiz commented Feb 22, 2022

@line-o Thank you!

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

Successfully merging this pull request may close these issues.

2 participants