Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add xsl:map/@select #1632

Closed
michaelhkay opened this issue Dec 4, 2024 · 0 comments · Fixed by #1694
Closed

Add xsl:map/@select #1632

michaelhkay opened this issue Dec 4, 2024 · 0 comments · Fixed by #1694
Labels
Enhancement A change or improvement to an existing feature PR Pending A PR has been raised to resolve this issue XSLT An issue related to XSLT

Comments

@michaelhkay
Copy link
Contributor

michaelhkay commented Dec 4, 2024

I was surprised to discover that the xsl:map instruction does not allow a select attribute.

For many use cases it might make the instruction equivalent to xsl:sequence:

<xsl:map select="map:build(.....)"/>

<xsl:map select="{'a': 1, 'b': 2}"/>

but it still has documentary value; and there are other cases where it's not merely cosmetic:

<xsl:map select="$map1, $map2 => map:remove('extra'), {'extra': 17}"/>

There's no change to the semantics, the value of the select attribute is handled just like the value of the sequence constructor.

@michaelhkay michaelhkay added XSLT An issue related to XSLT Enhancement A change or improvement to an existing feature labels Dec 4, 2024
@michaelhkay michaelhkay added the PR Pending A PR has been raised to resolve this issue label Jan 9, 2025
@ndw ndw closed this as completed in #1694 Jan 14, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Enhancement A change or improvement to an existing feature PR Pending A PR has been raised to resolve this issue XSLT An issue related to XSLT
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant