nodes of type xsd:decimal in ]-1;1[ interval are invalid

RESOLVED FIXED

Status

Core Graveyard
XForms
RESOLVED FIXED
12 years ago
11 months ago

People

(Reporter: Allan Beaufour, Assigned: Doron Rosenberg (IBM))

Tracking

({fixed1.8, testcase})

Trunk
fixed1.8, testcase

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
 
(Reporter)

Comment 1

12 years ago
Created attachment 192936 [details]
Testcase
(Assignee)

Comment 2

12 years ago
known issue, its fixed in my tree already :)
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Reporter)

Comment 3

12 years ago
(In reply to comment #2)
> known issue, its fixed in my tree already :)

Ah, then it's not a bug I guess. My bad for not checking your tree first ;-)
(Assignee)

Comment 4

12 years ago
Created attachment 199208 [details] [diff] [review]
patch

Since this is annoying, putting a patch here.

basically, someone contributed a patch that added leading 0 stripping, but the
code doesn't take 000.23 into consideration, and returned .23, which is
invalid. 

Therefore, if the decimal part is all zeros, we set it to "0".
(Assignee)

Updated

12 years ago
Attachment #199208 - Flags: review?(peterv)
Attachment #199208 - Flags: superreview+
Attachment #199208 - Flags: review?(peterv)
Attachment #199208 - Flags: review+
(Assignee)

Comment 5

12 years ago
checked into trunk
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 6

12 years ago
Comment on attachment 199208 [details] [diff] [review]
patch

a=mkaply
Attachment #199208 - Flags: approval1.8rc1+
(Assignee)

Comment 7

12 years ago
checked into branch
Keywords: fixed1.8
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.