W3C home > Mailing lists > Public > public-qt-comments@w3.org > March 2014

[Bug 25196] New: Availability of the xml-to-json packages

From: <bugzilla@jessica.w3.org>
Date: Fri, 28 Mar 2014 15:32:15 +0000
To: public-qt-comments@w3.org
Message-ID: <bug-25196-523@http.www.w3.org/Bugs/Public/>
https://www.w3.org/Bugs/Public/show_bug.cgi?id=25196

            Bug ID: 25196
           Summary: Availability of the xml-to-json packages
           Product: XPath / XQuery / XSLT
           Version: Last Call drafts
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSLT 3.0
          Assignee: mike@saxonica.com
          Reporter: abel.braaksma@xs4all.nl
        QA Contact: public-qt-comments@w3.org

Under [1], the section on XML To JSON, we do not currently mandate that the
packages in B.2 have to be made available by implementers. I suggest we add a
line something like the following:

An implementation must make the packages under B.2 available through their
name, 'http://www.w3.org/2013/XSLT/xml-to-json.xsl', whether or not the package
is also physically available (i.e. for editing) is implementation defined.

In addition, I noted the following minor issues:

1) the stylesheet under B.3 [2] uses xsl:import on a package with a relative
url. We have decided in the resolution for bug 24310 that packages cannot be
imported. 

2) the stylesheet under B.3 is not a package, which limits its usefulness and
does not make it readily available.

3) the stylesheet under B.3 does not re-use / redefine $j:start-map.

4) $indent-spaces should probably be $j:indent-spaces

We can solve (1) and (2) easily by embedding the stylesheet under B.3 inside
the package under B.2, with a default parameter for $indent-spaces of "0". In
addition, we can add a boolean, $j:indent, which, if set, changes the value of
other separators to include more whitespace (see for instance template
j:start-map).

We can solve (3) by rewriting $j:start-map dependent on the new $j:indent value
to include spacing.

Solving all these issues adds usability. We could simplify using the indented
version further by adding a function f:xml-to-json-indented, which calls the
overridden version of the templates (which will require a bit of redesigning of
the global parameters, allowing them to be overridden by function call and/or
apply-templates).

See also bug 25187, which mentions a few other (static) bugs in the stylesheets
in this section.

[1] http://www.w3.org/TR/xslt-30/#xml-to-json
[2] http://www.w3.org/TR/xslt-30/#xml-to-json-stylesheet-indented

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
Received on Friday, 28 March 2014 15:32:16 UTC

This archive was generated by hypermail 2.3.1 : Wednesday, 7 January 2015 15:45:55 UTC