Project

Profile

Help

Bug #2720

closed

Grouping separators in format-integer()

Added by Debbie Lockett over 8 years ago. Updated over 7 years ago.

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

100%

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

Description

Following the resolution of W3C bug #29488, clarifications have been made to the F&O 3.1 specification of format-integer() regarding the use of grouping separators. Two main issues arise, illustrated by the following:

  1. The expected result for test format-integer-030 has changed:

format-integer(602347826, '#(000)000-000') now expects '602)347-826' rather than '(602)347-826' (which is currently returned by Saxon).

  1. The expected result of format-integer(123456789, '000,00,00') is '12345,67,89'; but Saxon returns '1,23,45,67,89'. (See test format-integer-072)
Actions #1

Updated by Debbie Lockett over 8 years ago

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

Patches committed on 9.7 and 9.8 dev branches:

  1. In IrregularGroupFormatter: only include grouping separators at positions less than the length of the value.

  2. In FormatInteger: the leftmost group (defined by grouping separators in the picture string) is now also considered when determining whether grouping is regular.

Actions #2

Updated by O'Neil Delpratt over 8 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.

Actions #3

Updated by O'Neil Delpratt over 8 years ago

  • Sprint/Milestone set to 9.7.0.5
Actions #4

Updated by O'Neil Delpratt over 7 years ago

  • Applies to branch trunk added
  • Applies to branch deleted (9.6, 9.8)
Actions #5

Updated by O'Neil Delpratt over 7 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