434482 - Parse time zoon offset as decimal

Status: RESOLVED FIXED

(Mozilla Labs :: Operator, defect)

Description

[Takeshi Kurosawa :takenspc]

Attachment: Patch rv. 1.0

Some JavaScript file of "SemanticActions" parse time zone offset (and some other numbers) with |parseInt(foo)| like below:

> parseInt(hcalendar.dtend.substr(tzpos+1, 2), 10);

When the time zone offset hour is "08" or "09", it was parsed as 0 instead of 8 or 9.
# It was treated as an octal numeral.

We should use |parseInt(foo, 10)| instead of |parseInt(foo)|.

Comment 1

[Mike Kaply [:mkaply]]

Comment on attachment 321581 [details] [diff] [review]
Patch rv. 1.0

Thank you for this patch. It will be in the next Operator

Comment 2

[Mike Kaply [:mkaply]]

Fix checked in.