Bug #5563: SaxonJS allows elements in supposedly empty xsl:copy-of instruction - SaxonJS - Saxonica Developer Community

Actions

Send by e-mail Copy link

Bug #5563

closed

SaxonJS allows elements in supposedly empty xsl:copy-of instruction

Added by Martin Honnen almost 2 years ago. Updated over 1 year ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

XSLT Conformance

Sprint/Milestone:

SaxonJS 2.5

Start date:

2022-06-14

Due date:

% Done:

100%

Estimated time:

Applies to JS Branch:

2, Trunk

Fix Committed on JS Branch:

2, Trunk

Fixed in JS Release:

SaxonJS 2.5

SEF Generated with:

Platforms:

Company:

Contact person:

Additional contact persons:

Description

Due to some mental mistake I wrote some XSLT code using e.g. <xsl:copy-of select="."><xsl:apply-templates/></xsl:copy-of> and SaxonJS let met get away with it. When moving my code to SaxonJ, I found that it doesn't allow it and gives an error that xsl:copy-of is supposed to be empty, as indeed the XSLT spec defines it.

So I suppose this is a bug in terms of XSLT comformance of SaxonJS.

Actions

Copy link

Updated by John Lumley almost 2 years ago

Yes - this looks like an oversight - the original 'syntax check' for xsl:copy-of incorrectly declared the possibility of a sequence constructor child (as xsl:copy of course is permitted, and I assume I copy-and-pasted). Interestingly there must have been no XSLT test case which checked for a compiler error in such circumstances. The runtime code appears to ignore any children however.

Actions

Copy link

Updated by Norm Tovey-Walsh almost 2 years ago

Priority changed from Low to High
Sprint/Milestone changed from SaxonJS 2.4 to SaxonJS 3.0

Actions

Copy link

Updated by Michael Kay almost 2 years ago

Added XSLT3 test cases copy-0104 and copy-0105.

Actions

Copy link

Updated by Michael Kay almost 2 years ago

XSLT-elements.xsl line 81 reads

        <copy-of select="expression" copy-namespaces="boolean" copy-accumulators="boolean" type="eqname" validation="strict|lax|preserve|strip">
            <sequence-constructor/>
        </copy-of>

The <sequence-constructor/> is incorrect.

Actions

Copy link