Skip to content

780 Document incompatibility in format-number etc - #925

Merged
ndw merged 2 commits into
qt4cg:masterfrom
michaelhkay:780-format-number-incompatibility
Jan 9, 2024
Merged

780 Document incompatibility in format-number etc#925
ndw merged 2 commits into
qt4cg:masterfrom
michaelhkay:780-format-number-incompatibility

Conversation

@michaelhkay

Copy link
Copy Markdown
Contributor

Fix #780

Changes the XSLT and F+O specs to document a minor incompatibility arising from the change to functions such as format-number() to accept an argument of type union(xs:string, xs:QName) rather than xs:string.

In addition, in XSLT, all such functions now accept union(xs:string, xs:QName) rather than union(xs:QName, xs:string). This is primarily to make them all consistent.

@ChristianGruen ChristianGruen added the Propose Merge without Discussion Change is editorial or minor label Jan 4, 2024
@michaelhkay
michaelhkay marked this pull request as draft January 4, 2024 00:28
@michaelhkay

michaelhkay commented Jan 4, 2024

Copy link
Copy Markdown
Contributor Author

I added changes to the coercion rules that allow an untypedAtomic value to be supplied where the required type is (for example) union(string, QName). Without this, we would have an additional incompatibility from the format-number() change, because it would no longer be allowed to supply an untyped node as the third argument. (This is most likely to happen where the value is actually a text node resulting from evaluation of an xsl:variable declaration).

I also made the change suggested in issue #911 to change the table caption in the coercion rules from "type promotion" to "implicit casting".

@michaelhkay
michaelhkay marked this pull request as ready for review January 4, 2024 10:22
@ndw

ndw commented Jan 9, 2024

Copy link
Copy Markdown
Contributor

The CG agreed to merge this PR without discussion at meeting 060

@ndw
ndw merged commit f6d915d into qt4cg:master Jan 9, 2024
@michaelhkay michaelhkay added Tests Added Tests have been added to the test suites Completed PR has been applied, tests written and tagged, no further action needed and removed Propose Merge without Discussion Change is editorial or minor labels Jun 12, 2026
@michaelhkay

Copy link
Copy Markdown
Contributor Author

Note: the original change to allow an xs:QName argument to functions such function-available and element-available was made in the initial editor's draft accepted by the CG as its baseline. I have arbitrarily linked test cases etc to this PR as an identifier for the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Completed PR has been applied, tests written and tagged, no further action needed Tests Added Tests have been added to the test suites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

format-number() etc incompatibility

3 participants