29790 2016-08-23 20:57:30 +0000 [xslt30] Sample stylesheet for xml-to-json conversion uses a reserved namespace 2017-01-12 21:44:51 +0000 1 1 1 Unclassified XPath / XQuery / XSLT XSLT 3.0 Candidate Recommendation PC All RESOLVED FIXED P2 minor --- 1 mike mike public-qt-comments oldest_to_newest 127214 0 mike 2016-08-23 20:57:30 +0000 At some stage we changed the namespace used for elements in the "xml representation of JSON" schema to use the standard F+O namespace. We then changed the stylesheet for xml-to-json transformation accordingly. A consequence of this change is that the stylesheet now has used-defined functions in the F+O namespace, which is not allowed (the namespace is reserved). Also the following sentence in the spec (§22.4) is no longer true: This stylesheet includes a function j:xml-to-json which, apart from being in a different namespace, is functionally identical to the xml-to-json function described in the previous section. The j:xml-to-json namespace is currently in the same namespace as the standard fn:xml-to-json function. 127363 1 mike 2016-09-09 22:04:57 +0000 Reclassified as editorial. I have made the changes needed to the stylesheet. But leaving the bug open until the revised version is tested. 127669 2 mike 2016-10-06 12:34:20 +0000 I have been testing the revised stylesheet and came across a few issues, not all directly related to the stylesheet itself. The function entry points to the stylesheet should be declared as streamable stylesheet functions with streamability="absorbing". To test streamable use of the stylesheet I wanted to compare the results with streamed application of fn:xml-to-json() and I found that we had no tests for this, and it wasn't implemented in Saxon. So I've been developing tests (and an implementation that passes them) for fn:xml-to-json() and will come back to the stylesheet when that work is complete. 128360 3 mike 2017-01-12 00:23:13 +0000 I have now adapted the test set for fn:xml-to-json so the same tests are run with both the built-in function an the XSLT stylesheet, and both appear to give the same results, so I am now happy to close this bug.