This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 30139 - [xslt30ts] removal of visibility="absent"
Summary: [xslt30ts] removal of visibility="absent"
Status: RESOLVED FIXED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: XSLT 3.0 Test Suite (show other bugs)
Version: Proposed Recommendation
Hardware: PC All
: P2 normal
Target Milestone: ---
Assignee: Abel Braaksma
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-10 14:57 UTC by Michael Kay
Modified: 2019-02-25 18:53 UTC (History)
0 users

See Also:


Attachments

Description Michael Kay 2017-07-10 14:57:04 UTC
Bug 29574 removed the value visibility="absent" from the xsl:accept element, but this attribute value is still present in a number of test cases, to wit accept-040 to -047.

This raises the question of how these tests passed the validation performed by test catalog-005, which tests all non-error stylesheets against the schema for XSLT 3.0. I will raise this as a separate bug.
Comment 1 Michael Kay 2017-07-10 16:13:57 UTC
Also affects override-f-002
Comment 2 Michael Kay 2017-07-10 21:40:21 UTC
And override-f-008
Comment 3 Michael Kay 2017-07-10 22:37:43 UTC
The tests accept-040 to -047 need to be rethought. They are written on the basis that a using package is allowed to contain a static reference to an absent component, but cannot dynamically evaluate that reference. The consequence of bug 29574 is not just to change the keyword from "absent" to "hidden", but to change the rules as well. Being hidden, the relevant component is now not visible in the using package, so static references in the using package are not allowed. But for a component reference to the abstract component appearing within the used package, it remains true that static references are allowed, while dynamic references cause a dynamic error.
Comment 4 Michael Kay 2017-07-11 11:37:48 UTC
I have revised tests accept-040-047 accordingly.

Leaving open in case the WG wishes to review the changes.
Comment 5 Abel Braaksma 2019-02-25 18:53:57 UTC
Tests expose-916 - 924 and tests accept-901 - 915 have been added to deal with various combinations of (invalid) use of absent that has become possible after the here-mentioned spec change.

With the other, now historical changes in this test set, I think this can be closed as resolved. Please reopen if you find there are still issues with these test w.r.t. abstract.