Project

Profile

Help

Actions
Send by e-mail Copy link

Bug #4603

closed

format-number() - grouping separator before any digit character

Added by Michael Kay almost 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Michael Kay
Category:
XPath Conformance
Sprint/Milestone:
-
Start date:
2020-06-18
Due date:
% Done:

100%

Estimated time:
Applies to JS Branch:
2
Fix Committed on JS Branch:
2
Fixed in JS Release:
Saxon-JS 2.1
SEF Generated with:
Platforms:
Company:
-
Contact person:
-
Additional contact persons:
-

Description

In the profile.xsl stylesheet used for formatting a performance profile, we use format-number with the picture

format-number(@count, ',##0')

Saxon/J accepts this, Saxon-JS rejects it with the error "Invalid picture: grouping separator before any digit".

I suspect Saxon-JS is right and Saxon/J is wrong to accept it.

Actions
Copy link
 #1

Updated by Michael Kay almost 4 years ago

  • Project changed from Saxon to SaxonJS
  • Applies to JS Branch 2.0 added

Checking the spec, I can't see any rule that says this is not allowed. Therefore transferring to Saxon-JS project.

Actions
Copy link
 #2

Updated by Debbie Lockett almost 4 years ago

  • Category set to XPath Conformance
Actions
Copy link
 #3

Updated by Michael Kay almost 4 years ago

  • Status changed from New to Resolved
  • Priority changed from Low to Normal

Added QT3 test case numberformat320.

Patch committed to Git repository (remove check at Numberer.js line 859)

Actions
Copy link
 #4

Updated by Community Admin about 3 years ago

  • Applies to JS Branch 2 added
  • Applies to JS Branch deleted (2.0)
Actions
Copy link
 #5

Updated by Debbie Lockett about 3 years ago

  • Fix Committed on JS Branch 2 added
Actions
Copy link
 #6

Updated by Debbie Lockett about 3 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100
  • Fixed in JS Release set to Saxon-JS 2.1

Bug fix applied in the Saxon-JS 2.1 maintenance release.

Please register to edit this issue

Actions
Send by e-mail Copy link

Also available in: Atom PDF Tracking page