Project

Profile

Help

Bug #2721

Grouping separators in format-number()

Added by Debbie Lockett almost 4 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Category:
XPath conformance
Sprint/Milestone:
Start date:
2016-04-21
Due date:
% Done:

100%

Legacy ID:
Applies to branch:
9.7, trunk
Fix Committed on Branch:
9.7, trunk
Fixed in Maintenance Release:

Description

Following the resolution of W3C bug #29534, clarifications have been made to the F&O 3.1 specification of format-number() regarding regular grouping.

This affects Saxon, e.g. the expected result of format-number(xs:decimal('123456789'), '###,##,00') is 12345,67,89; but Saxon 9.7 returns 1,23,45,67,89. (See test cbcl-fn-format-number-015, and also cbcl-fn-format-number-027).

[Note that these particular tests pass under Saxon 9.6; while a number of other recently added tests (i.e. since 2015-09) which use grouping separators fail under Saxon 9.6. The implementation was updated in 9.7 following other F&O 3.1 specification clarifications related to the use of grouping separators (e.g see W3C bug #28567).]

History

#1 Updated by Debbie Lockett almost 4 years ago

  • Status changed from In Progress to Resolved
  • Priority changed from Low to Normal
  • Applies to branch 9.8 added
  • Fix Committed on Branch 9.7, 9.8 added

Patch committed to FormatNumber on 9.7 and 9.8 dev branches. The leftmost group (defined by grouping separators in the picture string) is now also considered when determining whether grouping is regular.

#2 Updated by O'Neil Delpratt almost 4 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100
  • Fixed in Maintenance Release 9.7.0.5 added

Bug fix applied in the Saxon 9.7.0.5 maintenance release.

#3 Updated by O'Neil Delpratt almost 4 years ago

  • Sprint/Milestone set to 9.7.0.5

#4 Updated by O'Neil Delpratt almost 3 years ago

  • Applies to branch trunk added
  • Applies to branch deleted (9.8)

#5 Updated by O'Neil Delpratt almost 3 years ago

  • Fix Committed on Branch trunk added
  • Fix Committed on Branch deleted (9.8)

Please register to edit this issue

Also available in: Atom PDF